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

From: Andrew Fish <af...@apple.com>
Sent: Wednesday, July 14, 2021 9:14 PM
To: edk2-devel-groups-io <devel@edk2.groups.io>; Arti Gupta 
<arti.gu...@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....@groups.io<mailto:arti.gupta=microsoft....@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/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to