[dpdk-dev] [PATCH v9 1/2] ring: add reset API to flush the ring when not in use

[Gavin Hu (Arm Technology China)]

Hi Olivier,  Thomas,

> -----Original Message-----
> From: Olivier Matz <olivier.matz at 6wind.com>
> Sent: Tuesday, July 16, 2019 6:32 AM
> To: Gavin Hu (Arm Technology China) <Gavin.Hu at arm.com>
> Cc: dev at dpdk.org; nd <nd at arm.com>; thomas at monjalon.net;
> jerinj at marvell.com; hemant.agrawal at nxp.com; Nipun.gupta at nxp.com;
> Honnappa Nagarahalli <Honnappa.Nagarahalli at arm.com>;
> i.maximets at samsung.com; stable at dpdk.org
> Subject: Re: [PATCH v9 1/2] ring: add reset API to flush the ring when not in
> use
> 
> On Tue, Jul 16, 2019 at 11:01:21AM +0200, Olivier Matz wrote:
> > On Fri, Jul 12, 2019 at 11:54:36PM +0800, Gavin Hu wrote:
> > > Currently, the flush is done by dequeuing the ring in a while loop. It is
> > > much simpler to flush the queue by resetting the head and tail indices.
> > >
> > > Fixes: af75078fece3 ("first public release")
> > > Cc: stable at dpdk.org
> 
> Actually it's not a fix, it adds a new API.
> 
> Is the patch in hash library intended to be backported? If yes, as it
> seems to be a performance optimization, you'll need to describe what
> scenario you're fixing and what is the performance gain. If no, the Cc
> stable can be removed.

As this is not in the data plan, I don't intend to backport. 
Do I need to submit a new version to remove the CC: lines? 

> > >
> > > Signed-off-by: Gavin Hu <gavin.hu at arm.com>
> > > Reviewed-by: Ruifeng Wang <ruifeng.wang at arm.com>
> > > Reviewed-by: Honnappa Nagarahalli <honnappa.nagarahalli at arm.com>
> >
> > Acked-by: Olivier Matz <olivier.matz at 6wind.com>