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

[CHERRY-PICK] DxePagingAuditTestApp: Remove MemoryOutsideEfiMemoryMapIsInaccessibleTest #568

Merged
merged 1 commit into from
Oct 1, 2024

Conversation

os-d
Copy link
Contributor

@os-d os-d commented Aug 28, 2024

Description

MemoryOutsideEfiMemoryMapIsInaccessible was attempting to test that memory outside the EFI_MEMORY_MAP was marked EFI_MEMORY_RP or unmapped, however this is not a valid test as we expect there to be ranges outside of the EFI_MEMORY_MAP, such as GCD non-existent memory and non-runtime MMIO ranges. This patch removes the test.

Cherry-picked from 2311.

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

How This Was Tested

From 2311.

Integration Instructions

N/A.

@os-d os-d requested review from Flickdm, kenlautner and apop5 August 28, 2024 22:22
@github-actions github-actions bot added impact:non-functional Does not have a functional impact impact:testing Affects testing labels Aug 28, 2024
@os-d os-d changed the base branch from release/202405 to dev/202405 October 1, 2024 21:24
… Test

MemoryOutsideEfiMemoryMapIsInaccessible was attempting to test that memory
outside the EFI_MEMORY_MAP was marked EFI_MEMORY_RP or unmapped, however this
is not a valid test as we expect there to be ranges outside of the EFI_MEMORY_MAP,
such as GCD non-existent memory and non-runtime MMIO ranges. This patch removes
the test.
@os-d os-d merged commit d9ae87d into microsoft:dev/202405 Oct 1, 2024
28 checks passed
@os-d os-d deleted the cp branch October 1, 2024 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact:non-functional Does not have a functional impact impact:testing Affects testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants