Re: [PATCH 2/3] cpufreq: schedutil: move slow path from workqueue to SCHED_FIFO task

From	Peter Zijlstra <peterz@infradead.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH 2/3] cpufreq: schedutil: move slow path from workqueue to SCHED_FIFO task
Date	2016-11-14 10:30 +0100
Message-ID	<sDlKp-55b-1@gated-at.bofh.it> (permalink)
References	<sChfQ-35W-17@gated-at.bofh.it> <sChfQ-35W-19@gated-at.bofh.it> <sCl9M-5wV-13@gated-at.bofh.it> <sCl9M-5wV-11@gated-at.bofh.it> <sCz33-6bf-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Sat, Nov 12, 2016 at 10:52:35AM +0530, Viresh Kumar wrote:
> On 11 November 2016 at 20:09, Peter Zijlstra <peterz@infradead.org> wrote:
> > On Fri, Nov 11, 2016 at 03:32:04PM +0100, Tommaso Cucinotta wrote:
> >> >+    struct sched_param param = { .sched_priority = 50 };
> >>
> >> won't you have a tunable here? (sysctl?)
> >
> > You can use the regular userspace tools, like schedtool and chrt to set
> > priorities.
> 
> I wanted to get some help from you on this Peter. The out of tree Interactive
> governor has always used MAX_RT_PRIORITY - 1 here instead of 50.
> 
> But Steve started with 50. What do you think should the value be ?

Any static prio value is wrong (static prio assignment requires system
knowledge that the kernel doesn't and cannot have), 50 is what threaded
IRQs default too as well IIRC, so it would at least be consistent with
that.

Thread

Re: [PATCH 2/3] cpufreq: schedutil: move slow path from workqueue to  SCHED_FIFO task Viresh Kumar <viresh.kumar@linaro.org> - 2016-11-12 06:30 +0100
  Re: [PATCH 2/3] cpufreq: schedutil: move slow path from workqueue to  SCHED_FIFO task Peter Zijlstra <peterz@infradead.org> - 2016-11-14 10:30 +0100
    Re: [PATCH 2/3] cpufreq: schedutil: move slow path from workqueue to  SCHED_FIFO task Viresh Kumar <viresh.kumar@linaro.org> - 2016-11-14 11:30 +0100

csiph-web