Re: [PATCH i-g-t] tests/intel/i915_pm_rps: Disable subtest thresholds tests when SLPC is enabled

[Riana Tauro <riana.tauro@intel.com>]

On 7/25/2024 5:17 PM, sk.anirban@intel.com wrote:
> From: Sk Anirban <sk.anirban@intel.com>
> 
> Skip these tests when SLPC is enabled as it fails frequently. These tests
> are designed for host turbo and sysfs is not available for SLPC.
                                                         ^ when SLPC is 
enabled
> 
> Closes: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/11455
> 
> v2: Add a skip all the threesolds subtests
typo
%s/threesolds/threshold
>      provide explanations for failures (Riana)
use either uppercase or lowercase
>      Addressed cosmetic review comments (Riana)


With the above comments fixed
Reviewed-by: Riana Tauro <riana.tauro@intel.com>
> 
> Signed-off-by: Sk Anirban <sk.anirban@intel.com>
> ---
>   tests/intel/i915_pm_rps.c | 11 ++++++++---
>   1 file changed, 8 insertions(+), 3 deletions(-)
> 
> diff --git a/tests/intel/i915_pm_rps.c b/tests/intel/i915_pm_rps.c
> index 99b83f315..ace22b00c 100644
> --- a/tests/intel/i915_pm_rps.c
> +++ b/tests/intel/i915_pm_rps.c
> @@ -1161,7 +1161,8 @@ igt_main
>   
>   	igt_subtest_with_dynamic("thresholds-idle") {
>   		int tmp, gt;
> -
> +		igt_skip_on_f(i915_is_slpc_enabled(drm_fd),
> +			      "This subtest is not supported when SLPC is enabled\n");
>   		i915_for_each_gt(drm_fd, tmp, gt) {
>   			igt_dynamic_f("gt%u", gt)
>   				test_thresholds(drm_fd, gt, TEST_IDLE);
> @@ -1170,7 +1171,8 @@ igt_main
>   
>   	igt_subtest_with_dynamic("thresholds") {
>   		int tmp, gt;
> -
> +		igt_skip_on_f(i915_is_slpc_enabled(drm_fd),
> +			      "This subtest is not supported when SLPC is enabled\n");
>   		i915_for_each_gt(drm_fd, tmp, gt) {
>   			igt_dynamic_f("gt%u", gt)
>   				test_thresholds(drm_fd, gt, 0);
> @@ -1179,7 +1181,8 @@ igt_main
>   
>   	igt_subtest_with_dynamic("thresholds-park") {
>   		int tmp, gt;
> -
> +		igt_skip_on_f(i915_is_slpc_enabled(drm_fd),
> +			      "This subtest is not supported when SLPC is enabled\n");
>   		i915_for_each_gt(drm_fd, tmp, gt) {
>   			igt_dynamic_f("gt%u", gt)
>   				test_thresholds(drm_fd, gt, TEST_PARK);
> @@ -1188,6 +1191,8 @@ igt_main
>   
>   	igt_subtest_with_dynamic("thresholds-idle-park") {
>   		int tmp, gt;
> +		igt_skip_on_f(i915_is_slpc_enabled(drm_fd),
> +			      "This subtest is not supported when SLPC is enabled\n");
>   
>   		i915_for_each_gt(drm_fd, tmp, gt) {
>   			igt_dynamic_f("gt%u", gt)