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

Petri Latvala petri.latvala at intel.com
Wed Mar 16 10:03:27 UTC 2022


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 at 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 at intel.com>


Merged, thanks!


-- 
Petri Latvala


More information about the igt-dev mailing list