From ed06b8a7cab2a2cc31a62e42ac8e655c03f2f40e Mon Sep 17 00:00:00 2001 From: Shiqing Gao Date: Thu, 27 Sep 2018 16:00:20 +0800 Subject: [PATCH] hv: fix 'Void procedure used in expression' MISRA-C states that a void procedure used in expressions is dangerous. This patch removes the improper 'return' when calling the void procedure 'send_dest_ipi'. Tracked-On: #861 Signed-off-by: Shiqing Gao Acked-by: Eddie Dong --- hypervisor/arch/x86/lapic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hypervisor/arch/x86/lapic.c b/hypervisor/arch/x86/lapic.c index a947d5f22..74e44af7e 100644 --- a/hypervisor/arch/x86/lapic.c +++ b/hypervisor/arch/x86/lapic.c @@ -438,7 +438,7 @@ void send_single_ipi(uint16_t pcpu_id, uint32_t vector) /* Get the lapic ID of the destination processor. */ dest_lapic_id = per_cpu(lapic_id, pcpu_id); - return send_dest_ipi(dest_lapic_id, vector, INTR_LAPIC_ICR_PHYSICAL); + send_dest_ipi(dest_lapic_id, vector, INTR_LAPIC_ICR_PHYSICAL); } int send_shorthand_ipi(uint8_t vector,