diff options
| author |   Chris Wilson <chris@chris-wilson.co.uk> | 2019-09-23 12:00:55 +0100 |
|---|---|---|
| committer |   Rodrigo Vivi <rodrigo.vivi@intel.com> | 2019-10-09 13:18:26 -0700 |
| commit | a8385f0c3fd3f31ab53726e26229b428d18a51bf (patch) | |
| tree | e4e9c09158a0dad11010767dfeb8abe2d18c21e5 /scripts/gdb/linux/utils.py | |
| parent | drm/i915/execlists: Drop redundant list_del_init(&rq->sched.link) (diff) | |
| download | wireguard-linux-a8385f0c3fd3f31ab53726e26229b428d18a51bf.tar.xz wireguard-linux-a8385f0c3fd3f31ab53726e26229b428d18a51bf.zip | |
drm/i915: Only enqueue already completed requests
If we are asked to submit a completed request, just move it onto the
active-list without modifying it's payload. If we try to emit the
modified payload of a completed request, we risk racing with the
ring->head update during retirement which may advance the head past our
breadcrumb and so we generate a warning for the emission being behind
the RING_HEAD.
v2: Commentary for the sneaky, shared responsibility between functions.
v3: Spelling mistakes and bonus assertion
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190923110056.15176-3-chris@chris-wilson.co.uk
(cherry picked from commit c0bb487dc19fc45dbeede7dcf8f513df51a3cd33)
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Diffstat (limited to 'scripts/gdb/linux/utils.py')
0 files changed, 0 insertions, 0 deletions