There is a enum type for the return value of traps: enum trap_return. Use the proper return type, wherever it is used.
Signed-off-by: Ralf Ramsauer <ralf.ramsa...@oth-regensburg.de> --- hypervisor/arch/arm-common/include/asm/smccc.h | 2 +- hypervisor/arch/arm-common/include/asm/traps.h | 4 ++-- hypervisor/arch/arm-common/smccc.c | 2 +- hypervisor/arch/arm/mmio.c | 2 +- hypervisor/arch/arm/traps.c | 10 +++++----- hypervisor/arch/arm64/mmio.c | 2 +- hypervisor/arch/arm64/traps.c | 6 +++--- 7 files changed, 14 insertions(+), 14 deletions(-) diff --git a/hypervisor/arch/arm-common/include/asm/smccc.h b/hypervisor/arch/arm-common/include/asm/smccc.h index 24bda10e..563ab9ef 100644 --- a/hypervisor/arch/arm-common/include/asm/smccc.h +++ b/hypervisor/arch/arm-common/include/asm/smccc.h @@ -33,4 +33,4 @@ #define SMCCC_IS_CONV_64(function_id) !!(function_id & (1 << 30)) -int handle_smc(struct trap_context *ctx); +enum trap_return handle_smc(struct trap_context *ctx); diff --git a/hypervisor/arch/arm-common/include/asm/traps.h b/hypervisor/arch/arm-common/include/asm/traps.h index e72d9fb9..ce3907d0 100644 --- a/hypervisor/arch/arm-common/include/asm/traps.h +++ b/hypervisor/arch/arm-common/include/asm/traps.h @@ -22,10 +22,10 @@ enum trap_return { TRAP_FORBIDDEN = -1, }; -typedef int (*trap_handler)(struct trap_context *ctx); +typedef enum trap_return (*trap_handler)(struct trap_context *ctx); void arch_skip_instruction(struct trap_context *ctx); -int arch_handle_dabt(struct trap_context *ctx); +enum trap_return arch_handle_dabt(struct trap_context *ctx); #endif /* !_JAILHOUSE_ASM_TRAPS_H */ diff --git a/hypervisor/arch/arm-common/smccc.c b/hypervisor/arch/arm-common/smccc.c index 463a0133..211d6cd7 100644 --- a/hypervisor/arch/arm-common/smccc.c +++ b/hypervisor/arch/arm-common/smccc.c @@ -30,7 +30,7 @@ static long handle_arch(struct trap_context *ctx) } } -int handle_smc(struct trap_context *ctx) +enum trap_return handle_smc(struct trap_context *ctx) { unsigned long *regs = ctx->regs; u32 *stats = this_cpu_public()->stats; diff --git a/hypervisor/arch/arm/mmio.c b/hypervisor/arch/arm/mmio.c index 19c03a0f..726ea751 100644 --- a/hypervisor/arch/arm/mmio.c +++ b/hypervisor/arch/arm/mmio.c @@ -62,7 +62,7 @@ static void arch_inject_dabt(struct trap_context *ctx, unsigned long addr) arm_write_sysreg(DFAR, addr); } -int arch_handle_dabt(struct trap_context *ctx) +enum trap_return arch_handle_dabt(struct trap_context *ctx) { enum mmio_result mmio_result; struct mmio_access mmio; diff --git a/hypervisor/arch/arm/traps.c b/hypervisor/arch/arm/traps.c index 0589a335..10b3fdbf 100644 --- a/hypervisor/arch/arm/traps.c +++ b/hypervisor/arch/arm/traps.c @@ -245,7 +245,7 @@ static void dump_guest_regs(struct trap_context *ctx) panic_printk("\n"); } -static int arch_handle_smc(struct trap_context *ctx) +static enum trap_return arch_handle_smc(struct trap_context *ctx) { unsigned long *regs = ctx->regs; @@ -261,7 +261,7 @@ static int arch_handle_smc(struct trap_context *ctx) return handle_smc(ctx); } -static int arch_handle_hvc(struct trap_context *ctx) +static enum trap_return arch_handle_hvc(struct trap_context *ctx) { unsigned long *regs = ctx->regs; unsigned long code = regs[0]; @@ -277,7 +277,7 @@ static int arch_handle_hvc(struct trap_context *ctx) return TRAP_HANDLED; } -static int arch_handle_cp15_32(struct trap_context *ctx) +static enum trap_return arch_handle_cp15_32(struct trap_context *ctx) { u32 hsr = ctx->hsr; u32 rt = (hsr >> 5) & 0xf; @@ -360,7 +360,7 @@ static int arch_handle_cp15_32(struct trap_context *ctx) return TRAP_HANDLED; } -static int arch_handle_cp15_64(struct trap_context *ctx) +static enum trap_return arch_handle_cp15_64(struct trap_context *ctx) { u32 hsr = ctx->hsr; u32 rt2 = (hsr >> 10) & 0xf; @@ -402,7 +402,7 @@ static int arch_handle_cp15_64(struct trap_context *ctx) return TRAP_HANDLED; } -static int handle_iabt(struct trap_context *ctx) +static enum trap_return handle_iabt(struct trap_context *ctx) { unsigned long hpfar, hdfar; diff --git a/hypervisor/arch/arm64/mmio.c b/hypervisor/arch/arm64/mmio.c index 772eb5f6..4961b030 100644 --- a/hypervisor/arch/arm64/mmio.c +++ b/hypervisor/arch/arm64/mmio.c @@ -30,7 +30,7 @@ static void arch_inject_dabt(struct trap_context *ctx, unsigned long addr) while (1); } -int arch_handle_dabt(struct trap_context *ctx) +enum trap_return arch_handle_dabt(struct trap_context *ctx) { enum mmio_result mmio_result; struct mmio_access mmio; diff --git a/hypervisor/arch/arm64/traps.c b/hypervisor/arch/arm64/traps.c index 734602ca..e2397b11 100644 --- a/hypervisor/arch/arm64/traps.c +++ b/hypervisor/arch/arm64/traps.c @@ -33,7 +33,7 @@ void arch_skip_instruction(struct trap_context *ctx) arm_write_sysreg(ELR_EL2, pc); } -static int handle_hvc(struct trap_context *ctx) +static enum trap_return handle_hvc(struct trap_context *ctx) { unsigned long *regs = ctx->regs; unsigned long code = regs[0]; @@ -51,7 +51,7 @@ static int handle_hvc(struct trap_context *ctx) return TRAP_HANDLED; } -static int handle_sysreg(struct trap_context *ctx) +static enum trap_return handle_sysreg(struct trap_context *ctx) { u32 esr = ctx->esr; u32 rt = (esr >> 5) & 0x1f; @@ -69,7 +69,7 @@ static int handle_sysreg(struct trap_context *ctx) return TRAP_UNHANDLED; } -static int handle_iabt(struct trap_context *ctx) +static enum trap_return handle_iabt(struct trap_context *ctx) { unsigned long hpfar, hdfar; -- 2.19.1 -- 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 jailhouse-dev+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.