Re: [Qemu-devel] [PULL 22/62] block: Support Archipelago as a QEMU block backend

[Chrysostomos Nanakos]

On 2015-04-09 17:05, Stefan Hajnoczi wrote:
> On Thu, Apr 9, 2015 at 1:48 PM, Chrysostomos Nanakos 
> <address@hidden> wrote:
> > On 2015-04-09 06:48, Andreas Färber wrote:
> > > Am 08.08.2014 um 19:39 schrieb Kevin Wolf:
> > > > From: Chrysostomos Nanakos <address@hidden>
> > > >
> > > > 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>
> > > > Reviewed-by: Stefan Hajnoczi <address@hidden>
> > > > Signed-off-by: Kevin Wolf <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
> > >
> > >
> > > Judging by configure output in v2.3.0-rc2, QEMU seems to rely on
> > > libxseg, which is GPL-3.0+: https://github.com/grnet/libxseg
> > >
> > > How can anyone legally build this backend then? o.O
> > >
> > > Any chance libxseg can be relicensed to GPL-2.0+?
> >
> > Hello Andreas,
> > indeed the license has changed a few months after submitting the 
> > patch for
> > the block driver to QEMU. I have already contacted the 
> > administration
> > of GRNET which is responsible for this change asking them about the
> > aforementioned problem. I am waiting for the reply and hopefully we 
> > will
> > resolve this license matter.
>
> In the meantime I've sent a patch to disable Archipelago by default 
> in
> ./configure ("[PATCH for-2.3] configure: disable by default and warn
> about libxseg GPLv3 license").  You are CCed on the email.
>
> The intention is to reduce the chance of users accidentally compiling
> binaries with license problems.  Archipelago support is still
> available but it must be explicitly enabled using ./configure
> --enable-archipelago.
>
> Stefan

For the moment I totally agree with the proposed patch.

Regards,
Chrysostomos.