From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=209.132.183.28; helo=mx1.redhat.com; envelope-from=lersek@redhat.com; receiver=edk2-devel@lists.01.org Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id F3CEC211E0931 for ; Wed, 20 Mar 2019 13:03:30 -0700 (PDT) Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id B63B180470; Wed, 20 Mar 2019 20:03:29 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-120-169.rdu2.redhat.com [10.10.120.169]) by smtp.corp.redhat.com (Postfix) with ESMTP id A2D495D70A; Wed, 20 Mar 2019 20:03:24 +0000 (UTC) To: Julien Grall , Lars Kurth , "Kinney, Michael D" Cc: "edk2-devel@lists.01.org" , Ard Biesheuvel , Jordan Justen , Anthony Perard , =?UTF-8?Q?Marc-Andr=c3=a9_Lureau?= , Stefan Berger , Stefano Stabellini References: <6de07821-a05e-9446-7ef6-c178eaf2fdfb@arm.com> <8F40F2BF-B40F-4338-A832-70AE84B26408@citrix.com> <6FBC013D-4BC9-454C-9D4D-87C96F435704@citrix.com> <720E0EE9-2AED-4110-827D-B87DE5F52862@citrix.com> <2439277a-b103-50d4-4de2-f1d3e17c53a3@redhat.com> From: Laszlo Ersek Message-ID: <4dec63ed-f3c4-4d47-3378-55d62fc932f0@redhat.com> Date: Wed, 20 Mar 2019 21:03:23 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.28]); Wed, 20 Mar 2019 20:03:29 +0000 (UTC) Subject: Re: PATCH] Change EDK II to BSD+Patent License X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 20 Mar 2019 20:03:32 -0000 Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit On 03/20/19 19:42, Julien Grall wrote: > Hi, > > On 20/03/2019 18:25, Laszlo Ersek wrote: >> On 03/20/19 14:03, Lars Kurth wrote: >>> On 15/03/2019, 17:48, "Lars Kurth" wrote: >>>      On 15/03/2019, 10:18, "Julien Grall" wrote: >>> Is the issue that you don’t trust that the license specified in the >>> files are correct? >> >> No -- the question is whether the license included in the files >> mentioned is indeed the MIT license, suitable for a replacement with the >> appropriate SPDX license ID. >> >>> >>>      > (2.2.2) Files that seem to be covered by the MIT license. >>>      > >>>      >            OvmfPkg/Include/IndustryStandard/Xen/arch-arm/xen.h >>> >>> I can't identify where in the Xen tree this file came from. There is >>> no corresponding xen.h file in the Xen tree at [xen.git] / xen / >>> include / public / arch-arm / >>> @Julien, @Anthony: can you clarify >> >> This file was first added to edk2 in b94c3ac93d57 ("Ovmf/Xen: implement >> XenHypercallLib for ARM", 2015-02-28). > > It is a copy of public/arch-arm.h. Somehow all the other projects > created the file under arch-arm/xen.h. > >> >> https://github.com/tianocore/edk2/commit/b94c3ac93d57 >> >> And from the Xen project (I think), it was Reviewed-by: Stefano >> Stabellini . (I vaguely recall that >> Stefano's emai has changed since.) > > That's correct. He is working for Xilinx now. > >> >>> >>>      >            >>> OvmfPkg/Include/IndustryStandard/Xen/arch-x86/xen-x86_32.h >>>      >            >>> OvmfPkg/Include/IndustryStandard/Xen/arch-x86/xen-x86_64.h >>>      >            OvmfPkg/Include/IndustryStandard/Xen/arch-x86/xen.h >>>      >            OvmfPkg/Include/IndustryStandard/Xen/event_channel.h >>>      >            OvmfPkg/Include/IndustryStandard/Xen/grant_table.h >>>      >            OvmfPkg/Include/IndustryStandard/Xen/hvm/hvm_op.h >>>      >            OvmfPkg/Include/IndustryStandard/Xen/hvm/params.h >>>      >            OvmfPkg/Include/IndustryStandard/Xen/io/blkif.h >>>      >            OvmfPkg/Include/IndustryStandard/Xen/io/console.h >>>      >            OvmfPkg/Include/IndustryStandard/Xen/io/protocols.h >>>      >            OvmfPkg/Include/IndustryStandard/Xen/io/ring.h >>>      >            OvmfPkg/Include/IndustryStandard/Xen/io/xenbus.h >>>      >            OvmfPkg/Include/IndustryStandard/Xen/io/xs_wire.h >>>      >            OvmfPkg/Include/IndustryStandard/Xen/memory.h >>>      >            OvmfPkg/Include/IndustryStandard/Xen/xen-compat.h >>>      >            OvmfPkg/Include/IndustryStandard/Xen/xen.h >>> >>> These all appear to originate from [xen.git] / xen / include / public >>> In the Xen tree these all have explicit MIT licenses, which implies >>> that the license headers are indeed correct. >> >> Thanks -- so can we replace the license blocks with >> >>    SPDX-License-Identifier: MIT >> >> ? (See e.g. .) > > I spoke with Lars today, this identifier would be suitable for the headers. > > Cheers, > Thanks! Laszlo