From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
To: Manfred Spraul <manfred@colorfullife.com>
Cc: mingo@elte.hu, linux-kernel@vger.kernel.org,
akpm@linux-foundation.org, oleg@tv-sign.ru, dipankar@in.ibm.com,
rostedt@goodmis.org, dvhltc@us.ibm.com, niv@us.ibm.com
Subject: Re: nohz_cpu_mask question (was: Re: [PATCH tip/core/rcu] classic RCU locking and memory-barrier cleanups)
Date: Tue, 19 Aug 2008 10:41:18 -0700 [thread overview]
Message-ID: <20080819174118.GD6724@linux.vnet.ibm.com> (raw)
In-Reply-To: <48AAFFF0.7020405@colorfullife.com>
On Tue, Aug 19, 2008 at 07:16:32PM +0200, Manfred Spraul wrote:
> Hi all,
>
> can someone explain me nohz_cpu_mask?
>
> If I understand it correctly, it merely means that this cpu does not take
> timer interrupts, correct?
> The cpu can still take "real" interupts, e.g. a nic interrupt. The cpu
> could also do softirq processing.
Yep, that is correct.
> Is that correct? Ingo?
>
> If nohz cpus can take normal interrupts, then it would be wrong to exclude
> these cpus from the mask of cpus that must pass a quiescent cycle - a
> softirq could hold on a pointers.
That is indeed the approach that preemptable RCU takes, and
I am considering making classic RCU also take this approach.
The main advantage is that sleeping CPUs need not be awakened
unless it is participating directly in the RCU grace period (as in
it has some callbacks). See rcu_irq_enter() and rcu_irq_exit() in
kernel/rcupreempt.c as well as rcu_enter_nohz() and rcu_exit_nohz()
in include/linux/rcupreempt.h. And http://lwn.net/Articles/279077/
for the proof of correctness. ;-)
Instead, classic RCU prevents any CPU from going into nohz mode in the
first place if RCU is waiting on it. I am not completely confident
in this algorithm, which is one reason that I am looking to switch to
the preemptable RCU approach. The other reason is power efficiency --
after all, why bother waking up a CPU just to have it tell RCU that it
is sitting idle?
Thanx, Paul
next prev parent reply other threads:[~2008-08-19 17:41 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-05 16:21 [PATCH tip/core/rcu] classic RCU locking and memory-barrier cleanups Paul E. McKenney
2008-08-05 16:48 ` Steven Rostedt
2008-08-05 17:40 ` Paul E. McKenney
2008-08-06 5:30 ` Manfred Spraul
2008-08-07 3:18 ` Paul E. McKenney
2008-08-18 9:13 ` Manfred Spraul
2008-08-18 14:04 ` Paul E. McKenney
2008-08-19 10:48 ` Manfred Spraul
2008-08-19 14:03 ` Paul E. McKenney
2008-08-19 17:16 ` nohz_cpu_mask question (was: Re: [PATCH tip/core/rcu] classic RCU locking and memory-barrier cleanups) Manfred Spraul
2008-08-19 17:41 ` Paul E. McKenney [this message]
2008-08-15 14:09 ` [PATCH tip/core/rcu] classic RCU locking and memory-barrier cleanups Ingo Molnar
2008-08-15 14:24 ` Ingo Molnar
2008-08-15 14:56 ` Ingo Molnar
2008-08-15 14:58 ` Paul E. McKenney
2008-08-17 14:37 ` [PATCH tip/core/rcu] classic RCU locking cleanup fix lockdep problem Paul E. McKenney
2008-08-17 15:38 ` Ingo Molnar
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20080819174118.GD6724@linux.vnet.ibm.com \
--to=paulmck@linux.vnet.ibm.com \
--cc=akpm@linux-foundation.org \
--cc=dipankar@in.ibm.com \
--cc=dvhltc@us.ibm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=manfred@colorfullife.com \
--cc=mingo@elte.hu \
--cc=niv@us.ibm.com \
--cc=oleg@tv-sign.ru \
--cc=rostedt@goodmis.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox