On Thu, 21 Feb 2013 14:47:08 +0100 Stefan Hajnoczi <stefa...@gmail.com> wrote:
> On Thu, Feb 21, 2013 at 06:21:32AM +0000, Dietmar Maurer wrote: > > > > +## > > > > +# @backup: > > > > +# > > > > +# Starts a VM backup. > > > > +# > > > > +# @backup-file: the backup file name > > > > +# > > > > +# @format: format of the backup file > > > > +# > > > > +# @config-filename: #optional name of a configuration file to include > > > > +into # the backup archive. > > > > +# > > > > +# @speed: #optional the maximum speed, in bytes per second # # > > > > +@devlist: #optional list of block device names (separated by ',', ';' > > > > +# or ':'). By default the backup includes all writable block devices. > > > > +# > > > > +# Returns: the uuid of the backup job # # Since: 1.5.0 ## { > > > > +'command': 'backup', 'data': { 'backup-file': 'str', > > > > + '*format': 'BackupFormat', > > > > + '*config-file': 'str', > > > > + '*devlist': 'str', '*speed': 'int' > > > > +}, > > > > > > devlist should be ['String']. > > > > I want to be able to use that command on the human monitor. > > That is no longer possible if I use ['String']? Unless I'm missing something, it should work just fine. The HMP command can have an argument which is a device list, then it will have to parse that list and transform it into the list expected by QAPI functions, which is what is going to be passed to the qmp_ function.