Re: [Qemu-devel] [PATCH v8 1/5] block: Support Archipelago as a QEMU block backend

[Stefan Hajnoczi <stefanha@redhat.com>]

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

On Mon, Aug 04, 2014 at 05:35:32PM +0300, Chrysostomos Nanakos wrote:
> VM Image on Archipelago volume is specified like this:
> 
> file.driver=archipelago,file.volume=<volumename>[,file.mport=<mapperd_port>[,
> file.vport=<vlmcd_port>][,file.segment=<segment_name>]]
> 
> 'archipelago' is the protocol.
> 
> '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' 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.
> 
> 'segment' is the name of the shared memory segment Archipelago stack is using.
> This is optional and if not specified, QEMU will make Archipelago to use the
> default value, 'archipelago'.
> 
> Examples:
> 
> file.driver=archipelago,file.volume=my_vm_volume
> file.driver=archipelago,file.volume=my_vm_volume,file.mport=123
> file.driver=archipelago,file.volume=my_vm_volume,file.mport=123,
> file.vport=1234
> file.driver=archipelago,file.volume=my_vm_volume,file.mport=123,
> file.vport=1234,file.segment=my_segment
> 
> Signed-off-by: Chrysostomos Nanakos <cnanakos@grnet.gr>
> ---
>  MAINTAINERS         |    6 +
>  block/Makefile.objs |    2 +
>  block/archipelago.c |  787 +++++++++++++++++++++++++++++++++++++++++++++++++++
>  configure           |   40 +++
>  4 files changed, 835 insertions(+)
>  create mode 100644 block/archipelago.c

The diff is that a flush op is used now:
+    case ARCHIP_OP_FLUSH:
+        req->op = X_FLUSH;
+        break;

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>

Kevin: Please replace the v7 commit with this new patch on the block
branch.

[-- Attachment #2: Type: application/pgp-signature, Size: 473 bytes --]