| Message ID | 1518097702-9665-1-git-send-email-leo.yan@linaro.org |
|---|---|
| State | Accepted |
| Commit | 43d1b29b27c76e7454cd6c85bec4d0e9cbb039f3 |
| Headers | show |
| Series | sched/cpufreq: Remove unused macro SUGOV_KTHREAD_PRIORITY | expand |
Forgot Cc'ing me ? :) On Thu, Feb 8, 2018 at 7:18 PM, Leo Yan <leo.yan@linaro.org> wrote: > Since schedutil kernel thread directly set priority to 0, the macro > SUGOV_KTHREAD_PRIORITY is not used. So remove it. > > Cc: Vikram Mulukutla <markivx@codeaurora.org> > Cc: Vincent Guittot <vincent.guittot@linaro.org> > Cc: Daniel Lezcano <daniel.lezcano@linaro.org> > Signed-off-by: Leo Yan <leo.yan@linaro.org> > --- > kernel/sched/cpufreq_schedutil.c | 2 -- > 1 file changed, 2 deletions(-) Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
On 08/02/2018 14:48, Leo Yan wrote: > Since schedutil kernel thread directly set priority to 0, the macro > SUGOV_KTHREAD_PRIORITY is not used. So remove it. > > Cc: Vikram Mulukutla <markivx@codeaurora.org> > Cc: Vincent Guittot <vincent.guittot@linaro.org> > Cc: Daniel Lezcano <daniel.lezcano@linaro.org> > Signed-off-by: Leo Yan <leo.yan@linaro.org> Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org> -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
On Thu, Feb 08, 2018 at 07:34:54PM +0530, Viresh Kumar wrote: > Forgot Cc'ing me ? :) I meant to copy you, but I wrongly added Vikram :) > On Thu, Feb 8, 2018 at 7:18 PM, Leo Yan <leo.yan@linaro.org> wrote: > > Since schedutil kernel thread directly set priority to 0, the macro > > SUGOV_KTHREAD_PRIORITY is not used. So remove it. > > > > Cc: Vikram Mulukutla <markivx@codeaurora.org> > > Cc: Vincent Guittot <vincent.guittot@linaro.org> > > Cc: Daniel Lezcano <daniel.lezcano@linaro.org> > > Signed-off-by: Leo Yan <leo.yan@linaro.org> > > --- > > kernel/sched/cpufreq_schedutil.c | 2 -- > > 1 file changed, 2 deletions(-) > > Acked-by: Viresh Kumar <viresh.kumar@linaro.org> Thanks for acking, Viresh and Daniel.
diff --git a/kernel/sched/cpufreq_schedutil.c b/kernel/sched/cpufreq_schedutil.c index dd062a1..7936f54 100644 --- a/kernel/sched/cpufreq_schedutil.c +++ b/kernel/sched/cpufreq_schedutil.c @@ -19,8 +19,6 @@ #include "sched.h" -#define SUGOV_KTHREAD_PRIORITY 50 - struct sugov_tunables { struct gov_attr_set attr_set; unsigned int rate_limit_us;
Since schedutil kernel thread directly set priority to 0, the macro SUGOV_KTHREAD_PRIORITY is not used. So remove it. Cc: Vikram Mulukutla <markivx@codeaurora.org> Cc: Vincent Guittot <vincent.guittot@linaro.org> Cc: Daniel Lezcano <daniel.lezcano@linaro.org> Signed-off-by: Leo Yan <leo.yan@linaro.org> --- kernel/sched/cpufreq_schedutil.c | 2 -- 1 file changed, 2 deletions(-) -- 2.7.4