On Wed, May 09, 2018 at 12:21:00PM +0200, Peter Zijlstra wrote: > On Mon, May 07, 2018 at 05:01:34PM -0400, Johannes Weiner wrote: > > + local_irq_disable(); > > + rq = this_rq(); > > + raw_spin_lock(&rq->lock); > > + rq_pin_lock(rq, &rf); > > Given that churn in sched.h, you seen rq_lock() and friends. > > Either write this like: > > local_irq_disable(); > rq = this_rq(); > rq_lock(rq, &rf); > > Or instroduce "rq = this_rq_lock_irq()", which we could also use in > do_sched_yield(). Sounds good, I'll add that. > > + update_rq_clock(rq); > > + > > + current->flags |= PF_MEMSTALL; > > + psi_task_change(current, rq_clock(rq), 0, TSK_MEMSTALL); > > + > > + rq_unpin_lock(rq, &rf); > > + raw_spin_unlock(&rq->lock); > > + local_irq_enable(); > > That's called rq_unlock_irq(). I'll use that. This code was first written against a kernel that didn't have 8a8c69c32778 ("sched/core: Add rq->lock wrappers.") yet ;) -- To unsubscribe from this list: send the line "unsubscribe cgroups" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html