Il giorno mer 26 lug 2023 alle ore 14:02 Jan Beulich <jbeul...@suse.com> ha scritto:
> 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 > Yes, it is. I accidentally removed the v4 from the object. In this submission, I've dropped the commits that have been already merged. -- Simone Ballarin, M.Sc. Field Application Engineer, BUGSENG (https://bugseng.com <http://bugseng.com>)