As per SBBR specification, "A UEFI runtime environment compliant with SBBR must not be written with any assumption of an identity mapping between virtual and physical memory maps."
Test case implementation was failing the test, if it is not identity mapped, which is incorrect. Corrected test case to warn the user that UEFI runtime environment is identity mapped, instead of failure. Signed-off-by: Sakar Arora <[email protected]> Reported-by: Felix Poludov <[email protected]> --- .../SbbrBootServices/BlackBoxTest/SbbrBootServicesBBTestFunction.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/SbbrBootServices/BlackBoxTest/SbbrBootServicesBBTestFunction.c b/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/SbbrBootServices/BlackBoxTest/SbbrBootServicesBBTestFunction.c index fb50702..c88d60b 100644 --- a/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/SbbrBootServices/BlackBoxTest/SbbrBootServicesBBTestFunction.c +++ b/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/SbbrBootServices/BlackBoxTest/SbbrBootServicesBBTestFunction.c @@ -201,13 +201,13 @@ BBTestMemoryMapTest ( // // Checking for identity mapping // - if (MemoryMapDescriptor->PhysicalStart != MemoryMapDescriptor->VirtualStart) { + if (MemoryMapDescriptor->PhysicalStart == MemoryMapDescriptor->VirtualStart) { StandardLib->RecordAssertion ( StandardLib, - EFI_TEST_ASSERTION_FAILED, + EFI_TEST_ASSERTION_WARNING, gSbbrBootServicesAssertion001Guid, L"MemoryMap", - L"%a:%d - MemoryMap 0x%X Not Identity Mapped", + L"%a:%d - MemoryMap 0x%X is Identity Mapped. UEFI runtime environment must not be written with any assumption of an identity mapping between virtual and physical memory maps.", __FILE__, __LINE__, MemoryMapDescriptor -- 2.7.4 _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
