[dpdk-dev] [PATCH v2 1/2] service: add API to retrieve service core active

[Phil Yang]

> -----Original Message-----
> From: dev <dev-bounces at dpdk.org> On Behalf Of Harry van Haaren
> Sent: Monday, July 20, 2020 10:38 PM
> To: dev at dpdk.org
> Cc: david.marchand at redhat.com; igor.romanov at oktetlabs.ru; Honnappa
> Nagarahalli <Honnappa.Nagarahalli at arm.com>; ferruh.yigit at intel.com; nd
> <nd at arm.com>; aconole at redhat.com;
> l.wojciechow at partner.samsung.com; Harry van Haaren
> <harry.van.haaren at intel.com>
> Subject: [dpdk-dev] [PATCH v2 1/2] service: add API to retrieve service core
> active
> 
> This commit adds a new experimental API which allows the user
> to retrieve the active state of an lcore. Knowing when the service
> lcore is completed its polling loop can be useful to applications
> to avoid race conditions when e.g. finalizing statistics.
> 
> The service thread itself now has a variable to indicate if its
> thread is active. When zero the service thread has completed its
> service, and has returned from the service_runner_func() function.
> 
> Suggested-by: Lukasz Wojciechowski <l.wojciechow at partner.samsung.com>
> Signed-off-by: Harry van Haaren <harry.van.haaren at intel.com>
> 

It looks good to me.

Reviewed-by: Phil Yang <phil.yang at arm.com>

Thanks,
Phil
<...>