Re: [Qemu-devel] [PATCH v4 07/11] qapi: Convert savevm

2013-04-10 Thread Markus Armbruster
[Note Cc: Kevin] Eric Blake writes: > On 04/09/2013 11:23 AM, Markus Armbruster wrote: >>> If we mandate that a tag always be supplied, then we cannot create qcow2 >>> files with a snapshot that lacks a tag using just QMP; but even if we do >> >> Are you sure you can do that with the proposed p

Re: [Qemu-devel] [PATCH v4 07/11] qapi: Convert savevm

2013-04-09 Thread Eric Blake
On 04/09/2013 11:23 AM, Markus Armbruster wrote: >> If we mandate that a tag always be supplied, then we cannot create qcow2 >> files with a snapshot that lacks a tag using just QMP; but even if we do > > Are you sure you can do that with the proposed patches? If I read them > correctly, we make

Re: [Qemu-devel] [PATCH v4 07/11] qapi: Convert savevm

2013-04-09 Thread Markus Armbruster
Eric Blake writes: > On 04/09/2013 10:04 AM, Markus Armbruster wrote: > >>> + >>> +## >>> +# @vm-snapshot-save: >>> +# >>> +# Create a snapshot of the whole virtual machine. If tag is >>> provided as @name, >>> +# it is used as human readable identifier. If there is already a snapshot >>> +# with

Re: [Qemu-devel] [PATCH v4 07/11] qapi: Convert savevm

2013-04-09 Thread Eric Blake
On 04/09/2013 10:04 AM, Markus Armbruster wrote: >> + >> +## >> +# @vm-snapshot-save: >> +# >> +# Create a snapshot of the whole virtual machine. If tag is provided as >> @name, >> +# it is used as human readable identifier. If there is already a snapshot >> +# with the same tag or id, the force

Re: [Qemu-devel] [PATCH v4 07/11] qapi: Convert savevm

2013-04-09 Thread Markus Armbruster
Patch no longer applies. Pavel Hrdina writes: > QMP command "vm-snapshot-save" has also extra optional force parameter > to specify whether replace existing snapshot or not. It also returns > information about created snapshot. Took me a minute to realize that you're comparing it to HMP command

Re: [Qemu-devel] [PATCH v4 07/11] qapi: Convert savevm

2013-03-29 Thread Pavel Hrdina
On 29.3.2013 17:12, Eric Blake wrote: On 03/29/2013 08:12 AM, Pavel Hrdina wrote: QMP command "vm-snapshot-save" has also extra optional force parameter to specify whether replace existing snapshot or not. It also returns information about created snapshot. Signed-off-by: Pavel Hrdina --- hm

Re: [Qemu-devel] [PATCH v4 07/11] qapi: Convert savevm

2013-03-29 Thread Eric Blake
On 03/29/2013 08:12 AM, Pavel Hrdina wrote: > QMP command "vm-snapshot-save" has also extra optional force parameter > to specify whether replace existing snapshot or not. It also returns > information about created snapshot. > > Signed-off-by: Pavel Hrdina > --- > hmp-commands.hx | 4 +

[Qemu-devel] [PATCH v4 07/11] qapi: Convert savevm

2013-03-29 Thread Pavel Hrdina
QMP command "vm-snapshot-save" has also extra optional force parameter to specify whether replace existing snapshot or not. It also returns information about created snapshot. Signed-off-by: Pavel Hrdina --- hmp-commands.hx | 4 ++-- hmp.c | 11 + hmp.h