[edk2-devel] [PATCH v1 01/16] ArmPkg/ArmMmuBaseLib: Disallow STANDALONE_MM

[Ryan Harkin]

Hi Leif,

There's a 00/16 in the series with a brief explanation that I'll paste here:

"This series of patch commits moves a number of modules and libraries around in order
to better un-silo the Arm-specific code. Unnecessary abstractions were removed, some
duplicate code was abstracted, and generally things were cleaned up so that common
code (e.g. MdePkg, MdeModulePkg, StandaloneMmPkg) does not require ArmPkg or
ArmPlatformPkg.

Also, some things that were previously hidden away under ArmPkg are now located
with the core functionality or build tooling that makes sense for it."

Cheers,
Ryan

________________________________
From: devel at edk2.groups.io <devel at edk2.groups.io> on behalf of Leif Lindholm via groups.io <leif=nuviainc.com at groups.io>
Sent: 02 November 2021 08:43
To: brbarkel at microsoft.com <bret at corthon.com>
Cc: devel at edk2.groups.io <devel at edk2.groups.io>; Ard Biesheuvel <ardb+tianocore at kernel.org>; Sean Brogan <sean.brogan at microsoft.com>
Subject: Re: [edk2-devel] [PATCH v1 01/16] ArmPkg/ArmMmuBaseLib: Disallow STANDALONE_MM

Hi Bret,

Is there some overall scope this set is aiming towards, or are these
16 unrelated patches to ArmPkg? Could you send out a cover-letter
separately to clarify?

Best Regards,

Leif

On Mon, Nov 01, 2021 at 12:56:33 -0700, brbarkel at microsoft.com wrote:
> From: Bret Barkelew <brbarkel at microsoft.com>
>
> Not sure what all module types this HAS been validated against, but it
> doesn't work with STANDALONE_MM or STANDALONE_MM_CORE.
>
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3647
>
> Cc: Leif Lindholm <leif at nuviainc.com>
> Cc: Ard Biesheuvel <ardb+tianocore at kernel.org>
> Cc: Sean Brogan <sean.brogan at microsoft.com>
> Signed-off-by: Bret Barkelew <bret.barkelew at microsoft.com>
> ---
>  ArmPkg/Library/ArmMmuLib/ArmMmuBaseLib.inf | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/ArmPkg/Library/ArmMmuLib/ArmMmuBaseLib.inf b/ArmPkg/Library/ArmMmuLib/ArmMmuBaseLib.inf
> index 2a7e7147958c..d0988b27d582 100644
> --- a/ArmPkg/Library/ArmMmuLib/ArmMmuBaseLib.inf
> +++ b/ArmPkg/Library/ArmMmuLib/ArmMmuBaseLib.inf
> @@ -13,7 +13,7 @@ [Defines]
>    FILE_GUID                      = da8f0232-fb14-42f0-922c-63104d2c70bd
>    MODULE_TYPE                    = BASE
>    VERSION_STRING                 = 1.0
> -  LIBRARY_CLASS                  = ArmMmuLib
> +  LIBRARY_CLASS                  = ArmMmuLib | DXE_DRIVER PEIM PEI_CORE DXE_CORE DXE_RUNTIME_DRIVER SEC
>
>  [Defines.AARCH64]
>    CONSTRUCTOR                    = ArmMmuBaseLibConstructor
> --
> 2.31.1.windows.1
>





IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.


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