[Intel-gfx] [PATCH v3 1/6] drm/i915/gen8: Add infrastructure to initialize WA batch buffers

Chris Wilson chris at chris-wilson.co.uk
Fri Jun 5 04:36:51 PDT 2015


On Fri, Jun 05, 2015 at 12:24:58PM +0100, Siluvery, Arun wrote:
> ok, it is possible to do the allocation and setup in
> logical_ring_init() itself. I wanted to group it with other wa which
> are setup in init_context().

Phew, I had worried I had missed something. The issue the current split
between ring init and default context init raises in my mind is that the
content has context dependencies upon it - whereas the w/a so far can be
reused globally. So imo the current split is more confusing than just
creating the w/a buffer entirely during ring init.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list