[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