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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jul 06, 2017 at 10:29:41AM -0700, Michel Thierry wrote:
> 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@xxxxxxxxx>
> > 
> > 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/

Since you've said that you have a consensus with Antionio, and the
patches LGTM.

Acked-by: Arkadiusz Hiler <arkadiusz.hiler@xxxxxxxxx>

Pushed, thanks!
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux