[dpdk-dev] config/thunderx: disable c11 mem model ring implementation

Message ID 20171203123730.26782-1-jerin.jacob@caviumnetworks.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Jerin Jacob Dec. 3, 2017, 12:37 p.m. UTC
  On thunderx and octeontx, ring_perf_autotest and
ring_pmd_perf_autotest test shows better performance
when disabling CONFIG_RTE_RING_USE_C11_MEM_MODEL.
On the other hand, Enabling CONFIG_RTE_RING_USE_C11_MEM_MODEL
shows better performance on thunderx2.
Since thunderx2 is using the default armv8 config,
no particular change is required.

Signed-off-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
---
This patch is depended on following patch from Jia He.
support c11 memory model barrier in librte_ring
http://dpdk.org/ml/archives/dev/2017-November/082368.html
---
 config/defconfig_arm64-thunderx-linuxapp-gcc | 1 +
 1 file changed, 1 insertion(+)
  

Comments

Thomas Monjalon Jan. 29, 2018, 3:14 p.m. UTC | #1
03/12/2017 13:37, Jerin Jacob:
> On thunderx and octeontx, ring_perf_autotest and
> ring_pmd_perf_autotest test shows better performance
> when disabling CONFIG_RTE_RING_USE_C11_MEM_MODEL.
> On the other hand, Enabling CONFIG_RTE_RING_USE_C11_MEM_MODEL
> shows better performance on thunderx2.
> Since thunderx2 is using the default armv8 config,
> no particular change is required.
> 
> Signed-off-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>

Applied, thanks
  

Patch

diff --git a/config/defconfig_arm64-thunderx-linuxapp-gcc b/config/defconfig_arm64-thunderx-linuxapp-gcc
index 45038b119..60c68ec96 100644
--- a/config/defconfig_arm64-thunderx-linuxapp-gcc
+++ b/config/defconfig_arm64-thunderx-linuxapp-gcc
@@ -36,6 +36,7 @@  CONFIG_RTE_MACHINE="thunderx"
 CONFIG_RTE_CACHE_LINE_SIZE=128
 CONFIG_RTE_MAX_NUMA_NODES=2
 CONFIG_RTE_MAX_LCORE=96
+CONFIG_RTE_RING_USE_C11_MEM_MODEL=n
 
 #
 # Compile PMD for octeontx sso event device