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

Patchwork patchwork at emeril.freedesktop.org
Tue Sep 10 00:49:05 UTC 2024


== Series Details ==

Series: drm/xe/oa: Insert wmb/sfence before enabling OA (rev4)
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
c62d7e164862503a3662a095da1c6c9014248cb2
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 6aca830d83c4eef3b7c1dd36f71a64f0a74e7f80
Author: Ashutosh Dixit <ashutosh.dixit at intel.com>
Date:   Mon Sep 9 17:43:13 2024 -0700

    drm/xe/oa: Insert wmb/sfence before enabling OA
    
    We are occasionally seeing 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 40360ec4ceee7e8c16a187299cadf3e6babc8114 drm-intel
6aca830d83c4 drm/xe/oa: Insert wmb/sfence before enabling OA
-:17: WARNING:BAD_REPORTED_BY_LINK: Reported-by: should be immediately followed by Closes: with a URL to the report
#17: 
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