From: "Liming Gao" <liming.gao@intel.com>
To: Ard Biesheuvel <ard.biesheuvel@linaro.org>,
Laszlo Ersek <lersek@redhat.com>,
"Kinney, Michael D" <michael.d.kinney@intel.com>
Cc: edk2-devel-groups-io <devel@edk2.groups.io>,
Leif Lindholm <leif.lindholm@linaro.org>
Subject: Re: [edk2-devel] [Patch v3 1/3] NetworkPkg DSC: Add the required ARM library to pass ARM build
Date: Mon, 29 Apr 2019 02:02:33 +0000 [thread overview]
Message-ID: <4A89E2EF3DFEDB4C8BFDE51014F606A14E43AF35@SHSMSX104.ccr.corp.intel.com> (raw)
In-Reply-To: <CAKv+Gu-h7vUMfSfOebs5MTyytddAKEqcCmFW5FaktU3snjjg8w@mail.gmail.com>
>-----Original Message-----
>From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]
>Sent: Saturday, April 27, 2019 6:19 AM
>To: Laszlo Ersek <lersek@redhat.com>; Kinney, Michael D
><michael.d.kinney@intel.com>
>Cc: edk2-devel-groups-io <devel@edk2.groups.io>; Gao, Liming
><liming.gao@intel.com>; Leif Lindholm <leif.lindholm@linaro.org>
>Subject: Re: [edk2-devel] [Patch v3 1/3] NetworkPkg DSC: Add the required
>ARM library to pass ARM build
>
>On Sat, 27 Apr 2019 at 00:04, Laszlo Ersek <lersek@redhat.com> wrote:
>>
>> On 04/25/19 14:37, Liming Gao wrote:
>> > Signed-off-by: Liming Gao <liming.gao@intel.com>
>> > ---
>> > NetworkPkg/NetworkPkg.dsc | 4 +++-
>> > 1 file changed, 3 insertions(+), 1 deletion(-)
>> >
>> > diff --git a/NetworkPkg/NetworkPkg.dsc b/NetworkPkg/NetworkPkg.dsc
>> > index 66d43bec12..955e45e84d 100644
>> > --- a/NetworkPkg/NetworkPkg.dsc
>> > +++ b/NetworkPkg/NetworkPkg.dsc
>> > @@ -55,6 +55,7 @@
>> > FileHandleLib|MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.inf
>> >
>FileExplorerLib|MdeModulePkg/Library/FileExplorerLib/FileExplorerLib.inf
>> > SortLib|MdeModulePkg/Library/UefiSortLib/UefiSortLib.inf
>> > + NULL|MdePkg/Library/BaseStackCheckLib/BaseStackCheckLib.inf
>>
>> "MdePkg/Library/BaseStackCheckLib/BaseStackCheckLib.inf" seems to be
>> ARM/AARCH64 only. Packages that are not inherently specific to
>> ARM/AARCH64 include this library resolution only in the following sections:
>>
>> [LibraryClasses.ARM, LibraryClasses.AARCH64]
>>
>> But the above hunk, from NetworkPkg.dsc, seems to fall under
>> [LibraryClasses]. I think that might break NetworkPkg.dsc builds on
>> IA32/X64.
Agree. I will move it to ARM arch section.
>>
>> >
>> > [LibraryClasses.common.UEFI_DRIVER]
>> > HobLib|MdePkg/Library/DxeHobLib/DxeHobLib.inf
>> > @@ -72,6 +73,7 @@
>> > # [LibraryClasses.ARM] and NULL mean link this library into all ARM
>images.
>> > #
>> > NULL|ArmPkg/Library/CompilerIntrinsicsLib/CompilerIntrinsicsLib.inf
>> > + ArmSoftFloatLib|ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.inf
>> >
>> > [PcdsFeatureFlag]
>> > gEfiMdePkgTokenSpaceGuid.PcdComponentName2Disable|TRUE
>>
>> I'll let Ard & Leif comment on this.
>>
>
>Sigh.
>
>The RNG code in OpenSSL uses a 'double' to record the current entropy
>level, even though the value is in the range [0 .. 31] (IIRC).
>Unfortunately, this does imply that any .DSC that incorporates TLS or
>other crypto code for ARM needs this resolution to be included.
>
>
>>
>> > @@ -115,7 +117,7 @@
>> > NetworkPkg/Application/IpsecConfig/IpSecConfig.inf
>> > NetworkPkg/Application/VConfig/VConfig.inf
>> >
>> > -[Components.IA32, Components.X64]
>> > +[Components.IA32, Components.X64, Components.ARM,
>Components.AARCH64]
>> > NetworkPkg/IpSecDxe/IpSecDxe.inf
>> > NetworkPkg/IScsiDxe/IScsiDxe.inf
>> > NetworkPkg/UefiPxeBcDxe/UefiPxeBcDxe.inf
>> >
>>
>> Shouldn't we just merge this section into [Components] above?
>>
>> Or are these modules unsuitable for EBC?
>>
>
>(+ Mike)
>
>This came up at the plugfest: is it really necessary to keep building
>arbitrary modules using EBC? EBC is no longer mandatory, and the
>compiler diagnostics of the other toolchains are much better than they
>were 10+ years ago.
Now, those modules can't pass build on EBC, because of CryptoLib. I agree EBC compiler is too old. I agree to remove it.
next prev parent reply other threads:[~2019-04-29 2:02 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-04-25 12:37 [Patch v3 0/3] Add package level include DSC/FDF in NetworkPkg Liming Gao
2019-04-25 12:37 ` [Patch v3 1/3] NetworkPkg DSC: Add the required ARM library to pass ARM build Liming Gao
2019-04-26 22:04 ` [edk2-devel] " Laszlo Ersek
2019-04-26 22:19 ` Ard Biesheuvel
2019-04-29 2:02 ` Liming Gao [this message]
2019-04-25 12:37 ` [Patch v3 2/3] NetworkPkg: Add DSC/FDF include segment files to NetworkPkg Liming Gao
2019-04-29 13:05 ` [edk2-devel] " Laszlo Ersek
2019-04-29 14:18 ` Liming Gao
2019-05-05 15:21 ` Liming Gao
2019-05-06 18:23 ` Laszlo Ersek
2019-05-06 18:49 ` Andrew Fish
2019-05-07 4:23 ` Liming Gao
2019-05-07 1:22 ` Siyuan, Fu
2019-04-25 12:37 ` [Patch v3 3/3] NetworkPkg: Add package level include DSC file Liming Gao
2019-04-29 12:03 ` [edk2-devel] " Laszlo Ersek
2019-04-29 14:15 ` Liming Gao
2019-04-29 13:10 ` [edk2-devel] [Patch v3 0/3] Add package level include DSC/FDF in NetworkPkg Laszlo Ersek
2019-04-29 14:19 ` Liming Gao
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4A89E2EF3DFEDB4C8BFDE51014F606A14E43AF35@SHSMSX104.ccr.corp.intel.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox