[igt-dev] [PATCH i-g-t 40/77] tests/i915/gem_ctx_persistence: Drop the clone subtest

[Zbigniew Kempczyński]

On Mon, Jun 14, 2021 at 11:36:55AM -0500, Jason Ekstrand wrote:
> Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
> ---
>  tests/i915/gem_ctx_persistence.c | 30 ------------------------------
>  1 file changed, 30 deletions(-)
> 
> diff --git a/tests/i915/gem_ctx_persistence.c b/tests/i915/gem_ctx_persistence.c
> index 10d057f1b..91c301769 100644
> --- a/tests/i915/gem_ctx_persistence.c
> +++ b/tests/i915/gem_ctx_persistence.c
> @@ -147,33 +147,6 @@ static void test_idempotent(int i915)
>  	igt_assert_eq(p.value, expected);
>  }
>  
> -static void test_clone(int i915)
> -{
> -	struct drm_i915_gem_context_param p = {
> -		.param = I915_CONTEXT_PARAM_PERSISTENCE,
> -	};
> -	uint32_t ctx, clone;
> -
> -	/*
> -	 * Check that persistence is inherited across a clone.
> -	 */
> -	igt_require( __gem_context_create(i915, &ctx) == 0);
> -
> -	p.ctx_id = ctx;
> -	p.value = 0;
> -	gem_context_set_param(i915, &p);
> -
> -	clone = gem_context_clone(i915, ctx, I915_CONTEXT_CLONE_FLAGS, 0);
> -	gem_context_destroy(i915, ctx);
> -
> -	p.ctx_id = clone;
> -	p.value = -1;
> -	gem_context_get_param(i915, &p);
> -	igt_assert_eq(p.value, 0);
> -
> -	gem_context_destroy(i915, clone);
> -}
> -
>  static void test_persistence(int i915, unsigned int engine)
>  {
>  	igt_spin_t *spin;
> @@ -1366,9 +1339,6 @@ igt_main
>  	igt_subtest("idempotent")
>  		test_idempotent(i915);
>  
> -	igt_subtest("clone")
> -		test_clone(i915);
> -
>  	igt_subtest("file")
>  		test_nonpersistent_file(i915);

Ok, this is for upcoming removing context cloning.

Reviewed-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>

--
Zbigniew

>  
> -- 
> 2.31.1
> 
> _______________________________________________
> igt-dev mailing list
> igt-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/igt-dev