[Intel-gfx] [PATCH] drm/i915: init per-engine WAs for all engines

Chris Wilson chris at chris-wilson.co.uk
Wed Jan 9 21:51:35 UTC 2019


Quoting Chris Wilson (2019-01-09 21:48:29)
> Quoting Daniele Ceraolo Spurio (2019-01-09 21:30:38)
> > commit 4a15c75c4246 ("drm/i915: Introduce per-engine workarounds")
> > refactored the workaround code to have functions per-engine, but didn't
> > call any of them from logical_xcs_ring_init. Since we do have a non-RCS
> > workaround for KBL (WaKBLVECSSemaphoreWaitPoll) we do need to call
> > intel_engine_init_workarounds for non-RCS engines.
> > Note that whitelist is still RCS-only.
> 
> Yeah, the danger is that our selftests are only as good as the code to
> setup the workaround lists. Hmm, really that code shouldn't be using the
> wa_list built into the engine, but building them itself to double check
> against such failures or corruption.

Fwiw, I think we should first create a patch for the selftests to detect
the missing initialisation.
-Chris


More information about the Intel-gfx mailing list