Re: [Qemu-devel] [PATCH v4 11/11] qmp-commands.hx: Update the supported 'transaction' operations

[Kashyap Chamarthy <kchamart@redhat.com>]

On Tue, May 19, 2015 at 11:37:32AM -0400, John Snow wrote:
> > On Mon, May 18, 2015 at 06:22:22PM +0200, Max Reitz wrote:
> >> On 12.05.2015 01:04, John Snow wrote:

[. . .]

> >>> diff --git a/qmp-commands.hx b/qmp-commands.hx
> >>> index 7506774..363126a 100644
> >>> --- a/qmp-commands.hx
> >>> +++ b/qmp-commands.hx
> >>> @@ -1238,11 +1238,14 @@ SQMP
> >>>  transaction
> >>>  -----------
> >>> -Atomically operate on one or more block devices.  The only supported operations
> >>> -for now are drive-backup, internal and external snapshotting.  A list of
> >>> -dictionaries is accepted, that contains the actions to be performed.
> >>> -If there is any failure performing any of the operations, all operations
> >>> -for the group are abandoned.
> >>> +Atomically operate on one or more block devices.  Operations that are
> >>> +currently supported: drive-backup, blockdev-backup,
> >>> +blockdev-snapshot-sync, blockdev-snapshot-internal-sync, abort,
> >>> +block-dirty-bitmap-add, block-dirty-bitmap-clear
> >>
> >> Hm, seven operations... Worth making it a real list?
> > 
> > I don't have a preference. FWIW, I think it still retains the
> > readability. And, not sure if it's worth the churn.
> > 
> >>> (refer to the
> >>> +qemu/qapi-schema.json file for minimum required QEMU versions for these
> >>> +operations).  A list of dictionaries is accepted, that contains the
> >>> +actions to be performed.  If there is any failure performing any of the
> >>> +operations, all operations for the group are abandoned.
> >>>  For external snapshots, the dictionary contains the device, the file to use for
> >>>  the new snapshot, and the format.  The default format, if not specified, is
> >>
> > 
> 
> I have to respin the series anyway, so if you want Kashyap, you can
> rewrite this and send it to me privately for inclusion

Done. Sent you (and Max) the revised patch. Max provided his R-b. Max
wondered whether you'd include this in your series on #qemu. I'll assume
you'll do.

Thanks.

-- 
/kashyap