Quoting Tvrtko Ursulin (2018-02-19 19:16:36)
>
> On 19/02/2018 18:35, Chris Wilson wrote:
> > Quoting Tvrtko Ursulin (2018-02-19 18:31:31)
> >>
> >> On 19/02/2018 14:01, Chris Wilson wrote:
> >>> If we fail to unbind the vma (due to a signal on an active buffer that
> >>> needs to be moved for the next execbuf), then we need to clear the
> >>> persistent tracking state we setup for this execbuf.
> >>>
> >>> Fixes: c7c6e46f913b ("drm/i915: Convert execbuf to use struct-of-array packing for critical fields")
> >>> Testcase: igt/gem_fenced_exec_thrash/no-spare-fences-busy*
> >>> Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> >>> Cc: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx>
> >>> Cc: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>
> >>> Cc: <stable@xxxxxxxxxxxxxxx> # v4.14+
> >>> ---
[snip]
> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx>
And grabbed because I want to see some improvement in the morning :)
Thanks for submitting yourself to the horrors of i915_gem_execbuf,
-Chris