All of lore.kernel.org
 help / color / mirror / Atom feed
From: Clark Williams <williams@redhat.com>
To: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
Cc: Steven Rostedt <rostedt@goodmis.org>,
	Ingo Molnar <mingo@kernel.org>,
	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 16:50:07 -0500	[thread overview]
Message-ID: <20150420165007.295f3fe0@sluggy> (raw)
In-Reply-To: <20150420211504.GW5561@linux.vnet.ibm.com>

[-- Attachment #1: Type: text/plain, Size: 1510 bytes --]

On Mon, 20 Apr 2015 14:15:04 -0700
"Paul E. McKenney" <paulmck@linux.vnet.ibm.com> wrote:

> On Mon, Apr 20, 2015 at 04:40:49PM -0400, Steven Rostedt 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.
> > 
> > Hmm, what priority is this for anyway. To change the priority of the boost
> > value at run time, do we only need to change the priority of the rcub threads?
> > 
> > And the priority of the other rcu threads can change as well with a simple
> > chrt?
> > 
> > If that's the case, then we don't need a sysctl knob at all.
> 
> For the grace-period kthreads and the boost kthread, that is the case.
> It is also the case for the per-CPU kthreads that invoke RCU callbacks
> for the non-offloaded RCU_BOOST configuration (and that replace all
> softirq RCU work in -rt).
> 
> So, should I just ditch all of the priority-setting within RCU and tell
> users to just use chrt?

Looks to me like all we need to do is tell people if they need a boost
higher than the compiled in default (RCU_KTHREAD_PRIO), then chrt the
priority of the rcub thread to the desired priority. 

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

  reply	other threads:[~2015-04-20 21:50 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-16 18:38 [GIT RFC PULL rcu/urgent] Prevent Kconfig from asking pointless questions Paul E. McKenney
2015-04-18 13:03 ` Ingo Molnar
2015-04-18 13:34   ` Paul E. McKenney
2015-04-18 14:32     ` Ingo Molnar
2015-04-19  2:05       ` 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
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 [this message]
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=20150420165007.295f3fe0@sluggy \
    --to=williams@redhat.com \
    --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=mingo@kernel.org \
    --cc=paulmck@linux.vnet.ibm.com \
    --cc=rostedt@goodmis.org \
    --cc=torvalds@linux-foundation.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 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.