* [PATCH] sched/cpufreq: Remove unused macro SUGOV_KTHREAD_PRIORITY
@ 2018-02-08 13:48 Leo Yan
2018-02-08 14:04 ` Viresh Kumar
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Leo Yan @ 2018-02-08 13:48 UTC (permalink / raw)
To: linux-kernel, Ingo Molnar, Peter Zijlstra, Rafael J . Wysocki
Cc: Leo Yan, Vikram Mulukutla, Vincent Guittot, Daniel Lezcano
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(-)
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;
--
2.7.4
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH] sched/cpufreq: Remove unused macro SUGOV_KTHREAD_PRIORITY
2018-02-08 13:48 [PATCH] sched/cpufreq: Remove unused macro SUGOV_KTHREAD_PRIORITY Leo Yan
@ 2018-02-08 14:04 ` Viresh Kumar
2018-02-08 15:23 ` Leo Yan
2018-02-08 14:06 ` Daniel Lezcano
2018-02-13 12:15 ` [tip:sched/urgent] sched/cpufreq: Remove unused SUGOV_KTHREAD_PRIORITY macro tip-bot for Leo Yan
2 siblings, 1 reply; 5+ messages in thread
From: Viresh Kumar @ 2018-02-08 14:04 UTC (permalink / raw)
To: Leo Yan
Cc: linux-kernel, Ingo Molnar, Peter Zijlstra, Rafael J . Wysocki,
Vikram Mulukutla, Vincent Guittot, Daniel Lezcano
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>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] sched/cpufreq: Remove unused macro SUGOV_KTHREAD_PRIORITY
2018-02-08 14:04 ` Viresh Kumar
@ 2018-02-08 15:23 ` Leo Yan
0 siblings, 0 replies; 5+ messages in thread
From: Leo Yan @ 2018-02-08 15:23 UTC (permalink / raw)
To: Viresh Kumar
Cc: linux-kernel, Ingo Molnar, Peter Zijlstra, Rafael J . Wysocki,
Vikram Mulukutla, Vincent Guittot, Daniel Lezcano
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.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] sched/cpufreq: Remove unused macro SUGOV_KTHREAD_PRIORITY
2018-02-08 13:48 [PATCH] sched/cpufreq: Remove unused macro SUGOV_KTHREAD_PRIORITY Leo Yan
2018-02-08 14:04 ` Viresh Kumar
@ 2018-02-08 14:06 ` Daniel Lezcano
2018-02-13 12:15 ` [tip:sched/urgent] sched/cpufreq: Remove unused SUGOV_KTHREAD_PRIORITY macro tip-bot for Leo Yan
2 siblings, 0 replies; 5+ messages in thread
From: Daniel Lezcano @ 2018-02-08 14:06 UTC (permalink / raw)
To: Leo Yan, linux-kernel, Ingo Molnar, Peter Zijlstra, Rafael J . Wysocki
Cc: Vikram Mulukutla, Vincent Guittot
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* [tip:sched/urgent] sched/cpufreq: Remove unused SUGOV_KTHREAD_PRIORITY macro
2018-02-08 13:48 [PATCH] sched/cpufreq: Remove unused macro SUGOV_KTHREAD_PRIORITY Leo Yan
2018-02-08 14:04 ` Viresh Kumar
2018-02-08 14:06 ` Daniel Lezcano
@ 2018-02-13 12:15 ` tip-bot for Leo Yan
2 siblings, 0 replies; 5+ messages in thread
From: tip-bot for Leo Yan @ 2018-02-13 12:15 UTC (permalink / raw)
To: linux-tip-commits
Cc: viresh.kumar, tglx, torvalds, daniel.lezcano, vincent.guittot,
markivx, linux-kernel, peterz, hpa, leo.yan, rafael.j.wysocki,
mingo
Commit-ID: 43d1b29b27c76e7454cd6c85bec4d0e9cbb039f3
Gitweb: https://git.kernel.org/tip/43d1b29b27c76e7454cd6c85bec4d0e9cbb039f3
Author: Leo Yan <leo.yan@linaro.org>
AuthorDate: Thu, 8 Feb 2018 21:48:22 +0800
Committer: Ingo Molnar <mingo@kernel.org>
CommitDate: Tue, 13 Feb 2018 13:04:03 +0100
sched/cpufreq: Remove unused SUGOV_KTHREAD_PRIORITY macro
Since schedutil kernel thread directly set priority to 0, the macro
SUGOV_KTHREAD_PRIORITY is not used. So remove it.
Signed-off-by: Leo Yan <leo.yan@linaro.org>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Rafael J . Wysocki <rafael.j.wysocki@intel.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Vikram Mulukutla <markivx@codeaurora.org>
Cc: Vincent Guittot <vincent.guittot@linaro.org>
Link: http://lkml.kernel.org/r/1518097702-9665-1-git-send-email-leo.yan@linaro.org
Signed-off-by: Ingo Molnar <mingo@kernel.org>
---
kernel/sched/cpufreq_schedutil.c | 2 --
1 file changed, 2 deletions(-)
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;
^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-02-13 12:16 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-08 13:48 [PATCH] sched/cpufreq: Remove unused macro SUGOV_KTHREAD_PRIORITY Leo Yan
2018-02-08 14:04 ` Viresh Kumar
2018-02-08 15:23 ` Leo Yan
2018-02-08 14:06 ` Daniel Lezcano
2018-02-13 12:15 ` [tip:sched/urgent] sched/cpufreq: Remove unused SUGOV_KTHREAD_PRIORITY macro tip-bot for Leo Yan
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.