[PATCH V4 00/16] Reset improvements for GC10+

[Alex Deucher]

This set improves per queue reset support for GC10+.
When we reset the queue, the queue is lost so we need
to re-emit the unprocessed state from subsequent submissions.
To that end, in order to make sure we actually restore
unprocessed state, we need to enable legacy enforce isolation
so that we can safely re-emit the unprocessed state.  If
we don't multiple jobs can run in parallel and we may not
end up resetting the correct one.  This is similar to how
windows handles queues.  This also gives us correct guilty
tracking for GC.

Tested on GC 10 and 11 chips with a game running and
then running hang tests.  The game pauses when the
hang happens, then continues after the queue reset.

I tried this same approach and GC8 and 9, but it
was not as reliable as soft recovery.  As such, I've dropped
the KGQ reset code for pre-GC10.

The same approach can be extended to SDMA and VCN in the future.
They don't need enforce isolation because those engines
are single threaded so they always operate serially.

Rework re-emit to signal the seq number of the bad job and
verify that to verify that the reset worked, then re-emit the
rest of the non-guilty state.  This way we are not waiting on
the rest of the state to complete, and if the subsequent state
also contains a bad job, we'll end up in queue reset again rather
than adapter reset.

v4: Drop explicit padding patches
    Drop new timeout macro
    Rework re-emit sequence

Alex Deucher (15):
  drm/amdgpu/gfx10: enable legacy enforce isolation
  drm/amdgpu/gfx11: enable legacy enforce isolation
  drm/amdgpu/gfx12: enable legacy enforce isolation
  drm/amdgpu/gfx7: drop reset_kgq
  drm/amdgpu/gfx8: drop reset_kgq
  drm/amdgpu/gfx9: drop reset_kgq
  drm/amdgpu: track ring state associated with a job
  drm/amdgpu/gfx10: re-emit unprocessed state on kgq reset
  drm/amdgpu/gfx11: re-emit unprocessed state on kgq reset
  drm/amdgpu/gfx12: re-emit unprocessed state on kgq reset
  drm/amdgpu/gfx9: re-emit unprocessed state on kcq reset
  drm/amdgpu/gfx9.4.3: re-emit unprocessed state on kcq reset
  drm/amdgpu/gfx10: re-emit unprocessed state on kcq reset
  drm/amdgpu/gfx11: re-emit unprocessed state on kcq reset
  drm/amdgpu/gfx12: re-emit unprocessed state on kcq reset

Christian König (1):
  drm/amdgpu: rework queue reset scheduler interaction

 drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c   |  6 ++
 drivers/gpu/drm/amd/amdgpu/amdgpu_job.c  | 32 ++++++-----
 drivers/gpu/drm/amd/amdgpu/amdgpu_job.h  |  2 +
 drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c | 23 ++++++++
 drivers/gpu/drm/amd/amdgpu/amdgpu_ring.h |  5 ++
 drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c   | 67 +++++++++++++---------
 drivers/gpu/drm/amd/amdgpu/gfx_v11_0.c   | 60 +++++++++++++-------
 drivers/gpu/drm/amd/amdgpu/gfx_v12_0.c   | 60 +++++++++++++-------
 drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c    | 71 ------------------------
 drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c    | 71 ------------------------
 drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c    | 65 ++++++----------------
 drivers/gpu/drm/amd/amdgpu/gfx_v9_4_3.c  | 21 ++++++-
 12 files changed, 214 insertions(+), 269 deletions(-)

-- 
2.49.0