✓ CI.checkpatch: success for drm/xe/gt: Fix assert in L3 bank mask generation (rev3)

Patchwork patchwork at emeril.freedesktop.org
Tue Apr 30 17:53:16 UTC 2024


== Series Details ==

Series: drm/xe/gt: Fix assert in L3 bank mask generation (rev3)
URL   : https://patchwork.freedesktop.org/series/133055/
State : success

== 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
0daf0be5bb95eb0a0e42275e00a0e42d8d8fd543
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 86cf50ba0e84c150f39bc8a9715cd53fa8eef2ae
Author: Francois Dugast <francois.dugast at intel.com>
Date:   Tue Apr 30 16:39:05 2024 +0000

    drm/xe/gt: Fix assert in L3 bank mask generation
    
    What needs to be asserted is that the pattern fits in the number
    of bits provided by the user in patternbits, otherwise it would
    be truncated when replicated according to the mask, which is
    likely not the intended use of this function.
    The pattern argument is a bitmap so use find_last_bit() instead
    of fls(). The bit position starts at index 0 so remove "or equal"
    from the comparison. XE_MAX_L3_BANK_MASK_BITS would be the
    returned value if the pattern is 0, which can be the case on some
    platforms.
    
    v2: Check the result does not overflow the array (Lucas De Marchi)
    
    Cc: Matt Roper <matthew.d.roper at intel.com>
    Cc: Lucas De Marchi <lucas.demarchi at intel.com>
    Signed-off-by: Francois Dugast <francois.dugast at intel.com>
+ /mt/dim checkpatch 8aebdf15c4291ea17c810781d54a16f7d12c19fb drm-intel
86cf50ba0e84 drm/xe/gt: Fix assert in L3 bank mask generation




More information about the Intel-xe mailing list