Re: [PATCH] drm/i915: Update ring space correctly on lrc context reset

Arun Siluvery
2015-09-02T11:21:00Z

On 20/08/2015 16:27, Chris Wilson wrote:
> On Thu, Aug 20, 2015 at 05:34:59PM +0300, Mika Kuoppala wrote:
>> If we leave the last_retired_head to pre-reset value, we might
>> end up in a situation where intel_ring_space() returns wrong
>> value on next hardware init.
>
> http://patchwork.freedesktop.org/patch/46612/
> and earlier
> -Chris
>
Hi Chris,

I see the warning even with below batch,

[Intel-gfx] [PATCH 50/70] drm/i915: The argument for postfix is redundant
http://patchwork.freedesktop.org/patch/46601/

the following patch need to be updated as it uses olr,
[Intel-gfx] [PATCH 51/70] drm/i915: Record the position of the start of
the request
http://patchwork.freedesktop.org/patch/46612/

Do we need some of the previous patches in series as well?

This patch is fixing the issue in the current code, do you think we can
get this in its current state?

regards
Arun