From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jia He Subject: [PATCH v9 0/3] support c11 memory model barrier in librte_ring Date: Sun, 21 Jan 2018 20:41:25 -0800 Message-ID: <1516596088-10364-1-git-send-email-hejianet@gmail.com> References: <1516161831-28719-1-git-send-email-hejianet@gmail.com> Cc: Jerin Jacob , Jianbo Liu , Jan Viktorin , Olivier Matz , konstantin.ananyev@intel.com, bruce.richardson@intel.com, Jia He To: dev@dpdk.org, Thomas Monjalon , Hemant Agrawal Return-path: Received: from mail-pf0-f195.google.com (mail-pf0-f195.google.com [209.85.192.195]) by dpdk.org (Postfix) with ESMTP id 6C7A11E33 for ; Mon, 22 Jan 2018 05:41:47 +0100 (CET) Received: by mail-pf0-f195.google.com with SMTP id e76so6192870pfk.1 for ; Sun, 21 Jan 2018 20:41:47 -0800 (PST) In-Reply-To: <1516161831-28719-1-git-send-email-hejianet@gmail.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" There are 2 model barrier options in librte_ring suggested by Jerin: 1. use rte_smp_rmb 2. use load_acquire/store_release CONFIG_RTE_RING_USE_C11_MEM_MODEL is provided for supporting C11 memory model barrier in librte_ring. By default it is "y" on arm64, "n" on any other architectures so far. Already fuctionally tested on the machines as follows: - on X86 - on arm64 with CONFIG_RTE_RING_USE_C11_MEM_MODEL=y - on arm64 with CONFIG_RTE_RING_USE_C11_MEM_MODEL=n --- Changelog: V9: remove the SPDX tag and refine commit logs V8: Change the license to SPDX tag. Change USE_C11_MEM_MODEL to "n" on any architectures by default V7: fix check-git-log warnings which is suggested by Jerin V6: minor change in subject and log V5: split it into 2 patchset due to the milestone concerns, this is the 2st one. Also fix checkpatch.pl warnings V4: split into small patches V3: arch specific implementation for enqueue/dequeue barrier V2: let users choose whether using load_acquire/store_release V1: rte_smp_rmb() between 2 loads Jia He (3): eal/arm64: remove the braces {} for dmb() and dsb() ring: introduce new header file to include common functions ring: introduce new header file to support C11 memory model config/common_armv8a_linuxapp | 2 + config/common_base | 5 + .../common/include/arch/arm/rte_atomic_64.h | 4 +- lib/librte_eventdev/rte_event_ring.h | 6 +- lib/librte_ring/Makefile | 4 +- lib/librte_ring/rte_ring.h | 173 ++-------------- lib/librte_ring/rte_ring_c11_mem.h | 218 ++++++++++++++++++++ lib/librte_ring/rte_ring_generic.h | 228 +++++++++++++++++++++ 8 files changed, 476 insertions(+), 164 deletions(-) create mode 100644 lib/librte_ring/rte_ring_c11_mem.h create mode 100644 lib/librte_ring/rte_ring_generic.h -- 2.7.4