s390/pci: Sort PCI functions prior to creating virtual busses

[ Upstream commit 0467cdde8c4320bbfdb31a8cff1277b202f677fc ]

Instead of relying on the observed but not architected firmware behavior
that PCI functions from the same card are listed in ascending RID order
in clp_list_pci() ensure this by sorting. To allow for sorting separate
the initial clp_list_pci() and creation of the virtual PCI busses.

Note that fundamentally in our per-PCI function hotplug design non RID
order of discovery is still possible. For example when the two PFs of
a two port NIC are hotplugged after initial boot and in descending RID
order. In this case the virtual PCI bus would be created by the second
PF using that PF's UID as domain number instead of that of the first PF.
Thus the domain number would then change from the UID of the second PF
to that of the first PF on reboot but there is really nothing we can do
about that since changing domain numbers at runtime seems even worse.
This only impacts the domain number as the RIDs are consistent and thus
even with just the second PF visible it will show up in the correct
position on the virtual bus.

Reviewed-by: Gerd Bayer <gbayer@linux.ibm.com>
Signed-off-by: Niklas Schnelle <schnelle@linux.ibm.com>
Signed-off-by: Heiko Carstens <hca@linux.ibm.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>

diff --git a/arch/s390/include/asm/pci.h b/arch/s390/include/asm/pci.h
index 30b20ce9a70033cc7c1bc8c3b35c82444cb53a2e..b10801fa73569252ac1b65c8159be89688cf3ba2 100644 (file)
--- a/arch/s390/include/asm/pci.h
+++ b/arch/s390/include/asm/pci.h
@@ -129,6 +129,7 @@ struct zpci_dev {
        u16             vfn;            /* virtual function number */
        u16             pchid;          /* physical channel ID */
        u16             maxstbl;        /* Maximum store block size */
+       u16             rid;            /* RID as supplied by firmware */
        u8              pfgid;          /* function group ID */
        u8              pft;            /* pci function type */
        u8              port;
@@ -210,12 +211,14 @@ extern struct airq_iv *zpci_aif_sbv;
 ----------------------------------------------------------------------------- */
 /* Base stuff */
 struct zpci_dev *zpci_create_device(u32 fid, u32 fh, enum zpci_state state);
+int zpci_add_device(struct zpci_dev *zdev);
 int zpci_enable_device(struct zpci_dev *);
 int zpci_disable_device(struct zpci_dev *);
 int zpci_scan_configured_device(struct zpci_dev *zdev, u32 fh);
 int zpci_deconfigure_device(struct zpci_dev *zdev);
 void zpci_device_reserved(struct zpci_dev *zdev);
 bool zpci_is_device_configured(struct zpci_dev *zdev);
+int zpci_scan_devices(void);
 
 int zpci_hot_reset_device(struct zpci_dev *zdev);
 int zpci_register_ioat(struct zpci_dev *, u8, u64, u64, u64, u8 *);
@@ -225,7 +228,7 @@ void zpci_update_fh(struct zpci_dev *zdev, u32 fh);
 
 /* CLP */
 int clp_setup_writeback_mio(void);
-int clp_scan_pci_devices(void);
+int clp_scan_pci_devices(struct list_head *scan_list);
 int clp_query_pci_fn(struct zpci_dev *zdev);
 int clp_enable_fh(struct zpci_dev *zdev, u32 *fh, u8 nr_dma_as);
 int clp_disable_fh(struct zpci_dev *zdev, u32 *fh);

diff --git a/arch/s390/pci/pci.c b/arch/s390/pci/pci.c
index 635fd8f2acbaa2a3723a711cf6c990200a21f7e8..2a5299888ca86bcc0533e4e82e0b38b75b8294a5 100644 (file)
--- a/arch/s390/pci/pci.c
+++ b/arch/s390/pci/pci.c
@@ -29,6 +29,7 @@
 #include <linux/pci.h>
 #include <linux/printk.h>
 #include <linux/lockdep.h>
+#include <linux/list_sort.h>
 
 #include <asm/isc.h>
 #include <asm/airq.h>
@@ -788,7 +789,6 @@ struct zpci_dev *zpci_create_device(u32 fid, u32 fh, enum zpci_state state)
        struct zpci_dev *zdev;
        int rc;
 
-       zpci_dbg(1, "add fid:%x, fh:%x, c:%d\n", fid, fh, state);
        zdev = kzalloc(sizeof(*zdev), GFP_KERNEL);
        if (!zdev)
                return ERR_PTR(-ENOMEM);
@@ -808,6 +808,19 @@ struct zpci_dev *zpci_create_device(u32 fid, u32 fh, enum zpci_state state)
        mutex_init(&zdev->fmb_lock);
        mutex_init(&zdev->kzdev_lock);
 
+       return zdev;
+
+error:
+       zpci_dbg(0, "crt fid:%x, rc:%d\n", fid, rc);
+       kfree(zdev);
+       return ERR_PTR(rc);
+}
+
+int zpci_add_device(struct zpci_dev *zdev)
+{
+       int rc;
+
+       zpci_dbg(1, "add fid:%x, fh:%x, c:%d\n", zdev->fid, zdev->fh, zdev->state);
        rc = zpci_init_iommu(zdev);
        if (rc)
                goto error;
@@ -819,15 +832,13 @@ struct zpci_dev *zpci_create_device(u32 fid, u32 fh, enum zpci_state state)
        spin_lock(&zpci_list_lock);
        list_add_tail(&zdev->entry, &zpci_list);
        spin_unlock(&zpci_list_lock);
-
-       return zdev;
+       return 0;
 
 error_destroy_iommu:
        zpci_destroy_iommu(zdev);
 error:
-       zpci_dbg(0, "add fid:%x, rc:%d\n", fid, rc);
-       kfree(zdev);
-       return ERR_PTR(rc);
+       zpci_dbg(0, "add fid:%x, rc:%d\n", zdev->fid, rc);
+       return rc;
 }
 
 bool zpci_is_device_configured(struct zpci_dev *zdev)
@@ -1069,6 +1080,49 @@ bool zpci_is_enabled(void)
        return s390_pci_initialized;
 }
 
+static int zpci_cmp_rid(void *priv, const struct list_head *a,
+                       const struct list_head *b)
+{
+       struct zpci_dev *za = container_of(a, struct zpci_dev, entry);
+       struct zpci_dev *zb = container_of(b, struct zpci_dev, entry);
+
+       /*
+        * PCI functions without RID available maintain original order
+        * between themselves but sort before those with RID.
+        */
+       if (za->rid == zb->rid)
+               return za->rid_available > zb->rid_available;
+       /*
+        * PCI functions with RID sort by RID ascending.
+        */
+       return za->rid > zb->rid;
+}
+
+static void zpci_add_devices(struct list_head *scan_list)
+{
+       struct zpci_dev *zdev, *tmp;
+
+       list_sort(NULL, scan_list, &zpci_cmp_rid);
+       list_for_each_entry_safe(zdev, tmp, scan_list, entry) {
+               list_del_init(&zdev->entry);
+               zpci_add_device(zdev);
+       }
+}
+
+int zpci_scan_devices(void)
+{
+       LIST_HEAD(scan_list);
+       int rc;
+
+       rc = clp_scan_pci_devices(&scan_list);
+       if (rc)
+               return rc;
+
+       zpci_add_devices(&scan_list);
+       zpci_bus_scan_busses();
+       return 0;
+}
+
 static int __init pci_base_init(void)
 {
        int rc;
@@ -1098,10 +1152,9 @@ static int __init pci_base_init(void)
        if (rc)
                goto out_irq;
 
-       rc = clp_scan_pci_devices();
+       rc = zpci_scan_devices();
        if (rc)
                goto out_find;
-       zpci_bus_scan_busses();
 
        s390_pci_initialized = 1;
        return 0;

diff --git a/arch/s390/pci/pci_clp.c b/arch/s390/pci/pci_clp.c
index 6f55a59a08711519acd36e8afddc234ebb155b8e..f7430086e97391bae436dbaf38db039d4cba98cc 100644 (file)
--- a/arch/s390/pci/pci_clp.c
+++ b/arch/s390/pci/pci_clp.c
@@ -164,8 +164,10 @@ static int clp_store_query_pci_fn(struct zpci_dev *zdev,
        zdev->port = response->port;
        zdev->uid = response->uid;
        zdev->fmb_length = sizeof(u32) * response->fmb_len;
-       zdev->rid_available = response->rid_avail;
        zdev->is_physfn = response->is_physfn;
+       zdev->rid_available = response->rid_avail;
+       if (zdev->rid_available)
+               zdev->rid = response->rid;
        if (!s390_pci_no_rid && zdev->rid_available)
                zdev->devfn = response->rid & ZPCI_RID_MASK_DEVFN;
 
@@ -407,6 +409,7 @@ static int clp_find_pci(struct clp_req_rsp_list_pci *rrb, u32 fid,
 
 static void __clp_add(struct clp_fh_list_entry *entry, void *data)
 {
+       struct list_head *scan_list = data;
        struct zpci_dev *zdev;
 
        if (!entry->vendor_id)
@@ -417,10 +420,11 @@ static void __clp_add(struct clp_fh_list_entry *entry, void *data)
                zpci_zdev_put(zdev);
                return;
        }
-       zpci_create_device(entry->fid, entry->fh, entry->config_state);
+       zdev = zpci_create_device(entry->fid, entry->fh, entry->config_state);
+       list_add_tail(&zdev->entry, scan_list);
 }
 
-int clp_scan_pci_devices(void)
+int clp_scan_pci_devices(struct list_head *scan_list)
 {
        struct clp_req_rsp_list_pci *rrb;
        int rc;
@@ -429,7 +433,7 @@ int clp_scan_pci_devices(void)
        if (!rrb)
                return -ENOMEM;
 
-       rc = clp_list_pci(rrb, NULL, __clp_add);
+       rc = clp_list_pci(rrb, scan_list, __clp_add);
 
        clp_free_block(rrb);
        return rc;

diff --git a/arch/s390/pci/pci_event.c b/arch/s390/pci/pci_event.c
index d4f19d33914cbc91b6280bceabc5c6482d764d0b..47f934f4e828e9ac6f8cae46474346d14fb5375b 100644 (file)
--- a/arch/s390/pci/pci_event.c
+++ b/arch/s390/pci/pci_event.c
@@ -340,6 +340,7 @@ static void __zpci_event_availability(struct zpci_ccdf_avail *ccdf)
                        zdev = zpci_create_device(ccdf->fid, ccdf->fh, ZPCI_FN_STATE_CONFIGURED);
                        if (IS_ERR(zdev))
                                break;
+                       zpci_add_device(zdev);
                } else {
                        /* the configuration request may be stale */
                        if (zdev->state != ZPCI_FN_STATE_STANDBY)
@@ -349,10 +350,14 @@ static void __zpci_event_availability(struct zpci_ccdf_avail *ccdf)
                zpci_scan_configured_device(zdev, ccdf->fh);
                break;
        case 0x0302: /* Reserved -> Standby */
-               if (!zdev)
-                       zpci_create_device(ccdf->fid, ccdf->fh, ZPCI_FN_STATE_STANDBY);
-               else
+               if (!zdev) {
+                       zdev = zpci_create_device(ccdf->fid, ccdf->fh, ZPCI_FN_STATE_STANDBY);
+                       if (IS_ERR(zdev))
+                               break;
+                       zpci_add_device(zdev);
+               } else {
                        zpci_update_fh(zdev, ccdf->fh);
+               }
                break;
        case 0x0303: /* Deconfiguration requested */
                if (zdev) {
@@ -381,7 +386,7 @@ static void __zpci_event_availability(struct zpci_ccdf_avail *ccdf)
                break;
        case 0x0306: /* 0x308 or 0x302 for multiple devices */
                zpci_remove_reserved_devices();
-               clp_scan_pci_devices();
+               zpci_scan_devices();
                break;
        case 0x0308: /* Standby -> Reserved */
                if (!zdev)