drm/i915/gt: Mark up the nested engine-pm timeline lock as irqsafe
authorChris Wilson <chris@chris-wilson.co.uk>
Mon, 19 Aug 2019 07:58:19 +0000 (08:58 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Mon, 19 Aug 2019 09:12:23 +0000 (10:12 +0100)
commitd67739268cf0ee928cdc5e8224090c59efacd653
tree75324d9b48a47246d177528f0e6678a5c6843d32
parent6a736ebf314a0b0813bae248ad3daa9154463798
drm/i915/gt: Mark up the nested engine-pm timeline lock as irqsafe

We use a fake timeline->mutex lock to reassure lockdep that the timeline
is always locked when emitting requests. However, the use inside
__engine_park() may be inside hardirq and so lockdep now complains about
the mixed irq-state of the nested locked. Disable irqs around the
lockdep tracking to keep it happy.

Fixes: 6c69a45445af ("drm/i915/gt: Mark context->active_count as protected by timeline->mutex")
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Reviewed-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190819075835.20065-3-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/gt/intel_engine_pm.c