From d9d6ab05f041bce5bc65c34f2568812878060775 Mon Sep 17 00:00:00 2001 From: Min He Date: Thu, 14 Feb 2019 07:06:34 +0000 Subject: [PATCH] hv: fix an issue in vlapic_trigger_lvt In commit e24899d9c993 ("fix "Recursion in procedure calls found""), it added a check if vector < 16U in vlapic_trigger_lvt() by a mistake, since the vector variable in vlapic_trigger_lvt() is not the interrupt vector. This patch fixed the issue mentioned above. Tracked-On:#2598 Signed-off-by: Min He --- hypervisor/arch/x86/guest/vlapic.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/hypervisor/arch/x86/guest/vlapic.c b/hypervisor/arch/x86/guest/vlapic.c index 5d24ac526..d0b801b6c 100644 --- a/hypervisor/arch/x86/guest/vlapic.c +++ b/hypervisor/arch/x86/guest/vlapic.c @@ -982,11 +982,7 @@ vlapic_trigger_lvt(struct acrn_vlapic *vlapic, uint32_t vector) default: return -EINVAL; } - if (vector < 16U) { - vlapic_set_error(vlapic, APIC_ESR_RECEIVE_ILLEGAL_VECTOR); - } else { - vlapic_fire_lvt(vlapic, lvt); - } + vlapic_fire_lvt(vlapic, lvt); return 0; }