From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
To: Andi Kleen <andi@firstfloor.org>
Cc: linux-kernel@vger.kernel.org, mingo@elte.hu,
laijs@cn.fujitsu.com, dipankar@in.ibm.com,
akpm@linux-foundation.org, mathieu.desnoyers@efficios.com,
josh@joshtriplett.org, niv@us.ibm.com, tglx@linutronix.de,
peterz@infradead.org, rostedt@goodmis.org, dhowells@redhat.com,
edumazet@google.com, darren@dvhart.com, fweisbec@gmail.com,
sbw@mit.edu
Subject: Re: [PATCH RFC nohz_full 0/7] v2 Provide infrastructure for full-system idle
Date: Mon, 1 Jul 2013 12:19:49 -0700 [thread overview]
Message-ID: <20130701191949.GS3773@linux.vnet.ibm.com> (raw)
In-Reply-To: <20130701161910.GN6123@two.firstfloor.org>
On Mon, Jul 01, 2013 at 06:19:10PM +0200, Andi Kleen wrote:
> > I am guessing that you want CONFIG_NO_HZ_FULL to implicitly enable
> > the sysidle code so that CONFIG_NO_HZ_FULL_SYSIDLE can be eliminated.
> > I will be happy to take that step, but only after I gain full confidence
> > in the correctness and performance of the sysidle code.
>
> FWIW if you want useful testing you need to enable it by default
> (as part of NO_IDLE_HZ) anyways. Users will most likely pick
> whatever is "default" in Kconfig.
At this point in the process, I want testers who choose to test. Hapless
victim testers come later. Well, other than randconfig testers, but I
consider them to be voluntary hapless victims. ;-)
> > > If you want a switch for testing I would advise a sysctl or sysfs knob
> >
> > This would work well for the correctness part, but not for the performance
> > part.
>
> What performance part?
>
> Are you saying this adds so many checks to hot paths that normal runtime
> if() with a flag is too expensive?
I am saying that I don't know, and that I want to make it easy for people
to find out by comparing to the base configuration -- and for me to be
able to detect this from their .config file.
Thanx, Paul
next prev parent reply other threads:[~2013-07-01 19:19 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-28 20:09 [PATCH RFC nohz_full 0/7] v2 Provide infrastructure for full-system idle Paul E. McKenney
2013-06-28 20:10 ` [PATCH RFC nohz_full v2 1/7] nohz_full: Add Kconfig parameter for scalable detection of all-idle state Paul E. McKenney
2013-06-28 20:10 ` [PATCH RFC nohz_full v2 2/7] nohz_full: Add rcu_dyntick data " Paul E. McKenney
2013-07-01 15:31 ` Josh Triplett
2013-07-01 15:52 ` Paul E. McKenney
2013-07-01 18:16 ` Josh Triplett
2013-07-01 18:23 ` Paul E. McKenney
2013-07-01 18:34 ` Josh Triplett
2013-07-01 19:16 ` Paul E. McKenney
2013-07-02 5:10 ` Mike Galbraith
2013-07-02 5:58 ` Paul E. McKenney
2013-06-28 20:10 ` [PATCH RFC nohz_full v2 3/7] nohz_full: Add per-CPU idle-state tracking Paul E. McKenney
2013-07-01 15:33 ` Josh Triplett
2013-06-28 20:10 ` [PATCH RFC nohz_full v2 4/7] nohz_full: Add full-system idle states and variables Paul E. McKenney
2013-06-28 20:10 ` [PATCH RFC nohz_full v2 5/7] nohz_full: Add full-system-idle arguments to API Paul E. McKenney
2013-06-28 20:10 ` [PATCH RFC nohz_full v2 6/7] nohz_full: Add full-system-idle state machine Paul E. McKenney
2013-07-01 16:35 ` Frederic Weisbecker
2013-07-01 18:10 ` Paul E. McKenney
2013-07-01 20:55 ` Frederic Weisbecker
2013-07-01 16:53 ` Frederic Weisbecker
2013-07-01 18:17 ` Paul E. McKenney
2013-07-01 21:38 ` Frederic Weisbecker
2013-07-01 22:51 ` Paul E. McKenney
2013-06-28 20:10 ` [PATCH RFC nohz_full v2 7/7] nohz_full: Force RCU's grace-period kthreads onto timekeeping CPU Paul E. McKenney
2013-07-01 15:19 ` [PATCH RFC nohz_full 0/7] v2 Provide infrastructure for full-system idle Andi Kleen
2013-07-01 16:03 ` Paul E. McKenney
2013-07-01 16:19 ` Andi Kleen
2013-07-01 19:19 ` Paul E. McKenney [this message]
2013-07-01 19:43 ` Christoph Lameter
2013-07-01 19:56 ` Paul E. McKenney
2013-07-01 20:24 ` Christoph Lameter
2013-07-01 20:43 ` Thomas Gleixner
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=20130701191949.GS3773@linux.vnet.ibm.com \
--to=paulmck@linux.vnet.ibm.com \
--cc=akpm@linux-foundation.org \
--cc=andi@firstfloor.org \
--cc=darren@dvhart.com \
--cc=dhowells@redhat.com \
--cc=dipankar@in.ibm.com \
--cc=edumazet@google.com \
--cc=fweisbec@gmail.com \
--cc=josh@joshtriplett.org \
--cc=laijs@cn.fujitsu.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mathieu.desnoyers@efficios.com \
--cc=mingo@elte.hu \
--cc=niv@us.ibm.com \
--cc=peterz@infradead.org \
--cc=rostedt@goodmis.org \
--cc=sbw@mit.edu \
--cc=tglx@linutronix.de \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.