]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Core/PiSmmCore/MemoryAttributesTable.c
MdeModulePkg: Apply uncrustify changes
[mirror_edk2.git] / MdeModulePkg / Core / PiSmmCore / MemoryAttributesTable.c
index 3e8a80dd7db4a3cc0a22bb9d9a590e7b463d6ead..394fdae507413b7726ea8bd36a13b95370b6e758 100644 (file)
@@ -25,33 +25,33 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
 #define PREVIOUS_MEMORY_DESCRIPTOR(MemoryDescriptor, Size) \\r
   ((EFI_MEMORY_DESCRIPTOR *)((UINT8 *)(MemoryDescriptor) - (Size)))\r
 \r
-#define IMAGE_PROPERTIES_RECORD_CODE_SECTION_SIGNATURE SIGNATURE_32 ('I','P','R','C')\r
+#define IMAGE_PROPERTIES_RECORD_CODE_SECTION_SIGNATURE  SIGNATURE_32 ('I','P','R','C')\r
 \r
 typedef struct {\r
-  UINT32                 Signature;\r
-  LIST_ENTRY             Link;\r
-  EFI_PHYSICAL_ADDRESS   CodeSegmentBase;\r
-  UINT64                 CodeSegmentSize;\r
+  UINT32                  Signature;\r
+  LIST_ENTRY              Link;\r
+  EFI_PHYSICAL_ADDRESS    CodeSegmentBase;\r
+  UINT64                  CodeSegmentSize;\r
 } IMAGE_PROPERTIES_RECORD_CODE_SECTION;\r
 \r
-#define IMAGE_PROPERTIES_RECORD_SIGNATURE SIGNATURE_32 ('I','P','R','D')\r
+#define IMAGE_PROPERTIES_RECORD_SIGNATURE  SIGNATURE_32 ('I','P','R','D')\r
 \r
 typedef struct {\r
-  UINT32                 Signature;\r
-  LIST_ENTRY             Link;\r
-  EFI_PHYSICAL_ADDRESS   ImageBase;\r
-  UINT64                 ImageSize;\r
-  UINTN                  CodeSegmentCount;\r
-  LIST_ENTRY             CodeSegmentList;\r
+  UINT32                  Signature;\r
+  LIST_ENTRY              Link;\r
+  EFI_PHYSICAL_ADDRESS    ImageBase;\r
+  UINT64                  ImageSize;\r
+  UINTN                   CodeSegmentCount;\r
+  LIST_ENTRY              CodeSegmentList;\r
 } IMAGE_PROPERTIES_RECORD;\r
 \r
-#define IMAGE_PROPERTIES_PRIVATE_DATA_SIGNATURE SIGNATURE_32 ('I','P','P','D')\r
+#define IMAGE_PROPERTIES_PRIVATE_DATA_SIGNATURE  SIGNATURE_32 ('I','P','P','D')\r
 \r
 typedef struct {\r
-  UINT32                 Signature;\r
-  UINTN                  ImageRecordCount;\r
-  UINTN                  CodeSegmentCountMax;\r
-  LIST_ENTRY             ImageRecordList;\r
+  UINT32        Signature;\r
+  UINTN         ImageRecordCount;\r
+  UINTN         CodeSegmentCountMax;\r
+  LIST_ENTRY    ImageRecordList;\r
 } IMAGE_PROPERTIES_PRIVATE_DATA;\r
 \r
 IMAGE_PROPERTIES_PRIVATE_DATA  mImagePropertiesPrivateData = {\r
@@ -63,7 +63,7 @@ IMAGE_PROPERTIES_PRIVATE_DATA  mImagePropertiesPrivateData = {
 \r
 #define EFI_MEMORY_ATTRIBUTES_RUNTIME_MEMORY_PROTECTION_NON_EXECUTABLE_PE_DATA  BIT0\r
 \r
-UINT64 mMemoryProtectionAttribute = EFI_MEMORY_ATTRIBUTES_RUNTIME_MEMORY_PROTECTION_NON_EXECUTABLE_PE_DATA;\r
+UINT64  mMemoryProtectionAttribute = EFI_MEMORY_ATTRIBUTES_RUNTIME_MEMORY_PROTECTION_NON_EXECUTABLE_PE_DATA;\r
 \r
 //\r
 // Below functions are for MemoryMap\r
@@ -82,7 +82,7 @@ UINT64 mMemoryProtectionAttribute = EFI_MEMORY_ATTRIBUTES_RUNTIME_MEMORY_PROTECT
 STATIC\r
 UINT64\r
 EfiPagesToSize (\r
-  IN UINT64 Pages\r
+  IN UINT64  Pages\r
   )\r
 {\r
   return LShiftU64 (Pages, EFI_PAGE_SHIFT);\r
@@ -102,13 +102,12 @@ EfiPagesToSize (
 STATIC\r
 UINT64\r
 EfiSizeToPages (\r
-  IN UINT64 Size\r
+  IN UINT64  Size\r
   )\r
 {\r
   return RShiftU64 (Size, EFI_PAGE_SHIFT) + ((((UINTN)Size) & EFI_PAGE_MASK) ? 1 : 0);\r
 }\r
 \r
-\r
 /**\r
   Sort memory map entries based upon PhysicalStart, from low to high.\r
 \r
@@ -125,30 +124,30 @@ SortMemoryMap (
   IN UINTN                      DescriptorSize\r
   )\r
 {\r
-  EFI_MEMORY_DESCRIPTOR       *MemoryMapEntry;\r
-  EFI_MEMORY_DESCRIPTOR       *NextMemoryMapEntry;\r
-  EFI_MEMORY_DESCRIPTOR       *MemoryMapEnd;\r
-  EFI_MEMORY_DESCRIPTOR       TempMemoryMap;\r
+  EFI_MEMORY_DESCRIPTOR  *MemoryMapEntry;\r
+  EFI_MEMORY_DESCRIPTOR  *NextMemoryMapEntry;\r
+  EFI_MEMORY_DESCRIPTOR  *MemoryMapEnd;\r
+  EFI_MEMORY_DESCRIPTOR  TempMemoryMap;\r
 \r
-  MemoryMapEntry = MemoryMap;\r
+  MemoryMapEntry     = MemoryMap;\r
   NextMemoryMapEntry = NEXT_MEMORY_DESCRIPTOR (MemoryMapEntry, DescriptorSize);\r
-  MemoryMapEnd = (EFI_MEMORY_DESCRIPTOR *) ((UINT8 *) MemoryMap + MemoryMapSize);\r
+  MemoryMapEnd       = (EFI_MEMORY_DESCRIPTOR *)((UINT8 *)MemoryMap + MemoryMapSize);\r
   while (MemoryMapEntry < MemoryMapEnd) {\r
     while (NextMemoryMapEntry < MemoryMapEnd) {\r
       if (MemoryMapEntry->PhysicalStart > NextMemoryMapEntry->PhysicalStart) {\r
-        CopyMem (&TempMemoryMap, MemoryMapEntry, sizeof(EFI_MEMORY_DESCRIPTOR));\r
-        CopyMem (MemoryMapEntry, NextMemoryMapEntry, sizeof(EFI_MEMORY_DESCRIPTOR));\r
-        CopyMem (NextMemoryMapEntry, &TempMemoryMap, sizeof(EFI_MEMORY_DESCRIPTOR));\r
+        CopyMem (&TempMemoryMap, MemoryMapEntry, sizeof (EFI_MEMORY_DESCRIPTOR));\r
+        CopyMem (MemoryMapEntry, NextMemoryMapEntry, sizeof (EFI_MEMORY_DESCRIPTOR));\r
+        CopyMem (NextMemoryMapEntry, &TempMemoryMap, sizeof (EFI_MEMORY_DESCRIPTOR));\r
       }\r
 \r
       NextMemoryMapEntry = NEXT_MEMORY_DESCRIPTOR (NextMemoryMapEntry, DescriptorSize);\r
     }\r
 \r
-    MemoryMapEntry      = NEXT_MEMORY_DESCRIPTOR (MemoryMapEntry, DescriptorSize);\r
-    NextMemoryMapEntry  = NEXT_MEMORY_DESCRIPTOR (MemoryMapEntry, DescriptorSize);\r
+    MemoryMapEntry     = NEXT_MEMORY_DESCRIPTOR (MemoryMapEntry, DescriptorSize);\r
+    NextMemoryMapEntry = NEXT_MEMORY_DESCRIPTOR (MemoryMapEntry, DescriptorSize);\r
   }\r
 \r
-  return ;\r
+  return;\r
 }\r
 \r
 /**\r
@@ -170,25 +169,26 @@ MergeMemoryMap (
   IN UINTN                      DescriptorSize\r
   )\r
 {\r
-  EFI_MEMORY_DESCRIPTOR       *MemoryMapEntry;\r
-  EFI_MEMORY_DESCRIPTOR       *MemoryMapEnd;\r
-  UINT64                      MemoryBlockLength;\r
-  EFI_MEMORY_DESCRIPTOR       *NewMemoryMapEntry;\r
-  EFI_MEMORY_DESCRIPTOR       *NextMemoryMapEntry;\r
+  EFI_MEMORY_DESCRIPTOR  *MemoryMapEntry;\r
+  EFI_MEMORY_DESCRIPTOR  *MemoryMapEnd;\r
+  UINT64                 MemoryBlockLength;\r
+  EFI_MEMORY_DESCRIPTOR  *NewMemoryMapEntry;\r
+  EFI_MEMORY_DESCRIPTOR  *NextMemoryMapEntry;\r
 \r
-  MemoryMapEntry = MemoryMap;\r
+  MemoryMapEntry    = MemoryMap;\r
   NewMemoryMapEntry = MemoryMap;\r
-  MemoryMapEnd = (EFI_MEMORY_DESCRIPTOR *) ((UINT8 *) MemoryMap + *MemoryMapSize);\r
+  MemoryMapEnd      = (EFI_MEMORY_DESCRIPTOR *)((UINT8 *)MemoryMap + *MemoryMapSize);\r
   while ((UINTN)MemoryMapEntry < (UINTN)MemoryMapEnd) {\r
-    CopyMem (NewMemoryMapEntry, MemoryMapEntry, sizeof(EFI_MEMORY_DESCRIPTOR));\r
+    CopyMem (NewMemoryMapEntry, MemoryMapEntry, sizeof (EFI_MEMORY_DESCRIPTOR));\r
     NextMemoryMapEntry = NEXT_MEMORY_DESCRIPTOR (MemoryMapEntry, DescriptorSize);\r
 \r
     do {\r
-      MemoryBlockLength = (UINT64) (EfiPagesToSize (MemoryMapEntry->NumberOfPages));\r
+      MemoryBlockLength = (UINT64)(EfiPagesToSize (MemoryMapEntry->NumberOfPages));\r
       if (((UINTN)NextMemoryMapEntry < (UINTN)MemoryMapEnd) &&\r
           (MemoryMapEntry->Type == NextMemoryMapEntry->Type) &&\r
           (MemoryMapEntry->Attribute == NextMemoryMapEntry->Attribute) &&\r
-          ((MemoryMapEntry->PhysicalStart + MemoryBlockLength) == NextMemoryMapEntry->PhysicalStart)) {\r
+          ((MemoryMapEntry->PhysicalStart + MemoryBlockLength) == NextMemoryMapEntry->PhysicalStart))\r
+      {\r
         MemoryMapEntry->NumberOfPages += NextMemoryMapEntry->NumberOfPages;\r
         if (NewMemoryMapEntry != MemoryMapEntry) {\r
           NewMemoryMapEntry->NumberOfPages += NextMemoryMapEntry->NumberOfPages;\r
@@ -202,13 +202,13 @@ MergeMemoryMap (
       }\r
     } while (TRUE);\r
 \r
-    MemoryMapEntry = NEXT_MEMORY_DESCRIPTOR (MemoryMapEntry, DescriptorSize);\r
+    MemoryMapEntry    = NEXT_MEMORY_DESCRIPTOR (MemoryMapEntry, DescriptorSize);\r
     NewMemoryMapEntry = NEXT_MEMORY_DESCRIPTOR (NewMemoryMapEntry, DescriptorSize);\r
   }\r
 \r
   *MemoryMapSize = (UINTN)NewMemoryMapEntry - (UINTN)MemoryMap;\r
 \r
-  return ;\r
+  return;\r
 }\r
 \r
 /**\r
@@ -228,29 +228,30 @@ EnforceMemoryMapAttribute (
   IN UINTN                      DescriptorSize\r
   )\r
 {\r
-  EFI_MEMORY_DESCRIPTOR       *MemoryMapEntry;\r
-  EFI_MEMORY_DESCRIPTOR       *MemoryMapEnd;\r
+  EFI_MEMORY_DESCRIPTOR  *MemoryMapEntry;\r
+  EFI_MEMORY_DESCRIPTOR  *MemoryMapEnd;\r
 \r
   MemoryMapEntry = MemoryMap;\r
-  MemoryMapEnd   = (EFI_MEMORY_DESCRIPTOR *) ((UINT8 *) MemoryMap + MemoryMapSize);\r
+  MemoryMapEnd   = (EFI_MEMORY_DESCRIPTOR *)((UINT8 *)MemoryMap + MemoryMapSize);\r
   while ((UINTN)MemoryMapEntry < (UINTN)MemoryMapEnd) {\r
     if (MemoryMapEntry->Attribute != 0) {\r
       // It is PE image, the attribute is already set.\r
     } else {\r
       switch (MemoryMapEntry->Type) {\r
-      case EfiRuntimeServicesCode:\r
-        MemoryMapEntry->Attribute = EFI_MEMORY_RO;\r
-        break;\r
-      case EfiRuntimeServicesData:\r
-      default:\r
-        MemoryMapEntry->Attribute |= EFI_MEMORY_XP;\r
-        break;\r
+        case EfiRuntimeServicesCode:\r
+          MemoryMapEntry->Attribute = EFI_MEMORY_RO;\r
+          break;\r
+        case EfiRuntimeServicesData:\r
+        default:\r
+          MemoryMapEntry->Attribute |= EFI_MEMORY_XP;\r
+          break;\r
       }\r
     }\r
+\r
     MemoryMapEntry = NEXT_MEMORY_DESCRIPTOR (MemoryMapEntry, DescriptorSize);\r
   }\r
 \r
-  return ;\r
+  return;\r
 }\r
 \r
 /**\r
@@ -268,15 +269,16 @@ GetImageRecordByAddress (
   IN UINT64                Length\r
   )\r
 {\r
-  IMAGE_PROPERTIES_RECORD    *ImageRecord;\r
-  LIST_ENTRY                 *ImageRecordLink;\r
-  LIST_ENTRY                 *ImageRecordList;\r
+  IMAGE_PROPERTIES_RECORD  *ImageRecord;\r
+  LIST_ENTRY               *ImageRecordLink;\r
+  LIST_ENTRY               *ImageRecordList;\r
 \r
   ImageRecordList = &mImagePropertiesPrivateData.ImageRecordList;\r
 \r
   for (ImageRecordLink = ImageRecordList->ForwardLink;\r
        ImageRecordLink != ImageRecordList;\r
-       ImageRecordLink = ImageRecordLink->ForwardLink) {\r
+       ImageRecordLink = ImageRecordLink->ForwardLink)\r
+  {\r
     ImageRecord = CR (\r
                     ImageRecordLink,\r
                     IMAGE_PROPERTIES_RECORD,\r
@@ -285,7 +287,8 @@ GetImageRecordByAddress (
                     );\r
 \r
     if ((Buffer <= ImageRecord->ImageBase) &&\r
-        (Buffer + Length >= ImageRecord->ImageBase + ImageRecord->ImageSize)) {\r
+        (Buffer + Length >= ImageRecord->ImageBase + ImageRecord->ImageSize))\r
+    {\r
       return ImageRecord;\r
     }\r
   }\r
@@ -309,43 +312,43 @@ GetImageRecordByAddress (
 STATIC\r
 UINTN\r
 SetNewRecord (\r
-  IN IMAGE_PROPERTIES_RECORD       *ImageRecord,\r
-  IN OUT EFI_MEMORY_DESCRIPTOR     *NewRecord,\r
-  IN EFI_MEMORY_DESCRIPTOR         *OldRecord,\r
-  IN UINTN                         DescriptorSize\r
+  IN IMAGE_PROPERTIES_RECORD    *ImageRecord,\r
+  IN OUT EFI_MEMORY_DESCRIPTOR  *NewRecord,\r
+  IN EFI_MEMORY_DESCRIPTOR      *OldRecord,\r
+  IN UINTN                      DescriptorSize\r
   )\r
 {\r
-  EFI_MEMORY_DESCRIPTOR                     TempRecord;\r
-  IMAGE_PROPERTIES_RECORD_CODE_SECTION      *ImageRecordCodeSection;\r
-  LIST_ENTRY                                *ImageRecordCodeSectionLink;\r
-  LIST_ENTRY                                *ImageRecordCodeSectionEndLink;\r
-  LIST_ENTRY                                *ImageRecordCodeSectionList;\r
-  UINTN                                     NewRecordCount;\r
-  UINT64                                    PhysicalEnd;\r
-  UINT64                                    ImageEnd;\r
-\r
-  CopyMem (&TempRecord, OldRecord, sizeof(EFI_MEMORY_DESCRIPTOR));\r
-  PhysicalEnd = TempRecord.PhysicalStart + EfiPagesToSize(TempRecord.NumberOfPages);\r
+  EFI_MEMORY_DESCRIPTOR                 TempRecord;\r
+  IMAGE_PROPERTIES_RECORD_CODE_SECTION  *ImageRecordCodeSection;\r
+  LIST_ENTRY                            *ImageRecordCodeSectionLink;\r
+  LIST_ENTRY                            *ImageRecordCodeSectionEndLink;\r
+  LIST_ENTRY                            *ImageRecordCodeSectionList;\r
+  UINTN                                 NewRecordCount;\r
+  UINT64                                PhysicalEnd;\r
+  UINT64                                ImageEnd;\r
+\r
+  CopyMem (&TempRecord, OldRecord, sizeof (EFI_MEMORY_DESCRIPTOR));\r
+  PhysicalEnd    = TempRecord.PhysicalStart + EfiPagesToSize (TempRecord.NumberOfPages);\r
   NewRecordCount = 0;\r
 \r
   //\r
   // Always create a new entry for non-PE image record\r
   //\r
   if (ImageRecord->ImageBase > TempRecord.PhysicalStart) {\r
-    NewRecord->Type = TempRecord.Type;\r
+    NewRecord->Type          = TempRecord.Type;\r
     NewRecord->PhysicalStart = TempRecord.PhysicalStart;\r
     NewRecord->VirtualStart  = 0;\r
-    NewRecord->NumberOfPages = EfiSizeToPages(ImageRecord->ImageBase - TempRecord.PhysicalStart);\r
+    NewRecord->NumberOfPages = EfiSizeToPages (ImageRecord->ImageBase - TempRecord.PhysicalStart);\r
     NewRecord->Attribute     = TempRecord.Attribute;\r
-    NewRecord = NEXT_MEMORY_DESCRIPTOR (NewRecord, DescriptorSize);\r
-    NewRecordCount ++;\r
+    NewRecord                = NEXT_MEMORY_DESCRIPTOR (NewRecord, DescriptorSize);\r
+    NewRecordCount++;\r
     TempRecord.PhysicalStart = ImageRecord->ImageBase;\r
-    TempRecord.NumberOfPages = EfiSizeToPages(PhysicalEnd - TempRecord.PhysicalStart);\r
+    TempRecord.NumberOfPages = EfiSizeToPages (PhysicalEnd - TempRecord.PhysicalStart);\r
   }\r
 \r
   ImageRecordCodeSectionList = &ImageRecord->CodeSegmentList;\r
 \r
-  ImageRecordCodeSectionLink = ImageRecordCodeSectionList->ForwardLink;\r
+  ImageRecordCodeSectionLink    = ImageRecordCodeSectionList->ForwardLink;\r
   ImageRecordCodeSectionEndLink = ImageRecordCodeSectionList;\r
   while (ImageRecordCodeSectionLink != ImageRecordCodeSectionEndLink) {\r
     ImageRecordCodeSection = CR (\r
@@ -360,31 +363,31 @@ SetNewRecord (
       //\r
       // DATA\r
       //\r
-      NewRecord->Type = EfiRuntimeServicesData;\r
+      NewRecord->Type          = EfiRuntimeServicesData;\r
       NewRecord->PhysicalStart = TempRecord.PhysicalStart;\r
       NewRecord->VirtualStart  = 0;\r
-      NewRecord->NumberOfPages = EfiSizeToPages(ImageRecordCodeSection->CodeSegmentBase - NewRecord->PhysicalStart);\r
+      NewRecord->NumberOfPages = EfiSizeToPages (ImageRecordCodeSection->CodeSegmentBase - NewRecord->PhysicalStart);\r
       NewRecord->Attribute     = TempRecord.Attribute | EFI_MEMORY_XP;\r
       if (NewRecord->NumberOfPages != 0) {\r
         NewRecord = NEXT_MEMORY_DESCRIPTOR (NewRecord, DescriptorSize);\r
-        NewRecordCount ++;\r
+        NewRecordCount++;\r
       }\r
 \r
       //\r
       // CODE\r
       //\r
-      NewRecord->Type = EfiRuntimeServicesCode;\r
+      NewRecord->Type          = EfiRuntimeServicesCode;\r
       NewRecord->PhysicalStart = ImageRecordCodeSection->CodeSegmentBase;\r
       NewRecord->VirtualStart  = 0;\r
-      NewRecord->NumberOfPages = EfiSizeToPages(ImageRecordCodeSection->CodeSegmentSize);\r
+      NewRecord->NumberOfPages = EfiSizeToPages (ImageRecordCodeSection->CodeSegmentSize);\r
       NewRecord->Attribute     = (TempRecord.Attribute & (~EFI_MEMORY_XP)) | EFI_MEMORY_RO;\r
       if (NewRecord->NumberOfPages != 0) {\r
         NewRecord = NEXT_MEMORY_DESCRIPTOR (NewRecord, DescriptorSize);\r
-        NewRecordCount ++;\r
+        NewRecordCount++;\r
       }\r
 \r
-      TempRecord.PhysicalStart = ImageRecordCodeSection->CodeSegmentBase + EfiPagesToSize (EfiSizeToPages(ImageRecordCodeSection->CodeSegmentSize));\r
-      TempRecord.NumberOfPages = EfiSizeToPages(PhysicalEnd - TempRecord.PhysicalStart);\r
+      TempRecord.PhysicalStart = ImageRecordCodeSection->CodeSegmentBase + EfiPagesToSize (EfiSizeToPages (ImageRecordCodeSection->CodeSegmentSize));\r
+      TempRecord.NumberOfPages = EfiSizeToPages (PhysicalEnd - TempRecord.PhysicalStart);\r
       if (TempRecord.NumberOfPages == 0) {\r
         break;\r
       }\r
@@ -397,12 +400,12 @@ SetNewRecord (
   // Final DATA\r
   //\r
   if (TempRecord.PhysicalStart < ImageEnd) {\r
-    NewRecord->Type = EfiRuntimeServicesData;\r
+    NewRecord->Type          = EfiRuntimeServicesData;\r
     NewRecord->PhysicalStart = TempRecord.PhysicalStart;\r
     NewRecord->VirtualStart  = 0;\r
     NewRecord->NumberOfPages = EfiSizeToPages (ImageEnd - TempRecord.PhysicalStart);\r
     NewRecord->Attribute     = TempRecord.Attribute | EFI_MEMORY_XP;\r
-    NewRecordCount ++;\r
+    NewRecordCount++;\r
   }\r
 \r
   return NewRecordCount;\r
@@ -420,25 +423,26 @@ SetNewRecord (
 STATIC\r
 UINTN\r
 GetMaxSplitRecordCount (\r
-  IN EFI_MEMORY_DESCRIPTOR *OldRecord\r
+  IN EFI_MEMORY_DESCRIPTOR  *OldRecord\r
   )\r
 {\r
-  IMAGE_PROPERTIES_RECORD *ImageRecord;\r
-  UINTN                   SplitRecordCount;\r
-  UINT64                  PhysicalStart;\r
-  UINT64                  PhysicalEnd;\r
+  IMAGE_PROPERTIES_RECORD  *ImageRecord;\r
+  UINTN                    SplitRecordCount;\r
+  UINT64                   PhysicalStart;\r
+  UINT64                   PhysicalEnd;\r
 \r
   SplitRecordCount = 0;\r
-  PhysicalStart = OldRecord->PhysicalStart;\r
-  PhysicalEnd = OldRecord->PhysicalStart + EfiPagesToSize(OldRecord->NumberOfPages);\r
+  PhysicalStart    = OldRecord->PhysicalStart;\r
+  PhysicalEnd      = OldRecord->PhysicalStart + EfiPagesToSize (OldRecord->NumberOfPages);\r
 \r
   do {\r
     ImageRecord = GetImageRecordByAddress (PhysicalStart, PhysicalEnd - PhysicalStart);\r
     if (ImageRecord == NULL) {\r
       break;\r
     }\r
+\r
     SplitRecordCount += (2 * ImageRecord->CodeSegmentCount + 2);\r
-    PhysicalStart = ImageRecord->ImageBase + ImageRecord->ImageSize;\r
+    PhysicalStart     = ImageRecord->ImageBase + ImageRecord->ImageSize;\r
   } while ((ImageRecord != NULL) && (PhysicalStart < PhysicalEnd));\r
 \r
   return SplitRecordCount;\r
@@ -462,19 +466,19 @@ GetMaxSplitRecordCount (
 STATIC\r
 UINTN\r
 SplitRecord (\r
-  IN EFI_MEMORY_DESCRIPTOR     *OldRecord,\r
-  IN OUT EFI_MEMORY_DESCRIPTOR *NewRecord,\r
-  IN UINTN                     MaxSplitRecordCount,\r
-  IN UINTN                     DescriptorSize\r
+  IN EFI_MEMORY_DESCRIPTOR      *OldRecord,\r
+  IN OUT EFI_MEMORY_DESCRIPTOR  *NewRecord,\r
+  IN UINTN                      MaxSplitRecordCount,\r
+  IN UINTN                      DescriptorSize\r
   )\r
 {\r
-  EFI_MEMORY_DESCRIPTOR   TempRecord;\r
-  IMAGE_PROPERTIES_RECORD *ImageRecord;\r
-  IMAGE_PROPERTIES_RECORD *NewImageRecord;\r
-  UINT64                  PhysicalStart;\r
-  UINT64                  PhysicalEnd;\r
-  UINTN                   NewRecordCount;\r
-  UINTN                   TotalNewRecordCount;\r
+  EFI_MEMORY_DESCRIPTOR    TempRecord;\r
+  IMAGE_PROPERTIES_RECORD  *ImageRecord;\r
+  IMAGE_PROPERTIES_RECORD  *NewImageRecord;\r
+  UINT64                   PhysicalStart;\r
+  UINT64                   PhysicalEnd;\r
+  UINTN                    NewRecordCount;\r
+  UINTN                    TotalNewRecordCount;\r
 \r
   if (MaxSplitRecordCount == 0) {\r
     CopyMem (NewRecord, OldRecord, DescriptorSize);\r
@@ -486,9 +490,9 @@ SplitRecord (
   //\r
   // Override previous record\r
   //\r
-  CopyMem (&TempRecord, OldRecord, sizeof(EFI_MEMORY_DESCRIPTOR));\r
+  CopyMem (&TempRecord, OldRecord, sizeof (EFI_MEMORY_DESCRIPTOR));\r
   PhysicalStart = TempRecord.PhysicalStart;\r
-  PhysicalEnd = TempRecord.PhysicalStart + EfiPagesToSize(TempRecord.NumberOfPages);\r
+  PhysicalEnd   = TempRecord.PhysicalStart + EfiPagesToSize (TempRecord.NumberOfPages);\r
 \r
   ImageRecord = NULL;\r
   do {\r
@@ -501,28 +505,30 @@ SplitRecord (
         //\r
         // Always create a new entry for non-PE image record\r
         //\r
-        NewRecord->Type = TempRecord.Type;\r
+        NewRecord->Type          = TempRecord.Type;\r
         NewRecord->PhysicalStart = TempRecord.PhysicalStart;\r
         NewRecord->VirtualStart  = 0;\r
         NewRecord->NumberOfPages = TempRecord.NumberOfPages;\r
         NewRecord->Attribute     = TempRecord.Attribute;\r
-        TotalNewRecordCount ++;\r
+        TotalNewRecordCount++;\r
       }\r
+\r
       break;\r
     }\r
+\r
     ImageRecord = NewImageRecord;\r
 \r
     //\r
     // Set new record\r
     //\r
-    NewRecordCount = SetNewRecord (ImageRecord, NewRecord, &TempRecord, DescriptorSize);\r
+    NewRecordCount       = SetNewRecord (ImageRecord, NewRecord, &TempRecord, DescriptorSize);\r
     TotalNewRecordCount += NewRecordCount;\r
-    NewRecord = (EFI_MEMORY_DESCRIPTOR *)((UINT8 *)NewRecord + NewRecordCount * DescriptorSize);\r
+    NewRecord            = (EFI_MEMORY_DESCRIPTOR *)((UINT8 *)NewRecord + NewRecordCount * DescriptorSize);\r
 \r
     //\r
     // Update PhysicalStart, in order to exclude the image buffer already splitted.\r
     //\r
-    PhysicalStart = ImageRecord->ImageBase + ImageRecord->ImageSize;\r
+    PhysicalStart            = ImageRecord->ImageBase + ImageRecord->ImageSize;\r
     TempRecord.PhysicalStart = PhysicalStart;\r
     TempRecord.NumberOfPages = EfiSizeToPages (PhysicalEnd - PhysicalStart);\r
   } while ((ImageRecord != NULL) && (PhysicalStart < PhysicalEnd));\r
@@ -590,12 +596,12 @@ SplitTable (
   IN UINTN                      DescriptorSize\r
   )\r
 {\r
-  INTN        IndexOld;\r
-  INTN        IndexNew;\r
-  UINTN       MaxSplitRecordCount;\r
-  UINTN       RealSplitRecordCount;\r
-  UINTN       TotalSplitRecordCount;\r
-  UINTN       AdditionalRecordCount;\r
+  INTN   IndexOld;\r
+  INTN   IndexNew;\r
+  UINTN  MaxSplitRecordCount;\r
+  UINTN  RealSplitRecordCount;\r
+  UINTN  TotalSplitRecordCount;\r
+  UINTN  AdditionalRecordCount;\r
 \r
   AdditionalRecordCount = (2 * mImagePropertiesPrivateData.CodeSegmentCountMax + 2) * mImagePropertiesPrivateData.ImageRecordCount;\r
 \r
@@ -608,12 +614,12 @@ SplitTable (
   // Let new record point to end of full MemoryMap buffer.\r
   //\r
   IndexNew = ((*MemoryMapSize) / DescriptorSize) - 1 + AdditionalRecordCount;\r
-  for (; IndexOld >= 0; IndexOld--) {\r
+  for ( ; IndexOld >= 0; IndexOld--) {\r
     MaxSplitRecordCount = GetMaxSplitRecordCount ((EFI_MEMORY_DESCRIPTOR *)((UINT8 *)MemoryMap + IndexOld * DescriptorSize));\r
     //\r
     // Split this MemoryMap record\r
     //\r
-    IndexNew -= MaxSplitRecordCount;\r
+    IndexNew            -= MaxSplitRecordCount;\r
     RealSplitRecordCount = SplitRecord (\r
                              (EFI_MEMORY_DESCRIPTOR *)((UINT8 *)MemoryMap + IndexOld * DescriptorSize),\r
                              (EFI_MEMORY_DESCRIPTOR *)((UINT8 *)MemoryMap + IndexNew * DescriptorSize),\r
@@ -630,10 +636,12 @@ SplitTable (
         (RealSplitRecordCount + 1) * DescriptorSize\r
         );\r
     }\r
-    IndexNew = IndexNew + MaxSplitRecordCount - RealSplitRecordCount;\r
+\r
+    IndexNew               = IndexNew + MaxSplitRecordCount - RealSplitRecordCount;\r
     TotalSplitRecordCount += RealSplitRecordCount;\r
-    IndexNew --;\r
+    IndexNew--;\r
   }\r
+\r
   //\r
   // Move all records to the beginning.\r
   //\r
@@ -660,7 +668,7 @@ SplitTable (
   //\r
   MergeMemoryMap (MemoryMap, MemoryMapSize, DescriptorSize);\r
 \r
-  return ;\r
+  return;\r
 }\r
 \r
 /**\r
@@ -724,7 +732,7 @@ SmmCoreGetMemoryMapMemoryAttributesTable (
   AdditionalRecordCount = (2 * mImagePropertiesPrivateData.CodeSegmentCountMax + 2) * mImagePropertiesPrivateData.ImageRecordCount;\r
 \r
   OldMemoryMapSize = *MemoryMapSize;\r
-  Status = SmmCoreGetMemoryMap (MemoryMapSize, MemoryMap, MapKey, DescriptorSize, DescriptorVersion);\r
+  Status           = SmmCoreGetMemoryMap (MemoryMapSize, MemoryMap, MapKey, DescriptorSize, DescriptorVersion);\r
   if (Status == EFI_BUFFER_TOO_SMALL) {\r
     *MemoryMapSize = *MemoryMapSize + (*DescriptorSize) * AdditionalRecordCount;\r
   } else if (Status == EFI_SUCCESS) {\r
@@ -738,7 +746,7 @@ SmmCoreGetMemoryMapMemoryAttributesTable (
       //\r
       // Split PE code/data\r
       //\r
-      ASSERT(MemoryMap != NULL);\r
+      ASSERT (MemoryMap != NULL);\r
       SplitTable (MemoryMapSize, MemoryMap, *DescriptorSize);\r
     }\r
   }\r
@@ -762,7 +770,8 @@ SetMemoryAttributesTableSectionAlignment (
   )\r
 {\r
   if (((SectionAlignment & (RUNTIME_PAGE_ALLOCATION_GRANULARITY - 1)) != 0) &&\r
-      ((mMemoryProtectionAttribute & EFI_MEMORY_ATTRIBUTES_RUNTIME_MEMORY_PROTECTION_NON_EXECUTABLE_PE_DATA) != 0)) {\r
+      ((mMemoryProtectionAttribute & EFI_MEMORY_ATTRIBUTES_RUNTIME_MEMORY_PROTECTION_NON_EXECUTABLE_PE_DATA) != 0))\r
+  {\r
     DEBUG ((DEBUG_VERBOSE, "SMM SetMemoryAttributesTableSectionAlignment - Clear\n"));\r
     mMemoryProtectionAttribute &= ~((UINT64)EFI_MEMORY_ATTRIBUTES_RUNTIME_MEMORY_PROTECTION_NON_EXECUTABLE_PE_DATA);\r
   }\r
@@ -777,11 +786,11 @@ SetMemoryAttributesTableSectionAlignment (
 STATIC\r
 VOID\r
 SwapImageRecordCodeSection (\r
-  IN IMAGE_PROPERTIES_RECORD_CODE_SECTION      *FirstImageRecordCodeSection,\r
-  IN IMAGE_PROPERTIES_RECORD_CODE_SECTION      *SecondImageRecordCodeSection\r
+  IN IMAGE_PROPERTIES_RECORD_CODE_SECTION  *FirstImageRecordCodeSection,\r
+  IN IMAGE_PROPERTIES_RECORD_CODE_SECTION  *SecondImageRecordCodeSection\r
   )\r
 {\r
-  IMAGE_PROPERTIES_RECORD_CODE_SECTION      TempImageRecordCodeSection;\r
+  IMAGE_PROPERTIES_RECORD_CODE_SECTION  TempImageRecordCodeSection;\r
 \r
   TempImageRecordCodeSection.CodeSegmentBase = FirstImageRecordCodeSection->CodeSegmentBase;\r
   TempImageRecordCodeSection.CodeSegmentSize = FirstImageRecordCodeSection->CodeSegmentSize;\r
@@ -801,21 +810,21 @@ SwapImageRecordCodeSection (
 STATIC\r
 VOID\r
 SortImageRecordCodeSection (\r
-  IN IMAGE_PROPERTIES_RECORD              *ImageRecord\r
+  IN IMAGE_PROPERTIES_RECORD  *ImageRecord\r
   )\r
 {\r
-  IMAGE_PROPERTIES_RECORD_CODE_SECTION      *ImageRecordCodeSection;\r
-  IMAGE_PROPERTIES_RECORD_CODE_SECTION      *NextImageRecordCodeSection;\r
-  LIST_ENTRY                                *ImageRecordCodeSectionLink;\r
-  LIST_ENTRY                                *NextImageRecordCodeSectionLink;\r
-  LIST_ENTRY                                *ImageRecordCodeSectionEndLink;\r
-  LIST_ENTRY                                *ImageRecordCodeSectionList;\r
+  IMAGE_PROPERTIES_RECORD_CODE_SECTION  *ImageRecordCodeSection;\r
+  IMAGE_PROPERTIES_RECORD_CODE_SECTION  *NextImageRecordCodeSection;\r
+  LIST_ENTRY                            *ImageRecordCodeSectionLink;\r
+  LIST_ENTRY                            *NextImageRecordCodeSectionLink;\r
+  LIST_ENTRY                            *ImageRecordCodeSectionEndLink;\r
+  LIST_ENTRY                            *ImageRecordCodeSectionList;\r
 \r
   ImageRecordCodeSectionList = &ImageRecord->CodeSegmentList;\r
 \r
-  ImageRecordCodeSectionLink = ImageRecordCodeSectionList->ForwardLink;\r
+  ImageRecordCodeSectionLink     = ImageRecordCodeSectionList->ForwardLink;\r
   NextImageRecordCodeSectionLink = ImageRecordCodeSectionLink->ForwardLink;\r
-  ImageRecordCodeSectionEndLink = ImageRecordCodeSectionList;\r
+  ImageRecordCodeSectionEndLink  = ImageRecordCodeSectionList;\r
   while (ImageRecordCodeSectionLink != ImageRecordCodeSectionEndLink) {\r
     ImageRecordCodeSection = CR (\r
                                ImageRecordCodeSectionLink,\r
@@ -833,10 +842,11 @@ SortImageRecordCodeSection (
       if (ImageRecordCodeSection->CodeSegmentBase > NextImageRecordCodeSection->CodeSegmentBase) {\r
         SwapImageRecordCodeSection (ImageRecordCodeSection, NextImageRecordCodeSection);\r
       }\r
+\r
       NextImageRecordCodeSectionLink = NextImageRecordCodeSectionLink->ForwardLink;\r
     }\r
 \r
-    ImageRecordCodeSectionLink = ImageRecordCodeSectionLink->ForwardLink;\r
+    ImageRecordCodeSectionLink     = ImageRecordCodeSectionLink->ForwardLink;\r
     NextImageRecordCodeSectionLink = ImageRecordCodeSectionLink->ForwardLink;\r
   }\r
 }\r
@@ -852,22 +862,22 @@ SortImageRecordCodeSection (
 STATIC\r
 BOOLEAN\r
 IsImageRecordCodeSectionValid (\r
-  IN IMAGE_PROPERTIES_RECORD              *ImageRecord\r
+  IN IMAGE_PROPERTIES_RECORD  *ImageRecord\r
   )\r
 {\r
-  IMAGE_PROPERTIES_RECORD_CODE_SECTION      *ImageRecordCodeSection;\r
-  IMAGE_PROPERTIES_RECORD_CODE_SECTION      *LastImageRecordCodeSection;\r
-  LIST_ENTRY                                *ImageRecordCodeSectionLink;\r
-  LIST_ENTRY                                *ImageRecordCodeSectionEndLink;\r
-  LIST_ENTRY                                *ImageRecordCodeSectionList;\r
+  IMAGE_PROPERTIES_RECORD_CODE_SECTION  *ImageRecordCodeSection;\r
+  IMAGE_PROPERTIES_RECORD_CODE_SECTION  *LastImageRecordCodeSection;\r
+  LIST_ENTRY                            *ImageRecordCodeSectionLink;\r
+  LIST_ENTRY                            *ImageRecordCodeSectionEndLink;\r
+  LIST_ENTRY                            *ImageRecordCodeSectionList;\r
 \r
   DEBUG ((DEBUG_VERBOSE, "SMM ImageCode SegmentCount - 0x%x\n", ImageRecord->CodeSegmentCount));\r
 \r
   ImageRecordCodeSectionList = &ImageRecord->CodeSegmentList;\r
 \r
-  ImageRecordCodeSectionLink = ImageRecordCodeSectionList->ForwardLink;\r
+  ImageRecordCodeSectionLink    = ImageRecordCodeSectionList->ForwardLink;\r
   ImageRecordCodeSectionEndLink = ImageRecordCodeSectionList;\r
-  LastImageRecordCodeSection = NULL;\r
+  LastImageRecordCodeSection    = NULL;\r
   while (ImageRecordCodeSectionLink != ImageRecordCodeSectionEndLink) {\r
     ImageRecordCodeSection = CR (\r
                                ImageRecordCodeSectionLink,\r
@@ -878,15 +888,19 @@ IsImageRecordCodeSectionValid (
     if (ImageRecordCodeSection->CodeSegmentSize == 0) {\r
       return FALSE;\r
     }\r
+\r
     if (ImageRecordCodeSection->CodeSegmentBase < ImageRecord->ImageBase) {\r
       return FALSE;\r
     }\r
+\r
     if (ImageRecordCodeSection->CodeSegmentBase >= MAX_ADDRESS - ImageRecordCodeSection->CodeSegmentSize) {\r
       return FALSE;\r
     }\r
+\r
     if ((ImageRecordCodeSection->CodeSegmentBase + ImageRecordCodeSection->CodeSegmentSize) > (ImageRecord->ImageBase + ImageRecord->ImageSize)) {\r
       return FALSE;\r
     }\r
+\r
     if (LastImageRecordCodeSection != NULL) {\r
       if ((LastImageRecordCodeSection->CodeSegmentBase + LastImageRecordCodeSection->CodeSegmentSize) > ImageRecordCodeSection->CodeSegmentBase) {\r
         return FALSE;\r
@@ -909,22 +923,22 @@ IsImageRecordCodeSectionValid (
 STATIC\r
 VOID\r
 SwapImageRecord (\r
-  IN IMAGE_PROPERTIES_RECORD      *FirstImageRecord,\r
-  IN IMAGE_PROPERTIES_RECORD      *SecondImageRecord\r
+  IN IMAGE_PROPERTIES_RECORD  *FirstImageRecord,\r
+  IN IMAGE_PROPERTIES_RECORD  *SecondImageRecord\r
   )\r
 {\r
-  IMAGE_PROPERTIES_RECORD      TempImageRecord;\r
+  IMAGE_PROPERTIES_RECORD  TempImageRecord;\r
 \r
-  TempImageRecord.ImageBase = FirstImageRecord->ImageBase;\r
-  TempImageRecord.ImageSize = FirstImageRecord->ImageSize;\r
+  TempImageRecord.ImageBase        = FirstImageRecord->ImageBase;\r
+  TempImageRecord.ImageSize        = FirstImageRecord->ImageSize;\r
   TempImageRecord.CodeSegmentCount = FirstImageRecord->CodeSegmentCount;\r
 \r
-  FirstImageRecord->ImageBase = SecondImageRecord->ImageBase;\r
-  FirstImageRecord->ImageSize = SecondImageRecord->ImageSize;\r
+  FirstImageRecord->ImageBase        = SecondImageRecord->ImageBase;\r
+  FirstImageRecord->ImageSize        = SecondImageRecord->ImageSize;\r
   FirstImageRecord->CodeSegmentCount = SecondImageRecord->CodeSegmentCount;\r
 \r
-  SecondImageRecord->ImageBase = TempImageRecord.ImageBase;\r
-  SecondImageRecord->ImageSize = TempImageRecord.ImageSize;\r
+  SecondImageRecord->ImageBase        = TempImageRecord.ImageBase;\r
+  SecondImageRecord->ImageSize        = TempImageRecord.ImageSize;\r
   SecondImageRecord->CodeSegmentCount = TempImageRecord.CodeSegmentCount;\r
 \r
   SwapListEntries (&FirstImageRecord->CodeSegmentList, &SecondImageRecord->CodeSegmentList);\r
@@ -939,18 +953,18 @@ SortImageRecord (
   VOID\r
   )\r
 {\r
-  IMAGE_PROPERTIES_RECORD      *ImageRecord;\r
-  IMAGE_PROPERTIES_RECORD      *NextImageRecord;\r
-  LIST_ENTRY                   *ImageRecordLink;\r
-  LIST_ENTRY                   *NextImageRecordLink;\r
-  LIST_ENTRY                   *ImageRecordEndLink;\r
-  LIST_ENTRY                   *ImageRecordList;\r
+  IMAGE_PROPERTIES_RECORD  *ImageRecord;\r
+  IMAGE_PROPERTIES_RECORD  *NextImageRecord;\r
+  LIST_ENTRY               *ImageRecordLink;\r
+  LIST_ENTRY               *NextImageRecordLink;\r
+  LIST_ENTRY               *ImageRecordEndLink;\r
+  LIST_ENTRY               *ImageRecordList;\r
 \r
   ImageRecordList = &mImagePropertiesPrivateData.ImageRecordList;\r
 \r
-  ImageRecordLink = ImageRecordList->ForwardLink;\r
+  ImageRecordLink     = ImageRecordList->ForwardLink;\r
   NextImageRecordLink = ImageRecordLink->ForwardLink;\r
-  ImageRecordEndLink = ImageRecordList;\r
+  ImageRecordEndLink  = ImageRecordList;\r
   while (ImageRecordLink != ImageRecordEndLink) {\r
     ImageRecord = CR (\r
                     ImageRecordLink,\r
@@ -968,10 +982,11 @@ SortImageRecord (
       if (ImageRecord->ImageBase > NextImageRecord->ImageBase) {\r
         SwapImageRecord (ImageRecord, NextImageRecord);\r
       }\r
+\r
       NextImageRecordLink = NextImageRecordLink->ForwardLink;\r
     }\r
 \r
-    ImageRecordLink = ImageRecordLink->ForwardLink;\r
+    ImageRecordLink     = ImageRecordLink->ForwardLink;\r
     NextImageRecordLink = ImageRecordLink->ForwardLink;\r
   }\r
 }\r
@@ -985,16 +1000,17 @@ DumpImageRecord (
   VOID\r
   )\r
 {\r
-  IMAGE_PROPERTIES_RECORD      *ImageRecord;\r
-  LIST_ENTRY                   *ImageRecordLink;\r
-  LIST_ENTRY                   *ImageRecordList;\r
-  UINTN                        Index;\r
+  IMAGE_PROPERTIES_RECORD  *ImageRecord;\r
+  LIST_ENTRY               *ImageRecordLink;\r
+  LIST_ENTRY               *ImageRecordList;\r
+  UINTN                    Index;\r
 \r
   ImageRecordList = &mImagePropertiesPrivateData.ImageRecordList;\r
 \r
-  for (ImageRecordLink = ImageRecordList->ForwardLink, Index= 0;\r
+  for (ImageRecordLink = ImageRecordList->ForwardLink, Index = 0;\r
        ImageRecordLink != ImageRecordList;\r
-       ImageRecordLink = ImageRecordLink->ForwardLink, Index++) {\r
+       ImageRecordLink = ImageRecordLink->ForwardLink, Index++)\r
+  {\r
     ImageRecord = CR (\r
                     ImageRecordLink,\r
                     IMAGE_PROPERTIES_RECORD,\r
@@ -1015,25 +1031,26 @@ SmmInsertImageRecord (
   IN EFI_SMM_DRIVER_ENTRY  *DriverEntry\r
   )\r
 {\r
-  VOID                                 *ImageAddress;\r
-  EFI_IMAGE_DOS_HEADER                 *DosHdr;\r
-  UINT32                               PeCoffHeaderOffset;\r
-  UINT32                               SectionAlignment;\r
-  EFI_IMAGE_SECTION_HEADER             *Section;\r
-  EFI_IMAGE_OPTIONAL_HEADER_PTR_UNION  Hdr;\r
-  UINT8                                *Name;\r
-  UINTN                                Index;\r
-  IMAGE_PROPERTIES_RECORD              *ImageRecord;\r
-  CHAR8                                *PdbPointer;\r
-  IMAGE_PROPERTIES_RECORD_CODE_SECTION *ImageRecordCodeSection;\r
+  VOID                                  *ImageAddress;\r
+  EFI_IMAGE_DOS_HEADER                  *DosHdr;\r
+  UINT32                                PeCoffHeaderOffset;\r
+  UINT32                                SectionAlignment;\r
+  EFI_IMAGE_SECTION_HEADER              *Section;\r
+  EFI_IMAGE_OPTIONAL_HEADER_PTR_UNION   Hdr;\r
+  UINT8                                 *Name;\r
+  UINTN                                 Index;\r
+  IMAGE_PROPERTIES_RECORD               *ImageRecord;\r
+  CHAR8                                 *PdbPointer;\r
+  IMAGE_PROPERTIES_RECORD_CODE_SECTION  *ImageRecordCodeSection;\r
 \r
   DEBUG ((DEBUG_VERBOSE, "SMM InsertImageRecord - 0x%x\n", DriverEntry));\r
   DEBUG ((DEBUG_VERBOSE, "SMM InsertImageRecord - 0x%016lx - 0x%08x\n", DriverEntry->ImageBuffer, DriverEntry->NumberOfPage));\r
 \r
-  ImageRecord = AllocatePool (sizeof(*ImageRecord));\r
+  ImageRecord = AllocatePool (sizeof (*ImageRecord));\r
   if (ImageRecord == NULL) {\r
-    return ;\r
+    return;\r
   }\r
+\r
   ImageRecord->Signature = IMAGE_PROPERTIES_RECORD_SIGNATURE;\r
 \r
   DEBUG ((DEBUG_VERBOSE, "SMM ImageRecordCount - 0x%x\n", mImagePropertiesPrivateData.ImageRecordCount));\r
@@ -1042,11 +1059,11 @@ SmmInsertImageRecord (
   // Step 1: record whole region\r
   //\r
   ImageRecord->ImageBase = DriverEntry->ImageBuffer;\r
-  ImageRecord->ImageSize = EfiPagesToSize(DriverEntry->NumberOfPage);\r
+  ImageRecord->ImageSize = EfiPagesToSize (DriverEntry->NumberOfPage);\r
 \r
   ImageAddress = (VOID *)(UINTN)DriverEntry->ImageBuffer;\r
 \r
-  PdbPointer = PeCoffLoaderGetPdbPointer ((VOID*) (UINTN) ImageAddress);\r
+  PdbPointer = PeCoffLoaderGetPdbPointer ((VOID *)(UINTN)ImageAddress);\r
   if (PdbPointer != NULL) {\r
     DEBUG ((DEBUG_VERBOSE, "SMM   Image - %a\n", PdbPointer));\r
   }\r
@@ -1054,13 +1071,13 @@ SmmInsertImageRecord (
   //\r
   // Check PE/COFF image\r
   //\r
-  DosHdr = (EFI_IMAGE_DOS_HEADER *) (UINTN) ImageAddress;\r
+  DosHdr             = (EFI_IMAGE_DOS_HEADER *)(UINTN)ImageAddress;\r
   PeCoffHeaderOffset = 0;\r
   if (DosHdr->e_magic == EFI_IMAGE_DOS_SIGNATURE) {\r
     PeCoffHeaderOffset = DosHdr->e_lfanew;\r
   }\r
 \r
-  Hdr.Pe32 = (EFI_IMAGE_NT_HEADERS32 *)((UINT8 *) (UINTN) ImageAddress + PeCoffHeaderOffset);\r
+  Hdr.Pe32 = (EFI_IMAGE_NT_HEADERS32 *)((UINT8 *)(UINTN)ImageAddress + PeCoffHeaderOffset);\r
   if (Hdr.Pe32->Signature != EFI_IMAGE_NT_SIGNATURE) {\r
     DEBUG ((DEBUG_VERBOSE, "SMM Hdr.Pe32->Signature invalid - 0x%x\n", Hdr.Pe32->Signature));\r
     goto Finish;\r
@@ -1070,29 +1087,34 @@ SmmInsertImageRecord (
   // Get SectionAlignment\r
   //\r
   if (Hdr.Pe32->OptionalHeader.Magic == EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) {\r
-    SectionAlignment  = Hdr.Pe32->OptionalHeader.SectionAlignment;\r
+    SectionAlignment = Hdr.Pe32->OptionalHeader.SectionAlignment;\r
   } else {\r
-    SectionAlignment  = Hdr.Pe32Plus->OptionalHeader.SectionAlignment;\r
+    SectionAlignment = Hdr.Pe32Plus->OptionalHeader.SectionAlignment;\r
   }\r
 \r
   SetMemoryAttributesTableSectionAlignment (SectionAlignment);\r
   if ((SectionAlignment & (RUNTIME_PAGE_ALLOCATION_GRANULARITY - 1)) != 0) {\r
-    DEBUG ((DEBUG_WARN, "SMM !!!!!!!!  InsertImageRecord - Section Alignment(0x%x) is not %dK  !!!!!!!!\n",\r
-      SectionAlignment, RUNTIME_PAGE_ALLOCATION_GRANULARITY >> 10));\r
-    PdbPointer = PeCoffLoaderGetPdbPointer ((VOID*) (UINTN) ImageAddress);\r
+    DEBUG ((\r
+      DEBUG_WARN,\r
+      "SMM !!!!!!!!  InsertImageRecord - Section Alignment(0x%x) is not %dK  !!!!!!!!\n",\r
+      SectionAlignment,\r
+      RUNTIME_PAGE_ALLOCATION_GRANULARITY >> 10\r
+      ));\r
+    PdbPointer = PeCoffLoaderGetPdbPointer ((VOID *)(UINTN)ImageAddress);\r
     if (PdbPointer != NULL) {\r
       DEBUG ((DEBUG_WARN, "SMM !!!!!!!!  Image - %a  !!!!!!!!\n", PdbPointer));\r
     }\r
+\r
     goto Finish;\r
   }\r
 \r
-  Section = (EFI_IMAGE_SECTION_HEADER *) (\r
-               (UINT8 *) (UINTN) ImageAddress +\r
-               PeCoffHeaderOffset +\r
-               sizeof(UINT32) +\r
-               sizeof(EFI_IMAGE_FILE_HEADER) +\r
-               Hdr.Pe32->FileHeader.SizeOfOptionalHeader\r
-               );\r
+  Section = (EFI_IMAGE_SECTION_HEADER *)(\r
+                                         (UINT8 *)(UINTN)ImageAddress +\r
+                                         PeCoffHeaderOffset +\r
+                                         sizeof (UINT32) +\r
+                                         sizeof (EFI_IMAGE_FILE_HEADER) +\r
+                                         Hdr.Pe32->FileHeader.SizeOfOptionalHeader\r
+                                         );\r
   ImageRecord->CodeSegmentCount = 0;\r
   InitializeListHead (&ImageRecord->CodeSegmentList);\r
   for (Index = 0; Index < Hdr.Pe32->FileHeader.NumberOfSections; Index++) {\r
@@ -1124,10 +1146,11 @@ SmmInsertImageRecord (
       //\r
       // Step 2: record code section\r
       //\r
-      ImageRecordCodeSection = AllocatePool (sizeof(*ImageRecordCodeSection));\r
+      ImageRecordCodeSection = AllocatePool (sizeof (*ImageRecordCodeSection));\r
       if (ImageRecordCodeSection == NULL) {\r
-        return ;\r
+        return;\r
       }\r
+\r
       ImageRecordCodeSection->Signature = IMAGE_PROPERTIES_RECORD_CODE_SECTION_SIGNATURE;\r
 \r
       ImageRecordCodeSection->CodeSegmentBase = (UINTN)ImageAddress + Section[Index].VirtualAddress;\r
@@ -1143,10 +1166,11 @@ SmmInsertImageRecord (
   if (ImageRecord->CodeSegmentCount == 0) {\r
     SetMemoryAttributesTableSectionAlignment (1);\r
     DEBUG ((DEBUG_ERROR, "SMM !!!!!!!!  InsertImageRecord - CodeSegmentCount is 0  !!!!!!!!\n"));\r
-    PdbPointer = PeCoffLoaderGetPdbPointer ((VOID*) (UINTN) ImageAddress);\r
+    PdbPointer = PeCoffLoaderGetPdbPointer ((VOID *)(UINTN)ImageAddress);\r
     if (PdbPointer != NULL) {\r
       DEBUG ((DEBUG_ERROR, "SMM !!!!!!!!  Image - %a  !!!!!!!!\n", PdbPointer));\r
     }\r
+\r
     goto Finish;\r
   }\r
 \r
@@ -1172,10 +1196,9 @@ SmmInsertImageRecord (
   SortImageRecord ();\r
 \r
 Finish:\r
-  return ;\r
+  return;\r
 }\r
 \r
-\r
 /**\r
   Publish MemoryAttributesTable to SMM configuration table.\r
 **/\r
@@ -1184,27 +1207,27 @@ PublishMemoryAttributesTable (
   VOID\r
   )\r
 {\r
-  UINTN                                MemoryMapSize;\r
-  EFI_MEMORY_DESCRIPTOR                *MemoryMap;\r
-  UINTN                                MapKey;\r
-  UINTN                                DescriptorSize;\r
-  UINT32                               DescriptorVersion;\r
-  UINTN                                Index;\r
-  EFI_STATUS                           Status;\r
-  UINTN                                RuntimeEntryCount;\r
-  EDKII_PI_SMM_MEMORY_ATTRIBUTES_TABLE *MemoryAttributesTable;\r
-  EFI_MEMORY_DESCRIPTOR                *MemoryAttributesEntry;\r
-  UINTN                                MemoryAttributesTableSize;\r
+  UINTN                                 MemoryMapSize;\r
+  EFI_MEMORY_DESCRIPTOR                 *MemoryMap;\r
+  UINTN                                 MapKey;\r
+  UINTN                                 DescriptorSize;\r
+  UINT32                                DescriptorVersion;\r
+  UINTN                                 Index;\r
+  EFI_STATUS                            Status;\r
+  UINTN                                 RuntimeEntryCount;\r
+  EDKII_PI_SMM_MEMORY_ATTRIBUTES_TABLE  *MemoryAttributesTable;\r
+  EFI_MEMORY_DESCRIPTOR                 *MemoryAttributesEntry;\r
+  UINTN                                 MemoryAttributesTableSize;\r
 \r
   MemoryMapSize = 0;\r
-  MemoryMap = NULL;\r
-  Status = SmmCoreGetMemoryMapMemoryAttributesTable (\r
-             &MemoryMapSize,\r
-             MemoryMap,\r
-             &MapKey,\r
-             &DescriptorSize,\r
-             &DescriptorVersion\r
-             );\r
+  MemoryMap     = NULL;\r
+  Status        = SmmCoreGetMemoryMapMemoryAttributesTable (\r
+                    &MemoryMapSize,\r
+                    MemoryMap,\r
+                    &MapKey,\r
+                    &DescriptorSize,\r
+                    &DescriptorVersion\r
+                    );\r
   ASSERT (Status == EFI_BUFFER_TOO_SMALL);\r
 \r
   do {\r
@@ -1228,9 +1251,9 @@ PublishMemoryAttributesTable (
   //\r
   // Allocate MemoryAttributesTable\r
   //\r
-  RuntimeEntryCount = MemoryMapSize/DescriptorSize;\r
-  MemoryAttributesTableSize = sizeof(EDKII_PI_SMM_MEMORY_ATTRIBUTES_TABLE) + DescriptorSize * RuntimeEntryCount;\r
-  MemoryAttributesTable = AllocatePool (sizeof(EDKII_PI_SMM_MEMORY_ATTRIBUTES_TABLE) + DescriptorSize * RuntimeEntryCount);\r
+  RuntimeEntryCount         = MemoryMapSize/DescriptorSize;\r
+  MemoryAttributesTableSize = sizeof (EDKII_PI_SMM_MEMORY_ATTRIBUTES_TABLE) + DescriptorSize * RuntimeEntryCount;\r
+  MemoryAttributesTable     = AllocatePool (sizeof (EDKII_PI_SMM_MEMORY_ATTRIBUTES_TABLE) + DescriptorSize * RuntimeEntryCount);\r
   ASSERT (MemoryAttributesTable != NULL);\r
   MemoryAttributesTable->Version         = EDKII_PI_SMM_MEMORY_ATTRIBUTES_TABLE_VERSION;\r
   MemoryAttributesTable->NumberOfEntries = (UINT32)RuntimeEntryCount;\r
@@ -1249,16 +1272,15 @@ PublishMemoryAttributesTable (
     DEBUG ((DEBUG_VERBOSE, "  VirtualStart      - 0x%016lx\n", MemoryAttributesEntry->VirtualStart));\r
     DEBUG ((DEBUG_VERBOSE, "  NumberOfPages     - 0x%016lx\n", MemoryAttributesEntry->NumberOfPages));\r
     DEBUG ((DEBUG_VERBOSE, "  Attribute         - 0x%016lx\n", MemoryAttributesEntry->Attribute));\r
-    MemoryAttributesEntry = NEXT_MEMORY_DESCRIPTOR(MemoryAttributesEntry, DescriptorSize);\r
+    MemoryAttributesEntry = NEXT_MEMORY_DESCRIPTOR (MemoryAttributesEntry, DescriptorSize);\r
 \r
-    MemoryMap = NEXT_MEMORY_DESCRIPTOR(MemoryMap, DescriptorSize);\r
+    MemoryMap = NEXT_MEMORY_DESCRIPTOR (MemoryMap, DescriptorSize);\r
   }\r
 \r
   Status = gSmst->SmmInstallConfigurationTable (gSmst, &gEdkiiPiSmmMemoryAttributesTableGuid, MemoryAttributesTable, MemoryAttributesTableSize);\r
   ASSERT_EFI_ERROR (Status);\r
 }\r
 \r
-\r
 /**\r
   This function installs all SMM image record information.\r
 **/\r
@@ -1267,12 +1289,12 @@ SmmInstallImageRecord (
   VOID\r
   )\r
 {\r
-  EFI_STATUS                  Status;\r
-  UINTN                       NoHandles;\r
-  EFI_HANDLE                  *HandleBuffer;\r
-  EFI_LOADED_IMAGE_PROTOCOL   *LoadedImage;\r
-  UINTN                       Index;\r
-  EFI_SMM_DRIVER_ENTRY        DriverEntry;\r
+  EFI_STATUS                 Status;\r
+  UINTN                      NoHandles;\r
+  EFI_HANDLE                 *HandleBuffer;\r
+  EFI_LOADED_IMAGE_PROTOCOL  *LoadedImage;\r
+  UINTN                      Index;\r
+  EFI_SMM_DRIVER_ENTRY       DriverEntry;\r
 \r
   Status = SmmLocateHandleBuffer (\r
              ByProtocol,\r
@@ -1282,7 +1304,7 @@ SmmInstallImageRecord (
              &HandleBuffer\r
              );\r
   if (EFI_ERROR (Status)) {\r
-    return ;\r
+    return;\r
   }\r
 \r
   for (Index = 0; Index < NoHandles; Index++) {\r
@@ -1294,18 +1316,19 @@ SmmInstallImageRecord (
     if (EFI_ERROR (Status)) {\r
       continue;\r
     }\r
+\r
     DEBUG ((DEBUG_VERBOSE, "LoadedImage - 0x%x 0x%x ", LoadedImage->ImageBase, LoadedImage->ImageSize));\r
     {\r
-      VOID *PdbPointer;\r
+      VOID  *PdbPointer;\r
       PdbPointer = PeCoffLoaderGetPdbPointer (LoadedImage->ImageBase);\r
       if (PdbPointer != NULL) {\r
         DEBUG ((DEBUG_VERBOSE, "(%a) ", PdbPointer));\r
       }\r
     }\r
     DEBUG ((DEBUG_VERBOSE, "\n"));\r
-    ZeroMem (&DriverEntry, sizeof(DriverEntry));\r
+    ZeroMem (&DriverEntry, sizeof (DriverEntry));\r
     DriverEntry.ImageBuffer  = (UINTN)LoadedImage->ImageBase;\r
-    DriverEntry.NumberOfPage = EFI_SIZE_TO_PAGES((UINTN)LoadedImage->ImageSize);\r
+    DriverEntry.NumberOfPage = EFI_SIZE_TO_PAGES ((UINTN)LoadedImage->ImageSize);\r
     SmmInsertImageRecord (&DriverEntry);\r
   }\r
 \r
@@ -1354,8 +1377,8 @@ SmmCoreInitializeMemoryAttributesTable (
   VOID\r
   )\r
 {\r
-  EFI_STATUS                        Status;\r
-  VOID                              *Registration;\r
+  EFI_STATUS  Status;\r
+  VOID        *Registration;\r
 \r
   Status = gSmst->SmmRegisterProtocolNotify (\r
                     &gEfiSmmEndOfDxeProtocolGuid,\r
@@ -1364,5 +1387,5 @@ SmmCoreInitializeMemoryAttributesTable (
                     );\r
   ASSERT_EFI_ERROR (Status);\r
 \r
-  return ;\r
+  return;\r
 }\r