[PATCH i-g-t 5/5] tests: Remove collisions with Subcategory and Functionality

Kamil Konieczny kamil.konieczny at linux.intel.com
Wed Apr 10 15:31:54 UTC 2024


Hi Katarzyna,
On 2024-04-09 at 11:19:15 +0000, Katarzyna Piecielska wrote:
> Let's remove Sub-category/Functionality name collisions:
> - changes in Functionality:
> 	s/DRM/drm_mm/
> 	s/dmabuf/dmabuf test/
> 	s/compute/compute test/
> 	s/synchronization/fences/

There is one left:
tests/intel/kms_fence_pin_leak.c: * Functionality: kms_core, synchronization

Regards,
Kamil

> 	s/vma/virtual address/
> 	s/kunit/kunit test/
> 	s/mocs/mocs configuration/
> 	s/intel-nouveau/intel-nouveau uapi/
> 
> Signed-off-by: Katarzyna Piecielska <katarzyna.piecielska at intel.com>
> Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> ---
>  tests/dmabuf.c                       | 2 +-
>  tests/dmabuf_sync_file.c             | 2 +-
>  tests/drm_mm.c                       | 2 +-
>  tests/drm_read.c                     | 2 +-
>  tests/intel/gem_exec_endless.c       | 4 ++--
>  tests/intel/gem_fence_thrash.c       | 2 +-
>  tests/intel/gem_fence_upload.c       | 2 +-
>  tests/intel/gem_fenced_exec_thrash.c | 6 +++---
>  tests/intel/i915_selftest.c          | 4 ++--
>  tests/intel/xe_exec_compute_mode.c   | 2 +-
>  tests/intel/xe_live_ktest.c          | 6 +++---
>  tests/prime_nv_api.c                 | 2 +-
>  12 files changed, 18 insertions(+), 18 deletions(-)
> 
> diff --git a/tests/dmabuf.c b/tests/dmabuf.c
> index 549cfa27e..71285ff0b 100644
> --- a/tests/dmabuf.c
> +++ b/tests/dmabuf.c
> @@ -29,7 +29,7 @@
>   * Feature: mapping, prime
>   * Category: Infrastructure
>   * Description: drm_mm range manager SW validation
> - * Functionality: DRM
> + * Functionality: drm_mm
>   * Sub-category: Memory management
>   * Test category: GEM_Legacy
>   *
> diff --git a/tests/dmabuf_sync_file.c b/tests/dmabuf_sync_file.c
> index f4eb2e63f..402816afc 100644
> --- a/tests/dmabuf_sync_file.c
> +++ b/tests/dmabuf_sync_file.c
> @@ -14,7 +14,7 @@ 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
> + * Functionality: dmabuf test
>   * Sub-category: Core
>   *
>   * SUBTEST: export-basic
> diff --git a/tests/drm_mm.c b/tests/drm_mm.c
> index b8b72635b..dcd4c7676 100644
> --- a/tests/drm_mm.c
> +++ b/tests/drm_mm.c
> @@ -28,7 +28,7 @@
>   * Description: Basic sanity check of DRM's range manager (struct drm_mm)
>   * Feature: mapping
>   * Category: Infrastructure
> - * Functionality: DRM
> + * Functionality: drm_mm
>   * Sub-category: Memory management
>   * Test category: GEM_Legacy
>   *
> diff --git a/tests/drm_read.c b/tests/drm_read.c
> index a976a59a2..94c53fded 100644
> --- a/tests/drm_read.c
> +++ b/tests/drm_read.c
> @@ -54,7 +54,7 @@ IGT_TEST_DESCRIPTION("Call read(drm) and see if it behaves.");
>   * TEST: drm read
>   * Description: boundary testing of read(drm_fd)
>   * Category: Infrastructure
> - * Functionality: DRM memory managemnt
> + * Functionality: drm_mm
>   * Sub-category: uapi
>   *
>   * SUBTEST: invalid-buffer
> diff --git a/tests/intel/gem_exec_endless.c b/tests/intel/gem_exec_endless.c
> index 5bec7adee..f0bf82b54 100644
> --- a/tests/intel/gem_exec_endless.c
> +++ b/tests/intel/gem_exec_endless.c
> @@ -33,10 +33,10 @@
>  /**
>   * TEST: gem exec endless
>   * Category: Server
> - * Feature: cmd_submission
> - * Functionality: compute
>   * Sub-category: CMD submission
> + * Functionality: compute test
>   * Test category: Compute
> + * Feature: cmd_submission
>   *
>   * SUBTEST: dispatch
>   */
> diff --git a/tests/intel/gem_fence_thrash.c b/tests/intel/gem_fence_thrash.c
> index 0df7319fc..aac2a667b 100644
> --- a/tests/intel/gem_fence_thrash.c
> +++ b/tests/intel/gem_fence_thrash.c
> @@ -48,7 +48,7 @@
>   * Feature: gtt, synchronization feature
>   * Sub-category: CMD submission
>   * Category: Desktop client
> - * Functionality: synchronization
> + * Functionality: fences
>   *
>   * SUBTEST: bo-copy
>   *
> diff --git a/tests/intel/gem_fence_upload.c b/tests/intel/gem_fence_upload.c
> index b80d53a25..8b92d3976 100644
> --- a/tests/intel/gem_fence_upload.c
> +++ b/tests/intel/gem_fence_upload.c
> @@ -45,7 +45,7 @@
>   * TEST: gem fence upload
>   * Sub-category: CMD submission
>   * Category: Desktop client
> - * Functionality: synchronization
> + * Functionality: fences
>   *
>   * SUBTEST: performance
>   *
> diff --git a/tests/intel/gem_fenced_exec_thrash.c b/tests/intel/gem_fenced_exec_thrash.c
> index d0055dfb5..614967615 100644
> --- a/tests/intel/gem_fenced_exec_thrash.c
> +++ b/tests/intel/gem_fenced_exec_thrash.c
> @@ -36,10 +36,10 @@
>  /**
>   * TEST: gem fenced exec thrash
>   * Description: Test execbuf fence accounting.
> - * Feature: gtt, synchronization feature
> - * Sub-category: CMD submission
>   * Category: Desktop client
> - * Functionality: synchronization
> + * Sub-category: CMD submission
> + * Functionality: fences
> + * Feature: gtt, synchronization feature
>   *
>   * SUBTEST: 2-spare-fences
>   *
> diff --git a/tests/intel/i915_selftest.c b/tests/intel/i915_selftest.c
> index 1d03fb388..21420f133 100644
> --- a/tests/intel/i915_selftest.c
> +++ b/tests/intel/i915_selftest.c
> @@ -349,7 +349,7 @@
>   * SUBTEST: live at vma
>   * Category: Selftest
>   * Description: Per-object virtual address management
> - * Functionality: vma
> + * Functionality: virtual address
>   * Test category: i915
>   * Sub-category: Memory management
>   *
> @@ -516,7 +516,7 @@
>   * SUBTEST: mock at vma
>   * Category: Selftest
>   * Description: API checks for virtual address management
> - * Functionality: vma
> + * Functionality: virtual address
>   * Test category: i915
>   * Sub-category: uapi
>   *
> diff --git a/tests/intel/xe_exec_compute_mode.c b/tests/intel/xe_exec_compute_mode.c
> index 3ec848164..a73a7343c 100644
> --- a/tests/intel/xe_exec_compute_mode.c
> +++ b/tests/intel/xe_exec_compute_mode.c
> @@ -7,7 +7,7 @@
>   * TEST: Basic tests for execbuf compute machine functionality
>   * Category: Hardware building block
>   * Sub-category: CMD submission
> - * Functionality: compute
> + * Functionality: compute test
>   */
>  
>  #include <fcntl.h>
> diff --git a/tests/intel/xe_live_ktest.c b/tests/intel/xe_live_ktest.c
> index c50aefea8..a894c6257 100644
> --- a/tests/intel/xe_live_ktest.c
> +++ b/tests/intel/xe_live_ktest.c
> @@ -6,7 +6,7 @@
>   * Description: Xe driver live dmabuf unit tests
>   * Category: Software building block
>   * Sub-category: kunit
> - * Functionality: kunit
> + * Functionality: kunit test
>   * Test category: functionality test
>   *
>   * SUBTEST: xe_bo
> @@ -17,7 +17,7 @@
>   *
>   * SUBTEST: xe_dma_buf
>   * Description: Kernel dynamic selftests for dmabuf functionality.
> - * Functionality: dmabuf
> + * Functionality: dmabuf test
>   *
>   * SUBTEST: xe_migrate
>   * Description:
> @@ -28,7 +28,7 @@
>   * SUBTEST: xe_mocs
>   * Description:
>   *	Kernel dynamic selftests to check mocs configuration.
> - * Functionality: mocs
> + * Functionality: mocs configuration
>   */
>  
>  static const char *live_tests[] = {
> diff --git a/tests/prime_nv_api.c b/tests/prime_nv_api.c
> index 29cc9a8a7..d9c1849cf 100644
> --- a/tests/prime_nv_api.c
> +++ b/tests/prime_nv_api.c
> @@ -12,7 +12,7 @@
>   * TEST: prime_nv_api
>   * Description: Basic api tests between intel and nouveau
>   * Sub-category: uapi
> - * Functionality: intel-nouveau
> + * Functionality: intel-nouveau uapi
>   *
>   * SUBTEST: i915_nv_import_twice
>   * SUBTEST: i915_nv_import_twice_check_flink_name
> -- 
> 2.34.1
> 


More information about the igt-dev mailing list