[PATCH i-g-t v2 1/5] tests: Documentation refactoring part 1

Katarzyna Piecielska katarzyna.piecielska at intel.com
Fri Mar 1 09:08:40 UTC 2024


Unify documentation for Intel i915 tests that we have by moving
common parts to be available for all tests.

v2: rebase on master, add 3 more files with refactoring
(tests/intel/api_intel_allocator.c, tests/intel/api_intel_bb.c,
tests/intel/gem_busy.c)

Signed-off-by: Katarzyna Piecielska <katarzyna.piecielska at intel.com>
---
 tests/core_hotunplug.c            |  57 ++++-------
 tests/device_reset.c              |  11 ++-
 tests/dmabuf.c                    |  13 +--
 tests/drm_mm.c                    |  84 +----------------
 tests/intel/api_intel_allocator.c |  46 ++-------
 tests/intel/api_intel_bb.c        | 152 ++----------------------------
 tests/intel/debugfs_test.c        |  32 ++-----
 tests/intel/gem_basic.c           |  22 +----
 tests/intel/gem_blits.c           |   2 +-
 tests/intel/gem_busy.c            |  42 +++------
 tests/intel/gem_ccs.c             |   9 +-
 tests/intel/gem_close_race.c      |  39 +++-----
 tests/intel/gem_create.c          |  87 +++++------------
 13 files changed, 115 insertions(+), 481 deletions(-)

diff --git a/tests/core_hotunplug.c b/tests/core_hotunplug.c
index be80d479f..f578afa4a 100644
--- a/tests/core_hotunplug.c
+++ b/tests/core_hotunplug.c
@@ -44,64 +44,43 @@
  * Description: Examine behavior of a driver on device hot unplug
  * Feature: SMI, core
  * Test category: functionality test
- *
- * SUBTEST: hotrebind
  * Category: Software building block
  * Sub-category: driver
  * Functionality: device hotplug
- * Description:
- *   Check if the driver can be cleanly rebound to a device with a still open hot unbound driver
- *   instance
  * Run type: FULL
  *
+ * SUBTEST: hotrebind
+ * Description: Check if the driver can be cleanly rebound to a device with a
+ *		still open hot unbound driver instance
+ *
  * SUBTEST: hotrebind-lateclose
- * Category: Software building block
- * Sub-category: driver
- * Functionality: device hotplug
- * Description: Check if a hot unbound driver instance still open after hot rebind can be cleanly released
- * Run type: FULL
+ * Description: Check if a hot unbound driver instance still open after hot
+ *		rebind can be cleanly released
  *
  * SUBTEST: hotreplug
- * Category: Software building block
- * Sub-category: driver
- * Functionality: device hotplug
- * Description: Check if a hot unplugged and still open device can be cleanly restored
- * Run type: FULL
+ * Description: Check if a hot unplugged and still open device can be cleanly
+ *		restored
  *
  * SUBTEST: hotreplug-lateclose
- * Category: Software building block
- * Sub-category: driver
- * Functionality: device hotplug
- * Description: Check if an instance of a still open while hot replugged device can be cleanly released
- * Run type: FULL
+ * Description: Check if an instance of a still open while hot replugged device
+ *		can be cleanly released
  *
  * SUBTEST: hotunbind-rebind
- * Category: Software building block
- * Sub-category: driver
- * Functionality: device hotplug
- * Description: Check if the driver can be cleanly unbound from an open device, then released and rebound
- * Run type: FULL
+ * Description: Check if the driver can be cleanly unbound from an open device,
+ *		then released and rebound
  *
  * SUBTEST: hotunplug-rescan
- * Category: Software building block
- * Sub-category: driver
- * Functionality: device hotplug
- * Description: Check if an open device can be cleanly unplugged, then released and restored
- * Run type: FULL
+ * Description: Check if an open device can be cleanly unplugged, then released
+ *		and restored
  *
  * SUBTEST: unbind-rebind
- * Category: Software building block
- * Sub-category: driver
- * Functionality: device hotplug
- * Description: Check if the driver can be cleanly unbound from a device believed to be closed, then rebound
+ * Description: Check if the driver can be cleanly unbound from a device
+ *		believed to be closed, then rebound
  * Run type: BAT
  *
  * SUBTEST: unplug-rescan
- * Category: Software building block
- * Sub-category: driver
- * Functionality: device hotplug
- * Description: Check if a device believed to be closed can be cleanly unplugged, then restored
- * Run type: FULL
+ * Description: Check if a device believed to be closed can be cleanly
+ *		unplugged, then restored
  */
 
 IGT_TEST_DESCRIPTION("Examine behavior of a driver on device hot unplug");
diff --git a/tests/device_reset.c b/tests/device_reset.c
index 13bd5ab5f..817cbc285 100644
--- a/tests/device_reset.c
+++ b/tests/device_reset.c
@@ -18,29 +18,30 @@
  * TEST: device reset
  * Category: Hardware building block
  * Description: Examine behavior of a driver on device sysfs reset
+ * Category: Server
+ * Feature: reset
+ * Functionality: reset
+ * Sub-category: Reset
+ * Test category: GEM_Legacy
  * Run type: FULL
  *
  * SUBTEST: cold-reset-bound
  * Description: Cold Resets device with bound driver
- * Feature: reset
  *
  * SUBTEST: reset-bound
  * Description: Resets device with bound driver
- * Feature: reset
  *
  * SUBTEST: unbind-cold-reset-rebind
  * Description: Unbinds driver from device, initiates cold reset then rebinds driver to device
- * Feature: reset
  *
  * SUBTEST: unbind-reset-rebind
  * Description:
  *   Unbinds driver from device, initiates reset then rebinds driver to device
  *   validating device resets
  * Category: Infrastructure
- * Sub-category: sysman
+ * Sub-category: SysMan
  * Functionality: reset
  * Feature: reset, sriov-reset
- * Test category: GEM_Legacy
  */
 
 IGT_TEST_DESCRIPTION("Examine behavior of a driver on device sysfs reset");
diff --git a/tests/dmabuf.c b/tests/dmabuf.c
index 2ac105423..977b04043 100644
--- a/tests/dmabuf.c
+++ b/tests/dmabuf.c
@@ -27,21 +27,18 @@
  * TEST: dmabuf
  * Description: Kernel selftests for the dmabuf API
  * Feature: mapping, prime
+ * Category: Infrastructure
+ * Description: drm_mm range manager SW validation
+ * Functionality: DRM
+ * Sub-category: Memory management
+ * Test category: GEM_Legacy
  * Run type: BAT
  *
  * SUBTEST: all-tests
  *
  * SUBTEST: all-tests at dma_fence
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: all-tests at sanitycheck
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  */
 
 IGT_TEST_DESCRIPTION("Kernel selftests for the dmabuf API");
diff --git a/tests/drm_mm.c b/tests/drm_mm.c
index e6ba22474..f2e6c6cd1 100644
--- a/tests/drm_mm.c
+++ b/tests/drm_mm.c
@@ -27,129 +27,53 @@
  * TEST: drm mm
  * Description: Basic sanity check of DRM's range manager (struct drm_mm)
  * Feature: mapping
+ * Category: Infrastructure
+ * Functionality: DRM memory mangemnt
+ * Sub-category: Memory management
+ * Test category: GEM_Legacy
  * Run type: FULL
  *
  * SUBTEST: drm_mm
  *
  * SUBTEST: drm_mm at align
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: drm_mm at align32
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: drm_mm at align64
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: drm_mm at bottomup
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: drm_mm at color
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: drm_mm at color_evict
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: drm_mm at color_evict_range
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: drm_mm at debug
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: drm_mm at evict
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: drm_mm at evict_range
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: drm_mm at frag
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: drm_mm at highest
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: drm_mm at init
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: drm_mm at insert
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: drm_mm at insert_range
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: drm_mm at lowest
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: drm_mm at replace
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: drm_mm at reserve
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: drm_mm at sanitycheck
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  *
  * SUBTEST: drm_mm at topdown
- * Category: Infrastructure
- * Description: drm_mm range manager SW validation
- * Functionality: DRM memory mangemnt
- * Test category: GEM_Legacy
  */
 
 IGT_TEST_DESCRIPTION("Basic sanity check of DRM's range manager (struct drm_mm)");
diff --git a/tests/intel/api_intel_allocator.c b/tests/intel/api_intel_allocator.c
index b2e6770fe..f260250d4 100644
--- a/tests/intel/api_intel_allocator.c
+++ b/tests/intel/api_intel_allocator.c
@@ -16,79 +16,49 @@
 /**
  * TEST: api intel allocator
  * Category: Infrastructure
- * Functionality: virtual address ranges
+ * Functionality: virtual address
+ * Description: checking the virtual address ranges
  * Run type: FULL
  * Sub-category: IGT Lib
  * Test category: GEM_Legacy
+ * Feature: igt_core
  *
  * SUBTEST: alloc-simple
- * Description: checking the virtual address ranges
- * Feature: igt_core
  *
  * SUBTEST: default-alignment
- * Description:
- *   For simple allocator check does default alignment is properly handled in open and alloc
- *   functions
- *   checking the virtual address ranges
- * Feature: igt_core
+ * Description: For simple allocator check does default alignment is properly
+ *		handled in open and alloc functions checking the virtual
+ *		address ranges
  *
  * SUBTEST: execbuf-with-allocator
- * Description: checking the virtual address ranges
- * Feature: igt_core
  *
  * SUBTEST: fork-simple-once
- * Description: checking the virtual address ranges
- * Feature: igt_core
  *
  * SUBTEST: fork-simple-stress
- * Description: checking the virtual address ranges
- * Feature: igt_core
  *
  * SUBTEST: fork-simple-stress-signal
- * Description: checking the virtual address ranges
- * Feature: igt_core
  *
  * SUBTEST: gem-pool
- * Description:
- *   Verifies creating and executing bb from gem pool
- *   checking the virtual address ranges
- * Feature: igt_core
+ * Description: Verifies creating and executing bb from gem pool checking
+ *		the virtual address ranges
  *
  * SUBTEST: open-vm
- * Description: checking the virtual address ranges
- * Feature: igt_core
  *
  * SUBTEST: reloc-allocator
- * Description: checking the virtual address ranges
- * Feature: igt_core
  *
  * SUBTEST: reopen
- * Description: checking the virtual address ranges
- * Feature: igt_core
  *
  * SUBTEST: reopen-fork
- * Description: checking the virtual address ranges
- * Feature: igt_core
  *
  * SUBTEST: reserve-simple
- * Description: checking the virtual address ranges
- * Feature: igt_core
  *
  * SUBTEST: simple-allocator
- * Description: checking the virtual address ranges
- * Feature: igt_core
  *
  * SUBTEST: standalone
- * Description: checking the virtual address ranges
- * Feature: igt_core
  *
  * SUBTEST: two-level-inception
- * Description: checking the virtual address ranges
- * Feature: igt_core
  *
  * SUBTEST: two-level-inception-interruptible
- * Description: checking the virtual address ranges
- * Feature: igt_core
  */
 
 #define OBJ_SIZE 1024
diff --git a/tests/intel/api_intel_bb.c b/tests/intel/api_intel_bb.c
index 8a210f2ab..74e334581 100644
--- a/tests/intel/api_intel_bb.c
+++ b/tests/intel/api_intel_bb.c
@@ -43,210 +43,72 @@
 #include "intel_blt.h"
 /**
  * TEST: api intel bb
- * Description: intel_bb API check.
- * Run type: FULL
- *
- * SUBTEST: add-remove-objects
  * Category: Infrastructure
  * Description: checking dummy buffer creation & submission
  * Feature: igt_core
  * Functionality: dummy buffer creation & submission
  * Sub-category: IGT Lib
  * Test category: GEM_Legacy
+ * Run type: FULL
+ *
+ * SUBTEST: add-remove-objects
  *
  * SUBTEST: bb-with-allocator
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  *
  * SUBTEST: blit-noreloc-keep-cache
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  *
  * SUBTEST: blit-noreloc-purge-cache
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  *
  * SUBTEST: blit-reloc-keep-cache
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  *
  * SUBTEST: blit-reloc-purge-cache
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  *
  * SUBTEST: crc32
  * Description: Compare cpu and gpu crc32 sums on input object
- * Feature: igt_core
- * Test category: GEM_Legacy
  *
  * SUBTEST: delta-check
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  *
  * SUBTEST: destroy-bb
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  *
  * SUBTEST: full-batch
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  *
  * SUBTEST: intel-bb-blit-none
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  *
  * SUBTEST: intel-bb-blit-x
- * Feature: igt_core
  *
  * SUBTEST: intel-bb-blit-y
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  *
  * SUBTEST: lot-of-buffers
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  *
  * SUBTEST: misplaced-blitter
- * Category: Infrastructure
- * Description:
- *   Execute intel_bb with set of engines provided by userspace
- *   checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
+ * Description: Execute intel_bb with set of engines provided by userspace
+ *		checking dummy buffer creation & submission
  *
  * SUBTEST: object-noreloc-keep-cache-simple
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  *
  * SUBTEST: object-noreloc-purge-cache-simple
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  *
  * SUBTEST: object-reloc-keep-cache
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  *
  * SUBTEST: object-reloc-purge-cache
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  *
  * SUBTEST: offset-control
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  *
  * SUBTEST: purge-bb
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  *
  * SUBTEST: render
- * Feature: igt_core
  *
  * SUBTEST: render-ccs
- * Feature: igt_core
  *
  * SUBTEST: reset-bb
- * Category: Infrastructure
- * Description:
- *   Ensure reset is possible on fresh bb
- *   checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
+ * Description: Ensure reset is possible on fresh bb checking dummy buffer
+ *		creation & submission
  *
  * SUBTEST: reset-flags
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  *
  * SUBTEST: simple-bb
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  *
  * SUBTEST: simple-bb-ctx
- * Category: Infrastructure
- * Description: checking dummy buffer creation & submission
- * Feature: igt_core
- * Functionality: dummy buffer creation & submission
- * Sub-category: IGT Lib
- * Test category: GEM_Legacy
  */
 
 #define PAGE_SIZE 4096
diff --git a/tests/intel/debugfs_test.c b/tests/intel/debugfs_test.c
index b67bfabac..9706b81e1 100644
--- a/tests/intel/debugfs_test.c
+++ b/tests/intel/debugfs_test.c
@@ -33,43 +33,29 @@
  * TEST: debugfs test
  * Description: Read entries from debugfs, hwmon and sysfs paths.
  * Feature: core
+ * Category: Infrastructure
+ * Functionality: debugfs
+ * Sub-category: uapi
+ * Test category: uapi
+ * Run type: FULL
  *
  * SUBTEST: basic-hwmon
- * Category: Infrastructure
- * Description:
- *   Read all entries from hwmon path
- *   validating debugfs entries
- * Functionality: debugfs entries
+ * Description: Read all entries from hwmon path validating debugfs entries
  * Run type: BAT
- * Test category: GEM_Legacy
  *
  * SUBTEST: read_all_entries
- * Category: Infrastructure
- * Description:
- *   Read all entries from debugfs path.
- *   validating debugfs entries
- * Functionality: debugfs entries
+ * Description: Read all entries from debugfs path validating debugfs entries
  * Run type: BAT
- * Sub-category: i915
- * Test category: GEM_Legacy
  *
  * SUBTEST: read_all_entries_display_off
  * Description: Read all debugfs entries with display on/off.
- * Run type: FULL
  *
  * SUBTEST: read_all_entries_display_on
  * Description: Read all debugfs entries with display on/off.
- * Run type: FULL
  *
  * SUBTEST: sysfs
- * Category: Infrastructure
- * Description:
- *   Read all entries from sysfs path.
- *   validating debugfs entries
- * Functionality: debugfs entries
- * Run type: FULL
- * Sub-category: i915
- * Test category: GEM_Legacy
+ * Description: Read all entries from sysfs path validating debugfs entries
+ *
  */
 
 IGT_TEST_DESCRIPTION("Read entries from debugfs, hwmon and sysfs paths.");
diff --git a/tests/intel/gem_basic.c b/tests/intel/gem_basic.c
index 3485f92c8..513113cd1 100644
--- a/tests/intel/gem_basic.c
+++ b/tests/intel/gem_basic.c
@@ -44,33 +44,21 @@
 /**
  * TEST: gem basic
  * Description: Tests basic gem_create and gem_close IOCTLs
- *
- * SUBTEST: bad-close
  * Category: Infrastructure
- * Description: Verify that gem_close fails with bad params.
  * Feature: gem_core
  * Functionality: driver handler
- * Run type: BAT
- * Sub-category: i915
+ * Sub-category: uapi
  * Test category: GEM_Legacy
+ * Run type: BAT
+ *
+ * SUBTEST: bad-close
+ * Description: Verify that gem_close fails with bad params.
  *
  * SUBTEST: create-close
- * Category: Infrastructure
  * Description: Verify basic functionality of gem_create and gem_close.
- * Feature: gem_core
- * Functionality: driver handler
- * Run type: BAT
- * Sub-category: i915
- * Test category: GEM_Legacy
  *
  * SUBTEST: create-fd-close
- * Category: Infrastructure
  * Description: Verify that closing drm driver is possible with opened gem object.
- * Feature: gem_core
- * Functionality: driver handler
- * Run type: BAT
- * Sub-category: i915
- * Test category: GEM_Legacy
  *
  * SUBTEST: multigpu-create-close
  * Description: Verify basic functionality of gem_create and gem_close on multi-GPU.
diff --git a/tests/intel/gem_blits.c b/tests/intel/gem_blits.c
index e567cdf33..b9d0b5f87 100644
--- a/tests/intel/gem_blits.c
+++ b/tests/intel/gem_blits.c
@@ -34,7 +34,7 @@
  * Feature: blitter
  * Functionality: command streamer
  * Run type: FULL
- * Sub-category: HW
+ * Sub-category: CMD submission
  * Test category: GEM_Legacy
  *
  * SUBTEST: basic
diff --git a/tests/intel/gem_busy.c b/tests/intel/gem_busy.c
index c60654b1e..7baaac41f 100644
--- a/tests/intel/gem_busy.c
+++ b/tests/intel/gem_busy.c
@@ -33,57 +33,43 @@
 /**
  * TEST: gem busy
  * Description: Basic check of busy-ioctl ABI.
+ * Category: Desktop client
+ * Functionality: semaphore
+ * Sub-category: uapi
+ * Test category: GEM_Legacy
  * Feature: cmd_submission
  * Run type: FULL
  *
  * SUBTEST: busy
- * Category: Desktop client
  * Description: Basic test to check busyness of each engine.
- * Functionality: semaphore
- * Sub-category: i915
- * Test category: GEM_Legacy
  *
  * SUBTEST: busy at all
- * Functionality: Semaphore
  *
  * SUBTEST: busy at all-engines
  * Run type: BAT
  *
  * SUBTEST: close-race
- * Category: Desktop client
- * Description: Test to check race condition by randomly closing the handle using gem_close.
- * Functionality: semaphore
- * Sub-category: i915
- * Test category: GEM_Legacy
+ * Description: Test to check race condition by randomly closing the handle
+ *		using gem_close.
  *
  * SUBTEST: extended
- * Category: Desktop client
  * Description: Extended test to check busyness of dwstore-capable engines.
- * Functionality: semaphore
- * Sub-category: i915
- * Test category: GEM_Legacy
  *
  * SUBTEST: hang
- * Description: Basic test to check hang state behaviour of engines with increased timeout.
+ * Description: Basic test to check hang state behaviour of engines with
+ *		increased timeout.
  *
  * SUBTEST: hang-extended
- * Description:
- *   Extended test to check hang state behaviour of dwstore-capable engines with increased
- *   timeout.
+ * Description: Extended test to check hang state behaviour of dwstore-capable
+ *		engines with increased timeout.
  *
  * SUBTEST: parallel
- * Category: Desktop client
- * Description:
- *   Extended test to check busyness of dwstore-capable engines while doing parallel
- *   execution.
- * Functionality: semaphore
- * Sub-category: i915
- * Test category: GEM_Legacy
+ * Description: Extended test to check busyness of dwstore-capable engines
+ *		while doing parallel execution.
  *
  * SUBTEST: semaphore
- * Description:
- *   Test to check busyness of engine on submitting a new batch while engine is busy in executing
- *   previous batch.
+ * Description: Test to check busyness of engine on submitting a new batch
+ *		while engine is busy in executing previous batch.
  */
 
 #define PAGE_ALIGN(x) ALIGN(x, 4096)
diff --git a/tests/intel/gem_ccs.c b/tests/intel/gem_ccs.c
index e8f16d7d8..3442ecfd3 100644
--- a/tests/intel/gem_ccs.c
+++ b/tests/intel/gem_ccs.c
@@ -19,11 +19,11 @@
 /**
  * TEST: gem ccs
  * Description: Exercise gen12 blitter with and without flatccs compression
+ * Feature: flat_ccs_mapping
  * Run type: FULL
  *
  * SUBTEST: block-copy-compressed
  * Description: Check block-copy flatccs compressed blit
- * Feature: flat_ccs_mapping
  *
  * SUBTEST: block-copy-uncompressed
  * Description: Check block-copy uncompressed blit
@@ -31,19 +31,16 @@
  *
  * SUBTEST: block-multicopy-compressed
  * Description: Check block-multicopy flatccs compressed blit
- * Feature: flat_ccs_mapping
  *
  * SUBTEST: block-multicopy-inplace
  * Description: Check block-multicopy flatccs inplace decompression blit
- * Feature: flat_ccs_mapping
  *
  * SUBTEST: ctrl-surf-copy
  * Description: Check flatccs data can be copied from/to surface
- * Feature: flat_ccs_mapping
  *
  * SUBTEST: ctrl-surf-copy-new-ctx
- * Description: Check flatccs data are physically tagged and visible in different contexts
- * Feature: flat_ccs_mapping
+ * Description: Check flatccs data are physically tagged and visible i
+ *		different contexts
  *
  * SUBTEST: suspend-resume
  * Description: Check flatccs data persists after suspend / resume (S0)
diff --git a/tests/intel/gem_close_race.c b/tests/intel/gem_close_race.c
index cef0ff025..9c2034ad3 100644
--- a/tests/intel/gem_close_race.c
+++ b/tests/intel/gem_close_race.c
@@ -51,57 +51,42 @@
 /**
  * TEST: gem close race
  * Description: Test try to race gem_close against workload submission.
- *
- * SUBTEST: basic-process
  * Category: Infrastructure
- * Description: Basic workload submission.
  * Feature: synchronization
  * Functionality: driver handler
- * Run type: BAT
- * Sub-category: i915
+ * Sub-category: CMD submission
  * Test category: GEM_Legacy
+ * Run type: FULL
+ *
+ * SUBTEST: basic-process
+ * Description: Basic workload submission.
+ * Run type: BAT
  *
  * SUBTEST: basic-threads
- * Category: Infrastructure
- * Description:
- *   Share buffer handle across different drm fd's and trying to race gem_close against
- *   continuous workload with minimum timeout.
- * Feature: synchronization
- * Functionality: driver handler
+ * Description: Share buffer handle across different drm fd's and trying to
+ *		race gem_close against continuous workload with minimum timeout.
  * Run type: BAT
- * Sub-category: i915
- * Test category: GEM_Legacy
  *
  * SUBTEST: contexts
- * Description:
- *   Share buffer handle across different drm fd's and trying to race gem_close against
- *   continuous workload in other contexts.
- * Feature: synchronization
- * Run type: FULL
+ * Description: Share buffer handle across different drm fd's and trying to
+ *		race gem_close against continuous workload in other contexts.
  *
  * SUBTEST: gem-close-race
- * Description:
- *   Share buffer handle across different drm fd's and trying to race of gem_close against
- *   continuous workload.
- * Feature: synchronization
- * Run type: FULL
+ * Description: Share buffer handle across different drm fd's and trying to
+ *		race of gem_close against continuous workload.
  *
  * SUBTEST: multigpu-basic-process
  * Description: Basic workload submission on multi-GPU machine.
  * Feature: multigpu, synchronization
- * Run type: FULL
  * Test category: MultiGPU
  *
  * SUBTEST: multigpu-basic-threads
  * Description: Run basic-threads race on multi-GPU machine.
  * Feature: multigpu, synchronization
- * Run type: FULL
  * Test category: MultiGPU
  *
  * SUBTEST: process-exit
  * Description: Test try to race gem_close against submission of continuous workload.
- * Feature: synchronization
- * Run type: FULL
  */
 
 #define OBJECT_SIZE (256 * 1024)
diff --git a/tests/intel/gem_create.c b/tests/intel/gem_create.c
index 87fde2963..77ffceda4 100644
--- a/tests/intel/gem_create.c
+++ b/tests/intel/gem_create.c
@@ -64,101 +64,60 @@
 #include "i915_drm.h"
 /**
  * TEST: gem create
- * Description:
- *   Ensure that basic gem_create and gem_create_ext works and that invalid input combinations
- *   are rejected.
- * Run type: FULL
- *
- * SUBTEST: busy-create
+ * Description: Ensure that basic gem_create and gem_create_ext works and that
+ *		invalid input combinations are rejected.
  * Category: Infrastructure
- * Description: Create buffer objects while GPU is busy.
  * Feature: mapping
  * Functionality: buffer management
- * Sub-category: i915
+ * Sub-category: uapi
  * Test category: GEM_Legacy
+ * Run type: FULL
+ *
+ * SUBTEST: busy-create
+ * Description: Create buffer objects while GPU is busy.
  *
  * SUBTEST: create-clear
- * Category: Infrastructure
  * Description: Verify that all new objects are clear.
- * Feature: mapping
- * Functionality: buffer management
- * Sub-category: i915
- * Test category: GEM_Legacy
  *
  * SUBTEST: create-ext-cpu-access-big
- * Description:
- *   Verify the extreme cases with very large objects and.
- *   I915_GEM_CREATE_EXT_FLAG_NEEDS_CPU_ACCESS
- * Feature: mapping
+ * Description: Verify the extreme cases with very large objects and
+ *		I915_GEM_CREATE_EXT_FLAG_NEEDS_CPU_ACCESS
  *
  * SUBTEST: create-ext-cpu-access-sanity-check
- * Description:
- *   Verify the basic functionally and expected ABI contract around.
- *   I915_GEM_CREATE_EXT_FLAG_NEEDS_CPU_ACCESS
- * Feature: mapping
+ * Description: Verify the basic functionally and expected ABI contract around
+ *		I915_GEM_CREATE_EXT_FLAG_NEEDS_CPU_ACCESS
  *
  * SUBTEST: create-ext-placement-all
  * Category: Infrastructure
  * Description: Create objects in every memory region using create_ext.
- * Feature: mapping
- * Functionality: buffer management
- * Sub-category: i915
- * Test category: GEM_Legacy
  *
  * SUBTEST: create-ext-placement-each
- * Category: Infrastructure
- * Description: Create one object with memory pieces in each memory region using create_ext.
- * Feature: mapping
- * Functionality: buffer management
- * Sub-category: i915
- * Test category: GEM_Legacy
+ * Description: Create one object with memory pieces in each memory region
+ *		using create_ext.
  *
  * SUBTEST: create-ext-placement-sanity-check
- * Category: Infrastructure
  * Description: Exercise create_ext placements extension.
- * Feature: mapping
- * Functionality: buffer management
- * Sub-category: i915
- * Test category: GEM_Legacy
  *
  * SUBTEST: create-invalid-size
- * Category: Infrastructure
- * Description: Try to create a gem object of invalid size 0 and check if ioctl returns error.
- * Feature: mapping
- * Functionality: buffer management
- * Sub-category: i915
- * Test category: GEM_Legacy
+ * Description: Try to create a gem object of invalid size 0 and check if
+ *		ioctl returns error.
  *
  * SUBTEST: create-massive
- * Category: Infrastructure
- * Description: Exercise creation of buffer object with impossible size and check for the expected error.
- * Feature: mapping
- * Functionality: buffer management
- * Sub-category: i915
- * Test category: GEM_Legacy
+ * Description: Exercise creation of buffer object with impossible size and
+ *		check for the expected error.
  *
  * SUBTEST: create-size-update
- * Category: Infrastructure
- * Description: Try to create a gem object with size 15 and check actual created size.
- * Feature: mapping
- * Functionality: buffer management
- * Sub-category: i915
- * Test category: GEM_Legacy
+ * Description: Try to create a gem object with size 15 and check actual
+ *		created size.
  *
  * SUBTEST: create-valid-nonaligned
- * Category: Infrastructure
- * Description:
- *   Try to create an object with non-aligned size, check we got one with size aligned up to page
- *   size and test we can write into the padded extra memory.
- * Feature: mapping
- * Functionality: buffer management
- * Sub-category: i915
- * Test category: GEM_Legacy
+ * Description: Try to create an object with non-aligned size, check we got one
+ *		with size aligned up to page size and test we can write into
+ *		the padded extra memory.
  *
  * SUBTEST: hog-create
  * Description: Create buffer objects while GPU is busy.
- * Feature: mapping
- * Test category: GEM_Legacy
+ *
  */
 
 IGT_TEST_DESCRIPTION("Ensure that basic gem_create and gem_create_ext works"
-- 
2.34.1



More information about the igt-dev mailing list