Thread (102 messages) 102 messages, 3 authors, 2021-08-06

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

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>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help