On Tue, 2015-03-31 at 14:07 +0530, Preeti U Murthy wrote: > On 03/31/2015 12:25 AM, Jason Low wrote:
> > diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c > > index fdae26e..ba8ec1a 100644 > > --- a/kernel/sched/fair.c > > +++ b/kernel/sched/fair.c > > @@ -7644,7 +7644,7 @@ static void nohz_idle_balance(struct rq *this_rq, > > enum cpu_idle_type idle) > > * balancing owner will pick it up. > > */ > > if (need_resched()) > > - break; > > + goto end; > > Why is this hunk needed? In terms of the change in the need_resched() case, if the current CPU doesn't complete iterating all of the CPUs, then this will make it not update nohz.next_balance. This is so we can continue the balancing with the next balancing owner without too much delay. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
