[PATCH] eal/x86: don't export internal alignment macro
Thomas Monjalon
thomas at monjalon.net
Wed Jun 22 11:32:26 CEST 2022
21/06/2022 16:28, Morten Brørup:
> > 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>
> > ---
> > --- a/lib/eal/x86/include/rte_memcpy.h
> > +++ b/lib/eal/x86/include/rte_memcpy.h
> > +#undef ALIGNMENT_MASK
>
> 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>
Fixes: f5472703c0bd ("eal: optimize aligned memcpy on x86")
Cc: stable at dpdk.org
Applied, thanks.
More information about the dev
mailing list