drm/i915: Only skip requests once a context is banned
authorChris Wilson <chris@chris-wilson.co.uk>
Thu, 5 Jan 2017 17:00:59 +0000 (17:00 +0000)
committerChris Wilson <chris@chris-wilson.co.uk>
Thu, 5 Jan 2017 21:06:20 +0000 (21:06 +0000)
commit7ec73b7e36d0bf165cea77770a490fcdf4418e93
tree751dbaf60a7ebb6f41c0d0bff0afb6c63bfd3fca
parentb7163936bd99dab90a71c2496295f62f2b4874ab
drm/i915: Only skip requests once a context is banned

If we skip before banning, we have an inconsistent interface between
execbuf still queueing valid request but those requests already queued
being cancelled. If we only cancel the pending requests once we stop
accepting new requests, the user interface is more consistent.

Reported-by: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
Fixes: 821ed7df6e2a ("drm/i915: Update reset path to fix incomplete requests")
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Mika Kuoppala <mika.kuoppala@intel.com>
Cc: <stable@vger.kernel.org> # v4.9+
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170105170059.344-1-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/i915_gem.c