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

[Kumar, Janga Rahul]

> -----Original Message-----
> From: Vivi, Rodrigo <rodrigo.vivi at intel.com>
> Sent: Friday, May 17, 2024 1:04 AM
> To: De Marchi, Lucas <lucas.demarchi at intel.com>
> Cc: Kumar, Janga Rahul <janga.rahul.kumar at intel.com>; igt-
> dev at lists.freedesktop.org; Gandi, Ramadevi <ramadevi.gandi at intel.com>;
> Nilawar, Badal <badal.nilawar at intel.com>
> Subject: Re: [PATCH i-g-t] tests/intel/xe_pm: Disable display for mocs-rpm test
> 
> 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 at intel.com
> wrote:
> > > From: Janga Rahul Kumar <janga.rahul.kumar at 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 at intel.com>
> > > Cc: Lucas De Marchi <lucas.demarchi at intel.com>
> > > Signed-off-by: Janga Rahul Kumar <janga.rahul.kumar at intel.com>
> >
> > I think this is
> >
> > 	Closes: https://gitlab.freedesktop.org/drm/xe/kernel/-/issues/1818
Yes

Janga Rahul Kumar
> 
> very likely yes
> 
> >
> > right?
> >
> > Reviewed-by: Lucas De Marchi <lucas.demarchi at 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
> > >