[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[patch V2 21/31] soc: ti: ti_sci_inta_msi: Rework MSI descriptor allocation



Protect the allocation properly and use the core allocation and free
mechanism.

No functional change intended.

Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
---
 drivers/soc/ti/ti_sci_inta_msi.c |   71 +++++++++++++--------------------------
 1 file changed, 25 insertions(+), 46 deletions(-)

--- a/drivers/soc/ti/ti_sci_inta_msi.c
+++ b/drivers/soc/ti/ti_sci_inta_msi.c
@@ -51,6 +51,7 @@ struct irq_domain *ti_sci_inta_msi_creat
        struct irq_domain *domain;
 
        ti_sci_inta_msi_update_chip_ops(info);
+       info->flags |= MSI_FLAG_FREE_MSI_DESCS;
 
        domain = msi_create_irq_domain(fwnode, info, parent);
        if (domain)
@@ -60,50 +61,31 @@ struct irq_domain *ti_sci_inta_msi_creat
 }
 EXPORT_SYMBOL_GPL(ti_sci_inta_msi_create_irq_domain);
 
-static void ti_sci_inta_msi_free_descs(struct device *dev)
-{
-       struct msi_desc *desc, *tmp;
-
-       list_for_each_entry_safe(desc, tmp, dev_to_msi_list(dev), list) {
-               list_del(&desc->list);
-               free_msi_entry(desc);
-       }
-}
-
 static int ti_sci_inta_msi_alloc_descs(struct device *dev,
                                       struct ti_sci_resource *res)
 {
-       struct msi_desc *msi_desc;
+       struct msi_desc msi_desc;
        int set, i, count = 0;
 
+       memset(&msi_desc, 0, sizeof(msi_desc));
+
        for (set = 0; set < res->sets; set++) {
-               for (i = 0; i < res->desc[set].num; i++) {
-                       msi_desc = alloc_msi_entry(dev, 1, NULL);
-                       if (!msi_desc) {
-                               ti_sci_inta_msi_free_descs(dev);
-                               return -ENOMEM;
-                       }
-
-                       msi_desc->msi_index = res->desc[set].start + i;
-                       INIT_LIST_HEAD(&msi_desc->list);
-                       list_add_tail(&msi_desc->list, dev_to_msi_list(dev));
-                       count++;
+               for (i = 0; i < res->desc[set].num; i++, count++) {
+                       msi_desc.msi_index = res->desc[set].start + i;
+                       if (msi_add_msi_desc(dev, &msi_desc))
+                               goto fail;
                }
-               for (i = 0; i < res->desc[set].num_sec; i++) {
-                       msi_desc = alloc_msi_entry(dev, 1, NULL);
-                       if (!msi_desc) {
-                               ti_sci_inta_msi_free_descs(dev);
-                               return -ENOMEM;
-                       }
-
-                       msi_desc->msi_index = res->desc[set].start_sec + i;
-                       INIT_LIST_HEAD(&msi_desc->list);
-                       list_add_tail(&msi_desc->list, dev_to_msi_list(dev));
-                       count++;
+
+               for (i = 0; i < res->desc[set].num_sec; i++, count++) {
+                       msi_desc.msi_index = res->desc[set].start_sec + i;
+                       if (msi_add_msi_desc(dev, &msi_desc))
+                               goto fail;
                }
        }
-
        return count;
+fail:
+       msi_free_msi_descs(dev);
+       return -ENOMEM;
 }
 
 int ti_sci_inta_msi_domain_alloc_irqs(struct device *dev,
@@ -124,20 +106,18 @@ int ti_sci_inta_msi_domain_alloc_irqs(st
        if (ret)
                return ret;
 
+       msi_lock_descs(dev);
        nvec = ti_sci_inta_msi_alloc_descs(dev, res);
-       if (nvec <= 0)
-               return nvec;
-
-       ret = msi_domain_alloc_irqs(msi_domain, dev, nvec);
-       if (ret) {
-               dev_err(dev, "Failed to allocate IRQs %d\n", ret);
-               goto cleanup;
+       if (nvec <= 0) {
+               ret = nvec;
+               goto unlock;
        }
 
-       return 0;
-
-cleanup:
-       ti_sci_inta_msi_free_descs(&pdev->dev);
+       ret = msi_domain_alloc_irqs_descs_locked(msi_domain, dev, nvec);
+       if (ret)
+               dev_err(dev, "Failed to allocate IRQs %d\n", ret);
+unlock:
+       msi_unlock_descs(dev);
        return ret;
 }
 EXPORT_SYMBOL_GPL(ti_sci_inta_msi_domain_alloc_irqs);
@@ -145,6 +125,5 @@ EXPORT_SYMBOL_GPL(ti_sci_inta_msi_domain
 void ti_sci_inta_msi_domain_free_irqs(struct device *dev)
 {
        msi_domain_free_irqs(dev->msi.domain, dev);
-       ti_sci_inta_msi_free_descs(dev);
 }
 EXPORT_SYMBOL_GPL(ti_sci_inta_msi_domain_free_irqs);




 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.