✗ CI.checkpatch: warning for drm/xe/pf: Release all VFs configs on device removal (rev2)

[Patchwork]

== Series Details ==

Series: drm/xe/pf: Release all VFs configs on device removal (rev2)
URL   : https://patchwork.freedesktop.org/series/144628/
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
22f9cda3436b4fe965b5c5f31d2f2c1bcb483189
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 4835b7f81e2047a6e36711b2b2308e394018cbe8
Author: Michal Wajdeczko <michal.wajdeczko at intel.com>
Date:   Tue Feb 11 16:50:34 2025 +0100

    drm/xe/pf: Release all VFs configs on device removal
    
    If we try to manually provision VFs using debugfs and then we
    try to unload the driver, we will see complains like:
    
     [ ] Memory manager not clean during takedown.
     [ ] RIP: 0010:drm_mm_takedown+0x3f/0x100
     [ ] [drm:drm_mm_takedown] *ERROR* node [fedff000 + 00001000]: inserted at
          drm_mm_insert_node_in_range+0x2bd/0x520
          xe_ggtt_node_insert+0x52/0x90 [xe]
          pf_provision_vf_ggtt+0x1fa/0xac0 [xe]
          xe_gt_sriov_pf_config_set_ggtt+0x79/0x7a0 [xe]
          ggtt_set+0x53/0x80 [xe]
          simple_attr_write_xsigned.isra.0+0xd2/0x150
          simple_attr_write+0x14/0x30
          debugfs_attr_write+0x4e/0x80
    
     [ ] xe 0000:00:02.0: [drm] *ERROR* GT0: GUC ID manager unclean (1/65535)
     [ ] xe 0000:00:02.0: [drm] GT0:      total 65535
     [ ] xe 0000:00:02.0: [drm] GT0:      used 1
     [ ] xe 0000:00:02.0: [drm] GT0:      range 65534..65534 (1)
    
     [ ] xe 0000:00:02.0: [drm] *ERROR* GT0: GuC doorbells manager unclean (1/256)
     [ ] xe 0000:00:02.0: [drm] GT0:      count: 256
     [ ] xe 0000:00:02.0: [drm] GT0:      available range: 1..255 (255)
     [ ] xe 0000:00:02.0: [drm] GT0:      available total: 255
     [ ] xe 0000:00:02.0: [drm] GT0:      reserved range: 0..0 (1)
     [ ] xe 0000:00:02.0: [drm] GT0:      reserved total: 1
    
    This could be easily fixed by adding config release action.
    
    Signed-off-by: Michal Wajdeczko <michal.wajdeczko at intel.com>
    Cc: Piotr Piórkowski <piotr.piorkowski at intel.com>
+ /mt/dim checkpatch 5a6f3d00770c0c384a79c34a74e049c80d8b235d drm-intel
4835b7f81e20 drm/xe/pf: Release all VFs configs on device removal
-:29: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#29: 
 [ ] xe 0000:00:02.0: [drm] *ERROR* GT0: GuC doorbells manager unclean (1/256)

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