[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [XEN PATCH] docs/misra: add x86_64 and arm64 asm-offset.c to exclude-list



Hi Jan,

On 2024-02-08 09:05, Jan Beulich wrote:
On 08.02.2024 08:55, Nicola Vetrini wrote:
These two files contain several deliberate violations of MISRA C rules and they are not linked in the final Xen binary, therefore they can be exempted
from MISRA compliance.

No functional change.

Signed-off-by: Nicola Vetrini <nicola.vetrini@xxxxxxxxxxx>
---
Since the exclude list only contains arm64 and x86 files I reasoned that introducing an entry that would match all architectures would not be desirable
(e.g., arm32). I'm happy to change that, though.

Just wanted to ask - if globs are permitted, I'd favor covering all possible architectures. It is certainly expected that they all follow suit. Just that in the absence of sub-architectures the path would be xen/arch/*/asm-offsets.c (and it's quite possible that we may, over time, morph x86 to a sub-arch-less
form).

Jan


Ok, I'll modify it. Thanks,

--
Nicola Vetrini, BSc
Software Engineer, BUGSENG srl (https://bugseng.com)



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.