From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Gavin Hu (Arm Technology China)" Subject: Re: [PATCH v4 0/3] add rte ring reset api and use it to flush a ring by hash Date: Thu, 3 Jan 2019 02:43:54 +0000 Message-ID: References: <1546390533-53868-1-git-send-email-gavin.hu@arm.com> <3304696.9C1FQTdO3e@xps> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Cc: "dev@dpdk.org" , "jerinj@marvell.com" , "hemant.agrawal@nxp.com" , "bruce.richardson@intel.com" , "chaozhu@linux.vnet.ibm.com" , nd , "olivier.matz@6wind.com" , nd , nd To: Honnappa Nagarahalli , Thomas Monjalon Return-path: Received: from EUR03-DB5-obe.outbound.protection.outlook.com (mail-eopbgr40057.outbound.protection.outlook.com [40.107.4.57]) by dpdk.org (Postfix) with ESMTP id EB92C1B129 for ; Thu, 3 Jan 2019 03:43:56 +0100 (CET) In-Reply-To: Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" > -----Original Message----- > From: Honnappa Nagarahalli > Sent: Thursday, January 3, 2019 2:41 AM > To: Thomas Monjalon ; Gavin Hu (Arm Technology > China) > Cc: dev@dpdk.org; jerinj@marvell.com; hemant.agrawal@nxp.com; > bruce.richardson@intel.com; chaozhu@linux.vnet.ibm.com; nd > ; olivier.matz@6wind.com; nd > Subject: RE: [PATCH v4 0/3] add rte ring reset api and use it to flush a = ring by > hash >=20 > > > > 02/01/2019 01:55, Gavin Hu: > > > V4: Include the ring perf test case enhancement patch in the series. > This change is not related to this patch. Should be a separate patch? I included it in this patch set to avoid patches scattering here and there,= =20 Anyway I updated the title of the cover letter to reflect this. > There were comments provided: > http://mails.dpdk.org/archives/dev/2018-December/121893.html > http://mails.dpdk.org/archives/dev/2018-December/122157.html >=20 > Do you plan to address these? One was addressed in v4 and the other was address in new v5. Thanks! >=20 > > > > > > V3: Allow experimental API for meson build > > > > > > V2: Fix the coding style issue(commit message line too long) > > > > > > V1: To flush a ring not in use, dequeue one by one is wasting cpu cyc= les. > > > The patch is to just resetting the head and tail indices to save cpu > > > cycle. > > > > It is too late for adding this API in 19.02, but we should review and g= ive > > opinion, so it will be ready to integrate in early February. > >