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

Patchwork patchwork at emeril.freedesktop.org
Tue May 27 21:45:47 UTC 2025


== Series Details ==

Series: drm/xe: Fix out-of-bounds field write in MI_STORE_DATA_IMM (rev2)
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 c70ccc4b0b22efc270e835cf351ebee53d919149
Author: Jia Yao <jia.yao at intel.com>
Date:   Tue May 27 16:30:50 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)
    
    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>
    Suggested-by: Shuicheng Lin <shuicheng.lin at intel.com>
    Signed-off-by: Jia Yao <jia.yao at intel.com>
    Reviewed-by: Maciej Patelczyk <maciej.patelczyk at intel.com>
    Reviewed-by: Matthew Brost <matthew.brost at intel.com>
+ /mt/dim checkpatch 07b6736f75df3e60d5e20eda53ecb97f6d9a66c0 drm-intel
c70ccc4b0b22 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, 24 lines checked




More information about the Intel-xe mailing list