提交 3dbca8e6 编写于 作者: S Stefan Hajnoczi 提交者: Michael S. Tsirkin

virtio-pci: Rename bugs field to flags

The VirtIOPCIProxy bugs field is currently used to enable workarounds
for older guests.  Rename it to flags so that other per-device behavior
can be tracked.

A later patch uses the flags field to remember whether ioeventfd should
be used for virtqueue host notification.
Signed-off-by: NStefan Hajnoczi <stefanha@linux.vnet.ibm.com>
Signed-off-by: NMichael S. Tsirkin <mst@redhat.com>
上级 2f6bfe3b
...@@ -80,9 +80,8 @@ ...@@ -80,9 +80,8 @@
* 12 is historical, and due to x86 page size. */ * 12 is historical, and due to x86 page size. */
#define VIRTIO_PCI_QUEUE_ADDR_SHIFT 12 #define VIRTIO_PCI_QUEUE_ADDR_SHIFT 12
/* We can catch some guest bugs inside here so we continue supporting older /* Flags track per-device state like workarounds for quirks in older guests. */
guests. */ #define VIRTIO_PCI_FLAG_BUS_MASTER_BUG (1 << 0)
#define VIRTIO_PCI_BUG_BUS_MASTER (1 << 0)
/* QEMU doesn't strictly need write barriers since everything runs in /* QEMU doesn't strictly need write barriers since everything runs in
* lock-step. We'll leave the calls to wmb() in though to make it obvious for * lock-step. We'll leave the calls to wmb() in though to make it obvious for
...@@ -95,7 +94,7 @@ ...@@ -95,7 +94,7 @@
typedef struct { typedef struct {
PCIDevice pci_dev; PCIDevice pci_dev;
VirtIODevice *vdev; VirtIODevice *vdev;
uint32_t bugs; uint32_t flags;
uint32_t addr; uint32_t addr;
uint32_t class_code; uint32_t class_code;
uint32_t nvectors; uint32_t nvectors;
...@@ -159,7 +158,7 @@ static int virtio_pci_load_config(void * opaque, QEMUFile *f) ...@@ -159,7 +158,7 @@ static int virtio_pci_load_config(void * opaque, QEMUFile *f)
in ready state. Then we have a buggy guest OS. */ in ready state. Then we have a buggy guest OS. */
if ((proxy->vdev->status & VIRTIO_CONFIG_S_DRIVER_OK) && if ((proxy->vdev->status & VIRTIO_CONFIG_S_DRIVER_OK) &&
!(proxy->pci_dev.config[PCI_COMMAND] & PCI_COMMAND_MASTER)) { !(proxy->pci_dev.config[PCI_COMMAND] & PCI_COMMAND_MASTER)) {
proxy->bugs |= VIRTIO_PCI_BUG_BUS_MASTER; proxy->flags |= VIRTIO_PCI_FLAG_BUS_MASTER_BUG;
} }
return 0; return 0;
} }
...@@ -185,7 +184,7 @@ static void virtio_pci_reset(DeviceState *d) ...@@ -185,7 +184,7 @@ static void virtio_pci_reset(DeviceState *d)
VirtIOPCIProxy *proxy = container_of(d, VirtIOPCIProxy, pci_dev.qdev); VirtIOPCIProxy *proxy = container_of(d, VirtIOPCIProxy, pci_dev.qdev);
virtio_reset(proxy->vdev); virtio_reset(proxy->vdev);
msix_reset(&proxy->pci_dev); msix_reset(&proxy->pci_dev);
proxy->bugs = 0; proxy->flags = 0;
} }
static void virtio_ioport_write(void *opaque, uint32_t addr, uint32_t val) static void virtio_ioport_write(void *opaque, uint32_t addr, uint32_t val)
...@@ -235,7 +234,7 @@ static void virtio_ioport_write(void *opaque, uint32_t addr, uint32_t val) ...@@ -235,7 +234,7 @@ static void virtio_ioport_write(void *opaque, uint32_t addr, uint32_t val)
some safety checks. */ some safety checks. */
if ((val & VIRTIO_CONFIG_S_DRIVER_OK) && if ((val & VIRTIO_CONFIG_S_DRIVER_OK) &&
!(proxy->pci_dev.config[PCI_COMMAND] & PCI_COMMAND_MASTER)) { !(proxy->pci_dev.config[PCI_COMMAND] & PCI_COMMAND_MASTER)) {
proxy->bugs |= VIRTIO_PCI_BUG_BUS_MASTER; proxy->flags |= VIRTIO_PCI_FLAG_BUS_MASTER_BUG;
} }
break; break;
case VIRTIO_MSI_CONFIG_VECTOR: case VIRTIO_MSI_CONFIG_VECTOR:
...@@ -403,7 +402,7 @@ static void virtio_write_config(PCIDevice *pci_dev, uint32_t address, ...@@ -403,7 +402,7 @@ static void virtio_write_config(PCIDevice *pci_dev, uint32_t address,
if (PCI_COMMAND == address) { if (PCI_COMMAND == address) {
if (!(val & PCI_COMMAND_MASTER)) { if (!(val & PCI_COMMAND_MASTER)) {
if (!(proxy->bugs & VIRTIO_PCI_BUG_BUS_MASTER)) { if (!(proxy->flags & VIRTIO_PCI_FLAG_BUS_MASTER_BUG)) {
virtio_set_status(proxy->vdev, virtio_set_status(proxy->vdev,
proxy->vdev->status & ~VIRTIO_CONFIG_S_DRIVER_OK); proxy->vdev->status & ~VIRTIO_CONFIG_S_DRIVER_OK);
} }
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册