motorhead/qemu
1
0
Fork 0
mirror of https://github.com/Motorhead1991/qemu.git synced 2025-08-06 01:03:55 -06:00
Code Issues Projects Releases Packages Wiki Activity Actions

vfio/migration: Add Error** argument to .vfio_save_config() handler

Browse source 
Use vmstate_save_state_with_err() to improve error reporting in the
callers and store a reported error under the migration stream. Add
documentation while at it.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Reviewed-by: Avihai Horon <avihaih@nvidia.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>
This commit is contained in:
Cédric Le Goater 2024-05-16 14:46:54 +02:00
parent fbd2469a66
commit 3783f814e7
3 changed files with 45 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

23
hw/vfio/migration.c
View file

@ -193,21 +193,30 @@ static int vfio_load_buffer(QEMUFile *f, VFIODevice *vbasedev,
return ret; return ret;
} }
static int vfio_save_device_config_state(QEMUFile *f, void *opaque) static int vfio_save_device_config_state(QEMUFile *f, void *opaque,
Error **errp)
{ {
VFIODevice *vbasedev = opaque; VFIODevice *vbasedev = opaque;
int ret;
qemu_put_be64(f, VFIO_MIG_FLAG_DEV_CONFIG_STATE); qemu_put_be64(f, VFIO_MIG_FLAG_DEV_CONFIG_STATE);
if (vbasedev->ops && vbasedev->ops->vfio_save_config) { if (vbasedev->ops && vbasedev->ops->vfio_save_config) {
vbasedev->ops->vfio_save_config(vbasedev, f); ret = vbasedev->ops->vfio_save_config(vbasedev, f, errp);
if (ret) {
return ret;
}
} }
qemu_put_be64(f, VFIO_MIG_FLAG_END_OF_STATE); qemu_put_be64(f, VFIO_MIG_FLAG_END_OF_STATE);
trace_vfio_save_device_config_state(vbasedev->name); trace_vfio_save_device_config_state(vbasedev->name);
return qemu_file_get_error(f); ret = qemu_file_get_error(f);
if (ret < 0) {
error_setg_errno(errp, -ret, "Failed to save state");
}
return ret;
} }
static int vfio_load_device_config_state(QEMUFile *f, void *opaque) static int vfio_load_device_config_state(QEMUFile *f, void *opaque)
@ -592,13 +601,15 @@ static int vfio_save_complete_precopy(QEMUFile *f, void *opaque)
static void vfio_save_state(QEMUFile *f, void *opaque) static void vfio_save_state(QEMUFile *f, void *opaque)
{ {
VFIODevice *vbasedev = opaque; VFIODevice *vbasedev = opaque;
Error *local_err = NULL;
int ret; int ret;
ret = vfio_save_device_config_state(f, opaque); ret = vfio_save_device_config_state(f, opaque, &local_err);
if (ret) { if (ret) {
error_report("%s: Failed to save device config space", error_prepend(&local_err,
"vfio: Failed to save device config space of %s - ",
vbasedev->name); vbasedev->name);
qemu_file_set_error(f, ret); qemu_file_set_error_obj(f, ret, local_err);
} }
} }

5
hw/vfio/pci.c
View file

@ -2586,11 +2586,12 @@ static const VMStateDescription vmstate_vfio_pci_config = {
} }
}; };
static void vfio_pci_save_config(VFIODevice *vbasedev, QEMUFile *f) static int vfio_pci_save_config(VFIODevice *vbasedev, QEMUFile *f, Error **errp)
{ {
VFIOPCIDevice *vdev = container_of(vbasedev, VFIOPCIDevice, vbasedev); VFIOPCIDevice *vdev = container_of(vbasedev, VFIOPCIDevice, vbasedev);
vmstate_save_state(f, &vmstate_vfio_pci_config, vdev, NULL); return vmstate_save_state_with_err(f, &vmstate_vfio_pci_config, vdev, NULL,
errp);
} }
static int vfio_pci_load_config(VFIODevice *vbasedev, QEMUFile *f) static int vfio_pci_load_config(VFIODevice *vbasedev, QEMUFile *f)

25
include/hw/vfio/vfio-common.h
View file

@ -133,7 +133,30 @@ struct VFIODeviceOps {
int (*vfio_hot_reset_multi)(VFIODevice *vdev); int (*vfio_hot_reset_multi)(VFIODevice *vdev);
void (*vfio_eoi)(VFIODevice *vdev); void (*vfio_eoi)(VFIODevice *vdev);
Object *(*vfio_get_object)(VFIODevice *vdev); Object *(*vfio_get_object)(VFIODevice *vdev);
void (*vfio_save_config)(VFIODevice *vdev, QEMUFile *f);
/**
* @vfio_save_config
*
* Save device config state
*
* @vdev: #VFIODevice for which to save the config
* @f: #QEMUFile where to send the data
* @errp: pointer to Error*, to store an error if it happens.
*
* Returns zero to indicate success and negative for error
*/
int (*vfio_save_config)(VFIODevice *vdev, QEMUFile *f, Error **errp);
/**
* @vfio_load_config
*
* Load device config state
*
* @vdev: #VFIODevice for which to load the config
* @f: #QEMUFile where to get the data
*
* Returns zero to indicate success and negative for error
*/
int (*vfio_load_config)(VFIODevice *vdev, QEMUFile *f); int (*vfio_load_config)(VFIODevice *vdev, QEMUFile *f);
}; };