✗ CI.checkpatch: warning for drm/xe/oa: Insert wmb/sfence before enabling OA (rev3)

Patchwork patchwork at emeril.freedesktop.org
Wed Aug 28 16:02:05 UTC 2024


== Series Details ==

Series: drm/xe/oa: Insert wmb/sfence before enabling OA (rev3)
URL   : https://patchwork.freedesktop.org/series/137816/
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
9fe5037901cabbcdf27a6fe0dfb047ca1474d363
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 439964f02d55c4d72d261e067b6e585989fca2db
Author: Ashutosh Dixit <ashutosh.dixit at intel.com>
Date:   Wed Aug 28 08:55:52 2024 -0700

    drm/xe/oa: Insert wmb/sfence before enabling OA
    
    In some simulation environments, we occasionally see warnings such as "OAG
    Buffer register is not programmed (has value 0). Register offset db08".
    This means OA has been enabled before it has been fully configured. Or, the
    register write enabling OA has overtaken previous OA configuration register
    writes.
    
    Therefore, insert a wmb/sfence to preserve OA register write ordering
    before enabling OA.
    
    v2: s/wmb()/xe_device_wmb()/
    
    Fixes: e936f885f1e9 ("drm/xe/oa/uapi: Expose OA stream fd")
    Reported-by: Guy Zadicario <guy.zadicario at intel.com>
    Signed-off-by: Ashutosh Dixit <ashutosh.dixit at intel.com>
+ /mt/dim checkpatch da0e5d5784bc90f24d02e5944051f2e8a3f53b56 drm-intel
439964f02d55 drm/xe/oa: Insert wmb/sfence before enabling OA
-:18: WARNING:BAD_REPORTED_BY_LINK: Reported-by: should be immediately followed by Closes: with a URL to the report
#18: 
Reported-by: Guy Zadicario <guy.zadicario at intel.com>
Signed-off-by: Ashutosh Dixit <ashutosh.dixit at intel.com>

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




More information about the Intel-xe mailing list