Re: [PATCH v2] device-assignment: Register as un-migratable

[Jan Kiszka <jan.kiszka@web.de>]

[-- Attachment #1: Type: text/plain, Size: 1786 bytes --]

Am 16.11.2010 00:06, Alex Williamson wrote:
> Use register_device_unmigratable() to declare ourselves as
> non-migratable.
> 
> Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
> ---
> 
>  v2: Use dummy vmsd instead of dummy save_state
> 
>  hw/device-assignment.c |   10 ++++++++++
>  1 files changed, 10 insertions(+), 0 deletions(-)
> 
> diff --git a/hw/device-assignment.c b/hw/device-assignment.c
> index bde231d..154bb1a 100644
> --- a/hw/device-assignment.c
> +++ b/hw/device-assignment.c
> @@ -1434,6 +1434,10 @@ static void assigned_dev_unregister_msix_mmio(AssignedDevice *dev)
>      dev->msix_table_page = NULL;
>  }
>  
> +static const VMStateDescription vmstate_assigned_device = {
> +    .name = "pci-assign"
> +};
> +
>  static int assigned_initfn(struct PCIDevice *pci_dev)
>  {
>      AssignedDevice *dev = DO_UPCAST(AssignedDevice, dev, pci_dev);
> @@ -1490,6 +1494,11 @@ static int assigned_initfn(struct PCIDevice *pci_dev)
>  
>      assigned_dev_load_option_rom(dev);
>      QLIST_INSERT_HEAD(&devs, dev, next);
> +
> +    /* Register a vmsd so that we can mark it unmigratable. */
> +    vmstate_register(&dev->dev.qdev, 0, &vmstate_assigned_device, dev);

Almost: You can register this vmstate description via assign_info
(.qdev.vmsd = ....).

Jan

> +    register_device_unmigratable(&dev->dev.qdev, "pci-assign", dev);
> +
>      return 0;
>  
>  assigned_out:
> @@ -1503,6 +1512,7 @@ static int assigned_exitfn(struct PCIDevice *pci_dev)
>  {
>      AssignedDevice *dev = DO_UPCAST(AssignedDevice, dev, pci_dev);
>  
> +    vmstate_unregister(&dev->dev.qdev, &vmstate_assigned_device, dev);
>      QLIST_REMOVE(dev, next);
>      deassign_device(dev);
>      free_assigned_device(dev);
> 



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 259 bytes --]