From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: redhat.com, ip: 209.132.183.28, mailfrom: lersek@redhat.com) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by groups.io with SMTP; Thu, 11 Jul 2019 03:24:50 -0700 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 9B9D1307CDD1; Thu, 11 Jul 2019 10:24:49 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-117-231.ams2.redhat.com [10.36.117.231]) by smtp.corp.redhat.com (Postfix) with ESMTP id 0A8075D71A; Thu, 11 Jul 2019 10:24:46 +0000 (UTC) Subject: Re: [PATCH V2 2/4] UefiCpuPkg/CpuDxe: Implement Cpu2 protocol To: "Gao, Zhichao" , "Dong, Eric" , "devel@edk2.groups.io" Cc: Sean Brogan , "Ni, Ray" , "Gao, Liming" , Michael Turner , Bret Barkelew , Michael Kinney , Leif Lindholm , Andrew Fish References: <20190709083956.13024-1-zhichao.gao@intel.com> <20190709083956.13024-3-zhichao.gao@intel.com> <3CE959C139B4C44DBEA1810E3AA6F9000B807187@SHSMSX101.ccr.corp.intel.com> From: "Laszlo Ersek" Message-ID: <95279da1-9212-4f34-6600-474fa1703517@redhat.com> Date: Thu, 11 Jul 2019 12:24:46 +0200 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: <3CE959C139B4C44DBEA1810E3AA6F9000B807187@SHSMSX101.ccr.corp.intel.com> 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.49]); Thu, 11 Jul 2019 10:24:49 +0000 (UTC) Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit On 07/11/19 04:36, Gao, Zhichao wrote: > > >> -----Original Message----- >> From: Dong, Eric >> Sent: Thursday, July 11, 2019 10:22 AM >> To: Gao, Zhichao ; devel@edk2.groups.io >> Cc: Sean Brogan ; Ni, Ray ; >> Laszlo Ersek ; Gao, Liming ; >> Michael Turner ; Bret Barkelew >> >> Subject: RE: [PATCH V2 2/4] UefiCpuPkg/CpuDxe: Implement Cpu2 protocol >> >> Hi Zhizhao, >> >> The new add files don't have copyright info, is it ok? Good catch! > I forgot add file comment for Ia32/EnableInterruptsAndSleep.c. Also lose the BSD+Patent License. I would add them in the future. > For the copyright: > The code is contributed by Microsoft at https://github.com/Microsoft/mu_basecore/blob/release/201808/MdePkg/Library/BaseLib/X64/EnableInterrupts.nasm#L48 > I move the code from MdePkg to UefiCpuPkg to a single file. It is inappropriate to add both intel (not contributor) and Microsoft (they didn't add in the above link). > So I keep the copyright blank. I don't know whether it is ok without copyright. Every single source file must have both copyright notice(s) and licensing information. - Licensing information without copyright holder(s) makes zero sense -- the licensing terms are offered / dictated *by* the copyright holders. - A copyright notice in itself, without a suitable license (including the "no license" case) does not provide us with the necessary rights to carry the source file in the edk2 project. Regarding "EnableInterrupts.nasm". This looks like a tricky situation. Under the above link, the file is currently missing a Microsoft (C) notice (the last commit to modify the file is apparently from Microsoft, b621971). So that should be addressed in Project Mu first, in my opinion. Then, Microsoft should please relicense the file under BSD+Patent, from pure 2-BSD. If they disagree, then the 2-BSD is still acceptable for edk2. Then, the file could be imported into edk2, carrying - the MS (C) notice, - the Intel (C) notice (extended to 2019), - the license -- BSD+Patent, or else 2-BSD --, expressed with an SPDX ID (not open-coded license text). Note: I'm not a lawyer; the above is just my opinion. CC'ing the other stewards. Thanks, Laszlo > > Hi Liming, > > What's your opinion? > > Thanks, > Zhichao > >> >> Thanks, >> Eric >> >>> -----Original Message----- >>> From: Gao, Zhichao >>> Sent: Tuesday, July 9, 2019 4:40 PM >>> To: devel@edk2.groups.io >>> Cc: Sean Brogan ; Dong, Eric >>> ; Ni, Ray ; Laszlo Ersek >>> ; Gao, Liming ; Michael >>> Turner ; Bret Barkelew >>> >>> Subject: [PATCH V2 2/4] UefiCpuPkg/CpuDxe: Implement Cpu2 protocol >>> >>> From: Sean Brogan >>> >>> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1400 >>> >>> Implement Cp2 protocol: it has one interface to enable the interrupt >>> and put cpu to sleep and wait for an interrupt. >>> >>> Cc: Eric Dong >>> Cc: Ray Ni >>> Cc: Laszlo Ersek >>> Cc: Liming Gao >>> Cc: Sean Brogan >>> Cc: Michael Turner >>> Cc: Bret Barkelew >>> Signed-off-by: Zhichao Gao >>> --- >>> UefiCpuPkg/CpuDxe/CpuDxe.c | 38 +++++++++++++++++++ >>> UefiCpuPkg/CpuDxe/CpuDxe.h | 25 ++++++++++++ >>> UefiCpuPkg/CpuDxe/CpuDxe.inf | 3 ++ >>> .../CpuDxe/Ia32/EnableInterruptsAndSleep.c | 24 ++++++++++++ >>> .../CpuDxe/X64/EnableInterruptsAndSleep.nasm | 31 +++++++++++++++ >>> 5 files changed, 121 insertions(+) >>> create mode 100644 >> UefiCpuPkg/CpuDxe/Ia32/EnableInterruptsAndSleep.c >>> create mode 100644 >>> UefiCpuPkg/CpuDxe/X64/EnableInterruptsAndSleep.nasm >>>