[igt-dev] [PATCH i-g-t v6 1/1] i915_pm_rpm: remove gem-execbuf-stress-extra-wait because same as gem-execbuf-stress

[Chris Wilson]

Quoting Caz Yokoyama (2019-03-29 22:26:13)
> Extra 5sec delay does not add any value more than gem-execbuf-stress.
> It waits until suspend state after a job is added by gem_execbuf().
> There is no need to do more when GPU becomes suspended state.
> I confirm this by looking at pm_runtime_force_suspend() which exits
> on suspend state.
> 
> Signed-off-by: Caz Yokoyama <caz.yokoyama at intel.com>
> Cc: Chris Wilson <chris at chris-wilson.co.uk>

There is no difference wrt to the driver that extra-wait would cover
that isn't covered by the other execbuf-stress variants. The wait is
pure paranoia that the HW does something underneath the covers that
interferes with us. The likelihood of this test even spotting that?
Slim. Still I can't rule out that it may, so useful to keep around for
investigations.

Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
-Chris