[PATCH v6 0/3] drm/xe: Add devcoredump locking and reason string
John.C.Harrison at Intel.com
John.C.Harrison at Intel.com
Thu Nov 28 21:08:21 UTC 2024
From: John Harrison <John.C.Harrison at Intel.com>
There are now many ways to trigger a devcoredump capture. So include a
reason message to say why it happened.
Also, add locking to avoid a races with double captures and free vs read.
v2: Change to spin_lock to mutex and a few other tweaks (review
feedback from Matthew Brost).
v3: Move mutex locks and add reclaim annotation (review feedback from
Matthew Brost).
v4: Fix typo - when adding lockdep specific code it helps to actually
have CONFIG_LOCKDEP enabled!
v5: New patch to move the call to dev_coredumpm_timeout to prevent an
existing issue highlighted by the new lockdep checking.
v6: Rebase due to conflict with recent change to worker function.
Signed-off-by: John Harrison <John.C.Harrison at Intel.com>
John Harrison (3):
drm/xe: Add a reason string to the devcoredump
drm/xe: Move the coredump registration to the worker thread
drm/xe: Add mutex locking to devcoredump
drivers/gpu/drm/xe/xe_devcoredump.c | 125 +++++++++++++++-------
drivers/gpu/drm/xe/xe_devcoredump.h | 5 +-
drivers/gpu/drm/xe/xe_devcoredump_types.h | 8 +-
drivers/gpu/drm/xe/xe_guc_submit.c | 14 ++-
4 files changed, 107 insertions(+), 45 deletions(-)
--
2.47.0
More information about the Intel-xe
mailing list