On 26.07.2023 13:03, Simone Ballarin wrote: > The xen sources contains violations of MISRA C:2012 Rule 7.2 whose headline > states: > "A 'u' or 'U' suffix shall be applied to all integer constants that are > represented in an unsigned type". > > These violations are caused by the missing "u" or "U" suffix in unsigned > integer constants, such as: > > xen/arch/x86/hvm/hypercall.c:132.17-132.26 > if ( (eax & 0x80000000) && is_viridian_domain(currd) ) > > If a rule is not met, changes are needed in order to achieve compliance. > The patches in this series achieve compliance for MISRA C:2012 Rule 7.2 by > adding the 'U' suffix to integers literals with unsigned type and also to > other > literals used in the same contexts or near violations, when their positive > nature is immediately clear. The latter changes are done for the sake of > uniformity. > > Gianluca Luparini (4): > x86/vmx: address violations of MISRA C:2012 Rule 7.2 > xen/vpci: address violations of MISRA C:2012 Rule 7.2 > x86/viridian: address violations of MISRA C:2012 Rule 7.2 > xen/x86: address violations of MISRA C:2012 Rule 7.2
Looks like this is v4 of a previously larger series, where fair parts have gone in already? Jan
