On Thu, 15 Aug 2019 22:45:43 -0700 Andrii Nakryiko <andr...@fb.com> wrote:
> bpf_helpers.h and bpf_endian.h contain useful macros and BPF helper > definitions essential to almost every BPF program. Which makes them > useful not just for selftests. To be able to expose them as part of > libbpf, though, we need them to be dual-licensed as LGPL-2.1 OR > BSD-2-Clause. This patch updates licensing of those two files. I've already ACKed this, and is fine with (LGPL-2.1 OR BSD-2-Clause). I just want to understand, why "BSD-2-Clause" and not "Apache-2.0" ? The original argument was that this needed to be compatible with "Apache-2.0", then why not simply add this in the "OR" ? > Acked-by: Alexei Starovoitov <a...@kernel.org> > Acked-by: Hechao Li <hech...@fb.com> > Acked-by: Martin KaFai Lau <ka...@fb.com> > Acked-by: Andrey Ignatov <r...@fb.com> > Acked-by: Yonghong Song <y...@fb.com> > Acked-by: Lawrence Brakmo <bra...@fb.com> > Acked-by: Adam Barth <a...@fb.com> > Acked-by: Roman Gushchin <g...@fb.com> > Acked-by: Josef Bacik <jba...@fb.com> > Acked-by: Joe Stringer <j...@wand.net.nz> > Acked-by: Daniel Borkmann <dan...@iogearbox.net> > Acked-by: Joel Fernandes (Google) <j...@joelfernandes.org> > Acked-by: David Ahern <dsah...@gmail.com> > Acked-by: Jesper Dangaard Brouer <bro...@redhat.com> Confirming I acked this. > Acked-by: Ilya Leoshkevich <i...@linux.ibm.com> > Acked-by: Lorenz Bauer <l...@cloudflare.com> > Acked-by: Adrian Ratiu <adrian.ra...@collabora.com> > Acked-by: Nikita V. Shirokov <tehn...@tehnerd.com> > Acked-by: Willem de Bruijn <will...@google.com> > Acked-by: Petar Penkov <ppen...@google.com> > Acked-by: Teng Qin <palmte...@gmail.com> > Cc: Michael Holzheu <holz...@linux.vnet.ibm.com> > Cc: Naveen N. Rao <naveen.n....@linux.vnet.ibm.com> > Cc: David S. Miller <da...@davemloft.net> > Cc: Michal Rostecki <mroste...@opensuse.org> > Cc: John Fastabend <john.fastab...@gmail.com> > Cc: Sargun Dhillon <sar...@sargun.me> > Signed-off-by: Andrii Nakryiko <andr...@fb.com> > --- > tools/testing/selftests/bpf/bpf_endian.h | 2 +- > tools/testing/selftests/bpf/bpf_helpers.h | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/tools/testing/selftests/bpf/bpf_endian.h > b/tools/testing/selftests/bpf/bpf_endian.h > index 05f036df8a4c..ff3593b0ae03 100644 > --- a/tools/testing/selftests/bpf/bpf_endian.h > +++ b/tools/testing/selftests/bpf/bpf_endian.h > @@ -1,4 +1,4 @@ > -/* SPDX-License-Identifier: GPL-2.0 */ > +/* SPDX-License-Identifier: (LGPL-2.1 OR BSD-2-Clause) */ > #ifndef __BPF_ENDIAN__ > #define __BPF_ENDIAN__ > > diff --git a/tools/testing/selftests/bpf/bpf_helpers.h > b/tools/testing/selftests/bpf/bpf_helpers.h > index 8b503ea142f0..6c4930bc6e2e 100644 > --- a/tools/testing/selftests/bpf/bpf_helpers.h > +++ b/tools/testing/selftests/bpf/bpf_helpers.h > @@ -1,4 +1,4 @@ > -/* SPDX-License-Identifier: GPL-2.0 */ > +/* SPDX-License-Identifier: (LGPL-2.1 OR BSD-2-Clause) */ > #ifndef __BPF_HELPERS_H > #define __BPF_HELPERS_H > -- Best regards, Jesper Dangaard Brouer MSc.CS, Principal Kernel Engineer at Red Hat LinkedIn: http://www.linkedin.com/in/brouer
