Thread (99 messages) 99 messages, 8 authors, 2017-07-01

Re: [PATCH v8 2/2] config: enable vhost numa awareness by default

From: Thomas Monjalon <hidden>
Date: 2017-06-27 09:21:51

27/06/2017 11:18, Hemant Agrawal:
On 6/27/2017 2:16 PM, Ilya Maximets wrote:
quoted
It is safe to enable LIBRTE_VHOST_NUMA by default for all
configurations where libnuma is already a default dependency.

Signed-off-by: Ilya Maximets <redacted>
---
 config/common_linuxapp                    | 1 +
 config/defconfig_arm-armv7a-linuxapp-gcc  | 1 +
 config/defconfig_arm64-dpaa2-linuxapp-gcc | 1 +
 3 files changed, 3 insertions(+)
[...]
quoted
--- a/config/defconfig_arm64-dpaa2-linuxapp-gcc
+++ b/config/defconfig_arm64-dpaa2-linuxapp-gcc
@@ -47,6 +47,7 @@ CONFIG_RTE_PKTMBUF_HEADROOM=256

 # Doesn't support NUMA
 CONFIG_RTE_EAL_NUMA_AWARE_HUGEPAGES=y
+CONFIG_RTE_LIBRTE_VHOST_NUMA=n

 #
 # Compile Support Libraries for DPAA2
-1
It should also be disabled for generic ARM64. This patch is breaking 
generic arm64 config tests on our platforms and creating a unnecessary 
dependency.
What do you mean? Which ARM64 platform is it breaking?
We can specifically disable it on more platforms.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help