Re: [PATCH v11 0/3] drm/xe/guc: Compress GuC log and CTB dump with zstd

[Rodrigo Vivi <rodrigo.vivi@intel.com>]

On Tue, Oct 13, 2026 at 03:28:44PM -0700, Alan Previn wrote:
> GuC snapshot output is currently expensive in both size and latency: large
> log buffers are emitted as raw ascii85, and the surrounding dump text
> includes a bit of avoidable noise. This series makes xe snapshot dumps more
> compact and easier to handle by compressing the large binary payloads,
> tightening the textual output.
> 
> Select ZSTD_COMPRESS in DRM_XE so the xe module pulls in the zstd
> compressor when built.
> 
> Signed-off-by: Zhanjun Dong <zhanjun.dong@intel.com>
> Signed-off-by: Alan Previn <alan.previn.teres.alexis@intel.com>

pushed to drm-xe-next, thank you!

> 
> ---
> History:
> V11: Patch3: In ct_dead_worker_func, call xe_guc_log/ct_snapshot_free
>      after xe_guc_ct_snapshot_free (Sashiko).
> v10: Add a new patch to introduce dump_cache to optimize GuC CT/ Log
>      compression e2e flows and manage errors across parses. (Sashiko
>      and Rodrigo).
>      Patch2: Reorder Kconfig compression dependency (Rodrigo) and round
>      up buffer size to dword as per final output-dump expectation.
> v9: Rebase to latest drm-tip, no functional changes, added Signed-
>     off by Alan on 2nd patch. (Rodrigo)
> v8: Improve patch #2 description on clarity and wording (Rodrigo)
> v7: Add *why* to the commit message (Rodrigo)
> v6: Remove intermediate build files(Julia)
>     Remvoe unnecessary comment(Julia)
> v5: Update cover letter messages to remove shrink log size.
>     Fix kernel-doc warning.
> v4: Remove ZSTD_DECOMPRESS dependency.
> v3: Add Kconfig option to control compression of GuC log and CTB dumps,
>     allowing users to choose between reduced dump sizes or lower memory
>     usage.
>     Remove "reduce GuC log size from 8MB to 2MB" patch, to be added later
>     if needed.
>     Replace pr_err with xe_gt_err and add more error print.
> v2: Fix Kunit test error by selecting ZSTD_DECOMPRESS in DRM_XE config,
>     required by zstd_is_error/zstd_get_error_name.
> 
> Alan Previn (1):
>   drm/xe/guc: Add dump_cache to optimize GuC CT/Log compression flow
> 
> Zhanjun Dong (2):
>   drm/xe: Remove extra newlines from LRC snapshot dump
>   drm/xe/guc: Compress GuC log and CTB dump with zstd
> 
>  drivers/gpu/drm/xe/Kconfig            |  15 ++
>  drivers/gpu/drm/xe/xe_devcoredump.c   |   4 +-
>  drivers/gpu/drm/xe/xe_guc_ct.c        |  31 ++--
>  drivers/gpu/drm/xe/xe_guc_ct.h        |   4 +-
>  drivers/gpu/drm/xe/xe_guc_ct_types.h  |   3 +
>  drivers/gpu/drm/xe/xe_guc_log.c       | 222 ++++++++++++++++++++++++--
>  drivers/gpu/drm/xe/xe_guc_log.h       |  11 +-
>  drivers/gpu/drm/xe/xe_guc_log_types.h |  20 +++
>  drivers/gpu/drm/xe/xe_lrc.c           |   4 +-
>  9 files changed, 281 insertions(+), 33 deletions(-)
> 
> 
> base-commit: 83b6bc1390606facc1c0426727905f9fcd75836d
> -- 
> 2.43.0
>