[PATCH] eal/x86: don't export internal alignment macro
Morten Brørup
mb at smartsharesystems.com
Tue Jun 21 16:28:29 CEST 2022
> From: David Marchand [mailto:david.marchand at redhat.com]
> Sent: Tuesday, 21 June 2022 16.13
>
> ALIGNMENT_MASK is only used internally.
> Besides it lacks a DPDK-related prefix.
> Hide it from external eyes.
>
> Reported-by: Morten Brørup <mb at smartsharesystems.com>
> Signed-off-by: David Marchand <david.marchand at redhat.com>
> ---
> lib/eal/x86/include/rte_memcpy.h | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/lib/eal/x86/include/rte_memcpy.h
> b/lib/eal/x86/include/rte_memcpy.h
> index 18aa4e43a7..b678b5c942 100644
> --- a/lib/eal/x86/include/rte_memcpy.h
> +++ b/lib/eal/x86/include/rte_memcpy.h
> @@ -851,6 +851,8 @@ rte_memcpy(void *dst, const void *src, size_t n)
> return rte_memcpy_generic(dst, src, n);
> }
>
> +#undef ALIGNMENT_MASK
> +
> #if defined(RTE_TOOLCHAIN_GCC) && (GCC_VERSION >= 100000)
> #pragma GCC diagnostic pop
> #endif
> --
> 2.36.1
>
You could consider also giving another name with a lower risk of collision.
However, I think the risk is very low, since it can only collide with other headers being included before this header file, so this suffices.
Reviewed-by: Morten Brørup <mb at smartsharesystems.com>
More information about the dev
mailing list