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

[Kevin Wolf]

Am 05.08.2014 um 10:20 hat Stefan Hajnoczi geschrieben:
> 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 <address@hidden>
> > ---
> >  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 <address@hidden>
> 
> Kevin: Please replace the v7 commit with this new patch on the block
> branch.

Already did that, will add your Reviewed-by now.

Kevin

pgp2rDSQdMOD1.pgp
Description: PGP signature