Re: [PATCH i-g-t] tests/intel/xe_pm: Disable display for mocs-rpm test

[Rodrigo Vivi <rodrigo.vivi@intel.com>]

On Thu, May 16, 2024 at 11:21:45AM -0500, Lucas De Marchi wrote:
> On Thu, May 16, 2024 at 04:15:13PM GMT, janga.rahul.kumar@intel.com wrote:
> > From: Janga Rahul Kumar <janga.rahul.kumar@intel.com>
> > 
> > If display is enabled, the always-on components will always keeps
> > runtime pm references, unless you put DPMS_OFF.
> > 
> > Cc: Badal Nilawar <badal.nilawar@intel.com>
> > Cc: Lucas De Marchi <lucas.demarchi@intel.com>
> > Signed-off-by: Janga Rahul Kumar <janga.rahul.kumar@intel.com>
> 
> I think this is
> 
> 	Closes: https://gitlab.freedesktop.org/drm/xe/kernel/-/issues/1818

very likely yes

> 
> right?
> 
> Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
> 
> thanks
> Lucas De Marchi
> 
> > ---
> > tests/intel/xe_pm.c | 5 ++++-
> > 1 file changed, 4 insertions(+), 1 deletion(-)
> > 
> > diff --git a/tests/intel/xe_pm.c b/tests/intel/xe_pm.c
> > index 9cb00c7f1..b4a8c4d15 100644
> > --- a/tests/intel/xe_pm.c
> > +++ b/tests/intel/xe_pm.c
> > @@ -835,8 +835,11 @@ igt_main
> > 			igt_pm_set_autosuspend_delay(device.pci_xe, delay_ms);
> > 		}
> > 
> > -		igt_subtest("mocs-rpm")
> > +		igt_subtest("mocs-rpm") {
> > +			dpms_on_off(device, DRM_MODE_DPMS_OFF);
> > 			test_mocs_suspend_resume(device, NO_SUSPEND);
> > +			dpms_on_off(device, DRM_MODE_DPMS_ON);
> > +		}
> > 	}
> > 
> > 	igt_fixture {
> > -- 
> > 2.25.1
> >