Refine BaseMemoryTestLib and add instance with full functionality.

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@7831 6f19259b-4bc3-4df7-8a09-765794883524

1 files changed, 31 insertions, 16 deletions

diff --git a/MdeModulePkg/Library/BaseMemoryTestLibNull/BaseMemoryTestLibNull.c b/MdeModulePkg/Library/BaseMemoryTestLibNull/BaseMemoryTestLibNull.c
index 4163e61bb3..e88a22536a 100644
--- a/MdeModulePkg/Library/BaseMemoryTestLibNull/BaseMemoryTestLibNull.c
+++ b/MdeModulePkg/Library/BaseMemoryTestLibNull/BaseMemoryTestLibNull.c
@@ -15,23 +15,22 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #include <Library/BaseMemoryTestLib.h>

 

 /**

-  Test a system memory range with sparsely sampled memory units.

+  Perform a quick system memory range test.

 

-  This function tests a system memory range, whose memory units

-  are sampled sparsely. It leads to quick performance but less

-  reliability.

+  This function performs a quick system memory range test. It leads to quick performance

+  but least reliability.

 

   @param  StartAddress           Start address of the memory range to test.

   @param  Length                 Length of the memory range to test.

   @param  ErrorAddress           Address of the memory where error is encountered.

 

   @retval RETURN_SUCCESS         The memory range passes test.

-  @retval RETURNEFI_DEVICE_ERROR The memory range does not pass test.

+  @retval RETURN_DEVICE_ERROR    The memory range does not pass test.

 

 **/

 RETURN_STATUS

 EFIAPI

-SparseMemoryTest (

+QuickMemoryTest (

   IN  VOID      *StartAddress,

   IN  UINT64    Length,

   OUT VOID      **ErrorAddress

@@ -41,23 +40,23 @@ SparseMemoryTest (
 }

 

 /**

-  Test a system memory range with extensively sampled memory units.

+  Test a system memory range with sparsely sampled memory units.

 

   This function tests a system memory range, whose memory units

-  are sampled extensively. Compared with SparseMemoryTest, it achieves

-  more reliability and less performance.

+  are sampled sparsely. It leads to relatively good performance

+  and partial reliability.

 

   @param  StartAddress           Start address of the memory range to test.

   @param  Length                 Length of the memory range to test.

   @param  ErrorAddress           Address of the memory where error is encountered.

 

   @retval RETURN_SUCCESS         The memory range passes test.

-  @retval RETURNEFI_DEVICE_ERROR The memory range does not pass test.

+  @retval RETURN_DEVICE_ERROR    The memory range does not pass test.

 

 **/

 RETURN_STATUS

 EFIAPI

-ExtensiveMemoryTest (

+SparseMemoryTest (

   IN  VOID      *StartAddress,

   IN  UINT64    Length,

   OUT VOID      **ErrorAddress

@@ -67,23 +66,23 @@ ExtensiveMemoryTest (
 }

 

 /**

-  Test a system memory range with every memory unit checked.

+  Test a system memory range with extensively sampled memory units.

 

   This function tests a system memory range, whose memory units

-  are fully checked. It leads to complete reliability with the

-  cost of performance.

+  are sampled extensively. Compared with SparseMemoryTest, it achieves

+  more reliability and less performance.

 

   @param  StartAddress           Start address of the memory range to test.

   @param  Length                 Length of the memory range to test.

   @param  ErrorAddress           Address of the memory where error is encountered.

 

   @retval RETURN_SUCCESS         The memory range passes test.

-  @retval RETURNEFI_DEVICE_ERROR The memory range does not pass test.

+  @retval RETURN_DEVICE_ERROR    The memory range does not pass test.

 

 **/

 RETURN_STATUS

 EFIAPI

-FullMemoryTest (

+ExtensiveMemoryTest (

   IN  VOID      *StartAddress,

   IN  UINT64    Length,

   OUT VOID      **ErrorAddress

@@ -91,3 +90,19 @@ FullMemoryTest (
 {

   return RETURN_SUCCESS;

 }

+

+/**

+  Check if soft ECC initialzation is needed for system

+

+  @retval TRUE         Soft ECC initialzation is needed.

+  @retval FALSE        Soft ECC initialzation is not needed.

+

+**/

+BOOLEAN

+EFIAPI

+IsSoftEccInitRequired (

+  VOID

+  )

+{

+  return FALSE;

+}