Intel-XE Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 0/3] drm: Create an app info option for wedge events
@ 2025-05-11 22:47 André Almeida
  2025-05-11 22:47 ` [PATCH v2 1/3] " André Almeida
                   ` (10 more replies)
  0 siblings, 11 replies; 17+ messages in thread
From: André Almeida @ 2025-05-11 22:47 UTC (permalink / raw)
  To: Alex Deucher, Christian König, siqueira, airlied, simona,
	Raag Jadav, rodrigo.vivi, jani.nikula, Xaver Hugl,
	Pierre-Loup A . Griffais
  Cc: dri-devel, linux-kernel, kernel-dev, amd-gfx, intel-xe, intel-gfx,
	André Almeida

This patchset implements a request made by Xaver Hugl about wedge events:

"I'd really like to have the PID of the client that triggered the GPU
reset, so that we can kill it if multiple resets are triggered in a
row (or switch to software rendering if it's KWin itself) and show a
user-friendly notification about why their app(s) crashed, but that
can be added later."

From https://lore.kernel.org/dri-devel/CAFZQkGwJ4qgHV8WTp2=svJ_VXhb-+Y8_VNtKB=jLsk6DqMYp9w@mail.gmail.com/

For testing, I've used amdgpu's debug_mask options debug_disable_soft_recovery
and debug_disable_gpu_ring_reset to test both wedge event paths in the driver.
To trigger a ring timeout, I've used this app:
https://gitlab.freedesktop.org/andrealmeid/gpu-timeout

Thanks!

Changelog:

v2:
  - Rebased on top of drm/drm-next
  - Added new patch for documentation


André Almeida (3):
  drm: Create an app info option for wedge events
  drm/doc: Add a section about "App information" for the wedge API
  drm/amdgpu: Make use of drm_wedge_app_info

 Documentation/gpu/drm-uapi.rst             | 15 +++++++++++++++
 drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 19 +++++++++++++++++--
 drivers/gpu/drm/amd/amdgpu/amdgpu_job.c    |  6 +++++-
 drivers/gpu/drm/drm_drv.c                  | 16 +++++++++++++---
 drivers/gpu/drm/i915/gt/intel_reset.c      |  3 ++-
 drivers/gpu/drm/xe/xe_device.c             |  3 ++-
 include/drm/drm_device.h                   |  8 ++++++++
 include/drm/drm_drv.h                      |  3 ++-
 8 files changed, 64 insertions(+), 9 deletions(-)

-- 
2.49.0


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

end of thread, other threads:[~2025-05-12 18:37 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-05-11 22:47 [PATCH v2 0/3] drm: Create an app info option for wedge events André Almeida
2025-05-11 22:47 ` [PATCH v2 1/3] " André Almeida
2025-05-12  6:08   ` Krzysztof Karas
2025-05-12 13:40     ` André Almeida
2025-05-12 14:53     ` Christian König
2025-05-12 15:34   ` Rodrigo Vivi
2025-05-11 22:47 ` [PATCH v2 2/3] drm/doc: Add a section about "App information" for the wedge API André Almeida
2025-05-12 18:37   ` Raag Jadav
2025-05-11 22:47 ` [PATCH v2 3/3] drm/amdgpu: Make use of drm_wedge_app_info André Almeida
2025-05-11 22:53 ` ✓ CI.Patch_applied: success for drm: Create an app info option for wedge events (rev2) Patchwork
2025-05-11 22:54 ` ✗ CI.checkpatch: warning " Patchwork
2025-05-11 22:55 ` ✓ CI.KUnit: success " Patchwork
2025-05-11 23:03 ` ✓ CI.Build: " Patchwork
2025-05-11 23:05 ` ✓ CI.Hooks: " Patchwork
2025-05-11 23:07 ` ✗ CI.checksparse: warning " Patchwork
2025-05-11 23:35 ` ✓ Xe.CI.BAT: success " Patchwork
2025-05-12  0:48 ` ✗ Xe.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