> -----Original Message----- > From: Laatz, Kevin <kevin.la...@intel.com> > Sent: Friday, October 16, 2020 10:08 AM > To: dev@dpdk.org > Cc: Van Haaren, Harry <harry.van.haa...@intel.com>; > david.march...@redhat.com; l.wojciec...@partner.samsung.com; > honnappa.nagaraha...@arm.com; phil.y...@arm.com; acon...@redhat.com; > Laatz, Kevin <kevin.la...@intel.com> > Subject: [PATCH] test/service: fix race condition on stopping lcore > > There is a potential race condition in 'service_attr_get' which will cause > test failures since the service core thread is still running while the > values are being retrieved/reset. > > This patch fixes the race condition by waiting for the service core thread > to stop before continuing with the unit test checks. > > Signed-off-by: Kevin Laatz <kevin.la...@intel.com>
Thanks Kevin for handling; can't reproduce race-cond here, but by code review this is the correct fix, thanks also for refactoring the wait into its own function. Acked-by: Harry van Haaren <harry.van.haa...@intel.com>
