On 2017-07-20 16:24, Claudio Scordino wrote:
> Dear developer,
> 
> your name is listed among the authors of some file within the inmates/lib/
> directory of Jailhouse, and therefore you are among the copyright holders of a
> part of such library.
> 
> The library is currently licensed under GPL, meaning that it cannot be linked
> to proprietary code unless by releasing such code under GPL too. This prevents
> the usage of the hypervisor in all industrial contexts where the inmate code
> cannot be disclosed.
> 
> For this reason, we want to re-license the inmate library to a dual GPL|BSD-2
> license, as already done for the Jailhouse headers [1]. The license of the
> other components of Jailhouse will remain untouched.
> 
> IMPORTANT: In order to perform these license changes, we need the agreement of
> all contributors to the affected files. The proposed change is pasted below.
> I KINDLY ASK YOU TO REPLY TO THE PATCH WITH YOUR SIGNED-OFF-BY LINE IF YOU
> AGREE TO THIS CHANGE. The patch won't be committed until everyone involved did
> this. Please also speak up if you think you or someone else has been missed in
> this process.
> 
> Don't hesitate to contact me in case you need any further clarification.
> 
> Many thanks for your time and your contribution.
> 
> Best regards.
> 
>               Claudio Scordino
> 
> [1] https://groups.google.com/forum/#!topic/jailhouse-dev/eskAY0BGhWc
> 
> 
> From 662c63e33e3581532b90bbceb34046b29725abe9 Mon Sep 17 00:00:00 2001
> From: Claudio Scordino <[email protected]>
> Date: Wed, 14 Jun 2017 10:36:34 +0200
> Subject: [PATCH] Provide the inmate library under BSD-2 license too.
> 
> This patch does not introduce any functional change, but "only" deals
> with copyright and license aspects.
> 
> The rationale is simply that, as was already stated in COPYING, cell
> inmates, AKA guests, are not derived work of Jailhouse, even if written
> exclusively to run in its special environment. The inmate library
> provides a set of functionalities to let inmates run on Jailhouse.
> In order to avoid that people need to rewrite such library, possibly
> introducing bugs or incompatibilities this way, we should rather enable
> the reuse of the "official" one. Thus, it should be provided also under a
> permissive license. BSD 2-clause is a well-established and easy-to-deal
> with license, already used for the Jailhouse headers.
> 
> All contributors to the affected code have signed-off to confirm that
> they agree with this relicensing.
> 
> Signed-off-by: Claudio Scordino <[email protected]>
> ---
>  inmates/lib/arm-common/Makefile.lib     | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm-common/gic-v2.c         | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm-common/gic.c            | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm-common/include/gic.h    | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm-common/include/inmate.h | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm-common/include/uart.h   | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm-common/printk.c         | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm-common/timer.c          | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm-common/uart-8250.c      | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm-common/uart-jailhouse.c | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm-common/uart-pl011.c     | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm-common/uart-xuartps.c   | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm/Makefile                | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm/Makefile.lib            | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm/gic-v3.c                | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm/header.S                | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm/include/arch/gic.h      | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm/include/arch/inmate.h   | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm/include/mach.h          | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm/inmate.lds.S            | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm64/Makefile              | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm64/Makefile.lib          | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm64/header.S              | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm64/include/arch/gic.h    | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm64/include/arch/inmate.h | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm64/include/mach.h        | 26 ++++++++++++++++++++++++++
>  inmates/lib/arm64/inmate.lds.S          | 26 ++++++++++++++++++++++++++
>  inmates/lib/cmdline.c                   | 26 ++++++++++++++++++++++++++
>  inmates/lib/inmate_common.h             | 26 ++++++++++++++++++++++++++
>  inmates/lib/pci.c                       | 26 ++++++++++++++++++++++++++
>  inmates/lib/string.c                    | 26 ++++++++++++++++++++++++++
>  inmates/lib/x86/Makefile                | 26 ++++++++++++++++++++++++++
>  inmates/lib/x86/Makefile.lib            | 26 ++++++++++++++++++++++++++
>  inmates/lib/x86/header-32.S             | 26 ++++++++++++++++++++++++++
>  inmates/lib/x86/header.S                | 26 ++++++++++++++++++++++++++
>  inmates/lib/x86/hypercall.c             | 26 ++++++++++++++++++++++++++
>  inmates/lib/x86/inmate.h                | 26 ++++++++++++++++++++++++++
>  inmates/lib/x86/inmate.lds              | 26 ++++++++++++++++++++++++++
>  inmates/lib/x86/int.c                   | 26 ++++++++++++++++++++++++++
>  inmates/lib/x86/ioapic.c                | 26 ++++++++++++++++++++++++++
>  inmates/lib/x86/mem.c                   | 26 ++++++++++++++++++++++++++
>  inmates/lib/x86/pci.c                   | 26 ++++++++++++++++++++++++++
>  inmates/lib/x86/printk.c                | 26 ++++++++++++++++++++++++++
>  inmates/lib/x86/smp.c                   | 26 ++++++++++++++++++++++++++
>  inmates/lib/x86/timing.c                | 26 ++++++++++++++++++++++++++
>  45 files changed, 1170 insertions(+)
> 

Thanks to Ralf's digging on other ends, I just realized that this is not
yet enough to build the inmates lib completely under dual license:

- hypervisor/arch/arm[64[/include/asm/sysregs.h
- hypervisor/printk-core.c

The sysregs.h for arm (32 bit) has a lot of copyright holders, so
relicensing it would require a second, larger round. But I'm not a fan
of this reuse anyway because it's not very obvious, and would welcome
any patch that rewrites the required parts for inmates under dual
GPL/BSD, both for arm and arm64.

The printk-core has only be written by me and is, thus, Siemens owned. I
can issue a relicensing, but maybe this is also a good chance to look
for something more sophisticated for the inmates (check other RTOSes or
libraries with the right license) and divorce the users.

There is another finding along this:

- hypervisor/arch/arm64/include/asm/jailhouse_hypercall.h is GPL-only

But that's likely just a mistake because the header that pulls this,
jailhouse/hypercall.h, was always dual-licensed. Tony, could you send a
correcting patch for that header, or should I prepare it?

Thanks,
Jan

-- 
Siemens AG, Corporate Technology, CT RDA ITP SES-DE
Corporate Competence Center Embedded Linux

-- 
You received this message because you are subscribed to the Google Groups 
"Jailhouse" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to