[igt-dev] [PATCH i-g-t] i915/gem_render_linear_blits: Add test description

[Katarzyna Dec]

On Thu, Nov 12, 2020 at 10:19:20AM +0530, sai.gowtham.ch at intel.com wrote:
> From: sai gowtham <sai.gowtham.ch at intel.com>
> 
> Added more test description to the test and basic subtest.
> 
> Cc: Melkaveri Arjun <arjun.melkaveri at intel.com>
> Signed-off-by: sai gowtham <sai.gowtham.ch at intel.com>
> ---
>  tests/i915/gem_render_linear_blits.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/tests/i915/gem_render_linear_blits.c b/tests/i915/gem_render_linear_blits.c
> index 8e646127..0cbd205c 100644
> --- a/tests/i915/gem_render_linear_blits.c
> +++ b/tests/i915/gem_render_linear_blits.c
> @@ -55,6 +55,9 @@
>  #define HEIGHT 512
>  #define SIZE (HEIGHT*STRIDE)
>  
> +IGT_TEST_DESCRIPTION("This is a test of doing many blits, with a working set"
> +		" larger than the aperture size. The goal is to simply ensure"
> +		" the basics work.");
Leave a blank line between.
>  static uint32_t linear[WIDTH*HEIGHT];
>  static igt_render_copyfunc_t render_copy;
>  
> @@ -170,6 +173,8 @@ igt_main
>  		igt_require_gem(fd);
>  	}
>  
> +	igt_describe("This is a basic test for render blit where it"
> +			" does many blits, with a working set larger than the aperture size.");
More specific:
igt_describe("Copy 2 buffers forward, backward and random.");
>  	igt_subtest("basic") {
>  		run_test(fd, 2);
>  	}

Maybe you could add descriptions to other cases too?
These cases are similar just more exahustive, using maximum memory.

Kasia :)

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