2014-10-27 09:43:01

by Yao Dongdong

[permalink] [raw]
Subject: [PATCH v2]sched:add check of shallowest_idle_cpu before search for least_loaded_cpu

Idle cpu is idler than non-idle cpu, so we needn't search for least_loaded_cpu
after we have found an idle cpu.

Signed-off-by:[email protected]
Reviewed-by: Srikar Dronamraju <[email protected]>
---
kernel/sched/fair.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
index 0b069bf..2445a23 100644
--- a/kernel/sched/fair.c
+++ b/kernel/sched/fair.c
@@ -4446,7 +4446,7 @@ find_idlest_cpu(struct sched_group *group, struct task_struct *p, int this_cpu)
latest_idle_timestamp = rq->idle_stamp;
shallowest_idle_cpu = i;
}
- } else {
+ } else if (shallowest_idle_cpu == -1) {
load = weighted_cpuload(i);
if (load < min_load || (load == min_load && i == this_cpu)) {
min_load = load;
--
1.8.0.1


2014-10-27 12:31:06

by Peter Zijlstra

[permalink] [raw]
Subject: Re: [PATCH v2]sched:add check of shallowest_idle_cpu before search for least_loaded_cpu


Please as to try again:

Applying patch patches/yao_dongdong-sched_add_check_of_shallowest_idle_cpu_before_search_for_least_loaded_cpu.patch
patching file kernel/sched/fair.c
Hunk #1 FAILED at 4446.
1 out of 1 hunk FAILED -- rejects in file kernel/sched/fair.c
Patch patches/yao_dongdong-sched_add_check_of_shallowest_idle_cpu_before_search_for_least_loaded_cpu.patch does not apply (enforce with -f)
quilt returned 1, with 0 fuzz and 1 rejects
use quilt -f? y/n [y]: y
Applying patch patches/yao_dongdong-sched_add_check_of_shallowest_idle_cpu_before_search_for_least_loaded_cpu.patch
patching file kernel/sched/fair.c
Hunk #1 FAILED at 4446.
1 out of 1 hunk FAILED -- saving rejects to file kernel/sched/fair.c.rej
Applied patch patches/yao_dongdong-sched_add_check_of_shallowest_idle_cpu_before_search_for_least_loaded_cpu.patch (forced; needs refresh)
reject patch patches/yao_dongdong-sched_add_check_of_shallowest_idle_cpu_before_search_for_least_loaded_cpu.patch file kernel/sched/fair.c hunk 1 line 4
[a]uto[m]erge,edit [f]ile [p]atch [r]ej,[n]ext,[d]elete: [amfdprn]: m
rej started:
[a]uto[m]erge,edit [f]ile [p]atch [r]ej,[n]ext,[d]elete: [amfdprn]: kernel/sched/fair.c: 1 matched, 0 conflicts remain
n
patches/yao_dongdong-sched_add_check_of_shallowest_idle_cpu_before_search_for_least_loaded_cpu.patch done [s]top, [n]ext, [r]efresh [snr]: r
WARNING: Use a single space after Signed-off-by:
#13:
Signed-off-by:[email protected]

ERROR: code indent should use tabs where possible
#25: FILE: kernel/sched/fair.c:4644:
+ } else if (shallowest_idle_cpu == -1) {$

WARNING: please, no spaces at the start of a line
#25: FILE: kernel/sched/fair.c:4644:
+ } else if (shallowest_idle_cpu == -1) {$

total: 1 errors, 2 warnings, 8 lines checked

NOTE: whitespace errors detected, you may wish to use scripts/cleanpatch or
scripts/cleanfile

2014-10-28 01:47:26

by Yao Dongdong

[permalink] [raw]
Subject: Re: [PATCH v2]sched:add check of shallowest_idle_cpu before search for least_loaded_cpu

On 2014/10/27 20:30, Peter Zijlstra wrote:
> Please as to try again:
>
> Applying patch patches/yao_dongdong-sched_add_check_of_shallowest_idle_cpu_before_search_for_least_loaded_cpu.patch
> patching file kernel/sched/fair.c
> Hunk #1 FAILED at 4446.
> 1 out of 1 hunk FAILED -- rejects in file kernel/sched/fair.c
> Patch patches/yao_dongdong-sched_add_check_of_shallowest_idle_cpu_before_search_for_least_loaded_cpu.patch does not apply (enforce with -f)
> quilt returned 1, with 0 fuzz and 1 rejects
> use quilt -f? y/n [y]: y
> Applying patch patches/yao_dongdong-sched_add_check_of_shallowest_idle_cpu_before_search_for_least_loaded_cpu.patch
> patching file kernel/sched/fair.c
> Hunk #1 FAILED at 4446.
> 1 out of 1 hunk FAILED -- saving rejects to file kernel/sched/fair.c.rej
> Applied patch patches/yao_dongdong-sched_add_check_of_shallowest_idle_cpu_before_search_for_least_loaded_cpu.patch (forced; needs refresh)
> reject patch patches/yao_dongdong-sched_add_check_of_shallowest_idle_cpu_before_search_for_least_loaded_cpu.patch file kernel/sched/fair.c hunk 1 line 4
> [a]uto[m]erge,edit [f]ile [p]atch [r]ej,[n]ext,[d]elete: [amfdprn]: m
> rej started:
> [a]uto[m]erge,edit [f]ile [p]atch [r]ej,[n]ext,[d]elete: [amfdprn]: kernel/sched/fair.c: 1 matched, 0 conflicts remain
> n
> patches/yao_dongdong-sched_add_check_of_shallowest_idle_cpu_before_search_for_least_loaded_cpu.patch done [s]top, [n]ext, [r]efresh [snr]: r
> WARNING: Use a single space after Signed-off-by:
> #13:
> Signed-off-by:[email protected]
>
> ERROR: code indent should use tabs where possible
> #25: FILE: kernel/sched/fair.c:4644:
> + } else if (shallowest_idle_cpu == -1) {$
>
> WARNING: please, no spaces at the start of a line
> #25: FILE: kernel/sched/fair.c:4644:
> + } else if (shallowest_idle_cpu == -1) {$
>
> total: 1 errors, 2 warnings, 8 lines checked
>
> NOTE: whitespace errors detected, you may wish to use scripts/cleanpatch or
> scripts/cleanfile
>
> .
>
I 'm sorry, and i will fix and resend another one.

Thanks,
Yao Dongdong