summaryrefslogtreecommitdiff
path: root/drivers/gpu/drm/i915/intel_engine_cs.c
diff options
context:
space:
mode:
authorChris Wilson <chris@chris-wilson.co.uk>2017-12-22 17:19:59 +0300
committerChris Wilson <chris@chris-wilson.co.uk>2017-12-22 21:34:05 +0300
commit2d453c7895c0ca6028500050ff9305c70fd7252f (patch)
tree1c9def2aee240b0780be2c0772db95d4b31d0bdd /drivers/gpu/drm/i915/intel_engine_cs.c
parent193a98dc7cc4770580ebcd0fa64a6c23a50093af (diff)
downloadlinux-2d453c7895c0ca6028500050ff9305c70fd7252f.tar.xz
drm/i915: Assert that the request is on the execution queue before being removed
We should only attempt to remove requests from the execution queue that are on the execution queue. These are the requests that have been assigned a global_seqno, so we can assert that we only attempt to remove requests with a nonzero global_seqno. Afterwards we assert that we remove them in order, i.e. the global_seqno matches the engine's seqno, but that leaves a small loophole for an unattached request on an unused engine. We can then make the same assertion on queuing the request to the execution engine, it must have a zero global_seqno or else we are queuing the same request twice. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com> Cc: MichaƂ Winiarski <michal.winiarski@intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/20171222141959.3006-1-chris@chris-wilson.co.uk Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Diffstat (limited to 'drivers/gpu/drm/i915/intel_engine_cs.c')
0 files changed, 0 insertions, 0 deletions