✗ CI.checkpatch: warning for drm/xe/guc: Compress Gu C log and CTB dump with zstd (rev3)

Patchwork <[email protected]> Fri, 31 Jul 2026 16:00:11 -0000
Newsgroups org.freedesktop.lists.intel-xe
Message-ID <178551361190.9497.10525515933257507126@61270ab9df2a>
== Series Details ==

Series: drm/xe/guc: Compress GuC log and CTB dump with zstd (rev3)
URL   : https://patchwork.freedesktop.org/series/170589/
State : warning

== Summary ==

+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
061140b9bc586ae7f40abc1249c97e1cc72d1b9d
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 067d55f3f5ceacc80cd72f91791415c630abbdb6
Author: Zhanjun Dong <[email protected]>
Date:   Fri Jul 31 11:54:11 2026 -0400

    drm/xe/guc: Compress GuC log and CTB dump with zstd
    
    With the default settings, the GuC log is several megabytes in size.
    Printing the entire log is time-consuming. In debugging scenarios,
    printing the GuC log to dmesg while forwarding the dmesg output to a
    serial console can take several minutes. It can also exhaust the storage
    allocated to CI logs. Follow i915's approach and compress the GuC dump
    data to address these issues.
    
    Replace the raw ascii85 dumps of GuC log snapshots with a
    zstd-compressed ascii85 stream and reuse the same helper for CTB
    snapshot output.
    
    Use zstd streaming with the default compression level of 3 and an
    estimated source size of 0 so the compression workspace stays bounded
    even for multi-megabyte logs.
    
    Add a new Kconfig option DRM_XE_COMPRESS_DUMP to control whether
    GuC log and CTB dumps are compressed with ZSTD. This allows users to
    choose between reduced dump sizes and lower memory usage.
    
    Signed-off-by: Zhanjun Dong <[email protected]>
+ /mt/dim checkpatch f9ef9f302ee01854066ec6da30550da6ea6cb03a drm-intel
57914a8d885c drm/xe: Remove extra newlines from LRC snapshot dump
067d55f3f5ce drm/xe/guc: Compress GuC log and CTB dump with zstd
-:199: CHECK:CAMELCASE: Avoid CamelCase: <cParams>
#199: FILE: drivers/gpu/drm/xe/xe_guc_log.c:255:
+	wksp_size = zstd_cstream_workspace_bound(&params.cParams);

-:226: CHECK:CAMELCASE: Avoid CamelCase: <ZSTD_isError>
#226: FILE: drivers/gpu/drm/xe/xe_guc_log.c:282:
+			if (ZSTD_isError(ret)) {

-:228: CHECK:CAMELCASE: Avoid CamelCase: <ZSTD_getErrorName>
#228: FILE: drivers/gpu/drm/xe/xe_guc_log.c:284:
+					  ZSTD_getErrorName(ret));

total: 0 errors, 0 warnings, 3 checks, 321 lines checked