hw/ppc/spapr_pci: Do not reject VFs created after a PF
authorAkihiko Odaki <akihiko.odaki@daynix.com>
Thu, 16 Jan 2025 09:00:53 +0000 (18:00 +0900)
committerMichael S. Tsirkin <mst@redhat.com>
Thu, 20 Feb 2025 23:23:19 +0000 (18:23 -0500)
A PF may automatically create VFs and the PF may be function 0.

Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Reviewed-by: Shivaprasad G Bhat <sbhat@linux.ibm.com>
Tested-by: Shivaprasad G Bhat <sbhat@linux.ibm.com>
Message-Id: <20250116-reuse-v20-2-7cb370606368@daynix.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
hw/ppc/spapr_pci.c

index b94e4ba1314f96644e6a82fe3cedff075e853cf2..e0a9d50edc3df3a04a19fbb48a0d9786371439ed 100644 (file)
@@ -1549,7 +1549,9 @@ static void spapr_pci_pre_plug(HotplugHandler *plug_handler,
      * hotplug, we do not allow functions to be hotplugged to a
      * slot that already has function 0 present
      */
-    if (plugged_dev->hotplugged && bus->devices[PCI_DEVFN(slotnr, 0)] &&
+    if (plugged_dev->hotplugged &&
+        !pci_is_vf(pdev) &&
+        bus->devices[PCI_DEVFN(slotnr, 0)] &&
         PCI_FUNC(pdev->devfn) != 0) {
         error_setg(errp, "PCI: slot %d function 0 already occupied by %s,"
                    " additional functions can no longer be exposed to guest.",