2017-12-07 13:25:24

by Cheng Jian

Subject: [PATCH] sched: remove unused parameter from wakeup_gran

The first parameter 'curr' in wakeup_gran is unnecessary now.

Signed-off-by: Cheng Jian <[email protected]>
---
kernel/sched/fair.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
index 4037e19..005e196 100644
--- a/kernel/sched/fair.c
+++ b/kernel/sched/fair.c
@@ -6405,8 +6405,7 @@ static void task_dead_fair(struct task_struct *p)
}
#endif /* CONFIG_SMP */

-static unsigned long
-wakeup_gran(struct sched_entity *curr, struct sched_entity *se)
+static unsigned long wakeup_gran(struct sched_entity *se)
{
unsigned long gran = sysctl_sched_wakeup_granularity;

@@ -6448,7 +6447,7 @@ static void task_dead_fair(struct task_struct *p)
if (vdiff <= 0)
return -1;

- gran = wakeup_gran(curr, se);
+ gran = wakeup_gran(se);
if (vdiff > gran)
return 1;

--
1.8.3.1

2017-12-08 08:44:27

by tip-bot for Vasyl Gomonovych

Subject: [tip:sched/core] sched/fair: Remove unused 'curr' parameter from wakeup_gran

Commit-ID: a555e9d86ee384d9d3cb3310a57aed33f7e053d4
Gitweb: https://git.kernel.org/tip/a555e9d86ee384d9d3cb3310a57aed33f7e053d4
Author: Cheng Jian <[email protected]>
AuthorDate: Thu, 7 Dec 2017 21:30:43 +0800
Committer: Ingo Molnar <[email protected]>
CommitDate: Fri, 8 Dec 2017 07:51:53 +0100

sched/fair: Remove unused 'curr' parameter from wakeup_gran

The first parameter of wakeup_gran(), 'curr', is unnecessary now.

Signed-off-by: Cheng Jian <[email protected]>
Cc: Linus Torvalds <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Cc: Thomas Gleixner <[email protected]>
Cc: [email protected]
Cc: [email protected]
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Ingo Molnar <[email protected]>
---
kernel/sched/fair.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
index 2fe3aa8..2915c0d 100644
--- a/kernel/sched/fair.c
+++ b/kernel/sched/fair.c
@@ -6449,8 +6449,7 @@ static void task_dead_fair(struct task_struct *p)
}
#endif /* CONFIG_SMP */

-static unsigned long
-wakeup_gran(struct sched_entity *curr, struct sched_entity *se)
+static unsigned long wakeup_gran(struct sched_entity *se)
{
unsigned long gran = sysctl_sched_wakeup_granularity;

@@ -6492,7 +6491,7 @@ wakeup_preempt_entity(struct sched_entity *curr, struct sched_entity *se)
if (vdiff <= 0)
return -1;

- gran = wakeup_gran(curr, se);
+ gran = wakeup_gran(se);
if (vdiff > gran)
return 1;