All of lore.kernel.org
 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 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.