✗ CI.checkpatch: warning for DONOT-Review: Pin vram limit (rev3)

Patchwork <[email protected]> Tue, 04 Aug 2026 15:33:18 -0000
Newsgroups org.freedesktop.lists.intel-xe
Message-ID <178585759800.18880.4949417180972015291@61270ab9df2a>
== Series Details ==

Series: DONOT-Review: Pin vram limit (rev3)
URL   : https://patchwork.freedesktop.org/series/171544/
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 8a52b385f4fe012e6c440727a4385e5a45cb0895
Author: Himal Prasad Ghimiray <[email protected]>
Date:   Tue Aug 4 20:34:37 2026 +0530

    DONOT-Review: Pin vram limit
    
    drm/xe/dma-buf: propagate pin error
    
    Return the error from xe_bo_pin_external() instead of asserting
    success, so callers observe failures such as -ENOSPC from the pin limit.
    
    Signed-off-by: Himal Prasad Ghimiray <[email protected]>
    
    drm/xe: add pin_vram_percent modparam
    
    Add a module parameter that caps the percentage of each VRAM region
    which may be pinned via dma-buf. A value of 0 disables the limit.
    
    Signed-off-by: Himal Prasad Ghimiray <[email protected]>
    
    drm/xe/vram: add pinned vram counters
    
    Add per-region counters tracking dma-buf and total pinned VRAM bytes
    along with the enforced dma-buf pin limit.
    
    Signed-off-by: Himal Prasad Ghimiray <[email protected]>
    
    drm/xe/vram: init dma-buf pin limit
    
    Derive the per-region dma-buf pin limit from pin_vram_percent during
    VRAM manager initialization.
    
    Signed-off-by: Himal Prasad Ghimiray <[email protected]>
    
    drm/xe/bo: account dma-buf pinned vram
    
    Track dma-buf pinned VRAM in the per-region counters on external
    pin and unpin.
    
    Signed-off-by: Himal Prasad Ghimiray <[email protected]>
    
    drm/xe/bo: account kernel pinned vram
    
    Track kernel pinned VRAM in the per-region total counter on pin
    and unpin.
    
    Signed-off-by: Himal Prasad Ghimiray <[email protected]>
    
    drm/xe/bo: enforce dma-buf pin limit
    
    Reject external VRAM pins that would exceed the per-region dma-buf
    limit with -ENOSPC.
    
    Signed-off-by: Himal Prasad Ghimiray <[email protected]>
    
    drm/xe/vram: show pinned vram in debug
    
    Expose the pinned VRAM counters and the dma-buf limit in the VRAM
    manager debug output.
    
    Signed-off-by: Himal Prasad Ghimiray <[email protected]>
    
    drm/xe/tests: add VRAM pin limit KUnit tests
    
    Test under-limit success, over-limit rejection, counter recovery after
    unpin, and the limit=0 unlimited case in xe_bo tests. Add an end-to-end
    dma-buf pin limit test in xe_dma_buf tests.
    
    Signed-off-by: Himal Prasad Ghimiray <[email protected]>
+ /mt/dim checkpatch 814bd7de7f09446c4b9401eecb8b30c3285af78b drm-intel
8a52b385f4fe DONOT-Review: Pin vram limit
-:18: WARNING:BAD_SIGN_OFF: Duplicate signature
#18: 
Signed-off-by: Himal Prasad Ghimiray <[email protected]>

-:25: WARNING:BAD_SIGN_OFF: Duplicate signature
#25: 
Signed-off-by: Himal Prasad Ghimiray <[email protected]>

-:32: WARNING:BAD_SIGN_OFF: Duplicate signature
#32: 
Signed-off-by: Himal Prasad Ghimiray <[email protected]>

-:39: WARNING:BAD_SIGN_OFF: Duplicate signature
#39: 
Signed-off-by: Himal Prasad Ghimiray <[email protected]>

-:46: WARNING:BAD_SIGN_OFF: Duplicate signature
#46: 
Signed-off-by: Himal Prasad Ghimiray <[email protected]>

-:53: WARNING:BAD_SIGN_OFF: Duplicate signature
#53: 
Signed-off-by: Himal Prasad Ghimiray <[email protected]>

-:60: WARNING:BAD_SIGN_OFF: Duplicate signature
#60: 
Signed-off-by: Himal Prasad Ghimiray <[email protected]>

-:68: WARNING:BAD_SIGN_OFF: Duplicate signature
#68: 
Signed-off-by: Himal Prasad Ghimiray <[email protected]>

-:90: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#90: FILE: drivers/gpu/drm/xe/tests/xe_bo.c:835:
+static struct xe_ttm_vram_mgr *pin_limit_get_mgr(struct xe_device *xe,
+						  struct kunit *test)

total: 0 errors, 8 warnings, 1 checks, 574 lines checked