Re: [PATCH i-g-t] tests/gem_exec_capture: adjust the pi subtest

["Das, Nirmoy" <nirmoy.das@xxxxxxxxxxxxxxx>]

Reviewed-by: Nirmoy Das <nirmoy.das@xxxxxxxxx>

On 7/5/2022 12:49 PM, Matthew Auld wrote:
> This was broken by the kernel change: 71b1669ea9bd ("drm/i915/uapi:
> tweak error capture on recoverable contexts"). Update the test to use
> non-recoverable context if needed.
>
> Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/6350
> Signed-off-by: Matthew Auld <matthew.auld@xxxxxxxxx>
> Cc: Nirmoy Das <nirmoy.das@xxxxxxxxxxxxxxx>
> ---
>   tests/i915/gem_exec_capture.c | 9 +++++++++
>   1 file changed, 9 insertions(+)
>
> diff --git a/tests/i915/gem_exec_capture.c b/tests/i915/gem_exec_capture.c
> index c2639082..a25f529b 100644
> --- a/tests/i915/gem_exec_capture.c
> +++ b/tests/i915/gem_exec_capture.c
> @@ -721,6 +721,15 @@ static void prioinv(int fd, int dir, const intel_ctx_t *ctx,
>   
>   		ctx2 = intel_ctx_create_all_physical(fd);
>   		igt_assert(ctx2);
> +		if (needs_recoverable_ctx(fd)) {
> +			struct drm_i915_gem_context_param param = {
> +				.ctx_id = ctx2->id,
> +				.param = I915_CONTEXT_PARAM_RECOVERABLE,
> +				.value = 0,
> +			};
> +
> +			gem_context_set_param(fd, &param);
> +		}
>   
>   		intel_allocator_init();
>   		/* Reopen the allocator in the new process. */