CorebootModulePkg: Fix memmap issue
authorgdong1 <guo.dong@intel.com>
Wed, 26 Oct 2016 23:48:40 +0000 (16:48 -0700)
committerMaurice Ma <maurice.ma@intel.com>
Thu, 27 Oct 2016 00:11:54 +0000 (17:11 -0700)
Some reserved memory (e.g. CSE reserved memory) might be in the
middle of usable physical memory. The current memory map caculation
could not handle this case. This patch fixed this issue.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: gdong1 <guo.dong@intel.com>
Reviewed-by: Maurice Ma <maurice.ma@intel.com>
CorebootModulePkg/CbSupportPei/CbSupportPei.c
CorebootModulePkg/CbSupportPei/CbSupportPei.h
CorebootModulePkg/Include/Library/CbParseLib.h
CorebootModulePkg/Library/CbParseLib/CbParseLib.c

index 80fd611..8fa0ac5 100755 (executable)
@@ -140,6 +140,72 @@ CbPeiReportRemainedFvs (
   }\r
 }\r
 \r
+/**\r
+  Based on memory base, size and type, build resource descripter HOB.\r
+\r
+  @param  Base    Memory base address.\r
+  @param  Size    Memory size.\r
+  @param  Type    Memory type.\r
+  @param  Param   A pointer to CB_MEM_INFO.\r
+\r
+  @retval EFI_SUCCESS if it completed successfully.\r
+**/\r
+EFI_STATUS\r
+CbMemInfoCallback (\r
+  UINT64                  Base,\r
+  UINT64                  Size,\r
+  UINT32                  Type,\r
+  VOID                    *Param\r
+  )\r
+{\r
+  CB_MEM_INFO             *MemInfo;\r
+  UINTN                   Attribue;\r
+\r
+  Attribue = EFI_RESOURCE_ATTRIBUTE_PRESENT |\r
+             EFI_RESOURCE_ATTRIBUTE_INITIALIZED |\r
+             EFI_RESOURCE_ATTRIBUTE_TESTED |\r
+             EFI_RESOURCE_ATTRIBUTE_UNCACHEABLE |\r
+             EFI_RESOURCE_ATTRIBUTE_WRITE_COMBINEABLE |\r
+             EFI_RESOURCE_ATTRIBUTE_WRITE_THROUGH_CACHEABLE |\r
+             EFI_RESOURCE_ATTRIBUTE_WRITE_BACK_CACHEABLE;\r
+\r
+  MemInfo = (CB_MEM_INFO *)Param;\r
+  if (Base >= 0x100000) {\r
+    if (Type == CB_MEM_RAM) {\r
+      if (Base < 0x100000000ULL) {\r
+        MemInfo->UsableLowMemTop = (UINT32)(Base + Size);\r
+      } else {\r
+        Attribue &= ~EFI_RESOURCE_ATTRIBUTE_TESTED;\r
+      }\r
+      BuildResourceDescriptorHob (\r
+        EFI_RESOURCE_SYSTEM_MEMORY,\r
+        Attribue,\r
+        (EFI_PHYSICAL_ADDRESS)Base,\r
+        Size\r
+        );\r
+    } else if (Type == CB_MEM_TABLE) {\r
+      BuildResourceDescriptorHob (\r
+        EFI_RESOURCE_MEMORY_RESERVED,\r
+        Attribue,\r
+        (EFI_PHYSICAL_ADDRESS)Base,\r
+        Size\r
+        );\r
+      MemInfo->SystemLowMemTop = ((UINT32)(Base + Size) + 0x0FFFFFFF) & 0xF0000000;\r
+    } else if (Type == CB_MEM_RESERVED) {\r
+      if ((MemInfo->SystemLowMemTop == 0) || (Base < MemInfo->SystemLowMemTop)) {\r
+        BuildResourceDescriptorHob (\r
+          EFI_RESOURCE_MEMORY_RESERVED,\r
+          Attribue,\r
+          (EFI_PHYSICAL_ADDRESS)Base,\r
+          Size\r
+          ); \r
+      }\r
+    }\r
+  }\r
+  \r
+  return EFI_SUCCESS;\r
+}\r
+\r
 /**\r
   This is the entrypoint of PEIM\r
 \r
@@ -155,9 +221,9 @@ CbPeiEntryPoint (
   IN CONST EFI_PEI_SERVICES     **PeiServices\r
   )\r
 {\r
-  EFI_STATUS Status;\r
-  UINT64 LowMemorySize, HighMemorySize;\r
-  UINT64 PeiMemSize = SIZE_64MB;   // 64 MB\r
+  EFI_STATUS           Status;\r
+  UINT64               LowMemorySize;\r
+  UINT64               PeiMemSize = SIZE_64MB;   // 64 MB\r
   EFI_PHYSICAL_ADDRESS PeiMemBase = 0;\r
   UINT32               RegEax;\r
   UINT8                PhysicalAddressBits;\r
@@ -173,23 +239,12 @@ CbPeiEntryPoint (
   UINTN                PmCtrlRegBase, PmTimerRegBase, ResetRegAddress, ResetValue;\r
   UINTN                PmEvtBase;\r
   UINTN                PmGpeEnBase;\r
-\r
-  LowMemorySize = 0;\r
-  HighMemorySize = 0;\r
-\r
-  Status = CbParseMemoryInfo (&LowMemorySize, &HighMemorySize);\r
-  if (EFI_ERROR(Status))\r
-    return Status;\r
-\r
-  DEBUG((EFI_D_ERROR, "LowMemorySize: 0x%lx.\n", LowMemorySize));\r
-  DEBUG((EFI_D_ERROR, "HighMemorySize: 0x%lx.\n", HighMemorySize));\r
-\r
-  ASSERT (LowMemorySize > 0);\r
+  CB_MEM_INFO          CbMemInfo;\r
 \r
   //\r
   // Report lower 640KB of RAM. Attribute EFI_RESOURCE_ATTRIBUTE_TESTED  \r
- // is intentionally omitted to prevent erasing of the coreboot header  \r
- // record before it is processed by CbParseMemoryInfo.\r
 // is intentionally omitted to prevent erasing of the coreboot header  \r
 // record before it is processed by CbParseMemoryInfo.\r
   //\r
   BuildResourceDescriptorHob (\r
     EFI_RESOURCE_SYSTEM_MEMORY,\r
@@ -221,37 +276,16 @@ CbPeiEntryPoint (
     (UINT64)(0x60000)\r
     );\r
 \r
-   BuildResourceDescriptorHob (\r
-    EFI_RESOURCE_SYSTEM_MEMORY,\r
-    (\r
-       EFI_RESOURCE_ATTRIBUTE_PRESENT |\r
-       EFI_RESOURCE_ATTRIBUTE_INITIALIZED |\r
-       EFI_RESOURCE_ATTRIBUTE_TESTED |\r
-       EFI_RESOURCE_ATTRIBUTE_UNCACHEABLE |\r
-       EFI_RESOURCE_ATTRIBUTE_WRITE_COMBINEABLE |\r
-       EFI_RESOURCE_ATTRIBUTE_WRITE_THROUGH_CACHEABLE |\r
-       EFI_RESOURCE_ATTRIBUTE_WRITE_BACK_CACHEABLE\r
-    ),\r
-    (EFI_PHYSICAL_ADDRESS)(0x100000),\r
-    (UINT64) (LowMemorySize - 0x100000)\r
-    );\r
-\r
-  if (HighMemorySize > 0) {\r
-    BuildResourceDescriptorHob (\r
-    EFI_RESOURCE_SYSTEM_MEMORY,\r
-    (\r
-       EFI_RESOURCE_ATTRIBUTE_PRESENT |\r
-       EFI_RESOURCE_ATTRIBUTE_INITIALIZED |\r
-       EFI_RESOURCE_ATTRIBUTE_UNCACHEABLE |\r
-       EFI_RESOURCE_ATTRIBUTE_WRITE_COMBINEABLE |\r
-       EFI_RESOURCE_ATTRIBUTE_WRITE_THROUGH_CACHEABLE |\r
-       EFI_RESOURCE_ATTRIBUTE_WRITE_BACK_CACHEABLE\r
-    ),\r
-    (EFI_PHYSICAL_ADDRESS)(0x100000000ULL),\r
-    HighMemorySize\r
-    );\r
+  ZeroMem (&CbMemInfo, sizeof(CbMemInfo));\r
+  Status = CbParseMemoryInfo (CbMemInfoCallback, (VOID *)&CbMemInfo);\r
+  if (EFI_ERROR(Status)) {\r
+    return Status;\r
   }\r
 \r
+  LowMemorySize = CbMemInfo.UsableLowMemTop;\r
+  DEBUG ((EFI_D_INFO, "Low memory 0x%lx\n", LowMemorySize));\r
+  DEBUG ((EFI_D_INFO, "SystemLowMemTop 0x%x\n", CbMemInfo.SystemLowMemTop));\r
+\r
   //\r
   // Should be 64k aligned\r
   //\r
index f897dc4..ed5cdb4 100644 (file)
@@ -38,5 +38,11 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #include <Guid/AcpiBoardInfoGuid.h>\r
 \r
 #include <Ppi/MasterBootMode.h>\r
+#include "Coreboot.h"\r
+\r
+typedef struct {\r
+  UINT32  UsableLowMemTop;\r
+  UINT32  SystemLowMemTop;\r
+} CB_MEM_INFO;\r
 \r
 #endif\r
index a023246..064baf3 100644 (file)
 **/\r
 #include <Guid/FrameBufferInfoGuid.h>\r
 \r
+typedef RETURN_STATUS \\r
+        (*CB_MEM_INFO_CALLBACK) (UINT64 Base, UINT64 Size, UINT32 Type, VOID *Param);\r
+\r
 /**\r
   Acquire the memory information from the coreboot table in memory.\r
 \r
-  @param  pLowMemorySize     Pointer to the variable of low memory size\r
-  @param  pHighMemorySize    Pointer to the variable of high memory size\r
+  @param  MemInfoCallback     The callback routine\r
+  @param  pParam              Pointer to the callback routine parameter\r
 \r
   @retval RETURN_SUCCESS     Successfully find out the memory information.\r
-  @retval RETURN_INVALID_PARAMETER    Invalid input parameters.\r
   @retval RETURN_NOT_FOUND   Failed to find the memory information.\r
 \r
 **/\r
 RETURN_STATUS\r
 CbParseMemoryInfo (\r
-  IN UINT64*    pLowMemorySize,\r
-  IN UINT64*    pHighMemorySize\r
+  IN  CB_MEM_INFO_CALLBACK  MemInfoCallback,\r
+  IN  VOID                  *pParam\r
   );\r
 
 /**\r
index 7c81a51..305e38f 100644 (file)
@@ -2,7 +2,7 @@
   This library will parse the coreboot table in memory and extract those required\r
   information.\r
 \r
-  Copyright (c) 2014 - 2015, Intel Corporation. All rights reserved.<BR>\r
+  Copyright (c) 2014 - 2016, Intel Corporation. All rights reserved.<BR>\r
   This program and the accompanying materials\r
   are licensed and made available under the terms and conditions of the BSD License\r
   which accompanies this distribution.  The full text of the license may be found at\r
@@ -33,7 +33,7 @@
   @return          the UNIT64 value after convertion.\r
 \r
 **/\r
-UINT64
+UINT64\r
 cb_unpack64 (\r
   IN struct cbuint64 val\r
   )\r
@@ -216,8 +216,8 @@ FindCbMemTable (
         *pMemTableSize = Entries[Idx].size;\r
       }\r
 \r
-      DEBUG ((EFI_D_INFO, "Find CbMemTable Id 0x%x, base %p, size 0x%x\n",
-        TableId, *pMemTable, Entries[Idx].size));
+      DEBUG ((EFI_D_INFO, "Find CbMemTable Id 0x%x, base %p, size 0x%x\n",\r
+        TableId, *pMemTable, Entries[Idx].size));\r
       return RETURN_SUCCESS;\r
     }\r
   }\r
@@ -229,18 +229,17 @@ FindCbMemTable (
 /**\r
   Acquire the memory information from the coreboot table in memory.\r
 \r
-  @param  pLowMemorySize     Pointer to the variable of low memory size\r
-  @param  pHighMemorySize    Pointer to the variable of high memory size\r
+  @param  MemInfoCallback     The callback routine\r
+  @param  pParam              Pointer to the callback routine parameter\r
 \r
   @retval RETURN_SUCCESS     Successfully find out the memory information.\r
-  @retval RETURN_INVALID_PARAMETER    Invalid input parameters.\r
   @retval RETURN_NOT_FOUND   Failed to find the memory information.\r
 \r
 **/\r
 RETURN_STATUS\r
 CbParseMemoryInfo (\r
-  OUT UINT64     *pLowMemorySize,\r
-  OUT UINT64     *pHighMemorySize\r
+  IN  CB_MEM_INFO_CALLBACK  MemInfoCallback,\r
+  IN  VOID                  *pParam\r
   )\r
 {\r
   struct cb_memory         *rec;\r
@@ -249,10 +248,6 @@ CbParseMemoryInfo (
   UINT64                   Size;\r
   UINTN                    Index;\r
 \r
-  if ((pLowMemorySize == NULL) || (pHighMemorySize == NULL)) {\r
-    return RETURN_INVALID_PARAMETER;\r
-  }\r
-\r
   //\r
   // Get the coreboot memory table\r
   //\r
@@ -265,9 +260,6 @@ CbParseMemoryInfo (
     return RETURN_NOT_FOUND;\r
   }\r
 \r
-  *pLowMemorySize = 0;\r
-  *pHighMemorySize = 0;\r
-\r
   for (Index = 0; Index < MEM_RANGE_COUNT(rec); Index++) {\r
     Range = MEM_RANGE_PTR(rec, Index);\r
     Start = cb_unpack64(Range->start);\r
@@ -275,19 +267,9 @@ CbParseMemoryInfo (
     DEBUG ((EFI_D_INFO, "%d. %016lx - %016lx [%02x]\n",\r
             Index, Start, Start + Size - 1, Range->type));\r
 \r
-    if (Range->type != CB_MEM_RAM) {\r
-      continue;\r
-    }\r
-\r
-    if (Start + Size < 0x100000000ULL) {\r
-      *pLowMemorySize = Start + Size;\r
-    } else {\r
-      *pHighMemorySize = Start + Size - 0x100000000ULL;\r
-    }\r
+    MemInfoCallback (Start, Size, Range->type, pParam);\r
   }\r
 \r
-  DEBUG ((EFI_D_INFO, "Low memory 0x%lx, High Memory 0x%lx\n", *pLowMemorySize, *pHighMemorySize));\r
-\r
   return RETURN_SUCCESS;\r
 }\r
 \r
@@ -469,25 +451,25 @@ CbParseFadtInfo (
         }\r
         DEBUG ((EFI_D_INFO, "Reset Value 0x%x\n", Fadt->ResetValue));\r
 \r
-        if (pPmEvtReg != NULL) {
+        if (pPmEvtReg != NULL) {\r
           *pPmEvtReg = Fadt->Pm1aEvtBlk;\r
           DEBUG ((EFI_D_INFO, "PmEvt Reg 0x%x\n", Fadt->Pm1aEvtBlk));\r
         }\r
 \r
-        if (pPmGpeEnReg != NULL) {
+        if (pPmGpeEnReg != NULL) {\r
           *pPmGpeEnReg = Fadt->Gpe0Blk + Fadt->Gpe0BlkLen / 2;\r
           DEBUG ((EFI_D_INFO, "PmGpeEn Reg 0x%x\n", *pPmGpeEnReg));\r
         }\r
 \r
-        //
-        // Verify values for proper operation
-        //
-        ASSERT(Fadt->Pm1aCntBlk != 0);
-        ASSERT(Fadt->PmTmrBlk != 0);
-        ASSERT(Fadt->ResetReg.Address != 0);
-        ASSERT(Fadt->Pm1aEvtBlk != 0);
-        ASSERT(Fadt->Gpe0Blk != 0);
-
+        //\r
+        // Verify values for proper operation\r
+        //\r
+        ASSERT(Fadt->Pm1aCntBlk != 0);\r
+        ASSERT(Fadt->PmTmrBlk != 0);\r
+        ASSERT(Fadt->ResetReg.Address != 0);\r
+        ASSERT(Fadt->Pm1aEvtBlk != 0);\r
+        ASSERT(Fadt->Gpe0Blk != 0);\r
+\r
         return RETURN_SUCCESS;\r
       }\r
     }\r
@@ -519,15 +501,15 @@ CbParseFadtInfo (
           *pResetValue = Fadt->ResetValue;\r
         DEBUG ((EFI_D_ERROR, "Reset Value 0x%x\n", Fadt->ResetValue));\r
 \r
-        if (pPmEvtReg != NULL) {
+        if (pPmEvtReg != NULL) {\r
           *pPmEvtReg = Fadt->Pm1aEvtBlk;\r
            DEBUG ((EFI_D_INFO, "PmEvt Reg 0x%x\n", Fadt->Pm1aEvtBlk));\r
         }\r
 \r
-        if (pPmGpeEnReg != NULL) {
+        if (pPmGpeEnReg != NULL) {\r
           *pPmGpeEnReg = Fadt->Gpe0Blk + Fadt->Gpe0BlkLen / 2;\r
           DEBUG ((EFI_D_INFO, "PmGpeEn Reg 0x%x\n", *pPmGpeEnReg));\r
-        }
+        }\r
         return RETURN_SUCCESS;\r
       }\r
     }\r
@@ -541,10 +523,10 @@ CbParseFadtInfo (
 \r
   @param  pRegBase           Pointer to the base address of serial port registers\r
   @param  pRegAccessType     Pointer to the access type of serial port registers\r
-  @param  pRegWidth          Pointer to the register width in bytes
+  @param  pRegWidth          Pointer to the register width in bytes\r
   @param  pBaudrate          Pointer to the serial port baudrate\r
-  @param  pInputHertz        Pointer to the input clock frequency
-  @param  pUartPciAddr       Pointer to the UART PCI bus, dev and func address
+  @param  pInputHertz        Pointer to the input clock frequency\r
+  @param  pUartPciAddr       Pointer to the UART PCI bus, dev and func address\r
 \r
   @retval RETURN_SUCCESS     Successfully find the serial port information.\r
   @retval RETURN_NOT_FOUND   Failed to find the serial port information .\r
@@ -554,10 +536,10 @@ RETURN_STATUS
 CbParseSerialInfo (\r
   OUT UINT32      *pRegBase,\r
   OUT UINT32      *pRegAccessType,\r
-  OUT UINT32      *pRegWidth,
-  OUT UINT32      *pBaudrate,
-  OUT UINT32      *pInputHertz,
-  OUT UINT32      *pUartPciAddr
+  OUT UINT32      *pRegWidth,\r
+  OUT UINT32      *pBaudrate,\r
+  OUT UINT32      *pInputHertz,\r
+  OUT UINT32      *pUartPciAddr\r
   )\r
 {\r
   struct cb_serial    *CbSerial;\r
@@ -575,10 +557,10 @@ CbParseSerialInfo (
     *pRegBase = CbSerial->baseaddr;\r
   }\r
 \r
-  if (pRegWidth != NULL) {
-    *pRegWidth = CbSerial->regwidth;
-  }
-
+  if (pRegWidth != NULL) {\r
+    *pRegWidth = CbSerial->regwidth;\r
+  }\r
+\r
   if (pRegAccessType != NULL) {\r
     *pRegAccessType = CbSerial->type;\r
   }\r
@@ -587,14 +569,14 @@ CbParseSerialInfo (
     *pBaudrate = CbSerial->baud;\r
   }\r
 \r
-  if (pInputHertz != NULL) {
-    *pInputHertz = CbSerial->input_hertz;
-  }
-
-  if (pUartPciAddr != NULL) {
-    *pUartPciAddr = CbSerial->uart_pci_addr;
-  }
-
+  if (pInputHertz != NULL) {\r
+    *pInputHertz = CbSerial->input_hertz;\r
+  }\r
+\r
+  if (pUartPciAddr != NULL) {\r
+    *pUartPciAddr = CbSerial->uart_pci_addr;\r
+  }\r
+\r
   return RETURN_SUCCESS;\r
 }\r
 \r