From 91c8a7d5cef1e9729c1c2bad549ebc69afeadb3b Mon Sep 17 00:00:00 2001 From: Bin Meng Date: Sun, 11 Dec 2022 14:54:22 +0800 Subject: [PATCH] lib: utils/irqchip: plic: Fix the off-by-one error in plic_context_init() The number of interrupt enable register in words was once correct, but was wrongly changed to have an off-by-one error since commit 8c362e7d065e ("lib: irqchip/plic: Factor out a context init function"). Fixes: 8c362e7d065e ("lib: irqchip/plic: Factor out a context init function") Signed-off-by: Bin Meng Reviewed-by: Anup Patel Reviewed-by: Samuel Holland --- lib/utils/irqchip/plic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/utils/irqchip/plic.c b/lib/utils/irqchip/plic.c index c5c18796..5b87d3bc 100644 --- a/lib/utils/irqchip/plic.c +++ b/lib/utils/irqchip/plic.c @@ -121,7 +121,7 @@ int plic_context_init(const struct plic_data *plic, int context_id, if (!plic || context_id < 0) return SBI_EINVAL; - ie_words = (plic->num_src + 31) / 32; + ie_words = plic->num_src / 32 + 1; ie_value = enable ? 0xffffffffU : 0U; for (u32 i = 0; i < ie_words; i++)