[PATCH i-g-t 06/24] tests/intel: Introduce Context subcategory

[Piecielska, Katarzyna]

You are right 😊 I will keep context as functionality and move those tests to CMD submission.
There are other 'Context' tests that need to be change - I will do that as well.

Kasia

-----Original Message-----
From: Mauro Carvalho Chehab <mauro.chehab at linux.intel.com> 
Sent: Monday, March 4, 2024 4:07 PM
To: Piecielska, Katarzyna <katarzyna.piecielska at intel.com>
Cc: igt-dev at lists.freedesktop.org; Chris Wilson <chris at chris-wilson.co.uk>; Kamil Konieczny <kamil.konieczny at linux.intel.com>; Dandamudi, Priyanka <priyanka.dandamudi at intel.com>; Ch, Sai Gowtham <sai.gowtham.ch at intel.com>
Subject: Re: [PATCH i-g-t 06/24] tests/intel: Introduce Context subcategory

On Mon,  4 Mar 2024 11:15:59 +0000
Katarzyna Piecielska <katarzyna.piecielska at intel.com> wrote:

> Let's introduce Context subcategory.



> 
> Signed-off-by: Katarzyna Piecielska <katarzyna.piecielska at intel.com>
> Cc: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> Cc: Priyanka Dandamudi <priyanka.dandamudi at intel.com>
> Cc: Mauro Carvalho Chehab <mauro.chehab at linux.intel.com>
> Cc: Sai Gowtham Ch <sai.gowtham.ch at intel.com>
> ---
>  tests/intel/gem_ctx_bad_destroy.c | 4 ++--
>  tests/intel/gem_ctx_engines.c     | 4 ++--
>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/tests/intel/gem_ctx_bad_destroy.c 
> b/tests/intel/gem_ctx_bad_destroy.c
> index 65f6b5583..b6575e2d8 100644
> --- a/tests/intel/gem_ctx_bad_destroy.c
> +++ b/tests/intel/gem_ctx_bad_destroy.c
> @@ -39,9 +39,9 @@
>   * Category: Infrastructure
>   * Description: Negative test cases for destroy contexts.
>   * Feature: context
> - * Functionality: context management
> + * Functionality: context
>   * Run type: FULL
> - * Sub-category: i915
> + * Sub-category: Context

As explained on patch 3, having two buckets with the same name is confusing.

>   * Test category: GEM_Legacy
>   *
>   * SUBTEST: double-destroy
> diff --git a/tests/intel/gem_ctx_engines.c 
> b/tests/intel/gem_ctx_engines.c index 61718494e..75770e1f1 100644
> --- a/tests/intel/gem_ctx_engines.c
> +++ b/tests/intel/gem_ctx_engines.c
> @@ -47,9 +47,9 @@
>   * TEST: gem ctx engines
>   * Category: Infrastructure
>   * Feature: context
> - * Functionality: context management
> + * Functionality: context
>   * Run type: FULL
> - * Sub-category: i915
> + * Sub-category: Context
>   * Test category: GEM_Legacy
>   *
>   * SUBTEST: execute-allforone