linux-rt-users.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@kernel.org>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>,
	Clark Williams <williams@redhat.com>,
	linux-kernel@vger.kernel.org, torvalds@linux-foundation.org,
	a.p.zijlstra@chello.nl, akpm@linux-foundation.org,
	linux-rt-users@vger.kernel.org
Subject: Re: [GIT RFC PULL rcu/urgent] Prevent Kconfig from asking pointless questions
Date: Mon, 20 Apr 2015 20:09:04 +0200	[thread overview]
Message-ID: <20150420180904.GA19229@gmail.com> (raw)
In-Reply-To: <20150420180107.GE24936@home.goodmis.org>


* Steven Rostedt <rostedt@goodmis.org> wrote:

> On Mon, Apr 20, 2015 at 10:09:03AM -0700, Paul E. McKenney wrote:
> > 
> > The sysfs knob might be nice, but as far as I know nobody has been
> > complaining about it.
> > 
> > Besides, we already have the rcutree.kthread_prio= kernel-boot 
> > parameter. So how about if the Kconfig parameter selects either 
> > SCHED_OTHER (the default) or SCHED_FIFO:1, and then the boot 
> > parameter can be used to select other values.
> > 
> > That said, if the lack of a sysfs knob has been causing real 
> > problems, let's make that happen.
> 
> But then it's too late, because the time of something getting into 
> the kernel to the time people can use it can be months if not years.
> 
> I see no harm in adding one. Pretty much every kernel parameter I 
> added for ftrace, has a sysctrl knob for it. (Not a sysfs knob, but 
> a /proc/sys/kernel knob which is different).

So the disadvantage is that if a boot default is wrong, we'll hear 
about it eventually and can fix/improve it.

If a sysctl knob is wrong, people will just 'tune' it and forget to 
propagate it to the kernel proper (why should they).

Which is fine for something like ftrace and other ad-hoc 
instrumentation that is generally very fine tuned to a given bug or 
given piece of hardware, but for something like the RCU implementation 
of the kernel - even if it's just a RT side thought of it - I'm not so 
sure about it.

Thanks,

	Ingo

  reply	other threads:[~2015-04-20 18:09 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20150416183812.GA5571@linux.vnet.ibm.com>
     [not found] ` <20150418130340.GA26931@gmail.com>
     [not found]   ` <20150418133444.GD23685@linux.vnet.ibm.com>
     [not found]     ` <20150418143238.GA2337@gmail.com>
2015-04-19  2:05       ` [GIT RFC PULL rcu/urgent] Prevent Kconfig from asking pointless questions Paul E. McKenney
2015-04-20 16:35         ` Clark Williams
2015-04-20 17:09           ` Paul E. McKenney
2015-04-20 17:59             ` Clark Williams
2015-04-20 18:20               ` Daniel Bristot de Oliveira
2015-04-20 18:01             ` Steven Rostedt
2015-04-20 18:09               ` Ingo Molnar [this message]
2015-04-20 18:21                 ` Steven Rostedt
2015-04-20 18:28                   ` Ingo Molnar
2015-04-20 18:34                     ` Steven Rostedt
2015-04-21  6:42                       ` Ingo Molnar
2015-04-21 13:18                         ` Steven Rostedt
2015-04-21  3:37                   ` Mike Galbraith
2015-04-20 20:40             ` Steven Rostedt
2015-04-20 21:15               ` Paul E. McKenney
2015-04-20 21:50                 ` Clark Williams
2015-04-21  1:22                   ` Paul E. McKenney
2015-04-21 13:12                     ` Steven Rostedt
2015-04-21 15:01                       ` Paul E. McKenney
2015-04-21 15:50                         ` Steven Rostedt
2015-04-21 15:54                           ` Paul E. McKenney

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=20150420180904.GA19229@gmail.com \
    --to=mingo@kernel.org \
    --cc=a.p.zijlstra@chello.nl \
    --cc=akpm@linux-foundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-rt-users@vger.kernel.org \
    --cc=paulmck@linux.vnet.ibm.com \
    --cc=rostedt@goodmis.org \
    --cc=torvalds@linux-foundation.org \
    --cc=williams@redhat.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;
as well as URLs for NNTP newsgroup(s).