> -----Original Message-----
> From: Steven Lariau <steven.lar...@arm.com>
> Sent: Wednesday, August 5, 2020 10:57 AM
> To: Eads, Gage <gage.e...@intel.com>; Olivier Matz
> <olivier.m...@6wind.com>
> Cc: dev@dpdk.org; honnappa.nagaraha...@arm.com;
> dharmik.thak...@arm.com; n...@arm.com; Steven Lariau
> <steven.lar...@arm.com>
> 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 <steven.lar...@arm.com>
> Reviewed-by: Dharmik Thakkar <dharmik.thak...@arm.com>
> Reviewed-by: Phil Yang <phil.y...@arm.com>
> Reviewed-by: Ruifeng Wang <ruifeng.w...@arm.com>
Nice simplification, and thanks for the good contributions to these tests.
Acked-by: Gage Eads <gage.e...@intel.com