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

2019-07-16 Thread Gavin Hu (Arm Technology China)
Hi Thomas, > -Original Message- > From: Thomas Monjalon > Sent: Tuesday, July 16, 2019 10:07 AM > To: Gavin Hu (Arm Technology China) > Cc: Olivier Matz ; dev@dpdk.org; nd > ; jer...@marvell.com; hemant.agra...@nxp.com; > nipun.gu...@nxp.com; Honnappa Nagarahalli > ; i.maxim...@samsung.co

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

2019-07-16 Thread Thomas Monjalon
16/07/2019 16:03, Gavin Hu (Arm Technology China): > From: Olivier Matz > > 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 > > >

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

2019-07-16 Thread Gavin Hu (Arm Technology China)
Hi Olivier, Thomas, > -Original Message- > From: Olivier Matz > Sent: Tuesday, July 16, 2019 6:32 AM > To: Gavin Hu (Arm Technology China) > Cc: dev@dpdk.org; nd ; tho...@monjalon.net; > jer...@marvell.com; hemant.agra...@nxp.com; nipun.gu...@nxp.com; > Honnappa Nagarahalli ; > i.maxim.

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

2019-07-16 Thread Olivier Matz
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: af75078

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

2019-07-16 Thread Olivier Matz
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: sta...@dpdk.org > > Signed-off-by:

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

2019-07-12 Thread Gavin Hu
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: sta...@dpdk.org Signed-off-by: Gavin Hu Reviewed-by: Ruifeng Wang Reviewed-by: Honnappa Nagarahalli