Re: [igt-dev] [PATCH i-g-t] test/i915_pm_rpm: Remove igt_describe() from dynamic subtest

["Gupta, Anshuman" <anshuman.gupta@intel.com>]

> -----Original Message-----
> From: Latvala, Petri <petri.latvala@intel.com>
> Sent: Wednesday, March 16, 2022 3:33 PM
> To: Nilawar, Badal <badal.nilawar@intel.com>
> Cc: Gupta, Anshuman <anshuman.gupta@intel.com>; igt-
> dev@lists.freedesktop.org
> Subject: Re: [igt-dev] [PATCH i-g-t] test/i915_pm_rpm: Remove igt_describe()
> from dynamic subtest
> 
> On Wed, Mar 16, 2022 at 11:28:54AM +0530, Nilawar, Badal wrote:
> >
> >
> > On 16-03-2022 10:30, Anshuman Gupta wrote:
> > > Remove igt_describe() from gem-execbuf-stress() dynamic subtest to
> > > avoid the igt internal assert.
> > >
> > > Fixes: aa6eb64bac51 ("i915_pm_rpm: Add placement to
> > > gem_exec_stress")
> > > Signed-off-by: Anshuman Gupta <anshuman.gupta@intel.com>
> > > ---
> > >   tests/i915/i915_pm_rpm.c | 6 ++----
> > >   1 file changed, 2 insertions(+), 4 deletions(-)
> > >
> > > diff --git a/tests/i915/i915_pm_rpm.c b/tests/i915/i915_pm_rpm.c
> > > index 8ae407e50..461730e8a 100644
> > > --- a/tests/i915/i915_pm_rpm.c
> > > +++ b/tests/i915/i915_pm_rpm.c
> > > @@ -2170,14 +2170,12 @@ igt_main_args("", long_options, help_str,
> opt_handler, NULL)
> > >   		system_suspend_subtest(SUSPEND_STATE_DISK,
> SUSPEND_TEST_NONE);
> > >   	/* GEM stress */
> > > +	igt_describe("Validate execbuf submission while exercising rpm "
> > > +		     "suspend/resume cycles.");
> > This change looks fine.
> > Reviewed-by: Badal Nilawar <badal.nilawar@intel.com>
> 
> 
> Merged, thanks!
Thanks Badal and Petri for review and merge.
Br,
Anshuman Gupta.
> 
> 
> --
> Petri Latvala