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

[Nilawar, Badal]

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>
>   	igt_subtest_with_dynamic("gem-execbuf-stress") {
>   		for_each_memory_region(r, drm_fd) {
> -			igt_describe("Validate execbuf submission while exercising rpm "
> -				     "suspend/resume cycles.");
>   			igt_dynamic_f("%s", r->name)
>   				gem_execbuf_stress_subtest(rounds, WAIT_STATUS, &r->ci);
> -			igt_describe("Validate execbuf submission while exercising rpm "
> -				     "suspend/resume cycles with extra wait.");
>   			igt_dynamic_f("%s-%s", "extra-wait", r->name)
>   				gem_execbuf_stress_subtest(rounds, WAIT_STATUS | WAIT_EXTRA, &r->ci);
>   		}