xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Lars Kurth <lars.kurth@citrix.com>
To: Dario Faggioli <dario.faggioli@citrix.com>,
	Andrew Cooper <Andrew.Cooper3@citrix.com>
Cc: Ian Campbell <Ian.Campbell@citrix.com>,
	Brian Warner <brian.warner@linuxfoundation.org>,
	"julien.grall@linaro.org" <julien.grall@linaro.org>,
	"Tim (Xen.org)" <tim@xen.org>,
	"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>,
	Oleksandr Tyshchenko <oleksandr.tyshchenko@globallogic.com>,
	Stefano Stabellini <Stefano.Stabellini@citrix.com>
Subject: Re: [PATCH] xen/tools: Introduce QNX IFS loader
Date: Wed, 20 Aug 2014 04:09:00 +0000	[thread overview]
Message-ID: <D019834F.F9EC%lars.kurth@citrix.com> (raw)
In-Reply-To: <1408465606.3725.53.camel@Solace.lan>



On 19/08/2014 11:26, "Dario Faggioli" <dario.faggioli@citrix.com> wrote:

>[Adding Lars, as he may be of help with licensing things]
>
>On mar, 2014-08-19 at 17:13 +0100, Andrew Cooper wrote:
>> On 19/08/14 16:51, Oleksandr Tyshchenko wrote:
>> > Add ability to load QNX IFS image. Based on IPL code (U-Boot for QNX).
>> >
>> > Signed-off-by: Oleksandr Tyshchenko
>><oleksandr.tyshchenko@globallogic.com>
>> > ---
>> >  tools/libxc/Makefile              |   1 +
>> >  tools/libxc/xc_dom_qnxifsloader.c | 189
>>++++++++++++++++++++++++++++++++++++++
>> >  2 files changed, 190 insertions(+)
>> >  create mode 100644 tools/libxc/xc_dom_qnxifsloader.c
>> >
>> > diff --git a/tools/libxc/Makefile b/tools/libxc/Makefile
>> > index 22eef8e..812cc7e 100644
>> > --- a/tools/libxc/Makefile
>> > +++ b/tools/libxc/Makefile
>> > @@ -67,6 +67,7 @@ GUEST_SRCS-y                 += xc_dom_elfloader.c
>> >  GUEST_SRCS-$(CONFIG_X86)     += xc_dom_bzimageloader.c
>> >  GUEST_SRCS-$(CONFIG_X86)     += xc_dom_decompress_lz4.c
>> >  GUEST_SRCS-$(CONFIG_ARM)     += xc_dom_armzimageloader.c
>> > +GUEST_SRCS-$(CONFIG_ARM)     += xc_dom_qnxifsloader.c
>> >  GUEST_SRCS-y                 += xc_dom_binloader.c
>> >  GUEST_SRCS-y                 += xc_dom_compat_linux.c
>> >  
>> > diff --git a/tools/libxc/xc_dom_qnxifsloader.c
>>b/tools/libxc/xc_dom_qnxifsloader.c
>> > new file mode 100644
>> > index 0000000..45d007d
>> > --- /dev/null
>> > +++ b/tools/libxc/xc_dom_qnxifsloader.c
>> > @@ -0,0 +1,189 @@
>> > +/*
>> > + * Xen domain builder -- QNX IFS bits
>> > + *
>> > + * Parse and load QNX IFS image.
>> > + *
>> > + * Copyright (C) 2014, Globallogic.
>> > + *
>> > + * Licensed under the Apache License, Version 2.0 (the "License").
>>You
>> > + * may not reproduce, modify or distribute this software except in
>> > + * compliance with the License. You may obtain a copy of the License
>> > + * at: http://www.apache.org/licenses/LICENSE-2.0
>> > + *
>> > + * Unless required by applicable law or agreed to in writing,
>>software
>> > + * distributed under the License is distributed on an "AS IS" basis,
>> > + * WITHOUT WARRANTIES OF ANY KIND, either express or implied.
>> > + *
>> > + * This file may contain contributions from others, either as
>> > + * contributors under the License or as licensors under other terms.
>> > + * Please review this entire file for other proprietary rights or
>>license
>> > + * notices, as well as the QNX Development Suite License Guide at
>> > + * http://licensing.qnx.com/license-guide/ for other information.
>> > + */
>> 
>> The rest of libxc is licensed under GPLv2.1, which is incompatible with
>> Apache 2.0

That is correct. 

The question is whether this code needs to be Apache 2.0. My understanding
was that the QNX recommendation is that a base port of QNX (aka the
drivers) are recommended to be Apache 2. But it is not clear to me whether
the boot loader needs to be. I don¹t have enough context and can¹t find
any reference to the file system itself in the license guide.

I do not know enough about the subtleties of licenses here and am adding
Brian, in case we need to get some more advice on how to mitigate this
issue.

@Oleksandr, can you also check with your legal guys?

Regards
Lars

  reply	other threads:[~2014-08-20  4:09 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-19 15:51 [PATCH] xen/tools: Introduce QNX IFS loader Oleksandr Tyshchenko
2014-08-19 16:13 ` Andrew Cooper
2014-08-19 16:26   ` Dario Faggioli
2014-08-20  4:09     ` Lars Kurth [this message]
2014-08-20 10:07       ` Artem Mygaiev
2014-08-20 10:11         ` Dario Faggioli
2014-08-20 13:56           ` Artem Mygaiev
2014-08-20 14:00             ` Dario Faggioli
2014-08-26 21:09             ` Ian Campbell
2014-08-27  7:53               ` Andrii Tseglytskyi
2014-08-20 10:20       ` Oleksandr Tyshchenko
2014-08-19 16:41 ` Julien Grall
2014-08-20 11:10   ` Oleksandr Tyshchenko
2014-08-20 13:00   ` Oleksandr Tyshchenko
2014-08-20 14:55     ` Julien Grall
2014-08-20 16:35       ` Oleksandr Tyshchenko

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=D019834F.F9EC%lars.kurth@citrix.com \
    --to=lars.kurth@citrix.com \
    --cc=Andrew.Cooper3@citrix.com \
    --cc=Ian.Campbell@citrix.com \
    --cc=Stefano.Stabellini@citrix.com \
    --cc=brian.warner@linuxfoundation.org \
    --cc=dario.faggioli@citrix.com \
    --cc=julien.grall@linaro.org \
    --cc=oleksandr.tyshchenko@globallogic.com \
    --cc=tim@xen.org \
    --cc=xen-devel@lists.xen.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).