All of lore.kernel.org
 help / color / mirror / Atom feed
* [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 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
* 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
* [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.