[PATCH i-g-t v2 02/16] tests: Add Memory management tests to General Core Mega feature

Kamil Konieczny kamil.konieczny at linux.intel.com
Thu Apr 25 17:19:32 UTC 2024


Hi Katarzyna,
On 2024-04-23 at 10:48:59 +0000, Katarzyna Piecielska wrote:
> Add Memory management tests to General Core mega feature
> Group:
> 	SUBTEST
> 	Description
> 	Category
> 	Mega feature
> 	Sub-category
> 	Functionality
> 	Feature

grep -E -i ':.*memory.management' n.*

n.Functionality:../tests/core_getstats.c: * Functionality: DRM memory management

n.Functionality:../tests/intel/prime_mmap_kms.c: * Functionality: memory management

n.Sub-category:../tests/dmabuf.c: * Sub-category: Memory management
^^^^
imho there is missing 'tests' in this sub-category, it should be:

n.Sub-category:../tests/dmabuf.c: * Sub-category: Memory management tests

Regards,
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>
> Cc: Aditya Chauhan <aditya.chauhan at intel.com>
> Cc: Girotra Himanshu <himanshu.girotra at intel.com>
> CC: Sandeep Kumar Parupalli <sandeep.kumar.parupalli at intel.com>
> ---
>  tests/dmabuf.c                               |  4 +-
>  tests/drm_mm.c                               |  5 +-
>  tests/intel/gem_close.c                      |  5 +-
>  tests/intel/gem_cs_tlb.c                     |  7 ++-
>  tests/intel/gem_gtt_cpu_tlb.c                |  8 ++-
>  tests/intel/gem_gtt_hog.c                    |  3 +-
>  tests/intel/gem_gtt_speed.c                  |  3 +-
>  tests/intel/gem_lmem_evict.c                 |  5 +-
>  tests/intel/gem_lmem_swapping.c              |  7 ++-
>  tests/intel/gem_madvise.c                    |  7 ++-
>  tests/intel/gem_mmap.c                       |  7 ++-
>  tests/intel/gem_mmap_gtt.c                   |  5 +-
>  tests/intel/gem_mmap_offset.c                |  5 +-
>  tests/intel/gem_mmap_wc.c                    |  5 +-
>  tests/intel/gem_ppgtt.c                      |  5 +-
>  tests/intel/gem_read_read_speed.c            |  3 +-
>  tests/intel/gem_readwrite.c                  |  5 +-
>  tests/intel/gem_set_tiling_vs_gtt.c          |  5 +-
>  tests/intel/gem_set_tiling_vs_pwrite.c       |  5 +-
>  tests/intel/gem_streaming_writes.c           |  3 +-
>  tests/intel/gem_tiled_blits.c                |  5 +-
>  tests/intel/gem_tiled_fence_blits.c          |  5 +-
>  tests/intel/gem_tiled_partial_pwrite_pread.c |  5 +-
>  tests/intel/gem_tiled_pread_basic.c          |  5 +-
>  tests/intel/gem_tiled_pread_pwrite.c         |  5 +-
>  tests/intel/gem_tiled_swapping.c             |  5 +-
>  tests/intel/gem_tiled_wb.c                   |  5 +-
>  tests/intel/gem_tiled_wc.c                   |  5 +-
>  tests/intel/gem_tiling_max_stride.c          |  5 +-
>  tests/intel/gem_unfence_active_buffers.c     |  5 +-
>  tests/intel/i915_fb_tiling.c                 |  5 +-
>  tests/intel/i915_selftest.c                  | 64 ++++++++++++--------
>  tests/intel/prime_busy.c                     |  7 ++-
>  tests/intel/prime_mmap.c                     |  7 ++-
>  tests/intel/prime_mmap_coherency.c           |  7 ++-
>  tests/intel/prime_self_import.c              |  7 ++-
>  tests/intel/xe_evict.c                       |  2 +
>  tests/intel/xe_mmap.c                        |  3 +-
>  tests/intel/xe_pat.c                         |  1 +
>  tests/intel/xe_vm.c                          |  1 +
>  40 files changed, 155 insertions(+), 101 deletions(-)
> 
> diff --git a/tests/dmabuf.c b/tests/dmabuf.c
> index 71285ff0b..f99f0e7fb 100644
> --- a/tests/dmabuf.c
> +++ b/tests/dmabuf.c
> @@ -26,11 +26,11 @@
>  /**
>   * TEST: dmabuf
>   * Description: Kernel selftests for the dmabuf API
> - * Feature: mapping, prime
>   * Category: Infrastructure
> - * Description: drm_mm range manager SW validation
> + * Mega feature: General Core features
>   * Functionality: drm_mm
>   * Sub-category: Memory management
> + * Feature: mapping, prime
>   * Test category: GEM_Legacy
>   *
>   * SUBTEST: all-tests
> diff --git a/tests/drm_mm.c b/tests/drm_mm.c
> index dcd4c7676..41b363cf0 100644
> --- a/tests/drm_mm.c
> +++ b/tests/drm_mm.c
> @@ -26,10 +26,11 @@
>  /**
>   * TEST: drm mm
>   * Description: Basic sanity check of DRM's range manager (struct drm_mm)
> - * Feature: mapping
>   * Category: Infrastructure
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: drm_mm
> - * Sub-category: Memory management
> + * Feature: mapping
>   * Test category: GEM_Legacy
>   *
>   * SUBTEST: drm_mm
> diff --git a/tests/intel/gem_close.c b/tests/intel/gem_close.c
> index b3c5cb108..33da151e1 100644
> --- a/tests/intel/gem_close.c
> +++ b/tests/intel/gem_close.c
> @@ -28,9 +28,10 @@
>  /**
>   * TEST: gem close
>   * Category: Infrastructure
> - * Feature: mapping
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: driver handler
> - * Sub-category: Memory management
> + * Feature: mapping
>   * Test category: GEM_Legacy
>   *
>   * SUBTEST: basic
> diff --git a/tests/intel/gem_cs_tlb.c b/tests/intel/gem_cs_tlb.c
> index f18a8fdaa..065d6b330 100644
> --- a/tests/intel/gem_cs_tlb.c
> +++ b/tests/intel/gem_cs_tlb.c
> @@ -54,11 +54,12 @@
>  #include "igt.h"
>  /**
>   * TEST: gem cs tlb
> - * Category: Infrastructure
>   * Description: Check whether we correctly invalidate the cs tlb.
> - * Feature: mapping
> + * Category: Infrastructure
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: tlb
> - * Sub-category: Memory management
> + * Feature: mapping
>   * Test category: GEM_Legacy
>   *
>   * SUBTEST: engines
> diff --git a/tests/intel/gem_gtt_cpu_tlb.c b/tests/intel/gem_gtt_cpu_tlb.c
> index c7a1cbe63..7fd876e5c 100644
> --- a/tests/intel/gem_gtt_cpu_tlb.c
> +++ b/tests/intel/gem_gtt_cpu_tlb.c
> @@ -46,10 +46,12 @@
>  /**
>   * TEST: gem gtt cpu tlb
>   * Description: Check whether gtt tlbs for cpu access are correctly invalidated.
> - * Feature: gtt, mapping
> - * Sub-category: Memory management
>   * Category: Desktop client
> - * Functionality: mmap
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
> + * Functionality: mapping
> + * Feature: gtt, mapping
> + *
>   *
>   * SUBTEST:
>   */
> diff --git a/tests/intel/gem_gtt_hog.c b/tests/intel/gem_gtt_hog.c
> index 12f291020..cfeea8ba1 100644
> --- a/tests/intel/gem_gtt_hog.c
> +++ b/tests/intel/gem_gtt_hog.c
> @@ -44,8 +44,9 @@
>  #include "igt.h"
>  /**
>   * TEST: gem gtt hog
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: gtt
>   *
>   * SUBTEST:
> diff --git a/tests/intel/gem_gtt_speed.c b/tests/intel/gem_gtt_speed.c
> index 46ebafdd8..14f0ffa88 100644
> --- a/tests/intel/gem_gtt_speed.c
> +++ b/tests/intel/gem_gtt_speed.c
> @@ -44,8 +44,9 @@
>  #include "i915/gem_create.h"
>  /**
>   * TEST: gem gtt speed
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: gtt
>   *
>   * SUBTEST:
> diff --git a/tests/intel/gem_lmem_evict.c b/tests/intel/gem_lmem_evict.c
> index f756565cb..2453f95b2 100644
> --- a/tests/intel/gem_lmem_evict.c
> +++ b/tests/intel/gem_lmem_evict.c
> @@ -10,10 +10,11 @@
>  /**
>   * TEST: gem lmem evict
>   * Description: Force tiny lmem size for easily testing eviction scenarios.
> - * Feature: local_memory
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: local memory eviction
> + * Feature: local_memory
>   * Test category: GEM_Legacy
>   *
>   * SUBTEST: dontneed-evict-race
> diff --git a/tests/intel/gem_lmem_swapping.c b/tests/intel/gem_lmem_swapping.c
> index a10a6e96e..3f9a8c90a 100644
> --- a/tests/intel/gem_lmem_swapping.c
> +++ b/tests/intel/gem_lmem_swapping.c
> @@ -26,11 +26,12 @@
>  #include "intel_mocs.h"
>  /**
>   * TEST: gem lmem swapping
> - * Category: Server
>   * Description: Exercise local memory swapping.
> - * Feature: local_memory
> + * Category: Server
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: buffer management
> - * Sub-category: Memory management
> + * Feature: local_memory
>   * Test category: GEM_Legacy
>   *
>   * SUBTEST: basic
> diff --git a/tests/intel/gem_madvise.c b/tests/intel/gem_madvise.c
> index da5ac7d6d..71139f27d 100644
> --- a/tests/intel/gem_madvise.c
> +++ b/tests/intel/gem_madvise.c
> @@ -40,12 +40,13 @@
>  #include "i915/gem_create.h"
>  /**
>   * TEST: gem madvise
> - * Category: Desktop client
>   * Description: Checks that the kernel reports EFAULT when trying to use purged bo.
> - * Feature: caching, mapping
> + * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: buffer management
> - * Sub-category: Memory management
>   * Test category: GEM_Legacy
> + * Feature: caching, mapping
>   *
>   * SUBTEST: dontneed-after-mmap
>   * Description: Check signal for Segmentation Fault and bus error after
> diff --git a/tests/intel/gem_mmap.c b/tests/intel/gem_mmap.c
> index 179001a20..5e8a787d1 100644
> --- a/tests/intel/gem_mmap.c
> +++ b/tests/intel/gem_mmap.c
> @@ -41,10 +41,11 @@
>  /**
>   * TEST: gem mmap
>   * Description: Basic MMAP IOCTL tests for memory regions.
> - * Feature: mapping
> - * Sub-category: Memory management
>   * Category: Desktop client
> - * Functionality: mmap
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
> + * Functionality: mapping
> + * Feature: mapping
>   *
>   * SUBTEST: bad-object
>   * Description: Verify mapping to invalid gem objects won't be created.
> diff --git a/tests/intel/gem_mmap_gtt.c b/tests/intel/gem_mmap_gtt.c
> index 7218ef9bc..79be68a22 100644
> --- a/tests/intel/gem_mmap_gtt.c
> +++ b/tests/intel/gem_mmap_gtt.c
> @@ -49,10 +49,11 @@
>  /**
>   * TEST: gem mmap gtt
>   * Description: Ensure that all operations around MMAP_GTT ioctl works.
> - * Feature: gtt
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: map gtt
> + * Feature: gtt
>   *
>   * SUBTEST: bad-object
>   * Description: Verify mapping to invalid gem objects fails.
> diff --git a/tests/intel/gem_mmap_offset.c b/tests/intel/gem_mmap_offset.c
> index 95d2158ca..90a4a0f1a 100644
> --- a/tests/intel/gem_mmap_offset.c
> +++ b/tests/intel/gem_mmap_offset.c
> @@ -41,9 +41,10 @@
>   * TEST: gem mmap offset
>   * Description: Basic MMAP_OFFSET IOCTL tests for mem regions
>   * Category: Infrastructure
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
> + * Functionality: mapping
>   * Feature: mapping
> - * Functionality: mmap
> - * Sub-category: Memory management
>   * Test category: GEM_Legacy
>   *
>   * SUBTEST: bad-extensions
> diff --git a/tests/intel/gem_mmap_wc.c b/tests/intel/gem_mmap_wc.c
> index 0a1bef7a2..beecd0b3a 100644
> --- a/tests/intel/gem_mmap_wc.c
> +++ b/tests/intel/gem_mmap_wc.c
> @@ -41,10 +41,11 @@
>  #include "i915/gem_create.h"
>  /**
>   * TEST: gem mmap wc
> - * Feature: caching, mapping
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: cache
> + * Feature: caching, mapping
>   *
>   * SUBTEST: bad-object
>   * SUBTEST: bad-offset
> diff --git a/tests/intel/gem_ppgtt.c b/tests/intel/gem_ppgtt.c
> index e6a4651da..0dee7ad1a 100644
> --- a/tests/intel/gem_ppgtt.c
> +++ b/tests/intel/gem_ppgtt.c
> @@ -42,10 +42,11 @@
>  /**
>   * TEST: gem ppgtt
>   * Category: Desktop client
> - * Feature: mapping, xorg_dri2
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: buffer management
> - * Sub-category: Memory management
>   * Test category: GEM_Legacy
> + * Feature: mapping, xorg_dri2
>   *
>   * SUBTEST: blt-vs-render-ctx0
>   *
> diff --git a/tests/intel/gem_read_read_speed.c b/tests/intel/gem_read_read_speed.c
> index 47c3dfecd..f6834f74c 100644
> --- a/tests/intel/gem_read_read_speed.c
> +++ b/tests/intel/gem_read_read_speed.c
> @@ -45,8 +45,9 @@
>  /**
>   * TEST: gem read read speed
>   * Description: Test speed of concurrent reads between engines.
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: read/write memory
>   *
>   * SUBTEST: read-read-1024x1024
> diff --git a/tests/intel/gem_readwrite.c b/tests/intel/gem_readwrite.c
> index 9970c5f33..b5c6e7a26 100644
> --- a/tests/intel/gem_readwrite.c
> +++ b/tests/intel/gem_readwrite.c
> @@ -40,10 +40,11 @@
>  #include "i915/gem_create.h"
>  /**
>   * TEST: gem readwrite
> - * Feature: mapping
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: read/write memory
> + * Feature: mapping
>   *
>   * SUBTEST: beyond-EOB
>   *
> diff --git a/tests/intel/gem_set_tiling_vs_gtt.c b/tests/intel/gem_set_tiling_vs_gtt.c
> index 3305a2297..621dc6fbf 100644
> --- a/tests/intel/gem_set_tiling_vs_gtt.c
> +++ b/tests/intel/gem_set_tiling_vs_gtt.c
> @@ -41,10 +41,11 @@
>  /**
>   * TEST: gem set tiling vs gtt
>   * Description: Check set_tiling vs gtt mmap coherency.
> - * Feature: gtt
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: tiling vs gtt coherency
> + * Feature: gtt
>   *
>   * SUBTEST:
>   */
> diff --git a/tests/intel/gem_set_tiling_vs_pwrite.c b/tests/intel/gem_set_tiling_vs_pwrite.c
> index 241e72ab8..10bfcb71f 100644
> --- a/tests/intel/gem_set_tiling_vs_pwrite.c
> +++ b/tests/intel/gem_set_tiling_vs_pwrite.c
> @@ -41,10 +41,11 @@
>  /**
>   * TEST: gem set tiling vs pwrite
>   * Description: Check set_tiling vs pwrite coherency.
> - * Feature: gtt
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: tiling vs pwrite coherency
> + * Feature: gtt
>   *
>   * SUBTEST:
>   */
> diff --git a/tests/intel/gem_streaming_writes.c b/tests/intel/gem_streaming_writes.c
> index d707c703a..d80ac6fe1 100644
> --- a/tests/intel/gem_streaming_writes.c
> +++ b/tests/intel/gem_streaming_writes.c
> @@ -43,8 +43,9 @@
>  /**
>   * TEST: gem streaming writes
>   * Description: Test of streaming writes into active GPU sources
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: barrier
>   *
>   * SUBTEST: batch-cpu
> diff --git a/tests/intel/gem_tiled_blits.c b/tests/intel/gem_tiled_blits.c
> index 73adadef4..f3845d213 100644
> --- a/tests/intel/gem_tiled_blits.c
> +++ b/tests/intel/gem_tiled_blits.c
> @@ -57,10 +57,11 @@
>  /**
>   * TEST: gem tiled blits
>   * Description: Test doing many tiled blits, with a working set larger than the aperture size.
> - * Feature: gtt, mapping
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: tiled blits
> + * Feature: gtt, mapping
>   *
>   * SUBTEST: basic
>   * Description: Check basic functionality.
> diff --git a/tests/intel/gem_tiled_fence_blits.c b/tests/intel/gem_tiled_fence_blits.c
> index faebe0efb..bf33b8d5f 100644
> --- a/tests/intel/gem_tiled_fence_blits.c
> +++ b/tests/intel/gem_tiled_fence_blits.c
> @@ -50,10 +50,11 @@
>   * Description:
>   *   Tests fence management problems related to tiled blits by performing many blits on tiled
>   *   buffer objects with fences enabled and with working set larger than the aperture size.
> - * Feature: gtt, mapping
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: tiled blits
> + * Feature: gtt, mapping
>   *
>   * SUBTEST: basic
>   * Description: Check basic functionality.
> diff --git a/tests/intel/gem_tiled_partial_pwrite_pread.c b/tests/intel/gem_tiled_partial_pwrite_pread.c
> index 9c24f1901..551fb4306 100644
> --- a/tests/intel/gem_tiled_partial_pwrite_pread.c
> +++ b/tests/intel/gem_tiled_partial_pwrite_pread.c
> @@ -41,10 +41,11 @@
>  /**
>   * TEST: gem tiled partial pwrite pread
>   * Description: Test pwrite/pread consistency when touching partial cachelines.
> - * Feature: gtt, mapping
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: tiled pread/pwrite
> + * Feature: gtt, mapping
>   *
>   * SUBTEST: reads
>   *
> diff --git a/tests/intel/gem_tiled_pread_basic.c b/tests/intel/gem_tiled_pread_basic.c
> index 913800880..f26db3592 100644
> --- a/tests/intel/gem_tiled_pread_basic.c
> +++ b/tests/intel/gem_tiled_pread_basic.c
> @@ -51,10 +51,11 @@
>  /**
>   * TEST: gem tiled pread basic
>   * Description: Test pread behavior on tiled objects with respect to the reported swizzling value.
> - * Feature: gtt, mapping
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: tiled pread/pwrite
> + * Feature: gtt, mapping
>   *
>   * SUBTEST:
>   */
> diff --git a/tests/intel/gem_tiled_pread_pwrite.c b/tests/intel/gem_tiled_pread_pwrite.c
> index 7ec6a9291..b65e0458a 100644
> --- a/tests/intel/gem_tiled_pread_pwrite.c
> +++ b/tests/intel/gem_tiled_pread_pwrite.c
> @@ -61,10 +61,11 @@
>  /**
>   * TEST: gem tiled pread pwrite
>   * Description: Test swizzling by testing pwrite does the inverse of pread.
> - * Feature: gtt, mapping
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: tiled pread/pwrite
> + * Feature: gtt, mapping
>   *
>   * SUBTEST:
>   */
> diff --git a/tests/intel/gem_tiled_swapping.c b/tests/intel/gem_tiled_swapping.c
> index 1b5c4bd14..af8976a25 100644
> --- a/tests/intel/gem_tiled_swapping.c
> +++ b/tests/intel/gem_tiled_swapping.c
> @@ -61,10 +61,11 @@
>  /**
>   * TEST: gem tiled swapping
>   * Description: Exercise swizzle code for swapping.
> - * Feature: gtt, mapping
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: swapping
> + * Feature: gtt, mapping
>   *
>   * SUBTEST: non-threaded
>   *
> diff --git a/tests/intel/gem_tiled_wb.c b/tests/intel/gem_tiled_wb.c
> index 53c1ea217..6a61fcace 100644
> --- a/tests/intel/gem_tiled_wb.c
> +++ b/tests/intel/gem_tiled_wb.c
> @@ -50,10 +50,11 @@
>   * Description:
>   *   This is a test of write-combining mmap's behavior on tiled objects with respect to the reported
>   *   swizzling value.
> - * Feature: caching, gtt
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: cache
> + * Feature: caching, gtt
>   *
>   * SUBTEST:
>   */
> diff --git a/tests/intel/gem_tiled_wc.c b/tests/intel/gem_tiled_wc.c
> index 7b8e88427..ca10ab02d 100644
> --- a/tests/intel/gem_tiled_wc.c
> +++ b/tests/intel/gem_tiled_wc.c
> @@ -47,10 +47,11 @@
>  #include "i915/gem_create.h"
>  /**
>   * TEST: gem tiled wc
> - * Feature: caching, gtt
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: cache
> + * Feature: caching, gtt
>   *
>   * SUBTEST:
>   */
> diff --git a/tests/intel/gem_tiling_max_stride.c b/tests/intel/gem_tiling_max_stride.c
> index 5877b887c..4021f3e7a 100644
> --- a/tests/intel/gem_tiling_max_stride.c
> +++ b/tests/intel/gem_tiling_max_stride.c
> @@ -42,10 +42,11 @@
>  /**
>   * TEST: gem tiling max stride
>   * Description: Check that max fence stride works.
> - * Feature: gtt, mapping
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: fences
> + * Feature: gtt, mapping
>   *
>   * SUBTEST:
>   */
> diff --git a/tests/intel/gem_unfence_active_buffers.c b/tests/intel/gem_unfence_active_buffers.c
> index cde4771c6..14ed1167a 100644
> --- a/tests/intel/gem_unfence_active_buffers.c
> +++ b/tests/intel/gem_unfence_active_buffers.c
> @@ -55,10 +55,11 @@
>  /**
>   * TEST: gem unfence active buffers
>   * Description: Check for use-after-free in the fence stealing code.
> - * Feature: synchronization feature
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: fences
> + * Feature: synchronization
>   *
>   * SUBTEST:
>   */
> diff --git a/tests/intel/i915_fb_tiling.c b/tests/intel/i915_fb_tiling.c
> index 1e42555b2..eab1d661d 100644
> --- a/tests/intel/i915_fb_tiling.c
> +++ b/tests/intel/i915_fb_tiling.c
> @@ -26,10 +26,11 @@
>  /**
>   * TEST: i915 fb tiling
>   * Description: Object tiling must be fixed after framebuffer creation.
> - * Feature: mapping
> - * Sub-category: Memory management
>   * Category: Desktop client
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: tiling
> + * Feature: mapping
>   *
>   * SUBTEST:
>   */
> diff --git a/tests/intel/i915_selftest.c b/tests/intel/i915_selftest.c
> index ce43042c8..5d17f3bf1 100644
> --- a/tests/intel/i915_selftest.c
> +++ b/tests/intel/i915_selftest.c
> @@ -54,11 +54,12 @@
>   * Sub-category: uapi
>   *
>   * SUBTEST: live at coherency
> - * Category: Selftest
>   * Description: Cache management
> + * Category: Selftest
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: cache
>   * Test category: i915 / HW
> - * Sub-category: Memory management
>   *
>   * SUBTEST: live at debugger
>   * Category: Selftest
> @@ -74,15 +75,17 @@
>   *
>   * SUBTEST: live at dmabuf
>   * Category: Selftest
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: buffer management
>   * Test category: i915
> - * Sub-category: Memory management
>   *
>   * SUBTEST: live at evict
>   * Category: Selftest
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: GTT eviction
>   * Test category: i915
> - * Sub-category: Memory management
>   *
>   * SUBTEST: live at execlists
>   * Description: command submission backend
> @@ -140,11 +143,12 @@
>   * Test category: i915
>   *
>   * SUBTEST: live at gt_gtt
> - * Category: Selftest
>   * Description: Validation of virtual address management and execution
> + * Category: Selftest
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: gtt
>   * Test category: HW
> - * Sub-category: Memory management
>   *
>   * SUBTEST: live at gt_heartbeat
>   * Category: Selftest
> @@ -183,16 +187,18 @@
>   *
>   * SUBTEST: live at gt_tlb
>   * Category: Selftest
> - * Test category: Memory Management
> - * Sub-category: Memory management
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: tlb
> + * Test category: Memory Management
>   *
>   * SUBTEST: live at gtt
> - * Category: Selftest
>   * Description: Virtual address management interface validation
> + * Category: Selftest
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: virtual address
>   * Test category: i915
> - * Sub-category: Memory management
>   *
>   * SUBTEST: live at gtt_l4wa
>   * Category: Selftest
> @@ -237,11 +243,12 @@
>   * Sub-category: Reset
>   *
>   * SUBTEST: live at hugepages
> - * Category: Selftest
>   * Description: Large page support validation
> + * Category: Selftest
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: large page
>   * Test category: i915
> - * Sub-category: Memory management
>   *
>   * SUBTEST: live at late_gt_pm
>   * Category: Selftest
> @@ -251,32 +258,36 @@
>   * Sub-category: Power management
>   *
>   * SUBTEST: live at lmem
> + * Description: Basic i915 driver module selftests
>   * Category: Selftest
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Feature: local_memory
> - * Description: Basic i915 driver module selftests
>   * Functionality: local memory
> - * Sub-category: Memory management
>   *
>   * SUBTEST: live at memory_region
> - * Category: Selftest
>   * Description: memory topology validation and migration checks
> + * Category: Selftest
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: memory topology
>   * Test category: i915 / HW
> - * Sub-category: Memory management
>   *
>   * SUBTEST: live at memory_region_cross_tile
>   * Category: Selftest
>   * Mega feature: General Core features
>   * Description: Multi-tile memory topology validation
> - * Functionality: memory topology
> + * Category: Selftest
>   * Sub-category: MultiTile
> + * Functionality: memory topology
>   *
>   * SUBTEST: live at mman
> - * Category: Selftest
>   * Description: memory management validation
> + * Category: Selftest
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: mapping
>   * Test category: i915
> - * Sub-category: Memory management
>   *
>   * SUBTEST: live at obj_lock
>   * Category: Selftest
> @@ -357,11 +368,12 @@
>   * Sub-category: Power management
>   *
>   * SUBTEST: live at vma
> - * Category: Selftest
>   * Description: Per-object virtual address management
> + * Category: Selftest
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: virtual address
>   * Test category: i915
> - * Sub-category: Memory management
>   *
>   * SUBTEST: live at win_blt_copy
>   * Category: Selftest
> @@ -384,11 +396,12 @@
>   * Feature: gem_core
>   *
>   * SUBTEST: mock at buddy
> - * Category: Selftest
>   * Description: Buddy allocation
> + * Category: Selftest
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: buddy allocation
>   * Test category: DRM
> - * Sub-category: Memory management
>   *
>   * SUBTEST: mock at contexts
>   * Category: Selftest
> @@ -511,9 +524,10 @@
>   *
>   * SUBTEST: mock at tlb
>   * Category: Selftest
> - * Test category: Memory Management
> - * Sub-category: Memory management
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: tlb
> + * Test category: Memory Management
>   *
>   * SUBTEST: mock at uncore
>   * Category: Selftest
> diff --git a/tests/intel/prime_busy.c b/tests/intel/prime_busy.c
> index 66af5f8a3..a7c3f3499 100644
> --- a/tests/intel/prime_busy.c
> +++ b/tests/intel/prime_busy.c
> @@ -28,12 +28,13 @@
>  #include "igt.h"
>  /**
>   * TEST: prime busy
> - * Category: Infrastructure
>   * Description: Basic check of polling for prime fences.
> - * Feature: prime
> + * Category: Infrastructure
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: buffer management
> - * Sub-category: Memory management
>   * Test category: GEM_Legacy
> + * Feature: prime
>   *
>   * SUBTEST: after
>   *
> diff --git a/tests/intel/prime_mmap.c b/tests/intel/prime_mmap.c
> index 3c0a0c0e5..c874bbd1f 100644
> --- a/tests/intel/prime_mmap.c
> +++ b/tests/intel/prime_mmap.c
> @@ -53,10 +53,11 @@
>  /**
>   * TEST: prime mmap
>   * Category: Infrastructure
> - * Feature: prime
> - * Functionality: prime mmap
> - * Sub-category: Memory management
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
> + * Functionality: prime
>   * Test category: GEM_Legacy
> + * Feature: prime
>   *
>   * SUBTEST: test_aperture_limit
>   *
> diff --git a/tests/intel/prime_mmap_coherency.c b/tests/intel/prime_mmap_coherency.c
> index 3db79156d..73449d8bb 100644
> --- a/tests/intel/prime_mmap_coherency.c
> +++ b/tests/intel/prime_mmap_coherency.c
> @@ -33,13 +33,14 @@
>  #include "igt.h"
>  /**
>   * TEST: prime mmap coherency
> - * Category: Infrastructure
>   * Description:
>   *   Test dma-buf mmap on !llc platforms mostly and provoke coherency bugs so we know for sure where
>   *   we need the sync ioctls.
> + * Category: Infrastructure
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
> + * Functionality: prime
>   * Feature: prime
> - * Functionality: prime mmap
> - * Sub-category: Memory management
>   * Test category: GEM_Legacy
>   *
>   * SUBTEST: ioctl-errors
> diff --git a/tests/intel/prime_self_import.c b/tests/intel/prime_self_import.c
> index 7bf947cf4..7603c1763 100644
> --- a/tests/intel/prime_self_import.c
> +++ b/tests/intel/prime_self_import.c
> @@ -47,12 +47,13 @@
>  #include "i915/gem_create.h"
>  /**
>   * TEST: prime self import
> - * Category: Infrastructure
>   * Description: Check whether prime import/export works on the same device... but with different fds.
> - * Feature: prime
> + * Category: Infrastructure
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: buffer management
> - * Sub-category: Memory management
>   * Test category: GEM_Legacy
> + * Feature: prime
>   *
>   * SUBTEST: basic-llseek-bad
>   * Description: Check dmabuf llseek support with invalid values.
> diff --git a/tests/intel/xe_evict.c b/tests/intel/xe_evict.c
> index 5994ebb14..d5f6cea32 100644
> --- a/tests/intel/xe_evict.c
> +++ b/tests/intel/xe_evict.c
> @@ -6,6 +6,7 @@
>  /**
>   * TEST: Check VMA eviction
>   * Category: Software building block
> + * Mega feature: General Core features
>   * Sub-category: VMA
>   * Functionality: eviction
>   * GPU requirements: GPU needs to have dedicated VRAM
> @@ -486,6 +487,7 @@ static uint64_t calc_bo_size(uint64_t vram_size, int mul, int div)
>   * @beng-large-multi-vm:	large multi VM bind exec_queue
>   */
>  /**
> + *
>   * SUBTEST: evict-%s
>   * Description:  %arg[1] evict test.
>   * Feature: compute machine
> diff --git a/tests/intel/xe_mmap.c b/tests/intel/xe_mmap.c
> index dcd36a50d..353d157da 100644
> --- a/tests/intel/xe_mmap.c
> +++ b/tests/intel/xe_mmap.c
> @@ -6,7 +6,8 @@
>  /**
>   * TEST: Test if the driver is capable of doing mmap on different memory regions
>   * Category: Software building block
> - * Sub-category: Memory management
> + * Mega feature: General Core features
> + * Sub-category: Memory management tests
>   * Functionality: mmap
>   */
>  
> diff --git a/tests/intel/xe_pat.c b/tests/intel/xe_pat.c
> index d20100d7e..a48cd084b 100644
> --- a/tests/intel/xe_pat.c
> +++ b/tests/intel/xe_pat.c
> @@ -6,6 +6,7 @@
>  /**
>   * TEST: Test for selecting per-VMA pat_index
>   * Category: Software building block
> + * Mega feature: General Core features
>   * Sub-category: VMA
>   * Functionality: pat_index
>   */
> diff --git a/tests/intel/xe_vm.c b/tests/intel/xe_vm.c
> index 13cca2752..695a7882c 100644
> --- a/tests/intel/xe_vm.c
> +++ b/tests/intel/xe_vm.c
> @@ -6,6 +6,7 @@
>  /**
>   * TEST: Check if VMA functionality is working
>   * Category: Software building block
> + * Mega feature: General Core features
>   * Sub-category: VMA
>   */
>  
> -- 
> 2.34.1
> 


More information about the igt-dev mailing list