> -----Original Message-----
> From: Van Haaren, Harry <harry.van.haa...@intel.com>
> Sent: Friday, April 3, 2020 7:58 PM
> To: Phil Yang <phil.y...@arm.com>; tho...@monjalon.net; Ananyev,
> Konstantin <konstantin.anan...@intel.com>;
> step...@networkplumber.org; maxime.coque...@redhat.com;
> dev@dpdk.org
> Cc: david.march...@redhat.com; jer...@marvell.com;
> hemant.agra...@nxp.com; Honnappa Nagarahalli
> <honnappa.nagaraha...@arm.com>; Gavin Hu <gavin...@arm.com>;
> Ruifeng Wang <ruifeng.w...@arm.com>; Joyce Kong
> <joyce.k...@arm.com>; nd <n...@arm.com>; sta...@dpdk.org
> Subject: RE: [PATCH v3 07/12] service: remove rte prefix from static functions
>
> > From: Phil Yang <phil.y...@arm.com>
> > Sent: Tuesday, March 17, 2020 1:18 AM
> > To: tho...@monjalon.net; Van Haaren, Harry
> <harry.van.haa...@intel.com>;
> > Ananyev, Konstantin <konstantin.anan...@intel.com>;
> > step...@networkplumber.org; maxime.coque...@redhat.com;
> dev@dpdk.org
> > Cc: david.march...@redhat.com; jer...@marvell.com;
> hemant.agra...@nxp.com;
> > honnappa.nagaraha...@arm.com; gavin...@arm.com;
> ruifeng.w...@arm.com;
> > joyce.k...@arm.com; n...@arm.com; sta...@dpdk.org
> > Subject: [PATCH v3 07/12] service: remove rte prefix from static functions
> >
> > Fixes: 3cf5eb1546ed ("service: fix and refactor atomic service accesses")
> > Fixes: 21698354c832 ("service: introduce service cores concept")
> > Cc: sta...@dpdk.org
> >
> > Signed-off-by: Phil Yang <phil.y...@arm.com>
> > Reviewed-by: Honnappa Nagarahalli <honnappa.nagaraha...@arm.com>
>
>
> This patchset needs a rebase since the EAL file movement got merged,
> however I'll review here so we can include some Acks etc and make
> progress.
>
> Is this really a "Fix"? The internal function names were not exported
> in the .map file, so are not part of public ABI. This is an internal
> naming improvement (thanks for doing cleanup), but I don't think the
> Fixes: tags make sense?
>
> Also I'm not sure if we want to port this patch back to stable? Changing
> (internal) function names seems like unnecessary churn, and hence risk to a
> stable release, without any benefit?
OK.
I will remove these tags in the next version and split the service core patches
from the original series into a series by itself.
Thanks,
Phil
>
> ---
>
> <snip patch diff>
