qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v4 1/1] Introduce "xen-load-devices-state"


From: Changlong Xie
Subject: Re: [Qemu-devel] [PATCH v4 1/1] Introduce "xen-load-devices-state"
Date: Thu, 2 Jun 2016 18:37:10 +0800
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0

On 06/02/2016 06:16 PM, Dr. David Alan Gilbert wrote:
* Stefano Stabellini (address@hidden) wrote:
On Thu, 2 Jun 2016, Stefano Stabellini wrote:
On Wed, 1 Jun 2016, Dr. David Alan Gilbert wrote:
* Amit Shah (address@hidden) wrote:
Dave, can you take a look?

Yes, I think I'm happy with it;

Reviewed-by: Dr. David Alan Gilbert <address@hidden>

I think people might try interesting non-Xen things with it; for
example I think if you backed QEMU's main RAM by a /tmp file
perhaps you could start a guest up even under KVM just
by loading the devices.

I added this patch to the Xen queue.

I spoke too quickly. I get:

/local/qemu/migration/savevm.c: In function 'qmp_xen_load_devices_state':
/local/qemu/migration/savevm.c:2089:5: error: implicit declaration of function 
'qemu_fopen' [-Werror=implicit-function-declaration]
/local/qemu/migration/savevm.c:2089:5: error: nested extern declaration of 
'qemu_fopen' [-Werror=nested-externs]
/local/qemu/migration/savevm.c:2089:7: error: assignment makes pointer from 
integer without a cast [-Werror]

Ah, yes, your problem is that Dan's crypto changes have probably just removed 
that;
see qmp_xen_save_devices_state and 8925839f  that changed it to use the new 
qio_channel_file_new_path.


Yes, i just send another patchset.

Dave

   CC    i386-softmmu/hw/display/virtio-vga.o
cc1: all warnings being treated as errors
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK


.






reply via email to

[Prev in Thread] Current Thread [Next in Thread]