✗ CI.checkpatch: warning for PF: Mitigate unexpected GuC VF config checks

Patchwork patchwork at emeril.freedesktop.org
Wed Jan 29 21:11:39 UTC 2025


== Series Details ==

Series: PF: Mitigate unexpected GuC VF config checks
URL   : https://patchwork.freedesktop.org/series/144120/
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
30ab6715fc09baee6cc14cb3c89ad8858688d474
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 48df448f65dc05c9fff244f05682a4a8be5f2f0b
Author: Michal Wajdeczko <michal.wajdeczko at intel.com>
Date:   Wed Jan 29 20:59:47 2025 +0100

    drm/xe/pf: Reset GuC VF config when unprovisioning critical resource
    
    GuC firmware counts received VF configuration KLVs and may start
    validation of the complete VF config even if some resources where
    unprovisioned in the meantime, leading to unexpected errors like:
    
     $ echo 1 | sudo tee /sys/kernel/debug/dri/0000:00:02.0/gt0/vf1/contexts_quota
     $ echo 0 | sudo tee /sys/kernel/debug/dri/0000:00:02.0/gt0/vf1/contexts_quota
     $ echo 1 | sudo tee /sys/kernel/debug/dri/0000:00:02.0/gt0/vf1/doorbells_quota
     $ echo 0 | sudo tee /sys/kernel/debug/dri/0000:00:02.0/gt0/vf1/doorbells_quota
     $ echo 1 | sudo tee /sys/kernel/debug/dri/0000:00:02.0/gt0/vf1/ggtt_quota
     tee: '/sys/kernel/debug/dri/0000:00:02.0/gt0/vf1/ggtt_quota': Input/output error
    
    To mitigate this problem trigger explicit VF config reset after
    unprovisioning any of the critical resources (GGTT, context or
    doorbell IDs) that GuC is monitoring.
    
    Signed-off-by: Michal Wajdeczko <michal.wajdeczko at intel.com>
+ /mt/dim checkpatch c2a5da40b8b1c5af77dcdabed8516069949fea3b drm-intel
921df509a0ce drm/xe/pf: Don't send BEGIN_ID if VF has no context/doorbells
-:19: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#19: 
 [] xe 0000:00:02.0: [drm] GT0: { key 0x0001 : 64b value 0x176a000 } # ggtt_start

total: 0 errors, 1 warnings, 0 checks, 16 lines checked
48df448f65dc drm/xe/pf: Reset GuC VF config when unprovisioning critical resource
-:11: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#11: 
 $ echo 1 | sudo tee /sys/kernel/debug/dri/0000:00:02.0/gt0/vf1/contexts_quota

total: 0 errors, 1 warnings, 0 checks, 67 lines checked




More information about the Intel-xe mailing list