[igt-dev] [PATCH i-g-t v2] i915/gem_vm_create: Added test description for test case

[Lukasz Fiedorowicz]

<priyanka.dandamudi at intel.com> writes:

> From: Priyanka Dandamudi <priyanka.dandamudi at intel.com>
>
> Added test description for test and to all the available subtests.
>
> v2:Modified 5th subtest description.
>
> Cc: Melkaveri, Arjun <Arjun.Melkaveri at intel.com>
> Signed-off-by: Priyanka Dandamudi <priyanka.dandamudi at intel.com>

LGTM.
Reviewed-by: Lukasz Fiedorowicz <lukasz.fiedorowicz at intel.com>

> ---
>  tests/i915/gem_vm_create.c | 22 +++++++++++++++++-----
>  1 file changed, 17 insertions(+), 5 deletions(-)
>
> diff --git a/tests/i915/gem_vm_create.c b/tests/i915/gem_vm_create.c
> index e8af68f1..e9172291 100644
> --- a/tests/i915/gem_vm_create.c
> +++ b/tests/i915/gem_vm_create.c
> @@ -26,6 +26,11 @@
>  #include "igt.h"
>  #include "igt_dummyload.h"
>  
> +IGT_TEST_DESCRIPTION("Test the basic vm creation ioctl,"
> +		 "where it checks if invalid combinations are rejected and "
> +		 "checks isolation that the same vm_id on different fd are indeed "
> +		 "different vm.");
> +
>  static int vm_create_ioctl(int i915, struct drm_i915_gem_vm_control *ctl)
>  {
>  	int err = 0;
> @@ -395,10 +400,10 @@ igt_main
>  		igt_require_gem(i915);
>  		igt_require(has_vm(i915));
>  	}
> -
> +	igt_describe("Checks if vm creation ioctl returns error for invalid flags and extensions.");
>  	igt_subtest("invalid-create")
>  		invalid_create(i915);
> -
> +	igt_describe("Checks if vm destroy ioctl returns error for invalid vmid's, flags and extensions.");
>  	igt_subtest("invalid-destroy")
>  		invalid_destroy(i915);
>  
> @@ -406,16 +411,23 @@ igt_main
>  		igt_fixture {
>  			gem_context_require_param(i915, I915_CONTEXT_PARAM_VM);
>  		}
> -
> +		igt_describe("Checks if offset for same context can be different for different vm's.");
>  		igt_subtest("execbuf")
>  			execbuf(i915);
>  
> +		igt_describe("Checks if same vmid in different fd are different vm, also " 
> +			     "proves that it cannot be written to vm from another fd.");
>  		igt_subtest("isolation")
>  			isolation(i915);
> -
> +		
> +		igt_describe("Test creates a vm and is allowed to share between two contexts on the same fd "
> +			     "and checks if offset set by one context of an already active vma can be modified"
> +			     " by another context or not."); 
>  		igt_subtest("create-ext")
>  			create_ext(i915);
> -
> +		
> +		igt_describe("Test tries to access same vm and context for "
> +			     "two different spin batches when vm has been destroyed asynchronously.");
>  		igt_subtest("async-destroy")
>  			async_destroy(i915);
>  	}
> -- 
> 2.26.2
>

-- 
Thanks,
Lukasz