Re: [PATCH 11/14] sched: filter task pull request

2013-05-22 Thread Vincent Guittot
On 22 May 2013 17:56, Morten Rasmussen wrote: > On Fri, Apr 26, 2013 at 11:00:58AM +0100, Vincent Guittot wrote: >> Part of this patch is missing, the fix below is needed >> >> @@ -3497,7 +3497,9 @@ static bool is_buddy_full(int cpu) >> static bool is_my_buddy(int cpu, int buddy) >> { >> int

Re: [PATCH 11/14] sched: filter task pull request

2013-05-22 Thread Morten Rasmussen
On Fri, Apr 26, 2013 at 11:00:58AM +0100, Vincent Guittot wrote: > Part of this patch is missing, the fix below is needed > > @@ -3497,7 +3497,9 @@ static bool is_buddy_full(int cpu) > static bool is_my_buddy(int cpu, int buddy) > { > int my_buddy = per_cpu(sd_pack_buddy, cpu); > - return

Re: [PATCH 11/14] sched: filter task pull request

2013-05-22 Thread Morten Rasmussen
On Fri, Apr 26, 2013 at 11:00:58AM +0100, Vincent Guittot wrote: Part of this patch is missing, the fix below is needed @@ -3497,7 +3497,9 @@ static bool is_buddy_full(int cpu) static bool is_my_buddy(int cpu, int buddy) { int my_buddy = per_cpu(sd_pack_buddy, cpu); - return (my_buddy

Re: [PATCH 11/14] sched: filter task pull request

2013-05-22 Thread Vincent Guittot
On 22 May 2013 17:56, Morten Rasmussen morten.rasmus...@arm.com wrote: On Fri, Apr 26, 2013 at 11:00:58AM +0100, Vincent Guittot wrote: Part of this patch is missing, the fix below is needed @@ -3497,7 +3497,9 @@ static bool is_buddy_full(int cpu) static bool is_my_buddy(int cpu, int buddy)

Re: [PATCH 11/14] sched: filter task pull request

2013-04-26 Thread Vincent Guittot
Part of this patch is missing, the fix below is needed @@ -3497,7 +3497,9 @@ static bool is_buddy_full(int cpu) static bool is_my_buddy(int cpu, int buddy) { int my_buddy = per_cpu(sd_pack_buddy, cpu); - return (my_buddy == -1) || (buddy == my_buddy); + + return ((sysctl_sched_packing_mode ==

Re: [PATCH 11/14] sched: filter task pull request

2013-04-26 Thread Vincent Guittot
Part of this patch is missing, the fix below is needed @@ -3497,7 +3497,9 @@ static bool is_buddy_full(int cpu) static bool is_my_buddy(int cpu, int buddy) { int my_buddy = per_cpu(sd_pack_buddy, cpu); - return (my_buddy == -1) || (buddy == my_buddy); + + return ((sysctl_sched_packing_mode ==

[PATCH 11/14] sched: filter task pull request

2013-04-25 Thread Vincent Guittot
Only CPUs that participates to the packing effort can pull tasks on a busiest group. Signed-off-by: Vincent Guittot --- kernel/sched/fair.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index 28f8ea7..6f63fb9

[PATCH 11/14] sched: filter task pull request

2013-04-25 Thread Vincent Guittot
Only CPUs that participates to the packing effort can pull tasks on a busiest group. Signed-off-by: Vincent Guittot vincent.guit...@linaro.org --- kernel/sched/fair.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c