mirrors/linux
1
0
Fork 1
mirror of https://github.com/torvalds/linux.git synced 2025-10-31 16:48:26 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

genirq/msi: Remove filter from msi_free_descs_free_range()

Browse source 
When a range of descriptors is freed then all of them are not associated to
a linux interrupt. Remove the filter and add a warning to the free function.

Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Reviewed-by: Ashok Raj <ashok.raj@intel.com>
Reviewed-by: Jason Gunthorpe <jgg@nvidia.com>
Link: https://lore.kernel.org/r/20221111122013.888850936@linutronix.de
This commit is contained in:
Thomas Gleixner 2022-11-11 14:54:22 +01:00
parent befd780253
commit 2f2940d168
3 changed files with 13 additions and 13 deletions
repo.diff.show_diff_stats Download patch file Download diff file Expand all files Collapse all files

2
drivers/base/platform-msi.c
View file

@ -325,7 +325,7 @@ void platform_msi_device_domain_free(struct irq_domain *domain, unsigned int vir
msi_lock_descs(data->dev); msi_lock_descs(data->dev);
irq_domain_free_irqs_common(domain, virq, nr_irqs); irq_domain_free_irqs_common(domain, virq, nr_irqs);
msi_free_msi_descs_range(data->dev, MSI_DESC_ALL, virq, virq + nr_irqs - 1); msi_free_msi_descs_range(data->dev, virq, virq + nr_irqs - 1);
msi_unlock_descs(data->dev); msi_unlock_descs(data->dev);
} }

5
include/linux/msi.h
View file

@ -247,8 +247,7 @@ static inline void pci_write_msi_msg(unsigned int irq, struct msi_msg *msg)
#endif /* CONFIG_PCI_MSI */ #endif /* CONFIG_PCI_MSI */
int msi_add_msi_desc(struct device *dev, struct msi_desc *init_desc); int msi_add_msi_desc(struct device *dev, struct msi_desc *init_desc);
void msi_free_msi_descs_range(struct device *dev, enum msi_desc_filter filter, void msi_free_msi_descs_range(struct device *dev, unsigned int first_index, unsigned int last_index);
unsigned int first_index, unsigned int last_index);
/** /**
* msi_free_msi_descs - Free MSI descriptors of a device * msi_free_msi_descs - Free MSI descriptors of a device
@ -256,7 +255,7 @@ void msi_free_msi_descs_range(struct device *dev, enum msi_desc_filter filter,
*/ */
static inline void msi_free_msi_descs(struct device *dev) static inline void msi_free_msi_descs(struct device *dev)
{ {
msi_free_msi_descs_range(dev, MSI_DESC_ALL, 0, MSI_MAX_INDEX); msi_free_msi_descs_range(dev, 0, MSI_MAX_INDEX);
} }
void __pci_read_msi_msg(struct msi_desc *entry, struct msi_msg *msg); void __pci_read_msi_msg(struct msi_desc *entry, struct msi_msg *msg);

19
kernel/irq/msi.c
View file

@ -120,7 +120,7 @@ static int msi_add_simple_msi_descs(struct device *dev, unsigned int index, unsi
fail_mem: fail_mem:
ret = -ENOMEM; ret = -ENOMEM;
fail: fail:
msi_free_msi_descs_range(dev, MSI_DESC_ALL, index, last); msi_free_msi_descs_range(dev, index, last);
return ret; return ret;
} }
@ -141,12 +141,11 @@ static bool msi_desc_match(struct msi_desc *desc, enum msi_desc_filter filter)
/** /**
* msi_free_msi_descs_range - Free MSI descriptors of a device * msi_free_msi_descs_range - Free MSI descriptors of a device
* @dev: Device to free the descriptors * @dev: Device to free the descriptors
* @filter: Descriptor state filter
* @first_index: Index to start freeing from * @first_index: Index to start freeing from
* @last_index: Last index to be freed * @last_index: Last index to be freed
*/ */
void msi_free_msi_descs_range(struct device *dev, enum msi_desc_filter filter, void msi_free_msi_descs_range(struct device *dev, unsigned int first_index,
unsigned int first_index, unsigned int last_index) unsigned int last_index)
{ {
struct xarray *xa = &dev->msi.data->__store; struct xarray *xa = &dev->msi.data->__store;
struct msi_desc *desc; struct msi_desc *desc;
@ -155,10 +154,12 @@ void msi_free_msi_descs_range(struct device *dev, enum msi_desc_filter filter,
lockdep_assert_held(&dev->msi.data->mutex); lockdep_assert_held(&dev->msi.data->mutex);
xa_for_each_range(xa, idx, desc, first_index, last_index) { xa_for_each_range(xa, idx, desc, first_index, last_index) {
if (msi_desc_match(desc, filter)) { xa_erase(xa, idx);
xa_erase(xa, idx);
msi_free_desc(desc); /* Leak the descriptor when it is still referenced */
} if (WARN_ON_ONCE(msi_desc_match(desc, MSI_DESC_ASSOCIATED)))
continue;
msi_free_desc(desc);
} }
} }
@ -739,7 +740,7 @@ int msi_domain_populate_irqs(struct irq_domain *domain, struct device *dev,
fail: fail:
for (--virq; virq >= virq_base; virq--) for (--virq; virq >= virq_base; virq--)
irq_domain_free_irqs_common(domain, virq, 1); irq_domain_free_irqs_common(domain, virq, 1);
msi_free_msi_descs_range(dev, MSI_DESC_ALL, virq_base, virq_base + nvec - 1); msi_free_msi_descs_range(dev, virq_base, virq_base + nvec - 1);
unlock: unlock:
msi_unlock_descs(dev); msi_unlock_descs(dev);
return ret; return ret;