diff options
| author |   Chris Wilson <chris@chris-wilson.co.uk> | 2017-03-03 12:19:46 +0000 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2017-03-03 13:08:04 +0000 |
| commit | 5400367a864ddafdfb9d5d04cf6f9979665ef349 (patch) | |
| tree | a3d952b72a9163877627907d3b6927cef765bd0c /drivers/gpu/drm/i915/i915_gem_gtt.h | |
| parent | drm/i915: Remove duplicate DDI enabling logic from MST path (diff) | |
| download | linux-dev-5400367a864ddafdfb9d5d04cf6f9979665ef349.tar.xz linux-dev-5400367a864ddafdfb9d5d04cf6f9979665ef349.zip | |
drm/i915: Ensure the engine is idle before manually changing HWS
During reset_all_global_seqno() on seqno rollover, we have to update the
HWS. This causes all in flight requests to be completed, so first we
wait. However, we were only waiting for the requests themselves to be
completed and clearing out the waiter rbtrees - what I had missed was
the extra reference in execlists->port[]. Since commit fe9ae7a3bfdb
("drm/i915/execlists: Detect an out-of-order context switch") we can
detect when the request is retired before the context switch interrupt
is completed. The impact should be neglible outside of debugging.
Testcase: igt/gem_exec_whisper
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Mika Kuoppala <mika.kuoppala@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170303121947.20482-1-chris@chris-wilson.co.uk
Reviewed-by: Mika Kuoppala <mika.kuoppala@intel.com>
Diffstat (limited to 'drivers/gpu/drm/i915/i915_gem_gtt.h')
0 files changed, 0 insertions, 0 deletions