> -----Original Message-----
> From: Steven Lariau <[email protected]>
> Sent: Wednesday, August 5, 2020 10:57 AM
> To: Eads, Gage <[email protected]>; Olivier Matz
> <[email protected]>
> Cc: [email protected]; [email protected];
> [email protected]; [email protected]; Steven Lariau
> <[email protected]>
> Subject: [PATCH 4/4] test/stack: remove atomics operations
>
> Remove the part that checks if there is enough room in the stack, it's
> always true as long as size of stack >= MAX_BULK*rte_lcore_count().
> This check used an atomic cmpset, and read / write to a shared size
> variable. These operations result in some form of synchronization
> that might get in the way of the actual stack testing.
>
> Signed-off-by: Steven Lariau <[email protected]>
> Reviewed-by: Dharmik Thakkar <[email protected]>
> Reviewed-by: Phil Yang <[email protected]>
> Reviewed-by: Ruifeng Wang <[email protected]>
Nice simplification, and thanks for the good contributions to these tests.
Acked-by: Gage Eads <[email protected]
