✗ CI.checkpatch: warning for drm/xe: Fix out-of-bounds field write in MI_STORE_DATA_IMM (rev3)

Patchwork patchwork at emeril.freedesktop.org
Fri Jun 13 07:29:32 UTC 2025


== Series Details ==

Series: drm/xe: Fix out-of-bounds field write in MI_STORE_DATA_IMM (rev3)
URL   : https://patchwork.freedesktop.org/series/149435/
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 651becb74c543f10e4326b26310cad06b12cd500
Author: Jia Yao <jia.yao at intel.com>
Date:   Thu Jun 12 22:46:20 2025 +0000

    drm/xe: Fix out-of-bounds field write in MI_STORE_DATA_IMM
    
    According to Bspec, bits 0~9 of MI_STORE_DATA_IMM must not exceed 0x3FE.
    The macro MI_SDI_NUM_QW(x) evaluates to 2 * x + 1, which means the
    condition 2 * x + 1 <= 0x3FE must be satisfied. Therefore, the maximum
    valid value for x is 0x1FE, not 0x1FF.
    
    v2
     - Replace 0x1fe with macro MAX_PTE_PER_SDI (Auld, Matthew & Patelczyk, Maciej)
    
    v3
     - Change macro MAX_PTE_PER_SDI from 0x1fe to 0x1feU (De Marchi, Lucas)
    
    Bspec: 60246
    
    Fixes: 9c44fd5f6e8a ("drm/xe: Add migrate layer functions for SVM support")
    Cc: Matthew Brost <matthew.brost at intel.com>
    Cc: Brian3 Nguyen <brian3.nguyen at intel.com>
    Cc: Alex Zuo <alex.zuo at intel.com>
    Cc: Matthew Auld <matthew.auld at intel.com>
    Cc: Maciej Patelczyk <maciej.patelczyk at intel.com>
    Cc: Lucas De Marchi <lucas.demarchi at intel.com>
    Suggested-by: Shuicheng Lin <shuicheng.lin at intel.com>
    Signed-off-by: Jia Yao <jia.yao at intel.com>
+ /mt/dim checkpatch 93593ca3b525390c11682f32f6d803032acd56b2 drm-intel
651becb74c54 drm/xe: Fix out-of-bounds field write in MI_STORE_DATA_IMM
-:12: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#12: 
 - Replace 0x1fe with macro MAX_PTE_PER_SDI (Auld, Matthew & Patelczyk, Maciej)

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




More information about the Intel-xe mailing list