Thread (17 messages) 17 messages, 10 authors, 2021-09-29

Re: [PATCH] samples/bpf: relicense bpf_insn.h as GPL-2.0-only OR BSD-2-Clause

From: Luca Boccassi <hidden>
Date: 2021-09-27 14:07:47

On Thu, 2021-09-23 at 11:41 +0100, Luca Boccassi wrote:
On Thu, 2021-09-23 at 01:05 +0100, luca.boccassi@gmail.com wrote:
quoted
From: Luca Boccassi <redacted>

libbpf and bpftool have been dual-licensed to facilitate inclusion in
software that is not compatible with GPL2-only (ie: Apache2), but the
samples are still GPL2-only.

Given these files are samples, they get naturally copied around. For
example
it is the case for samples/bpf/bpf_insn.h which was copied into the
systemd
tree:
https://github.com/systemd/systemd/blob/main/src/shared/linux/bpf_insn.h

Dual-license this header as GPL-2.0-only OR BSD-2-Clause to follow
the same licensing used by libbpf and bpftool:

1bc38b8ff6cc ("libbpf: relicense libbpf as LGPL-2.1 OR BSD-2-Clause")
907b22365115 ("tools: bpftool: dual license all files")

Signed-off-by: Luca Boccassi <redacted>
---
Most of systemd is (L)GPL2-or-later, which means there is no
perceived
incompatibility with Apache2 softwares and can thus be linked with
OpenSSL 3.0. But given this GPL2-only header is included this is
currently
not possible.
Dual-licensing this header solves this problem for us as we are
scoping
moving to OpenSSL 3.0, see:

https://lists.freedesktop.org/archives/systemd-devel/2021-September/046882.html

The authors of this file according to git log are:

Alexei Starovoitov [off-list ref]
Björn Töpel [off-list ref]
Brendan Jackman [off-list ref]
Chenbo Feng [off-list ref]
Daniel Borkmann [off-list ref]
Daniel Mack [off-list ref]
Jakub Kicinski [off-list ref]
Jiong Wang [off-list ref]
Joe Stringer [off-list ref]
Josef Bacik [off-list ref]

(excludes a commit adding the SPDX header)

All authors and maintainers are CC'ed. An Acked-by from everyone in
the
above list of authors will be necessary.

One could probably argue for relicensing all the samples/bpf/ files
given both
libbpf and bpftool are, however the authors list would be much larger
and thus
it would be much more difficult, so I'd really appreciate if this
header could
be handled first by itself, as it solves a real license
incompatibility issue
we are currently facing.

 samples/bpf/bpf_insn.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/samples/bpf/bpf_insn.h b/samples/bpf/bpf_insn.h
index aee04534483a..29c3bb6ad1cd 100644
--- a/samples/bpf/bpf_insn.h
+++ b/samples/bpf/bpf_insn.h
@@ -1,4 +1,4 @@
-/* SPDX-License-Identifier: GPL-2.0 */
+/* SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) */
 /* eBPF instruction mini library */
 #ifndef __BPF_INSN_H
 #define __BPF_INSN_H
Got "address not found" for the following:

Björn Töpel [off-list ref]
Jakub Kicinski [off-list ref]
Jiong Wang [off-list ref]

Trying again with different aliases from more recent commits for Björn
and Jakub.

I cannot find other commits from Jiong with a different email address -
Jakub, do you happen to know how we can reach Jiong? Perhaps it's not
necessary as it's Netronome that owns the copyright and thus your ack
would cover both contributions?
Gentle ping. We got ACKs from Netronome and Google so far (thanks!).

-- 
Kind regards,
Luca Boccassi

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help