@@ -570,7 +570,7 @@ static void ivshmem_save(QEMUFile* f, void *opaque)
visit_start_struct(v, NULL, NULL, "ivshmem", 0, &err);
- pci_device_save(&proxy->dev, f);
+ pci_device_save(&proxy->dev, v, &err);
if (ivshmem_has_feature(proxy, IVSHMEM_MSI)) {
msix_save(&proxy->dev, v, &err);
@@ -609,7 +609,7 @@ static int ivshmem_load(QEMUFile* f, void *opaque, int version_id)
goto out;
}
- ret = pci_device_load(&proxy->dev, f);
+ ret = pci_device_load(&proxy->dev, v, &err);
if (ret) {
goto out;
}
@@ -1149,7 +1149,7 @@ static void openpic_save(QEMUFile* f, void *opaque)
error_report("error saving openpic state: %s", error_get_pretty(err));
error_free(err);
}
- pci_device_save(&opp->pci_dev, f);
+ pci_device_save(&opp->pci_dev, v, &err);
}
static int openpic_load(QEMUFile* f, void *opaque, int version_id)
@@ -1239,7 +1239,7 @@ static int openpic_load(QEMUFile* f, void *opaque, int version_id)
error_report("error loading openpic state: %s", error_get_pretty(err));
error_free(err);
}
- return pci_device_load(&opp->pci_dev, f);
+ return pci_device_load(&opp->pci_dev, v, &err);
}
static void openpic_irq_raise(openpic_t *opp, int n_CPU, IRQ_src_t *src)
@@ -465,22 +465,23 @@ static inline const VMStateDescription *pci_get_vmstate(PCIDevice *s)
return pci_is_express(s) ? &vmstate_pcie_device : &vmstate_pci_device;
}
-void pci_device_save(PCIDevice *s, QEMUFile *f)
+void pci_device_save(PCIDevice *s, Visitor *v, Error **errp)
{
/* Clear interrupt status bit: it is implicit
* in irq_state which we are saving.
* This makes us compatible with old devices
* which never set or clear this bit. */
s->config[PCI_STATUS] &= ~PCI_STATUS_INTERRUPT;
- vmstate_save_state(f, pci_get_vmstate(s), s);
+ vmstate_process(v, pci_get_vmstate(s), s, 0, VMS_SAVE, errp);
/* Restore the interrupt status bit. */
pci_update_irq_status(s);
}
-int pci_device_load(PCIDevice *s, QEMUFile *f)
+int pci_device_load(PCIDevice *s, Visitor *v, Error **errp)
{
int ret;
- ret = vmstate_load_state(f, pci_get_vmstate(s), s, s->version_id);
+ ret = vmstate_process(v, pci_get_vmstate(s), s, s->version_id, VMS_LOAD,
+ errp);
/* Restore the interrupt status bit. */
pci_update_irq_status(s);
return ret;
@@ -11,6 +11,8 @@
#include "isa.h"
#include "pcie.h"
+#include "qapi/qapi-visit-core.h"
+#include "error.h"
/* PCI bus */
@@ -215,8 +217,8 @@ uint32_t pci_default_read_config(PCIDevice *d,
uint32_t address, int len);
void pci_default_write_config(PCIDevice *d,
uint32_t address, uint32_t val, int len);
-void pci_device_save(PCIDevice *s, QEMUFile *f);
-int pci_device_load(PCIDevice *s, QEMUFile *f);
+void pci_device_save(PCIDevice *s, Visitor *v, Error **errp);
+int pci_device_load(PCIDevice *s, Visitor *v, Error **errp);
MemoryRegion *pci_address_space(PCIDevice *dev);
MemoryRegion *pci_address_space_io(PCIDevice *dev);
@@ -201,7 +201,7 @@ static int i440fx_load_old(QEMUFile* f, void *opaque, int version_id)
visit_start_struct(v, NULL, NULL, "i440fx", 0, &err);
- ret = pci_device_load(&d->dev, f);
+ ret = pci_device_load(&d->dev, v, &err);
if (ret < 0) {
goto out;
}
@@ -106,7 +106,7 @@ static void virtio_pci_save_config(void * opaque, QEMUFile *f)
Visitor *v = qemu_file_get_visitor(f);
Error *err = NULL;
- pci_device_save(&proxy->pci_dev, f);
+ pci_device_save(&proxy->pci_dev, v, &err);
msix_save(&proxy->pci_dev, v, &err);
if (msix_present(&proxy->pci_dev)) {
visit_type_uint16(v, &proxy->vdev->config_vector,
@@ -144,7 +144,7 @@ static int virtio_pci_load_config(void * opaque, QEMUFile *f)
Error *err = NULL;
int ret;
- ret = pci_device_load(&proxy->pci_dev, f);
+ ret = pci_device_load(&proxy->pci_dev, v, &err);
if (ret) {
goto out;
}
With all callers converted to Visitors, we can change the interface to accept a Visitor directly, along with an Error** for error propagation. Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com> --- hw/ivshmem.c | 4 ++-- hw/openpic.c | 4 ++-- hw/pci.c | 9 +++++---- hw/pci.h | 6 ++++-- hw/piix_pci.c | 2 +- hw/virtio-pci.c | 4 ++-- 6 files changed, 16 insertions(+), 13 deletions(-)