[PATCH v1 1/2] irqchip/gic-v3-its: don't set bitmap for LPI which user didn't allocate

From: Luo Jiaxing
Date: Mon Feb 08 2021 - 06:10:34 EST


The driver sets the LPI bitmap of device based on get_count_order(nvecs).
This means that when the number of LPI interrupts does not meet the power
of two, redundant bits are set in the LPI bitmap. However, when free
interrupt, these redundant bits is not cleared. As a result, device will
fails to allocate the same numbers of interrupts next time.

Therefore, clear the redundant bits set in LPI bitmap.

Fixes: 4615fbc3788d ("genirq/irqdomain: Don't try to free an interrupt that has no mapping")

Signed-off-by: Luo Jiaxing <luojiaxing@xxxxxxxxxx>
---
drivers/irqchip/irq-gic-v3-its.c | 4 ++++
1 file changed, 4 insertions(+)

diff --git a/drivers/irqchip/irq-gic-v3-its.c b/drivers/irqchip/irq-gic-v3-its.c
index ed46e60..027f7ef 100644
--- a/drivers/irqchip/irq-gic-v3-its.c
+++ b/drivers/irqchip/irq-gic-v3-its.c
@@ -3435,6 +3435,10 @@ static int its_alloc_device_irq(struct its_device *dev, int nvecs, irq_hw_number

*hwirq = dev->event_map.lpi_base + idx;

+ bitmap_clear(dev->event_map.lpi_map,
+ idx + nvecs,
+ roundup_pow_of_two(nvecs) - nvecs);
+
return 0;
}

--
2.7.4