On Tue, Aug 23, 2016 at 03:55:52PM -0400, Waiman Long wrote: > On 08/23/2016 08:46 AM, Peter Zijlstra wrote: > > I have 2 more comments about the code. > 1) There are a couple of places where you only use 0x3 in mutex.c. They > should be replaced by the symbolic name instead. > 2) We should make __mutex_lock_slowpath() a noinline function just like > __mutex_lock_killable_slowpath() or __mutex_lock_interruptible_slowpath().
3) I broken lockdep with the fastpath changes.. we used to only take the slowpath with debugging, so only the slow paths are annotated. Now we uncondtionally use the fast paths.
