[PATCH] build: add missing arch define for Arm
Thomas Monjalon
thomas at monjalon.net
Thu Jan 13 18:32:52 CET 2022
17/12/2021 09:54, Ruifeng Wang:
> As per design document, RTE_ARCH is the name of the architecture.
> However, the definition was missing on Arm with meson build.
> It impacts applications that refers to this string.
>
> Added for Arm builds.
>
> Fixes: b1d48c41189a ("build: support ARM with meson")
> Cc: stable at dpdk.org
>
> Signed-off-by: Ruifeng Wang <ruifeng.wang at arm.com>
> ---
> ['RTE_ARCH_ARMv8_AARCH32', true],
> + ['RTE_ARCH', 'arm64_aarch32'],
Why not armv8_aarch32?
[...]
> dpdk_conf.set('RTE_ARCH_ARMv7', true)
> + dpdk_conf.set('RTE_ARCH', 'armv7')
[...]
> # armv8 build
> + dpdk_conf.set('RTE_ARCH', 'arm64')
Why not armv8?
What I prefer the most in silicon industry is the naming craziness :)
More information about the stable
mailing list