[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