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

Kamil Konieczny kamil.konieczny at linux.intel.com
Tue Mar 12 15:59:00 UTC 2024


Hi Katarzyna,
On 2024-03-12 at 14:13:59 +0000, Katarzyna Piecielska wrote:
> There is gem_workaround test that deserves it's own subcategory.
> 
> 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>
> Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> ---
>  tests/intel/gem_workarounds.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/intel/gem_workarounds.c b/tests/intel/gem_workarounds.c
> index d83c0a1db..43146f4b9 100644
> --- a/tests/intel/gem_workarounds.c
> +++ b/tests/intel/gem_workarounds.c
> @@ -36,8 +36,8 @@
>   * TEST: gem workarounds
>   * Category: Infrastructure
>   * Feature: workarounds
> - * Functionality: workarounds
> - * Sub-category: Context
> + * Functionality: context
> + * Sub-category: Workarounds

I did give you r-b but now that I know about names collision
it looks like Sub-category should be different from Feature,
what about:

 * Sub-category: GPU_Workarounds

Regards,
Kamil

>   * Test category: GEM_Legacy
>   * Run type: FULL
>   *
> -- 
> 2.34.1
> 


More information about the igt-dev mailing list