[igt-dev] [PATCH i-g-t] i915_pm_rpm: Use __igt_debugfs_dump for i915_runtime_pm_status

Dixit, Ashutosh ashutosh.dixit at intel.com
Wed Feb 23 15:57:49 UTC 2022


On Wed, 23 Feb 2022 01:59:51 -0800, Anshuman Gupta wrote:
>
> Use __igt_debugfs_dump() in wait_for_suspended() in order to
> specify explicit igt log level.
> We need to dump the i915_runtime_pm_status even in case of igt
> test's skips as well, that requires log level to be IGT_LOG_INFO.

Ok, by skip I believe you are referring to this:

	igt_require(setup_environment(false));

Therefore:

Reviewed-by: Ashutosh Dixit <ashutosh.dixit at intel.com>

> Signed-off-by: Anshuman Gupta <anshuman.gupta at intel.com>
> ---
>  tests/i915/i915_pm_rpm.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/i915/i915_pm_rpm.c b/tests/i915/i915_pm_rpm.c
> index 6c8ffbd5d..eb4c4f4e3 100644
> --- a/tests/i915/i915_pm_rpm.c
> +++ b/tests/i915/i915_pm_rpm.c
> @@ -207,7 +207,7 @@ static bool wait_for_suspended(void)
>		bool suspended = igt_wait_for_pm_status(IGT_RUNTIME_PM_STATUS_SUSPENDED);
>
>		if (!suspended)
> -			igt_debugfs_dump(drm_fd, "i915_runtime_pm_status");
> +			__igt_debugfs_dump(drm_fd, "i915_runtime_pm_status", IGT_LOG_INFO);
>
>		return suspended;
>	}
> --
> 2.26.2
>


More information about the igt-dev mailing list