✓ CI.checkpatch: success for drm/xe: Remove 'Force write completion' from MI_STORE_DATA_IMM

Patchwork patchwork at emeril.freedesktop.org
Mon Dec 23 18:35:03 UTC 2024


== Series Details ==

Series: drm/xe: Remove 'Force write completion' from MI_STORE_DATA_IMM
URL   : https://patchwork.freedesktop.org/series/142944/
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
30ab6715fc09baee6cc14cb3c89ad8858688d474
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit b132c7e377f5d1fc7dbe5a96657d128d7e1eeabb
Author: José Roberto de Souza <jose.souza at intel.com>
Date:   Mon Dec 23 10:29:18 2024 -0800

    drm/xe: Remove 'Force write completion' from MI_STORE_DATA_IMM
    
    In all places the MI_STORE_DATA_IMM are not followed by a read of
    the same memory address in the same batch buffer and the posted writes
    are flushed with PIPE_CONTROL or MI_FLUSH_DW in xe_ring_ops.c functions
    so there is no need to set this register.
    
    This is not a revert because I think it is worthy keep the register
    definition and the comment on top of it so future readers don't
    get the same wrong conclusion as I did.
    
    Cc: Thomas Hellström <thomas.hellstrom at linux.intel.com>
    Cc: Ashutosh Dixit <ashutosh.dixit at intel.com>
    Fixes: 1460bb1fef9c ("drm/xe: Force write completion of MI_STORE_DATA_IMM")
    Signed-off-by: José Roberto de Souza <jose.souza at intel.com>
+ /mt/dim checkpatch 8fd79761c1b6984d01b45a4bc8eb8ad81f6546e4 drm-intel
b132c7e377f5 drm/xe: Remove 'Force write completion' from MI_STORE_DATA_IMM




More information about the Intel-xe mailing list