drm/i915/gt: Lift set-wedged engine dumping out of user paths
authorChris Wilson <chris@chris-wilson.co.uk>
Mon, 27 Jan 2020 23:15:40 +0000 (23:15 +0000)
committerChris Wilson <chris@chris-wilson.co.uk>
Tue, 28 Jan 2020 13:09:32 +0000 (13:09 +0000)
commita28477826aef031745e6e1194a1eabb22ee7d494
tree2fecf297338026f066ec4fb8b944d64f937ac3d2
parentff3d4ff6c9e66ba1602d0b89255961ddd43e7db7
drm/i915/gt: Lift set-wedged engine dumping out of user paths

The user (e.g. gem_eio) can manipulate the driver into wedging itself,
allowing the user to trigger voluminous logging of inconsequential
details. If we lift the dump to direct calls to intel_gt_set_wedged(),
out of the intel_reset failure handling, we keep the detail logging for
what we expect are true HW or test failures without being tricked.

Reported-by: Tomi Sarvela <tomi.p.sarvela@intel.com>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Cc: Tomi Sarvela <tomi.p.sarvela@intel.com>
Reviewed-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200127231540.3302516-6-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/gt/intel_reset.c