[dpdk-dev] [PATCH V6 3/3] ring: introduce new header file to support C11 memory model

[Jerin Jacob]

-----Original Message-----
> Date: Sun, 26 Nov 2017 18:00:24 -0800
> From: Jia He <hejianet at gmail.com>
> To: jerin.jacob at caviumnetworks.com, dev at dpdk.org,
>  bruce.richardson at intel.com, konstantin.ananyev at intel.com
> Cc: olivier.matz at 6wind.com, jianbo.liu at arm.com, hemant.agrawal at nxp.com, Jia
>  He <hejianet at gmail.com>, Jia He <jia.he at hxt-semitech.com>
> Subject: [PATCH V6 3/3] ring: introduce new header file to support C11
>  memory model
> X-Mailer: git-send-email 2.7.4
> 
> To support C11 memory model barrier, 2 options are suggested by Jerin:
> 1. use rte_smp_rmb
> 2. use load_acquire/store_release(refer to [1]).
> CONFIG_RTE_RING_USE_C11_MEM_MODEL is provided, and by default it is "y"
> only on arm64 so far.
> 
> The reason why providing 2 options is due to the performance benchmark
> difference in different arm machines, refer to [2].
> 
> We haven't tested on ppc64. If anyone verifies it, he can add
> CONFIG_RTE_RING_USE_C11_MEM_MODEL=y to ppc64 config files.
> 
> [1] https://github.com/freebsd/freebsd/blob/master/sys/sys/buf_ring.h#L170
> [2] http://dpdk.org/ml/archives/dev/2017-October/080861.html
> 
> Signed-off-by: Jia He <jia.he at hxt-semitech.com>
> Suggested-by: Jerin Jacob <jerin.jacob at caviumnetworks.com>

Acked-by: Jerin Jacob <jerin.jacob at caviumnetworks.com>