[igt-dev] [PATCH i-g-t] tests/i915/gem_linear_blits: Added description to the subtest

[Zbigniew Kempczyński]

On Tue, Feb 15, 2022 at 10:42:36AM +0530, sai.gowtham.ch at intel.com wrote:
> From: Ch Sai Gowtham <sai.gowtham.ch at intel.com>
> 
> Added description to subtests.
> 
> Cc: Melkaveri Arjun <arjun.melkaveri at intel.com>
> Signed-off-by: Ch Sai Gowtham <sai.gowtham.ch at intel.com>
> ---
>  tests/i915/gem_linear_blits.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/tests/i915/gem_linear_blits.c b/tests/i915/gem_linear_blits.c
> index 6504c9f2..b570aa76 100644
> --- a/tests/i915/gem_linear_blits.c
> +++ b/tests/i915/gem_linear_blits.c
> @@ -258,9 +258,13 @@ igt_main
>  		count = (count + ncpus - 1) / ncpus;
>  	}
>  
> +	igt_describe("Excercise 2 buffer objects check if driver will"
                      ^ - exercise

> +			"rearrange the buffers");
>  	igt_subtest("basic")
>  		run_test(fd, 2, do_relocs);
>  
> +	igt_describe("The intent is to push beyond the GTT size to force the driver"
> +			"to have to rearrange the buffers");

Rearrange is vague, I would use "rebind" instead. Or "rearrange (rebind)", 
both are fine for me. Pick one.

--
Zbigniew


>  	igt_subtest("normal") {
>  		intel_allocator_multiprocess_start();
>  		igt_fork(child, ncpus)
> @@ -269,6 +273,8 @@ igt_main
>  		intel_allocator_multiprocess_stop();
>  	}
>  
> +	igt_describe("Pushes beyond the GTT size checks if driver will rearrange the"
> +			"buffers, with interrupts in between the parent process");
>  	igt_subtest("interruptible") {
>  		intel_allocator_multiprocess_start();
>  		igt_fork_signal_helper();
> -- 
> 2.35.0
>