On Wed, May 14, 2014 at 11:09:03AM +0200, Peter Zijlstra wrote: > On Wed, May 14, 2014 at 12:25:56AM +0200, Frederic Weisbecker wrote: > > @@ -670,10 +670,11 @@ bool sched_can_stop_tick(void) > > > > rq = this_rq(); > > > > - /* Make sure rq->nr_running update is visible after the IPI */ > > - smp_rmb(); > > - > > - /* More than one running task need preemption */ > > + /* > > + * More than one running task need preemption. > > + * nr_running update is assumed to be visible > > + * after IPI is sent from wakers. > > + */ > > if (rq->nr_running > 1) > > return false; > > Looks like whitespace damage on that comment's indenting.
Oops! -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

