lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 29 Aug 2014 11:08:48 +0800 From: Jason Wang <jasowang@...hat.com> To: Ingo Molnar <mingo@...nel.org> CC: Mike Galbraith <umgwanakikbuti@...il.com>, davem@...emloft.net, netdev@...r.kernel.org, linux-kernel@...r.kernel.org, mst@...hat.com, Peter Zijlstra <peterz@...radead.org>, Ingo Molnar <mingo@...e.hu> Subject: Re: [PATCH net-next 2/2] net: exit busy loop when another process is runnable On 08/22/2014 03:42 PM, Ingo Molnar wrote: > * Jason Wang <jasowang@...hat.com> wrote: > >> Polling could be done by either rx busy loop in process >> context or NAPI in softirq. [...] > Note that this shows another reason why it's a bad idea to > query nr_running directly: depending on the softirq processing > method, a softirq might run: > > - directly in process context > - in an idle thread's context > - or in a ksoftirqd context. > > 'nr_running' will have different values in these cases, causing > assymetries in busy-poll handling! Current rx busy polling code only works in process context, we can add BUG or warnings to make sure the helper was only called in process context. > > Another class of assymetry is when there are other softirq bits > pending, beyond NET_RX (or NET_TX): a nr_running check misses > them. Yes, but rx busy polling only works in process context and does not disable bh, so it may be not an issue. > The solution I outlined in the previous mail (using a > sched_expected_runtime() method) would be able to avoid most of > these artifacts. It only take cares the remaining runnable time for current process, this is a good hint for sk_busy_loop_end_time() which return the maximum time a process could do busy polling. But this is not this patch needs. We need check the state of other runnable process in current cpu to avoid damaging the performance of other process. > > Thanks, > > Ingo > -- > To unsubscribe from this list: send the line "unsubscribe netdev" in > the body of a message to majordomo@...r.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists