]> git.proxmox.com Git - mirror_edk2.git/blobdiff - BaseTools/Source/Python/BPDG/GenVpd.py
BaseTools: refactor the error for PCD value is negative or exceed max
[mirror_edk2.git] / BaseTools / Source / Python / BPDG / GenVpd.py
index 4ff464756f20dca1a9dff95ca6af7034356db632..d7852d29eed09df7b016ab7fadc8aff991973146 100644 (file)
@@ -2,7 +2,7 @@
 #  This file include GenVpd class for fix the Vpd type PCD offset, and PcdEntry for describe\r
 #  and process each entry of vpd type PCD.\r
 #\r
-#  Copyright (c) 2010 - 2016, Intel Corporation. All rights reserved.<BR>\r
+#  Copyright (c) 2010 - 2018, Intel Corporation. All rights reserved.<BR>\r
 #\r
 #  This program and the accompanying materials\r
 #  are licensed and made available under the terms and conditions of the BSD License\r
 #  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
 #\r
 \r
+from __future__ import absolute_import\r
 import Common.LongFilePathOs as os\r
-import StringIO\r
-import StringTable as st\r
+from io import BytesIO\r
+from . import StringTable as st\r
 import array\r
 import re\r
 from Common.LongFilePathSupport import OpenLongFilePath as open\r
 from struct import *\r
+from Common.DataType import MAX_SIZE_TYPE, MAX_VAL_TYPE\r
 import Common.EdkLogger as EdkLogger\r
 import Common.BuildToolError as BuildToolError\r
 \r
@@ -31,11 +33,11 @@ _FORMAT_CHAR = {1: 'B',
 \r
 ## The VPD PCD data structure for store and process each VPD PCD entry.\r
 #\r
-#  This class contain method to format and pack pcd's value.          \r
+#  This class contain method to format and pack pcd's value.\r
 #\r
 class PcdEntry:\r
-    def __init__(self, PcdCName, SkuId,PcdOffset, PcdSize, PcdValue, Lineno=None, FileName=None, PcdUnpackValue=None, \r
-                 PcdBinOffset=None, PcdBinSize=None):\r
+    def __init__(self, PcdCName, SkuId,PcdOffset, PcdSize, PcdValue, Lineno=None, FileName=None, PcdUnpackValue=None,\r
+                 PcdBinOffset=None, PcdBinSize=None, Alignment=None):\r
         self.PcdCName       = PcdCName.strip()\r
         self.SkuId          = SkuId.strip()\r
         self.PcdOffset      = PcdOffset.strip()\r
@@ -46,7 +48,8 @@ class PcdEntry:
         self.PcdUnpackValue = PcdUnpackValue\r
         self.PcdBinOffset   = PcdBinOffset\r
         self.PcdBinSize     = PcdBinSize\r
-        \r
+        self.Alignment       = Alignment\r
+\r
         if self.PcdValue == '' :\r
             EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID,\r
                             "Invalid PCD format(Name: %s File: %s line: %s) , no Value specified!" % (self.PcdCName, self.FileName, self.Lineno))\r
@@ -61,14 +64,14 @@ class PcdEntry:
 \r
         self._GenOffsetValue ()\r
 \r
-    ## Analyze the string value to judge the PCD's datum type euqal to Boolean or not.\r
-    # \r
+    ## Analyze the string value to judge the PCD's datum type equal to Boolean or not.\r
+    #\r
     #  @param   ValueString      PCD's value\r
     #  @param   Size             PCD's size\r
-    #  \r
+    #\r
     #  @retval  True   PCD's datum type is Boolean\r
-    #  @retval  False  PCD's datum type is not Boolean.              \r
-    #    \r
+    #  @retval  False  PCD's datum type is not Boolean.\r
+    #\r
     def _IsBoolean(self, ValueString, Size):\r
         if (Size == "1"):\r
             if ValueString.upper() in ["TRUE", "FALSE"]:\r
@@ -79,10 +82,10 @@ class PcdEntry:
         return False\r
 \r
     ## Convert the PCD's value from string to integer.\r
-    #              \r
+    #\r
     #  This function will try to convert the Offset value form string to integer\r
     #  for both hexadecimal and decimal.\r
-    #    \r
+    #\r
     def _GenOffsetValue(self):\r
         if self.PcdOffset != "*" :\r
             try:\r
@@ -95,10 +98,10 @@ class PcdEntry:
                                     "Invalid offset value %s for PCD %s (File: %s Line: %s)" % (self.PcdOffset, self.PcdCName, self.FileName, self.Lineno))\r
 \r
     ## Pack Boolean type VPD PCD's value form string to binary type.\r
-    # \r
+    #\r
     #  @param ValueString     The boolean type string for pack.\r
-    # \r
-    # \r
+    #\r
+    #\r
     def _PackBooleanValue(self, ValueString):\r
         if ValueString.upper() == "TRUE" or ValueString in ["1", "0x1", "0x01"]:\r
             try:\r
@@ -114,46 +117,27 @@ class PcdEntry:
                                 "Invalid size or value for PCD %s to pack(File: %s Line: %s)." % (self.PcdCName, self.FileName, self.Lineno))\r
 \r
     ## Pack Integer type VPD PCD's value form string to binary type.\r
-    # \r
+    #\r
     #  @param ValueString     The Integer type string for pack.\r
-    # \r
-    #                                \r
+    #\r
+    #\r
     def _PackIntValue(self, IntValue, Size):\r
-        if Size not in _FORMAT_CHAR.keys():\r
+        if Size not in _FORMAT_CHAR:\r
             EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID,\r
                             "Invalid size %d for PCD %s in integer datum size(File: %s Line: %s)." % (Size, self.PcdCName, self.FileName, self.Lineno))\r
 \r
-        if Size == 1:\r
-            if IntValue < 0:\r
-                EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID,\r
-                                "PCD can't be set to negative value %d for PCD %s in UINT8 datum type(File: %s Line: %s)." % (IntValue, self.PcdCName, self.FileName, self.Lineno))\r
-            elif IntValue >= 0x100:\r
-                EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID,\r
-                                "Too large PCD value %d for datum type UINT8 for PCD %s(File: %s Line: %s)." % (IntValue, self.PcdCName, self.FileName, self.Lineno))\r
-        elif Size == 2:\r
-            if IntValue < 0:\r
-                EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID,\r
-                                "PCD can't be set to negative value %d for PCD %s in UINT16 datum type(File: %s Line: %s)." % (IntValue, self.PcdCName, self.FileName, self.Lineno))\r
-            elif IntValue >= 0x10000:\r
-                EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID,\r
-                                "Too large PCD value %d for datum type UINT16 for PCD %s(File: %s Line: %s)." % (IntValue, self.PcdCName, self.FileName, self.Lineno))\r
-        elif Size == 4:\r
-            if IntValue < 0:\r
-                EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID,\r
-                                "PCD can't be set to negative value %d for PCD %s in UINT32 datum type(File: %s Line: %s)." % (IntValue, self.PcdCName, self.FileName, self.Lineno))\r
-            elif IntValue >= 0x100000000:\r
-                EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID,\r
-                                "Too large PCD value %d for datum type UINT32 for PCD %s(File: %s Line: %s)." % (IntValue, self.PcdCName, self.FileName, self.Lineno))\r
-        elif Size == 8:\r
-            if IntValue < 0:\r
-                EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID,\r
-                                "PCD can't be set to negative value %d for PCD %s in UINT32 datum type(File: %s Line: %s)." % (IntValue, self.PcdCName, self.FileName, self.Lineno))\r
-            elif IntValue >= 0x10000000000000000:\r
-                EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID,\r
-                                "Too large PCD value %d for datum type UINT32 for PCD %s(File: %s Line: %s)." % (IntValue, self.PcdCName, self.FileName, self.Lineno))\r
-        else:\r
-            EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID,\r
-                            "Invalid size %d for PCD %s in integer datum size(File: %s Line: %s)." % (Size, self.PcdCName, self.FileName, self.Lineno))\r
+        for Type, MaxSize in MAX_SIZE_TYPE.items():\r
+            if Type == 'BOOLEAN':\r
+                continue\r
+            if Size == MaxSize:\r
+                if IntValue < 0:\r
+                    EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID,\r
+                                    "PCD can't be set to negative value %d for PCD %s in %s datum type(File: %s Line: %s)." % (\r
+                                    IntValue, self.PcdCName, Type, self.FileName, self.Lineno))\r
+                elif IntValue > MAX_VAL_TYPE[Type]:\r
+                    EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID,\r
+                                    "Too large PCD value %d for datum type %s for PCD %s(File: %s Line: %s)." % (\r
+                                    IntValue, Type, self.PcdCName, self.FileName, self.Lineno))\r
 \r
         try:\r
             self.PcdValue = pack(_FORMAT_CHAR[Size], IntValue)\r
@@ -164,38 +148,41 @@ class PcdEntry:
     ## Pack VOID* type VPD PCD's value form string to binary type.\r
     #\r
     #  The VOID* type of string divided into 3 sub-type:\r
-    #    1:    L"String", Unicode type string.\r
-    #    2:    "String",  Ascii type string.\r
+    #    1:    L"String"/L'String', Unicode type string.\r
+    #    2:    "String"/'String',  Ascii type string.\r
     #    3:    {bytearray}, only support byte-array.\r
     #\r
     #  @param ValueString     The Integer type string for pack.\r
-    #       \r
+    #\r
     def _PackPtrValue(self, ValueString, Size):\r
-        if ValueString.startswith('L"'):\r
+        if ValueString.startswith('L"') or ValueString.startswith("L'"):\r
             self._PackUnicode(ValueString, Size)\r
         elif ValueString.startswith('{') and ValueString.endswith('}'):\r
             self._PackByteArray(ValueString, Size)\r
-        elif ValueString.startswith('"') and ValueString.endswith('"'):\r
+        elif (ValueString.startswith('"') and ValueString.endswith('"')) or (ValueString.startswith("'") and ValueString.endswith("'")):\r
             self._PackString(ValueString, Size)\r
         else:\r
             EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID,\r
                             "Invalid VOID* type PCD %s value %s (File: %s Line: %s)" % (self.PcdCName, ValueString, self.FileName, self.Lineno))\r
 \r
     ## Pack an Ascii PCD value.\r
-    #  \r
-    #  An Ascii string for a PCD should be in format as  "".\r
-    #                   \r
+    #\r
+    #  An Ascii string for a PCD should be in format as  ""/''.\r
+    #\r
     def _PackString(self, ValueString, Size):\r
         if (Size < 0):\r
             EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID,\r
                             "Invalid parameter Size %s of PCD %s!(File: %s Line: %s)" % (self.PcdBinSize, self.PcdCName, self.FileName, self.Lineno))\r
         if (ValueString == ""):\r
             EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID, "Invalid parameter ValueString %s of PCD %s!(File: %s Line: %s)" % (self.PcdUnpackValue, self.PcdCName, self.FileName, self.Lineno))\r
-        if (len(ValueString) < 2):\r
-            EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID, "For PCD: %s ,ASCII string %s at least contains two!(File: %s Line: %s)" % (self.PcdCName, self.PcdUnpackValue, self.FileName, self.Lineno))\r
+\r
+        QuotedFlag = True\r
+        if ValueString.startswith("'"):\r
+            QuotedFlag = False\r
 \r
         ValueString = ValueString[1:-1]\r
-        if len(ValueString) + 1 > Size:\r
+        # No null-terminator in 'string'\r
+        if (QuotedFlag and len(ValueString) + 1 > Size) or (not QuotedFlag and len(ValueString) > Size):\r
             EdkLogger.error("BPDG", BuildToolError.RESOURCE_OVERFLOW,\r
                             "PCD value string %s is exceed to size %d(File: %s Line: %s)" % (ValueString, Size, self.FileName, self.Lineno))\r
         try:\r
@@ -205,9 +192,9 @@ class PcdEntry:
                             "Invalid size or value for PCD %s to pack(File: %s Line: %s)." % (self.PcdCName, self.FileName, self.Lineno))\r
 \r
     ## Pack a byte-array PCD value.\r
-    #  \r
+    #\r
     #  A byte-array for a PCD should be in format as  {0x01, 0x02, ...}.\r
-    #         \r
+    #\r
     def _PackByteArray(self, ValueString, Size):\r
         if (Size < 0):\r
             EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID, "Invalid parameter Size %s of PCD %s!(File: %s Line: %s)" % (self.PcdBinSize, self.PcdCName, self.FileName, self.Lineno))\r
@@ -257,20 +244,21 @@ class PcdEntry:
         self.PcdValue = ReturnArray.tolist()\r
 \r
     ## Pack a unicode PCD value into byte array.\r
-    #  \r
-    #  A unicode string for a PCD should be in format as  L"".\r
+    #\r
+    #  A unicode string for a PCD should be in format as  L""/L''.\r
     #\r
     def _PackUnicode(self, UnicodeString, Size):\r
         if (Size < 0):\r
             EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID, "Invalid parameter Size %s of PCD %s!(File: %s Line: %s)" % \\r
                              (self.PcdBinSize, self.PcdCName, self.FileName, self.Lineno))\r
-        if (len(UnicodeString) < 3):\r
-            EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID, "For PCD: %s ,ASCII string %s at least contains two!(File: %s Line: %s)" % \\r
-                            (self.PcdCName, self.PcdUnpackValue, self.FileName, self.Lineno))\r
 \r
+        QuotedFlag = True\r
+        if UnicodeString.startswith("L'"):\r
+            QuotedFlag = False\r
         UnicodeString = UnicodeString[2:-1]\r
 \r
-        if (len(UnicodeString) + 1) * 2 > Size:\r
+        # No null-terminator in L'string'\r
+        if (QuotedFlag and (len(UnicodeString) + 1) * 2 > Size) or (not QuotedFlag and len(UnicodeString) * 2 > Size):\r
             EdkLogger.error("BPDG", BuildToolError.RESOURCE_OVERFLOW,\r
                             "The size of unicode string %s is too larger for size %s(File: %s Line: %s)" % \\r
                             (UnicodeString, Size, self.FileName, self.Lineno))\r
@@ -299,7 +287,7 @@ class PcdEntry:
 #       2. Format the input file data to remove unused lines;\r
 #       3. Fixed offset if needed;\r
 #       4. Generate output file, including guided.map and guided.bin file;\r
-#        \r
+#\r
 class GenVPD :\r
     ## Constructor of DscBuildData\r
     #\r
@@ -329,9 +317,9 @@ class GenVPD :
             EdkLogger.error("BPDG", BuildToolError.FILE_OPEN_FAILURE, "File open failed for %s" % InputFileName, None)\r
 \r
     ##\r
-    # Parser the input file which is generated by the build tool. Convert the value of each pcd's \r
+    # Parser the input file which is generated by the build tool. Convert the value of each pcd's\r
     # from string to it's real format. Also remove the useless line in the input file.\r
-    # \r
+    #\r
     def ParserInputFile (self):\r
         count = 0\r
         for line in self.FileLinesList:\r
@@ -344,7 +332,7 @@ class GenVPD :
                 #\r
                 # Enhanced for support "|" character in the string.\r
                 #\r
-                ValueList = ['', '', '', '','']\r
+                ValueList = ['', '', '', '', '']\r
 \r
                 ValueRe = re.compile(r'\s*L?\".*\|.*\"\s*$')\r
                 PtrValue = ValueRe.findall(line)\r
@@ -376,7 +364,7 @@ class GenVPD :
         # Delete useless lines\r
         while (True) :\r
             try :\r
-                if (self.FileLinesList[count] == None) :\r
+                if (self.FileLinesList[count] is None) :\r
                     del(self.FileLinesList[count])\r
                 else :\r
                     count += 1\r
@@ -385,7 +373,7 @@ class GenVPD :
         #\r
         # After remove the useless line, if there are no data remain in the file line list,\r
         # Report warning messages to user's.\r
-        # \r
+        #\r
         if len(self.FileLinesList) == 0 :\r
             EdkLogger.warn('BPDG', BuildToolError.RESOURCE_NOT_AVAILABLE,\r
                            "There are no VPD type pcds defined in DSC file, Please check it.")\r
@@ -393,8 +381,8 @@ class GenVPD :
         # Process the pcds one by one base on the pcd's value and size\r
         count = 0\r
         for line in self.FileLinesList:\r
-            if line != None :\r
-                PCD = PcdEntry(line[0], line[1], line[2], line[3], line[4],line[5], self.InputFileName)   \r
+            if line is not None :\r
+                PCD = PcdEntry(line[0], line[1], line[2], line[3], line[4], line[5], self.InputFileName)\r
                 # Strip the space char\r
                 PCD.PcdCName     = PCD.PcdCName.strip(' ')\r
                 PCD.SkuId        = PCD.SkuId.strip(' ')\r
@@ -434,6 +422,7 @@ class GenVPD :
                 else:\r
                     Alignment = 1\r
 \r
+                PCD.Alignment = Alignment\r
                 if PCD.PcdOffset != '*':\r
                     if PCD.PcdOccupySize % Alignment != 0:\r
                         if PCD.PcdUnpackValue.startswith("{"):\r
@@ -444,6 +433,7 @@ class GenVPD :
                     if PCD.PcdOccupySize % Alignment != 0:\r
                         PCD.PcdOccupySize = (PCD.PcdOccupySize / Alignment + 1) * Alignment\r
 \r
+                PackSize = PCD.PcdOccupySize\r
                 if PCD._IsBoolean(PCD.PcdValue, PCD.PcdSize):\r
                     PCD._PackBooleanValue(PCD.PcdValue)\r
                     self.FileLinesList[count] = PCD\r
@@ -473,14 +463,14 @@ class GenVPD :
                 continue\r
 \r
     ##\r
-    # This function used to create a clean list only contain useful information and reorganized to make it \r
+    # This function used to create a clean list only contain useful information and reorganized to make it\r
     # easy to be sorted\r
     #\r
     def FormatFileLine (self) :\r
 \r
         for eachPcd in self.FileLinesList :\r
             if eachPcd.PcdOffset != '*' :\r
-                # Use pcd's Offset value as key, and pcd's Value as value \r
+                # Use pcd's Offset value as key, and pcd's Value as value\r
                 self.PcdFixedOffsetSizeList.append(eachPcd)\r
             else :\r
                 # Use pcd's CName as key, and pcd's Size as value\r
@@ -490,18 +480,26 @@ class GenVPD :
     ##\r
     # This function is use to fix the offset value which the not specified in the map file.\r
     # Usually it use the star (meaning any offset) character in the offset field\r
-    #    \r
+    #\r
     def FixVpdOffset (self):\r
         # At first, the offset should start at 0\r
         # Sort fixed offset list in order to find out where has free spaces for the pcd's offset\r
-        # value is "*" to insert into.      \r
+        # value is "*" to insert into.\r
 \r
-        self.PcdFixedOffsetSizeList.sort(lambda x, y: cmp(x.PcdBinOffset, y.PcdBinOffset))\r
+        self.PcdFixedOffsetSizeList.sort(key=lambda x: x.PcdBinOffset)\r
 \r
         #\r
         # Sort the un-fixed pcd's offset by it's size.\r
         #\r
-        self.PcdUnknownOffsetList.sort(lambda x, y: cmp(x.PcdBinSize, y.PcdBinSize))\r
+        self.PcdUnknownOffsetList.sort(key=lambda x: x.PcdBinSize)\r
+\r
+        index =0\r
+        for pcd in self.PcdUnknownOffsetList:\r
+            index += 1\r
+            if pcd.PcdCName == ".".join(("gEfiMdeModulePkgTokenSpaceGuid", "PcdNvStoreDefaultValueBuffer")):\r
+                if index != len(self.PcdUnknownOffsetList):\r
+                    for i in range(len(self.PcdUnknownOffsetList) - index):\r
+                        self.PcdUnknownOffsetList[index+i -1 ], self.PcdUnknownOffsetList[index+i] = self.PcdUnknownOffsetList[index+i], self.PcdUnknownOffsetList[index+i -1]\r
 \r
         #\r
         # Process all Offset value are "*"\r
@@ -510,60 +508,62 @@ class GenVPD :
             # The offset start from 0\r
             NowOffset = 0\r
             for Pcd in self.PcdUnknownOffsetList :\r
+                if NowOffset % Pcd.Alignment != 0:\r
+                    NowOffset = (NowOffset/ Pcd.Alignment + 1) * Pcd.Alignment\r
                 Pcd.PcdBinOffset = NowOffset\r
                 Pcd.PcdOffset    = str(hex(Pcd.PcdBinOffset))\r
                 NowOffset       += Pcd.PcdOccupySize\r
-                \r
+\r
             self.PcdFixedOffsetSizeList = self.PcdUnknownOffsetList\r
             return\r
 \r
-        # Check the offset of VPD type pcd's offset start from 0.    \r
+        # Check the offset of VPD type pcd's offset start from 0.\r
         if self.PcdFixedOffsetSizeList[0].PcdBinOffset != 0 :\r
             EdkLogger.warn("BPDG", "The offset of VPD type pcd should start with 0, please check it.",\r
                             None)\r
 \r
         # Judge whether the offset in fixed pcd offset list is overlapped or not.\r
         lenOfList = len(self.PcdFixedOffsetSizeList)\r
-        count     = 0                       \r
+        count     = 0\r
         while (count < lenOfList - 1) :\r
             PcdNow  = self.PcdFixedOffsetSizeList[count]\r
             PcdNext = self.PcdFixedOffsetSizeList[count+1]\r
-            # Two pcd's offset is same            \r
+            # Two pcd's offset is same\r
             if PcdNow.PcdBinOffset == PcdNext.PcdBinOffset :\r
                 EdkLogger.error("BPDG", BuildToolError.ATTRIBUTE_GET_FAILURE,\r
                                 "The offset of %s at line: %s is same with %s at line: %s in file %s" % \\r
                                 (PcdNow.PcdCName, PcdNow.Lineno, PcdNext.PcdCName, PcdNext.Lineno, PcdNext.FileName),\r
                                 None)\r
 \r
-            # Overlapped   \r
+            # Overlapped\r
             if PcdNow.PcdBinOffset + PcdNow.PcdOccupySize > PcdNext.PcdBinOffset :\r
                 EdkLogger.error("BPDG", BuildToolError.ATTRIBUTE_GET_FAILURE,\r
                                 "The offset of %s at line: %s is overlapped with %s at line: %s in file %s" % \\r
                                 (PcdNow.PcdCName, PcdNow.Lineno, PcdNext.PcdCName, PcdNext.Lineno, PcdNext.FileName),\r
                                 None)\r
 \r
-            # Has free space, raise a warning message   \r
+            # Has free space, raise a warning message\r
             if PcdNow.PcdBinOffset + PcdNow.PcdOccupySize < PcdNext.PcdBinOffset :\r
                 EdkLogger.warn("BPDG", BuildToolError.ATTRIBUTE_GET_FAILURE,\r
                                "The offsets have free space of between %s at line: %s and %s at line: %s in file %s" % \\r
                                (PcdNow.PcdCName, PcdNow.Lineno, PcdNext.PcdCName, PcdNext.Lineno, PcdNext.FileName),\r
                                 None)\r
             count += 1\r
-                             \r
+\r
         LastOffset              = self.PcdFixedOffsetSizeList[0].PcdBinOffset\r
         FixOffsetSizeListCount  = 0\r
         lenOfList               = len(self.PcdFixedOffsetSizeList)\r
         lenOfUnfixedList        = len(self.PcdUnknownOffsetList)\r
-                \r
+\r
         ##\r
-        # Insert the un-fixed offset pcd's list into fixed offset pcd's list if has free space between those pcds. \r
-        # \r
+        # Insert the un-fixed offset pcd's list into fixed offset pcd's list if has free space between those pcds.\r
+        #\r
         while (FixOffsetSizeListCount < lenOfList) :\r
-            \r
-            eachFixedPcd     = self.PcdFixedOffsetSizeList[FixOffsetSizeListCount]                       \r
+\r
+            eachFixedPcd     = self.PcdFixedOffsetSizeList[FixOffsetSizeListCount]\r
             NowOffset        = eachFixedPcd.PcdBinOffset\r
-            \r
-            # Has free space               \r
+\r
+            # Has free space\r
             if LastOffset < NowOffset :\r
                 if lenOfUnfixedList != 0 :\r
                     countOfUnfixedList = 0\r
@@ -572,49 +572,51 @@ class GenVPD :
                         needFixPcdSize      = eachUnfixedPcd.PcdOccupySize\r
                         # Not been fixed\r
                         if eachUnfixedPcd.PcdOffset == '*' :\r
+                            if LastOffset % eachUnfixedPcd.Alignment != 0:\r
+                                LastOffset = (LastOffset / eachUnfixedPcd.Alignment + 1) * eachUnfixedPcd.Alignment\r
                             # The offset un-fixed pcd can write into this free space\r
                             if needFixPcdSize <= (NowOffset - LastOffset) :\r
                                 # Change the offset value of un-fixed pcd\r
                                 eachUnfixedPcd.PcdOffset    = str(hex(LastOffset))\r
                                 eachUnfixedPcd.PcdBinOffset = LastOffset\r
                                 # Insert this pcd into fixed offset pcd list.\r
-                                self.PcdFixedOffsetSizeList.insert(FixOffsetSizeListCount,eachUnfixedPcd)\r
-                                \r
+                                self.PcdFixedOffsetSizeList.insert(FixOffsetSizeListCount, eachUnfixedPcd)\r
+\r
                                 # Delete the item's offset that has been fixed and added into fixed offset list\r
                                 self.PcdUnknownOffsetList.pop(countOfUnfixedList)\r
-                                \r
+\r
                                 # After item added, should enlarge the length of fixed pcd offset list\r
-                                lenOfList               += 1                                \r
+                                lenOfList               += 1\r
                                 FixOffsetSizeListCount  += 1\r
-                                \r
+\r
                                 # Decrease the un-fixed pcd offset list's length\r
                                 lenOfUnfixedList        -= 1\r
-                                \r
-                                # Modify the last offset value \r
-                                LastOffset              += needFixPcdSize                            \r
+\r
+                                # Modify the last offset value\r
+                                LastOffset              += needFixPcdSize\r
                             else :\r
                                 # It can not insert into those two pcds, need to check still has other space can store it.\r
                                 LastOffset             = NowOffset + self.PcdFixedOffsetSizeList[FixOffsetSizeListCount].PcdOccupySize\r
                                 FixOffsetSizeListCount += 1\r
                                 break\r
-                                                                                 \r
+\r
                 # Set the FixOffsetSizeListCount = lenOfList for quit the loop\r
                 else :\r
-                    FixOffsetSizeListCount = lenOfList                    \r
-                        \r
-            # No free space, smoothly connect with previous pcd. \r
+                    FixOffsetSizeListCount = lenOfList\r
+\r
+            # No free space, smoothly connect with previous pcd.\r
             elif LastOffset == NowOffset :\r
                 LastOffset = NowOffset + eachFixedPcd.PcdOccupySize\r
                 FixOffsetSizeListCount += 1\r
-            # Usually it will not enter into this thunk, if so, means it overlapped. \r
+            # Usually it will not enter into this thunk, if so, means it overlapped.\r
             else :\r
                 EdkLogger.error("BPDG", BuildToolError.ATTRIBUTE_NOT_AVAILABLE,\r
                                 "The offset value definition has overlapped at pcd: %s, it's offset is: %s, in file: %s line: %s" % \\r
                                 (eachFixedPcd.PcdCName, eachFixedPcd.PcdOffset, eachFixedPcd.InputFileName, eachFixedPcd.Lineno),\r
                                 None)\r
                 FixOffsetSizeListCount += 1\r
-        \r
-        # Continue to process the un-fixed offset pcd's list, add this time, just append them behind the fixed pcd's offset list.    \r
+\r
+        # Continue to process the un-fixed offset pcd's list, add this time, just append them behind the fixed pcd's offset list.\r
         lenOfUnfixedList  = len(self.PcdUnknownOffsetList)\r
         lenOfList         = len(self.PcdFixedOffsetSizeList)\r
         while (lenOfUnfixedList > 0) :\r
@@ -622,20 +624,23 @@ class GenVPD :
             # The last pcd instance\r
             LastPcd    = self.PcdFixedOffsetSizeList[lenOfList-1]\r
             NeedFixPcd = self.PcdUnknownOffsetList[0]\r
-            \r
+\r
             NeedFixPcd.PcdBinOffset = LastPcd.PcdBinOffset + LastPcd.PcdOccupySize\r
+            if NeedFixPcd.PcdBinOffset % NeedFixPcd.Alignment != 0:\r
+                NeedFixPcd.PcdBinOffset = (NeedFixPcd.PcdBinOffset / NeedFixPcd.Alignment + 1) * NeedFixPcd.Alignment\r
+\r
             NeedFixPcd.PcdOffset    = str(hex(NeedFixPcd.PcdBinOffset))\r
-            \r
+\r
             # Insert this pcd into fixed offset pcd list's tail.\r
             self.PcdFixedOffsetSizeList.insert(lenOfList, NeedFixPcd)\r
             # Delete the item's offset that has been fixed and added into fixed offset list\r
             self.PcdUnknownOffsetList.pop(0)\r
-            \r
+\r
             lenOfList          += 1\r
-            lenOfUnfixedList   -= 1                                                                                                                \r
+            lenOfUnfixedList   -= 1\r
     ##\r
     # Write the final data into output files.\r
-    #   \r
+    #\r
     def GenerateVpdFile (self, MapFileName, BinFileName):\r
         #Open an VPD file to process\r
 \r
@@ -651,8 +656,8 @@ class GenVPD :
             # Open failed\r
             EdkLogger.error("BPDG", BuildToolError.FILE_OPEN_FAILURE, "File open failed for %s" % self.MapFileName, None)\r
 \r
-        # Use a instance of StringIO to cache data\r
-        fStringIO = StringIO.StringIO('')\r
+        # Use a instance of BytesIO to cache data\r
+        fStringIO = BytesIO('')\r
 \r
         # Write the header of map file.\r
         try :\r
@@ -663,7 +668,7 @@ class GenVPD :
         for eachPcd in self.PcdFixedOffsetSizeList  :\r
             # write map file\r
             try :\r
-                fMapFile.write("%s | %s | %s | %s | %s  \n" % (eachPcd.PcdCName, eachPcd.SkuId,eachPcd.PcdOffset, eachPcd.PcdSize,eachPcd.PcdUnpackValue))\r
+                fMapFile.write("%s | %s | %s | %s | %s  \n" % (eachPcd.PcdCName, eachPcd.SkuId, eachPcd.PcdOffset, eachPcd.PcdSize, eachPcd.PcdUnpackValue))\r
             except:\r
                 EdkLogger.error("BPDG", BuildToolError.FILE_WRITE_FAILURE, "Write data to file %s failed, please check whether the file been locked or using by other applications." % self.MapFileName, None)\r
 \r
@@ -683,4 +688,4 @@ class GenVPD :
         fStringIO.close ()\r
         fVpdFile.close ()\r
         fMapFile.close ()\r
-        \r
+\r