Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes:
> As we flush each test and wait for idle before the next, also switch
> back to the kernel context. This helps limit the amount of collateral
> damage a test may cause by resetting to the default state each time (and
> also helps clean up temporaries used by the test).
>
> Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> Cc: Mika Kuoppala <mika.kuoppala@xxxxxxxxxxxxxxx>
> ---
> drivers/gpu/drm/i915/selftests/igt_flush_test.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/selftests/igt_flush_test.c b/drivers/gpu/drm/i915/selftests/igt_flush_test.c
> index abff2f04ea84..7f35bddc2e95 100644
> --- a/drivers/gpu/drm/i915/selftests/igt_flush_test.c
> +++ b/drivers/gpu/drm/i915/selftests/igt_flush_test.c
> @@ -57,6 +57,11 @@ int igt_flush_test(struct drm_i915_private *i915, unsigned int flags)
>
> cond_resched();
>
> + if (i915_gem_switch_to_kernel_context(i915)) {
> + pr_err("Failed to switch back to kernel context; declaring wedged\n");
> + i915_gem_set_wedged(i915);
You don't want to give the error code? It comes from request_alloc.
Also if the test already wedged itself we would wedge again.
-Mika
> + }
> +
> wedge_on_timeout(&w, i915, HZ)
> i915_gem_wait_for_idle(i915, flags);
>
> --
> 2.17.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
