[PATCH i-g-t 01/24] tests: Introduce uapi subcategory
Mauro Carvalho Chehab
mauro.chehab at linux.intel.com
Mon Mar 4 14:57:26 UTC 2024
On Mon, 4 Mar 2024 11:15:54 +0000
Katarzyna Piecielska <katarzyna.piecielska at intel.com> wrote:
> Let's introduce uapi subcategory to explictly 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
>
> 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>
> ---
> 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_sseu.c | 3 +++
> tests/intel/gem_reg_read.c | 3 +++
> tests/intel/gem_reset_stats.c | 3 +++
> tests/intel/i915_getparams_basic.c | 2 +-
> tests/intel/i915_pm_freq_api.c | 3 +++
> tests/intel/sysfs_defaults.c | 6 +++---
> 10 files changed, 41 insertions(+), 21 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
For this series, it sounds ok to keep the description empty, but the better
would be to add descriptions for each subtest later on.
Reviewed-by: Mauro Carvalho Chehab <mchehab at kernel.org>
> + */
> +
> 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_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_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/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
> */
More information about the igt-dev
mailing list