[PATCH i-g-t v4 05/25] tests/intel: Let's introduce Workarounds subcategory

Kamil Konieczny kamil.konieczny at linux.intel.com
Tue Mar 19 17:09:21 UTC 2024


Hi Katarzyna,
On 2024-03-14 at 14:34:35 +0000, Katarzyna Piecielska wrote:
> There is gem_workaround test that deserves it's own subcategory.
> 
> v2: Make Sub-category/Functionality/Feature unique values (Kamil)
> 
> Signed-off-by: Katarzyna Piecielska <katarzyna.piecielska at intel.com>
> Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> Cc: Mauro Carvalho Chehab <mauro.chehab at linux.intel.com>

There are few 'workarounds' left:

957:intel/gem_workarounds.c: * Feature: hibernate, workarounds
958:intel/gem_workarounds.c: * Feature: hibernate, workarounds
959:intel/gem_workarounds.c: * Feature: hibernate, workarounds

Also what about using 'gpu_workaround'?

Regards,
Kamil

> ---
>  tests/intel/gem_workarounds.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/tests/intel/gem_workarounds.c b/tests/intel/gem_workarounds.c
> index d83c0a1db..ddb9bddbe 100644
> --- a/tests/intel/gem_workarounds.c
> +++ b/tests/intel/gem_workarounds.c
> @@ -35,9 +35,9 @@
>  /**
>   * TEST: gem workarounds
>   * Category: Infrastructure
> - * Feature: workarounds
> - * Functionality: workarounds
> - * Sub-category: Context
> + * Feature: workaround
> + * Functionality: context
> + * Sub-category: Workarounds
>   * Test category: GEM_Legacy
>   * Run type: FULL
>   *
> @@ -59,12 +59,12 @@
>   *
>   * SUBTEST: suspend-resume-context
>   * Description: Test to verify gem WA registers during suspend-resume
> - * Feature: suspend, workarounds
> + * Feature: suspend, workaround
>   * Test category: suspend
>   *
>   * SUBTEST: suspend-resume-fd
>   * Description: Test to verify gem WA registers during suspend-resume
> - * Feature: suspend, workarounds
> + * Feature: suspend, workaround
>   * Test category: suspend
>   */
>  
> -- 
> 2.34.1
> 


More information about the igt-dev mailing list