diff mbox series

[v2] mk: change TLS model for DPAA machine

Message ID 1530698085-1933-1-git-send-email-hemant.agrawal@nxp.com
State New
Headers show
Series [v2] mk: change TLS model for DPAA machine | expand

Commit Message

Hemant Agrawal July 4, 2018, 9:54 a.m. UTC
From: Sachin Saxena <sachin.saxena@nxp.com>


Random corruptions observed on platfoms with using
the dpdk library in shared mode with VPP software (plugin).

using traditional TLS scheme resolved the issue.

Tested with VPP with DPDK as a plugin.

Signed-off-by: Sachin Saxena <sachin.saxena@nxp.com>

---
v2: remove the armv8 machine changes

 mk/machine/dpaa/rte.vars.mk  | 3 +++
 mk/machine/dpaa2/rte.vars.mk | 3 +++
 2 files changed, 6 insertions(+)

-- 
2.7.4

Comments

Thomas Monjalon July 12, 2018, 11:11 a.m. UTC | #1
04/07/2018 11:54, Hemant Agrawal:
> From: Sachin Saxena <sachin.saxena@nxp.com>

> 

> Random corruptions observed on platfoms with using

> the dpdk library in shared mode with VPP software (plugin).

> 

> using traditional TLS scheme resolved the issue.

> 

> Tested with VPP with DPDK as a plugin.

> 

> Signed-off-by: Sachin Saxena <sachin.saxena@nxp.com>

> ---

> +# To avoid TLS corruption issue.

> +MACHINE_CFLAGS += -mtls-dialect=trad


Applied as a temporary fix.

What about generic issue for other CPU?
What about meson build?
diff mbox series

Patch

diff --git a/mk/machine/dpaa/rte.vars.mk b/mk/machine/dpaa/rte.vars.mk
index bddcb80..75df626 100644
--- a/mk/machine/dpaa/rte.vars.mk
+++ b/mk/machine/dpaa/rte.vars.mk
@@ -32,3 +32,6 @@  MACHINE_CFLAGS += -march=armv8-a+crc
 ifdef CONFIG_RTE_ARCH_ARM_TUNE
 MACHINE_CFLAGS += -mtune=$(CONFIG_RTE_ARCH_ARM_TUNE:"%"=%)
 endif
+
+# To avoid TLS corruption issue.
+MACHINE_CFLAGS += -mtls-dialect=trad
diff --git a/mk/machine/dpaa2/rte.vars.mk b/mk/machine/dpaa2/rte.vars.mk
index 2fd2eac..aaa03c4 100644
--- a/mk/machine/dpaa2/rte.vars.mk
+++ b/mk/machine/dpaa2/rte.vars.mk
@@ -32,3 +32,6 @@  MACHINE_CFLAGS += -march=armv8-a+crc
 ifdef CONFIG_RTE_ARCH_ARM_TUNE
 MACHINE_CFLAGS += -mtune=$(CONFIG_RTE_ARCH_ARM_TUNE:"%"=%)
 endif
+
+# To avoid TLS corruption issue.
+MACHINE_CFLAGS += -mtls-dialect=trad