All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eric Blake <eblake@redhat.com>
To: Chrysostomos Nanakos <cnanakos@grnet.gr>, qemu-devel@nongnu.org
Cc: kwolf@redhat.com, stefanha@redhat.com
Subject: Re: [Qemu-devel] [PATCH 3/3] QMP: Add support for Archipelago
Date: Wed, 04 Jun 2014 06:56:56 -0600	[thread overview]
Message-ID: <538F1798.2000001@redhat.com> (raw)
In-Reply-To: <1401881404-15823-4-git-send-email-cnanakos@grnet.gr>

[-- Attachment #1: Type: text/plain, Size: 2453 bytes --]

On 06/04/2014 05:30 AM, Chrysostomos Nanakos wrote:
> Introduce new enum BlockdevOptionsArchipelago.
> 
> @volume:              #Name of the Archipelago volume image
> 
> @mport:               #'mport' is the port number on which mapperd is
>                       listening. This is optional and if not specified,
>                       QEMU will make Archipelago to use the default port.
> 
> @vport:               #'vport' is the port number on which vlmcd is
>                       listening. This is optional and if not specified,
>                       QEMU will make Archipelago to use the default port.
> 
> Signed-off-by: Chrysostomos Nanakos <cnanakos@grnet.gr>
> ---
>  qapi-schema.json |   31 +++++++++++++++++++++++++++++--
>  1 file changed, 29 insertions(+), 2 deletions(-)
> 

> +
> +##
> +# @BlockdevOptionsArchipelago
> +#
> +# Driver specific block device options for Archipelago.
> +#
> +# @volume:              #Name of the Archipelago volume image
> +#

s/#Name/Name/ (we only use # for the initial comment, and for the
special marking '#optional' that designates when a parameter is optional)

> +#
> +# @mport:               #'mport' is the port number on which mapperd is
> +#                       listening. This is optional
> +#                       and if not specified, QEMU will make Archipelago to
> +#                       use the default port.

@mport: #optional The port number ...

s/to use/use/


> +#
> +# @vport:               #'vport' is the port number on which vlmcd is
> +#                       listening. This is optional
> +#                       and if not specified, QEMU will make Archipelago to
> +#                       use the default port.

@vport: #optional The port number...

s/to use/use/

> +# Since: 2.0

2.1

> +##
> +{ 'type': 'BlockdevOptionsArchipelago',
> +  'data': { 'volume': 'str',
> +            '*mport': 'int',
> +            '*vport': 'int' } }
> +
>  ##
>  # @BlkdebugEvent
>  #
> @@ -4616,6 +4642,7 @@
>        'vhdx':       'BlockdevOptionsGenericFormat',
>        'vmdk':       'BlockdevOptionsGenericCOWFormat',
>        'vpc':        'BlockdevOptionsGenericFormat',
> +      'archipelago': 'BlockdevOptionsArchipelago',
>        'quorum':     'BlockdevOptionsQuorum'
>    } }
>  
> 

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 604 bytes --]

  parent reply	other threads:[~2014-06-04 12:57 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-04 11:30 [Qemu-devel] [PATCH v3 0/3] Support Archipelago as a QEMU block backend Chrysostomos Nanakos
2014-06-04 11:30 ` [Qemu-devel] [PATCH 1/3] block: " Chrysostomos Nanakos
2014-06-04 11:30 ` [Qemu-devel] [PATCH 2/3] block/archipelago: Add support for creating images Chrysostomos Nanakos
2014-06-04 11:30 ` [Qemu-devel] [PATCH 3/3] QMP: Add support for Archipelago Chrysostomos Nanakos
2014-06-04 11:38   ` Kevin Wolf
2014-06-04 11:46     ` Chrysostomos Nanakos
2014-06-04 12:56   ` Eric Blake [this message]
2014-06-04 13:02     ` Chrysostomos Nanakos

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=538F1798.2000001@redhat.com \
    --to=eblake@redhat.com \
    --cc=cnanakos@grnet.gr \
    --cc=kwolf@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.