From: Maarten Lankhorst Date: Wed, 21 Feb 2024 13:30:17 +0000 (+0100) Subject: drm/xe: Clear all snapshot members after deleting coredump X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=bd71cdd209c63f3d526aef661282b5252a436c4d;p=linux.git drm/xe: Clear all snapshot members after deleting coredump It's not strictly needed to clear right now, but this prevents bugs from dangling pointers. Signed-off-by: Maarten Lankhorst Reviewed-by: Francois Dugast Link: https://patchwork.freedesktop.org/patch/msgid/20240221133024.898315-2-maarten.lankhorst@linux.intel.com --- diff --git a/drivers/gpu/drm/xe/xe_devcoredump.c b/drivers/gpu/drm/xe/xe_devcoredump.c index 08d3f6cb72292..ae26d8c6d01c5 100644 --- a/drivers/gpu/drm/xe/xe_devcoredump.c +++ b/drivers/gpu/drm/xe/xe_devcoredump.c @@ -124,6 +124,8 @@ static void xe_devcoredump_free(void *data) if (coredump->snapshot.hwe[i]) xe_hw_engine_snapshot_free(coredump->snapshot.hwe[i]); + /* To prevent stale data on next snapshot, clear everything */ + memset(&coredump->snapshot, 0, sizeof(coredump->snapshot)); coredump->captured = false; drm_info(&coredump_to_xe(coredump)->drm, "Xe device coredump has been deleted.\n");