[igt-dev] [PATCH i-g-t v2 3/4] i915/i915_pm_rpm: Add PC10 display off test

[Gupta, Anshuman]

> -----Original Message-----
> From: Chris Wilson <chris at chris-wilson.co.uk>
> Sent: Wednesday, February 3, 2021 9:48 PM
> To: Gupta, Anshuman <anshuman.gupta at intel.com>; igt-
> dev at lists.freedesktop.org
> Cc: Vivi, Rodrigo <rodrigo.vivi at intel.com>
> Subject: Re: [igt-dev] [PATCH i-g-t v2 3/4] i915/i915_pm_rpm: Add PC10
> display off test
> 
> Quoting Anshuman Gupta (2021-02-03 10:03:09)
> > Add a test to validate PC10 residencies while all output's display is
> > off.
> 
> The pc10 tests look nice and straightforward. Just a general question on
> whether 30s timeout is too lax or just right?
Thanks for review.
I had just followed timeout time from existing pc8 tests for HASWELL/BROADWEL
which was 30 second. It is not an engineering number.
 IMO i915 default auto-suspend ms time 10 sec
(cat /sys/devices/pci0000\:00/0000\:00\:02.0/power/autosuspend_delay_ms)
Considering above it may have some scope of optimization.
Could you please provide some input on it.
Thanks,
Anshuman Gupta.
> -Chris