[EXTERNAL] Re: [edk2-devel] BaseCryptLib in ARM incorrectly marked as a DXE_DRIVER

Sat Jul 17 04:21:41 UTC 2021

Thank you Andrew, you are right. I missed the fact that there are 2 INFs in there.

From: Andrew Fish <afish at apple.com>
Sent: Wednesday, July 14, 2021 9:14 PM
To: edk2-devel-groups-io <devel at edk2.groups.io>; Arti Gupta <arti.gupta at microsoft.com>
Subject: [EXTERNAL] Re: [edk2-devel] BaseCryptLib in ARM incorrectly marked as a DXE_DRIVER

Arti,

Can you be a little more specific? Which INF file? How is it used? Sorry I don't know a lot about this library.

The different INF files pull in different allocation strategies for the common library code.

https://github.com/tianocore/edk2/blob/master/CryptoPkg/Library/BaseCryptLib/RuntimeCryptLib.inf<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftianocore%2Fedk2%2Fblob%2Fmaster%2FCryptoPkg%2FLibrary%2FBaseCryptLib%2FRuntimeCryptLib.inf&data=04%7C01%7Carti.gupta%40microsoft.com%7C9a0bceb1743449d13bdf08d947470322%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637619193528251436%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C2000&sdata=l8ZQyxKzbJtGrcdwBpr9ayjex0NZ0k%2FoYW6QxHsClPI%3D&reserved=0>
SysCall/RuntimeMemAllocation.c
https://github.com/tianocore/edk2/blob/master/CryptoPkg/Library/BaseCryptLib/BaseCryptLib.inf<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftianocore%2Fedk2%2Fblob%2Fmaster%2FCryptoPkg%2FLibrary%2FBaseCryptLib%2FBaseCryptLib.inf&data=04%7C01%7Carti.gupta%40microsoft.com%7C9a0bceb1743449d13bdf08d947470322%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637619193528251436%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C2000&sdata=%2FLD37K8PZCqCDzNWL53WQPwx5PYMZYIvtCA0lT88RsM%3D&reserved=0>
SysCall/BaseMemAllocation.c

The most important line in the INF is:

LIBRARY_CLASS                  = BaseCryptLib|DXE_RUNTIME_DRIVER

or

LIBRARY_CLASS                  = BaseCryptLib|DXE_DRIVER DXE_CORE UEFI_APPLICATION UEFI_DRIVER

The list after the | is what module types this library can be linked into. I don't remember what issue a wrong MODULE_TYPE for a library would cause. I think the LIBRARY_CLASS list may override the MODULE_TYPE in terms of how this library gets consumed.

Thanks,

Andrew Fish

On Jul 14, 2021, at 1:34 PM, Arti Gupta via groups.io<https://nam06.safelinks.protection.outlook.com/?url=http%3A%2F%2Fgroups.io%2F&data=04%7C01%7Carti.gupta%40microsoft.com%7C9a0bceb1743449d13bdf08d947470322%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637619193528261394%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C2000&sdata=H%2BE00BTI2lFvJjRtlz2pendG2ZHh4pO8zMvVffUT1Vg%3D&reserved=0> <arti.gupta=microsoft.com at groups.io<mailto:arti.gupta=microsoft.com at groups.io>> wrote:

Hello,

I am seeing that the BaseCryptLib in ARM is declared as a DXE_DRIVER instead of a DXE_RUNTIME_DRIVER, however it has a RuntimeMemAllocation.c which registers for the virtualaddresschange event so that tells me that the lib is expected to live in runtime. Is this bug known? What are the next steps for getting this fixed?

Thanks,
Arti

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#77842): https://edk2.groups.io/g/devel/message/77842
Mute This Topic: https://groups.io/mt/84264574/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/20210717/615fca52/attachment.htm>