From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <thomas@monjalon.net>
Received: from out1-smtp.messagingengine.com (out1-smtp.messagingengine.com
 [66.111.4.25]) by dpdk.org (Postfix) with ESMTP id F19AA1B5E1
 for <dev@dpdk.org>; Wed, 19 Dec 2018 22:59:08 +0100 (CET)
Received: from compute1.internal (compute1.nyi.internal [10.202.2.41])
 by mailout.nyi.internal (Postfix) with ESMTP id D8E1D21EAD;
 Wed, 19 Dec 2018 16:59:07 -0500 (EST)
Received: from mailfrontend2 ([10.202.2.163])
 by compute1.internal (MEProxy); Wed, 19 Dec 2018 16:59:07 -0500
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h=
 from:to:cc:subject:date:message-id:in-reply-to:references
 :mime-version:content-transfer-encoding:content-type; s=mesmtp;
 bh=qD2fUyJdZwDP6vgprdOg6FRCcH7jGjtXfqKb664yHeM=; b=Fppvwfe3sTmj
 aoDXoi20hqYo6tGgk27vQNy+38rW05Lk9RhrL2HftQ2i5geEPp1MYug92cfdRC4n
 WAiXsmKmMmZKEthLwkrKcgJbAOF9lDeOwOgHp1N2OpbY8h+kakDwrFy4Cd7GuGTf
 0hI3+l2ZCh9cid1ShtieA9wIeNFDO24=
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=
 messagingengine.com; h=cc:content-transfer-encoding:content-type
 :date:from:in-reply-to:message-id:mime-version:references
 :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender
 :x-sasl-enc; s=fm1; bh=qD2fUyJdZwDP6vgprdOg6FRCcH7jGjtXfqKb664yH
 eM=; b=DwH/k+C0EcIfc88V+mcaMXi7GYvbREoYW/7kNrRfLr5x3ym7UJ5VPuz+a
 CkSTWeytMKY2JqBZqMs9kuQIgrVt+g2jrG5ew9owxcbVCIp/Zy4aiDw6c7nIeLGr
 WZFUKnnjzRmUWX3LeJ6FUBkC9izsLDIfIYEgZ/cBCtY1U2kHLtw7MHzhyBEKqSrZ
 8YEI7RaJu4T3KZ/fCm8EZCHZYk6AOLUfypij/wy/rwIHVREgr+ayIeOiZS38Ygli
 ddXk9eaGM73kGkR9xtamRes95Ww1lxfak0vdQz0748SfnscSETrh2ZYnjW7lxidE
 7GNBGy9ZB37rTibgJmBptrdECoD5Q==
X-ME-Sender: <xms:Kr8aXKZ0QU7AtivdgkfZS5q4N9XXBoOkIrZRotwY3PJ6-m1oE7mO1g>
X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedtkedrudejtddgudehjecutefuodetggdotefrod
 ftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpqfhuthenuceurghilhhouhhtmecu
 fedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmnecujfgurhephffvufffkf
 gjfhgggfgtsehtufertddttddvnecuhfhrohhmpefvhhhomhgrshcuofhonhhjrghlohhn
 uceothhhohhmrghssehmohhnjhgrlhhonhdrnhgvtheqnecukfhppeejjedrudefgedrvd
 dtfedrudekgeenucfrrghrrghmpehmrghilhhfrhhomhepthhhohhmrghssehmohhnjhgr
 lhhonhdrnhgvthenucevlhhushhtvghrufhiiigvpedt
X-ME-Proxy: <xmx:Kr8aXP-fIT2c0zSo75iDNavKE8xzxSmN0OuMjd8YTtuEHiNDdYTUyA>
 <xmx:Kr8aXF44Cs7mpC9QCkI8MhrWu6il3VSZCLNAsPSn8FntQGu_Ncn7qw>
 <xmx:Kr8aXBqSiOyYFlAGJ16P3oIypTuO8XJxZ-8JKSLHFYJV_Pl2HkmKjQ>
 <xmx:K78aXKlYstdagYwEt8vqccEPVMl2CfQEyi6T3TKiIXZ1WwbPkSU7YQ>
Received: from xps.localnet (184.203.134.77.rev.sfr.net [77.134.203.184])
 by mail.messagingengine.com (Postfix) with ESMTPA id A49C3100BA;
 Wed, 19 Dec 2018 16:59:05 -0500 (EST)
From: Thomas Monjalon <thomas@monjalon.net>
To: Gavin Hu <gavin.hu@arm.com>
Cc: dev@dpdk.org, nd@arm.com, jerin.jacob@caviumnetworks.com,
 hemant.agrawal@nxp.com, Honnappa.Nagarahalli@arm.com,
 bruce.richardson@intel.com, Olivier Matz <olivier.matz@6wind.com>
Date: Wed, 19 Dec 2018 22:59:04 +0100
Message-ID: <3213509.eEyrQtEzxV@xps>
In-Reply-To: <20181212111525.22996-1-gavin.hu@arm.com>
References: <20181212111525.22996-1-gavin.hu@arm.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"
Subject: Re: [dpdk-dev] [PATCH v3 0/2] add rte ring reset api and use it to
	flush a ring by hash
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Wed, 19 Dec 2018 21:59:09 -0000

+Cc Olivier, maintainer of the ring library.

12/12/2018 12:15, Gavin Hu:
> 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 cycles.
> The patch is to just resetting the head and tail indices to save cpu
> cycle.
> 
> Gavin Hu (2):
>   ring: add reset api to flush the ring when not in use
>   hash: flush the rings instead of dequeuing one by one
> 
>  lib/librte_hash/Makefile             |  2 +-
>  lib/librte_hash/meson.build          |  3 +++
>  lib/librte_hash/rte_cuckoo_hash.c    | 11 ++++-------
>  lib/librte_ring/rte_ring.h           | 20 ++++++++++++++++++++
>  lib/librte_ring/rte_ring_version.map |  7 +++++++
>  5 files changed, 35 insertions(+), 8 deletions(-)