Thanks for the previous review.
In this version, the unused-parameter is suppressed by default as soon
as Wextra is added. The warnings that can be triggered by Wextra are
manually listed with Wno-error. Eventually, all Wno-error are removed.
Thanks,
Andrea
Andrea Bastoni (22):
hypervisor: Makefile: hook-in Wextra
hypervisor: add wordsize dependent size_t type
hypervisor, arm-common: provide an explicit uint INVALID_CPU_ID
hypervisor: mmio: fix Wsign-compare by casting index to u32.
hypervisor: printk: Fix Wsign-compare in align()
inmates: lib: fix align() in printk similarly to the hypervisor
hypervisor: Wsign-compare: force the "right" promotion direction
arm-common: paging: fix old style declaration
(-Wold-style-declaration)
arm-common: provide unsigned int PSCI_INVALID_ADDRESS
arm-common: gic*: use uint instead of int when needed (Wsign-compare)
arm, arm-common: Wsign-compare: arm_dcaches_flush(): take unsigned
long as size
arm64: smmu-v3: use uint instead of int when needed (Wsign-compare)
arm64: smmu-v3: rename 'i' -> 'n', 'j' -> 's'
arm64: ti-pvu: fix Wsign-compare by casting size to u64.
arm64: smmu: Wsign-compare: make iterator uint
x86: fix simple Wsign-compare warnings
x86: apic: Wsign-compare change to uint and update comparison check
accordingly
x86: cat: fix Wsign-compare
x86: vtd: fix Wsign-compare
x86: vcpu: force cast to prevent Wsign-compare
x86: introduce EXCEPTION_NO_ERROR to avoid int to uint comparison on
error frame
hypervisor: Makefile: remove Wno-error from CFLAGS
hypervisor/Makefile | 7 ++--
hypervisor/arch/arm-common/gic-v2.c | 18 ++++-----
hypervisor/arch/arm-common/gic-v3.c | 10 ++---
.../arch/arm-common/include/asm/dcaches.h | 2 +-
hypervisor/arch/arm-common/include/asm/psci.h | 2 +-
hypervisor/arch/arm-common/lib.c | 2 +-
hypervisor/arch/arm-common/paging.c | 4 +-
hypervisor/arch/arm-common/psci.c | 4 +-
hypervisor/arch/arm/mmu_hyp.c | 4 +-
hypervisor/arch/arm64/smmu-v3.c | 39 ++++++++++---------
hypervisor/arch/arm64/smmu.c | 17 ++++----
hypervisor/arch/arm64/ti-pvu.c | 2 +-
hypervisor/arch/x86/apic.c | 4 +-
hypervisor/arch/x86/cat.c | 3 +-
hypervisor/arch/x86/control.c | 2 +-
hypervisor/arch/x86/entry.S | 3 +-
hypervisor/arch/x86/include/asm/processor.h | 2 +
hypervisor/arch/x86/paging.c | 2 +-
hypervisor/arch/x86/svm.c | 2 +-
hypervisor/arch/x86/vcpu.c | 2 +-
hypervisor/arch/x86/vtd.c | 11 +++---
hypervisor/control.c | 3 +-
hypervisor/include/jailhouse/control.h | 5 ++-
hypervisor/include/jailhouse/string.h | 5 ++-
hypervisor/include/jailhouse/types.h | 6 +++
hypervisor/lib.c | 5 +--
hypervisor/mmio.c | 2 +-
hypervisor/pci.c | 3 +-
hypervisor/printk.c | 5 ++-
hypervisor/setup.c | 4 +-
inmates/lib/printk.c | 5 ++-
31 files changed, 102 insertions(+), 83 deletions(-)
--
2.28.0
--
You received this message because you are subscribed to the Google Groups
"Jailhouse" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/jailhouse-dev/20201028210933.138379-1-andrea.bastoni%40tum.de.
