On Mon, Oct 7, 2019 at 5:37 PM Van Haaren, Harry <[email protected]> wrote: > > > -----Original Message----- > > From: David Marchand [mailto:[email protected]] > > Sent: Monday, October 7, 2019 3:53 PM > > To: Van Haaren, Harry <[email protected]> > > Cc: dev <[email protected]>; Rao, Nikhil <[email protected]> > > Subject: Re: [dpdk-dev] [PATCH] service: improve service run performance > > > > On Mon, Sep 16, 2019 at 12:01 PM Nikhil Rao <[email protected]> wrote: > > > > > > For a valid service, the core mask of the service > > > is checked against the current core and the corresponding > > > entry in the active_on_lcore array is set or reset. > > > > > > Upto 8 cores share the same cache line for their > > > service active_on_lcore array entries since each entry is a uint8_t. > > > Some number of these entries also share the cache line with > > > the internal_flags member of struct rte_service_spec_impl, > > > hence this false sharing also makes the service_valid() check > > > expensive. > > > > > > Eliminate false sharing by moving the active_on_lcore array to > > > a per-core data structure. The array is now indexed by service id.
Acked-by: Gage Eads <[email protected]> > > > > Harry, any comments on this patch? > > > Looks good to me, thanks Nikhil & David for the ping; > > Acked-by: Harry van Haaren <[email protected]> Applied, thanks. -- David Marchand
