Re: [igt-dev] [PATCH i-g-t] tests/i915/gem_lmem_swapping: Add parallel-random-verify-ccs subtest

[Ramalingam C <ramalingam.c@intel.com>]

On 2022-04-21 at 08:50:08 +0200, Zbigniew Kempczyński wrote:
> For this subtest we need to use allocator thread so parallel subtests
> need to be executed in allocator multi-process aware environment.
> 
> Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
Looks good to me. And tested too.

Reviewed-by: Ramalingam C <ramalingam.c@intel.com>

> Cc: Ramalingam C <ramalingam.c@intel.com>
> ---
>  tests/i915/gem_lmem_swapping.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/tests/i915/gem_lmem_swapping.c b/tests/i915/gem_lmem_swapping.c
> index e58a1f505f..5d93e9daae 100644
> --- a/tests/i915/gem_lmem_swapping.c
> +++ b/tests/i915/gem_lmem_swapping.c
> @@ -576,6 +576,7 @@ static void test_evict(int i915,
>  	if (flags & TEST_PARALLEL) {
>  		int fd = gem_reopen_driver(i915);
>  
> +		intel_allocator_multiprocess_start();
>  		ctx = intel_ctx_create_all_physical(fd);
>  		__gem_context_set_persistence(fd, ctx->id, false);
>  
> @@ -586,6 +587,7 @@ static void test_evict(int i915,
>  		igt_waitchildren();
>  		intel_ctx_destroy(fd, ctx);
>  		close(fd);
> +		intel_allocator_multiprocess_stop();
>  	} else {
>  		__do_evict(i915, ctx, &region->region, &params, params.seed);
>  	}
> @@ -747,6 +749,7 @@ igt_main_args("", long_options, help_str, opt_handler, NULL)
>  		{ "verify-random-ccs", TEST_CCS | TEST_RANDOM },
>  		{ "heavy-verify-random-ccs", TEST_CCS | TEST_RANDOM | TEST_HEAVY },
>  		{ "heavy-verify-multi-ccs", TEST_CCS | TEST_RANDOM | TEST_HEAVY | TEST_ENGINES | TEST_MULTI },
> +		{ "parallel-random-verify-ccs", TEST_PARALLEL | TEST_RANDOM | TEST_CCS },
>  		{ }
>  	};
>  	const intel_ctx_t *ctx;
> -- 
> 2.32.0
>