On Fri, 19 Aug 2022, Xenia Ragiadakou wrote: > In macros MPIDR_LEVEL_SHIFT() and MPIDR_AFFINITY_LEVEL(), add parentheses > around the macro parameters 'level' and 'mpidr', respectively, to prevent > against unintended expansions. > > Signed-off-by: Xenia Ragiadakou <burzalod...@gmail.com>
Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> > --- > xen/arch/arm/include/asm/processor.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/xen/arch/arm/include/asm/processor.h > b/xen/arch/arm/include/asm/processor.h > index 55f56b33bc..1dd81d7d52 100644 > --- a/xen/arch/arm/include/asm/processor.h > +++ b/xen/arch/arm/include/asm/processor.h > @@ -127,10 +127,10 @@ > #define MPIDR_LEVEL_MASK ((1 << MPIDR_LEVEL_BITS) - 1) > > #define MPIDR_LEVEL_SHIFT(level) \ > - (((1 << level) >> 1) << MPIDR_LEVEL_BITS_SHIFT) > + (((1 << (level)) >> 1) << MPIDR_LEVEL_BITS_SHIFT) > > #define MPIDR_AFFINITY_LEVEL(mpidr, level) \ > - ((mpidr >> MPIDR_LEVEL_SHIFT(level)) & MPIDR_LEVEL_MASK) > + (((mpidr) >> MPIDR_LEVEL_SHIFT(level)) & MPIDR_LEVEL_MASK) > > #define AFFINITY_MASK(level) ~((_AC(0x1,UL) << MPIDR_LEVEL_SHIFT(level)) > - 1) > > -- > 2.34.1 >
