On pe, 2016-10-07 at 10:46 +0100, Chris Wilson wrote: > @@ -4211,10 +4240,10 @@ i915_gem_madvise_ioctl(struct drm_device *dev, void *data, > i915_gem_object_truncate(obj); > > args->retained = obj->mm.madv != __I915_MADV_PURGED; > + mutex_unlock(&obj->mm.lock); > > +err: Shared, so 'out:' Other than that, looks good. Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> Am I to assume that now when we introduced a new lock, every single function that assumes the lock taken is now tagged with assert_lockdep_held? Regards, Joonas -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
