From 1d388b4fda2c4c9d00dc6ae91aaf35eb9fc04c26 Mon Sep 17 00:00:00 2001

From: Paolo Bonzini <pbonzini@redhat.com>

Date: Fri, 20 Sep 2013 16:57:51 +0200

Subject: [PATCH] virtio-pci: remove vdev field

The vdev field is complicated to synchronize.  Just access the

BusState's list of children.

Cc: qemu-stable@nongnu.org

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

---

 hw/s390x/virtio-ccw.h  |   1 -

 hw/virtio/virtio-pci.c | 107 +++++++++++++++++++++++++++++--------------------

 hw/virtio/virtio-pci.h |   1 -

 3 files changed, 63 insertions(+), 46 deletions(-)

diff --git a/hw/s390x/virtio-ccw.h b/hw/s390x/virtio-ccw.h

index 96d6f5d..00932c7 100644

--- a/hw/s390x/virtio-ccw.h

+++ b/hw/s390x/virtio-ccw.h

@@ -77,7 +77,6 @@ typedef struct VirtIOCCWDeviceClass {

 struct VirtioCcwDevice {

     DeviceState parent_obj;

     SubchDev *sch;

-    VirtIODevice *vdev;

     char *bus_id;

     uint32_t host_features[VIRTIO_CCW_FEATURE_SIZE];

     VirtioBusState bus;

diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pci.c

index 55617a6..6fd6d6d 100644

--- a/hw/virtio/virtio-pci.c

+++ b/hw/virtio/virtio-pci.c

@@ -112,31 +112,39 @@ static inline VirtIOPCIProxy *to_virtio_pci_proxy_fast(DeviceState *d)

 static void virtio_pci_notify(DeviceState *d, uint16_t vector)

 {

     VirtIOPCIProxy *proxy = to_virtio_pci_proxy_fast(d);

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

+

     if (msix_enabled(&proxy->pci_dev))

         msix_notify(&proxy->pci_dev, vector);

     else

-        qemu_set_irq(proxy->pci_dev.irq[0], proxy->vdev->isr & 1);

+        qemu_set_irq(proxy->pci_dev.irq[0], vdev->isr & 1);

 }

 static void virtio_pci_save_config(DeviceState *d, QEMUFile *f)

 {

     VirtIOPCIProxy *proxy = to_virtio_pci_proxy(d);

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

+

     pci_device_save(&proxy->pci_dev, f);

     msix_save(&proxy->pci_dev, f);

     if (msix_present(&proxy->pci_dev))

-        qemu_put_be16(f, proxy->vdev->config_vector);

+        qemu_put_be16(f, vdev->config_vector);

 }

 static void virtio_pci_save_queue(DeviceState *d, int n, QEMUFile *f)

 {

     VirtIOPCIProxy *proxy = to_virtio_pci_proxy(d);

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

+

     if (msix_present(&proxy->pci_dev))

-        qemu_put_be16(f, virtio_queue_vector(proxy->vdev, n));

+        qemu_put_be16(f, virtio_queue_vector(vdev, n));

 }

 static int virtio_pci_load_config(DeviceState *d, QEMUFile *f)

 {

     VirtIOPCIProxy *proxy = to_virtio_pci_proxy(d);

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

+

     int ret;

     ret = pci_device_load(&proxy->pci_dev, f);

     if (ret) {

@@ -145,12 +153,12 @@ static int virtio_pci_load_config(DeviceState *d, QEMUFile *f)

     msix_unuse_all_vectors(&proxy->pci_dev);

     msix_load(&proxy->pci_dev, f);

     if (msix_present(&proxy->pci_dev)) {

-        qemu_get_be16s(f, &proxy->vdev->config_vector);

+        qemu_get_be16s(f, &vdev->config_vector);

     } else {

-        proxy->vdev->config_vector = VIRTIO_NO_VECTOR;

+        vdev->config_vector = VIRTIO_NO_VECTOR;

     }

-    if (proxy->vdev->config_vector != VIRTIO_NO_VECTOR) {

-        return msix_vector_use(&proxy->pci_dev, proxy->vdev->config_vector);

+    if (vdev->config_vector != VIRTIO_NO_VECTOR) {

+        return msix_vector_use(&proxy->pci_dev, vdev->config_vector);

     }

     return 0;

 }

@@ -158,13 +166,15 @@ static int virtio_pci_load_config(DeviceState *d, QEMUFile *f)

 static int virtio_pci_load_queue(DeviceState *d, int n, QEMUFile *f)

 {

     VirtIOPCIProxy *proxy = to_virtio_pci_proxy(d);

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

+

     uint16_t vector;

     if (msix_present(&proxy->pci_dev)) {

         qemu_get_be16s(f, &vector);

     } else {

         vector = VIRTIO_NO_VECTOR;

     }

-    virtio_queue_set_vector(proxy->vdev, n, vector);

+    virtio_queue_set_vector(vdev, n, vector);

     if (vector != VIRTIO_NO_VECTOR) {

         return msix_vector_use(&proxy->pci_dev, vector);

     }

@@ -174,7 +184,8 @@ static int virtio_pci_load_queue(DeviceState *d, int n, QEMUFile *f)

 static int virtio_pci_set_host_notifier_internal(VirtIOPCIProxy *proxy,

                                                  int n, bool assign, bool set_handler)

 {

-    VirtQueue *vq = virtio_get_queue(proxy->vdev, n);

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

+    VirtQueue *vq = virtio_get_queue(vdev, n);

     EventNotifier *notifier = virtio_queue_get_host_notifier(vq);

     int r = 0;

@@ -199,6 +210,7 @@ static int virtio_pci_set_host_notifier_internal(VirtIOPCIProxy *proxy,

 static void virtio_pci_start_ioeventfd(VirtIOPCIProxy *proxy)

 {

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

     int n, r;

     if (!(proxy->flags & VIRTIO_PCI_FLAG_USE_IOEVENTFD) ||

@@ -208,7 +220,7 @@ static void virtio_pci_start_ioeventfd(VirtIOPCIProxy *proxy)

     }

     for (n = 0; n < VIRTIO_PCI_QUEUE_MAX; n++) {

-        if (!virtio_queue_get_num(proxy->vdev, n)) {

+        if (!virtio_queue_get_num(vdev, n)) {

             continue;

         }

@@ -222,7 +234,7 @@ static void virtio_pci_start_ioeventfd(VirtIOPCIProxy *proxy)

 assign_error:

     while (--n >= 0) {

-        if (!virtio_queue_get_num(proxy->vdev, n)) {

+        if (!virtio_queue_get_num(vdev, n)) {

             continue;

         }

@@ -235,6 +247,7 @@ assign_error:

 static void virtio_pci_stop_ioeventfd(VirtIOPCIProxy *proxy)

 {

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

     int r;

     int n;

@@ -243,7 +256,7 @@ static void virtio_pci_stop_ioeventfd(VirtIOPCIProxy *proxy)

     }

     for (n = 0; n < VIRTIO_PCI_QUEUE_MAX; n++) {

-        if (!virtio_queue_get_num(proxy->vdev, n)) {

+        if (!virtio_queue_get_num(vdev, n)) {

             continue;

         }

@@ -256,7 +269,7 @@ static void virtio_pci_stop_ioeventfd(VirtIOPCIProxy *proxy)

 static void virtio_ioport_write(void *opaque, uint32_t addr, uint32_t val)

 {

     VirtIOPCIProxy *proxy = opaque;

-    VirtIODevice *vdev = proxy->vdev;

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

     hwaddr pa;

     switch (addr) {

@@ -271,7 +284,7 @@ static void virtio_ioport_write(void *opaque, uint32_t addr, uint32_t val)

         pa = (hwaddr)val << VIRTIO_PCI_QUEUE_ADDR_SHIFT;

         if (pa == 0) {

             virtio_pci_stop_ioeventfd(proxy);

-            virtio_reset(proxy->vdev);

+            virtio_reset(vdev);

             msix_unuse_all_vectors(&proxy->pci_dev);

         }

         else

@@ -298,7 +311,7 @@ static void virtio_ioport_write(void *opaque, uint32_t addr, uint32_t val)

         }

         if (vdev->status == 0) {

-            virtio_reset(proxy->vdev);

+            virtio_reset(vdev);

             msix_unuse_all_vectors(&proxy->pci_dev);

         }

@@ -334,7 +347,7 @@ static void virtio_ioport_write(void *opaque, uint32_t addr, uint32_t val)

 static uint32_t virtio_ioport_read(VirtIOPCIProxy *proxy, uint32_t addr)

 {

-    VirtIODevice *vdev = proxy->vdev;

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

     uint32_t ret = 0xFFFFFFFF;

     switch (addr) {

@@ -380,6 +393,7 @@ static uint64_t virtio_pci_config_read(void *opaque, hwaddr addr,

                                        unsigned size)

 {

     VirtIOPCIProxy *proxy = opaque;

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

     uint32_t config = VIRTIO_PCI_CONFIG(&proxy->pci_dev);

     uint64_t val = 0;

     if (addr < config) {

@@ -389,16 +403,16 @@ static uint64_t virtio_pci_config_read(void *opaque, hwaddr addr,

     switch (size) {

     case 1:

-        val = virtio_config_readb(proxy->vdev, addr);

+        val = virtio_config_readb(vdev, addr);

         break;

     case 2:

-        val = virtio_config_readw(proxy->vdev, addr);

+        val = virtio_config_readw(vdev, addr);

         if (virtio_is_big_endian()) {

             val = bswap16(val);

         }

         break;

     case 4:

-        val = virtio_config_readl(proxy->vdev, addr);

+        val = virtio_config_readl(vdev, addr);

         if (virtio_is_big_endian()) {

             val = bswap32(val);

         }

@@ -412,6 +426,7 @@ static void virtio_pci_config_write(void *opaque, hwaddr addr,

 {

     VirtIOPCIProxy *proxy = opaque;

     uint32_t config = VIRTIO_PCI_CONFIG(&proxy->pci_dev);

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

     if (addr < config) {

         virtio_ioport_write(proxy, addr, val);

         return;

@@ -423,19 +438,19 @@ static void virtio_pci_config_write(void *opaque, hwaddr addr,

      */

     switch (size) {

     case 1:

-        virtio_config_writeb(proxy->vdev, addr, val);

+        virtio_config_writeb(vdev, addr, val);

         break;

     case 2:

         if (virtio_is_big_endian()) {

             val = bswap16(val);

         }

-        virtio_config_writew(proxy->vdev, addr, val);

+        virtio_config_writew(vdev, addr, val);

         break;

     case 4:

         if (virtio_is_big_endian()) {

             val = bswap32(val);

         }

-        virtio_config_writel(proxy->vdev, addr, val);

+        virtio_config_writel(vdev, addr, val);

         break;

     }

 }

@@ -454,6 +469,7 @@ static void virtio_write_config(PCIDevice *pci_dev, uint32_t address,

                                 uint32_t val, int len)

 {

     VirtIOPCIProxy *proxy = DO_UPCAST(VirtIOPCIProxy, pci_dev, pci_dev);

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

     pci_default_write_config(pci_dev, address, val, len);

@@ -461,8 +477,7 @@ static void virtio_write_config(PCIDevice *pci_dev, uint32_t address,

         !(pci_dev->config[PCI_COMMAND] & PCI_COMMAND_MASTER) &&

         !(proxy->flags & VIRTIO_PCI_FLAG_BUS_MASTER_BUG)) {

         virtio_pci_stop_ioeventfd(proxy);

-        virtio_set_status(proxy->vdev,

-                          proxy->vdev->status & ~VIRTIO_CONFIG_S_DRIVER_OK);

+        virtio_set_status(vdev, vdev->status & ~VIRTIO_CONFIG_S_DRIVER_OK);

     }

 }

@@ -505,7 +520,8 @@ static int kvm_virtio_pci_irqfd_use(VirtIOPCIProxy *proxy,

                                  unsigned int vector)

 {

     VirtIOIRQFD *irqfd = &proxy->vector_irqfd[vector];

-    VirtQueue *vq = virtio_get_queue(proxy->vdev, queue_no);

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

+    VirtQueue *vq = virtio_get_queue(vdev, queue_no);

     EventNotifier *n = virtio_queue_get_guest_notifier(vq);

     int ret;

     ret = kvm_irqchip_add_irqfd_notifier(kvm_state, n, irqfd->virq);

@@ -516,7 +532,8 @@ static void kvm_virtio_pci_irqfd_release(VirtIOPCIProxy *proxy,

                                       unsigned int queue_no,

                                       unsigned int vector)

 {

-    VirtQueue *vq = virtio_get_queue(proxy->vdev, queue_no);

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

+    VirtQueue *vq = virtio_get_queue(vdev, queue_no);

     EventNotifier *n = virtio_queue_get_guest_notifier(vq);

     VirtIOIRQFD *irqfd = &proxy->vector_irqfd[vector];

     int ret;

@@ -528,7 +545,7 @@ static void kvm_virtio_pci_irqfd_release(VirtIOPCIProxy *proxy,

 static int kvm_virtio_pci_vector_use(VirtIOPCIProxy *proxy, int nvqs)

 {

     PCIDevice *dev = &proxy->pci_dev;

-    VirtIODevice *vdev = proxy->vdev;

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

     VirtioDeviceClass *k = VIRTIO_DEVICE_GET_CLASS(vdev);

     unsigned int vector;

     int ret, queue_no;

@@ -577,7 +594,7 @@ undo:

 static void kvm_virtio_pci_vector_release(VirtIOPCIProxy *proxy, int nvqs)

 {

     PCIDevice *dev = &proxy->pci_dev;

-    VirtIODevice *vdev = proxy->vdev;

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

     unsigned int vector;

     int queue_no;

     VirtioDeviceClass *k = VIRTIO_DEVICE_GET_CLASS(vdev);

@@ -605,8 +622,9 @@ static int virtio_pci_vq_vector_unmask(VirtIOPCIProxy *proxy,

                                        unsigned int vector,

                                        MSIMessage msg)

 {

-    VirtioDeviceClass *k = VIRTIO_DEVICE_GET_CLASS(proxy->vdev);

-    VirtQueue *vq = virtio_get_queue(proxy->vdev, queue_no);

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

+    VirtioDeviceClass *k = VIRTIO_DEVICE_GET_CLASS(vdev);

+    VirtQueue *vq = virtio_get_queue(vdev, queue_no);

     EventNotifier *n = virtio_queue_get_guest_notifier(vq);

     VirtIOIRQFD *irqfd;

     int ret = 0;

@@ -625,10 +643,10 @@ static int virtio_pci_vq_vector_unmask(VirtIOPCIProxy *proxy,

      * Otherwise, set it up now.

      */

     if (k->guest_notifier_mask) {

-        k->guest_notifier_mask(proxy->vdev, queue_no, false);

+        k->guest_notifier_mask(vdev, queue_no, false);

         /* Test after unmasking to avoid losing events. */

         if (k->guest_notifier_pending &&

-            k->guest_notifier_pending(proxy->vdev, queue_no)) {

+            k->guest_notifier_pending(vdev, queue_no)) {

             event_notifier_set(n);

         }

     } else {

@@ -641,13 +659,14 @@ static void virtio_pci_vq_vector_mask(VirtIOPCIProxy *proxy,

                                              unsigned int queue_no,

                                              unsigned int vector)

 {

-    VirtioDeviceClass *k = VIRTIO_DEVICE_GET_CLASS(proxy->vdev);

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

+    VirtioDeviceClass *k = VIRTIO_DEVICE_GET_CLASS(vdev);

     /* If guest supports masking, keep irqfd but mask it.

      * Otherwise, clean it up now.

      */ 

     if (k->guest_notifier_mask) {

-        k->guest_notifier_mask(proxy->vdev, queue_no, true);

+        k->guest_notifier_mask(vdev, queue_no, true);

     } else {

         kvm_virtio_pci_irqfd_release(proxy, queue_no, vector);

     }

@@ -657,7 +676,7 @@ static int virtio_pci_vector_unmask(PCIDevice *dev, unsigned vector,

                                     MSIMessage msg)

 {

     VirtIOPCIProxy *proxy = container_of(dev, VirtIOPCIProxy, pci_dev);

-    VirtIODevice *vdev = proxy->vdev;

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

     int ret, queue_no;

     for (queue_no = 0; queue_no < proxy->nvqs_with_notifiers; queue_no++) {

@@ -687,7 +706,7 @@ undo:

 static void virtio_pci_vector_mask(PCIDevice *dev, unsigned vector)

 {

     VirtIOPCIProxy *proxy = container_of(dev, VirtIOPCIProxy, pci_dev);

-    VirtIODevice *vdev = proxy->vdev;

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

     int queue_no;

     for (queue_no = 0; queue_no < proxy->nvqs_with_notifiers; queue_no++) {

@@ -706,7 +725,7 @@ static void virtio_pci_vector_poll(PCIDevice *dev,

                                    unsigned int vector_end)

 {

     VirtIOPCIProxy *proxy = container_of(dev, VirtIOPCIProxy, pci_dev);

-    VirtIODevice *vdev = proxy->vdev;

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

     VirtioDeviceClass *k = VIRTIO_DEVICE_GET_CLASS(vdev);

     int queue_no;

     unsigned int vector;

@@ -738,8 +757,9 @@ static int virtio_pci_set_guest_notifier(DeviceState *d, int n, bool assign,

                                          bool with_irqfd)

 {

     VirtIOPCIProxy *proxy = to_virtio_pci_proxy(d);

-    VirtioDeviceClass *vdc = VIRTIO_DEVICE_GET_CLASS(proxy->vdev);

-    VirtQueue *vq = virtio_get_queue(proxy->vdev, n);

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

+    VirtioDeviceClass *vdc = VIRTIO_DEVICE_GET_CLASS(vdev);

+    VirtQueue *vq = virtio_get_queue(vdev, n);

     EventNotifier *notifier = virtio_queue_get_guest_notifier(vq);

     if (assign) {

@@ -754,7 +774,7 @@ static int virtio_pci_set_guest_notifier(DeviceState *d, int n, bool assign,

     }

     if (!msix_enabled(&proxy->pci_dev) && vdc->guest_notifier_mask) {

-        vdc->guest_notifier_mask(proxy->vdev, n, !assign);

+        vdc->guest_notifier_mask(vdev, n, !assign);

     }

     return 0;

@@ -769,7 +789,7 @@ static bool virtio_pci_query_guest_notifiers(DeviceState *d)

 static int virtio_pci_set_guest_notifiers(DeviceState *d, int nvqs, bool assign)

 {

     VirtIOPCIProxy *proxy = to_virtio_pci_proxy(d);

-    VirtIODevice *vdev = proxy->vdev;

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

     VirtioDeviceClass *k = VIRTIO_DEVICE_GET_CLASS(vdev);

     int r, n;

     bool with_irqfd = msix_enabled(&proxy->pci_dev) &&

@@ -863,11 +883,12 @@ static int virtio_pci_set_host_notifier(DeviceState *d, int n, bool assign)

 static void virtio_pci_vmstate_change(DeviceState *d, bool running)

 {

     VirtIOPCIProxy *proxy = to_virtio_pci_proxy(d);

+    VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);

     if (running) {

         /* Try to find out if the guest has bus master disabled, but is

            in ready state. Then we have a buggy guest OS. */

-        if ((proxy->vdev->status & VIRTIO_CONFIG_S_DRIVER_OK) &&

+        if ((vdev->status & VIRTIO_CONFIG_S_DRIVER_OK) &&

             !(proxy->pci_dev.config[PCI_COMMAND] & PCI_COMMAND_MASTER)) {

             proxy->flags |= VIRTIO_PCI_FLAG_BUS_MASTER_BUG;

         }

@@ -942,8 +963,6 @@ static void virtio_pci_device_plugged(DeviceState *d)

     uint8_t *config;

     uint32_t size;

-    proxy->vdev = virtio_bus_get_device(bus);

-

     config = proxy->pci_dev.config;

     if (proxy->class_code) {

         pci_config_set_class(config, proxy->class_code);

diff --git a/hw/virtio/virtio-pci.h b/hw/virtio/virtio-pci.h

index 917bcc5..dc332ae 100644

--- a/hw/virtio/virtio-pci.h

+++ b/hw/virtio/virtio-pci.h

@@ -82,7 +82,6 @@ typedef struct VirtioPCIClass {

 struct VirtIOPCIProxy {

     PCIDevice pci_dev;

-    VirtIODevice *vdev;

     MemoryRegion bar;

     uint32_t flags;

     uint32_t class_code;