From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gavin Hu Subject: [PATCH v3 0/1] ring: enforce reading the tail before reading ring slots Date: Wed, 13 Mar 2019 00:58:52 +0800 Message-ID: <1552409933-45684-1-git-send-email-gavin.hu@arm.com> References: <1551841661-42892-1-git-send-email-gavin.hu@arm.com> Cc: nd@arm.com, thomas@monjalon.net, konstantin.ananyev@intel.com, jerinj@marvell.com, hemant.agrawal@nxp.com, nipun.gupta@nxp.com, Honnappa.Nagarahalli@arm.com, gavin.hu@arm.com, i.maximets@samsung.com, chaozhu@linux.vnet.ibm.com To: dev@dpdk.org Return-path: Received: from foss.arm.com (usa-sjc-mx-foss1.foss.arm.com [217.140.101.70]) by dpdk.org (Postfix) with ESMTP id 5F2CB2BD3 for ; Tue, 12 Mar 2019 17:59:07 +0100 (CET) In-Reply-To: <1551841661-42892-1-git-send-email-gavin.hu@arm.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Change log: -V1: First version of the patch. -V2: Update the commit message to describe the ordering of detailed memory accesses. -V3: As suggested by Konstantin Ananyev, remove the rte_smp_rmb for enqueue case, as there is a control dependency in place, making rmb unnecessary. For dequeue case, it is still required as the control dependency does not order loads with loads. gavin hu (1): ring: enforce reading the tail before reading ring slots lib/librte_ring/rte_ring_generic.h | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) -- 2.7.4