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

StandaloneMmPkg: Introduce a PCD to disable shadow boot FV #10642

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

LeviYeoReum
Copy link
Contributor

@LeviYeoReum LeviYeoReum commented Jan 17, 2025

On some Arm platforms the boot firmware volume passed in the HOB by the secure world firmware (TF-A) is never freed and is always mapped as read-only.

On such platforms the heap memory can be saved by disabling the shadow copy of the boot firmware volume. This is useful as on some platforms the amount of memory available is limited.

Therefore, introduce a PCD PcdShadowBfv that platforms can configure to disable the shadow boot FV.
This PCD is set to TRUE by default.

@LeviYeoReum LeviYeoReum force-pushed the levi/disable_shadow_fv branch from c445148 to 466673b Compare January 17, 2025 16:02
@LeviYeoReum LeviYeoReum requested a review from lzeng14 January 17, 2025 16:04
@LeviYeoReum LeviYeoReum force-pushed the levi/disable_shadow_fv branch from 466673b to 3075a09 Compare January 17, 2025 16:25
On some Arm platforms the boot firmware volume passed in the HOB
by the secure world firmware (TF-A) is never freed and is always
mapped as read-only.

On such platforms the heap memory can be saved by disabling the
shadow copy of the boot firmware volume. This is useful as on
some platforms the amount of memory available is limited.

Therefore, introduce a PCD PcdShadowBfv that platforms
can configure to disable the shadow boot FV. This PCD is set to
TRUE by default.

Signed-off-by: Levi Yun <[email protected]>
@LeviYeoReum LeviYeoReum force-pushed the levi/disable_shadow_fv branch from 3075a09 to a571253 Compare January 17, 2025 16:39
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