tyler.durden
/
audk
mirror of https://github.com/acidanthera/audk.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

MdeModulePkg/NullMemoryTest: Fix bug in CompatibleRangeTest 

CompatibleRangeTest() contains two bugs:
1. It doesn't reject the memory above 16MB
2. it cannot handle the case when the partial or whole range of
   requested memory is already tested.

The patch fixes the two bugs.

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com>
Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>
This commit is contained in:
Ruiyu Ni 2018-03-06 11:31:57 +08:00
parent ee37e96479
commit 61c1742ba1
1 changed files with 45 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

53
MdeModulePkg/Universal/MemoryTest/NullMemoryTestDxe/NullMemoryTest.c
View File

@ -240,14 +240,51 @@ GenCompatibleRangeTest (
{
EFI_STATUS Status;
EFI_GCD_MEMORY_SPACE_DESCRIPTOR Descriptor;
EFI_PHYSICAL_ADDRESS CurrentBase;
UINT64 CurrentLength;
Status = gDS->GetMemorySpaceDescriptor (StartAddress, &Descriptor);
if (!EFI_ERROR (Status)) {
Status = ConvertToTestedMemory (
Descriptor.BaseAddress,
Descriptor.Length,
Descriptor.Capabilities
);
//
// Check if the parameter is below 16MB
//
if (StartAddress + Length > SIZE_16MB) {
return EFI_INVALID_PARAMETER;
}
return Status;
CurrentBase = StartAddress;
do {
//
// Check the required memory range status; if the required memory range span
// the different GCD memory descriptor, it may be cause different action.
//
Status = gDS->GetMemorySpaceDescriptor (
CurrentBase,
&Descriptor
);
if (EFI_ERROR (Status)) {
return Status;
}
if (Descriptor.GcdMemoryType == EfiGcdMemoryTypeReserved &&
(Descriptor.Capabilities & (EFI_MEMORY_PRESENT | EFI_MEMORY_INITIALIZED | EFI_MEMORY_TESTED)) ==
(EFI_MEMORY_PRESENT | EFI_MEMORY_INITIALIZED)
) {
CurrentLength = Descriptor.BaseAddress + Descriptor.Length - CurrentBase;
if (CurrentBase + CurrentLength > StartAddress + Length) {
CurrentLength = StartAddress + Length - CurrentBase;
}
Status = ConvertToTestedMemory (
CurrentBase,
CurrentLength,
Descriptor.Capabilities
);
if (EFI_ERROR (Status)) {
return Status;
}
}
CurrentBase = Descriptor.BaseAddress + Descriptor.Length;
} while (CurrentBase < StartAddress + Length);
//
// Here means the required range already be tested, so just return success.
//
return EFI_SUCCESS;
}