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+tianoc...@kernel.org>

Cc: Sami Mujawar <sami.muja...@arm.com>

Cc: Ray Ni <ray...@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/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to