Re: [Qemu-devel] [PULL] non-migratable devices

[Anthony Liguori]

On 07/20/2011 05:09 AM, Gerd Hoffmann wrote:
>    Hi,
>
> This patch series adds an easy way to tag devices as non-migratable
> and puts it into use for ahci, ehci and a number of usb devices.
>
> cheers,
>    Gerd
>
> The following changes since commit 03ff09580ef6cbc4a893b6e3e6bbff33180ec70a:
>
>    Merge remote-tracking branch 'agraf/xen-next' into staging (2011-07-19 
> 08:04:35 -0500)
>
> are available in the git repository at:
>
>    git://git.kraxel.org/qemu migration.1
>
> Gerd Hoffmann (9):
>        vmstate: add no_migrate flag to VMStateDescription
>        vmstate: complain about devices without vmstate

I appreciate the sentiment of this patch but this cannot go into 0.15. 
Lots of tools parse the output of QEMU and introducing something like 
this is going to create problems for those tools.

I'm not really sure this is totally appropriate for the development 
branch either.  I think something like a migration tainted flag that 
showed in info migrate would be much more appropriate.

Regards,

Anthony Liguori

>        ahci doesn't support migration
>        ehci doesn't support migration
>        usb storage: first migration support bits.
>        usb-wacom doesn't support migration
>        usb-bt doesn't support migration
>        usb-net doesn't support migration
>        usb-serial doesn't support migration
>
>   hw/hw.h         |    3 +++
>   hw/ide/ich.c    |    6 ++++++
>   hw/qdev.c       |    7 ++++++-
>   hw/usb-bt.c     |    6 ++++++
>   hw/usb-ehci.c   |    7 +++++++
>   hw/usb-msd.c    |   12 ++++++++++++
>   hw/usb-net.c    |    6 ++++++
>   hw/usb-serial.c |    7 +++++++
>   hw/usb-wacom.c  |    6 ++++++
>   savevm.c        |    1 +
>   10 files changed, 60 insertions(+), 1 deletions(-)