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
next prev parent 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).