> -----Original Message-----
> From: David Marchand <david.march...@redhat.com>
> Sent: Monday, September 14, 2020 9:37 AM
> To: Van Haaren, Harry <harry.van.haa...@intel.com>
> Cc: dev <dev@dpdk.org>; Igor Romanov <igor.roma...@oktetlabs.ru>; Honnappa
> Nagarahalli <honnappa.nagaraha...@arm.com>; Yigit, Ferruh
> <ferruh.yi...@intel.com>; nd <n...@arm.com>; Aaron Conole
> <acon...@redhat.com>; Lukasz Wojciechowski
> <l.wojciec...@partner.samsung.com>; Phil Yang <phil.y...@arm.com>
> Subject: Re: [PATCH v5 2/2] test/service: fix race condition on stopping lcore
>
> On Fri, Jul 24, 2020 at 3:44 PM Harry van Haaren
> <harry.van.haa...@intel.com> 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 <david.march...@redhat.com>
> > Signed-off-by: Harry van Haaren <harry.van.haa...@intel.com>
> > Reviewed-by: Phil Yang <phil.y...@arm.com>
> > Reviewed-by: Honnappa Nagarahalli <honnappa.nagaraha...@arm.com>
>
> There is still a compilation issue reported by the Intel CI.
> http://mails.dpdk.org/archives/test-report/2020-July/146535.html
>
> Please fix.
Done, v6 sent; http://patches.dpdk.org/project/dpdk/list/?series=12198
