On Wed, Aug 12, 2020 at 9:20 PM Steven Lariau <steven.lar...@arm.com> wrote: > > The current multithread DPDK stack test is using atomics operations to > share information between threads. > The lockfree stack implementation also uses atomic operations. > This is an issue for testing. The atomics operations for the test may > add some extra synchronization to the stack implementation, > that doesn't exist. > It makes it harder to find bugs related to memory orderings and data > races. The main goal of the patch is to remove all atomics operations > and any other form of data sharing in this test, to make sure that > most of the execution time is spent on the stack library. > > Furthermore, this patch uses more appropriate functions to start / > wait cores in order to simplify the code. > The patch also adds code to propagate errors on any slave core to the > master. > > v2: replace stack variable for arguments with a global variable. > > Steven Lariau (4): > test/stack: avoid trivial memory allocations > test/stack: launch tests with mp remote launch API > test/stack: propagate errors to main core > test/stack: remove atomics operations > > app/test/test_stack.c | 80 +++++++++---------------------------------- > 1 file changed, 17 insertions(+), 63 deletions(-) >
Series applied, thanks Steven. -- David Marchand