<html><head><meta name="color-scheme" content="light dark"></head><body><pre style="word-wrap: break-word; white-space: pre-wrap;">From 67d1a979738af24517c09068bc1b43c8026daa08 Mon Sep 17 00:00:00 2001
From: Laszlo Ersek &lt;lersek@redhat.com&gt;
Date: Mon, 13 Jul 2015 19:17:49 +0200
Subject: [PATCH 23/23] hw/pci-bridge: format special OFW unit address for PXB
 host

Message-id: &lt;1436815069-15106-13-git-send-email-lersek@redhat.com&gt;
Patchwork-id: 67003
O-Subject: [RHEV-7.2 qemu-kvm-rhev PATCH 12/12] hw/pci-bridge: format special OFW unit address for PXB host
Bugzilla: 1242479
RH-Acked-by: Marcel Apfelbaum &lt;marcel@redhat.com&gt;
RH-Acked-by: Paolo Bonzini &lt;pbonzini@redhat.com&gt;
RH-Acked-by: Michael S. Tsirkin &lt;mst@redhat.com&gt;

We have agreed that OpenFirmware device paths in the "bootorder" fw_cfg
file should follow the pattern

  /pci@i0cf8,%x/...

for devices that live behind an extra root bus. The extra root bus in
question is the %x'th among the extra root buses. (In other words, %x
gives the position of the affected extra root bus relative to the other
extra root buses, in bus_nr order.) %x starts at 1, and is formatted in
hex.

The portion of the unit address that comes before the comma is dynamically
taken from the main host bridge, similarly to sysbus_get_fw_dev_path().

Cc: Kevin O'Connor &lt;kevin@koconnor.net&gt;
Cc: Michael S. Tsirkin &lt;mst@redhat.com&gt;
Cc: Marcel Apfelbaum &lt;marcel@redhat.com&gt;
Signed-off-by: Laszlo Ersek &lt;lersek@redhat.com&gt;
Reviewed-by: Michael S. Tsirkin &lt;mst@redhat.com&gt;
Signed-off-by: Michael S. Tsirkin &lt;mst@redhat.com&gt;
(cherry picked from commit 48ea3dedc54dbcb3c738ddef02a336739910ecfd)
Signed-off-by: Miroslav Rezanina &lt;mrezanin@redhat.com&gt;
---
 hw/pci-bridge/pci_expander_bridge.c | 53 +++++++++++++++++++++++++++++++++++++
 1 file changed, 53 insertions(+)

diff --git a/hw/pci-bridge/pci_expander_bridge.c b/hw/pci-bridge/pci_expander_bridge.c
index 70708ef..57f8a37 100644
--- a/hw/pci-bridge/pci_expander_bridge.c
+++ b/hw/pci-bridge/pci_expander_bridge.c
@@ -43,6 +43,8 @@ typedef struct PXBDev {
     uint16_t numa_node;
 } PXBDev;
 
+static GList *pxb_dev_list;
+
 #define TYPE_PXB_HOST "pxb-host"
 
 static int pxb_bus_num(PCIBus *bus)
@@ -89,12 +91,45 @@ static const char *pxb_host_root_bus_path(PCIHostState *host_bridge,
     return bus-&gt;bus_path;
 }
 
+static char *pxb_host_ofw_unit_address(const SysBusDevice *dev)
+{
+    const PCIHostState *pxb_host;
+    const PCIBus *pxb_bus;
+    const PXBDev *pxb_dev;
+    int position;
+    const DeviceState *pxb_dev_base;
+    const PCIHostState *main_host;
+    const SysBusDevice *main_host_sbd;
+
+    pxb_host = PCI_HOST_BRIDGE(dev);
+    pxb_bus = pxb_host-&gt;bus;
+    pxb_dev = PXB_DEV(pxb_bus-&gt;parent_dev);
+    position = g_list_index(pxb_dev_list, pxb_dev);
+    assert(position &gt;= 0);
+
+    pxb_dev_base = DEVICE(pxb_dev);
+    main_host = PCI_HOST_BRIDGE(pxb_dev_base-&gt;parent_bus-&gt;parent);
+    main_host_sbd = SYS_BUS_DEVICE(main_host);
+
+    if (main_host_sbd-&gt;num_mmio &gt; 0) {
+        return g_strdup_printf(TARGET_FMT_plx ",%x",
+                               main_host_sbd-&gt;mmio[0].addr, position + 1);
+    }
+    if (main_host_sbd-&gt;num_pio &gt; 0) {
+        return g_strdup_printf("i%04x,%x",
+                               main_host_sbd-&gt;pio[0], position + 1);
+    }
+    return NULL;
+}
+
 static void pxb_host_class_init(ObjectClass *class, void *data)
 {
     DeviceClass *dc = DEVICE_CLASS(class);
+    SysBusDeviceClass *sbc = SYS_BUS_DEVICE_CLASS(class);
     PCIHostBridgeClass *hc = PCI_HOST_BRIDGE_CLASS(class);
 
     dc-&gt;fw_name = "pci";
+    sbc-&gt;explicit_ofw_unit_address = pxb_host_ofw_unit_address;
     hc-&gt;root_bus_path = pxb_host_root_bus_path;
 }
 
@@ -149,6 +184,15 @@ static int pxb_map_irq_fn(PCIDevice *pci_dev, int pin)
     return pin - PCI_SLOT(pxb-&gt;devfn);
 }
 
+static gint pxb_compare(gconstpointer a, gconstpointer b)
+{
+    const PXBDev *pxb_a = a, *pxb_b = b;
+
+    return pxb_a-&gt;bus_nr &lt; pxb_b-&gt;bus_nr ? -1 :
+           pxb_a-&gt;bus_nr &gt; pxb_b-&gt;bus_nr ?  1 :
+           0;
+}
+
 static int pxb_dev_initfn(PCIDevice *dev)
 {
     PXBDev *pxb = PXB_DEV(dev);
@@ -192,9 +236,17 @@ static int pxb_dev_initfn(PCIDevice *dev)
                                PCI_STATUS_66MHZ | PCI_STATUS_FAST_BACK);
     pci_config_set_class(dev-&gt;config, PCI_CLASS_BRIDGE_HOST);
 
+    pxb_dev_list = g_list_insert_sorted(pxb_dev_list, pxb, pxb_compare);
     return 0;
 }
 
+static void pxb_dev_exitfn(PCIDevice *pci_dev)
+{
+    PXBDev *pxb = PXB_DEV(pci_dev);
+
+    pxb_dev_list = g_list_remove(pxb_dev_list, pxb);
+}
+
 static Property pxb_dev_properties[] = {
     /* Note: 0 is not a legal a PXB bus number. */
     DEFINE_PROP_UINT8("bus_nr", PXBDev, bus_nr, 0),
@@ -208,6 +260,7 @@ static void pxb_dev_class_init(ObjectClass *klass, void *data)
     PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
 
     k-&gt;init = pxb_dev_initfn;
+    k-&gt;exit = pxb_dev_exitfn;
     k-&gt;vendor_id = PCI_VENDOR_ID_REDHAT;
     k-&gt;device_id = PCI_DEVICE_ID_REDHAT_PXB;
     k-&gt;class_id = PCI_CLASS_BRIDGE_HOST;
-- 
1.8.3.1

</pre></body></html>