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

[Peter Zijlstra]

On Sat, Nov 12, 2016 at 10:52:35AM +0530, Viresh Kumar wrote:
> On 11 November 2016 at 20:09, Peter Zijlstra <peterz@xxxxxxxxxxxxx> 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.