On Mon, Sep 14, 2020 at 4:30 PM Harry van Haaren <[email protected]> wrote: > > This commit fixes a potential race condition in the tests > where the lcore running a service would increment a counter > that was already reset by the test-suite thread. The resulting > race-condition incremented value could cause CI failures, as > indicated by DPDK's CI. > > This patch fixes the race-condition by making use of the > added rte_service_lcore_active() API, which indicates when > a service-core is no longer in the service-core polling loop. > > The unit test makes use of the above function to detect when > all statistics increments are done in the service-core thread, > and then the unit test continues finalizing and checking state. > > Fixes: f28f3594ded2 ("service: add attribute API") > > Reported-by: David Marchand <[email protected]> > Signed-off-by: Harry van Haaren <[email protected]> > Reviewed-by: Phil Yang <[email protected]> > Reviewed-by: Honnappa Nagarahalli <[email protected]>
Series applied, thanks. -- David Marchand
