回复: [edk2-devel] [edk2-platforms][PATCH 1/1] RaspberryPi: get RPi4 and RPi3 building again.
gaoliming
gaoliming at byosoft.com.cn
Fri Nov 20 07:20:24 UTC 2020
Andrei:
If you know the broken platforms, you can also contribute the patch to fix
them.
For this patch, it has passed the code review. The comment is to update the
commit message with below.
Fixes: tianocore/edk2 at b6490426e320 ("MdeModulePkg: Connect
VariablePolicy business logic to VariableServices")
Thanks
Liming
发件人: Andrei Warkentin <awarkentin at vmware.com>
发送时间: 2020年11月20日 3:53
收件人: Philippe Mathieu-Daudé <philmd at redhat.com>; Andrei Warkentin
<andrey.warkentin at gmail.com>; devel at edk2.groups.io; Bret Barkelew
<brbarkel at microsoft.com>; Dandan Bi <dandan.bi at intel.com>; Jian J Wang
<jian.j.wang at intel.com>; pete at akeo.ie
抄送: ard.biesheuvel at arm.com; leif at nuviainc.com; Liming Gao
<gaoliming at byosoft.com.cn>
主题: Re: [edk2-devel] [edk2-platforms][PATCH 1/1] RaspberryPi: get RPi4 and
RPi3 building again.
Sorry, folks, is there anything actionable for me here to do?
It does appear that all the other edk2-platforms are broken/affected as
well.
A
_____
From: devel at edk2.groups.io <mailto:devel at edk2.groups.io>
<devel at edk2.groups.io <mailto:devel at edk2.groups.io> > on behalf of Pete
Batard via groups.io <pete=akeo.ie at groups.io <mailto:pete=akeo.ie at groups.io>
>
Sent: Thursday, November 19, 2020 11:33 AM
To: Philippe Mathieu-Daudé <philmd at redhat.com <mailto:philmd at redhat.com> >;
Andrei Warkentin <andrey.warkentin at gmail.com <mailto:andrey.warkentin at gmail.
com> >; devel at edk2.groups.io <mailto:devel at edk2.groups.io>
<devel at edk2.groups.io <mailto:devel at edk2.groups.io> >; Bret Barkelew
<brbarkel at microsoft.com <mailto:brbarkel at microsoft.com> >; Dandan Bi
<dandan.bi at intel.com <mailto:dandan.bi at intel.com> >; Jian J Wang
<jian.j.wang at intel.com <mailto:jian.j.wang at intel.com> >
Cc: ard.biesheuvel at arm.com <mailto:ard.biesheuvel at arm.com>
<ard.biesheuvel at arm.com <mailto:ard.biesheuvel at arm.com> >; leif at nuviainc.com
<mailto:leif at nuviainc.com> <leif at nuviainc.com <mailto:leif at nuviainc.com> >;
Liming Gao <gaoliming at byosoft.com.cn <mailto:gaoliming at byosoft.com.cn> >
Subject: Re: [edk2-devel] [edk2-platforms][PATCH 1/1] RaspberryPi: get RPi4
and RPi3 building again.
Hi Philippe,
On 2020.11.19 17:28, Philippe Mathieu-Daudé wrote:
> On 11/19/20 6:14 PM, Pete Batard wrote:
>> For reference, this breakage was introduced with edk2 commit
>> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.
com%2Ftianocore%2Fedk2%2Fcommit%2Fb6490426e3208de7d6cbd756276a02e4329cd85a
<https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.co
m%2Ftianocore%2Fedk2%2Fcommit%2Fb6490426e3208de7d6cbd756276a02e4329cd85a&
;data=04%7C01%7Cawarkentin%40vmware.com%7C912c541f65924d5c6a6108d88cb139b8%7
Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C637414040092341605%7CUnknown%7CT
WFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3
D%7C1000&sdata=K%2Fe45o3ZvD29UaHNFzCKLBMil8lJ1adSB2mRiC0uVy0%3D&rese
rved=0>
&data=04%7C01%7Cawarkentin%40vmware.com%7C912c541f65924d5c6a6108d88cb139
b8%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C637414040092341605%7CUnknown
%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6M
n0%3D%7C1000&sdata=K%2Fe45o3ZvD29UaHNFzCKLBMil8lJ1adSB2mRiC0uVy0%3D&
reserved=0
>> (MdeModulePkg: Connect VariablePolicy business logic to
VariableServices).
>
> It is better if Andrei Cc the developers involved (Cc'ed them now).
Good point.
>
>>
>> It'll probably be worth adding a reference to it in the commit message
>> during integration.
>
> This can be added before the S-o-b:
>
> Fixes: b6490426e320 ("MdeModulePkg: Connect VariablePolicy business
> logic to VariableServices")
Just going to nitpick that since we're patching in edk2-platforms and
referencing an edk2 commit, we'll want:
Fixes: tianocore/edk2 at b6490426e320 ("MdeModulePkg: Connect
VariablePolicy business logic to VariableServices")
For GitHub to produce a clickable link.
Regards,
/Pete
>
>>
>> With this:
>>
>> On 2020.11.19 00:01, Andrei Warkentin wrote:
>>> Add VariablePolicyLib and its dependency.
>>>
>>> Testing: Pi 4 boot.
>>>
>>> Signed-off-by: Andrei Warkentin <andrey.warkentin at gmail.com
<mailto:andrey.warkentin at gmail.com> >
>>> ---
>>> Platform/RaspberryPi/RPi3/RPi3.dsc | 3 +++
>>> Platform/RaspberryPi/RPi4/RPi4.dsc | 3 +++
>>> 2 files changed, 6 insertions(+)
>>>
>>> diff --git a/Platform/RaspberryPi/RPi3/RPi3.dsc
>>> b/Platform/RaspberryPi/RPi3/RPi3.dsc
>>> index 325d7bdb..9408138d 100644
>>> --- a/Platform/RaspberryPi/RPi3/RPi3.dsc
>>> +++ b/Platform/RaspberryPi/RPi3/RPi3.dsc
>>> @@ -169,6 +169,8 @@
>>>
>>>
AuthVariableLib|MdeModulePkg/Library/AuthVariableLibNull/AuthVariableLibNull
.inf
>>>
>>> !endif
>>> VarCheckLib|MdeModulePkg/Library/VarCheckLib/VarCheckLib.inf
>>> +
>>>
VariablePolicyLib|MdeModulePkg/Library/VariablePolicyLib/VariablePolicyLib.i
nf
>>>
>>> +
>>>
VariablePolicyHelperLib|MdeModulePkg/Library/VariablePolicyHelperLib/Variabl
ePolicyHelperLib.inf
>>>
>>> GpioLib|Silicon/Broadcom/Bcm283x/Library/GpioLib/GpioLib.inf
>>> [LibraryClasses.common.SEC]
>>> @@ -218,6 +220,7 @@
>>>
>>> CapsuleLib|MdeModulePkg/Library/DxeCapsuleLibNull/DxeCapsuleLibNull.inf
>>>
EfiResetSystemLib|Platform/RaspberryPi/Library/ResetLib/ResetLib.inf
>>> ArmSmcLib|ArmPkg/Library/ArmSmcLib/ArmSmcLib.inf
>>> +
>>>
VariablePolicyLib|MdeModulePkg/Library/VariablePolicyLib/VariablePolicyLibRu
ntimeDxe.inf
>>>
>>> !if $(SECURE_BOOT_ENABLE) == TRUE
>>> BaseCryptLib|CryptoPkg/Library/BaseCryptLib/RuntimeCryptLib.inf
>>> diff --git a/Platform/RaspberryPi/RPi4/RPi4.dsc
>>> b/Platform/RaspberryPi/RPi4/RPi4.dsc
>>> index c994f56d..4e5a36ed 100644
>>> --- a/Platform/RaspberryPi/RPi4/RPi4.dsc
>>> +++ b/Platform/RaspberryPi/RPi4/RPi4.dsc
>>> @@ -169,6 +169,8 @@
>>>
>>>
AuthVariableLib|MdeModulePkg/Library/AuthVariableLibNull/AuthVariableLibNull
.inf
>>>
>>> !endif
>>> VarCheckLib|MdeModulePkg/Library/VarCheckLib/VarCheckLib.inf
>>> +
>>>
VariablePolicyLib|MdeModulePkg/Library/VariablePolicyLib/VariablePolicyLib.i
nf
>>>
>>> +
>>>
VariablePolicyHelperLib|MdeModulePkg/Library/VariablePolicyHelperLib/Variabl
ePolicyHelperLib.inf
>>>
>>> GpioLib|Silicon/Broadcom/Bcm283x/Library/GpioLib/GpioLib.inf
>>> #
>>> @@ -226,6 +228,7 @@
>>>
>>> CapsuleLib|MdeModulePkg/Library/DxeCapsuleLibNull/DxeCapsuleLibNull.inf
>>>
EfiResetSystemLib|Platform/RaspberryPi/Library/ResetLib/ResetLib.inf
>>> ArmSmcLib|ArmPkg/Library/ArmSmcLib/ArmSmcLib.inf
>>> +
>>>
VariablePolicyLib|MdeModulePkg/Library/VariablePolicyLib/VariablePolicyLibRu
ntimeDxe.inf
>>>
>>> !if $(SECURE_BOOT_ENABLE) == TRUE
>>> BaseCryptLib|CryptoPkg/Library/BaseCryptLib/RuntimeCryptLib.inf
>>>
>>
>> Reviewed-by: Pete Batard <pete at akeo.ie <mailto:pete at akeo.ie> >
>>
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#67753): https://edk2.groups.io/g/devel/message/67753
Mute This Topic: https://groups.io/mt/78384493/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20201120/eaf5b051/attachment.htm>
More information about the edk2-devel-archive
mailing list