]> git.proxmox.com Git - mirror_edk2.git/blobdiff - Tools/Source/FrameworkWizard/src/org/tianocore/frameworkwizard/platform/ui/FpdFileContents.java
draft UI implementation of module orders in FV.
[mirror_edk2.git] / Tools / Source / FrameworkWizard / src / org / tianocore / frameworkwizard / platform / ui / FpdFileContents.java
index 6d9235cdc87dc5334de95d33a127bba75dedd52f..e60426446fbe826b66cf251432da64e9b5db78e4 100644 (file)
@@ -40,6 +40,7 @@ import org.tianocore.FrameworkModulesDocument;
 import org.tianocore.IntermediateOutputType;\r
 import org.tianocore.LibrariesDocument;\r
 import org.tianocore.ModuleSADocument;\r
+import org.tianocore.ModuleSaBuildOptionsDocument;\r
 import org.tianocore.ModuleSurfaceAreaDocument;\r
 import org.tianocore.OptionDocument;\r
 import org.tianocore.OptionsDocument;\r
@@ -56,10 +57,11 @@ import org.tianocore.LicenseDocument;
 import org.tianocore.PlatformHeaderDocument;\r
 import org.tianocore.SkuInfoDocument;\r
 import org.tianocore.UserDefinedAntTasksDocument;\r
-import org.tianocore.frameworkwizard.platform.ui.global.GlobalData;\r
+import org.tianocore.UserExtensionsDocument;\r
+import org.tianocore.frameworkwizard.platform.ui.global.WorkspaceProfile;\r
 import org.tianocore.frameworkwizard.platform.ui.global.SurfaceAreaQuery;\r
-import org.tianocore.frameworkwizard.platform.ui.id.ModuleIdentification;\r
-import org.tianocore.frameworkwizard.platform.ui.id.PackageIdentification;\r
+import org.tianocore.frameworkwizard.module.Identifications.ModuleIdentification;\r
+import org.tianocore.frameworkwizard.packaging.PackageIdentification;\r
 \r
 /**\r
  This class processes fpd file contents such as add remove xml elements. \r
@@ -85,7 +87,9 @@ public class FpdFileContents {
     \r
     private DynamicPcdBuildDefinitionsDocument.DynamicPcdBuildDefinitions fpdDynPcdBuildDefs = null;\r
     \r
-    public static HashMap<String, ArrayList<String>> dynPcdMap = null;\r
+    private HashMap<String, ArrayList<String>> dynPcdMap = null;\r
+    \r
+    private HashMap<String, String> defaultPcdValue = new HashMap<String, String>();\r
     \r
     /**\r
      * look through all pcd data in all ModuleSA, create pcd -> ModuleSA mappings.\r
@@ -95,17 +99,19 @@ public class FpdFileContents {
           dynPcdMap = new HashMap<String, ArrayList<String>>();\r
           List<ModuleSADocument.ModuleSA> l = getfpdFrameworkModules().getModuleSAList();\r
           if (l == null) {\r
+              removeElement(getfpdFrameworkModules());\r
+              fpdFrameworkModules = null;\r
               return;\r
           }\r
           ListIterator<ModuleSADocument.ModuleSA> li = l.listIterator();\r
           while (li.hasNext()) {\r
-              ModuleSADocument.ModuleSA msa = li.next();\r
-              if (msa.getPcdBuildDefinition() == null || msa.getPcdBuildDefinition().getPcdDataList() == null) {\r
+              ModuleSADocument.ModuleSA moduleSa = li.next();\r
+              if (moduleSa.getPcdBuildDefinition() == null || moduleSa.getPcdBuildDefinition().getPcdDataList() == null) {\r
                   continue;\r
               }\r
-              String ModuleInfo = msa.getModuleGuid() + " " + msa.getModuleVersion() +\r
-               " " + msa.getPackageGuid() + " " + msa.getPackageVersion();\r
-              List<PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData> lp = msa.getPcdBuildDefinition().getPcdDataList();\r
+              String ModuleInfo = moduleSa.getModuleGuid() + " " + moduleSa.getModuleVersion() +\r
+               " " + moduleSa.getPackageGuid() + " " + moduleSa.getPackageVersion() + " " + listToString(moduleSa.getSupArchList());\r
+              List<PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData> lp = moduleSa.getPcdBuildDefinition().getPcdDataList();\r
               ListIterator<PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData> lpi = lp.listIterator();\r
               while (lpi.hasNext()) {\r
                   PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData pcdData = lpi.next();\r
@@ -122,6 +128,10 @@ public class FpdFileContents {
           }\r
       }\r
     }\r
+    \r
+    public ArrayList<String> getDynPcdMapValue(String key) {\r
+        return dynPcdMap.get(key);\r
+    }\r
     /**\r
      Constructor to create a new spd file\r
      **/\r
@@ -174,8 +184,36 @@ public class FpdFileContents {
         return fpdFrameworkModules;\r
     }\r
     \r
+    public void getFrameworkModuleGuid (String fvName, Vector<String> vGuid) {\r
+        if (getFrameworkModulesCount() == 0){\r
+            return;\r
+        }\r
+        \r
+        ListIterator li = getfpdFrameworkModules().getModuleSAList().listIterator();\r
+        while(li.hasNext()) {\r
+            ModuleSADocument.ModuleSA moduleSa = (ModuleSADocument.ModuleSA)li.next();\r
+            if (moduleSa.getModuleSaBuildOptions() == null) {\r
+                continue;\r
+            }\r
+            String fvBinding = moduleSa.getModuleSaBuildOptions().getFvBinding();\r
+            if (fvBinding == null) {\r
+                continue;\r
+            }\r
+            \r
+            String[] fvNames = fvBinding.split(" ");\r
+            for (int i = 0; i < fvNames.length; ++i) {\r
+                if (fvNames[i].equals(fvName) || fvNames[i].replaceAll("_", "").equals(fvName)) {\r
+                    vGuid.add(moduleSa.getModuleGuid());\r
+                    break;\r
+                }\r
+            }\r
+        }\r
+    }\r
+    \r
     public int getFrameworkModulesCount() {\r
-        if (getfpdFrameworkModules().getModuleSAList() == null){\r
+        if (getfpdFrameworkModules().getModuleSAList() == null || getfpdFrameworkModules().getModuleSAList().size() == 0){\r
+            removeElement(getfpdFrameworkModules());\r
+            fpdFrameworkModules = null;\r
             return 0;\r
         }\r
         return getfpdFrameworkModules().getModuleSAList().size();\r
@@ -189,53 +227,80 @@ public class FpdFileContents {
         ListIterator li = getfpdFrameworkModules().getModuleSAList().listIterator();\r
         int i = 0;\r
         while(li.hasNext()) {\r
-            ModuleSADocument.ModuleSA msa = (ModuleSADocument.ModuleSA)li.next();\r
-            saa[i][1] = msa.getModuleGuid();\r
-            saa[i][2] = msa.getModuleVersion();\r
+            ModuleSADocument.ModuleSA moduleSa = (ModuleSADocument.ModuleSA)li.next();\r
+            saa[i][0] = moduleSa.getModuleGuid();\r
+            saa[i][1] = moduleSa.getModuleVersion();\r
             \r
-            saa[i][3] = msa.getPackageGuid();\r
-            saa[i][4] = msa.getPackageVersion();\r
-//            saa[i][4] = listToString(msa.getSupArchList());\r
+            saa[i][2] = moduleSa.getPackageGuid();\r
+            saa[i][3] = moduleSa.getPackageVersion();\r
+            saa[i][4] = listToString(moduleSa.getSupArchList());\r
             ++i;\r
         }\r
     }\r
     \r
+    public void getFrameworkModuleInfo(int i, String[] sa) {\r
+        ModuleSADocument.ModuleSA msa = getModuleSA(i);\r
+        if (msa == null) {\r
+            return;\r
+        }\r
+        sa[0] = msa.getModuleGuid();\r
+        sa[1] = msa.getModuleVersion();\r
+        sa[2] = msa.getPackageGuid();\r
+        sa[3] = msa.getPackageVersion();\r
+        sa[4] = listToString(msa.getSupArchList());\r
+    }\r
+    \r
     public ModuleSADocument.ModuleSA getModuleSA(String key) {\r
         String[] s = key.split(" ");\r
-        if (getfpdFrameworkModules().getModuleSAList() == null) {\r
+        if (getfpdFrameworkModules().getModuleSAList() == null || getfpdFrameworkModules().getModuleSAList().size() == 0) {\r
+            removeElement(getfpdFrameworkModules());\r
+            fpdFrameworkModules = null;\r
             return null;\r
         }\r
         ListIterator li = getfpdFrameworkModules().getModuleSAList().listIterator();\r
         while(li.hasNext()) {\r
-            ModuleSADocument.ModuleSA msa = (ModuleSADocument.ModuleSA)li.next();\r
-            if (msa.getModuleGuid().equals(s[0]) && msa.getPackageGuid().equals(s[2])) {\r
-//                if (msa.getModuleVersion() != null) {\r
-//                    if (!msa.getModuleVersion().equals(s[1])) {\r
-//                        continue;\r
-//                    }\r
-//                }\r
-//                else{\r
-//                    if (s[1] != null) {\r
-//                        continue;\r
-//                    }\r
-//                }\r
-//                if (msa.getPackageVersion() != null) {\r
-//                    if (!msa.getPackageVersion().equals(s[3])) {\r
-//                        continue;\r
-//                    }\r
-//                }\r
-//                else{\r
-//                    if (s[3] != null) {\r
-//                        continue;\r
-//                    }\r
-//                }\r
-                return msa;\r
+            ModuleSADocument.ModuleSA moduleSa = (ModuleSADocument.ModuleSA)li.next();\r
+            if (moduleSa.getModuleGuid().equalsIgnoreCase(s[0]) && moduleSa.getPackageGuid().equalsIgnoreCase(s[2])) {\r
+                if (moduleSa.getModuleVersion() != null) {\r
+                    if (!moduleSa.getModuleVersion().equals(s[1])) {\r
+                        continue;\r
+                    }\r
+                }\r
+                if (moduleSa.getPackageVersion() != null) {\r
+                    if (!moduleSa.getPackageVersion().equals(s[3])) {\r
+                        continue;\r
+                    }\r
+                }\r
+                //ToDo add arch check for s[4]\r
+                if (moduleSa.getSupArchList() != null) {\r
+                    if (!listToString(moduleSa.getSupArchList()).equals(s[4])) {\r
+                        continue;\r
+                    }\r
+                }\r
+                return moduleSa;\r
             }\r
         }\r
         return null;\r
     }\r
+    \r
+    private ModuleSADocument.ModuleSA getModuleSA(int i) {\r
+        ModuleSADocument.ModuleSA moduleSa = null;\r
+        if (fpdRoot.getFrameworkModules() == null) {\r
+            return null;\r
+        }\r
+        XmlCursor cursor = fpdRoot.getFrameworkModules().newCursor();\r
+        if (cursor.toFirstChild()) {\r
+            for (int j = 0; j < i; ++j) {\r
+                cursor.toNextSibling();\r
+            }\r
+            moduleSa = (ModuleSADocument.ModuleSA)cursor.getObject();\r
+        }\r
+        cursor.dispose();\r
+        return moduleSa;\r
+    }\r
+    \r
     public void removeModuleSA(int i) {\r
-        XmlObject o = getfpdFrameworkModules();\r
+        XmlObject o = fpdRoot.getFrameworkModules();\r
         if (o == null) {\r
             return;\r
         }\r
@@ -250,113 +315,354 @@ public class FpdFileContents {
             //\r
             ModuleSADocument.ModuleSA moduleSa = (ModuleSADocument.ModuleSA)cursor.getObject();\r
             String moduleInfo = moduleSa.getModuleGuid() + " " + moduleSa.getModuleVersion() + " " +\r
-            moduleSa.getPackageGuid()+ " " + moduleSa.getPackageVersion();\r
+            moduleSa.getPackageGuid()+ " " + moduleSa.getPackageVersion() + " " + listToString(moduleSa.getSupArchList());\r
             PcdBuildDefinitionDocument.PcdBuildDefinition pcdBuildDef = moduleSa.getPcdBuildDefinition();\r
-            if (pcdBuildDef != null) {\r
-                maintainDynPcdMap(pcdBuildDef, moduleInfo);\r
+            if (pcdBuildDef != null && pcdBuildDef.getPcdDataList() != null) {\r
+                ListIterator<PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData> li = pcdBuildDef.getPcdDataList().listIterator();\r
+                while(li.hasNext()) {\r
+                    PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData pcdData = li.next();\r
+                    maintainDynPcdMap(pcdData.getCName() + " " + pcdData.getTokenSpaceGuidCName(), moduleInfo);\r
+                }\r
+            }\r
+            \r
+            cursor.push();\r
+            cursor.toPrevToken();\r
+            if (cursor.isComment()) {\r
+                cursor.removeXml();\r
             }\r
+            cursor.pop();\r
             cursor.removeXml();\r
+            if (getFrameworkModulesCount() == 0) {\r
+                cursor.toParent();\r
+                cursor.removeXml();\r
+            }\r
         }\r
         cursor.dispose();\r
     }\r
     \r
-    private void maintainDynPcdMap(PcdBuildDefinitionDocument.PcdBuildDefinition o, String moduleInfo) {\r
-        XmlCursor cursor = o.newCursor();\r
-        boolean fromLibInstance = false;\r
-        if (!cursor.toFirstChild()){\r
-            return;\r
+    public boolean adjustPcd (int seqModuleSa) throws Exception {\r
+        boolean dataModified = false;\r
+        ModuleSADocument.ModuleSA moduleSa = getModuleSA(seqModuleSa);\r
+        int pcdCount = getPcdDataCount(seqModuleSa);\r
+        String[][] saaModuleSaPcd = new String[pcdCount][7];\r
+        getPcdData(seqModuleSa, saaModuleSaPcd);\r
+        String mg = moduleSa.getModuleGuid();\r
+        String mv = moduleSa.getModuleVersion();\r
+        String pg = moduleSa.getPackageGuid();\r
+        String pv = moduleSa.getPackageVersion();\r
+        String arch = listToString(moduleSa.getSupArchList());\r
+        //\r
+        // delete pcd in ModuleSA but not in MSA files any longer.\r
+        //\r
+        String moduleKey = mg + " " + mv + " " + pg + " " + pv + " " + arch;\r
+        int libCount = getLibraryInstancesCount(moduleKey);\r
+        String[][] saaLib = new String[libCount][5];\r
+        getLibraryInstances(moduleKey, saaLib);\r
+        ModuleIdentification mi = WorkspaceProfile.getModuleId(moduleKey);\r
+        Vector<ModuleIdentification> vMi = new Vector<ModuleIdentification>();\r
+        vMi.add(mi);\r
+        try {\r
+    nextPcd:for (int i = 0; i < saaModuleSaPcd.length; ++i) {\r
+                if (WorkspaceProfile.pcdInMsa(saaModuleSaPcd[i][0], saaModuleSaPcd[i][1], mi)){\r
+                    continue;\r
+                }\r
+                for (int j = 0; j < saaLib.length; ++j) {\r
+                    String libKey = saaLib[j][1] + " " + saaLib[j][2] + " " + saaLib[j][3] + " " + saaLib[j][4];\r
+                    ModuleIdentification libMi = WorkspaceProfile.getModuleId(libKey);\r
+                    vMi.add(libMi);\r
+                    if (WorkspaceProfile.pcdInMsa(saaModuleSaPcd[i][0], saaModuleSaPcd[i][1], libMi)) {\r
+                        continue nextPcd;\r
+                    }\r
+                }\r
+                removePcdData(seqModuleSa, saaModuleSaPcd[i][0], saaModuleSaPcd[i][1]);\r
+                dataModified = true;\r
+            }\r
+        }\r
+        catch (Exception e) {\r
+            \r
         }\r
         //\r
-        // deal with first child, same process in the while loop below for siblings.\r
+        // add new Pcd from MSA file to ModuleSA.\r
         //\r
-        PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData pcdData = (PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData)cursor.getObject();\r
-        String pcdKey = pcdData.getCName() + " " + pcdData.getTokenSpaceGuidCName();\r
+        try {\r
+       \r
+            for (int i = 0; i < vMi.size(); ++i) {\r
+                ModuleSurfaceAreaDocument.ModuleSurfaceArea msa = (ModuleSurfaceAreaDocument.ModuleSurfaceArea) WorkspaceProfile\r
+                                                                                                                          .getModuleXmlObject(vMi\r
+                                                                                                                                                 .get(i));\r
+                if (msa.getPcdCoded() == null || msa.getPcdCoded().getPcdEntryList() == null) {\r
+                    continue;\r
+                }\r
+                ListIterator li = msa.getPcdCoded().getPcdEntryList().listIterator();\r
+     msaPcdIter:while (li.hasNext()) {\r
+                    PcdCodedDocument.PcdCoded.PcdEntry msaPcd = (PcdCodedDocument.PcdCoded.PcdEntry) li.next();\r
+                    ArrayList<String> al = getDynPcdMapValue(msaPcd.getCName() + " " + msaPcd.getTokenSpaceGuidCName());\r
+                    if (al != null) {\r
+                        for (int j = 0; j < al.size(); ++j) {\r
+                            if (al.get(j).contains(moduleKey)) {\r
+                                continue msaPcdIter;\r
+                            }\r
+                        }\r
+                    }\r
+                    \r
+                    Map<String, XmlObject> m = new HashMap<String, XmlObject>();\r
+                    m.put("ModuleSurfaceArea", msa);\r
+                    SurfaceAreaQuery.setDoc(m);\r
+                    PackageIdentification[] depPkgs = SurfaceAreaQuery.getDependencePkg(null, vMi.get(i));\r
+                    PcdDeclarationsDocument.PcdDeclarations.PcdEntry spdPcd = LookupPcdDeclaration(msaPcd, depPkgs);\r
+                    if (spdPcd == null) {\r
+                        //\r
+                        // ToDo Error \r
+                        //\r
+                        throw new PcdDeclNotFound("No Declaration for PCD Entry " + msaPcd.getCName() + " in Module "\r
+                                                  + mi.getName());\r
+                    }\r
+                    //\r
+                    // AddItem to ModuleSA PcdBuildDefinitions\r
+                    //\r
+                    String defaultVal = msaPcd.getDefaultValue() == null ? spdPcd.getDefaultValue()\r
+                                                                        : msaPcd.getDefaultValue();\r
+\r
+                    genPcdData(msaPcd.getCName(), spdPcd.getToken(), msaPcd.getTokenSpaceGuidCName(),\r
+                               msaPcd.getPcdItemType().toString(), spdPcd.getDatumType() + "", defaultVal, moduleSa);\r
+                    dataModified = true;\r
+                 }\r
+\r
+            }\r
+        }\r
+        catch (Exception e){\r
+           throw e;\r
+        }\r
+        \r
+        return dataModified;\r
+    }\r
+    \r
+    private void maintainDynPcdMap(String pcdKey, String moduleInfo) {\r
+        \r
         ArrayList<String> al = dynPcdMap.get(pcdKey);\r
+        if (al == null) {\r
+            return;\r
+        }\r
+        String[] s = moduleInfo.split(" ");\r
         for(int i = 0; i < al.size(); ++i){\r
-            if (al.get(i).startsWith(moduleInfo)){\r
-                fromLibInstance = true;\r
+            String consumer = al.get(i);\r
+            if (consumer.contains(s[0]) && consumer.contains(s[2])){\r
+                String[] consumerPart = consumer.split(" ");\r
+                if (!consumerPart[4].equals(s[4])) {\r
+                    continue;\r
+                }\r
+                al.remove(consumer);\r
                 break;\r
             }\r
         }\r
-        al.remove(moduleInfo + " " + pcdData.getItemType().toString());\r
+\r
         if (al.size() == 0) {\r
+            defaultPcdValue.remove(pcdKey);\r
             dynPcdMap.remove(pcdKey);\r
+            String[] s1 = pcdKey.split(" ");\r
+            removeDynamicPcdBuildData(s1[0], s1[1]);\r
+        }\r
+        \r
+    }\r
+    //\r
+    // key for ModuleSA : "ModuleGuid ModuleVer PackageGuid PackageVer Arch"\r
+    //\r
+    public int getPcdDataCount (int i){\r
+        ModuleSADocument.ModuleSA msa = getModuleSA(i);\r
+        \r
+        if (msa == null || msa.getPcdBuildDefinition() == null || msa.getPcdBuildDefinition().getPcdDataList() == null){\r
+            return 0;\r
+        }\r
+        return msa.getPcdBuildDefinition().getPcdDataList().size();\r
+        \r
+    }\r
+    \r
+    public void getPcdData (int i, String[][] saa) {\r
+        ModuleSADocument.ModuleSA msa = getModuleSA(i);\r
+        \r
+        if (msa == null || msa.getPcdBuildDefinition() == null || msa.getPcdBuildDefinition().getPcdDataList() == null){\r
+            return;\r
+        }\r
+        ListIterator<PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData>li = msa.getPcdBuildDefinition().getPcdDataList().listIterator();\r
+        for (int k = 0; k < saa.length; ++k) {\r
+            PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData pcdData = li.next();\r
+            saa[k][0] = pcdData.getCName();\r
+            saa[k][1] = pcdData.getTokenSpaceGuidCName();\r
+            saa[k][2] = pcdData.getItemType()+"";\r
+            saa[k][3] = pcdData.getToken().toString();\r
+            saa[k][4] = pcdData.getMaxDatumSize()+"";\r
+            saa[k][5] = pcdData.getDatumType()+"";\r
+            saa[k][6] = pcdData.getValue();\r
+            \r
+        }\r
+    }\r
+    \r
+    public void removePcdData (int seqModuleSa, String cName, String tsGuid) {\r
+        ModuleSADocument.ModuleSA moduleSa = getModuleSA(seqModuleSa);\r
+        if (moduleSa == null || moduleSa.getPcdBuildDefinition() == null){\r
+            return;\r
         }\r
         \r
-        if (pcdData.getItemType().toString().equals("DYNAMIC")) {\r
-            if (dynPcdMap.get(pcdKey) == null) {\r
-                removeDynamicPcdBuildData(pcdData.getCName(), pcdData.getTokenSpaceGuidCName());\r
+        String mg = moduleSa.getModuleGuid();\r
+        String mv = moduleSa.getModuleVersion();\r
+        String pg = moduleSa.getPackageGuid();\r
+        String pv = moduleSa.getPackageVersion();\r
+        String arch = listToString(moduleSa.getSupArchList());\r
+        String moduleKey = mg + " " + mv + " " + pg + " " + pv + " " + arch;\r
+        \r
+        XmlCursor cursor = moduleSa.getPcdBuildDefinition().newCursor();\r
+        if (cursor.toFirstChild()){\r
+            \r
+            do {\r
+                PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData pcdData = (PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData)cursor.getObject();\r
+                if (pcdData.getCName().equals(cName) && pcdData.getTokenSpaceGuidCName().equals(tsGuid)) {\r
+                    maintainDynPcdMap(cName + " " + tsGuid, moduleKey);\r
+                    if (getPcdDataCount(seqModuleSa) == 1) {\r
+                        cursor.toParent();\r
+                    }\r
+                    cursor.removeXml();\r
+                    break;\r
+                }\r
             }\r
+            while(cursor.toNextSibling());\r
+            \r
         }\r
-        if (fromLibInstance){\r
-            cursor.removeXml();\r
+        cursor.dispose();\r
+    }\r
+    \r
+    public void updatePcdData (String key, String cName, String tsGuid, String itemType, String maxSize, String value){\r
+        ModuleSADocument.ModuleSA moduleSa = getModuleSA(key);\r
+        if (moduleSa == null || moduleSa.getPcdBuildDefinition() == null){\r
+            return;\r
         }\r
-        while(cursor.toNextSibling()) {\r
-            fromLibInstance = false;\r
-            pcdData = (PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData)cursor.getObject();\r
-            //\r
-            // remove each pcd record from dynPcdMap\r
-            //\r
-            pcdKey = pcdData.getCName() + " " + pcdData.getTokenSpaceGuidCName();\r
-            al = dynPcdMap.get(pcdKey);\r
-            for(int i = 0; i < al.size(); ++i){\r
-                if (al.get(i).startsWith(moduleInfo)){\r
-                    fromLibInstance = true;\r
+        \r
+        XmlCursor cursor = moduleSa.getPcdBuildDefinition().newCursor();\r
+        if (cursor.toFirstChild()){\r
+            do {\r
+                PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData pcdData = (PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData)cursor.getObject();\r
+                if (pcdData.getCName().equals(cName) && pcdData.getTokenSpaceGuidCName().equals(tsGuid)) {\r
+                    pcdData.setItemType(PcdItemTypes.Enum.forString(itemType));\r
+                    if(pcdData.getDatumType().equals("VOID*")) {\r
+                        pcdData.setMaxDatumSize(new Integer(maxSize));\r
+                    }\r
+                    pcdData.setValue(value);\r
+                    defaultPcdValue.put(cName + " " + tsGuid, value);\r
                     break;\r
                 }\r
             }\r
-            al.remove(moduleInfo + " " + pcdData.getItemType().toString());\r
-            if (al.size() == 0) {\r
-                dynPcdMap.remove(pcdKey);\r
+            while(cursor.toNextSibling());\r
+        }\r
+        cursor.dispose();\r
+    }\r
+    \r
+    /**Get original Pcd info from MSA & SPD files.\r
+     * @param mi ModuleIdentification from which MSA & SPD come\r
+     * @param cName PCD cName\r
+     * @param sa Results: HelpText, Original item type.\r
+     * @return\r
+     */\r
+    public boolean getPcdBuildDataInfo(ModuleIdentification mi, String cName, String tsGuid, String[] sa) throws Exception{\r
+        try {\r
+           \r
+            ModuleSurfaceAreaDocument.ModuleSurfaceArea msa = (ModuleSurfaceAreaDocument.ModuleSurfaceArea)WorkspaceProfile.getModuleXmlObject(mi);\r
+            if (msa.getPcdCoded() == null) {\r
+                return false;\r
             }\r
             \r
-            if (pcdData.getItemType().toString().equals("DYNAMIC")) {\r
-                //\r
-                // First check whether this is the only consumer of this dyn pcd.\r
-                //\r
-                if (dynPcdMap.get(pcdKey) == null) {\r
+            Map<String, XmlObject> m = new HashMap<String, XmlObject>();\r
+            m.put("ModuleSurfaceArea", msa);\r
+            SurfaceAreaQuery.setDoc(m);\r
+            PackageIdentification[] depPkgs = SurfaceAreaQuery.getDependencePkg(null, mi);\r
+            //\r
+            // First look through MSA pcd entries.\r
+            //\r
+            List<PcdCodedDocument.PcdCoded.PcdEntry> l = msa.getPcdCoded().getPcdEntryList();\r
+            ListIterator li = l.listIterator();\r
+            while(li.hasNext()) {\r
+                PcdCodedDocument.PcdCoded.PcdEntry msaPcd = (PcdCodedDocument.PcdCoded.PcdEntry)li.next();\r
+                if (!msaPcd.getCName().equals(cName)) {\r
+                    continue;\r
+                }\r
+                if (!msaPcd.getTokenSpaceGuidCName().equals(tsGuid)) {\r
+                    continue;\r
+                }\r
+                PcdDeclarationsDocument.PcdDeclarations.PcdEntry spdPcd = LookupPcdDeclaration(msaPcd, depPkgs);\r
+                if (spdPcd == null) {\r
                     //\r
-                    // delete corresponding entry in DynamicPcdBuildData\r
+                    // ToDo Error \r
                     //\r
-                    removeDynamicPcdBuildData(pcdData.getCName(), pcdData.getTokenSpaceGuidCName());\r
+                    throw new PcdDeclNotFound(mi.getName() + " " + msaPcd.getCName());\r
                 }\r
+                //\r
+                // Get Pcd help text and original item type.\r
+                //\r
+                sa[0] = spdPcd.getHelpText() + msaPcd.getHelpText();\r
+                sa[1] = msaPcd.getPcdItemType()+"";\r
+                return true;\r
             }\r
-            if (fromLibInstance){\r
-                cursor.removeXml();\r
-            }\r
+            \r
+            \r
         }\r
-    }\r
-    //\r
-    // key for ModuleSA : "ModuleGuid ModuleVer PackageGuid PackageVer"\r
-    //\r
-    public int getPcdDataCount(String key){\r
-        ModuleSADocument.ModuleSA msa = getModuleSA(key);\r
-        if (msa == null || msa.getPcdBuildDefinition() == null || msa.getPcdBuildDefinition().getPcdDataList() == null){\r
-            return 0;\r
+        catch (Exception e){\r
+            e.printStackTrace();\r
+            throw e;\r
         }\r
-        return msa.getPcdBuildDefinition().getPcdDataList().size();\r
+        \r
+        return false;\r
     }\r
     \r
-    public void getPcdData(String key, String[][] saa) {\r
-        ModuleSADocument.ModuleSA msa = getModuleSA(key);\r
-        if (msa == null || msa.getPcdBuildDefinition() == null || msa.getPcdBuildDefinition().getPcdDataList() == null){\r
-            return;\r
+    /**Remove PCDBuildDefinition entries from ModuleSA\r
+     * @param moduleKey identifier of ModuleSA.\r
+     * @param consumer where these entries come from.\r
+     */\r
+    public void removePcdData(String moduleKey, ModuleIdentification consumer) {\r
+        try {\r
+            ModuleSurfaceAreaDocument.ModuleSurfaceArea msa = (ModuleSurfaceAreaDocument.ModuleSurfaceArea)WorkspaceProfile.getModuleXmlObject(consumer);\r
+            if (msa.getPcdCoded() == null) {\r
+                return;\r
+            }\r
+            \r
+            List<PcdCodedDocument.PcdCoded.PcdEntry> l = msa.getPcdCoded().getPcdEntryList();\r
+            ListIterator li = l.listIterator();\r
+            \r
+            while(li.hasNext()) {\r
+                PcdCodedDocument.PcdCoded.PcdEntry msaPcd = (PcdCodedDocument.PcdCoded.PcdEntry)li.next();\r
+                ModuleSADocument.ModuleSA moduleSA = getModuleSA(moduleKey);\r
+                if (moduleSA.getPcdBuildDefinition() != null) {\r
+                    XmlCursor cursor = moduleSA.getPcdBuildDefinition().newCursor();\r
+                    if (cursor.toFirstChild()) {\r
+                        PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData pcdData = (PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData) cursor\r
+                                                                                                                                                      .getObject();\r
+                        if (msaPcd.getCName().equals(pcdData.getCName())\r
+                            && msaPcd.getTokenSpaceGuidCName().equals(pcdData.getTokenSpaceGuidCName())) {\r
+\r
+                            maintainDynPcdMap(pcdData.getCName() + " " + pcdData.getTokenSpaceGuidCName(), moduleKey);\r
+                            cursor.removeXml();\r
+                            break;\r
+                        }\r
+                        while (cursor.toNextSibling()) {\r
+                            pcdData = (PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData) cursor.getObject();\r
+                            if (msaPcd.getCName().equals(pcdData.getCName())\r
+                                && msaPcd.getTokenSpaceGuidCName().equals(pcdData.getTokenSpaceGuidCName())) {\r
+                                maintainDynPcdMap(pcdData.getCName() + " " + pcdData.getTokenSpaceGuidCName(),\r
+                                                  moduleKey);\r
+                                cursor.removeXml();\r
+                                break;\r
+                            }\r
+                        }\r
+                    }\r
+                    cursor.dispose();\r
+                }\r
+            }\r
+            \r
         }\r
-        ListIterator<PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData>li = msa.getPcdBuildDefinition().getPcdDataList().listIterator();\r
-        for (int i = 0; i < saa.length; ++i) {\r
-            PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData pcdData = li.next();\r
-            saa[i][0] = pcdData.getCName();\r
-            saa[i][1] = pcdData.getTokenSpaceGuidCName();\r
-            saa[i][2] = pcdData.getItemType()+"";\r
-            saa[i][3] = pcdData.getToken().toString();\r
-            saa[i][4] = pcdData.getMaxDatumSize()+"";\r
-            saa[i][5] = pcdData.getDatumType()+"";\r
-            saa[i][6] = pcdData.getValue();\r
+        catch (Exception e){\r
+            e.printStackTrace();\r
             \r
         }\r
     }\r
     //\r
-    // key for ModuleSA : "ModuleGuid ModuleVer PackageGuid PackageVer"\r
+    // key for ModuleSA : "ModuleGuid ModuleVer PackageGuid PackageVer Arch"\r
     //\r
     public int getLibraryInstancesCount(String key) {\r
         ModuleSADocument.ModuleSA msa = getModuleSA(key);\r
@@ -382,18 +688,34 @@ public class FpdFileContents {
         }\r
     }\r
     \r
-    public void removeLibraryInstances(String key) {\r
+    public void removeLibraryInstance(String key, int i) {\r
         ModuleSADocument.ModuleSA msa = getModuleSA(key);\r
         if (msa == null || msa.getLibraries() == null){\r
             return ;\r
         }\r
         \r
         XmlCursor cursor = msa.getLibraries().newCursor();\r
-        cursor.removeXml();\r
+        if (cursor.toFirstChild()) {\r
+            for (int j = 0; j < i; ++j) {\r
+                cursor.toNextSibling();\r
+            }\r
+            cursor.push();\r
+            cursor.toPrevToken();\r
+            if (cursor.isComment()) {\r
+                cursor.removeXml();\r
+            }\r
+            cursor.pop();\r
+            cursor.removeXml();\r
+            if (getLibraryInstancesCount(key) == 0) {\r
+                cursor.toParent();\r
+                cursor.removeXml();\r
+            }\r
+        }\r
+        \r
         cursor.dispose();\r
     }\r
     \r
-    public void genLibraryInstance(String mg, String mv, String pg, String pv, String key) {\r
+    public void genLibraryInstance(ModuleIdentification libMi, String key) {\r
         ModuleSADocument.ModuleSA msa = getModuleSA(key);\r
         if (msa == null){\r
             msa = getfpdFrameworkModules().addNewModuleSA();\r
@@ -403,7 +725,26 @@ public class FpdFileContents {
             libs = msa.addNewLibraries();\r
         }\r
         \r
+        String mn = libMi.getName();\r
+        String mg = libMi.getGuid();\r
+        String mv = libMi.getVersion();\r
+        String pn = libMi.getPackageId().getName();\r
+        String pg = libMi.getPackageId().getGuid();\r
+        String pv = libMi.getPackageId().getVersion();\r
         LibrariesDocument.Libraries.Instance instance = libs.addNewInstance();\r
+        XmlCursor cursor = instance.newCursor();\r
+        try{\r
+            String comment = "Pkg: " + pn + " Mod: " + mn \r
+                + " Path: " + libMi.getPath().substring(System.getenv("WORKSPACE").length() + 1);\r
+            cursor.insertComment(comment);\r
+        }\r
+        catch (Exception e){\r
+            e.printStackTrace();\r
+        }\r
+        finally {\r
+            cursor.dispose();\r
+        }\r
+        \r
         instance.setModuleGuid(mg);\r
         instance.setModuleVersion(mv);\r
         instance.setPackageGuid(pg);\r
@@ -424,11 +765,18 @@ public class FpdFileContents {
         if (msa == null ) {\r
             return;\r
         }\r
-        if(msa.getModuleSaBuildOptions() == null){\r
-            msa.addNewModuleSaBuildOptions().setFvBinding(fvBinding);\r
-            return;\r
+        if (fvBinding == null || fvBinding.length() == 0) {\r
+            if(msa.getModuleSaBuildOptions() != null){\r
+                msa.getModuleSaBuildOptions().unsetFvBinding();\r
+            }\r
+        }\r
+        else {\r
+            if(msa.getModuleSaBuildOptions() == null){\r
+                msa.addNewModuleSaBuildOptions().setFvBinding(fvBinding);\r
+                return;\r
+            }\r
+            msa.getModuleSaBuildOptions().setFvBinding(fvBinding);\r
         }\r
-        msa.getModuleSaBuildOptions().setFvBinding(fvBinding);\r
     }\r
     \r
     public String getFfsFileNameGuid(String moduleKey){\r
@@ -445,10 +793,21 @@ public class FpdFileContents {
             return;\r
         }\r
         if(msa.getModuleSaBuildOptions() == null){\r
-            msa.addNewModuleSaBuildOptions().setFfsFileNameGuid(fileGuid);\r
-            return;\r
+            msa.addNewModuleSaBuildOptions();\r
+            \r
+        }\r
+        ModuleSaBuildOptionsDocument.ModuleSaBuildOptions msaBuildOpts= msa.getModuleSaBuildOptions();\r
+        if (fileGuid != null) {\r
+            msaBuildOpts.setFfsFileNameGuid(fileGuid);\r
+        }\r
+        else{\r
+            XmlCursor cursor = msaBuildOpts.newCursor();\r
+            if (cursor.toChild(xmlNs, "FfsFileNameGuid")) {\r
+                cursor.removeXml();\r
+            }\r
+            cursor.dispose();\r
         }\r
-        msa.getModuleSaBuildOptions().setFfsFileNameGuid(fileGuid);\r
+        \r
     }\r
     \r
     public String getFfsFormatKey(String moduleKey){\r
@@ -468,12 +827,25 @@ public class FpdFileContents {
             msa.addNewModuleSaBuildOptions().setFfsFormatKey(ffsKey);\r
             return;\r
         }\r
-        msa.getModuleSaBuildOptions().setFvBinding(ffsKey);\r
+        msa.getModuleSaBuildOptions().setFfsFormatKey(ffsKey);\r
+    }\r
+    \r
+    public void setModuleSAForceDebug(int i, boolean dbgEnable) {\r
+        ModuleSADocument.ModuleSA moduleSa = getModuleSA(i);\r
+        moduleSa.setForceDebug(dbgEnable);\r
+    }\r
+    \r
+    public boolean getModuleSAForceDebug (int i) {\r
+        ModuleSADocument.ModuleSA moduleSa = getModuleSA(i);\r
+        if (moduleSa.getForceDebug() == true) {\r
+            return true;\r
+        }\r
+        return false;\r
     }\r
     \r
     public void getModuleSAOptions(String moduleKey, String[][] saa) {\r
         ModuleSADocument.ModuleSA msa = getModuleSA(moduleKey);\r
-        if (msa.getModuleSaBuildOptions() == null || msa.getModuleSaBuildOptions().getOptions() == null\r
+        if (msa == null || msa.getModuleSaBuildOptions() == null || msa.getModuleSaBuildOptions().getOptions() == null\r
                         || msa.getModuleSaBuildOptions().getOptions().getOptionList() == null) {\r
             return ;\r
         }\r
@@ -487,12 +859,14 @@ public class FpdFileContents {
                 saa[i][0] = listToString(opt.getBuildTargets());\r
             }\r
             saa[i][1] = opt.getToolChainFamily();\r
+            saa[i][2] = opt.getTagName();\r
+            saa[i][3] = opt.getToolCode();\r
+            \r
             if (opt.getSupArchList() != null){\r
-                saa[i][2] = listToString(opt.getSupArchList());\r
+                saa[i][4] = listToString(opt.getSupArchList());\r
 \r
             }\r
-            saa[i][3] = opt.getToolCode();\r
-            saa[i][4] = opt.getTagName();\r
+            \r
             saa[i][5] = opt.getStringValue();\r
              \r
             ++i;\r
@@ -501,7 +875,7 @@ public class FpdFileContents {
     \r
     public int getModuleSAOptionsCount(String moduleKey){\r
         ModuleSADocument.ModuleSA msa = getModuleSA(moduleKey);\r
-        if (msa.getModuleSaBuildOptions() == null || msa.getModuleSaBuildOptions().getOptions() == null\r
+        if (msa == null || msa.getModuleSaBuildOptions() == null || msa.getModuleSaBuildOptions().getOptions() == null\r
                         || msa.getModuleSaBuildOptions().getOptions().getOptionList() == null) {\r
             return 0;\r
         }\r
@@ -557,21 +931,23 @@ public class FpdFileContents {
      * @param mi\r
      * @param moduleSa if null, generate a new ModuleSA.\r
      */\r
-    public void addFrameworkModulesPcdBuildDefs(ModuleIdentification mi, ModuleSADocument.ModuleSA moduleSa) throws Exception {\r
+    public void addFrameworkModulesPcdBuildDefs(ModuleIdentification mi, String arch, ModuleSADocument.ModuleSA moduleSa) throws Exception {\r
         //ToDo add Arch filter\r
         \r
         try {\r
-            ModuleSurfaceAreaDocument.ModuleSurfaceArea msa = (ModuleSurfaceAreaDocument.ModuleSurfaceArea)GlobalData.getModuleXmlObject(mi);\r
+            if (moduleSa == null) {\r
+                moduleSa = genModuleSA(mi, arch);\r
+            }\r
+            \r
+            ModuleSurfaceAreaDocument.ModuleSurfaceArea msa = (ModuleSurfaceAreaDocument.ModuleSurfaceArea)WorkspaceProfile.getModuleXmlObject(mi);\r
             if (msa.getPcdCoded() == null) {\r
                 return;\r
             }\r
-            if (moduleSa == null) {\r
-                moduleSa = genModuleSA(mi);\r
-            }\r
+            \r
             Map<String, XmlObject> m = new HashMap<String, XmlObject>();\r
             m.put("ModuleSurfaceArea", msa);\r
             SurfaceAreaQuery.setDoc(m);\r
-            PackageIdentification[] depPkgs = SurfaceAreaQuery.getDependencePkg(null);\r
+            PackageIdentification[] depPkgs = SurfaceAreaQuery.getDependencePkg(null, mi);\r
             //\r
             // Implementing InitializePlatformPcdBuildDefinitions\r
             //\r
@@ -584,18 +960,19 @@ public class FpdFileContents {
                     //\r
                     // ToDo Error \r
                     //\r
-                    throw new PcdDeclNotFound(mi.getName() + " " + msaPcd.getCName());\r
+                    throw new PcdDeclNotFound("No Declaration for PCD Entry " + msaPcd.getCName() + " in Module " + mi.getName());\r
                 }\r
                 //\r
                 // AddItem to ModuleSA PcdBuildDefinitions\r
                 //\r
                 String defaultVal = msaPcd.getDefaultValue() == null ? spdPcd.getDefaultValue() : msaPcd.getDefaultValue();\r
+                \r
                 genPcdData(msaPcd.getCName(), spdPcd.getToken(), msaPcd.getTokenSpaceGuidCName(), msaPcd.getPcdItemType().toString(), spdPcd.getDatumType()+"", defaultVal, moduleSa);\r
             }\r
             \r
         }\r
         catch (Exception e){\r
-            e.printStackTrace();\r
+            \r
             throw e; \r
         }\r
         \r
@@ -603,12 +980,10 @@ public class FpdFileContents {
     \r
     private PcdDeclarationsDocument.PcdDeclarations.PcdEntry LookupPcdDeclaration (PcdCodedDocument.PcdCoded.PcdEntry msaPcd, PackageIdentification[] depPkgs) {\r
         \r
-        Map<String, XmlObject> m = new HashMap<String, XmlObject>();\r
         PcdDeclarationsDocument.PcdDeclarations.PcdEntry spdPcd = null;\r
         for (int i = 0; i < depPkgs.length; ++i) {\r
-            m.put("PackageSurfaceArea", GlobalData.getPackageXmlObject(depPkgs[i]));\r
-            SurfaceAreaQuery.setDoc(m);\r
-            XmlObject[] xo = SurfaceAreaQuery.getSpdPcdDeclarations();\r
+\r
+            XmlObject[] xo = SurfaceAreaQuery.getSpdPcdDeclarations(depPkgs[i]);\r
             if (xo == null) {\r
                 continue;\r
             }\r
@@ -631,13 +1006,30 @@ public class FpdFileContents {
         return null;\r
     }\r
     \r
-    private ModuleSADocument.ModuleSA genModuleSA (ModuleIdentification mi) {\r
-        PackageIdentification pi = GlobalData.getPackageForModule(mi);\r
+    private ModuleSADocument.ModuleSA genModuleSA (ModuleIdentification mi, String arch) {\r
+        PackageIdentification pi = WorkspaceProfile.getPackageForModule(mi);\r
         ModuleSADocument.ModuleSA msa = getfpdFrameworkModules().addNewModuleSA();\r
+        XmlCursor cursor = msa.newCursor();\r
+        try{\r
+            String comment = "Mod: " + mi.getName() + " Type: " + SurfaceAreaQuery.getModuleType(mi) + " Path: "\r
+                            + mi.getPath().substring(System.getenv("WORKSPACE").length() + 1);\r
+            cursor.insertComment(comment);\r
+        }\r
+        catch(Exception e){\r
+            e.printStackTrace();\r
+        }\r
+        finally { \r
+            cursor.dispose();\r
+        }\r
         msa.setModuleGuid(mi.getGuid());\r
         msa.setModuleVersion(mi.getVersion());\r
         msa.setPackageGuid(pi.getGuid());\r
         msa.setPackageVersion(pi.getVersion());\r
+        if (arch != null) {\r
+            Vector<String> v = new Vector<String>();\r
+            v.add(arch);\r
+            msa.setSupArchList(v); \r
+        }\r
         \r
         return msa;\r
     }\r
@@ -655,39 +1047,18 @@ public class FpdFileContents {
         if (pcdConsumer == null) {\r
             pcdConsumer = new ArrayList<String>();\r
         }\r
+        //\r
+        // Using existing Pcd type, if this pcd already exists in other ModuleSA\r
+        //\r
+        if (pcdConsumer.size() > 0) {\r
+            String[] valPart = pcdConsumer.get(0).split(" ");\r
+            itemType = valPart[5];\r
+        }\r
         String listValue = moduleSa.getModuleGuid() + " " + moduleSa.getModuleVersion() \r
-        + " " + moduleSa.getPackageGuid() + " " + moduleSa.getPackageVersion() \r
+        + " " + moduleSa.getPackageGuid() + " " + moduleSa.getPackageVersion() + " " + listToString(moduleSa.getSupArchList())\r
         + " " + itemType;\r
         pcdConsumer.add(listValue);\r
         dynPcdMap.put(cName + " " + tsGuid, pcdConsumer);\r
-        //\r
-        // Special dynamic type, if this pcd already exists in other ModuleSA\r
-        //\r
-        if (itemType.equals("DYNAMIC")) {\r
-            \r
-            ListIterator li = pcdConsumer.listIterator();\r
-            while(li.hasNext()) {\r
-                String value = li.next().toString();\r
-                String[] valuePart= value.split(" ");\r
-                if (!valuePart[4].equals("DYNAMIC")) {\r
-                    //ToDo error for same pcd, other type than dynamic\r
-                    pcdConsumer.remove(listValue);\r
-                    throw new PcdItemTypeConflictException(value);\r
-                }\r
-            }\r
-        }\r
-        else {\r
-            ListIterator li = pcdConsumer.listIterator();\r
-            while(li.hasNext()) {\r
-                String value = li.next().toString();\r
-                String[] valuePart= value.split(" ");\r
-                if (valuePart[4].equals("DYNAMIC")) {\r
-                    //ToDo error for same pcd, other type than non-dynamic\r
-                    pcdConsumer.remove(listValue);\r
-                    throw new PcdItemTypeConflictException(value);\r
-                }\r
-            }\r
-        }\r
         \r
         PcdBuildDefinitionDocument.PcdBuildDefinition.PcdData fpdPcd = moduleSa.getPcdBuildDefinition().addNewPcdData();\r
         fpdPcd.setCName(cName);\r
@@ -696,6 +1067,51 @@ public class FpdFileContents {
         fpdPcd.setDatumType(PcdDataTypes.Enum.forString(dataType));\r
         fpdPcd.setItemType(PcdItemTypes.Enum.forString(itemType));\r
         \r
+        if (defaultVal != null){\r
+            fpdPcd.setValue(defaultVal);\r
+        }\r
+        else {\r
+            if (dataType.equals("UINT8") || dataType.equals("UINT16") || dataType.equals("UINT32") || dataType.equals("UINT64")) {\r
+                fpdPcd.setValue("0");\r
+            }\r
+            if (dataType.equals("BOOLEAN")){\r
+                fpdPcd.setValue("false");\r
+            }\r
+            if (dataType.equals("VOID*")) {\r
+                fpdPcd.setValue("");\r
+            }\r
+        }\r
+        //\r
+        // Using existing pcd value, if this pcd already exists in other moduleSa.\r
+        //\r
+        if (defaultPcdValue.get(cName + " " + tsGuid) == null) {\r
+            defaultPcdValue.put(cName + " " + tsGuid, fpdPcd.getValue());\r
+        }\r
+        else {\r
+            fpdPcd.setValue(defaultPcdValue.get(cName + " " + tsGuid));\r
+        }\r
+        \r
+        if (dataType.equals("UINT8")){\r
+            fpdPcd.setMaxDatumSize(1);\r
+        }\r
+        if (dataType.equals("UINT16")) {\r
+            fpdPcd.setMaxDatumSize(2);\r
+        }\r
+        if (dataType.equals("UINT32")) {\r
+            fpdPcd.setMaxDatumSize(4);\r
+        }\r
+        if (dataType.equals("UINT64")){\r
+            fpdPcd.setMaxDatumSize(8);\r
+        }\r
+        if (dataType.equals("BOOLEAN")){\r
+            fpdPcd.setMaxDatumSize(1);\r
+        }\r
+        if (dataType.equals("VOID*")) {\r
+            int maxSize = setMaxSizeForPointer(fpdPcd.getValue());\r
+            fpdPcd.setMaxDatumSize(maxSize);\r
+        }\r
+        \r
+        \r
         if (itemType.equals("DYNAMIC") || itemType.equals("DYNAMIC_EX")) {\r
             ArrayList<String> al = LookupDynamicPcdBuildDefinition(cName + " " + tsGuid);\r
             //\r
@@ -706,44 +1122,10 @@ public class FpdFileContents {
                 addDynamicPcdBuildData(cName, token, tsGuid, itemType, dataType, defaultVal);\r
             }\r
         }\r
-        else {\r
-            if (defaultVal != null){\r
-                fpdPcd.setValue(defaultVal);\r
-            }\r
-            else {\r
-                if (dataType.equals("UINT8") || dataType.equals("UINT16") || dataType.equals("UINT32") || dataType.equals("UINT64")) {\r
-                    fpdPcd.setValue("0");\r
-                }\r
-                if (dataType.equals("BOOLEAN")){\r
-                    fpdPcd.setValue("false");\r
-                }\r
-                if (dataType.equals("VOID*")) {\r
-                    fpdPcd.setValue("");\r
-                }\r
-            }\r
-            if (dataType.equals("UINT8")){\r
-                fpdPcd.setMaxDatumSize(1);\r
-            }\r
-            if (dataType.equals("UINT16")) {\r
-                fpdPcd.setMaxDatumSize(2);\r
-            }\r
-            if (dataType.equals("UINT32")) {\r
-                fpdPcd.setMaxDatumSize(4);\r
-            }\r
-            if (dataType.equals("UINT64")){\r
-                fpdPcd.setMaxDatumSize(8);\r
-            }\r
-            if (dataType.equals("BOOLEAN")){\r
-                fpdPcd.setMaxDatumSize(1);\r
-            }\r
-            if (dataType.equals("VOID*")) {\r
-                int maxSize = setMaxSizeForPointer(fpdPcd.getValue());\r
-                fpdPcd.setMaxDatumSize(maxSize);\r
-            }\r
-        }\r
+        \r
     }\r
     \r
-    private int setMaxSizeForPointer(String datum) throws PcdValueMalFormed{\r
+    public int setMaxSizeForPointer(String datum) throws PcdValueMalFormed{\r
         if (datum == null) {\r
             return 0;\r
         }\r
@@ -794,7 +1176,7 @@ public class FpdFileContents {
                 \r
                 if (value > 0xFF) {\r
 //                   "[FPD file error] The datum type of PCD %s in %s is VOID*, "+\r
-//                   "it is byte array in fact. But the element of %s exceed the byte range",\r
+//                   "it must be a byte array. But the element of %s exceed the byte range",\r
                     throw new PcdValueMalFormed (datum);                               \r
                 }\r
             }\r
@@ -820,18 +1202,22 @@ public class FpdFileContents {
     \r
     private ArrayList<String> LookupPlatformPcdData(String pcdKey) {\r
         \r
-        return dynPcdMap.get("pcdKey");\r
+        return dynPcdMap.get(pcdKey);\r
     }\r
     \r
     public int getDynamicPcdBuildDataCount() {\r
-        if (getfpdDynPcdBuildDefs().getPcdBuildDataList() == null) {\r
+        if (getfpdDynPcdBuildDefs().getPcdBuildDataList() == null || getfpdDynPcdBuildDefs().getPcdBuildDataList().size() == 0) {\r
+            removeElement(getfpdDynPcdBuildDefs());\r
+            fpdDynPcdBuildDefs = null;\r
             return 0;\r
         }\r
         return getfpdDynPcdBuildDefs().getPcdBuildDataList().size();\r
     }\r
     \r
     public void getDynamicPcdBuildData(String[][] saa) {\r
-        if (getfpdDynPcdBuildDefs().getPcdBuildDataList() == null) {\r
+        if (getfpdDynPcdBuildDefs().getPcdBuildDataList() == null || getfpdDynPcdBuildDefs().getPcdBuildDataList().size() == 0) {\r
+            removeElement(getfpdDynPcdBuildDefs());\r
+            fpdDynPcdBuildDefs = null;\r
             return ;\r
         }\r
         List<DynamicPcdBuildDefinitionsDocument.DynamicPcdBuildDefinitions.PcdBuildData> l = getfpdDynPcdBuildDefs().getPcdBuildDataList();\r
@@ -849,7 +1235,7 @@ public class FpdFileContents {
         }\r
     }\r
     \r
-    private void addDynamicPcdBuildData(String cName, Object token, String tsGuid, String itemType, String dataType, String defaultVal) \r
+    public void addDynamicPcdBuildData(String cName, Object token, String tsGuid, String itemType, String dataType, String defaultVal) \r
     throws PcdValueMalFormed{\r
         DynamicPcdBuildDefinitionsDocument.DynamicPcdBuildDefinitions.PcdBuildData dynPcdData = getfpdDynPcdBuildDefs().addNewPcdBuildData();\r
         dynPcdData.setItemType(PcdItemTypes.Enum.forString(itemType));\r
@@ -905,19 +1291,28 @@ public class FpdFileContents {
         }\r
     }\r
     \r
-    private void removeDynamicPcdBuildData(String cName, String tsGuid) {\r
-        XmlObject o = getfpdDynPcdBuildDefs();\r
+    public void removeDynamicPcdBuildData(String cName, String tsGuid) {\r
+        XmlObject o = fpdRoot.getDynamicPcdBuildDefinitions();\r
+        if (o == null) {\r
+            return;\r
+        }\r
         \r
         XmlCursor cursor = o.newCursor();\r
         if (cursor.toFirstChild()) {\r
-            DynamicPcdBuildDefinitionsDocument.DynamicPcdBuildDefinitions.PcdBuildData pcdBuildData = \r
-                (DynamicPcdBuildDefinitionsDocument.DynamicPcdBuildDefinitions.PcdBuildData)cursor.getObject();\r
-            while (!(pcdBuildData.getCName().equals(cName) && pcdBuildData.getTokenSpaceGuidCName().equals(tsGuid))) {\r
-                cursor.toNextSibling();\r
-                pcdBuildData = (DynamicPcdBuildDefinitionsDocument.DynamicPcdBuildDefinitions.PcdBuildData)cursor.getObject();\r
+            do {\r
+                DynamicPcdBuildDefinitionsDocument.DynamicPcdBuildDefinitions.PcdBuildData pcdBuildData = \r
+                    (DynamicPcdBuildDefinitionsDocument.DynamicPcdBuildDefinitions.PcdBuildData)cursor.getObject();\r
+                if (pcdBuildData.getCName().equals(cName) && pcdBuildData.getTokenSpaceGuidCName().equals(tsGuid)) {\r
+                    \r
+                    if (getDynamicPcdBuildDataCount() == 1) {\r
+                        cursor.toParent();\r
+                    }\r
+                    cursor.removeXml();\r
+                    cursor.dispose();\r
+                    return;\r
+                }\r
             }\r
-            \r
-            cursor.removeXml();\r
+            while (cursor.toNextSibling());\r
         }\r
         cursor.dispose();\r
     }\r
@@ -925,7 +1320,8 @@ public class FpdFileContents {
     // Get the Sku Info count of ith dyn pcd element.\r
     //\r
     public int getDynamicPcdSkuInfoCount(int i){\r
-        if (getfpdDynPcdBuildDefs().getPcdBuildDataList() == null || getfpdDynPcdBuildDefs().getPcdBuildDataList().size() == 0) {\r
+        if (fpdRoot.getDynamicPcdBuildDefinitions() == null || fpdRoot.getDynamicPcdBuildDefinitions().getPcdBuildDataList() == null \r
+                        || fpdRoot.getDynamicPcdBuildDefinitions().getPcdBuildDataList().size() == 0) {\r
             return 0;\r
         }\r
         \r
@@ -949,6 +1345,8 @@ public class FpdFileContents {
     \r
     public void getDynamicPcdSkuInfos(int i, String[][] saa){\r
         if (getfpdDynPcdBuildDefs().getPcdBuildDataList() == null || getfpdDynPcdBuildDefs().getPcdBuildDataList().size() == 0) {\r
+            removeElement(getfpdDynPcdBuildDefs());\r
+            fpdDynPcdBuildDefs = null;\r
             return;\r
         }\r
         \r
@@ -986,6 +1384,8 @@ public class FpdFileContents {
     public String getDynamicPcdBuildDataValue(int i){\r
         String value = null;\r
         if (getfpdDynPcdBuildDefs().getPcdBuildDataList() == null || getfpdDynPcdBuildDefs().getPcdBuildDataList().size() == 0) {\r
+            removeElement(getfpdDynPcdBuildDefs());\r
+            fpdDynPcdBuildDefs = null;\r
             return value;\r
         }\r
         \r
@@ -1009,6 +1409,8 @@ public class FpdFileContents {
     public String getDynamicPcdBuildDataVpdOffset(int i){\r
         String vpdOffset = null;\r
         if (getfpdDynPcdBuildDefs().getPcdBuildDataList() == null || getfpdDynPcdBuildDefs().getPcdBuildDataList().size() == 0) {\r
+            removeElement(getfpdDynPcdBuildDefs());\r
+            fpdDynPcdBuildDefs = null;\r
             return vpdOffset;\r
         }\r
         \r
@@ -1031,6 +1433,8 @@ public class FpdFileContents {
     \r
     public void removeDynamicPcdBuildDataSkuInfo(int i) {\r
         if (getfpdDynPcdBuildDefs().getPcdBuildDataList() == null || getfpdDynPcdBuildDefs().getPcdBuildDataList().size() == 0) {\r
+            removeElement(getfpdDynPcdBuildDefs());\r
+            fpdDynPcdBuildDefs = null;\r
             return;\r
         }\r
         \r
@@ -1057,9 +1461,9 @@ public class FpdFileContents {
     //\r
     public void genDynamicPcdBuildDataSkuInfo(String id, String varName, String varGuid, String varOffset, \r
                                               String hiiDefault, String vpdOffset, String value, int i) {\r
-        if (getfpdDynPcdBuildDefs().getPcdBuildDataList() == null || getfpdDynPcdBuildDefs().getPcdBuildDataList().size() == 0) {\r
-            return;\r
-        }\r
+//        if (getfpdDynPcdBuildDefs().getPcdBuildDataList() == null || getfpdDynPcdBuildDefs().getPcdBuildDataList().size() == 0) {\r
+//            return;\r
+//        }\r
         \r
         XmlCursor cursor = getfpdDynPcdBuildDefs().newCursor();\r
         if (cursor.toFirstChild()) {\r
@@ -1086,9 +1490,9 @@ public class FpdFileContents {
     \r
     public void updateDynamicPcdBuildDataSkuInfo(String id, String varName, String varGuid, String varOffset, \r
                                                  String hiiDefault, String vpdOffset, String value, int i){\r
-        if (getfpdDynPcdBuildDefs().getPcdBuildDataList() == null || getfpdDynPcdBuildDefs().getPcdBuildDataList().size() == 0) {\r
-            return;\r
-        }\r
+//        if (getfpdDynPcdBuildDefs().getPcdBuildDataList() == null || getfpdDynPcdBuildDefs().getPcdBuildDataList().size() == 0) {\r
+//            return;\r
+//        }\r
         \r
         XmlCursor cursor = getfpdDynPcdBuildDefs().newCursor();\r
         if (cursor.toFirstChild()) {\r
@@ -1118,15 +1522,6 @@ public class FpdFileContents {
         }\r
     }\r
     \r
-    public void removePcdDataFromLibraryInstance(String moduleKey, String libInstanceKey){\r
-        ModuleSADocument.ModuleSA moduleSa = getModuleSA(moduleKey);\r
-        //\r
-        //  should better maintain pcd from lib instance only, but maintain all is acceptable now. \r
-        //\r
-        maintainDynPcdMap(moduleSa.getPcdBuildDefinition(), libInstanceKey);\r
-        \r
-    }\r
-    \r
     public BuildOptionsDocument.BuildOptions getfpdBuildOpts() {\r
         if (fpdBuildOpts == null) {\r
             fpdBuildOpts = fpdRoot.addNewBuildOptions();\r
@@ -1134,6 +1529,118 @@ public class FpdFileContents {
         return fpdBuildOpts;\r
     }\r
     \r
+    public void genBuildOptionsUserExtensions(String fvName, String outputFileName, String[][] includeModules) {\r
+        UserExtensionsDocument.UserExtensions userExts = getfpdBuildOpts().addNewUserExtensions();\r
+        userExts.setUserID("IMAGES");\r
+        userExts.setIdentifier(new BigInteger("1"));\r
+        XmlCursor cursor = userExts.newCursor();\r
+        cursor.toEndToken();\r
+        \r
+        cursor.beginElement("FvName");\r
+        cursor.insertChars(fvName);\r
+        cursor.toNextToken();\r
+        \r
+        cursor.beginElement("InfFileName");\r
+        cursor.insertChars(fvName + ".inf");\r
+        cursor.toNextToken();\r
+        \r
+        cursor.beginElement("IncludeModules");\r
+        for (int i = 0; i < includeModules.length; ++i) {\r
+            cursor.beginElement("Module");\r
+            cursor.insertAttributeWithValue("ModuleGuid", includeModules[i][0]);\r
+            cursor.insertAttributeWithValue("BaseName", includeModules[i][1]);\r
+            cursor.toEndToken();\r
+            cursor.toNextToken();\r
+        }\r
+        cursor.dispose();\r
+    }\r
+    \r
+    public int getUserExtsIncModCount (String fvName) {\r
+        if (getfpdBuildOpts().getUserExtensionsList() == null) {\r
+            return -1;\r
+        }\r
+        ListIterator<UserExtensionsDocument.UserExtensions> li = getfpdBuildOpts().getUserExtensionsList().listIterator();\r
+        while (li.hasNext()) {\r
+            UserExtensionsDocument.UserExtensions ues = li.next();\r
+            if (!ues.getUserID().equals("IMAGES")) {\r
+                continue;\r
+            }\r
+            XmlCursor cursor = ues.newCursor();\r
+            cursor.toFirstChild();\r
+            String elementName = cursor.getTextValue();\r
+            if (elementName.equals(fvName)) {\r
+                cursor.toNextSibling(new QName("", "IncludeModules"));\r
+                if (cursor.toFirstChild()) {\r
+                    int i = 1;\r
+                    for (i = 1; cursor.toNextSibling(); ++i);\r
+                    cursor.dispose();\r
+                    return i;\r
+                }\r
+                cursor.dispose();\r
+                return 0;\r
+            }\r
+            cursor.dispose();\r
+        }\r
+        return -1;\r
+    }\r
+    \r
+    public void getUserExtsIncMods(String fvName, String[][] saa) {\r
+        if (getfpdBuildOpts().getUserExtensionsList() == null) {\r
+            return;\r
+        }\r
+        ListIterator<UserExtensionsDocument.UserExtensions> li = getfpdBuildOpts().getUserExtensionsList().listIterator();\r
+        while (li.hasNext()) {\r
+            UserExtensionsDocument.UserExtensions ues = li.next();\r
+            if (!ues.getUserID().equals("IMAGES")) {\r
+                continue;\r
+            }\r
+            XmlCursor cursor = ues.newCursor();\r
+            cursor.toFirstChild();\r
+            String elementName = cursor.getTextValue();\r
+            if (elementName.equals(fvName)) {\r
+                cursor.toNextSibling(new QName("", "IncludeModules"));\r
+                if (cursor.toFirstChild()) {\r
+                    int i = 0;\r
+                    do {\r
+                        saa[i][0] = cursor.getAttributeText(new QName("ModuleGuid"));\r
+                        saa[i][1] = cursor.getAttributeText(new QName("BaseName"));\r
+                        ++i;\r
+                    }while (cursor.toNextSibling());\r
+                }\r
+                cursor.dispose();\r
+                return;\r
+            }\r
+            cursor.dispose();\r
+        }\r
+        \r
+    }\r
+    \r
+    public void removeBuildOptionsUserExtensions (String fvName) {\r
+        if (getfpdBuildOpts().getUserExtensionsList() == null) {\r
+            return;\r
+        }\r
+        \r
+        ListIterator<UserExtensionsDocument.UserExtensions> li = getfpdBuildOpts().getUserExtensionsList().listIterator();\r
+        while (li.hasNext()) {\r
+            UserExtensionsDocument.UserExtensions ues = li.next();\r
+            if (!ues.getUserID().equals("IMAGES")) {\r
+                continue;\r
+            }\r
+            XmlCursor cursor = ues.newCursor();\r
+            cursor.toFirstChild();\r
+            String elementName = cursor.getTextValue();\r
+            if (elementName.equals(fvName)) {\r
+                cursor.toParent();\r
+                cursor.removeXml();\r
+                cursor.dispose();\r
+                return;\r
+            }\r
+            cursor.dispose();\r
+        }\r
+        \r
+    }\r
+    \r
+    \r
     public void genBuildOptionsUserDefAntTask (String id, String fileName, String execOrder) {\r
         UserDefinedAntTasksDocument.UserDefinedAntTasks udats = getfpdBuildOpts().getUserDefinedAntTasks();\r
         if (udats == null) {\r
@@ -1146,12 +1653,20 @@ public class FpdFileContents {
     \r
     private void setBuildOptionsUserDefAntTask(String id, String fileName, String execOrder, AntTaskDocument.AntTask at) {\r
         at.setId(new Integer(id));\r
+        XmlCursor cursor = at.newCursor();\r
         if (fileName != null){\r
             at.setFilename(fileName);\r
         }\r
+        else if (cursor.toChild(xmlNs, "Filename")) {\r
+            cursor.removeXml();\r
+        }\r
         if (execOrder != null) {\r
             at.setAntCmdOptions(execOrder);\r
         }\r
+        else if (cursor.toChild(xmlNs, "AntCmdOptions")) {\r
+            cursor.removeXml();\r
+        }\r
+        cursor.dispose();\r
     }\r
     \r
     public void removeBuildOptionsUserDefAntTask(int i) {\r
@@ -1165,6 +1680,10 @@ public class FpdFileContents {
                 cursor.toNextSibling();\r
             }\r
             cursor.removeXml();\r
+            if (getBuildOptionsUserDefAntTaskCount() == 0) {\r
+                cursor.toParent();\r
+                cursor.removeXml();\r
+            }\r
         }\r
         cursor.dispose();\r
     }\r
@@ -1233,7 +1752,14 @@ public class FpdFileContents {
         opt.setTagName(tagName);\r
         opt.setToolCode(toolCmd);\r
         \r
-        opt.setSupArchList(archList);\r
+        if (archList != null) {\r
+            opt.setSupArchList(archList);\r
+        }\r
+        else {\r
+            if (opt.isSetSupArchList()) {\r
+                opt.unsetSupArchList();\r
+            }\r
+        }\r
     }\r
     \r
     public void removeBuildOptionsOpt(int i){\r
@@ -1249,6 +1775,10 @@ public class FpdFileContents {
                 cursor.toNextSibling();\r
             }\r
             cursor.removeXml();\r
+            if (getBuildOptionsOptCount() == 0) {\r
+                cursor.toParent();\r
+                cursor.removeXml();\r
+            }\r
         }\r
         cursor.dispose();\r
     }\r
@@ -1484,10 +2014,15 @@ public class FpdFileContents {
                 cursor.toNextSibling(qSections);\r
             }\r
             if (cursor.toFirstChild()) {\r
-                for (int m = 0; m < k; ++m) {\r
+                int m = 0;\r
+                for (; m < k; ++m) {\r
                     cursor.toNextSibling();\r
                 }\r
                 cursor.removeXml();\r
+                if (m == 0) {\r
+                    cursor.toParent();\r
+                    cursor.removeXml();\r
+                }\r
             }\r
         }\r
         cursor.dispose();\r
@@ -1657,7 +2192,16 @@ public class FpdFileContents {
     }\r
     \r
     public void setPlatformDefsSupportedArchs(Vector<Object> archs) {\r
-        getfpdPlatformDefs().setSupportedArchitectures(archs);\r
+        if (archs != null) {\r
+            getfpdPlatformDefs().setSupportedArchitectures(archs);\r
+        }\r
+//        else {\r
+//            XmlCursor cursor = getfpdPlatformDefs().newCursor();\r
+//            if (cursor.toChild(xmlNs, "SupportedArchitectures")) {\r
+//                cursor.removeXml();\r
+//            }\r
+//            cursor.dispose();\r
+//        }\r
     }\r
     \r
     public void getPlatformDefsBuildTargets(Vector<Object> targets) {\r
@@ -1702,6 +2246,10 @@ public class FpdFileContents {
     \r
     public void getPlatformDefsSkuInfos(String[][] saa){\r
         if (getfpdPlatformDefs().getSkuInfo() == null || getfpdPlatformDefs().getSkuInfo().getUiSkuNameList() == null) {\r
+            if (getfpdDynPcdBuildDefs().getPcdBuildDataList() == null) {\r
+                removeElement(getfpdDynPcdBuildDefs());\r
+                fpdDynPcdBuildDefs = null;\r
+            }\r
             return ;\r
         }\r
         \r
@@ -1766,7 +2314,16 @@ public class FpdFileContents {
     }\r
     \r
     public void setPlatformDefsOutputDir(String outputDir) {\r
-        getfpdPlatformDefs().setOutputDirectory(outputDir);\r
+        if (outputDir != null && outputDir.length() > 0) {\r
+            getfpdPlatformDefs().setOutputDirectory(outputDir);\r
+        }\r
+        else{\r
+            XmlCursor cursor = getfpdPlatformDefs().newCursor();\r
+            if (cursor.toChild(new QName(xmlNs, "OutputDirectory"))) {\r
+                cursor.removeXml();\r
+            }\r
+            cursor.dispose();\r
+        }\r
     }\r
     \r
     public FlashDocument.Flash getfpdFlash() {\r
@@ -1794,8 +2351,6 @@ public class FpdFileContents {
         return fdf.getStringValue();\r
     }\r
     \r
-    \r
-    \r
     public void genFvImagesNameValue(String name, String value) {\r
       \r
         FvImagesDocument.FvImages fi = getfpdFlash().getFvImages();\r
@@ -2049,7 +2604,7 @@ public class FpdFileContents {
         if (fpdHdr == null) {\r
             fpdHdr = fpdRoot.addNewPlatformHeader();\r
         }\r
-        genPlatformDefsSkuInfo("0", "DEFAULT");\r
+        \r
         return fpdHdr;\r
     }\r
     \r
@@ -2095,7 +2650,7 @@ public class FpdFileContents {
     \r
     public String getFpdHdrSpec() {\r
 \r
-        return "FRAMEWORK_BUILD_PACKAGING_SPECIFICATION 0x00000052";\r
+        return "FRAMEWORK_BUILD_PACKAGING_SPECIFICATION   0x00000052";\r
 //        return getFpdHdr().getSpecification();\r
     }\r
     \r
@@ -2141,7 +2696,7 @@ public class FpdFileContents {
     }\r
     \r
     public void setFpdHdrSpec(String s){\r
-        s = "FRAMEWORK_BUILD_PACKAGING_SPECIFICATION 0x00000052";\r
+        s = "FRAMEWORK_BUILD_PACKAGING_SPECIFICATION   0x00000052";\r
         getFpdHdr().setSpecification(s);\r
     }\r
     /**\r
@@ -2177,6 +2732,12 @@ public class FpdFileContents {
         }\r
         return s.trim();\r
     }\r
+    \r
+    private void removeElement(XmlObject o) {\r
+        XmlCursor cursor = o.newCursor();\r
+        cursor.removeXml();\r
+        cursor.dispose();\r
+    }\r
 }\r
 \r
 class PcdItemTypeConflictException extends Exception {\r
@@ -2187,8 +2748,9 @@ class PcdItemTypeConflictException extends Exception {
     private static final long serialVersionUID = 1L;\r
     private String details = null;\r
     \r
-    PcdItemTypeConflictException(String info){\r
-        details = info;\r
+    PcdItemTypeConflictException(String pcdName, String info){\r
+        ModuleIdentification mi = WorkspaceProfile.getModuleId(info);\r
+        details = pcdName + " ItemType Conflicts with " + mi.getName() + " in Pkg " + mi.getPackageId().getName();\r
     }\r
     \r
     public String getMessage() {\r
@@ -2205,7 +2767,7 @@ class PcdDeclNotFound extends Exception {
     private String details = null;\r
     \r
     PcdDeclNotFound(String info) {\r
-        details = info;\r
+        details = "PcdDeclNotFound: " + info;\r
     }\r
     \r
     public String getMessage() {\r
@@ -2222,7 +2784,7 @@ class PcdValueMalFormed extends Exception {
     private String details = null;\r
     \r
     PcdValueMalFormed(String info) {\r
-        details = info;\r
+        details = "PcdValueMalFormed: " + info;\r
     }\r
     \r
     public String getMessage() {\r