[CI] drm/xe: Replace xe_device_wmb by wmb
Maarten Lankhorst
maarten.lankhorst at linux.intel.com
Tue Aug 27 10:03:13 UTC 2024
Hey,
Den 2024-08-27 kl. 01:41, skrev Ashutosh Dixit:
> CI ONLY for now
>
> In xe_device_wmb(), it is not clear what the purpose of register write
> following wmb() is. Replace xe_device_wmb() with just wmb() to see if we
> see any failures in CI.
>
> Signed-off-by: Ashutosh Dixit <ashutosh.dixit at intel.com>
After doing some digging, I came across it was originally added by brost:
"drm/xe: Add wmb after updating page tables with CPU
wmb is a MMIO write on VRAM devices."
I think removing it would break discrete graphics as internal caches need to be flushed.
CC´d Brost to be sure.
Cheers,
Maarten
More information about the Intel-xe
mailing list