Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Platform hook notify #10679

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Platform hook notify #10679

wants to merge 2 commits into from

Conversation

dyap0
Copy link

@dyap0 dyap0 commented Jan 27, 2025

Description

Provide PlatformHook NotifyProtocolGuid before and after SMM dispatch.

How This Was Tested

Manual test case run from platform code by hook on NotifyProtocolGuid.

Integration Instructions

N/A

dyap0 added 2 commits January 26, 2025 23:15
Created StandaloneMM Entry/Exit Notify Protocol for PlatformHook,
similar to SMM PlatformHook Before/After SmmDispatch.

Signed-off-by: Diat Siah Yap <diat.siah.yap@intel.com>
Created StandaloneMM Entry/Exit Notify Protocol for PlatformHook,
similar to SMM PlatformHook Before/After SmmDispatch.

Signed-off-by: Diat Siah Yap <diat.siah.yap@intel.com>
@ardbiesheuvel
Copy link
Member

Hello Diat,

Description

Provide PlatformHook NotifyProtocolGuid before and after SMM dispatch.

We can tell from the code that this is what your series does. You will also need to describe which problem it solves, and why this is the right way to solve it. In this case, you will also need to explain how these changes are supported by the PI specification, given that you are adding protocol definitions to MdePkg.

Thanks,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants