On Fri, Aug 9, 2013 at 11:34 PM, Paulo Zanoni <przanoni@xxxxxxxxx> wrote: > 2013/8/9 Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>: >> Quick note... >> >> On Fri, Aug 09, 2013 at 05:10:05PM -0300, Paulo Zanoni wrote: >>> + WARN_ON(!mutex_is_locked(&dev_priv->pc8.lock)); >> >> Preferred form is now lockdep_assert_held(&dev_priv->pc8.lock); > > Should I also convert all our other usages of > WARN_ON(!mutex_is_locked()) and BUG_ON(!mutex_is_locked()) too? On a > separate patch, of course. We have currently no usage of > lockdep_assert_held, and I like consistency, so fully switching to the > preferred form is good IMHO. Tbh I don't understand really why lockdep_assert_held is better ... it's right that it also checks that indeed the current task is holding the lock (and not some random other imposter). But the downside is that it's a noop without CONFIG_PROVE_LOCKING. And due to the massive perf impact of that option not many people actually run with it. At least I tend to only enable it when doing tricky locking work on my dev machines and not in general ... -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
