Re: [Patch] BaseTools: Fix a bug override Pcd by DSC Components section

["Gao, Liming" <liming.gao@intel.com>]

Reviewed-by: Liming Gao <liming.gao@intel.com>

>-----Original Message-----
>From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
>Yonghong Zhu
>Sent: Monday, February 26, 2018 3:49 PM
>To: edk2-devel@lists.01.org
>Subject: [edk2] [Patch] BaseTools: Fix a bug override Pcd by DSC Components
>section
>
>The case is: define a VOID* pcd in DEC file, eg: Value is {0x1}.
>then override this PCD on DSC component section, eg: Value is
>{0x1, 0x2, 0x3}, the max size of this PCD is calculate wrong
>which cause build error.
>
>Contributed-under: TianoCore Contribution Agreement 1.1
>Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com>
>---
> BaseTools/Source/Python/AutoGen/AutoGen.py        | 15 ++++++++++++++-
> BaseTools/Source/Python/Workspace/DscBuildData.py |  5 +++--
> 2 files changed, 17 insertions(+), 3 deletions(-)
>
>diff --git a/BaseTools/Source/Python/AutoGen/AutoGen.py
>b/BaseTools/Source/Python/AutoGen/AutoGen.py
>index 405bfa1..19c65b3 100644
>--- a/BaseTools/Source/Python/AutoGen/AutoGen.py
>+++ b/BaseTools/Source/Python/AutoGen/AutoGen.py
>@@ -2412,11 +2412,11 @@ class PlatformAutoGen(AutoGen):
>                                 ExtraData="%s.%s" % (ToPcd.TokenSpaceGuidCName,
>TokenCName))
>             ToPcd.validateranges = FromPcd.validateranges
>             ToPcd.validlists = FromPcd.validlists
>             ToPcd.expressions = FromPcd.expressions
>
>-        if ToPcd.DatumType == "VOID*" and ToPcd.MaxDatumSize in ['', None]:
>+        if FromPcd != None and ToPcd.DatumType == "VOID*" and
>ToPcd.MaxDatumSize in ['', None]:
>             EdkLogger.debug(EdkLogger.DEBUG_9, "No MaxDatumSize specified
>for PCD %s.%s" \
>                             % (ToPcd.TokenSpaceGuidCName, TokenCName))
>             Value = ToPcd.DefaultValue
>             if Value in [None, '']:
>                 ToPcd.MaxDatumSize = '1'
>@@ -2485,10 +2485,23 @@ class PlatformAutoGen(AutoGen):
>                             ToPcd = Pcds[PcdItem]
>                             Flag = True
>                             break
>                 if Flag:
>                     self._OverridePcd(ToPcd, PlatformModule.Pcds[Key], Module)
>+        # use PCD value to calculate the MaxDatumSize when it is not specified
>+        for Name, Guid in Pcds:
>+            Pcd = Pcds[Name, Guid]
>+            if Pcd.DatumType == "VOID*" and Pcd.MaxDatumSize in ['', None]:
>+                Value = Pcd.DefaultValue
>+                if Value in [None, '']:
>+                    Pcd.MaxDatumSize = '1'
>+                elif Value[0] == 'L':
>+                    Pcd.MaxDatumSize = str((len(Value) - 2) * 2)
>+                elif Value[0] == '{':
>+                    Pcd.MaxDatumSize = str(len(Value.split(',')))
>+                else:
>+                    Pcd.MaxDatumSize = str(len(Value) - 1)
>         return Pcds.values()
>
>     ## Resolve library names to library modules
>     #
>     # (for Edk.x modules)
>diff --git a/BaseTools/Source/Python/Workspace/DscBuildData.py
>b/BaseTools/Source/Python/Workspace/DscBuildData.py
>index 75b877a..32b1df9 100644
>--- a/BaseTools/Source/Python/Workspace/DscBuildData.py
>+++ b/BaseTools/Source/Python/Workspace/DscBuildData.py
>@@ -677,12 +677,13 @@ class DscBuildData(PlatformBuildClassObject):
>                          MODEL_PCD_FEATURE_FLAG, MODEL_PCD_DYNAMIC,
>MODEL_PCD_DYNAMIC_EX]:
>                 RecordList = self._RawData[Type, self._Arch, None, ModuleId]
>                 for TokenSpaceGuid, PcdCName, Setting, Dummy1, Dummy2,
>Dummy3, Dummy4,Dummy5 in RecordList:
>                     TokenList = GetSplitValueList(Setting)
>                     DefaultValue = TokenList[0]
>-                    if len(TokenList) > 1:
>-                        MaxDatumSize = TokenList[1]
>+                    # the format is PcdName| Value | VOID* | MaxDatumSize
>+                    if len(TokenList) > 2:
>+                        MaxDatumSize = TokenList[2]
>                     else:
>                         MaxDatumSize = ''
>                     TypeString = self._PCD_TYPE_STRING_[Type]
>                     Pcd = PcdClassObject(
>                             PcdCName,
>--
>2.6.1.windows.1
>
>_______________________________________________
>edk2-devel mailing list
>edk2-devel@lists.01.org
>https://lists.01.org/mailman/listinfo/edk2-devel