On Mon, Mar 30, 2015 at 03:29:09PM +0200, Vincent Guittot wrote: > On 30 March 2015 at 14:24, Peter Zijlstra <pet...@infradead.org> wrote: > > @@ -7647,6 +7648,8 @@ static void nohz_idle_balance(struct rq *this_rq, > > enum cpu_idle_type idle) > > break; > > > > rq = cpu_rq(balance_cpu); > > + if (rq == this_rq) > > + done = true; > > AFAICT, this can't happen because we start the for_each _cpu loop with: > if (balance_cpu == this_cpu || !idle_cpu(balance_cpu)) > continue;
Oh hey, look at me being blind ;-) -- 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/