[Intel-gfx] [PATCH i-g-t 3/3] tests/gem_reset_stats: Enforce full chip reset mode before run

Michel Thierry michel.thierry at intel.com
Thu Jul 6 17:29:41 UTC 2017


On 06/07/17 04:12, Arkadiusz Hiler wrote:
> On Tue, Jun 20, 2017 at 11:25:02AM -0700, Michel Thierry wrote:
>> Platforms with per-engine reset enabled (i915.reset=2) are unlikely to
>> perform a full chip reset, keeping the reset_count unmodified. In order
>> to keep the expectations of this test, enforce that full GPU reset is
>> enabled (i915.reset=1).
>>
>> Later on, we can expand the reset_stats ioctl to also return the number
>> of per-engine resets and use reset_count + reset_engine_count when
>> checking for the updated reset count.
>>
>> Signed-off-by: Michel Thierry <michel.thierry at intel.com>
> 
> This no longer applies due to changes in the context. It would be nice
> if you would send rebased version as well :-)

Hi Arek,

I think the v2 of these patches still apply cleanly,

https://patchwork.freedesktop.org/patch/164248/ and
https://patchwork.freedesktop.org/patch/164249/

> 
> @Antonio: any optionion on the patches? They LGTM, but an opinion of
> someone more informed wouldn't hurt.
> 

Indeed.

-Michel


More information about the Intel-gfx mailing list