mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Frederic Weisbecker <fweisbec@gmail.com>
To: Ingo Molnar <mingo@kernel.org>
Cc: LKML <linux-kernel@vger.kernel.org>,
	Frederic Weisbecker <fweisbec@gmail.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	Chris Metcalf <cmetcalf@tilera.com>,
	Christoph Lameter <cl@linux.com>,
	Geoff Levand <geoff@infradead.org>,
	Gilad Ben Yossef <gilad@benyossef.com>,
	Hakan Akkan <hakanakkan@gmail.com>,
	Kevin Hilman <khilman@linaro.org>,
	Li Zhong <zhong@linux.vnet.ibm.com>,
	Namhyung Kim <namhyung.kim@lge.com>,
	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>,
	Paul Gortmaker <paul.gortmaker@windriver.com>,
	Peter Zijlstra <peterz@infradead.org>,
	Steven Rostedt <rostedt@goodmis.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	Michal Marek <mmarek@suse.cz>
Subject: [RFC GIT PULL] nohz: Kconfig layout improvements
Date: Wed,  3 Apr 2013 14:46:26 +0200	[thread overview]
Message-ID: <1364993190-13784-1-git-send-email-fweisbec@gmail.com> (raw)

Ingo,

This set addresses your review concerning the Kconfig layout.
Please note two things here that derive from what we agreed
on due to technical limitations:

* Now the full dynticks Kconfig is not hidden anymore behind its
high level dependencies. (ie: passive dependencies are now active).
There is an exception though with CONFIG_VIRT_CPU_ACCOUNTING_GEN
(Full dynticks cputime accounting) that is part of a choice menu
like PREEMPT_*. It seems such kconfig layout prevent from doing a remote
select on its choices. So it stays a passive dependency for now, until
Kconfig/Kbuild supports that (Cc'ing Michel Marek) or somebody shows
me what I did wrong ;)

* Ideally we want to reuse CONFIG_NO_HZ as a Kconfig that consolidate
the common code between CONFIG_NO_HZ_IDLE and CONFIG_NO_HZ_EXTENDED.
But we also want CONFIG_NO_HZ from old config files to map to CONFIG_NO_HZ_IDLE.
Both at the same time is not possible or we have a Kconfig circular
dependency. So I introduced a new CONFIG_NO_HZ_COMMON for common nohz code
and CONFIG_NO_HZ stays for backward compatibility by enabling CONFIG_NO_HZ_IDLE
by default.

If you're ok, please pull from:

git://git.kernel.org/pub/scm/linux/kernel/git/frederic/linux-dynticks.git
	timers/nohz-v2

Thanks.

---
Frederic Weisbecker (4):
  nohz: Unhide full dynticks feature from its dependencies
  nohz: Rename CONFIG_NO_HZ to CONFIG_NO_HZ_COMMON
  nohz: Pack nohz Kconfig option in a menu of choices
  nohz: Print final full dynticks CPUs range on boot

 Documentation/RCU/stallwarn.txt         |    2 +-
 Documentation/cpu-freq/governors.txt    |    4 +-
 arch/um/include/shared/common-offsets.h |    4 +-
 arch/um/os-Linux/time.c                 |    2 +-
 include/linux/sched.h                   |    8 ++--
 include/linux/tick.h                    |    8 ++--
 init/Kconfig                            |    2 +-
 kernel/hrtimer.c                        |    4 +-
 kernel/sched/core.c                     |   18 +++++-----
 kernel/sched/fair.c                     |   10 +++---
 kernel/sched/sched.h                    |    4 +-
 kernel/softirq.c                        |    2 +-
 kernel/time/Kconfig                     |   54 ++++++++++++++++++++++++------
 kernel/time/tick-sched.c                |   22 +++++++++---
 kernel/timer.c                          |    4 +-
 15 files changed, 95 insertions(+), 53 deletions(-)

-- 
1.7.5.4


             reply	other threads:[~2013-04-03 12:46 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-03 12:46 Frederic Weisbecker [this message]
2013-04-03 12:46 ` [PATCH 1/4] nohz: Unhide full dynticks feature from its dependencies Frederic Weisbecker
2013-04-03 12:46 ` [PATCH 2/4] nohz: Rename CONFIG_NO_HZ to CONFIG_NO_HZ_COMMON Frederic Weisbecker
2013-04-03 12:46 ` [PATCH 3/4] nohz: Pack nohz Kconfig option in a menu of choices Frederic Weisbecker
2013-04-03 12:46 ` [PATCH 4/4] nohz: Print final full dynticks CPUs range on boot Frederic Weisbecker
2013-04-04 18:10 ` [RFC GIT PULL] nohz: Kconfig layout improvements Christoph Lameter
     [not found]   ` <CAOtvUMcs5F89biz_xtVSBAPVftfJk+0VDvxLfm7-kQ3q6x0Ynw@mail.gmail.com>
     [not found]     ` <0000013dd64a36f3-60dd0774-a44b-4780-93b7-af6b8baac87f-000000@email.amazonses.com>
2013-04-04 19:48       ` Gilad Ben-Yossef
2013-04-04 20:03         ` Christoph Lameter
2013-04-10 13:47   ` Frederic Weisbecker
2013-04-10 14:08     ` Christoph Lameter
2013-04-08 11:19 ` Ingo Molnar
2013-04-10 16:01   ` Frederic Weisbecker
2013-04-10 17:24     ` Ingo Molnar
2013-04-12 14:23       ` Frederic Weisbecker

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=1364993190-13784-1-git-send-email-fweisbec@gmail.com \
    --to=fweisbec@gmail.com \
    --cc=akpm@linux-foundation.org \
    --cc=cl@linux.com \
    --cc=cmetcalf@tilera.com \
    --cc=geoff@infradead.org \
    --cc=gilad@benyossef.com \
    --cc=hakanakkan@gmail.com \
    --cc=khilman@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@kernel.org \
    --cc=mmarek@suse.cz \
    --cc=namhyung.kim@lge.com \
    --cc=paul.gortmaker@windriver.com \
    --cc=paulmck@linux.vnet.ibm.com \
    --cc=peterz@infradead.org \
    --cc=rostedt@goodmis.org \
    --cc=tglx@linutronix.de \
    --cc=zhong@linux.vnet.ibm.com \
    /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

all inboxes | Powered by JetHome®