Update comments on setting >= 8 bytes Fv Alignment.
authorlgao4 <lgao4@de2fecce-e211-0410-80a6-f3fac2684e05>
Mon, 26 Oct 2009 09:14:47 +0000 (09:14 +0000)
committerlgao4 <lgao4@de2fecce-e211-0410-80a6-f3fac2684e05>
Mon, 26 Oct 2009 09:14:47 +0000 (09:14 +0000)
git-svn-id: https://edk2.tianocore.org/svn/edk2/trunk@9372 de2fecce-e211-0410-80a6-f3fac2684e05

edk2/MdeModulePkg/Core/Dxe/Dispatcher/Dispatcher.c
edk2/MdeModulePkg/Core/Dxe/FwVolBlock/FwVolBlock.c
edk2/MdeModulePkg/Core/Pei/FwVol/FwVol.c

index 7e6ddce..6a18fe9 100644 (file)
@@ -845,9 +845,12 @@ CoreProcessFvImageFile (
     // FvImage should be at its required alignment.\r
     //\r
     FvHeader = (EFI_FIRMWARE_VOLUME_HEADER *) Buffer;\r
+    //\r
+    // Get FvHeader alignment\r
+    //\r
     FvAlignment = 1 << ((FvHeader->Attributes & EFI_FVB2_ALIGNMENT) >> 16);\r
     //\r
-    // FvAlignment must be more than 8 bytes required by FvHeader structure.\r
+    // FvAlignment must be greater than or equal to 8 bytes of the minimum FFS alignment value. \r
     //\r
     if (FvAlignment < 8) {\r
       FvAlignment = 8;\r
index 9b7b166..5a3b243 100644 (file)
@@ -450,6 +450,9 @@ ProduceFVBProtocolOnBuffer (
   // Get FvHeader alignment\r
   //\r
   FvAlignment = 1 << ((FwVolHeader->Attributes & EFI_FVB2_ALIGNMENT) >> 16);\r
+  //\r
+  // FvAlignment must be greater than or equal to 8 bytes of the minimum FFS alignment value. \r
+  //\r
   if (FvAlignment < 8) {\r
     FvAlignment = 8;\r
   }\r
index 54501f5..a86e4e5 100644 (file)
@@ -893,7 +893,7 @@ ProcessFvFile (
   ASSERT_EFI_ERROR (Status);\r
   \r
   //\r
-  // FvAlignment must be more than 8 bytes required by FvHeader structure.\r
+  // FvAlignment must be greater than or equal to 8 bytes of the minimum FFS alignment value.\r
   //\r
   FvAlignment = 1 << ((FvImageInfo.FvAttributes & EFI_FVB2_ALIGNMENT) >> 16);\r
   if (FvAlignment < 8) {\r