[Intel-gfx] [PATCH 6/6] drm/i915: Stop caching the "golden" renderstate

[Oscar Mateo]

On 11/02/2017 07:56 AM, Joonas Lahtinen wrote:
> On Thu, 2017-11-02 at 07:54 -0700, Rodrigo Vivi wrote:
>> On Thu, Nov 02, 2017 at 02:43:16PM +0000, Joonas Lahtinen wrote:
>>> On Thu, 2017-11-02 at 12:42 +0000, Chris Wilson wrote:
>>>> As we now record the default HW state and so only emit the "golden"
>>>> renderstate once to prepare the HW, there is no advantage in keeping the
>>>> renderstate batch around as it will never be used again.
>> So, with this in place we really don't need that null context for CNL.
>> to fullfill all Mesa needs, right?!
> Separate issue, this only fixes isolation. This patch just releases it
> from memory after it's been applied to the default context states to be
> stored.
>
> But yes, we also decided not to have null context for new platforms.

At last until, two years from now, we find out that there is a very 
subtle reason why we need it :)

> Regards, Joonas