[igt-dev] [PATCH i-g-t v3 17/52] tests/gem_eio: Adopt to use allocator

[Zbigniew Kempczyński]

On Thu, Aug 05, 2021 at 02:44:39PM -0700, Dixit, Ashutosh wrote:
> On Mon, 26 Jul 2021 12:59:51 -0700, Zbigniew Kempczyński wrote:
> >
> > -static void reset_stress(int fd, const intel_ctx_t *ctx0,
> > +static void reset_stress(int fd, uint64_t ahnd, const intel_ctx_t *ctx0,
> 
> I think it would have been ok to allocate ahnd inside reset_stress() but
> looks like we wanted to keep it tied to ctx_create/destroy, so this is:

Yes, spin_sync() is running within ctx0 so I would keep ahnd for ctx0
for whole test (test_reset_stress() goes over rings within ctx0 so they
share vm). If we would recreate ahnd for ctx0 for each run we would 
just evict vmas from previous call.

--
Zbigniew

> 
> Reviewed-by: Ashutosh Dixit <ashutosh.dixit at intel.com>