✗ CI.checkpatch: warning for drm/xe/display: fix potential overflow when multiplying 2 u32 (rev2)

Patchwork patchwork at emeril.freedesktop.org
Mon Mar 18 18:21:54 UTC 2024


== Series Details ==

Series: drm/xe/display: fix potential overflow when multiplying 2 u32 (rev2)
URL   : https://patchwork.freedesktop.org/series/131013/
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
a9eb1ac8298ef9f9146567c29fa762d8e9efa1ef
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 6b7961ce316c3cfd6940d6cabb644ed51e2d77cd
Author: Arun R Murthy <arun.r.murthy at intel.com>
Date:   Mon Mar 18 16:31:03 2024 +0530

    drm/xe/display: fix potential overflow when multiplying 2 u32
    
    Multiplying XE_PAGE_SIZE with another u32 and the product stored in
    u64 can potentially lead to overflow, use mul_u32_u32 instead.
    
    Signed-off-by: Arun R Murthy <arun.r.murthy at intel.com>
+ /mt/dim checkpatch ce8cc731d53f9197a853b0d00386d7835f2b80e6 drm-intel
6b7961ce316c drm/xe/display: fix potential overflow when multiplying 2 u32
-:20: WARNING:LONG_LINE: line length of 101 exceeds 100 columns
#20: FILE: drivers/gpu/drm/xe/display/xe_fb_pin.c:32:
+			u64 pte = ggtt->pt_ops->pte_encode_bo(bo, mul_u32_u32(src_idx, XE_PAGE_SIZE),

-:30: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#30: FILE: drivers/gpu/drm/xe/display/xe_fb_pin.c:65:
+				     pte_encode_bo(bo, mul_u32_u32(src_idx, XE_PAGE_SIZE),
 				     xe->pat.idx[XE_CACHE_WB]));

-:47: WARNING:LONG_LINE: line length of 101 exceeds 100 columns
#47: FILE: drivers/gpu/drm/xe/display/xe_fb_pin.c:167:
+			u64 pte = ggtt->pt_ops->pte_encode_bo(bo, mul_u32_u32(src_idx, XE_PAGE_SIZE),

total: 0 errors, 2 warnings, 1 checks, 38 lines checked




More information about the Intel-xe mailing list