Re: [edk2-devel] [PATCH 0/2] Add Platform Hook Lib into StandaloneMmCore

["Michael Kubacki" <mikuback@linux.microsoft.com>]

This allows ambiguous "platform" code in the critical path of the MM 
core. Is this necessary?

Do you need this for one feature that others might too and can be 
abstracted? Or, do you plan to perform an unknown and arbitrary number 
of changes behind the hook over time?

Thanks,
Michael

On 10/26/2023 11:28 PM, Xu, Wei6 wrote:
> This patch set is to add StandaloneMmCorePlatformHookLib into StandaloneMmCore.
> 
> This library class defines a set of platform hooks called by the Standalone Mm Core. With this library, platform can perform specific tasks before and after invoking registered MMI handlers.
> We need this library to implement our feature.
> 
> PR: https://github.com/tianocore/edk2/pull/4949
> 
> 
> 
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> 
> Cc: Sami Mujawar <sami.mujawar@arm.com>
> 
> Cc: Ray Ni <ray.ni@intel.com>
> 
> 
> Wei6 Xu (2):
>    StandaloneMmPkg: Add Standalone Mm Core platform hook lib.
>    StandaloneMmPkg/Core: Consumes Standalone Mm Core Platform Hook Lib.
> 
>   StandaloneMmPkg/Core/StandaloneMmCore.c       |  7 ++-
>   .../StandaloneMmCorePlatformHookLibNull.c     | 45 +++++++++++++++++++
>   StandaloneMmPkg/Core/StandaloneMmCore.h       |  1 +
>   StandaloneMmPkg/Core/StandaloneMmCore.inf     |  1 +
>   .../Library/StandaloneMmCorePlatformHookLib.h | 44 ++++++++++++++++++
>   .../StandaloneMmCorePlatformHookLibNull.inf   | 30 +++++++++++++
>   StandaloneMmPkg/StandaloneMmPkg.dec           |  4 ++
>   StandaloneMmPkg/StandaloneMmPkg.dsc           |  2 +
>   8 files changed, 133 insertions(+), 1 deletion(-)
>   create mode 100644 StandaloneMmPkg/Library/StandaloneMmCorePlatformHookLibNull/StandaloneMmCorePlatformHookLibNull.c
>   create mode 100644 StandaloneMmPkg/Include/Library/StandaloneMmCorePlatformHookLib.h
>   create mode 100644 StandaloneMmPkg/Library/StandaloneMmCorePlatformHookLibNull/StandaloneMmCorePlatformHookLibNull.inf
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#110220): https://edk2.groups.io/g/devel/message/110220
Mute This Topic: https://groups.io/mt/102214566/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-