[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