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

Kamil Konieczny kamil.konieczny at linux.intel.com
Fri Mar 1 14:24:52 UTC 2024


Hi Katarzyna,
On 2024-03-01 at 09:08:41 +0000, Katarzyna Piecielska wrote:
> Unify documentation for Intel i915 tests that we have by moving
> common parts to be available for all tests.
> 
> v2: fix checkpatch warnings
> 
> Signed-off-by: Katarzyna Piecielska <katarzyna.piecielska at intel.com>

Acked-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>

> ---
>  tests/intel/gem_ctx_create.c      |  77 ++++++----------
>  tests/intel/gem_ctx_exec.c        |  51 +++--------
>  tests/intel/gem_ctx_isolation.c   |  43 ++-------
>  tests/intel/gem_ctx_param.c       | 144 ++++++------------------------
>  tests/intel/gem_ctx_persistence.c | 117 ++----------------------
>  tests/intel/gem_ctx_shared.c      |  42 ++-------
>  tests/intel/gem_eio.c             | 133 ++-------------------------
>  tests/intel/gem_exec_balancer.c   | 139 ++--------------------------
>  8 files changed, 99 insertions(+), 647 deletions(-)
> 
> diff --git a/tests/intel/gem_ctx_create.c b/tests/intel/gem_ctx_create.c
> index 6d0877954..5fb0f9a1f 100644
> --- a/tests/intel/gem_ctx_create.c
> +++ b/tests/intel/gem_ctx_create.c
> @@ -38,91 +38,64 @@
>  /**
>   * TEST: gem ctx create
>   * Description: Test the context create ioctls
> - *
> - * SUBTEST: active
> - * Description:
> - *   For each engine calculate the average performance of context creation execution and
> - *   exercise context reclaim
> + * Category: Infrastructure
>   * Feature: context
> + * Functionality: context
> + * Sub-category: Context
> + * Test category: GEM_Legacy
>   * Run type: FULL
>   *
> + * SUBTEST: active
> + * Description: For each engine calculate the average performance of context
> + *		creation execution and exercise context reclaim
> + *
>   * SUBTEST: active-all
> - * Description: Calculate the average performance of context creation and it's execution using all engines
> - * Feature: context
> - * Run type: FULL
> + * Description: Calculate the average performance of context creation and it's
> + *		execution using all engines
>   *
>   * SUBTEST: basic
> - * Category: Infrastructure
>   * Description: Test random context creation
> - * Feature: context
> - * Functionality: context management
>   * Run type: BAT
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: basic-files
> - * Category: Infrastructure
>   * Description: Exercise implicit per-fd context creation
> - * Feature: context
> - * Functionality: context management
>   * Run type: BAT
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: ext-param
>   * Description: Verify valid and invalid context extensions
> - * Feature: context
> - * Run type: FULL
>   *
>   * SUBTEST: files
>   * Description: Exercise implicit per-fd context creation on 1 CPU for long duration
> - * Feature: context
> - * Run type: FULL
>   *
>   * SUBTEST: forked-active
> - * Description:
> - *   For each engine calculate the average performance of context creation and execution on
> - *   multiple parallel processes
> - * Feature: context
> - * Run type: FULL
> + * Description: For each engine calculate the average performance of context
> + *		creation and execution on multiple parallel processes
>   *
>   * SUBTEST: forked-active-all
> - * Description:
> - *   Calculate the average performance of context creation and it's execution using all engines
> - *   on multiple parallel processes
> - * Feature: context
> - * Run type: FULL
> + * Description: Calculate the average performance of context creation and it's
> + *		execution using all engines on multiple parallel processes
>   *
>   * SUBTEST: forked-files
> - * Description: Exercise implicit per-fd context creation on all CPUs for long duration
> - * Feature: context
> - * Run type: FULL
> + * Description: Exercise implicit per-fd context creation on all CPUs for long
> + *		duration
>   *
>   * SUBTEST: hog
> - * Description:
> - *   For each engine calculate the average performance of context creation and execution while
> - *   all other engines are hogging the resources
> - * Feature: context
> - * Run type: FULL
> + * Description: For each engine calculate the average performance of context
> + *		creation and execution while all other engines are hogging the
> + *		resources
>   *
>   * SUBTEST: iris-pipeline
>   * Description: Set, validate and execute particular context params
> - * Feature: context
> - * Run type: FULL
>   *
>   * SUBTEST: maximum-mem
> - * Description:
> - *   Create contexts upto available RAM size, calculate the average performance of their
> - *   execution on multiple parallel processes
> - * Feature: context
> - * Run type: FULL
> + * Description: Create contexts up to available RAM size, calculate the average
> + *		performance of their execution on multiple parallel processes
>   *
>   * SUBTEST: maximum-swap
> - * Description:
> - *   Create contexts upto available RAM+SWAP size, calculate the average performance of their
> - *   execution on multiple parallel processes
> - * Feature: context
> - * Run type: FULL
> + * Description: Create contexts up to available RAM+SWAP size, calculate the
> + *		average performance of their execution on multiple parallel
> + *		processes
> + *
>   */
>  
>  IGT_TEST_DESCRIPTION("Test the context create ioctls");
> diff --git a/tests/intel/gem_ctx_exec.c b/tests/intel/gem_ctx_exec.c
> index 4376397a2..20d42d5b0 100644
> --- a/tests/intel/gem_ctx_exec.c
> +++ b/tests/intel/gem_ctx_exec.c
> @@ -50,61 +50,38 @@
>  /**
>   * TEST: gem ctx exec
>   * Description: Test context batch buffer execution.
> + * Category: Infrastructure
> + * Feature: context
> + * Functionality: execbuf
> + * Sub-category: Context
> + * Test category: GEM_Legacy
> + * Run type: FULL
>   *
>   * SUBTEST: basic
> - * Category: Infrastructure
>   * Description: Check the basic context batch buffer execution.
> - * Feature: context
> - * Functionality: command submission
>   * Run type: BAT
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: basic-close-race
> - * Category: Infrastructure
> - * Description: Race the execution and interrupt handlers along a context, while closing it at a random time.
> - * Feature: context
> - * Functionality: command submission
> - * Run type: FULL
> - * Sub-category: i915
> - * Test category: GEM_Legacy
> + * Description: Race the execution and interrupt handlers along a context,
> + *		while closing it at a random time.
>   *
>   * SUBTEST: basic-invalid-context
> - * Category: Infrastructure
>   * Description: Verify that execbuf with invalid context fails.
> - * Feature: context
> - * Functionality: command submission
> - * Run type: FULL
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: basic-nohangcheck
> - * Category: Server
> - * Description: Verify that contexts are automatically shotdown on close, if hangchecking is disabled.
> - * Feature: context
> - * Functionality: command submission
> - * Run type: FULL
> - * Sub-category: i915
> - * Test category: GEM_Legacy
> + * Description: Verify that contexts are automatically shutdown on close,
> + *		if hangchecking is disabled.
>   *
>   * SUBTEST: basic-norecovery
> - * Category: Server
> - * Description: Check the status of context after a hang by setting and unsetting the RECOVERABLE.
> - * Feature: context
> - * Functionality: command submission
> - * Run type: FULL
> - * Sub-category: i915
> - * Test category: GEM_Legacy
> + * Description: Check the status of context after a hang by setting and
> + *		unsetting the RECOVERABLE.
>   *
>   * SUBTEST: eviction
>   * Description: Check maximum number of buffers it can evict for a context.
> - * Feature: context
> - * Run type: FULL
>   *
>   * SUBTEST: reset-pin-leak
> - * Description: Check if the kernel doesn't leak the vma pin_count for the last context on reset.
> - * Feature: context
> - * Run type: FULL
> + * Description: Check if the kernel doesn't leak the vma pin_count for the last
> + *		context on reset.
>   */
>  
>  IGT_TEST_DESCRIPTION("Test context batch buffer execution.");
> diff --git a/tests/intel/gem_ctx_isolation.c b/tests/intel/gem_ctx_isolation.c
> index d50a90597..89c023e09 100644
> --- a/tests/intel/gem_ctx_isolation.c
> +++ b/tests/intel/gem_ctx_isolation.c
> @@ -28,62 +28,31 @@
>  #include "igt_types.h"
>  /**
>   * TEST: gem ctx isolation
> - * Run type: FULL
> - *
> - * SUBTEST: clean
>   * Category: Infrastructure
>   * Feature: context
> - * Functionality: security
> - * Sub-category: HW
> + * Functionality: priv/non-priv registers
> + * Sub-category: Security
>   * Test category: GEM_Legacy
> + * Run type: FULL
> + *
> + * SUBTEST: clean
>   *
>   * SUBTEST: dirty-create
> - * Category: Infrastructure
> - * Feature: context
> - * Functionality: security
> - * Sub-category: HW
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: dirty-switch
> - * Category: Infrastructure
> - * Feature: context
> - * Functionality: security
> - * Sub-category: HW
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: nonpriv
> - * Category: Infrastructure
> - * Feature: context
> - * Functionality: security
> - * Sub-category: HW
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: nonpriv-switch
> - * Category: Infrastructure
> - * Feature: context
> - * Functionality: security
> - * Sub-category: HW
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: preservation
> - * Category: Infrastructure
> - * Feature: context
> - * Functionality: security
> - * Sub-category: HW
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: preservation-S3
> - * Feature: context
>   *
>   * SUBTEST: preservation-S4
> - * Feature: context
>   *
>   * SUBTEST: preservation-reset
> - * Category: Infrastructure
> - * Feature: context
> - * Functionality: security
> - * Sub-category: HW
> - * Test category: GEM_Legacy
> + *
>   */
>  
>  /* MAX_REG must be greater than the maximum register address. */
> diff --git a/tests/intel/gem_ctx_param.c b/tests/intel/gem_ctx_param.c
> index 9b44c86a0..c6fa31875 100644
> --- a/tests/intel/gem_ctx_param.c
> +++ b/tests/intel/gem_ctx_param.c
> @@ -33,166 +33,80 @@
>  /**
>   * TEST: gem ctx param
>   * Description: Basic test for context set/get param input validation.
> + * Category: Infrastructure
> + * Feature: context
> + * Functionality: context
> + * Sub-category: uapi
> + * Test category: GEM_Legacy
>   * Run type: FULL
>   *
>   * SUBTEST: basic
> - * Category: Infrastructure
>   * Description: Basic test for context get/set param ioctls using valid context
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: basic-default
> - * Category: Infrastructure
>   * Description: Basic test for context get/set param ioctls using default context
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: get-priority-new-ctx
> - * Category: Infrastructure
> - * Description:
> - *   Test performed with context param set to priority.
> - *   Verify that priority is default for newly created context
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: i915
> - * Test category: GEM_Legacy
> + * Description: Test performed with context param set to priority. Verify that
> + *		priority is default for newly created context
>   *
>   * SUBTEST: invalid-ctx-get
> - * Category: Infrastructure
> - * Description: Verify that context get param ioctl using invalid context returns relevant error
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: i915
> - * Test category: GEM_Legacy
> + * Description: Verify that context get param ioctl using invalid context
> + *		returns relevant error
>   *
>   * SUBTEST: invalid-ctx-set
> - * Category: Infrastructure
> - * Description: Verify that context set param ioctl using invalid context returns relevant error
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: i915
> - * Test category: GEM_Legacy
> + * Description: Verify that context set param ioctl using invalid context
> + *		returns relevant error
>   *
>   * SUBTEST: invalid-get-engines
> - * Category: Infrastructure
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: invalid-get-no-zeromap
> - * Category: Infrastructure
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: invalid-get-ringsize
> - * Category: Infrastructure
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: invalid-param-get
> - * Category: Infrastructure
> - * Description: Checks that fetching context parameters using an unused param value is erroneous
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: i915
> - * Test category: GEM_Legacy
> + * Description: Checks that fetching context parameters using an unused param
> + *		value is erroneous
>   *
>   * SUBTEST: invalid-param-set
> - * Category: Infrastructure
> - * Description: Checks that setting context parameters using an unused param value is erroneous
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: i915
> - * Test category: GEM_Legacy
> + * Description: Checks that setting context parameters using an unused param
> + *		value is erroneous
>   *
>   * SUBTEST: invalid-set-no-zeromap
> - * Category: Infrastructure
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: invalid-set-ringsize
> - * Category: Infrastructure
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: invalid-size-get
> - * Category: Infrastructure
> - * Description: Verify that context get param ioctl returns valid size for valid context
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: i915
> - * Test category: GEM_Legacy
> + * Description: Verify that context get param ioctl returns valid size for
> + *		valid context
>   *
>   * SUBTEST: invalid-size-set
> - * Category: Infrastructure
> - * Description: Verify that context set param ioctl using invalid size returns relevant error
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: i915
> - * Test category: GEM_Legacy
> + * Description: Verify that context set param ioctl using invalid size returns
> + *		relevant error
>   *
>   * SUBTEST: non-root-set
> - * Category: Infrastructure
> - * Description: Verify that context set param ioctl returns relevant error in non root mode
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: i915
> - * Test category: GEM_Legacy
> + * Description: Verify that context set param ioctl returns relevant error in
> + *		non root mode
>   *
>   * SUBTEST: root-set
> - * Category: Infrastructure
>   * Description: Verify that context set param ioctl works fine in root mode
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: set-priority-invalid-size
> - * Category: Infrastructure
> - * Description:
> - *   Test performed with context param set to priority.
> - *   Verify that relevant error is returned on setting invalid ctx size with default priority
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: i915
> - * Test category: GEM_Legacy
> + * Description: Test performed with context param set to priority. Verify that
> + *		relevant error is returned on setting invalid ctx size with
> + *		default priority
>   *
>   * SUBTEST: set-priority-not-supported
> - * Description:
> - *   Verify that context set param ioctl returns relevant error if driver doesn't supports
> - *   assigning custom priorities from userspace
> - * Feature: context, no-set-priority
> + * Description: Verify that context set param ioctl returns relevant error if
> + *		driver doesn't supports assigning custom priorities from
> + *		userspace
>   *
>   * SUBTEST: set-priority-range
> - * Category: Infrastructure
> - * Description:
> - *   Test performed with context param set to priority.
> - *   Change priority range to test value overflow
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: i915
> - * Test category: GEM_Legacy
> + * Description: Test performed with context param set to priority. Change
> + *		priority range to test value overflow
>   *
>   * SUBTEST: vm
> - * Category: Infrastructure
>   * Description: Tests that multiple contexts can share the same VMA
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   */
>  
>  IGT_TEST_DESCRIPTION("Basic test for context set/get param input validation.");
> diff --git a/tests/intel/gem_ctx_persistence.c b/tests/intel/gem_ctx_persistence.c
> index c6e2c5d77..3f10c0fa3 100644
> --- a/tests/intel/gem_ctx_persistence.c
> +++ b/tests/intel/gem_ctx_persistence.c
> @@ -48,183 +48,76 @@
>  /**
>   * TEST: gem ctx persistence
>   * Feature: context
> - * Run type: FULL
> - *
> - * SUBTEST: engines-cleanup
>   * Category: Server
> - * Functionality: security
> + * Functionality: context
>   * Sub-category: Compute
>   * Test category: GEM_Legacy
> + * Run type: FULL
> + *
> + * SUBTEST: engines-cleanup
>   *
>   * SUBTEST: engines-hang
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: engines-hostile
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: engines-hostile-preempt
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: engines-mixed
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: engines-mixed-process
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: engines-persistence
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: engines-queued
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: file
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: hang
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: heartbeat-close
>   *
>   * SUBTEST: heartbeat-hang
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: heartbeat-hostile
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: heartbeat-many
>   *
>   * SUBTEST: heartbeat-stop
>   *
>   * SUBTEST: hostile
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: idempotent
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: legacy-engines-cleanup
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: legacy-engines-hang
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: legacy-engines-hostile
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: legacy-engines-hostile-preempt
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: legacy-engines-mixed
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: legacy-engines-mixed-process
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: legacy-engines-persistence
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: legacy-engines-queued
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: many-contexts
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: process
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: processes
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: saturated-hostile
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: saturated-hostile-nopreempt
>   *
>   * SUBTEST: smoketest
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: userptr
> - * Category: Server
> - * Functionality: security
> - * Sub-category: Compute
> - * Test category: GEM_Legacy
> + *
>   */
>  
>  #define RESET_TIMEOUT_MS 2 * MSEC_PER_SEC; /* default: 640ms */
> diff --git a/tests/intel/gem_ctx_shared.c b/tests/intel/gem_ctx_shared.c
> index fd429306e..8b045e20a 100644
> --- a/tests/intel/gem_ctx_shared.c
> +++ b/tests/intel/gem_ctx_shared.c
> @@ -50,71 +50,41 @@
>  /**
>   * TEST: gem ctx shared
>   * Description: Test shared contexts.
> + * Category: Desktop client
> + * Feature: context
> + * Functionality: shared contex
> + * Sub-category: Context
> + * Test category: GEM_Legacy
>   * Run type: FULL
>   *
>   * SUBTEST: Q-in-order
> - * Feature: context
>   *
>   * SUBTEST: Q-independent
> - * Feature: context
>   *
>   * SUBTEST: Q-out-order
> - * Feature: context
>   *
>   * SUBTEST: Q-promotion
> - * Feature: context
>   *
>   * SUBTEST: Q-smoketest
> - * Feature: context
>   *
>   * SUBTEST: Q-smoketest-all
> - * Feature: context
>   *
>   * SUBTEST: create-shared-gtt
> - * Category: Desktop client
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: Media
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: detached-shared-gtt
> - * Category: Desktop client
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: Media
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: disjoint-timelines
> - * Category: Desktop client
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: Media
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: exec-shared-gtt
> - * Category: Desktop client
> - * Feature: context
> - * Functionality: context management
> - * Sub-category: Media
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: exec-single-timeline
> - * Category: Desktop client
> - * Feature: context
> - * Functionality: context management
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: exhaust-shared-gtt
> - * Feature: context
>   *
>   * SUBTEST: exhaust-shared-gtt-lrc
> - * Feature: context
>   *
>   * SUBTEST: single-timeline
> - * Category: Desktop client
> - * Feature: context
> - * Functionality: context management
> - * Test category: GEM_Legacy
> + *
>   */
>  
>  #define LO 0
> diff --git a/tests/intel/gem_eio.c b/tests/intel/gem_eio.c
> index 99891ad15..d6d12bd08 100644
> --- a/tests/intel/gem_eio.c
> +++ b/tests/intel/gem_eio.c
> @@ -56,197 +56,78 @@
>  /**
>   * TEST: gem eio
>   * Description: Test that specific ioctls report a wedged GPU (EIO).
> - * Run type: FULL
> - *
> - * SUBTEST: banned
>   * Category: Infrastructure
>   * Feature: reset
>   * Functionality: reset
> - * Sub-category: i915
> + * Sub-category: uapi
>   * Test category: GEM_Legacy
> + * Run type: FULL
> + *
> + * SUBTEST: banned
>   *
>   * SUBTEST: context-create
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: create
> - * Category: Infrastructure
>   * Description: Validate i915_gem_create_ioctl, while gpu is wedged for fb scanout.
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: create-ext
> - * Category: Infrastructure
> - * Description:
> - *   Validate i915_gem_create_ext_ioctl and checks if returns clear backing store while gpu is
> - *   wedged for fb scanout.
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
> + * Description: Validate i915_gem_create_ext_ioctl and checks if returns clear
> + *		backing store while gpu is wedged for fb scanout.
>   *
>   * SUBTEST: execbuf
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: hibernate
>   * Description: Test that specific ioctls report a wedged GPU (EIO) during suspend
> - * Feature: hibernate, reset, suspend
> - * Test category: suspend
>   *
>   * SUBTEST: in-flight-10ms
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: in-flight-1us
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: in-flight-contexts-10ms
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: in-flight-contexts-1us
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: in-flight-contexts-immediate
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: in-flight-external
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: in-flight-immediate
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: in-flight-internal-10ms
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: in-flight-internal-1us
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: in-flight-internal-immediate
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: in-flight-suspend
>   * Description: Test that specific ioctls report a wedged GPU (EIO) during suspend
>   * Feature: reset, suspend
> - * Test category: suspend
>   *
>   * SUBTEST: kms
>   * Feature: kms_gem_interop, reset
>   *
>   * SUBTEST: reset-stress
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: suspend
>   * Description: Test that specific ioctls report a wedged GPU (EIO) during suspend
>   * Feature: reset, suspend
> - * Test category: suspend
>   *
>   * SUBTEST: throttle
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: unwedge-stress
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: wait-10ms
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: wait-1us
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: wait-immediate
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: wait-wedge-10ms
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: wait-wedge-1us
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: wait-wedge-immediate
> - * Category: Infrastructure
> - * Feature: reset
> - * Functionality: reset
> - * Sub-category: i915
> - * Test category: GEM_Legacy
> + *
>   */
>  
>  IGT_TEST_DESCRIPTION("Test that specific ioctls report a wedged GPU (EIO).");
> diff --git a/tests/intel/gem_exec_balancer.c b/tests/intel/gem_exec_balancer.c
> index 9190fc238..efa875889 100644
> --- a/tests/intel/gem_exec_balancer.c
> +++ b/tests/intel/gem_exec_balancer.c
> @@ -41,219 +41,94 @@
>  /**
>   * TEST: gem exec balancer
>   * Description: Exercise in-kernel load-balancing
> + * Category: Desktop \ Server
> + * Feature: cmd_submission, media_scalability
> + * Functionality: load balancing
> + * Sub-category: CMD submission
> + * Test category: Media & Compute
>   * Run type: FULL
>   *
>   * SUBTEST: bonded-chain
> - * Feature: cmd_submission, media_scalability
>   *
>   * SUBTEST: bonded-dual
> - * Feature: cmd_submission, media_scalability
>   *
>   * SUBTEST: bonded-false-hang
> - * Feature: cmd_submission, media_scalability
>   *
>   * SUBTEST: bonded-pair
> - * Feature: cmd_submission, media_scalability
>   *
>   * SUBTEST: bonded-semaphore
>   * Feature: cmd_submission, media_scalability, multictx
>   *
>   * SUBTEST: bonded-sync
> - * Feature: cmd_submission, media_scalability
>   *
>   * SUBTEST: bonded-true-hang
> - * Feature: cmd_submission, media_scalability
>   *
>   * SUBTEST: busy
> - * Category: Desktop \ Server
> - * Feature: cmd_submission, media_scalability
> - * Functionality: load balancing
> - * Sub-category: Media & Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: fairslice
> - * Feature: cmd_submission, media_scalability
>   *
>   * SUBTEST: full
> - * Category: Desktop \ Server
> - * Feature: cmd_submission, media_scalability, multictx
> - * Functionality: load balancing
> - * Sub-category: Media & Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: full-late
> - * Category: Desktop \ Server
>   * Feature: cmd_submission, media_scalability, multictx
> - * Functionality: load balancing
> - * Sub-category: Media & Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: full-late-pulse
> - * Category: Desktop \ Server
>   * Feature: cmd_submission, media_scalability, multictx
> - * Functionality: load balancing
> - * Sub-category: Media & Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: full-pulse
> - * Category: Desktop \ Server
>   * Feature: cmd_submission, media_scalability, multictx
> - * Functionality: load balancing
> - * Sub-category: Media & Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: hang
> - * Category: Desktop \ Server
> - * Feature: cmd_submission, media_scalability
> - * Functionality: load balancing
> - * Sub-category: Media & Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: hog
> - * Feature: cmd_submission, media_scalability
>   *
>   * SUBTEST: indices
> - * Category: Desktop \ Server
>   * Feature: cmd_submission, media_scalability, multictx
> - * Functionality: load balancing
> - * Sub-category: Media & Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: individual
> - * Category: Desktop \ Server
> - * Feature: cmd_submission, media_scalability
> - * Functionality: load balancing
> - * Sub-category: Media & Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: invalid-balancer
> - * Category: Desktop \ Server
> - * Feature: cmd_submission, media_scalability
> - * Functionality: load balancing
> - * Sub-category: Media & Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: invalid-bonds
> - * Feature: cmd_submission, media_scalability
>   *
>   * SUBTEST: nohangcheck
> - * Category: Desktop \ Server
> - * Feature: cmd_submission, media_scalability
> - * Functionality: load balancing
> - * Sub-category: Media & Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: noheartbeat
> - * Category: Desktop \ Server
> - * Feature: cmd_submission, media_scalability
> - * Functionality: load balancing
> - * Sub-category: Media & Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: nop
> - * Category: Desktop \ Server
> - * Feature: cmd_submission, media_scalability
> - * Functionality: load balancing
> - * Sub-category: Media & Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: parallel
> - * Category: Desktop client
> - * Feature: cmd_submission, media_scalability
> - * Functionality: command submission
> - * Sub-category: Media
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: parallel-balancer
> - * Category: Desktop client
> - * Feature: cmd_submission, media_scalability
> - * Functionality: command submission
> - * Sub-category: Media
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: parallel-bb-first
> - * Category: Desktop client
> - * Feature: cmd_submission, media_scalability
> - * Functionality: command submission
> - * Sub-category: Media
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: parallel-contexts
> - * Category: Desktop client
> - * Feature: cmd_submission, media_scalability
> - * Functionality: command submission
> - * Sub-category: Media
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: parallel-dmabuf-import-out-fence
> - * Description: Regression test to check that dmabuf imported sync file can handle fence array
> - * Feature: cmd_submission, media_scalability
> + * Description: Regression test to check that dmabuf imported sync file can
> + *		handle fence array
>   *
>   * SUBTEST: parallel-keep-in-fence
> - * Category: Desktop client
> - * Feature: cmd_submission, media_scalability
> - * Functionality: command submission
> - * Sub-category: Media
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: parallel-keep-submit-fence
> - * Category: Desktop client
> - * Feature: cmd_submission, media_scalability
> - * Functionality: command submission
> - * Sub-category: Media
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: parallel-ordering
> - * Category: Desktop client
> - * Feature: cmd_submission, media_scalability
> - * Functionality: command submission
> - * Sub-category: Media
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: parallel-out-fence
> - * Category: Desktop client
> - * Feature: cmd_submission, media_scalability
> - * Functionality: command submission
> - * Sub-category: Media
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: persistence
> - * Category: Desktop \ Server
> - * Feature: cmd_submission, media_scalability
> - * Functionality: load balancing
> - * Sub-category: Media & Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: semaphore
> - * Category: Desktop \ Server
>   * Feature: cmd_submission, media_scalability, multictx
> - * Functionality: load balancing
> - * Sub-category: Media & Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: sequential
> - * Category: Desktop \ Server
> - * Feature: cmd_submission, media_scalability
> - * Functionality: load balancing
> - * Sub-category: Media & Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: sliced
> - * Feature: cmd_submission, media_scalability
>   *
>   * SUBTEST: smoke
> - * Category: Desktop \ Server
> - * Feature: cmd_submission, media_scalability
> - * Functionality: load balancing
> - * Sub-category: Media & Compute
> - * Test category: GEM_Legacy
>   *
>   * SUBTEST: waits
> - * Category: Desktop \ Server
> - * Feature: cmd_submission, media_scalability
> - * Functionality: load balancing
> - * Sub-category: Media & Compute
> - * Test category: GEM_Legacy
>   */
>  
>  IGT_TEST_DESCRIPTION("Exercise in-kernel load-balancing");
> -- 
> 2.34.1
> 


More information about the igt-dev mailing list