✗ CI.checkpatch: warning for Track FAST_REQ H2Gs to report where errors came from (rev2)

Patchwork patchwork at emeril.freedesktop.org
Wed May 14 18:07:37 UTC 2025


== Series Details ==

Series: Track FAST_REQ H2Gs to report where errors came from (rev2)
URL   : https://patchwork.freedesktop.org/series/148912/
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
202708c00696422fd217223bb679a353a5936e23
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 4d61b5a2312c329e97b58faddee256401e1286f4
Author: John Harrison <John.C.Harrison at Intel.com>
Date:   Mon May 12 14:53:24 2025 -0700

    drm/xe/guc: Track FAST_REQ H2Gs to report where errors came from
    
    Most H2G messages are FAST_REQ which means no synchronous response is
    expected. The messages are sent as fire-and-forget with no tracking.
    However, errors can still be returned when something goes unexpectedly
    wrong. That leads to confusion due to not being able to match up the
    error response to the originating H2G.
    
    So add support for tracking the FAST_REQ H2Gs and matching up an error
    response to its originator. This is only enabled in XE_DEBUG builds
    given that such errors should never happen in a working system and
    there is an overhead for the tracking.
    
    Further, if XE_DEBUG_GUC is enabled then even more memory and time is
    used to record the call stack of each H2G and report that with the
    error. That makes it much easier to work out where a specific H2G came
    from if there are multiple code paths that can send it.
    
    v2: Some re-wording of comments and prints, more consistent use of #if
    vs stub functions - review feedback from Daniele & Michal).
    v3: Split config change to separate patch, improve a debug print
    (review feedback from Michal).
    v4: Bunch of minor tweaks (review feedback from Michal).
    
    Original-i915-code: Michal Wajdeczko <michal.wajdeczko at intel.com>
    Signed-off-by: John Harrison <John.C.Harrison at Intel.com>
    Reviewed-by: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
    Reviewed-by: Michal Wajdeczko <michal.wajdeczko at intel.com>
+ /mt/dim checkpatch dd6a856541dd90fa4ef755783898fcfbea3b9961 drm-intel
c4ffbb8faa6d drm/xe/guc: Remove double blank line
251adc2e8eac drm/xe/guc: Add missing H2G error code definitions
59e620420226 drm/xe/guc: Rename CONFIG_XE_LARGE_GUC_BUFFER
-:23: WARNING:CONFIG_DESCRIPTION: please write a help paragraph that fully describes the config symbol with at least 4 lines
#23: FILE: drivers/gpu/drm/xe/Kconfig.debug:89:
+config DRM_XE_DEBUG_GUC
+        bool "Enable extra GuC related debug options"
+        depends on DRM_XE_DEBUG
         default n
         help
           Choose this option when debugging guc issues.
+          The GuC log buffer is increased to the maximum allowed, which should
+          The GuC log buffer is increased to the maximum allowed, which should
+          be large enough for complex issues.
 
           Recommended for driver developers only.
 

total: 0 errors, 1 warnings, 0 checks, 25 lines checked
4d61b5a2312c drm/xe/guc: Track FAST_REQ H2Gs to report where errors came from




More information about the Intel-xe mailing list