Re: [igt-dev] [PATCH i-g-t v3 17/52] tests/gem_eio: Adopt to use allocator
From: Zbigniew Kempczyński <hidden>
Date: 2021-08-06 07:16:51
From: Zbigniew Kempczyński <hidden>
Date: 2021-08-06 07:16:51
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:quoted
-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 <redacted>