[Intel-gfx] [PATCH 2/3] drm/i915: struct_mutex is not required for allocating the framebuffer

Joonas Lahtinen joonas.lahtinen at linux.intel.com
Thu Feb 16 13:55:14 UTC 2017


On ke, 2017-02-15 at 10:59 +0000, Chris Wilson wrote:
> We do not need the BKL struct_mutex in order to allocate a GEM object,
> nor to create the framebuffer, so resist the temptation to take the BKL
> willy nilly. As this changes the locking contract around internal API
> calls, the patch is a little larger than a plain removal of a pair of
> mutex_lock/unlock.
> 
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>

Looks good, assuming we have enough lockdep_assert_held sprinkled;

Reviewed-by: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>

Regards, Joonas
-- 
Joonas Lahtinen
Open Source Technology Center
Intel Corporation


More information about the Intel-gfx mailing list