Intel-XE Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/1] Maintenence of devcoredump <-> GuC-Err-Capture plumbing
@ 2024-11-17 18:43 Alan Previn
  2024-11-17 18:44 ` [PATCH 1/1] drm/xe/guc/capture: " Alan Previn
                   ` (8 more replies)
  0 siblings, 9 replies; 12+ messages in thread
From: Alan Previn @ 2024-11-17 18:43 UTC (permalink / raw)
  To: intel-xe
  Cc: Alan Previn, dri-devel, Daniele Ceraolo Spurio, John Harrison,
	Matthew Brost, Zhanjun Dong

The GuC-Error-Capture is currently reaching into xe_devcoredump
structure to store its own place-holder snaphot to workaround
the race between G2H-Error-Capture-Notification vs Drm-Scheduler
triggering GuC-Submission-exec-queue-timeout/kill.

Part of that race workaround design included GuC-Error-Capture taking
on some of the front-end functions for xe_hw_engine_snapshot
generation because of the orthogonal debugfs for raw dumps of engine
registers without any job association. We want this to also be handled,
even if indirectly, by GuC-Error-Capture since there is a lot to manage
when it comes to reading and printing the register lists.

However, logically speaking, GuC-Error-Capture node management,
despite being the majority of an engine-snapshot work, is still
a subset of xe_hw_engine_snapshot.

This series intends to re-design the plumbing for future
maintenence and scalability, rearranging the layering
back to what its should be (xe_devcoredump_snapshot owns
xe_hw_engine_snapshot owns xe_guc_capture_snapshot)..

Alan Previn (1):
  drm/xe/guc/capture: Maintenence of devcoredump <-> GuC-Err-Capture
    plumbing

 drivers/gpu/drm/xe/xe_devcoredump.c           |   3 -
 drivers/gpu/drm/xe/xe_devcoredump_types.h     |   6 -
 drivers/gpu/drm/xe/xe_guc_capture.c           | 397 ++++++++----------
 drivers/gpu/drm/xe/xe_guc_capture.h           |  10 +-
 .../drm/xe/xe_guc_capture_snapshot_types.h    |  68 +++
 drivers/gpu/drm/xe/xe_guc_submit.c            |  21 +-
 drivers/gpu/drm/xe/xe_hw_engine.c             | 117 ++++--
 drivers/gpu/drm/xe/xe_hw_engine.h             |   4 +-
 drivers/gpu/drm/xe/xe_hw_engine_types.h       |  13 +-
 9 files changed, 353 insertions(+), 286 deletions(-)
 create mode 100644 drivers/gpu/drm/xe/xe_guc_capture_snapshot_types.h


base-commit: 2d4993e163401c81e138c6e0e6223752c2e90402
-- 
2.34.1


^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2024-12-03  3:12 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-11-17 18:43 [PATCH 0/1] Maintenence of devcoredump <-> GuC-Err-Capture plumbing Alan Previn
2024-11-17 18:44 ` [PATCH 1/1] drm/xe/guc/capture: " Alan Previn
2024-11-26 17:09   ` Dong, Zhanjun
2024-12-03  3:12     ` Teres Alexis, Alan Previn
2024-11-18  2:22 ` ✓ CI.Patch_applied: success for " Patchwork
2024-11-18  2:22 ` ✗ CI.checkpatch: warning " Patchwork
2024-11-18  2:23 ` ✓ CI.KUnit: success " Patchwork
2024-11-18  2:41 ` ✓ CI.Build: " Patchwork
2024-11-18  2:42 ` ✗ CI.Hooks: failure " Patchwork
2024-11-18  2:43 ` ✓ CI.checksparse: success " Patchwork
2024-11-18  3:02 ` ✓ CI.BAT: " Patchwork
2024-11-18  4:03 ` ✗ CI.FULL: failure " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox