[PATCH i-g-t 17/24] tests: Introduce Core subcategory

[Mauro Carvalho Chehab]

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

> Let's have Core subcategory instead of 'i915' or 'drm'.

LGTM.
Reviewed-by: Mauro Carvalho Chehab <mchehab at kernel.org>

> 
> 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>
> ---
>  tests/dmabuf_sync_file.c | 18 ++++++++++++++++++
>  tests/intel/i915_pciid.c |  2 +-
>  tests/vgem_basic.c       |  1 +
>  3 files changed, 20 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/dmabuf_sync_file.c b/tests/dmabuf_sync_file.c
> index cfa73117b..57f1ce2bc 100644
> --- a/tests/dmabuf_sync_file.c
> +++ b/tests/dmabuf_sync_file.c
> @@ -10,6 +10,24 @@
>  
>  IGT_TEST_DESCRIPTION("Tests for sync_file support in dma-buf");
>  
> +/**
> + * TEST: dmabuf_sync_file
> + * Category: Infrastructure
> + * Description: Tests for sync_file support in dma-buf
> + * Functionality: dmabuf
> + * Sub-category: Core
> + * Run type: FULL
> + *
> + * SUBTEST: export-basic
> + * SUBTEST: export-before-signal
> + * SUBTEST: export-multiwait
> + * SUBTEST: export-wait-after-attach
> + * SUBTEST: import-basic
> + * SUBTEST: import-multiple-read-only
> + * SUBTEST: import-multiple-read-write

We need to add descriptions for them later on.

> + *
> + */
> +
>  static void test_export_basic(int fd)
>  {
>  	struct vgem_bo bo;
> diff --git a/tests/intel/i915_pciid.c b/tests/intel/i915_pciid.c
> index 8322bd564..436acdaeb 100644
> --- a/tests/intel/i915_pciid.c
> +++ b/tests/intel/i915_pciid.c
> @@ -35,7 +35,7 @@
>   * Feature: core
>   * Functionality: device management
>   * Run type: BAT
> - * Sub-category: i915
> + * Sub-category: Core
>   * Test category: GEM_Legacy
>   *
>   * SUBTEST:
> diff --git a/tests/vgem_basic.c b/tests/vgem_basic.c
> index 6cfb02e82..fe201908c 100644
> --- a/tests/vgem_basic.c
> +++ b/tests/vgem_basic.c
> @@ -36,6 +36,7 @@
>   * Category: Infrastructure
>   * Description: Basic sanity check of Virtual GEM module (vGEM).
>   * Functionality: mock device
> + * Sub-category: Core
>   * Test category: GEM_Legacy
>   * Feature: vgem
>   * Run type: FULL