On 28.10.20 22:09, Andrea Bastoni wrote: > 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(-) >
Thanks, applied with minor changes as noted. Jan -- Siemens AG, T RDA IOT Corporate Competence Center Embedded Linux -- 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/46b28184-9435-f456-81ab-c368859774a4%40siemens.com.
