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.

Sign up for GitHub

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

Jump to bottom

StandaloneMmPkg/MmIpl: Correct unblocked memory regions attribute #6131

Merged
merged 1 commit into from
Sep 11, 2024
Merged

StandaloneMmPkg/MmIpl: Correct unblocked memory regions attribute #6131

merged 1 commit into from
Sep 11, 2024

Conversation

hongbin123
Copy link
Contributor

When PcdCpuSmmRestrictedMemoryAccess is FALSE, resource HOB should exclude unblocked memory regions.

  • Breaking change?
    • Breaking change - Does this PR cause a break in build or boot behavior?
    • Examples: Does it add a new library class or move a module to a different repo.
  • Impacts security?
    • Security - Does this PR have a direct security impact?
    • Examples: Crypto algorithm change or buffer overflow fix.
  • Includes tests?
    • Tests - Does this PR include any explicit test code?
    • Examples: Unit tests or integration tests.

@hongbin123 hongbin123 force-pushed the collect_unblockmemoryregions branch 3 times, most recently from de19ea8 to e1b285f Compare August 29, 2024 07:30
@hongbin123 hongbin123 changed the title StandaloneMmPkg/MmIpl: Collect unblocked memory regions StandaloneMmPkg/MmIpl: Correct unblocked memory regions attribute Aug 29, 2024
@hongbin123 hongbin123 force-pushed the collect_unblockmemoryregions branch 2 times, most recently from 59cf9c2 to 378ece1 Compare September 2, 2024 08:43
@hongbin123 hongbin123 force-pushed the collect_unblockmemoryregions branch 4 times, most recently from 99faca1 to 8035052 Compare September 9, 2024 11:16
@niruiyu niruiyu added the push Auto push patch series in PR if all checks pass label Sep 10, 2024
@hongbin123
StandaloneMmPkg/MmIpl: Correct unblocked memory regions attribute
0b06fba 
When CPU smm profile feature was enabled, unblocked memory should
not set logging attribute when building resource HOB.

Signed-off-by: Hongbin1 Zhang <[email protected]>
Cc: Jiewen Yao <[email protected]>
Cc: Ray Ni <[email protected]>
Cc: Star Zeng <[email protected]>
Cc: Jiaxin Wu <[email protected]>
Cc: Wei6 Xu <[email protected]>
Cc: Sami Mujawar <[email protected]>
Cc: Ard Biesheuvel <[email protected]>
Cc: Supreeth Venkatesh <[email protected]>
@mergify mergify bot merged commit 58b4bf7 into tianocore:master Sep 11, 2024
126 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@niruiyu niruiyu niruiyu approved these changes

@ardbiesheuvel ardbiesheuvel Awaiting requested review from ardbiesheuvel

@samimujawar samimujawar Awaiting requested review from samimujawar

@jiaxinwu jiaxinwu Awaiting requested review from jiaxinwu

Assignees
No one assigned
Labels
push Auto push patch series in PR if all checks pass
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hongbin123 @jiaxinwu @niruiyu