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

Siluvery, Arun arun.siluvery at linux.intel.com
Fri Jun 5 04:56:46 PDT 2015


On 05/06/2015 12:36, Chris Wilson wrote:
> 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
>
I am moving it to logical_render_ring_init() as these are only 
applicable for RCS.

regards
Arun



More information about the Intel-gfx mailing list