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.