Re: [Qemu-ppc] [Qemu-devel] [PATCH v4 2/2] spapr: generate DT node names

[Gavin Shan]

On Thu, Sep 24, 2015 at 12:27:39PM +0200, Laurent Vivier wrote:
>When DT node names for PCI devices are generated by SLOF,
>they are generated according to the type of the device
>(for instance, ethernet for virtio-net-pci device).
>
>Node name for hotplugged devices is generated by QEMU.
>This patch adds the mechanic to QEMU to create the node
>name according to the device type too.
>
>The data structure has been roughly copied from OpenBIOS/OpenHackware,
>node names from SLOF.
>
>Example:
>
>Hotplugging some PCI cards with QEMU monitor:
>
>device_add virtio-tablet-pci
>device_add virtio-serial-pci
>device_add virtio-mouse-pci
>device_add virtio-scsi-pci
>device_add virtio-gpu-pci
>device_add ne2k_pci
>device_add nec-usb-xhci
>device_add intel-hda
>
>What we can see in linux device tree:
>
>for dir in /proc/device-tree/address@hidden/address@hidden/; do
>    echo $dir
>    cat $dir/name
>    echo
>done
>
>WITHOUT this patch:
>
>/proc/device-tree/address@hidden/address@hidden/
>pci
>/proc/device-tree/address@hidden/address@hidden/
>pci
>/proc/device-tree/address@hidden/address@hidden/
>pci
>/proc/device-tree/address@hidden/address@hidden/
>pci
>/proc/device-tree/address@hidden/address@hidden/
>pci
>/proc/device-tree/address@hidden/address@hidden/
>pci
>/proc/device-tree/address@hidden/address@hidden/
>pci
>/proc/device-tree/address@hidden/address@hidden/
>pci
>
>WITH this patch:
>
>/proc/device-tree/address@hidden/address@hidden/
>communication-controller
>/proc/device-tree/address@hidden/address@hidden/
>display
>/proc/device-tree/address@hidden/address@hidden/
>ethernet
>/proc/device-tree/address@hidden/address@hidden/
>input-controller
>/proc/device-tree/address@hidden/address@hidden/
>mouse
>/proc/device-tree/address@hidden/address@hidden/
>multimedia-device
>/proc/device-tree/address@hidden/address@hidden/
>scsi
>/proc/device-tree/address@hidden/address@hidden/
>usb-xhci
>
>Signed-off-by: Laurent Vivier <address@hidden>
>Reviewed-by: Thomas Huth <address@hidden>
>---
> hw/ppc/spapr_pci.c | 292 ++++++++++++++++++++++++++++++++++++++++++++++++++---
> 1 file changed, 278 insertions(+), 14 deletions(-)
>
>diff --git a/hw/ppc/spapr_pci.c b/hw/ppc/spapr_pci.c
>index a2feb4c..63eb28c 100644
>--- a/hw/ppc/spapr_pci.c
>+++ b/hw/ppc/spapr_pci.c
>@@ -38,6 +38,7 @@
>
> #include "hw/pci/pci_bridge.h"
> #include "hw/pci/pci_bus.h"
>+#include "hw/pci/pci_ids.h"
> #include "hw/ppc/spapr_drc.h"
> #include "sysemu/device_tree.h"
>
>@@ -944,6 +945,276 @@ static void populate_resource_props(PCIDevice *d, 
>ResourceProps *rp)
>     rp->assigned_len = assigned_idx * sizeof(ResourceFields);
> }
>

One question would be: is there one reason why the logic, converting
class/subclass/iface code to tring, isn't put into generic PCI module?
If the code is put there, all platforms can reuse it.

Thanks,
Gavin

>+typedef struct PCIClass PCIClass;
>+typedef struct PCISubClass PCISubClass;
>+typedef struct PCIIFace PCIIFace;
>+
>+struct PCIIFace {
>+    uint8_t iface;
>+    const char *name;
>+};
>+
>+struct PCISubClass {
>+    uint8_t subclass;
>+    const char *name;
>+    const PCIIFace *iface;
>+};
>+#define SUBCLASS(a) ((uint8_t)a)
>+#define IFACE(a)    ((uint8_t)a)
>+
>+struct PCIClass {
>+    const char *name;
>+    const PCISubClass *subc;
>+};
>+
>+static const PCISubClass undef_subclass[] = {
>+    { IFACE(PCI_CLASS_NOT_DEFINED_VGA), "display", NULL },
>+    { 0xFF, NULL, NULL, NULL },
>+};
>+
>+static const PCISubClass mass_subclass[] = {
>+    { SUBCLASS(PCI_CLASS_STORAGE_SCSI), "scsi", NULL },
>+    { SUBCLASS(PCI_CLASS_STORAGE_IDE), "ide", NULL },
>+    { SUBCLASS(PCI_CLASS_STORAGE_FLOPPY), "fdc", NULL },
>+    { SUBCLASS(PCI_CLASS_STORAGE_IPI), "ipi", NULL },
>+    { SUBCLASS(PCI_CLASS_STORAGE_RAID), "raid", NULL },
>+    { SUBCLASS(PCI_CLASS_STORAGE_ATA), "ata", NULL },
>+    { SUBCLASS(PCI_CLASS_STORAGE_SATA), "sata", NULL },
>+    { SUBCLASS(PCI_CLASS_STORAGE_SAS), "sas", NULL },
>+    { 0xFF, NULL, NULL },
>+};
>+
>+static const PCISubClass net_subclass[] = {
>+    { SUBCLASS(PCI_CLASS_NETWORK_ETHERNET), "ethernet", NULL },
>+    { SUBCLASS(PCI_CLASS_NETWORK_TOKEN_RING), "token-ring", NULL },
>+    { SUBCLASS(PCI_CLASS_NETWORK_FDDI), "fddi", NULL },
>+    { SUBCLASS(PCI_CLASS_NETWORK_ATM), "atm", NULL },
>+    { SUBCLASS(PCI_CLASS_NETWORK_ISDN), "isdn", NULL },
>+    { SUBCLASS(PCI_CLASS_NETWORK_WORLDFIP), "worldfip", NULL },
>+    { SUBCLASS(PCI_CLASS_NETWORK_PICMG214), "picmg", NULL },
>+    { 0xFF, NULL, NULL },
>+};
>+
>+static const PCISubClass displ_subclass[] = {
>+    { SUBCLASS(PCI_CLASS_DISPLAY_VGA), "vga", NULL },
>+    { SUBCLASS(PCI_CLASS_DISPLAY_XGA), "xga", NULL },
>+    { SUBCLASS(PCI_CLASS_DISPLAY_3D), "3d-controller", NULL },
>+    { 0xFF, NULL, NULL },
>+};
>+
>+static const PCISubClass media_subclass[] = {
>+    { SUBCLASS(PCI_CLASS_MULTIMEDIA_VIDEO), "video", NULL },
>+    { SUBCLASS(PCI_CLASS_MULTIMEDIA_AUDIO), "sound", NULL },
>+    { SUBCLASS(PCI_CLASS_MULTIMEDIA_PHONE), "telephony", NULL },
>+    { 0xFF, NULL, NULL },
>+};
>+
>+static const PCISubClass mem_subclass[] = {
>+    { SUBCLASS(PCI_CLASS_MEMORY_RAM), "memory", NULL },
>+    { SUBCLASS(PCI_CLASS_MEMORY_FLASH), "flash", NULL },
>+    { 0xFF, NULL, NULL },
>+};
>+
>+static const PCISubClass bridg_subclass[] = {
>+    { SUBCLASS(PCI_CLASS_BRIDGE_HOST), "host", NULL },
>+    { SUBCLASS(PCI_CLASS_BRIDGE_ISA), "isa", NULL },
>+    { SUBCLASS(PCI_CLASS_BRIDGE_EISA), "eisa", NULL },
>+    { SUBCLASS(PCI_CLASS_BRIDGE_MC), "mca", NULL },
>+    { SUBCLASS(PCI_CLASS_BRIDGE_PCI), "pci", NULL },
>+    { SUBCLASS(PCI_CLASS_BRIDGE_PCMCIA), "pcmcia", NULL },
>+    { SUBCLASS(PCI_CLASS_BRIDGE_NUBUS), "nubus", NULL },
>+    { SUBCLASS(PCI_CLASS_BRIDGE_CARDBUS), "cardbus", NULL },
>+    { SUBCLASS(PCI_CLASS_BRIDGE_RACEWAY), "raceway", NULL },
>+    { SUBCLASS(PCI_CLASS_BRIDGE_PCI_SEMITP), "semi-transparent-pci", NULL },
>+    { SUBCLASS(PCI_CLASS_BRIDGE_IB_PCI), "infiniband", NULL },
>+    { 0xFF, NULL, NULL },
>+};
>+
>+static const PCISubClass comm_subclass[] = {
>+    { SUBCLASS(PCI_CLASS_COMMUNICATION_SERIAL), "serial", NULL },
>+    { SUBCLASS(PCI_CLASS_COMMUNICATION_PARALLEL), "parallel", NULL },
>+    { SUBCLASS(PCI_CLASS_COMMUNICATION_MULTISERIAL), "multiport-serial", NULL 
>},
>+    { SUBCLASS(PCI_CLASS_COMMUNICATION_MODEM), "modem", NULL },
>+    { SUBCLASS(PCI_CLASS_COMMUNICATION_GPIB), "gpib", NULL },
>+    { SUBCLASS(PCI_CLASS_COMMUNICATION_SC), "smart-card", NULL },
>+    { 0xFF, NULL, NULL, NULL },
>+};
>+
>+static const PCIIFace pic_iface[] = {
>+    { IFACE(PCI_CLASS_SYSTEM_PIC_IOAPIC), "io-apic" },
>+    { IFACE(PCI_CLASS_SYSTEM_PIC_IOXAPIC), "io-xapic" },
>+    { 0xFF, NULL },
>+};
>+
>+static const PCISubClass sys_subclass[] = {
>+    { SUBCLASS(PCI_CLASS_SYSTEM_PIC), "interrupt-controller", pic_iface },
>+    { SUBCLASS(PCI_CLASS_SYSTEM_DMA), "dma-controller", NULL },
>+    { SUBCLASS(PCI_CLASS_SYSTEM_TIMER), "timer", NULL },
>+    { SUBCLASS(PCI_CLASS_SYSTEM_RTC), "rtc", NULL },
>+    { SUBCLASS(PCI_CLASS_SYSTEM_PCI_HOTPLUG), "hot-plug-controller", NULL },
>+    { SUBCLASS(PCI_CLASS_SYSTEM_SDHCI), "sd-host-controller", NULL },
>+    { 0xFF, NULL, NULL },
>+};
>+
>+static const PCISubClass inp_subclass[] = {
>+    { SUBCLASS(PCI_CLASS_INPUT_KEYBOARD), "keyboard", NULL },
>+    { SUBCLASS(PCI_CLASS_INPUT_PEN), "pen", NULL },
>+    { SUBCLASS(PCI_CLASS_INPUT_MOUSE), "mouse", NULL },
>+    { SUBCLASS(PCI_CLASS_INPUT_SCANNER), "scanner", NULL },
>+    { SUBCLASS(PCI_CLASS_INPUT_GAMEPORT), "gameport", NULL },
>+    { 0xFF, NULL, NULL },
>+};
>+
>+static const PCISubClass dock_subclass[] = {
>+    { SUBCLASS(PCI_CLASS_DOCKING_GENERIC), "dock", NULL },
>+    { 0xFF, NULL, NULL },
>+};
>+
>+static const PCISubClass cpu_subclass[] = {
>+    { SUBCLASS(PCI_CLASS_PROCESSOR_PENTIUM), "pentium", NULL },
>+    { SUBCLASS(PCI_CLASS_PROCESSOR_POWERPC), "powerpc", NULL },
>+    { SUBCLASS(PCI_CLASS_PROCESSOR_MIPS), "mips", NULL },
>+    { SUBCLASS(PCI_CLASS_PROCESSOR_CO), "co-processor", NULL },
>+    { 0xFF, NULL, NULL },
>+};
>+
>+static const PCIIFace usb_iface[] = {
>+    { IFACE(PCI_CLASS_SERIAL_USB_UHCI), "usb-uhci" },
>+    { IFACE(PCI_CLASS_SERIAL_USB_OHCI), "usb-ohci", },
>+    { IFACE(PCI_CLASS_SERIAL_USB_EHCI), "usb-ehci" },
>+    { IFACE(PCI_CLASS_SERIAL_USB_XHCI), "usb-xhci" },
>+    { IFACE(PCI_CLASS_SERIAL_USB_UNKNOWN), "usb-unknown" },
>+    { IFACE(PCI_CLASS_SERIAL_USB_DEVICE), "usb-device" },
>+    { 0xFF, NULL },
>+};
>+
>+static const PCISubClass ser_subclass[] = {
>+    { SUBCLASS(PCI_CLASS_SERIAL_FIREWIRE), "firewire", NULL },
>+    { SUBCLASS(PCI_CLASS_SERIAL_ACCESS), "access-bus", NULL },
>+    { SUBCLASS(PCI_CLASS_SERIAL_SSA), "ssa", NULL },
>+    { SUBCLASS(PCI_CLASS_SERIAL_USB), "usb", usb_iface },
>+    { SUBCLASS(PCI_CLASS_SERIAL_FIBER), "fibre-channel", NULL },
>+    { SUBCLASS(PCI_CLASS_SERIAL_SMBUS), "smb", NULL },
>+    { SUBCLASS(PCI_CLASS_SERIAL_IB), "infiniband", NULL },
>+    { SUBCLASS(PCI_CLASS_SERIAL_IPMI), "ipmi", NULL },
>+    { SUBCLASS(PCI_CLASS_SERIAL_SERCOS), "sercos", NULL },
>+    { SUBCLASS(PCI_CLASS_SERIAL_CANBUS), "canbus", NULL },
>+    { 0xFF, NULL, NULL },
>+};
>+
>+static const PCISubClass wrl_subclass[] = {
>+    { SUBCLASS(PCI_CLASS_WIRELESS_IRDA), "irda", NULL },
>+    { SUBCLASS(PCI_CLASS_WIRELESS_CIR), "consumer-ir", NULL },
>+    { SUBCLASS(PCI_CLASS_WIRELESS_RF_CONTROLLER), "rf-controller", NULL },
>+    { SUBCLASS(PCI_CLASS_WIRELESS_BLUETOOTH), "bluetooth", NULL },
>+    { SUBCLASS(PCI_CLASS_WIRELESS_BROADBAND), "broadband", NULL },
>+    { 0xFF, NULL, NULL },
>+};
>+
>+static const PCISubClass sat_subclass[] = {
>+    { SUBCLASS(PCI_CLASS_SATELLITE_TV), "satellite-tv", NULL },
>+    { SUBCLASS(PCI_CLASS_SATELLITE_AUDIO), "satellite-audio", NULL },
>+    { SUBCLASS(PCI_CLASS_SATELLITE_VOICE), "satellite-voice", NULL },
>+    { SUBCLASS(PCI_CLASS_SATELLITE_DATA), "satellite-data", NULL },
>+    { 0xFF, NULL, NULL },
>+};
>+
>+static const PCISubClass crypt_subclass[] = {
>+    { SUBCLASS(PCI_CLASS_CRYPT_NETWORK), "network-encryption", NULL },
>+    { SUBCLASS(PCI_CLASS_CRYPT_ENTERTAINMENT),
>+      "entertainment-encryption", NULL },
>+    { 0xFF, NULL, NULL },
>+};
>+
>+static const PCISubClass spc_subclass[] = {
>+    { SUBCLASS(PCI_CLASS_SP_DPIO), "dpio", NULL },
>+    { SUBCLASS(PCI_CLASS_SP_PERF), "counter", NULL },
>+    { SUBCLASS(PCI_CLASS_SP_SYNCH), "measurement", NULL },
>+    { SUBCLASS(PCI_CLASS_SP_MANAGEMENT), "management-card", NULL },
>+    { 0xFF, NULL, NULL },
>+};
>+
>+static const PCIClass pci_classes[] = {
>+    { "legacy-device", undef_subclass },
>+    { "mass-storage",  mass_subclass },
>+    { "network", net_subclass },
>+    { "display", displ_subclass, },
>+    { "multimedia-device", media_subclass },
>+    { "memory-controller", mem_subclass },
>+    { "unknown-bridge", bridg_subclass },
>+    { "communication-controller", comm_subclass},
>+    { "system-peripheral", sys_subclass },
>+    { "input-controller", inp_subclass },
>+    { "docking-station", dock_subclass },
>+    { "cpu", cpu_subclass },
>+    { "serial-bus", ser_subclass },
>+    { "wireless-controller", wrl_subclass },
>+    { "intelligent-io", NULL },
>+    { "satellite-device", sat_subclass },
>+    { "encryption", crypt_subclass },
>+    { "data-processing-controller", spc_subclass },
>+};
>+
>+static const char *pci_find_device_name(uint8_t class, uint8_t subclass,
>+                                        uint8_t iface)
>+{
>+    const PCIClass *pclass;
>+    const PCISubClass *psubclass;
>+    const PCIIFace *piface;
>+    const char *name;
>+
>+    if (class >= ARRAY_SIZE(pci_classes)) {
>+        return "pci";
>+    }
>+
>+    pclass = pci_classes + class;
>+    name = pclass->name;
>+
>+    if (pclass->subc == NULL) {
>+        return name;
>+    }
>+
>+    psubclass = pclass->subc;
>+    while (psubclass->subclass != 0xff) {
>+        if (psubclass->subclass == subclass) {
>+            name = psubclass->name;
>+            break;
>+        }
>+        psubclass++;
>+    }
>+
>+    piface = psubclass->iface;
>+    if (piface == NULL) {
>+        return name;
>+    }
>+    while (piface->iface != 0xff) {
>+        if (piface->iface == iface) {
>+            name = piface->name;
>+            break;
>+        }
>+        piface++;
>+    }
>+
>+    return name;
>+}
>+
>+static void pci_get_node_name(char *nodename, int len, PCIDevice *dev)
>+{
>+    int slot = PCI_SLOT(dev->devfn);
>+    int func = PCI_FUNC(dev->devfn);
>+    uint32_t ccode = pci_default_read_config(dev, PCI_CLASS_PROG, 3);
>+    const char *name;
>+
>+    name = pci_find_device_name((ccode >> 16) & 0xff, (ccode >> 8) & 0xff,
>+                                ccode & 0xff);
>+
>+    if (func != 0) {
>+        snprintf(nodename, len, "address@hidden,%x", name, slot, func);
>+    } else {
>+        snprintf(nodename, len, "address@hidden", name, slot);
>+    }
>+}
>+
> static uint32_t spapr_phb_get_pci_drc_index(sPAPRPHBState *phb,
>                                             PCIDevice *pdev);
>
>@@ -955,6 +1226,7 @@ static int spapr_populate_pci_child_dt(PCIDevice *dev, 
>void *fdt, int offset,
>     int pci_status, err;
>     char *buf = NULL;
>     uint32_t drc_index = spapr_phb_get_pci_drc_index(sphb, dev);
>+    uint32_t ccode = pci_default_read_config(dev, PCI_CLASS_PROG, 3);
>
>     if (pci_default_read_config(dev, PCI_HEADER_TYPE, 1) ==
>         PCI_HEADER_TYPE_BRIDGE) {
>@@ -968,8 +1240,7 @@ static int spapr_populate_pci_child_dt(PCIDevice *dev, 
>void *fdt, int offset,
>                           pci_default_read_config(dev, PCI_DEVICE_ID, 2)));
>     _FDT(fdt_setprop_cell(fdt, offset, "revision-id",
>                           pci_default_read_config(dev, PCI_REVISION_ID, 1)));
>-    _FDT(fdt_setprop_cell(fdt, offset, "class-code",
>-                          pci_default_read_config(dev, PCI_CLASS_PROG, 3)));
>+    _FDT(fdt_setprop_cell(fdt, offset, "class-code", ccode));
>     if (pci_default_read_config(dev, PCI_INTERRUPT_PIN, 1)) {
>         _FDT(fdt_setprop_cell(fdt, offset, "interrupts",
>                  pci_default_read_config(dev, PCI_INTERRUPT_PIN, 1)));
>@@ -1010,11 +1281,10 @@ static int spapr_populate_pci_child_dt(PCIDevice *dev, 
>void *fdt, int offset,
>         _FDT(fdt_setprop(fdt, offset, "udf-supported", NULL, 0));
>     }
>
>-    /* NOTE: this is normally generated by firmware via path/unit name,
>-     * but in our case we must set it manually since it does not get
>-     * processed by OF beforehand
>-     */
>-    _FDT(fdt_setprop_string(fdt, offset, "name", "pci"));
>+    _FDT(fdt_setprop_string(fdt, offset, "name",
>+                            pci_find_device_name((ccode >> 16) & 0xff,
>+                                                 (ccode >> 8) & 0xff,
>+                                                 ccode & 0xff)));
>     buf = spapr_phb_get_loc_code(sphb, dev);
>     if (!buf) {
>         error_report("Failed setting the ibm,loc-code");
>@@ -1051,15 +1321,9 @@ static int spapr_create_pci_child_dt(sPAPRPHBState 
>*phb, PCIDevice *dev,
>                                      void *fdt, int node_offset)
> {
>     int offset, ret;
>-    int slot = PCI_SLOT(dev->devfn);
>-    int func = PCI_FUNC(dev->devfn);
>     char nodename[FDT_NAME_MAX];
>
>-    if (func != 0) {
>-        snprintf(nodename, FDT_NAME_MAX, "address@hidden,%x", slot, func);
>-    } else {
>-        snprintf(nodename, FDT_NAME_MAX, "address@hidden", slot);
>-    }
>+    pci_get_node_name(nodename, FDT_NAME_MAX, dev);
>     offset = fdt_add_subnode(fdt, node_offset, nodename);
>     ret = spapr_populate_pci_child_dt(dev, fdt, offset, phb);
>
>-- 
>2.4.3
>
>