]> git.proxmox.com Git - mirror_edk2.git/blobdiff - BaseTools/Source/Python/BPDG/GenVpd.py
UefiCpuPkg: Move AsmRelocateApLoopStart from Mpfuncs.nasm to AmdSev.nasm
[mirror_edk2.git] / BaseTools / Source / Python / BPDG / GenVpd.py
index 69a9665f5a761f3a38f2acac022359d545b6c446..049c082f40e2c74c7236205be6e42269fc544136 100644 (file)
@@ -4,22 +4,18 @@
 #\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
-#  which accompanies this distribution.  The full text of the license may be found at\r
-#  http://opensource.org/licenses/bsd-license.php\r
-#\r
-#  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
-#  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
+#  SPDX-License-Identifier: BSD-2-Clause-Patent\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, TAB_STAR\r
 import Common.EdkLogger as EdkLogger\r
 import Common.BuildToolError as BuildToolError\r
 \r
@@ -31,10 +27,10 @@ _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
+    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
@@ -47,7 +43,7 @@ class PcdEntry:
         self.PcdBinOffset   = PcdBinOffset\r
         self.PcdBinSize     = PcdBinSize\r
         self.Alignment       = Alignment\r
-        \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
@@ -63,13 +59,13 @@ class PcdEntry:
         self._GenOffsetValue ()\r
 \r
     ## Analyze the string value to judge the PCD's datum type equal to Boolean or not.\r
-    # \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
@@ -80,12 +76,12 @@ 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
+        if self.PcdOffset != TAB_STAR:\r
             try:\r
                 self.PcdBinOffset = int (self.PcdOffset)\r
             except:\r
@@ -96,10 +92,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
@@ -115,46 +111,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:\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
@@ -170,7 +147,7 @@ class PcdEntry:
     #    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"') or ValueString.startswith("L'"):\r
             self._PackUnicode(ValueString, Size)\r
@@ -183,9 +160,9 @@ class PcdEntry:
                             "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
+    #\r
     #  An Ascii string for a PCD should be in format as  ""/''.\r
-    #                   \r
+    #\r
     def _PackString(self, ValueString, Size):\r
         if (Size < 0):\r
             EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID,\r
@@ -198,20 +175,20 @@ class PcdEntry:
             QuotedFlag = False\r
 \r
         ValueString = ValueString[1:-1]\r
-        # No null-terminator in 'string' \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
-            self.PcdValue = pack('%ds' % Size, ValueString)\r
+            self.PcdValue = pack('%ds' % Size, ValueString.encode('utf-8'))\r
         except:\r
             EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID,\r
                             "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
@@ -229,7 +206,7 @@ class PcdEntry:
 \r
         ReturnArray = array.array('B')\r
 \r
-        for Index in xrange(len(ValueList)):\r
+        for Index in range(len(ValueList)):\r
             Value = None\r
             if ValueList[Index].lower().startswith('0x'):\r
                 # translate hex value\r
@@ -255,13 +232,13 @@ class PcdEntry:
 \r
             ReturnArray.append(Value)\r
 \r
-        for Index in xrange(len(ValueList), Size):\r
+        for Index in range(len(ValueList), Size):\r
             ReturnArray.append(0)\r
 \r
         self.PcdValue = ReturnArray.tolist()\r
 \r
     ## Pack a unicode PCD value into byte array.\r
-    #  \r
+    #\r
     #  A unicode string for a PCD should be in format as  L""/L''.\r
     #\r
     def _PackUnicode(self, UnicodeString, Size):\r
@@ -271,7 +248,7 @@ class PcdEntry:
 \r
         QuotedFlag = True\r
         if UnicodeString.startswith("L'"):\r
-            QuotedFlag = False \r
+            QuotedFlag = False\r
         UnicodeString = UnicodeString[2:-1]\r
 \r
         # No null-terminator in L'string'\r
@@ -290,7 +267,7 @@ class PcdEntry:
                                 "Invalid unicode character %s in unicode string %s(File: %s Line: %s)" % \\r
                                 (Value, UnicodeString, self.FileName, self.Lineno))\r
 \r
-        for Index in xrange(len(UnicodeString) * 2, Size):\r
+        for Index in range(len(UnicodeString) * 2, Size):\r
             ReturnArray.append(0)\r
 \r
         self.PcdValue = ReturnArray.tolist()\r
@@ -304,7 +281,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
@@ -323,7 +300,7 @@ class GenVPD :
         self.PcdFixedOffsetSizeList  = []\r
         self.PcdUnknownOffsetList    = []\r
         try:\r
-            fInputfile = open(InputFileName, "r", 0)\r
+            fInputfile = open(InputFileName, "r")\r
             try:\r
                 self.FileLinesList = fInputfile.readlines()\r
             except:\r
@@ -334,9 +311,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
-    # from string to it's real format. Also remove the useless line in the input file.\r
-    # \r
+    # Parser the input file which is generated by the build tool. Convert the value of each pcd's\r
+    # from string to its real format. Also remove the useless line in the input file.\r
+    #\r
     def ParserInputFile (self):\r
         count = 0\r
         for line in self.FileLinesList:\r
@@ -349,7 +326,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
@@ -390,7 +367,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
@@ -399,7 +376,7 @@ class GenVPD :
         count = 0\r
         for line in self.FileLinesList:\r
             if line is not None :\r
-                PCD = PcdEntry(line[0], line[1], line[2], line[3], line[4],line[5], self.InputFileName)   \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
@@ -440,7 +417,7 @@ class GenVPD :
                     Alignment = 1\r
 \r
                 PCD.Alignment = Alignment\r
-                if PCD.PcdOffset != '*':\r
+                if PCD.PcdOffset != TAB_STAR:\r
                     if PCD.PcdOccupySize % Alignment != 0:\r
                         if PCD.PcdUnpackValue.startswith("{"):\r
                             EdkLogger.warn("BPDG", "The offset value of PCD %s is not 8-byte aligned!" %(PCD.PcdCName), File=self.InputFileName)\r
@@ -448,7 +425,7 @@ class GenVPD :
                             EdkLogger.error("BPDG", BuildToolError.FORMAT_INVALID, 'The offset value of PCD %s should be %s-byte aligned.' % (PCD.PcdCName, Alignment))\r
                 else:\r
                     if PCD.PcdOccupySize % Alignment != 0:\r
-                        PCD.PcdOccupySize = (PCD.PcdOccupySize / Alignment + 1) * Alignment\r
+                        PCD.PcdOccupySize = (PCD.PcdOccupySize // Alignment + 1) * Alignment\r
 \r
                 PackSize = PCD.PcdOccupySize\r
                 if PCD._IsBoolean(PCD.PcdValue, PCD.PcdSize):\r
@@ -480,14 +457,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
+            if eachPcd.PcdOffset != TAB_STAR :\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
@@ -497,90 +474,90 @@ 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 TAB_STAR 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
+        # Sort the un-fixed pcd's offset by its 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 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
+                        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
+        # Process all Offset value are TAB_STAR\r
         #\r
         if (len(self.PcdFixedOffsetSizeList) == 0) and (len(self.PcdUnknownOffsetList) != 0) :\r
             # 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
+                    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
@@ -588,52 +565,52 @@ class GenVPD :
                         eachUnfixedPcd      = self.PcdUnknownOffsetList[countOfUnfixedList]\r
                         needFixPcdSize      = eachUnfixedPcd.PcdOccupySize\r
                         # Not been fixed\r
-                        if eachUnfixedPcd.PcdOffset == '*' :\r
+                        if eachUnfixedPcd.PcdOffset == TAB_STAR :\r
                             if LastOffset % eachUnfixedPcd.Alignment != 0:\r
-                                LastOffset = (LastOffset / eachUnfixedPcd.Alignment + 1) * eachUnfixedPcd.Alignment\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
+                                "The offset value definition has overlapped at pcd: %s, its 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
@@ -641,40 +618,40 @@ 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
+                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
         try:\r
-            fVpdFile = open(BinFileName, "wb", 0)\r
+            fVpdFile = open(BinFileName, "wb")\r
         except:\r
             # Open failed\r
             EdkLogger.error("BPDG", BuildToolError.FILE_OPEN_FAILURE, "File open failed for %s" % self.VpdFileName, None)\r
 \r
         try :\r
-            fMapFile = open(MapFileName, "w", 0)\r
+            fMapFile = open(MapFileName, "w")\r
         except:\r
             # 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
@@ -685,15 +662,19 @@ 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
             # Write Vpd binary file\r
             fStringIO.seek (eachPcd.PcdBinOffset)\r
             if isinstance(eachPcd.PcdValue, list):\r
-                ValueList = [chr(Item) for Item in eachPcd.PcdValue]\r
-                fStringIO.write(''.join(ValueList))\r
+                for i in range(len(eachPcd.PcdValue)):\r
+                    Value = eachPcd.PcdValue[i:i + 1]\r
+                    if isinstance(bytes(Value), str):\r
+                        fStringIO.write(chr(Value[0]))\r
+                    else:\r
+                        fStringIO.write(bytes(Value))\r
             else:\r
                 fStringIO.write (eachPcd.PcdValue)\r
 \r
@@ -705,4 +686,4 @@ class GenVPD :
         fStringIO.close ()\r
         fVpdFile.close ()\r
         fMapFile.close ()\r
-        \r
+\r