[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20251211224033.18079-1-osama.abdelkader@gmail.com>
Date: Thu, 11 Dec 2025 23:40:28 +0100
From: Osama Abdelkader <osama.abdelkader@...il.com>
To: Marc Zyngier <maz@...nel.org>,
Oliver Upton <oupton@...nel.org>,
Joey Gouly <joey.gouly@....com>,
Suzuki K Poulose <suzuki.poulose@....com>,
Zenghui Yu <yuzenghui@...wei.com>,
Catalin Marinas <catalin.marinas@....com>,
Will Deacon <will@...nel.org>,
linux-arm-kernel@...ts.infradead.org,
kvmarm@...ts.linux.dev,
linux-kernel@...r.kernel.org
Cc: Osama Abdelkader <osama.abdelkader@...il.com>
Subject: [PATCH] KVM: arm64: vgic: add default case to switch statement
The switch statement in vgic_validate_injection() handles all enum
values for irq->config, but lacked a default case. Add one to match
the pattern used in other switch statements in the same file and make
the defensive return explicit.
Signed-off-by: Osama Abdelkader <osama.abdelkader@...il.com>
---
arch/arm64/kvm/vgic/vgic.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/kvm/vgic/vgic.c b/arch/arm64/kvm/vgic/vgic.c
index 430aa98888fd..a035ce6231e2 100644
--- a/arch/arm64/kvm/vgic/vgic.c
+++ b/arch/arm64/kvm/vgic/vgic.c
@@ -369,9 +369,9 @@ static bool vgic_validate_injection(struct vgic_irq *irq, bool level, void *owne
return irq->line_level != level;
case VGIC_CONFIG_EDGE:
return level;
+ default:
+ return false;
}
-
- return false;
}
static bool vgic_model_needs_bcst_kick(struct kvm *kvm)
--
2.43.0
Powered by blists - more mailing lists