[igt-dev] [PATCH i-g-t, v3] i915/gem_ctx_create: Added test description for test case

[Kamil Konieczny]

Hi Apoorva,

Dnia 2022-02-15 at 18:33:00 +0530, apoorva1.singh at intel.com napisał(a):

> From: Apoorva Singh <apoorva1.singh at intel.com>
> 
> Added test and subtests description
> 
> v2: Modified description to be more precise
> 
> v3: Rebase
> 
> signed-off-by: apoorva singh <apoorva1.singh at intel.com>
> cc: melkaveri, arjun <arjun.melkaveri at intel.com>
> cc: dec, katarzyna <katarzyna.dec at intel.com>
> cc: petri latvala <petri.latvala at intel.com>
> Acked-by: Katarzyna Dec <katarzyna.dec at intel.com>
> ---
>  tests/i915/gem_ctx_create.c | 29 +++++++++++++++++++++++++++++
>  1 file changed, 29 insertions(+)
> 
> diff --git a/tests/i915/gem_ctx_create.c b/tests/i915/gem_ctx_create.c
> index 44846652..67219e9c 100644
> --- a/tests/i915/gem_ctx_create.c
> +++ b/tests/i915/gem_ctx_create.c
> @@ -36,6 +36,8 @@
>  #include "igt_rand.h"
>  #include "sw_sync.h"
>  
> +IGT_TEST_DESCRIPTION("Test the context create ioctls");
> +
>  #define ENGINE_FLAGS  (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
>  
>  static unsigned all_engines[I915_EXEC_RING_MASK + 1];
> @@ -585,6 +587,7 @@ igt_main
>  		igt_fork_hang_detector(fd);
>  	}
>  
> +	igt_describe("Test random context creation");
>  	igt_subtest("basic") {
>  		memset(&create, 0, sizeof(create));
>  		create.ctx_id = rand();
> @@ -594,41 +597,67 @@ igt_main
>  		gem_context_destroy(fd, create.ctx_id);
>  	}
>  
> +	igt_describe("Verify valid and invalid context extensions");
>  	igt_subtest("ext-param")
>  		basic_ext_param(fd);
> +
> +	igt_describe("Set, validate and execute particular context params");
>  	igt_subtest("iris-pipeline")
>  		iris_pipeline(fd);
>  
> +	igt_describe("Create contexts upto available RAM size, check the average "
s/check/calc/

Here test do not check the time, it measure a time, so replace
"check" with "calc",

or instead you can use "measure" or "calculate".
The same substitution is to be made below.

> +	       "performance of their execution on multiple parallel threads");
s/threads/processes/

>  	igt_subtest("maximum-mem")
>  		maximum(fd, &cfg, ncpus, CHECK_RAM);
> +
> +	igt_describe("Create contexts upto available RAM+SWAP size, check the average "
s/check/calc/

> +	       "performance of their execution on multiple parallel threads");
s/threads/processes/

>  	igt_subtest("maximum-swap")
>  		maximum(fd, &cfg, ncpus, CHECK_RAM | CHECK_SWAP);
>  
> +	igt_describe("Exercise implicit per-fd context creation");
>  	igt_subtest("basic-files")
>  		files(fd, &cfg, 2, 1);
> +
> +	igt_describe("Exercise implicit per-fd context creation on 1 CPU for long duration");
>  	igt_subtest("files")
>  		files(fd, &cfg, 20, 1);
> +
> +	igt_describe("Exercise implicit per-fd context creation on all CPUs for long duration");
>  	igt_subtest("forked-files")
>  		files(fd, &cfg, 20, ncpus);
>  
> +	igt_describe("Check the average performance of context creation and "
s/Check/Calc/

> +	       "it's execution using all engines");
>  	/* NULL value means all engines */
>  	igt_subtest("active-all")
>  		active(fd, &cfg, NULL, 20, 1);
> +
> +	igt_describe("Check the average performance of context creation and it's execution "
s/Check/Calc/

> +	       "using all engines on multiple parallel threads");
>  	igt_subtest("forked-active-all")
>  		active(fd, &cfg, NULL, 20, ncpus);
>  
> +	igt_describe("For each engine check the average performance of context creation "
s/check/calc/

> +	       "and execution");
>  	igt_subtest_with_dynamic("active") {
>  		for_each_ctx_cfg_engine(fd, &cfg, e) {
>  			igt_dynamic_f("%s", e->name)
>  				active(fd, &cfg, e, 20, 1);
>  		}
>  	}
> +
> +	igt_describe("For each engine check the average performance of context creation "
s/check/calc/

> +	       "and execution on multiple parallel threads");
>  	igt_subtest_with_dynamic("forked-active") {
>  		for_each_ctx_cfg_engine(fd, &cfg, e) {
>  			igt_dynamic_f("%s", e->name)
>  				active(fd, &cfg, e, 20, ncpus);
>  		}
>  	}
> +
> +	igt_describe("For each engine check the average performance of context creation "
s/check/calc/

> +	       "and execution while all other engines are hogging the resources");
>  	igt_subtest_with_dynamic("hog") {
>  		for_each_ctx_cfg_engine(fd, &cfg, e) {
>  			igt_dynamic_f("%s", e->name)
> -- 
> 2.25.1

With that changes you can add my r-b,

Regards,
Kamil Konieczny