[PATCH i-g-t v2 04/16] tests/intel: Introduce Power management Mega feature
Kamil Konieczny
kamil.konieczny at linux.intel.com
Thu Apr 25 17:11:43 UTC 2024
Hi Katarzyna,
On 2024-04-23 at 10:49:01 +0000, Katarzyna Piecielska wrote:
> Add mega feature to Power Management tests.
> Group:
> SUBTEST
> Description
> Category
> Mega feature
> Sub-category
> Functionality
>
> v2: move gem_pxp to General Mega feature
>
> 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/intel/gem_ctx_freq.c | 5 ++--
> tests/intel/i915_pm_freq_mult.c | 7 +++---
grep -E -i ':.*firmware' n.*
n.Category:../tests/intel/i915_pm_freq_mult.c: * Category: Firmware
n.Category:../tests/intel/xe_huc_copy.c: * Category: Firmware building block
n.Sub-category:../tests/intel/gem_huc_copy.c: * Sub-category: Firmware
n.Sub-category:../tests/intel/i915_selftest.c: * Sub-category: Firmware
n.Sub-category:../tests/intel/xe_huc_copy.c: * Sub-category: Firmware
so there is conflict where 'Firmware' is both in Category
and Sub-category.
Regards,
Kamil
> tests/intel/i915_pm_rc6_residency.c | 7 +++---
> tests/intel/i915_pm_rpm.c | 7 +++---
> tests/intel/i915_pm_rps.c | 7 +++---
> tests/intel/i915_pm_sseu.c | 5 ++--
> tests/intel/i915_power.c | 7 +++---
> tests/intel/i915_selftest.c | 38 +++++++++++++++--------------
> tests/intel/i915_suspend.c | 8 +++---
> tests/intel/xe_gt_freq.c | 3 ++-
> tests/intel/xe_pm.c | 3 ++-
> tests/intel/xe_pm_residency.c | 3 ++-
> 12 files changed, 56 insertions(+), 44 deletions(-)
>
> diff --git a/tests/intel/gem_ctx_freq.c b/tests/intel/gem_ctx_freq.c
> index db67cade5..c348fe338 100644
> --- a/tests/intel/gem_ctx_freq.c
> +++ b/tests/intel/gem_ctx_freq.c
> @@ -40,9 +40,10 @@
> /**
> * TEST: gem ctx freq
> * Category: Server
> - * Feature: context feature
> + * Mega feature: Power management
> + * Sub-category: Power management tests
> * Functionality: frequency management
> - * Sub-category: Performance
> + * Feature: context feature
> * Test category: GEM_Legacy
> *
> * SUBTEST: sysfs
> diff --git a/tests/intel/i915_pm_freq_mult.c b/tests/intel/i915_pm_freq_mult.c
> index 3c390d427..6d375c5aa 100644
> --- a/tests/intel/i915_pm_freq_mult.c
> +++ b/tests/intel/i915_pm_freq_mult.c
> @@ -14,10 +14,11 @@
> * TEST: i915 pm freq mult
> * Description: Tests for sysfs controls (or multipliers) for IP blocks which
> * run at frequencies different from the main GT frequency.
> - * Category: Firmware building block
> - * Functionality: freq multipliers
> + * Category: Firmware
> + * Mega feature: Power management
> + * Sub-category: Power management tests
> + * Functionality: freg multipliers
> * Test category: i915
> - * Sub-category: Power management
> *
> * SUBTEST: media-freq
> * Description: Tests for media frequency factor sysfs
> diff --git a/tests/intel/i915_pm_rc6_residency.c b/tests/intel/i915_pm_rc6_residency.c
> index be9f9117a..92dae43b3 100644
> --- a/tests/intel/i915_pm_rc6_residency.c
> +++ b/tests/intel/i915_pm_rc6_residency.c
> @@ -43,10 +43,11 @@
> #include "sw_sync.h"
> /**
> * TEST: i915 pm rc6 residency
> - * Feature: GuCRC, pm_rc6
> - * Functionality: rc6
> - * Sub-category: Power management
> * Category: Desktop client
> + * Mega feature: Power management
> + * Sub-category: Power management tests
> + * Functionality: rc6
> + * Feature: GuCRC, pm_rc6
> *
> * SUBTEST: media-rc6-accuracy
> * Feature: pm_rc6
> diff --git a/tests/intel/i915_pm_rpm.c b/tests/intel/i915_pm_rpm.c
> index 8ea3fd315..52e00489d 100644
> --- a/tests/intel/i915_pm_rpm.c
> +++ b/tests/intel/i915_pm_rpm.c
> @@ -42,11 +42,12 @@
> #include <sys/stat.h>
> /**
> * TEST: i915 pm rpm
> - * Feature: pm_rpm
> - * Functionality: D3 state entry-exit
> - * Sub-category: Power management
> * Category: Desktop client
> + * Mega feature: Power management
> + * Sub-category: Power management tests
> + * Functionality: D3 state entry-exit
> * Test category: pm_rpm
> + * Feature: pm_rpm
> *
> * SUBTEST: debugfs-forcewake-user
> * SUBTEST: debugfs-read
> diff --git a/tests/intel/i915_pm_rps.c b/tests/intel/i915_pm_rps.c
> index 648765b02..4ab137282 100644
> --- a/tests/intel/i915_pm_rps.c
> +++ b/tests/intel/i915_pm_rps.c
> @@ -47,11 +47,12 @@
> /**
> * TEST: i915 pm rps
> * Description: Render P-States tests - verify GPU frequency changes
> + * Category: Desktop client
> + * Mega feature: Power management
> + * Sub-category: Power management tests
> + * Functionality: pm_rps
> * Feature: pm_rps
> * Test category: pm_rps
> - * Functionality: rps
> - * Sub-category: Power management
> - * Category: Desktop client
> *
> * SUBTEST: basic-api
> *
> diff --git a/tests/intel/i915_pm_sseu.c b/tests/intel/i915_pm_sseu.c
> index 11df4f06c..1fb938893 100644
> --- a/tests/intel/i915_pm_sseu.c
> +++ b/tests/intel/i915_pm_sseu.c
> @@ -38,10 +38,11 @@
> /**
> * TEST: i915 pm sseu
> * Description: Tests slice/subslice/EU power gating functionality.
> - * Feature: pm_sseu
> - * Sub-category: Power management
> * Category: Desktop client
> + * Mega feature: Power management
> + * Sub-category: Power management tests
> * Functionality: sseu
> + * Feature: pm_sseu
> *
> * SUBTEST: full-enable
> */
> diff --git a/tests/intel/i915_power.c b/tests/intel/i915_power.c
> index fe8437a0f..ef92efb6f 100644
> --- a/tests/intel/i915_power.c
> +++ b/tests/intel/i915_power.c
> @@ -10,11 +10,12 @@
> /**
> * TEST: i915 power
> * Description: i915 power measurement tests
> - * Feature: hwmon
> - * Test category: GEM_Legacy
> - * Sub-category: Power management
> * Category: Desktop client
> + * Mega feature: Power management
> + * Sub-category: Power management tests
> * Functionality: hwmon
> + * Feature: hwmon
> + * Test category: GEM_Legacy
> *
> * SUBTEST: sanity
> * Description: Sanity check gpu power measurement
> diff --git a/tests/intel/i915_selftest.c b/tests/intel/i915_selftest.c
> index a673e9b03..cdaaaf635 100644
> --- a/tests/intel/i915_selftest.c
> +++ b/tests/intel/i915_selftest.c
> @@ -174,10 +174,11 @@
> * Sub-category: Mocs
> *
> * SUBTEST: live at gt_pm
> - * Category: Selftest
> * Description: Basic i915 driver module selftests
> + * Category: Selftest
> + * Mega feature: Power management
> + * Sub-category: Power management tests
> * Functionality: rps, rc6
> - * Sub-category: Power management
> *
> * SUBTEST: live at gt_timelines
> * Category: Selftest
> @@ -252,11 +253,12 @@
> * Test category: i915
> *
> * SUBTEST: live at late_gt_pm
> + * Description: Basic i915 driver module selftests
> * Category: Selftest
> - * Feature: rc6 feature
> - * Functionality: Basic i915 driver module selftests
> - * Test category: rc6
> - * Sub-category: Power management
> + * Mega feature: Power management
> + * Sub-category: Power management tests
> + * Feature: rc6
> + * Functionality: rc6
> *
> * SUBTEST: live at lmem
> * Description: Basic i915 driver module selftests
> @@ -353,20 +355,20 @@
> * Sub-category: Synchronization
> *
> * SUBTEST: live at slpc
> - * Category: Selftest
> - * Feature: slpc feature/ pm_rps
> - * Test category: slpc / pm_rps
> * Description: Basic i915 driver module selftests
> + * Category: Selftest
> + * Mega feature: Power management
> + * Sub-category: Power management tests
> * Functionality: slpc
> - * Sub-category: Power management
> + * Feature: slpc / pm_rps
> *
> * SUBTEST: live at uncore
> - * Category: Selftest
> * Description: Basic i915 driver module selftests
> - * Feature: forcewake feature
> - * Test category: forcewake
> + * Category: Selftest
> + * Mega feature: Power management
> + * Sub-category: Power management tests
> * Functionality: forcewake
> - * Sub-category: Power management
> + * Feature: forcewake
> *
> * SUBTEST: live at vma
> * Description: Per-object virtual address management
> @@ -532,12 +534,12 @@
> * Test category: Memory Management
> *
> * SUBTEST: mock at uncore
> - * Category: Selftest
> * Description: Basic i915 driver module selftests
> - * Feature: forcewake feature
> - * Test category: forcewake
> + * Category: Selftest
> + * Mega feature: Power management
> + * Sub-category: Power management tests
> * Functionality: forcewake
> - * Sub-category: Power management
> + * Feature: forcewake
> *
> * SUBTEST: mock at vma
> * Category: Selftest
> diff --git a/tests/intel/i915_suspend.c b/tests/intel/i915_suspend.c
> index ae4404101..1f99c2ddc 100644
> --- a/tests/intel/i915_suspend.c
> +++ b/tests/intel/i915_suspend.c
> @@ -46,12 +46,12 @@
> #include "igt_device_scan.h"
> /**
> * TEST: i915 suspend
> - * Feature: suspend feature
> + * Category: Desktop client
> + * Mega feature: Power management
> + * Sub-category: Power management tests
> * Functionality: s2idle w/o i915
> * Test category: suspend
> - * Functionality: suspend
> - * Sub-category: Power management
> - * Category: Desktop client
> + * Feature: suspend
> *
> * SUBTEST: basic-s2idle-without-i915
> * Description: Validate suspend-to-idle without i915 module
> diff --git a/tests/intel/xe_gt_freq.c b/tests/intel/xe_gt_freq.c
> index c12f6c52c..0c1acc688 100644
> --- a/tests/intel/xe_gt_freq.c
> +++ b/tests/intel/xe_gt_freq.c
> @@ -6,7 +6,8 @@
> /**
> * TEST: Test Xe GT frequency request functionality
> * Category: Infrastructure
> - * Sub-category: Power management
> + * Mega feature: Power management
> + * Sub-category: Power management tests
> * Functionality: frequency request
> * Test category: functionality test
> */
> diff --git a/tests/intel/xe_pm.c b/tests/intel/xe_pm.c
> index 51442537b..24cdeb7ed 100644
> --- a/tests/intel/xe_pm.c
> +++ b/tests/intel/xe_pm.c
> @@ -6,7 +6,8 @@
> /**
> * TEST: Check Power Management functionality
> * Category: Software building block
> - * Sub-category: Power management
> + * Mega feature: Power management
> + * Sub-category: Power management tests
> * Test category: functionality test
> */
>
> diff --git a/tests/intel/xe_pm_residency.c b/tests/intel/xe_pm_residency.c
> index ad70e2430..d58ebe804 100644
> --- a/tests/intel/xe_pm_residency.c
> +++ b/tests/intel/xe_pm_residency.c
> @@ -6,7 +6,8 @@
> /**
> * TEST: Test gtidle properties
> * Category: Software building block
> - * Sub-category: Power management
> + * Mega feature: Power management
> + * Sub-category: Power management tests
> * Functionality: GT C States
> * Test category: functionality test
> */
> --
> 2.34.1
>
More information about the igt-dev
mailing list