[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [XEN PATCH][for-4.19 v2 0/7] Fix or deviate various instances of missing declarations
The patches in this series aim to fix or deviate various instances where a function or variable do not have a declaration visible when such entity is defined (in violation of MISRA C:2012 Rule 8.4). An exception listed under docs/misra/rules.rst allows asm-only functions and variables to be exempted, while the other instances are either changed (e.g., making them static) or a missing header inclusion is added. Some of the patches in this series are potential candidates for bug fixes, or just general improvements that may be suited for inclusion in the next rc. The deviation comment used by these patches is still SAF-x-safe, which is being phased out in favour of some other name, as discussed here [1]. When such a name has been determined, a new (trivial) version of this series can be produced, the change could be made on commit or as part of the SAF rename troughout the codebase. Nicola Vetrini (7): xen: add declarations for variables where needed x86: add deviations for variables only used in asm code x86: add deviation comments for asm-only functions x86/grant: switch included header to make declarations visible x86/vm_event: add missing include for hvm_vm_event_do_resume xen/console: make function static inline x86/mem_access: make function static xen/arch/arm/include/asm/setup.h | 3 +++ xen/arch/arm/include/asm/smp.h | 3 +++ xen/arch/arm/platform_hypercall.c | 2 +- xen/arch/x86/cpu/mcheck/mce.c | 6 +++--- xen/arch/x86/hvm/grant_table.c | 3 +-- xen/arch/x86/hvm/svm/intr.c | 1 + xen/arch/x86/hvm/svm/nestedsvm.c | 1 + xen/arch/x86/hvm/svm/svm.c | 2 ++ xen/arch/x86/hvm/vm_event.c | 1 + xen/arch/x86/include/asm/asm_defns.h | 1 + xen/arch/x86/include/asm/compat.h | 1 + xen/arch/x86/include/asm/hvm/grant_table.h | 2 ++ xen/arch/x86/include/asm/setup.h | 1 + xen/arch/x86/irq.c | 2 +- xen/arch/x86/mm/mem_access.c | 2 +- xen/arch/x86/setup.c | 1 + xen/arch/x86/traps.c | 1 + xen/arch/x86/x86_64/traps.c | 1 + xen/common/symbols.c | 17 ----------------- xen/include/xen/consoled.h | 2 +- xen/include/xen/symbols.h | 18 ++++++++++++++++++ 21 files changed, 45 insertions(+), 26 deletions(-) -- 2.34.1
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |