✓ 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