[PATCH i-g-t v5 01/24] tests: Introduce uapi subcategory

Katarzyna Piecielska katarzyna.piecielska at intel.com
Thu Mar 21 15:38:24 UTC 2024


Let's introduce uapi subcategory to explicitly show where we are testing
the basics.
Also fix already set values:
- fix subcategory for create-ext-set-pat subtest
- fix functionality for gem_ctx_sseu: s/Sub-slice programming/sseu

v2: Add more tests to uapi from synchronization/semephore features. Make
Sub-category/Functionality/Feature unique. (Kamil)

Signed-off-by: Katarzyna Piecielska <katarzyna.piecielska at intel.com>
Cc: Mauro Carvalho Chehab <mauro.chehab at linux.intel.com>
Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>
Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>
---
 tests/core_getstats.c              |  4 ++--
 tests/drm_read.c                   | 17 +++++++++++++++++
 tests/dumb_buffer.c                |  6 +++---
 tests/intel/gem_create.c           | 15 +++------------
 tests/intel/gem_ctx_param.c        |  2 +-
 tests/intel/gem_ctx_sseu.c         |  3 +++
 tests/intel/gem_eio.c              |  2 +-
 tests/intel/gem_reg_read.c         |  3 +++
 tests/intel/gem_reset_stats.c      |  3 +++
 tests/intel/gem_wait.c             |  6 +++---
 tests/intel/i915_getparams_basic.c |  2 +-
 tests/intel/i915_pm_freq_api.c     |  3 +++
 tests/intel/sysfs_defaults.c       |  6 +++---
 tests/sw_sync.c                    |  7 +++----
 tests/syncobj_timeline.c           |  6 +++---
 tests/syncobj_wait.c               |  6 +++---
 16 files changed, 55 insertions(+), 36 deletions(-)

diff --git a/tests/core_getstats.c b/tests/core_getstats.c
index 8b5655053..1a8ad22c7 100644
--- a/tests/core_getstats.c
+++ b/tests/core_getstats.c
@@ -37,9 +37,9 @@
  * Category: Desktop client
  * Description: Tests the DRM_IOCTL_GET_STATS ioctl.
  * Feature: core
- * Functionality: permission management for clients
+ * Functionality: DRM memory management
  * Run type: FULL
- * Sub-category: DRM
+ * Sub-category: uapi
  * Test category: GEM_Legacy
  *
  * SUBTEST:
diff --git a/tests/drm_read.c b/tests/drm_read.c
index 36dc623ed..f8ef324fa 100644
--- a/tests/drm_read.c
+++ b/tests/drm_read.c
@@ -50,6 +50,23 @@
 
 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
+ * Sub-category: uapi
+ * Run type: FULL
+ *
+ * SUBTEST: invalid-buffer
+ * SUBTEST: fault-buffer
+ * SUBTEST: empty-block
+ * SUBTEST: empty-nonblock
+ * SUBTEST: short-buffer-block
+ * SUBTEST: short-buffer-nonblock
+ * SUBTEST: short-buffer-wakeup
+ */
+
 static void sighandler(int sig, siginfo_t * info, void *context)
 {
 }
diff --git a/tests/dumb_buffer.c b/tests/dumb_buffer.c
index 9db857f9f..d76f0c70f 100644
--- a/tests/dumb_buffer.c
+++ b/tests/dumb_buffer.c
@@ -54,10 +54,10 @@
  * Category: Desktop client
  * Description: This is a test for the generic dumb buffer interface.
  * Feature: prime
- * Functionality: IOCTL check for buffer management
+ * Functionality: ioctl
  * Run type: FULL
- * Sub-category: Compositor
- * Test category: GEM_Legacy
+ * Sub-category: uapi
+ * Test category: Compositor
  *
  * SUBTEST: create-clear
  *
diff --git a/tests/intel/gem_create.c b/tests/intel/gem_create.c
index 77ffceda4..ae9575089 100644
--- a/tests/intel/gem_create.c
+++ b/tests/intel/gem_create.c
@@ -98,6 +98,9 @@
  * SUBTEST: create-ext-placement-sanity-check
  * Description: Exercise create_ext placements extension.
  *
+ * SUBTEST: create-ext-set-pat
+ * Description:  Create GEM object with specific PAT index
+ *
  * SUBTEST: create-invalid-size
  * Description: Try to create a gem object of invalid size 0 and check if
  *		ioctl returns error.
@@ -594,18 +597,6 @@ static void create_ext_placement_each(int fd)
 	free(regions);
 }
 
-/**
-  * TEST: GEM create uAPI
-  * Category: Infrastructure
-  * Description: Tests the DRM_IOCTL_I915_GEM_CREATE_EXT ioctl.
-  * Feature: core
-  * Functionality: Create GEM object with specific PAT index
-  * Run type: BAT
-  * Sub-category: i915
-  * Test category: GEM_Legacy
-  *
-  * SUBTEST: create-ext-set-pat
-  */
 static void create_ext_set_pat(int fd)
 {
 	struct drm_i915_gem_create_ext_set_pat setparam_pat = {
diff --git a/tests/intel/gem_ctx_param.c b/tests/intel/gem_ctx_param.c
index c6fa31875..61b79e1ab 100644
--- a/tests/intel/gem_ctx_param.c
+++ b/tests/intel/gem_ctx_param.c
@@ -34,7 +34,7 @@
  * TEST: gem ctx param
  * Description: Basic test for context set/get param input validation.
  * Category: Infrastructure
- * Feature: context
+ * Feature: context feature
  * Functionality: context
  * Sub-category: uapi
  * Test category: GEM_Legacy
diff --git a/tests/intel/gem_ctx_sseu.c b/tests/intel/gem_ctx_sseu.c
index 3a01944e4..a9d400c2d 100644
--- a/tests/intel/gem_ctx_sseu.c
+++ b/tests/intel/gem_ctx_sseu.c
@@ -48,6 +48,9 @@
  * TEST: gem ctx sseu
  * Description: Test context render powergating programming.
  * Feature: pm_sseu
+ * Sub-category: uapi
+ * Category: Desktop client
+ * Functionality: sseu
  * Run type: FULL
  *
  * SUBTEST: engines
diff --git a/tests/intel/gem_eio.c b/tests/intel/gem_eio.c
index d6d12bd08..6fa68441f 100644
--- a/tests/intel/gem_eio.c
+++ b/tests/intel/gem_eio.c
@@ -57,7 +57,7 @@
  * TEST: gem eio
  * Description: Test that specific ioctls report a wedged GPU (EIO).
  * Category: Infrastructure
- * Feature: reset
+ * Feature: reset uapi
  * Functionality: reset
  * Sub-category: uapi
  * Test category: GEM_Legacy
diff --git a/tests/intel/gem_reg_read.c b/tests/intel/gem_reg_read.c
index de6788abe..03b12551d 100644
--- a/tests/intel/gem_reg_read.c
+++ b/tests/intel/gem_reg_read.c
@@ -35,6 +35,9 @@
 /**
  * TEST: gem reg read
  * Feature: gem_core
+ * Sub-category: uapi
+ * Category: Desktop client
+ * Functionality: ioctl
  * Run type: FULL
  *
  * SUBTEST: bad-register
diff --git a/tests/intel/gem_reset_stats.c b/tests/intel/gem_reset_stats.c
index 5dfc1fc67..a62c4061a 100644
--- a/tests/intel/gem_reset_stats.c
+++ b/tests/intel/gem_reset_stats.c
@@ -49,6 +49,9 @@
 #include "sw_sync.h"
 /**
  * TEST: gem reset stats
+ * Sub-category: uapi
+ * Category: Desktop client
+ * Functionality: reset
  * Run type: FULL
  *
  * SUBTEST: ban-blt
diff --git a/tests/intel/gem_wait.c b/tests/intel/gem_wait.c
index e2a598c64..99ac87536 100644
--- a/tests/intel/gem_wait.c
+++ b/tests/intel/gem_wait.c
@@ -35,9 +35,9 @@
  * TEST: gem wait
  * Category: Infrastructure
  * Description: Tests the GEM_WAIT ioctl
- * Feature: synchronization
- * Functionality: semaphore
- * Sub-category: i915
+ * Feature: synchronization feature
+ * Functionality: GEM_WAIT ioctl
+ * Sub-category: uapi
  * Test category: GEM_Legacy
  * Run type: FULL
  *
diff --git a/tests/intel/i915_getparams_basic.c b/tests/intel/i915_getparams_basic.c
index b22ab5d93..971c35e55 100644
--- a/tests/intel/i915_getparams_basic.c
+++ b/tests/intel/i915_getparams_basic.c
@@ -37,7 +37,7 @@
  * Feature: gem_core
  * Functionality: device topology
  * Run type: BAT
- * Sub-category: i915
+ * Sub-category: uapi
  * Test category: GEM_Legacy
  *
  * SUBTEST: basic-eu-total
diff --git a/tests/intel/i915_pm_freq_api.c b/tests/intel/i915_pm_freq_api.c
index 6018692a2..aa2de74c9 100644
--- a/tests/intel/i915_pm_freq_api.c
+++ b/tests/intel/i915_pm_freq_api.c
@@ -11,6 +11,9 @@
 /**
  * TEST: i915 pm freq api
  * Description: Test SLPC freq API
+ * Category: Infrastructure
+ * Sub-category: uapi
+ * Functionality: slpc
  * Run type: FULL
  *
  * SUBTEST: freq-basic-api
diff --git a/tests/intel/sysfs_defaults.c b/tests/intel/sysfs_defaults.c
index b686c19fc..a80f7126c 100644
--- a/tests/intel/sysfs_defaults.c
+++ b/tests/intel/sysfs_defaults.c
@@ -37,10 +37,10 @@
  * TEST: sysfs defaults
  * Category: Infrastructure
  * Feature: gem_core
- * Functionality: driver handler
- * Run type: FULL
- * Sub-category: i915
+ * Functionality: sysfs
+ * Sub-category: uapi
  * Test category: GEM_Legacy
+ * Run type: FULL
  *
  * SUBTEST: readonly
  */
diff --git a/tests/sw_sync.c b/tests/sw_sync.c
index 6c476ff46..a276c4a22 100644
--- a/tests/sw_sync.c
+++ b/tests/sw_sync.c
@@ -42,10 +42,9 @@
  * TEST: sw sync
  * Category: Infrastructure
  * Description: Test SW Sync Framework
- * Feature: synchronization
- * Functionality: semaphore
- * Run type: FULL
- * Sub-category: DRM
+ * Feature: semaphore
+ * Functionality: sw sync framework
+ * Sub-category: uapi
  * Test category: GEM_Legacy
  *
  * SUBTEST: alloc_fence
diff --git a/tests/syncobj_timeline.c b/tests/syncobj_timeline.c
index ea8341a30..f369c5fe1 100644
--- a/tests/syncobj_timeline.c
+++ b/tests/syncobj_timeline.c
@@ -34,10 +34,10 @@
  * TEST: syncobj timeline
  * Category: Infrastructure
  * Description: Tests for the drm timeline sync object API
- * Feature: synchronization
- * Functionality: semaphore
  * Run type: FULL
- * Sub-category: DRM
+ * Feature: synchronization feature
+ * Functionality: sync_object
+ * Sub-category: uapi
  * Test category: GEM_Legacy
  *
  * SUBTEST: 32bits-limit
diff --git a/tests/syncobj_wait.c b/tests/syncobj_wait.c
index e0ff69b79..bc4e3378c 100644
--- a/tests/syncobj_wait.c
+++ b/tests/syncobj_wait.c
@@ -34,10 +34,10 @@
  * TEST: syncobj wait
  * Category: Infrastructure
  * Description: Tests for the drm sync object wait API
- * Feature: synchronization
- * Functionality: semaphore
+ * Feature: synchronization feature
+ * Functionality: object wait
  * Run type: FULL
- * Sub-category: DRM
+ * Sub-category: uapi
  * Test category: GEM_Legacy
  *
  * SUBTEST: invalid-multi-wait-all-unsubmitted
-- 
2.34.1



More information about the igt-dev mailing list