[PATCH i-g-t v2 08/16] tests/intel: Introduce SysMan Mega feature

[Kamil Konieczny]

Hi Katarzyna,
On 2024-04-23 at 10:49:05 +0000, Katarzyna Piecielska wrote:
> Add tests to SysMan mega feature.
> Group:
> 	SUBTEST
> 	Description
> 	Category
> 	Mega feature
> 	Sub-category
> 	Functionality
> 
> Signed-off-by: Katarzyna Piecielska <katarzyna.piecielska at intel.com>
> Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> Cc: Mauro Carvalho Chehab <mauro.chehab at linux.intel.com>
> Cc: Aditya Chauhan <aditya.chauhan at intel.com>
> Cc: Girotra Himanshu <himanshu.girotra at intel.com>
> CC: Sandeep Kumar Parupalli <sandeep.kumar.parupalli at intel.com>
> ---
>  tests/device_reset.c                   | 3 ++-

'reset' is both in Functionality and Sub-Category:

grep -E -i ':.*reset' n.*
n.Functionality:../tests/device_reset.c: * Functionality: sysfs reset
n.Functionality:../tests/intel/gem_eio.c: * Functionality: reset
n.Functionality:../tests/intel/gem_reset_stats.c: * Functionality: reset
n.Functionality:../tests/intel/i915_selftest.c: * Functionality: engine/GT reset
n.Functionality:../tests/intel/xe_exec_reset.c: * Functionality: reset

n.Sub-category:../tests/device_reset.c: * Sub-category: Reset
n.Sub-category:../tests/intel/i915_selftest.c: * Sub-category: Reset

Regards,
Kamil

>  tests/intel/sysfs_heartbeat_interval.c | 7 ++++---
>  tests/intel/sysfs_preempt_timeout.c    | 7 ++++---
>  tests/intel/sysfs_timeslice_duration.c | 7 ++++---
>  tests/intel/xe_sysfs_defaults.c        | 6 ++++--
>  tests/intel/xe_sysfs_scheduler.c       | 3 ++-
>  6 files changed, 20 insertions(+), 13 deletions(-)
> 
> diff --git a/tests/device_reset.c b/tests/device_reset.c
> index de1a8a87b..9f77d0fce 100644
> --- a/tests/device_reset.c
> +++ b/tests/device_reset.c
> @@ -38,7 +38,8 @@
>   *   Unbinds driver from device, initiates reset then rebinds driver to device
>   *   validating device resets
>   * Category: Infrastructure
> - * Sub-category: SysMan
> + * Mega feature: SysMan
> + * Sub-category: SysMan tests
>   * Feature: reset, sriov-reset
>   */
>  
> diff --git a/tests/intel/sysfs_heartbeat_interval.c b/tests/intel/sysfs_heartbeat_interval.c
> index 5f82185bc..0af8879f6 100644
> --- a/tests/intel/sysfs_heartbeat_interval.c
> +++ b/tests/intel/sysfs_heartbeat_interval.c
> @@ -43,11 +43,12 @@
>  #include "sw_sync.h"
>  /**
>   * TEST: sysfs heartbeat interval
> - * Feature: SMI, cmd_submission
> - * Test category: SysMan
> - * Sub-category: SysMan
>   * Category: Desktop \ Server
> + * Mega feature: SysMan
> + * Sub-category: SysMan tests
>   * Functionality: sysfs heartbeat
> + * Feature: SMI, cmd_submission
> + * Test category: SysMan
>   *
>   * SUBTEST: idempotent
>   * Description: Test to check whether the heartbeat parameter reports the values set.
> diff --git a/tests/intel/sysfs_preempt_timeout.c b/tests/intel/sysfs_preempt_timeout.c
> index 84f8ba7d3..88f2cdfcd 100644
> --- a/tests/intel/sysfs_preempt_timeout.c
> +++ b/tests/intel/sysfs_preempt_timeout.c
> @@ -42,11 +42,12 @@
>  #include "sw_sync.h"
>  /**
>   * TEST: sysfs preempt timeout
> - * Feature: SMI, context
> - * Test category: SysMan
> - * Sub-category: SysMan
>   * Category: Desktop \ Server
> + * Mega feature: SysMan
> + * Sub-category: SysMan tests
>   * Functionality: sysfs preempt timeout
> + * Feature: SMI, context
> + * Test category: SysMan
>   *
>   * SUBTEST: idempotent
>   * Description: Test to check whether the preempt_timeout parameter reports the
> diff --git a/tests/intel/sysfs_timeslice_duration.c b/tests/intel/sysfs_timeslice_duration.c
> index b48152ecc..6baa000df 100644
> --- a/tests/intel/sysfs_timeslice_duration.c
> +++ b/tests/intel/sysfs_timeslice_duration.c
> @@ -45,11 +45,12 @@
>  #include "sw_sync.h"
>  /**
>   * TEST: sysfs timeslice duration
> + * Category: Desktop \ Server
> + * Mega feature: SysMan
> + * Sub-category: SysMan tests
> + * Functionality: timeslise sysfs
>   * Feature: SMI, cmd_submission
>   * Test category: SysMan
> - * Functionality: timeslise sysfs
> - * Sub-category: SysMan
> - * Category: Desktop \ Server
>   *
>   * SUBTEST: duration
>   * Description: Test to measure the timeslice duration by watching the
> diff --git a/tests/intel/xe_sysfs_defaults.c b/tests/intel/xe_sysfs_defaults.c
> index fd7dea77b..5c5db71a4 100644
> --- a/tests/intel/xe_sysfs_defaults.c
> +++ b/tests/intel/xe_sysfs_defaults.c
> @@ -5,12 +5,14 @@
>  
>  /**
>   * TEST: xe sysfs defaults
> + * Description: check if the sysfs engine .defaults node has all values.
>   * Category: Infrastructure
> + * Mega feature: SysMan
> + * Sub-category: SysMan tests
>   * Functionality: sysman defaults
> - * Sub-category: SysMan
>   * Test category: functionality test
> + *
>   * SUBTEST: engine-defaults
> - * Description: check if the sysfs engine .defaults node has all values.
>   */
>  
>  #include <dirent.h>
> diff --git a/tests/intel/xe_sysfs_scheduler.c b/tests/intel/xe_sysfs_scheduler.c
> index d1ef422f4..5c4fb1114 100644
> --- a/tests/intel/xe_sysfs_scheduler.c
> +++ b/tests/intel/xe_sysfs_scheduler.c
> @@ -6,7 +6,8 @@
>  /**
>   * TEST: xe sysfs scheduler
>   * Category: Infrastructure
> - * Sub-category: SysMan
> + * Mega feature: SysMan
> + * Sub-category: SysMan tests
>   * Functionality: scheduler control interface
>   *
>   * SUBTEST: %s-invalid
> -- 
> 2.34.1
>