Re: [dpdk-dev] [PATCH v9 0/3] support c11 memory model barrier in librte_ring

2018-01-29 Thread Thomas Monjalon
22/01/2018 05:41, Jia He: > 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" o

Re: [dpdk-dev] [PATCH v9 0/3] support c11 memory model barrier in librte_ring

2018-01-22 Thread Hemant Agrawal
> -Original Message- > From: Thomas Monjalon [mailto:tho...@monjalon.net] > Sent: Monday, January 22, 2018 1:23 PM > To: Jia He > Cc: dev@dpdk.org; Hemant Agrawal ; Jerin > Jacob ; Jianbo Liu > ; Jan Viktorin ; Olivier > Matz ; konstantin.anan...@intel.com; > bruce.richard...@intel.com >

Re: [dpdk-dev] [PATCH v9 0/3] support c11 memory model barrier in librte_ring

2018-01-22 Thread Olivier Matz
On Mon, Jan 22, 2018 at 09:26:49AM +0100, Olivier Matz wrote: > On Mon, Jan 22, 2018 at 08:53:01AM +0100, Thomas Monjalon wrote: > > 22/01/2018 05:41, Jia He: > > > Changelog: > > > V9: remove the SPDX tag and refine commit logs > > > > Why did you remove the SPDX tag? > > > > You need to fix the

Re: [dpdk-dev] [PATCH v9 0/3] support c11 memory model barrier in librte_ring

2018-01-22 Thread Olivier Matz
On Mon, Jan 22, 2018 at 08:53:01AM +0100, Thomas Monjalon wrote: > 22/01/2018 05:41, Jia He: > > Changelog: > > V9: remove the SPDX tag and refine commit logs > > Why did you remove the SPDX tag? > > You need to fix the licensing issue. > BSD-2-Clause is uncommon in DPDK. > Thomas, The licensi

Re: [dpdk-dev] [PATCH v9 0/3] support c11 memory model barrier in librte_ring

2018-01-21 Thread Thomas Monjalon
22/01/2018 05:41, Jia He: > Changelog: > V9: remove the SPDX tag and refine commit logs Why did you remove the SPDX tag? You need to fix the licensing issue. BSD-2-Clause is uncommon in DPDK.

[dpdk-dev] [PATCH v9 0/3] support c11 memory model barrier in librte_ring

2018-01-21 Thread Jia He
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. Alr