From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by gabe.freedesktop.org (Postfix) with ESMTPS id A2AE510E6BD for ; Wed, 23 Feb 2022 15:57:54 +0000 (UTC) Date: Wed, 23 Feb 2022 07:57:49 -0800 Message-ID: <87tucp60ea.wl-ashutosh.dixit@intel.com> From: "Dixit, Ashutosh" To: Anshuman Gupta In-Reply-To: <20220223095951.12430-1-anshuman.gupta@intel.com> References: <20220223095951.12430-1-anshuman.gupta@intel.com> MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=US-ASCII Subject: Re: [igt-dev] [PATCH i-g-t] i915_pm_rpm: Use __igt_debugfs_dump for i915_runtime_pm_status List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: igt-dev@lists.freedesktop.org, petri.latvala@intel.com, badal.nilawar@intel.com Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: 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 > Signed-off-by: Anshuman Gupta > --- > 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 >