diff options
author | Marc Zyngier <maz@kernel.org> | 2019-11-08 16:57:55 +0000 |
---|---|---|
committer | Marc Zyngier <maz@kernel.org> | 2019-11-10 18:47:50 +0000 |
commit | 898aa5ce6158c5ccfc256bfc17963bc81981eef8 (patch) | |
tree | 9f9746961519fce0552e8eb5f13927167a1009de | |
parent | bf93b04cd85d74c34335e76882cf8bf206caab6e (diff) | |
download | linux-898aa5ce6158c5ccfc256bfc17963bc81981eef8.tar.gz linux-898aa5ce6158c5ccfc256bfc17963bc81981eef8.tar.bz2 linux-898aa5ce6158c5ccfc256bfc17963bc81981eef8.zip |
irqchip/gic-v3-its: Free collection mapping on device teardown
We allocate the collection mapping on device creation, but somehow
free it on the irqdomain free path, which is pretty inconsistent
and has led to bugs in the past.
Move it to the point where we teardown the device, making the
alloc/free symetric.
Signed-off-by: Marc Zyngier <maz@kernel.org>
Reviewed-by: Zenghui Yu <yuzenghui@huawei.com>
Link: https://lore.kernel.org/r/20191108165805.3071-2-maz@kernel.org
-rw-r--r-- | drivers/irqchip/irq-gic-v3-its.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/irqchip/irq-gic-v3-its.c b/drivers/irqchip/irq-gic-v3-its.c index 021e0c70e87c..d5d8f8fc0973 100644 --- a/drivers/irqchip/irq-gic-v3-its.c +++ b/drivers/irqchip/irq-gic-v3-its.c @@ -2474,6 +2474,7 @@ static void its_free_device(struct its_device *its_dev) raw_spin_lock_irqsave(&its_dev->its->lock, flags); list_del(&its_dev->entry); raw_spin_unlock_irqrestore(&its_dev->its->lock, flags); + kfree(its_dev->event_map.col_map); kfree(its_dev->itt); kfree(its_dev); } @@ -2682,7 +2683,6 @@ static void its_irq_domain_free(struct irq_domain *domain, unsigned int virq, its_lpi_free(its_dev->event_map.lpi_map, its_dev->event_map.lpi_base, its_dev->event_map.nr_lpis); - kfree(its_dev->event_map.col_map); /* Unmap device/itt */ its_send_mapd(its_dev, 0); |