Re: [PATCH] locking/mutex: Set and clear owner using WRITE_ONCE()

From: Jason Low
Date: Mon May 23 2016 - 16:41:53 EST


On Fri, 2016-05-20 at 18:00 -0700, Davidlohr Bueso wrote:
> On Fri, 20 May 2016, Waiman Long wrote:
>
> >I think mutex-debug.h also needs similar changes for completeness.
>
> Maybe, but given that with debug the wait_lock is unavoidable, doesn't
> this send the wrong message?

The mutex_set_owner() and mutex_clear_owner() can still get called in
the fastpath without the wait_lock for the debug case too correct? If
that's the case, then the WRITE_ONCE would still make sense to have in
the debug case.