X-Git-Url: https://git.proxmox.com/?p=mirror_edk2.git;a=blobdiff_plain;f=Tools%2FSource%2FGenBuild%2Forg%2Ftianocore%2Fbuild%2Fpcd%2Faction%2FCollectPCDAction.java;h=8242a4c87c363dca42b14fbc237c6a635f683b38;hp=fe09be2b9ab136c8297cf413a8cba2c0431a322a;hb=eece174ad00f11db8688a36f7c7ef57c9ced98db;hpb=7db4ab705a654e2202b083968e92d5b552007d6b diff --git a/Tools/Source/GenBuild/org/tianocore/build/pcd/action/CollectPCDAction.java b/Tools/Source/GenBuild/org/tianocore/build/pcd/action/CollectPCDAction.java index fe09be2b9a..8242a4c87c 100644 --- a/Tools/Source/GenBuild/org/tianocore/build/pcd/action/CollectPCDAction.java +++ b/Tools/Source/GenBuild/org/tianocore/build/pcd/action/CollectPCDAction.java @@ -21,29 +21,30 @@ import java.io.BufferedReader; import java.io.File; import java.io.FileReader; import java.io.IOException; +import java.math.BigInteger; import java.util.ArrayList; -import java.util.Collections; import java.util.Comparator; import java.util.HashMap; +import java.util.Iterator; import java.util.List; import java.util.Map; +import java.util.Set; import java.util.UUID; +import java.util.regex.Matcher; +import java.util.regex.Pattern; import org.apache.xmlbeans.XmlException; import org.apache.xmlbeans.XmlObject; -import org.tianocore.DynamicPcdBuildDefinitionsDocument; import org.tianocore.DynamicPcdBuildDefinitionsDocument.DynamicPcdBuildDefinitions; -import org.tianocore.DynamicPcdBuildDefinitionsDocument.DynamicPcdBuildDefinitions.PcdBuildData.SkuInfo; -import org.tianocore.DynamicPcdBuildDefinitionsDocument.DynamicPcdBuildDefinitions.PcdBuildData; import org.tianocore.FrameworkModulesDocument; -import org.tianocore.FrameworkPlatformDescriptionDocument; -import org.tianocore.FrameworkPlatformDescriptionDocument.FrameworkPlatformDescription; import org.tianocore.ModuleSADocument; -import org.tianocore.ModuleSADocument.ModuleSA; -import org.tianocore.PackageSurfaceAreaDocument; +import org.tianocore.PcdBuildDefinitionDocument; import org.tianocore.PcdBuildDefinitionDocument.PcdBuildDefinition; +import org.tianocore.PlatformSurfaceAreaDocument; +import org.tianocore.build.autogen.CommonDefinition; +import org.tianocore.build.fpd.FpdParserTask; import org.tianocore.build.global.GlobalData; -import org.tianocore.build.global.SurfaceAreaQuery; +import org.tianocore.build.id.FpdModuleIdentification; import org.tianocore.build.pcd.action.ActionMessage; import org.tianocore.build.pcd.entity.DynamicTokenValue; import org.tianocore.build.pcd.entity.MemoryDatabaseManager; @@ -52,21 +53,47 @@ import org.tianocore.build.pcd.entity.Token; import org.tianocore.build.pcd.entity.UsageInstance; import org.tianocore.build.pcd.exception.EntityException; +/** + CStructTypeDeclaration + + This class is used to store the declaration string, such as + "UINT32 PcdPlatformFlashBaseAddress", of + each memember in the C structure, which is a standard C language + feature used to implement a simple and efficient database for + dynamic(ex) type PCD entry. +**/ + +class CStructTypeDeclaration { + String key; + int alignmentSize; + String cCode; + boolean initTable; + + public CStructTypeDeclaration (String key, int alignmentSize, String cCode, boolean initTable) { + this.key = key; + this.alignmentSize = alignmentSize; + this.cCode = cCode; + this.initTable = initTable; + } +} + +/** + StringTable + + This class is used to store the String in a PCD database. + +**/ class StringTable { private ArrayList al; private ArrayList alComments; private String phase; int len; - int bodyStart; - int bodyLineNum; public StringTable (String phase) { this.phase = phase; al = new ArrayList(); alComments = new ArrayList(); len = 0; - bodyStart = 0; - bodyLineNum = 0; } public String getSizeMacro () { @@ -80,70 +107,108 @@ class StringTable { return len == 0 ? 1 : len; } - public int getTableLen () { - return al.size() == 0 ? 1 : al.size(); - } - public String getExistanceMacro () { return String.format(PcdDatabase.StringTableExistenceMacro, phase, (al.size() == 0)? "TRUE":"FALSE"); } - - public String getTypeDeclaration () { - - String output; - + + public void genCode (ArrayList declaList, HashMap instTable) { final String stringTable = "StringTable"; - final String tab = "\t"; - final String newLine = ";\r\n"; - - output = "/* StringTable */\r\n"; - - if (al.size() == 0) { - output += tab + String.format("UINT16 %s[1] /* StringTable is Empty */", stringTable) + newLine; - } - - for (int i = 0; i < al.size(); i++) { - String str = al.get(i); - - if (i == 0) { - // - // StringTable is a well-known name in the PCD DXE driver - // - output += tab + String.format("UINT16 %s[%d] /* %s */", stringTable, str.length() + 1, alComments.get(i)) + newLine; - } else { - output += tab + String.format("UINT16 %s_%d[%d] /* %s */", stringTable, i, str.length() + 1, alComments.get(i)) + newLine; - } - } - - return output; - - } - - public ArrayList getInstantiation () { - ArrayList output = new ArrayList(); + final String tab = "\t"; + final String newLine = "\r\n"; + final String commaNewLine = ",\r\n"; + + CStructTypeDeclaration decl; - output.add("/* StringTable */"); + String cDeclCode = ""; + String cInstCode = ""; + // + // If we have a empty StringTable + // if (al.size() == 0) { - output.add("{ 0 }"); + cDeclCode += String.format("%-20s%s[1]; /* StringTable is empty */", "UINT16", stringTable) + newLine; + decl = new CStructTypeDeclaration ( + stringTable, + 2, + cDeclCode, + true + ); + declaList.add(decl); + + cInstCode = String.format("/* %s */", stringTable) + newLine + tab + "{ 0 }"; + instTable.put(stringTable, cInstCode); } else { - String str; + // + // If there is any String in the StringTable + // for (int i = 0; i < al.size(); i++) { - str = String.format("L\"%s\" /* %s */", al.get(i), alComments.get(i)); + String str = al.get(i); + String stringTableName; + + if (i == 0) { + // + // StringTable is a well-known name in the PCD DXE driver + // + stringTableName = stringTable; + + } else { + stringTableName = String.format("%s_%d", stringTable, i); + cDeclCode += tab; + } + cDeclCode += String.format("%-20s%s[%d]; /* %s */", "UINT16", + stringTableName, str.length() + 1, + alComments.get(i)) + + newLine; + + if (i == 0) { + cInstCode = "/* StringTable */" + newLine; + } + + cInstCode += tab + String.format("L\"%s\" /* %s */", al.get(i), alComments.get(i)); if (i != al.size() - 1) { - str += ","; + cInstCode += commaNewLine; } - output.add(str); } + + decl = new CStructTypeDeclaration ( + stringTable, + 2, + cDeclCode, + true + ); + declaList.add(decl); + + instTable.put(stringTable, cInstCode); } - - return output; } - public int add (String str, Token token) { + public int add (String inputStr, Token token) { int i; + int pos; + + String str = inputStr; + + // + // The input can be two types: + // "L\"Bootmode\"" or "Bootmode". + // We drop the L\" and \" for the first type. + if (str.startsWith("L\"") && str.endsWith("\"")) { + str = str.substring(2, str.length() - 1); + } + // + // Check if StringTable has this String already. + // If so, return the current pos. + // + for (i = 0, pos = 0; i < al.size(); i++) { + String s = al.get(i);; + if (str.equals(s)) { + return pos; + } + pos = s.length() + 1; + } + i = len; // // Include the NULL character at the end of String @@ -156,48 +221,82 @@ class StringTable { } } +/** + SizeTable + + This class is used to store the Size information for + POINTER TYPE PCD entry in a PCD database. + +**/ class SizeTable { - private ArrayList al; + private ArrayList> al; private ArrayList alComments; - private String phase; private int len; - private int bodyStart; - private int bodyLineNum; - + private String phase; + public SizeTable (String phase) { - this.phase = phase; - al = new ArrayList(); + al = new ArrayList>(); alComments = new ArrayList(); len = 0; - bodyStart = 0; - bodyLineNum = 0; + this.phase = phase; + } + + public String getSizeMacro () { + return String.format(PcdDatabase.SizeTableSizeMacro, phase, getSize()); + } + + private int getSize() { + return len == 0 ? 1 : len; } - public String getTypeDeclaration () { - return String.format(PcdDatabase.SizeTableDeclaration, phase); + public void genCode (ArrayList declaList, HashMap instTable, String phase) { + final String name = "SizeTable"; + + CStructTypeDeclaration decl; + String cCode; + + cCode = String.format(PcdDatabase.SizeTableDeclaration, phase); + decl = new CStructTypeDeclaration ( + name, + 2, + cCode, + true + ); + declaList.add(decl); + + + cCode = PcdDatabase.genInstantiationStr(getInstantiation()); + instTable.put(name, cCode); } - public ArrayList getInstantiation () { + private ArrayList getInstantiation () { + final String comma = ","; ArrayList Output = new ArrayList(); Output.add("/* SizeTable */"); Output.add("{"); - bodyStart = 2; - if (al.size() == 0) { - Output.add("0"); + Output.add("\t0"); } else { for (int index = 0; index < al.size(); index++) { - Integer n = al.get(index); - String str = n.toString(); + ArrayList ial = al.get(index); + + String str = "\t"; + + for (int index2 = 0; index2 < ial.size(); index2++) { + str += " " + ial.get(index2).toString(); + if (index2 != ial.size() - 1) { + str += comma; + } + } + str += " /* " + alComments.get(index) + " */"; + if (index != (al.size() - 1)) { - str += ","; + str += comma; } - str += " /* " + alComments.get(index) + " */"; Output.add(str); - bodyLineNum++; } } @@ -206,48 +305,37 @@ class SizeTable { return Output; } - public int getBodyStart() { - return bodyStart; - } - - public int getBodyLineNum () { - return bodyLineNum; - } + public void add (Token token) { - public int add (Token token) { - int index = len; + // + // We only have size information for POINTER type PCD entry. + // + if (token.datumType != Token.DATUM_TYPE.POINTER) { + return; + } + + ArrayList ial = token.getPointerTypeSize(); + + len+= ial.size(); - len++; - al.add(token.datumSize); + al.add(ial); alComments.add(token.getPrimaryKeyString()); - return index; + return; } - private int getDatumSize(Token token) { - /* - switch (token.datumType) { - case Token.DATUM_TYPE.UINT8: - return 1; - default: - return 0; - } - */ - return 0; - } - - public int getTableLen () { - return al.size() == 0 ? 1 : al.size(); - } - } +/** + GuidTable + + This class is used to store the GUIDs in a PCD database. +**/ class GuidTable { private ArrayList al; private ArrayList alComments; private String phase; private int len; - private int bodyStart; private int bodyLineNum; public GuidTable (String phase) { @@ -255,7 +343,6 @@ class GuidTable { al = new ArrayList(); alComments = new ArrayList(); len = 0; - bodyStart = 0; bodyLineNum = 0; } @@ -271,8 +358,24 @@ class GuidTable { return String.format(PcdDatabase.GuidTableExistenceMacro, phase, (al.size() == 0)? "TRUE":"FALSE"); } - public String getTypeDeclaration () { - return String.format(PcdDatabase.GuidTableDeclaration, phase); + public void genCode (ArrayList declaList, HashMap instTable, String phase) { + final String name = "GuidTable"; + + CStructTypeDeclaration decl; + String cCode = ""; + + cCode += String.format(PcdDatabase.GuidTableDeclaration, phase); + decl = new CStructTypeDeclaration ( + name, + 4, + cCode, + true + ); + declaList.add(decl); + + + cCode = PcdDatabase.genInstantiationStr(getInstantiation()); + instTable.put(name, cCode); } private String getUuidCString (UUID uuid) { @@ -280,7 +383,7 @@ class GuidTable { guidStrArray =(uuid.toString()).split("-"); - return String.format("{ 0x%s, 0x%s, 0x%s, { 0x%s, 0x%s, 0x%s, 0x%s, 0x%s, 0x%s, 0x%s, 0x%s } }", + return String.format("{0x%s, 0x%s, 0x%s, {0x%s, 0x%s, 0x%s, 0x%s, 0x%s, 0x%s, 0x%s, 0x%s}}", guidStrArray[0], guidStrArray[1], guidStrArray[2], @@ -295,22 +398,21 @@ class GuidTable { ); } - public ArrayList getInstantiation () { + private ArrayList getInstantiation () { ArrayList Output = new ArrayList(); Output.add("/* GuidTable */"); Output.add("{"); - bodyStart = 2; if (al.size() == 0) { - Output.add(getUuidCString(new UUID(0, 0))); + Output.add("\t" + getUuidCString(new UUID(0, 0))); } - for (Object u : al) { - UUID uuid = (UUID)u; - String str = getUuidCString(uuid); + for (int i = 0; i < al.size(); i++) { + String str = "\t" + getUuidCString(al.get(i)); - if (al.indexOf(u) != (al.size() - 1)) { + str += "/* " + alComments.get(i) + " */"; + if (i != (al.size() - 1)) { str += ","; } Output.add(str); @@ -322,45 +424,45 @@ class GuidTable { return Output; } - public int getBodyStart() { - return bodyStart; - } - - public int getBodyLineNum () { - return bodyLineNum; - } - public int add (UUID uuid, String name) { - int index = len; // - // Include the NULL character at the end of String + // Check if GuidTable has this entry already. + // If so, return the GuidTable index. // + for (int i = 0; i < al.size(); i++) { + if (al.get(i).compareTo(uuid) == 0) { + return i; + } + } + len++; al.add(uuid); + alComments.add(name); - return index; - } - - public int getTableLen () { - return al.size() == 0 ? 0 : al.size(); + // + // Return the previous Table Index + // + return len - 1; } } +/** + SkuIdTable + + This class is used to store the SKU IDs in a PCD database. + +**/ class SkuIdTable { private ArrayList al; private ArrayList alComment; private String phase; private int len; - private int bodyStart; - private int bodyLineNum; public SkuIdTable (String phase) { this.phase = phase; al = new ArrayList(); alComment = new ArrayList(); - bodyStart = 0; - bodyLineNum = 0; len = 0; } @@ -369,26 +471,57 @@ class SkuIdTable { } private int getSize () { - return (al.size() == 0)? 1 : al.size(); + return (len == 0)? 1 : len; } public String getExistanceMacro () { return String.format(PcdDatabase.SkuTableExistenceMacro, phase, (al.size() == 0)? "TRUE":"FALSE"); } - public String getTypeDeclaration () { - return String.format(PcdDatabase.SkuIdTableDeclaration, phase); + public void genCode (ArrayList declaList, HashMap instTable, String phase) { + final String name = "SkuIdTable"; + + CStructTypeDeclaration decl; + String cCode = ""; + + cCode += String.format(PcdDatabase.SkuIdTableDeclaration, phase); + decl = new CStructTypeDeclaration ( + name, + 1, + cCode, + true + ); + declaList.add(decl); + + + cCode = PcdDatabase.genInstantiationStr(getInstantiation()); + instTable.put(name, cCode); + + // + // SystemSkuId is in PEI phase PCD Database + // + if (phase.equalsIgnoreCase("PEI")) { + decl = new CStructTypeDeclaration ( + "SystemSkuId", + 1, + String.format("%-20sSystemSkuId;\r\n", "SKU_ID"), + true + ); + declaList.add(decl); + + instTable.put("SystemSkuId", "0"); + } + } - public ArrayList getInstantiation () { + private ArrayList getInstantiation () { ArrayList Output = new ArrayList (); Output.add("/* SkuIdTable */"); Output.add("{"); - bodyStart = 2; if (al.size() == 0) { - Output.add("0"); + Output.add("\t0"); } for (int index = 0; index < al.size(); index++) { @@ -400,16 +533,15 @@ class SkuIdTable { Integer[] ia = al.get(index); - str += ia[0].toString() + ", "; + str += "\t" + ia[0].toString() + ", "; for (int index2 = 1; index2 < ia.length; index2++) { str += ia[index2].toString(); - if (index != al.size() - 1) { + if (!((index2 == ia.length - 1) && (index == al.size() - 1))) { str += ", "; } } Output.add(str); - bodyLineNum++; } @@ -421,6 +553,31 @@ class SkuIdTable { public int add (Token token) { int index; + int pos; + + // + // Check if this SKU_ID Array is already in the table + // + pos = 0; + for (Object o: al) { + Integer [] s = (Integer[]) o; + boolean different = false; + if (s[0] == token.getSkuIdCount()) { + for (index = 1; index < s.length; index++) { + if (s[index] != token.skuData.get(index-1).id) { + different = true; + break; + } + } + } else { + different = true; + } + if (different) { + pos += s[0] + 1; + } else { + return pos; + } + } Integer [] skuIds = new Integer[token.skuData.size() + 1]; skuIds[0] = new Integer(token.skuData.size()); @@ -437,10 +594,6 @@ class SkuIdTable { return index; } - public int getTableLen () { - return al.size() == 0 ? 1 : al.size(); - } - } class LocalTokenNumberTable { @@ -470,24 +623,39 @@ class LocalTokenNumberTable { return String.format(PcdDatabase.DatabaseExistenceMacro, phase, (al.size() == 0)? "TRUE":"FALSE"); } - public String getTypeDeclaration () { - return String.format(PcdDatabase.LocalTokenNumberTableDeclaration, phase); + public void genCode (ArrayList declaList, HashMap instTable, String phase) { + final String name = "LocalTokenNumberTable"; + + CStructTypeDeclaration decl; + String cCode = ""; + + cCode += String.format(PcdDatabase.LocalTokenNumberTableDeclaration, phase); + decl = new CStructTypeDeclaration ( + name, + 4, + cCode, + true + ); + declaList.add(decl); + + cCode = PcdDatabase.genInstantiationStr(getInstantiation()); + instTable.put(name, cCode); } - public ArrayList getInstantiation () { + private ArrayList getInstantiation () { ArrayList output = new ArrayList(); output.add("/* LocalTokenNumberTable */"); output.add("{"); if (al.size() == 0) { - output.add("0"); + output.add("\t0"); } for (int index = 0; index < al.size(); index++) { String str; - str = (String)al.get(index); + str = "\t" + (String)al.get(index); str += " /* " + alComment.get(index) + " */ "; @@ -513,7 +681,7 @@ class LocalTokenNumberTable { str = String.format(PcdDatabase.offsetOfStrTemplate, phase, token.hasDefaultValue() ? "Init" : "Uninit", token.getPrimaryKeyString()); - if (token.isStringType()) { + if (token.isUnicodeStringType()) { str += " | PCD_TYPE_STRING"; } @@ -529,6 +697,25 @@ class LocalTokenNumberTable { str += " | PCD_TYPE_VPD"; } + switch (token.datumType) { + case UINT8: + case BOOLEAN: + str += " | PCD_DATUM_TYPE_UINT8"; + break; + case UINT16: + str += " | PCD_DATUM_TYPE_UINT16"; + break; + case UINT32: + str += " | PCD_DATUM_TYPE_UINT32"; + break; + case UINT64: + str += " | PCD_DATUM_TYPE_UINT64"; + break; + case POINTER: + str += " | PCD_DATUM_TYPE_POINTER"; + break; + } + al.add(str); alComment.add(token.getPrimaryKeyString()); @@ -536,8 +723,22 @@ class LocalTokenNumberTable { } } +/** + ExMapTable + + This class is used to store the table of mapping information + between DynamicEX ID pair(Guid, TokenNumber) and + the local token number assigned by PcdDatabase class. +**/ class ExMapTable { + /** + ExTriplet + + This class is used to store the mapping information + between DynamicEX ID pair(Guid, TokenNumber) and + the local token number assigned by PcdDatabase class. + **/ class ExTriplet { public Integer guidTableIdx; public Long exTokenNumber; @@ -551,18 +752,15 @@ class ExMapTable { } private ArrayList al; - private ArrayList alComment; + private Map alComment; private String phase; private int len; - private int bodyStart; private int bodyLineNum; - private int base; - + public ExMapTable (String phase) { this.phase = phase; al = new ArrayList(); - alComment = new ArrayList(); - bodyStart = 0; + alComment = new HashMap(); bodyLineNum = 0; len = 0; } @@ -572,27 +770,39 @@ class ExMapTable { + String.format(PcdDatabase.ExTokenNumber, phase, al.size()); } - private int getSize () { - return (al.size() == 0)? 1 : al.size(); - } - public String getExistanceMacro () { return String.format(PcdDatabase.ExMapTableExistenceMacro, phase, (al.size() == 0)? "TRUE":"FALSE"); } - public String getTypeDeclaration () { - return String.format(PcdDatabase.ExMapTableDeclaration, phase); - } + public void genCode (ArrayList declaList, HashMap instTable, String phase) { + final String exMapTableName = "ExMapTable"; + + sortTable(); + + CStructTypeDeclaration decl; + String cCode = ""; - public ArrayList getInstantiation () { + cCode += String.format(PcdDatabase.ExMapTableDeclaration, phase); + decl = new CStructTypeDeclaration ( + exMapTableName, + 4, + cCode, + true + ); + declaList.add(decl); + + + cCode = PcdDatabase.genInstantiationStr(getInstantiation()); + instTable.put(exMapTableName, cCode); + } + + private ArrayList getInstantiation () { ArrayList Output = new ArrayList(); Output.add("/* ExMapTable */"); Output.add("{"); - bodyStart = 2; - if (al.size() == 0) { - Output.add("{0, 0, 0}"); + Output.add("\t{0, 0, 0}"); } int index; @@ -601,11 +811,11 @@ class ExMapTable { ExTriplet e = (ExTriplet)al.get(index); - str = "{ " + e.exTokenNumber.toString() + ", "; + str = "\t" + "{ " + String.format("0x%08X", e.exTokenNumber) + ", "; str += e.localTokenIdx.toString() + ", "; str += e.guidTableIdx.toString(); - str += " /* " + alComment.get(index) + " */"; + str += "}" + " /* " + alComment.get(e) + " */" ; if (index != al.size() - 1) { str += ","; @@ -624,34 +834,77 @@ class ExMapTable { public int add (int localTokenIdx, long exTokenNum, int guidTableIdx, String name) { int index = len; - len++; - al.add(new ExTriplet(guidTableIdx, exTokenNum, localTokenIdx)); - alComment.add(name); + len++; + ExTriplet et = new ExTriplet(guidTableIdx, exTokenNum, localTokenIdx); + + al.add(et); + alComment.put(et, name); return index; } - public int getTableLen () { + private int getTableLen () { return al.size() == 0 ? 1 : al.size(); } + // + // To simplify the algorithm for GetNextToken and GetNextTokenSpace in + // PCD PEIM/Driver, we need to sort the ExMapTable according to the + // following order: + // 1) ExGuid + // 2) ExTokenNumber + // + class ExTripletComp implements Comparator { + public int compare (ExTriplet a, ExTriplet b) { + if (a.guidTableIdx == b.guidTableIdx ) { + // + // exTokenNumber is long, we can't use simple substraction. + // + if (a.exTokenNumber > b.exTokenNumber) { + return 1; + } else if (a.exTokenNumber == b.exTokenNumber) { + return 0; + } else { + return -1; + } + } + + return a.guidTableIdx - b.guidTableIdx; + } + } + + private void sortTable () { + java.util.Comparator comparator = new ExTripletComp(); + java.util.Collections.sort(al, comparator); + } } +/** + PcdDatabase + + This class is used to generate C code for Autogen.h and Autogen.c of + a PCD service DXE driver and PCD service PEIM. +**/ class PcdDatabase { - public final static String ExMapTableDeclaration = "DYNAMICEX_MAPPING ExMapTable[%s_EXMAPPING_TABLE_SIZE];\r\n"; - public final static String GuidTableDeclaration = "EFI_GUID GuidTable[%s_GUID_TABLE_SIZE];\r\n"; - public final static String LocalTokenNumberTableDeclaration = "UINT32 LocalTokenNumberTable[%s_LOCAL_TOKEN_NUMBER_TABLE_SIZE];\r\n"; - public final static String StringTableDeclaration = "UINT16 StringTable[%s_STRING_TABLE_SIZE];\r\n"; - public final static String SizeTableDeclaration = "UINT16 SizeTable[%s_LOCAL_TOKEN_NUMBER_TABLE_SIZE];\r\n"; - public final static String SkuIdTableDeclaration = "UINT8 SkuIdTable[%s_SKUID_TABLE_SIZE];\r\n"; + private final static int SkuHeadAlignmentSize = 4; + private final String newLine = "\r\n"; + private final String commaNewLine = ",\r\n"; + private final String tab = "\t"; + public final static String ExMapTableDeclaration = "DYNAMICEX_MAPPING ExMapTable[%s_EXMAPPING_TABLE_SIZE];\r\n"; + public final static String GuidTableDeclaration = "EFI_GUID GuidTable[%s_GUID_TABLE_SIZE];\r\n"; + public final static String LocalTokenNumberTableDeclaration = "UINT32 LocalTokenNumberTable[%s_LOCAL_TOKEN_NUMBER_TABLE_SIZE];\r\n"; + public final static String StringTableDeclaration = "UINT16 StringTable[%s_STRING_TABLE_SIZE];\r\n"; + public final static String SizeTableDeclaration = "SIZE_INFO SizeTable[%s_SIZE_TABLE_SIZE];\r\n"; + public final static String SkuIdTableDeclaration = "UINT8 SkuIdTable[%s_SKUID_TABLE_SIZE];\r\n"; public final static String ExMapTableSizeMacro = "#define %s_EXMAPPING_TABLE_SIZE %d\r\n"; public final static String ExTokenNumber = "#define %s_EX_TOKEN_NUMBER %d\r\n"; - public final static String GuidTableSizeMacro = "#define %s_GUID_TABLE_SIZE %d\r\n"; + public final static String GuidTableSizeMacro = "#define %s_GUID_TABLE_SIZE %d\r\n"; public final static String LocalTokenNumberTableSizeMacro = "#define %s_LOCAL_TOKEN_NUMBER_TABLE_SIZE %d\r\n"; public final static String LocalTokenNumberSizeMacro = "#define %s_LOCAL_TOKEN_NUMBER %d\r\n"; + public final static String SizeTableSizeMacro = "#define %s_SIZE_TABLE_SIZE %d\r\n"; public final static String StringTableSizeMacro = "#define %s_STRING_TABLE_SIZE %d\r\n"; public final static String SkuIdTableSizeMacro = "#define %s_SKUID_TABLE_SIZE %d\r\n"; @@ -663,7 +916,11 @@ class PcdDatabase { public final static String SkuTableExistenceMacro = "#define %s_SKUID_TABLE_EMPTY %s\r\n"; public final static String offsetOfSkuHeadStrTemplate = "offsetof(%s_PCD_DATABASE, %s.%s_SkuDataTable)"; + public final static String offsetOfVariableEnabledDefault = "offsetof(%s_PCD_DATABASE, %s.%s_VariableDefault_%d)"; public final static String offsetOfStrTemplate = "offsetof(%s_PCD_DATABASE, %s.%s)"; + + private final static String skuDataTableTemplate = "SkuDataTable"; + private StringTable stringTable; private GuidTable guidTable; @@ -676,6 +933,12 @@ class PcdDatabase { private String phase; private int assignedTokenNumber; + // + // Use two class global variable to store + // temperary + // + private String privateGlobalName; + private String privateGlobalCCode; // // After Major changes done to the PCD // database generation class PcdDatabase @@ -683,19 +946,26 @@ class PcdDatabase { // also update the version number in PCD // service PEIM and DXE driver accordingly. // - private final int version = 1; + private final int version = 2; private String hString; private String cString; - - class AlignmentSizeComp implements Comparator { - public int compare (Token a, Token b) { - return getAlignmentSize(b) - - getAlignmentSize(a); - } - } - + /** + Constructor for PcdDatabase class. + +

We have two PCD dynamic(ex) database for the Framework implementation. One + for PEI phase and the other for DXE phase.

+ + @param alTokens A ArrayList of Dynamic(EX) PCD entry. + @param exePhase The phase to generate PCD database for: valid input + is "PEI" or "DXE". + @param startLen The starting Local Token Number for the PCD database. For + PEI phase, the starting Local Token Number starts from 0. + For DXE phase, the starting Local Token Number starts + from the total number of PCD entry of PEI phase. + @return void + **/ public PcdDatabase (ArrayList alTokens, String exePhase, int startLen) { phase = exePhase; @@ -706,100 +976,210 @@ class PcdDatabase { sizeTable = new SizeTable(phase); exMapTable = new ExMapTable(phase); - assignedTokenNumber = startLen; + // + // Local token number 0 is reserved for INVALID_TOKEN_NUMBER. + // So we will increment 1 for the startLen passed from the + // constructor. + // + assignedTokenNumber = startLen + 1; this.alTokens = alTokens; } - private void getTwoGroupsOfTokens (ArrayList alTokens, List initTokens, List uninitTokens) { + private void getNonExAndExTokens (ArrayList alTokens, List nexTokens, List exTokens) { for (int i = 0; i < alTokens.size(); i++) { Token t = (Token)alTokens.get(i); - if (t.hasDefaultValue()) { - initTokens.add(t); + if (t.isDynamicEx()) { + exTokens.add(t); } else { - uninitTokens.add(t); + nexTokens.add(t); } } return; } - private int getAlignmentSize (Token token) { - if (token.getDefaultSku().type == DynamicTokenValue.VALUE_TYPE.HII_TYPE) { + private int getDataTypeAlignmentSize (Token token) { + switch (token.datumType) { + case UINT8: + return 1; + case UINT16: return 2; - } - - if (token.getDefaultSku().type == DynamicTokenValue.VALUE_TYPE.VPD_TYPE) { + case UINT32: return 4; + case UINT64: + return 8; + case POINTER: + return 1; + case BOOLEAN: + return 1; + default: + return 1; } - - if (token.isStringType()) { - return 2; - } - + } + + private int getHiiPtrTypeAlignmentSize(Token token) { switch (token.datumType) { - case UINT8: - return 1; - case UINT16: - return 2; - case UINT32: - return 4; - case UINT64: - return 8; - case POINTER: - return 1; - case BOOLEAN: - return 1; + case UINT8: + return 1; + case UINT16: + return 2; + case UINT32: + return 4; + case UINT64: + return 8; + case POINTER: + if (token.isHiiEnable()) { + if (token.isHiiDefaultValueUnicodeStringType()) { + return 2; + } } return 1; - } - - public String getCString () { + case BOOLEAN: + return 1; + default: + return 1; + } + } + + private int getAlignmentSize (Token token) { + if (token.getDefaultSku().type == DynamicTokenValue.VALUE_TYPE.HII_TYPE) { + return 2; + } + + if (token.getDefaultSku().type == DynamicTokenValue.VALUE_TYPE.VPD_TYPE) { + return 4; + } + + if (token.isUnicodeStringType()) { + return 2; + } + + return getDataTypeAlignmentSize(token); + } + + public String getCString () { return cString; } public String getHString () { return hString; } + + private void genCodeWorker(Token t, + ArrayList declaList, + HashMap instTable, String phase) + throws EntityException { - public void genCode () - throws EntityException { + CStructTypeDeclaration decl; - final String newLine = "\r\n"; - final String declNewLine = ";\r\n"; - final String tab = "\t"; - final String commaNewLine = ", \r\n"; + // + // Insert SKU_HEAD if isSkuEnable is true + // + if (t.isSkuEnable()) { + int tableIdx; + tableIdx = skuIdTable.add(t); + decl = new CStructTypeDeclaration(t.getPrimaryKeyString(), + SkuHeadAlignmentSize, getSkuEnabledTypeDeclaration(t), true); + declaList.add(decl); + instTable.put(t.getPrimaryKeyString(), + getSkuEnabledTypeInstantiaion(t, tableIdx)); + } - int i; - ArrayList decla; - ArrayList inst; + // + // Insert PCD_ENTRY declaration and instantiation + // + getCDeclarationString(t); - String macroStr = ""; - String initDeclStr = ""; - String initInstStr = ""; - String uninitDeclStr = ""; + decl = new CStructTypeDeclaration(privateGlobalName, + getAlignmentSize(t), privateGlobalCCode, t.hasDefaultValue()); + declaList.add(decl); + + if (t.hasDefaultValue()) { + instTable.put(privateGlobalName, + getTypeInstantiation(t, declaList, instTable, phase) + ); + } - List initTokens = new ArrayList (); - List uninitTokens = new ArrayList (); + } + + private void ProcessTokens (List tokens, + ArrayList cStructDeclList, + HashMap cStructInstTable, + String phase + ) + throws EntityException { - HashMap > initCode = new HashMap> (); - HashMap > uninitCode = new HashMap> (); + for (int idx = 0; idx < tokens.size(); idx++) { + Token t = tokens.get(idx); + + genCodeWorker (t, cStructDeclList, cStructInstTable, phase); + + sizeTable.add(t); + localTokenNumberTable.add(t); + t.tokenNumber = assignedTokenNumber++; + + // + // Add a mapping if this dynamic PCD entry is a EX type + // + if (t.isDynamicEx()) { + exMapTable.add((int)t.tokenNumber, + t.dynamicExTokenNumber, + guidTable.add(translateSchemaStringToUUID(t.tokenSpaceName), t.getPrimaryKeyString()), + t.getPrimaryKeyString() + ); + } + } - getTwoGroupsOfTokens (alTokens, initTokens, uninitTokens); + } + + public void genCode () throws EntityException { + + ArrayList cStructDeclList = new ArrayList(); + HashMap cStructInstTable = new HashMap(); + + List nexTokens = new ArrayList (); + List exTokens = new ArrayList (); - // - // Generate Structure Declaration for PcdTokens without Default Value - // PEI_PCD_DATABASE_INIT - // - java.util.Comparator comparator = new AlignmentSizeComp(); - java.util.Collections.sort(initTokens, comparator); - initCode = processTokens(initTokens); + getNonExAndExTokens (alTokens, nexTokens, exTokens); // - // Generate Structure Declaration for PcdTokens without Default Value - // PEI_PCD_DATABASE_UNINIT + // We have to process Non-Ex type PCD entry first. The reason is + // that our optimization assumes that the Token Number of Non-Ex + // PCD entry start from 1 (for PEI phase) and grows continously upwards. + // + // EX type token number starts from the last Non-EX PCD entry and + // grows continously upwards. // - java.util.Collections.sort(uninitTokens, comparator); - uninitCode = processTokens(uninitTokens); + ProcessTokens (nexTokens, cStructDeclList, cStructInstTable, phase); + ProcessTokens (exTokens, cStructDeclList, cStructInstTable, phase); + + stringTable.genCode(cStructDeclList, cStructInstTable); + skuIdTable.genCode(cStructDeclList, cStructInstTable, phase); + exMapTable.genCode(cStructDeclList, cStructInstTable, phase); + localTokenNumberTable.genCode(cStructDeclList, cStructInstTable, phase); + sizeTable.genCode(cStructDeclList, cStructInstTable, phase); + guidTable.genCode(cStructDeclList, cStructInstTable, phase); + + hString = genCMacroCode (); + + HashMap result; + + result = genCStructCode(cStructDeclList, + cStructInstTable, + phase + ); + + hString += result.get("initDeclStr"); + hString += result.get("uninitDeclStr"); + + hString += String.format("#define PCD_%s_SERVICE_DRIVER_VERSION %d", phase, version); + + cString = newLine + newLine + result.get("initInstStr"); + + } + + private String genCMacroCode () { + String macroStr = ""; // // Generate size info Macro for all Tables @@ -809,6 +1189,7 @@ class PcdDatabase { macroStr += skuIdTable.getSizeMacro(); macroStr += localTokenNumberTable.getSizeMacro(); macroStr += exMapTable.getSizeMacro(); + macroStr += sizeTable.getSizeMacro(); // // Generate existance info Macro for all Tables @@ -819,105 +1200,108 @@ class PcdDatabase { macroStr += localTokenNumberTable.getExistanceMacro(); macroStr += exMapTable.getExistanceMacro(); + macroStr += newLine; + + return macroStr; + } + + private HashMap genCStructCode( + ArrayList declaList, + HashMap instTable, + String phase + ) { + + int i; + HashMap result = new HashMap(); + HashMap > alignmentInitDecl = new HashMap>(); + HashMap > alignmentUninitDecl = new HashMap>(); + HashMap > alignmentInitInst = new HashMap>(); + // - // Generate Structure Declaration for PcdTokens with Default Value - // for example PEI_PCD_DATABASE_INIT + // Initialize the storage for each alignment // - initDeclStr += "typedef struct {" + newLine; - { - initDeclStr += tab + exMapTable.getTypeDeclaration(); - initDeclStr += tab + guidTable.getTypeDeclaration(); - initDeclStr += tab + localTokenNumberTable.getTypeDeclaration(); - initDeclStr += tab + stringTable.getTypeDeclaration(); - initDeclStr += tab + sizeTable.getTypeDeclaration(); - initDeclStr += tab + skuIdTable.getTypeDeclaration(); - if (phase.equalsIgnoreCase("PEI")) { - initDeclStr += tab + "SKU_ID SystemSkuId;" + newLine; - } - - decla = initCode.get(new String("Declaration")); - for (i = 0; i < decla.size(); i++) { - initDeclStr += tab + decla.get(i) + declNewLine; - } - - // - // Generate Structure Declaration for PcdToken with SkuEnabled - // - decla = initCode.get("DeclarationForSku"); - - for (i = 0; i < decla.size(); i++) { - initDeclStr += tab + decla.get(i) + declNewLine; - } - } - initDeclStr += String.format("} %s_PCD_DATABASE_INIT;\r\n\r\n", phase); + for (i = 8; i > 0; i>>=1) { + alignmentInitDecl.put(new Integer(i), new ArrayList()); + alignmentInitInst.put(new Integer(i), new ArrayList()); + alignmentUninitDecl.put(new Integer(i), new ArrayList()); + } + + String initDeclStr = "typedef struct {" + newLine; + String initInstStr = String.format("%s_PCD_DATABASE_INIT g%sPcdDbInit = { ", phase.toUpperCase(), phase.toUpperCase()) + newLine; + String uninitDeclStr = "typedef struct {" + newLine; // - // Generate MACRO for structure intialization of PCDTokens with Default Value - // The sequence must match the sequence of declaration of the memembers in the structure - String tmp = String.format("%s_PCD_DATABASE_INIT g%sPcdDbInit = { ", phase.toUpperCase(), phase.toUpperCase()); - initInstStr += tmp + newLine; - initInstStr += tab + genInstantiationStr(exMapTable.getInstantiation()) + commaNewLine; - initInstStr += tab + genInstantiationStr(guidTable.getInstantiation()) + commaNewLine; - initInstStr += tab + genInstantiationStr(localTokenNumberTable.getInstantiation()) + commaNewLine; - initInstStr += tab + genInstantiationStr(stringTable.getInstantiation()) + commaNewLine; - initInstStr += tab + genInstantiationStr(sizeTable.getInstantiation()) + commaNewLine; - initInstStr += tab + genInstantiationStr(skuIdTable.getInstantiation()) + commaNewLine; - // - // For SystemSkuId + // Sort all C declaration and instantiation base on Alignment Size // - if (phase.equalsIgnoreCase("PEI")) { - initInstStr += tab + "0" + tab + "/* SystemSkuId */" + commaNewLine; - } - - inst = initCode.get("Instantiation"); - for (i = 0; i < inst.size(); i++) { - initInstStr += tab + inst.get(i) + commaNewLine; - } - - inst = initCode.get("InstantiationForSku"); - for (i = 0; i < inst.size(); i++) { - initInstStr += tab + inst.get(i); - if (i != inst.size() - 1) { - initInstStr += commaNewLine; + for (Object d : declaList) { + CStructTypeDeclaration decl = (CStructTypeDeclaration) d; + + if (decl.initTable) { + alignmentInitDecl.get(new Integer(decl.alignmentSize)).add(decl.cCode); + alignmentInitInst.get(new Integer(decl.alignmentSize)).add(instTable.get(decl.key)); + } else { + alignmentUninitDecl.get(new Integer(decl.alignmentSize)).add(decl.cCode); } } - initInstStr += "};"; - - uninitDeclStr += "typedef struct {" + newLine; - { - decla = uninitCode.get("Declaration"); - if (decla.size() == 0) { - uninitDeclStr += "UINT8 dummy /* The UINT struct is empty */" + declNewLine; + // + // Generate code for every alignment size + // + boolean uinitDatabaseEmpty = true; + for (int align = 8; align > 0; align >>= 1) { + ArrayList declaListBasedOnAlignment = alignmentInitDecl.get(new Integer(align)); + ArrayList instListBasedOnAlignment = alignmentInitInst.get(new Integer(align)); + for (i = 0; i < declaListBasedOnAlignment.size(); i++) { + initDeclStr += tab + declaListBasedOnAlignment.get(i); + initInstStr += tab + instListBasedOnAlignment.get(i); + + // + // We made a assumption that both PEI_PCD_DATABASE and DXE_PCD_DATABASE + // has a least one data memember with alignment size of 1. So we can + // remove the last "," in the C structure instantiation string. Luckily, + // this is true as both data structure has SKUID_TABLE anyway. + // + if ((align == 1) && (i == declaListBasedOnAlignment.size() - 1)) { + initInstStr += newLine; } else { - - for (i = 0; i < decla.size(); i++) { - uninitDeclStr += tab + decla.get(i) + declNewLine; - } - - decla = uninitCode.get("DeclarationForSku"); - - for (i = 0; i < decla.size(); i++) { - uninitDeclStr += tab + decla.get(i) + declNewLine; - } + initInstStr += commaNewLine; } } - uninitDeclStr += String.format("} %s_PCD_DATABASE_UNINIT;\r\n\r\n", phase); - - cString = initInstStr + newLine; - hString = macroStr + newLine - + initDeclStr + newLine - + uninitDeclStr + newLine - + newLine; + + declaListBasedOnAlignment = alignmentUninitDecl.get(new Integer(align)); + + if (declaListBasedOnAlignment.size() != 0) { + uinitDatabaseEmpty = false; + } + + for (Object d : declaListBasedOnAlignment) { + String s = (String)d; + uninitDeclStr += tab + s; + } + } - hString += String.format("#define PCD_%s_SERVICE_DRIVER_VERSION %d", phase, version); + if (uinitDatabaseEmpty) { + uninitDeclStr += tab + String.format("%-20sdummy; /* PCD_DATABASE_UNINIT is emptry */\r\n", "UINT8"); + } + + initDeclStr += String.format("} %s_PCD_DATABASE_INIT;", phase) + newLine + newLine; + initInstStr += "};" + newLine; + uninitDeclStr += String.format("} %s_PCD_DATABASE_UNINIT;", phase) + newLine + newLine; + + result.put("initDeclStr", initDeclStr); + result.put("initInstStr", initInstStr); + result.put("uninitDeclStr", uninitDeclStr); + return result; } - private String genInstantiationStr (ArrayList alStr) { + public static String genInstantiationStr (ArrayList alStr) { String str = ""; for (int i = 0; i< alStr.size(); i++) { - str += "\t" + alStr.get(i); + if (i != 0) { + str += "\t"; + } + str += alStr.get(i); if (i != alStr.size() - 1) { str += "\r\n"; } @@ -926,134 +1310,86 @@ class PcdDatabase { return str; } - private HashMap> processTokens (List alToken) - throws EntityException { - - HashMap > map = new HashMap>(); - - ArrayList decl = new ArrayList(); - ArrayList declForSkuEnableType = new ArrayList(); - ArrayList inst = new ArrayList(); - ArrayList instForSkuEnableType = new ArrayList(); - - for (int index = 0; index < alToken.size(); index++) { - Token token = alToken.get(index); - - if (token.isSkuEnable()) { - // - // BugBug: Schema only support Data type now - // - int tableIdx; - - tableIdx = skuIdTable.add(token); - - decl.add(getSkuEnabledTypeDeclaration(token)); - if (token.hasDefaultValue()) { - inst.add(getSkuEnabledTypeInstantiaion(token, tableIdx)); - } - - declForSkuEnableType.add(getDataTypeDeclarationForSkuEnabled(token)); - if (token.hasDefaultValue()) { - instForSkuEnableType.add(getDataTypeInstantiationForSkuEnabled(token)); - } - - } else { - if (token.getDefaultSku().type == DynamicTokenValue.VALUE_TYPE.HII_TYPE) { - decl.add(getVariableEnableTypeDeclaration(token)); - inst.add(getVariableEnableInstantiation(token)); - } else if (token.getDefaultSku().type == DynamicTokenValue.VALUE_TYPE.VPD_TYPE) { - decl.add(getVpdEnableTypeDeclaration(token)); - inst.add(getVpdEnableTypeInstantiation(token)); - } else if (token.isStringType()) { - decl.add(getStringTypeDeclaration(token)); - inst.add(getStringTypeInstantiation(stringTable.add(token.getStringTypeString(), token), token)); - } - else { - decl.add(getDataTypeDeclaration(token)); - if (token.hasDefaultValue()) { - inst.add(getDataTypeInstantiation(token)); - } - } - } - - sizeTable.add(token); - localTokenNumberTable.add(token); - token.tokenNumber = assignedTokenNumber++; - - } - - map.put("Declaration", decl); - map.put("DeclarationForSku", declForSkuEnableType); - map.put("Instantiation", inst); - map.put("InstantiationForSku", instForSkuEnableType); - - return map; - } - private String getSkuEnabledTypeDeclaration (Token token) { - return String.format("SKU_HEAD %s;\r\n", token.getPrimaryKeyString()); + return String.format("%-20s%s;\r\n", "SKU_HEAD", token.getPrimaryKeyString()); } private String getSkuEnabledTypeInstantiaion (Token token, int SkuTableIdx) { String offsetof = String.format(PcdDatabase.offsetOfSkuHeadStrTemplate, phase, token.hasDefaultValue()? "Init" : "Uninit", token.getPrimaryKeyString()); - return String.format("{ %s, %d }", offsetof, SkuTableIdx); + return String.format("{ %s, %d } /* SKU_ENABLED: %s */", offsetof, SkuTableIdx, token.getPrimaryKeyString()); } - private String getDataTypeDeclarationForSkuEnabled (Token token) { - String typeStr = ""; - - if (token.datumType == Token.DATUM_TYPE.UINT8) { - typeStr = "UINT8 %s_%s[%d];\r\n"; - } else if (token.datumType == Token.DATUM_TYPE.UINT16) { - typeStr = "UINT16 %s_%s[%d];\r\n"; - } else if (token.datumType == Token.DATUM_TYPE.UINT32) { - typeStr = "UINT32 %s_%s[%d];\r\n"; - } else if (token.datumType == Token.DATUM_TYPE.UINT64) { - typeStr = "UINT64 %s_%s[%d];\r\n"; - } else if (token.datumType == Token.DATUM_TYPE.BOOLEAN) { - typeStr = "BOOLEAN %s_%s[%d];\r\n"; - } else if (token.datumType == Token.DATUM_TYPE.POINTER) { - return String.format("UINT8 %s_%s[%d];\r\n", token.getPrimaryKeyString(), "SkuDataTable", token.datumSize * token.skuData.size()); - } - - return String.format(typeStr, token.getPrimaryKeyString(), "SkuDataTable", token.skuData.size()); - + private String getDataTypeInstantiationForVariableDefault (Token token, String cName, int skuId) { + return String.format("%s /* %s */", token.skuData.get(skuId).value.hiiDefaultValue, cName); } - private String getDataTypeInstantiationForSkuEnabled (Token token) { - String str = ""; - - if (token.datumType == Token.DATUM_TYPE.POINTER) { - return String.format("UINT8 %s_%s[%d]", token.getPrimaryKeyString(), "SkuDataTable", token.datumSize * token.skuData.size()); - } else { - str = "{ "; - for (int idx = 0; idx < token.skuData.size(); idx++) { - str += token.skuData.get(idx).toString(); - if (idx != token.skuData.size() - 1) { - str += ", "; - } - } - str += "}"; - - return str; + private String getCType (Token t) + throws EntityException { + + if (t.isHiiEnable()) { + return "VARIABLE_HEAD"; + } + + if (t.isVpdEnable()) { + return "VPD_HEAD"; + } + + if (t.isUnicodeStringType()) { + return "STRING_HEAD"; + } + + switch (t.datumType) { + case UINT64: + return "UINT64"; + case UINT32: + return "UINT32"; + case UINT16: + return "UINT16"; + case UINT8: + return "UINT8"; + case BOOLEAN: + return "BOOLEAN"; + case POINTER: + return "UINT8"; + default: + throw new EntityException("Unknown type in getDataTypeCDeclaration"); } - } + + // + // privateGlobalName and privateGlobalCCode is used to pass output to caller of getCDeclarationString + // + private void getCDeclarationString(Token t) + throws EntityException { + + if (t.isSkuEnable()) { + privateGlobalName = String.format("%s_%s", t.getPrimaryKeyString(), skuDataTableTemplate); + } else { + privateGlobalName = t.getPrimaryKeyString(); + } - private String getDataTypeInstantiation (Token token) { - - if (token.datumType == Token.DATUM_TYPE.POINTER) { - return String.format("%s /* %s */", token.getDefaultSku().value, token.getPrimaryKeyString()); + String type = getCType(t); + if ((t.datumType == Token.DATUM_TYPE.POINTER) && (!t.isHiiEnable()) && (!t.isUnicodeStringType())) { + int bufferSize; + if (t.isASCIIStringType()) { + // + // Build tool will add a NULL string at the end of the ASCII string + // + bufferSize = t.datumSize + 1; + } else { + bufferSize = t.datumSize; + } + privateGlobalCCode = String.format("%-20s%s[%d][%d];\r\n", type, privateGlobalName, t.getSkuIdCount(), bufferSize); } else { - return String.format("%s /* %s */", token.getDefaultSku().value, token.getPrimaryKeyString()); + privateGlobalCCode = String.format("%-20s%s[%d];\r\n", type, privateGlobalName, t.getSkuIdCount()); } } + + private String getDataTypeDeclarationForVariableDefault (Token token, String cName, int skuId) + throws EntityException { - - private String getDataTypeDeclaration (Token token) { - - String typeStr = ""; + String typeStr; if (token.datumType == Token.DATUM_TYPE.UINT8) { typeStr = "UINT8"; @@ -1066,51 +1402,100 @@ class PcdDatabase { } else if (token.datumType == Token.DATUM_TYPE.BOOLEAN) { typeStr = "BOOLEAN"; } else if (token.datumType == Token.DATUM_TYPE.POINTER) { - return String.format("UINT8 %s[%d]", token.getPrimaryKeyString(), token.datumSize); + int size; + if (token.isHiiDefaultValueUnicodeStringType()) { + typeStr = "UINT16"; + // + // Include the NULL charactor + // + size = token.datumSize / 2 + 1; + } else { + typeStr = "UINT8"; + if (token.isHiiDefaultValueASCIIStringType()) { + // + // Include the NULL charactor + // + size = token.datumSize + 1; + } else { + size = token.datumSize; + } + } + return String.format("%-20s%s[%d];\r\n", typeStr, cName, size); } else { + throw new EntityException("Unknown DATUM_TYPE type in when generating code for VARIABLE_ENABLED PCD entry"); } - return String.format("%s %s", typeStr, token.getPrimaryKeyString()); - } - - private String getVpdEnableTypeDeclaration (Token token) { - return String.format("VPD_HEAD %s", token.getPrimaryKeyString()); - } - - private String getVpdEnableTypeInstantiation (Token token) { - return String.format("{ %s } /* %s */", token.getDefaultSku().vpdOffset, - token.getPrimaryKeyString()); - } - - private String getStringTypeDeclaration (Token token) { - return String.format("UINT16 %s", token.getPrimaryKeyString()); - } - - private String getStringTypeInstantiation (int StringTableIdx, Token token) { - return String.format ("%d /* %s */", StringTableIdx, - token.getPrimaryKeyString()); - } - - - private String getVariableEnableTypeDeclaration (Token token) { - return String.format("VARIABLE_HEAD %s", token.getPrimaryKeyString()); - } - - private String getVariableEnableInstantiation (Token token) - throws EntityException { - // - // Need scott fix - // - return String.format("{ %d, %d, %s } /* %s */", guidTable.add(token.getDefaultSku().variableGuid, token.getPrimaryKeyString()), - stringTable.add(token.getDefaultSku().getStringOfVariableName(), token), - token.getDefaultSku().variableOffset, - token.getPrimaryKeyString()); + return String.format("%-20s%s;\r\n", typeStr, cName); } + + private String getTypeInstantiation (Token t, ArrayList declaList, HashMap instTable, String phase) throws EntityException { + + int i; + + String s; + s = String.format("/* %s */", t.getPrimaryKeyString()) + newLine; + s += tab + "{" + newLine; + + for (i = 0; i < t.skuData.size(); i++) { + if (t.isUnicodeStringType()) { + s += tab + tab + String.format("{ %d }", stringTable.add(t.skuData.get(i).value.value, t)); + } else if (t.isHiiEnable()) { + /* VPD_HEAD definition + typedef struct { + UINT16 GuidTableIndex; // Offset in Guid Table in units of GUID. + UINT16 StringIndex; // Offset in String Table in units of UINT16. + UINT16 Offset; // Offset in Variable + UINT16 DefaultValueOffset; // Offset of the Default Value + } VARIABLE_HEAD ; + */ + String variableDefaultName = String.format("%s_VariableDefault_%d", t.getPrimaryKeyString(), i); + + s += tab + tab + String.format("{ %d, %d, %s, %s }", guidTable.add(t.skuData.get(i).value.variableGuid, t.getPrimaryKeyString()), + stringTable.add(t.skuData.get(i).value.getStringOfVariableName(), t), + t.skuData.get(i).value.variableOffset, + String.format("offsetof(%s_PCD_DATABASE, Init.%s)", phase, variableDefaultName) + ); + // + // We need to support the default value, so we add the declaration and + // the instantiation for the default value. + // + CStructTypeDeclaration decl = new CStructTypeDeclaration (variableDefaultName, + getHiiPtrTypeAlignmentSize(t), + getDataTypeDeclarationForVariableDefault(t, variableDefaultName, i), + true + ); + declaList.add(decl); + instTable.put(variableDefaultName, getDataTypeInstantiationForVariableDefault (t, variableDefaultName, i)); + } else if (t.isVpdEnable()) { + /* typedef struct { + UINT32 Offset; + } VPD_HEAD; + */ + s += tab + tab + String.format("{ %s }", t.skuData.get(i).value.vpdOffset); + } else { + if (t.isByteStreamType()) { + // + // Byte stream type input has their own "{" "}", so we won't help to insert. + // + s += tab + tab + String.format(" %s ", t.skuData.get(i).value.value); + } else { + s += tab + tab + String.format("{ %s }", t.skuData.get(i).value.value); + } + } + + if (i != t.skuData.size() - 1) { + s += commaNewLine; + } else { + s += newLine; + } - public int getTotalTokenNumber () { - return sizeTable.getTableLen(); + } + + s += tab + "}"; + + return s; } - + public static String getPcdDatabaseCommonDefinitions () throws EntityException { @@ -1180,15 +1565,119 @@ class PcdDatabase { return retStr; } + /** + Translate the schema string to UUID instance. + + In schema, the string of UUID is defined as following two types string: + 1) GuidArrayType: pattern = 0x[a-fA-F0-9]{1,8},( )*0x[a-fA-F0-9]{1,4},( + )*0x[a-fA-F0-9]{1,4}(,( )*\{)?(,?( )*0x[a-fA-F0-9]{1,2}){8}( )*(\})? + + 2) GuidNamingConvention: pattern = + [a-fA-F0-9]{8}-[a-fA-F0-9]{4}-[a-fA-F0-9]{4}-[a-fA-F0-9]{4}-[a-fA-F0-9]{12} + + This function will convert string and create uuid instance. + + @param uuidString UUID string in XML file + + @return UUID UUID instance + **/ + private UUID translateSchemaStringToUUID(String uuidString) + throws EntityException { + String temp; + String[] splitStringArray; + int index; + int chIndex; + int chLen; + + if (uuidString == null) { + return null; + } + + if (uuidString.length() == 0) { + return null; + } + + if (uuidString.equals("0") || + uuidString.equalsIgnoreCase("0x0")) { + return new UUID(0, 0); + } + + uuidString = uuidString.replaceAll("\\{", ""); + uuidString = uuidString.replaceAll("\\}", ""); + + // + // If the UUID schema string is GuidArrayType type then need translate + // to GuidNamingConvention type at first. + // + if ((uuidString.charAt(0) == '0') && ((uuidString.charAt(1) == 'x') || (uuidString.charAt(1) == 'X'))) { + splitStringArray = uuidString.split("," ); + if (splitStringArray.length != 11) { + throw new EntityException ("[FPD file error] Wrong format for UUID string: " + uuidString); + } + + // + // Remove blank space from these string and remove header string "0x" + // + for (index = 0; index < 11; index ++) { + splitStringArray[index] = splitStringArray[index].trim(); + splitStringArray[index] = splitStringArray[index].substring(2, splitStringArray[index].length()); + } + + // + // Add heading '0' to normalize the string length + // + for (index = 3; index < 11; index ++) { + chLen = splitStringArray[index].length(); + for (chIndex = 0; chIndex < 2 - chLen; chIndex ++) { + splitStringArray[index] = "0" + splitStringArray[index]; + } + } + + // + // construct the final GuidNamingConvention string + // + temp = String.format("%s-%s-%s-%s%s-%s%s%s%s%s%s", + splitStringArray[0], + splitStringArray[1], + splitStringArray[2], + splitStringArray[3], + splitStringArray[4], + splitStringArray[5], + splitStringArray[6], + splitStringArray[7], + splitStringArray[8], + splitStringArray[9], + splitStringArray[10]); + uuidString = temp; + } + + return UUID.fromString(uuidString); + } } +/** Module Info class is the data structure to hold information got from GlobalData. +*/ class ModuleInfo { - public ModuleSADocument.ModuleSA module; - public UsageInstance.MODULE_TYPE type; + /// + /// Module's ID for a + /// + private FpdModuleIdentification moduleId; + /// + /// xmlobject in FPD file for a + /// + private PcdBuildDefinitionDocument.PcdBuildDefinition pcdBuildDef; + + public ModuleInfo (FpdModuleIdentification moduleId, XmlObject pcdDef) { + this.moduleId = moduleId; + this.pcdBuildDef = ((PcdBuildDefinitionDocument)pcdDef).getPcdBuildDefinition(); + } + + public FpdModuleIdentification getModuleId (){ + return moduleId; + } - public ModuleInfo (ModuleSADocument.ModuleSA module, UsageInstance.MODULE_TYPE type) { - this.module = module; - this.type = type; + public PcdBuildDefinitionDocument.PcdBuildDefinition getPcdBuildDef(){ + return pcdBuildDef; } } @@ -1197,21 +1686,31 @@ class ModuleInfo { from buildAction or UIAction. **/ public class CollectPCDAction { + /// /// memoryDatabase hold all PCD information collected from SPD, MSA, FPD. + /// private MemoryDatabaseManager dbManager; - + /// /// Workspacepath hold the workspace information. + /// private String workspacePath; - + /// /// FPD file is the root file. + /// private String fpdFilePath; - + /// /// Message level for CollectPCDAction. + /// private int originalMessageLevel; - + /// /// Cache the fpd docment instance for private usage. - private FrameworkPlatformDescriptionDocument fpdDocInstance; - + /// + private PlatformSurfaceAreaDocument fpdDocInstance; + /// + /// xmlObject name + /// + private static String xmlObjectName = "PcdBuildDefinition"; + /** Set WorkspacePath parameter for this action class. @@ -1278,7 +1777,7 @@ public class CollectPCDAction { @throws EntityException Exception indicate failed to execute this action. **/ - private void execute() throws EntityException { + public void execute() throws EntityException { // // Get memoryDatabaseManager instance from GlobalData. // The memoryDatabaseManager should be initialized for whatever build @@ -1296,8 +1795,7 @@ public class CollectPCDAction { createTokenInDBFromFPD(); // - // Call Private function genPcdDatabaseSourceCode (void); ComponentTypeBsDriver - // 1) Generate for PEI, DXE PCD DATABASE's definition and initialization. + // Generate for PEI, DXE PCD DATABASE's definition and initialization. // genPcdDatabaseSourceCode (); @@ -1312,7 +1810,7 @@ public class CollectPCDAction { **/ private void genPcdDatabaseSourceCode() throws EntityException { - String PcdCommonHeaderString = PcdDatabase.getPcdDatabaseCommonDefinitions (); + String PcdCommonHeaderString = PcdDatabase.getPcdDatabaseCommonDefinitions(); ArrayList alPei = new ArrayList (); ArrayList alDxe = new ArrayList (); @@ -1320,18 +1818,15 @@ public class CollectPCDAction { dbManager.getTwoPhaseDynamicRecordArray(alPei, alDxe); PcdDatabase pcdPeiDatabase = new PcdDatabase (alPei, "PEI", 0); pcdPeiDatabase.genCode(); - dbManager.PcdPeimHString = PcdCommonHeaderString + pcdPeiDatabase.getHString() - + PcdDatabase.getPcdPeiDatabaseDefinitions(); - dbManager.PcdPeimCString = pcdPeiDatabase.getCString(); - - PcdDatabase pcdDxeDatabase = new PcdDatabase (alDxe, - "DXE", - alPei.size() - ); + MemoryDatabaseManager.PcdPeimHString = PcdCommonHeaderString + pcdPeiDatabase.getHString() + + PcdDatabase.getPcdPeiDatabaseDefinitions(); + MemoryDatabaseManager.PcdPeimCString = pcdPeiDatabase.getCString(); + + PcdDatabase pcdDxeDatabase = new PcdDatabase(alDxe, "DXE", alPei.size()); pcdDxeDatabase.genCode(); - dbManager.PcdDxeHString = dbManager.PcdPeimHString + pcdDxeDatabase.getHString() - + PcdDatabase.getPcdDxeDatabaseDefinitions(); - dbManager.PcdDxeCString = pcdDxeDatabase.getCString(); + MemoryDatabaseManager.PcdDxeHString = MemoryDatabaseManager.PcdPeimHString + pcdDxeDatabase.getHString() + + PcdDatabase.getPcdDxeDatabaseDefinitions(); + MemoryDatabaseManager.PcdDxeCString = pcdDxeDatabase.getCString(); } /** @@ -1344,84 +1839,23 @@ public class CollectPCDAction { */ private List getComponentsFromFPD() throws EntityException { - HashMap map = new HashMap(); - List allModules = new ArrayList(); - ModuleInfo current = null; - int index = 0; - org.tianocore.Components components = null; - FrameworkModulesDocument.FrameworkModules fModules = null; - java.util.List modules = null; - - - if (fpdDocInstance == null) { - try { - fpdDocInstance = (FrameworkPlatformDescriptionDocument)XmlObject.Factory.parse(new File(fpdFilePath)); - } catch(IOException ioE) { - throw new EntityException("File IO error for xml file:" + fpdFilePath + "\n" + ioE.getMessage()); - } catch(XmlException xmlE) { - throw new EntityException("Can't parse the FPD xml fle:" + fpdFilePath + "\n" + xmlE.getMessage()); - } + List allModules = new ArrayList(); + FrameworkModulesDocument.FrameworkModules fModules = null; + ModuleSADocument.ModuleSA[] modules = null; + Map pcdBuildDefinitions = null; - } - - // - // Check whether FPD contians - // - fModules = fpdDocInstance.getFrameworkPlatformDescription().getFrameworkModules(); - if (fModules == null) { + pcdBuildDefinitions = GlobalData.getFpdPcdBuildDefinitions(); + if (pcdBuildDefinitions == null) { return null; } // - // BUGBUG: The following is work around code, the final component type should be get from - // GlobalData class. - // - components = fModules.getSEC(); - if (components != null) { - modules = components.getModuleSAList(); - for (index = 0; index < modules.size(); index ++) { - allModules.add(new ModuleInfo(modules.get(index), UsageInstance.MODULE_TYPE.SEC)); - } - } - - components = fModules.getPEICORE(); - if (components != null) { - modules = components.getModuleSAList(); - for (index = 0; index < modules.size(); index ++) { - allModules.add(new ModuleInfo(modules.get(index), UsageInstance.MODULE_TYPE.PEI_CORE)); - } - } - - components = fModules.getPEIM(); - if (components != null) { - modules = components.getModuleSAList(); - for (index = 0; index < modules.size(); index ++) { - allModules.add(new ModuleInfo(modules.get(index), UsageInstance.MODULE_TYPE.PEIM)); - } - } - - components = fModules.getDXECORE(); - if (components != null) { - modules = components.getModuleSAList(); - for (index = 0; index < modules.size(); index ++) { - allModules.add(new ModuleInfo(modules.get(index), UsageInstance.MODULE_TYPE.DXE_CORE)); - } - } - - components = fModules.getDXEDRIVERS(); - if (components != null) { - modules = components.getModuleSAList(); - for (index = 0; index < modules.size(); index ++) { - allModules.add(new ModuleInfo(modules.get(index), UsageInstance.MODULE_TYPE.DXE_DRIVERS)); - } - } - - components = fModules.getOTHERCOMPONENTS(); - if (components != null) { - modules = components.getModuleSAList(); - for (index = 0; index < modules.size(); index ++) { - allModules.add(new ModuleInfo(modules.get(index), UsageInstance.MODULE_TYPE.OTHER_COMPONENTS)); - } + // Loop map to retrieve all PCD build definition and Module id + // + Iterator item = pcdBuildDefinitions.keySet().iterator(); + while (item.hasNext()){ + FpdModuleIdentification id = (FpdModuleIdentification) item.next(); + allModules.add(new ModuleInfo(id, pcdBuildDefinitions.get(id))); } return allModules; @@ -1445,8 +1879,6 @@ public class CollectPCDAction { List pcdBuildDataArray = new ArrayList(); PcdBuildDefinition.PcdData pcdBuildData = null; Token token = null; - SkuInstance skuInstance = null; - int skuIndex = 0; List modules = null; String primaryKey = null; String exceptionString = null; @@ -1456,10 +1888,11 @@ public class CollectPCDAction { boolean isDuplicate = false; Token.PCD_TYPE pcdType = Token.PCD_TYPE.UNKNOWN; Token.DATUM_TYPE datumType = Token.DATUM_TYPE.UNKNOWN; - int tokenNumber = 0; + long tokenNumber = 0; String moduleName = null; String datum = null; int maxDatumSize = 0; + String[] tokenSpaceStrRet = null; // // ---------------------------------------------- @@ -1478,44 +1911,16 @@ public class CollectPCDAction { // ------------------------------------------------------------------- // for (index = 0; index < modules.size(); index ++) { - isDuplicate = false; - for (index2 = 0; index2 < index; index2 ++) { - // - // BUGBUG: For transition schema, we can *not* get module's version from - // , It is work around code. - // - primaryKey1 = UsageInstance.getPrimaryKey(modules.get(index).module.getModuleName(), - translateSchemaStringToUUID(modules.get(index).module.getModuleGuid()), - modules.get(index).module.getPackageName(), - translateSchemaStringToUUID(modules.get(index).module.getPackageGuid()), - modules.get(index).module.getArch().toString(), - null); - primaryKey2 = UsageInstance.getPrimaryKey(modules.get(index2).module.getModuleName(), - translateSchemaStringToUUID(modules.get(index2).module.getModuleGuid()), - modules.get(index2).module.getPackageName(), - translateSchemaStringToUUID(modules.get(index2).module.getPackageGuid()), - modules.get(index2).module.getArch().toString(), - null); - if (primaryKey1.equalsIgnoreCase(primaryKey2)) { - isDuplicate = true; - break; - } - } - - if (isDuplicate) { - continue; - } - // // It is legal for a module does not contains ANY pcd build definitions. // - if (modules.get(index).module.getPcdBuildDefinition() == null) { + if (modules.get(index).getPcdBuildDef() == null) { continue; } - pcdBuildDataArray = modules.get(index).module.getPcdBuildDefinition().getPcdDataList(); + pcdBuildDataArray = modules.get(index).getPcdBuildDef().getPcdDataList(); - moduleName = modules.get(index).module.getModuleName(); + moduleName = modules.get(index).getModuleId().getModule().getName(); // // ---------------------------------------------------------------------- @@ -1524,14 +1929,37 @@ public class CollectPCDAction { // for (pcdIndex = 0; pcdIndex < pcdBuildDataArray.size(); pcdIndex ++) { pcdBuildData = pcdBuildDataArray.get(pcdIndex); - primaryKey = Token.getPrimaryKeyString(pcdBuildData.getCName(), - translateSchemaStringToUUID(pcdBuildData.getTokenSpaceGuid())); + + try { + tokenSpaceStrRet = GlobalData.getGuidInfoFromCname(pcdBuildData.getTokenSpaceGuidCName()); + } catch ( Exception e ) { + throw new EntityException ("Faile get Guid for token " + pcdBuildData.getCName() + ":" + e.getMessage()); + } + + if (tokenSpaceStrRet == null) { + throw new EntityException ("Fail to get Token space guid for token" + pcdBuildData.getCName()); + } + + primaryKey = Token.getPrimaryKeyString(pcdBuildData.getCName(), tokenSpaceStrRet[1]); pcdType = Token.getpcdTypeFromString(pcdBuildData.getItemType().toString()); datumType = Token.getdatumTypeFromString(pcdBuildData.getDatumType().toString()); - tokenNumber = Integer.decode(pcdBuildData.getToken().toString()); - datum = pcdBuildData.getValue(); + tokenNumber = Long.decode(pcdBuildData.getToken().toString()); + if (pcdBuildData.getValue() != null) { + datum = pcdBuildData.getValue().toString(); + } else { + datum = null; + } maxDatumSize = pcdBuildData.getMaxDatumSize(); + if ((pcdType == Token.PCD_TYPE.FEATURE_FLAG) && + (datumType != Token.DATUM_TYPE.BOOLEAN)){ + exceptionString = String.format("[FPD file error] For PCD %s in module %s, the PCD type is FEATRUE_FLAG but "+ + "datum type of this PCD entry is not BOOLEAN!", + pcdBuildData.getCName(), + moduleName); + throw new EntityException(exceptionString); + } + // // ------------------------------------------------------------------------------------------- // 2.1.1), Do some necessary checking work for FixedAtBuild, FeatureFlag and PatchableInModule @@ -1551,10 +1979,11 @@ public class CollectPCDAction { // // Check whether the datum size is matched datum type. // - if ((exceptionString = verifyDatumSize(pcdBuildData.getCName(), - moduleName, - maxDatumSize, - datumType)) != null) { + if ((exceptionString = verifyDatum(pcdBuildData.getCName(), + moduleName, + datum, + datumType, + maxDatumSize)) != null) { throw new EntityException(exceptionString); } } @@ -1640,8 +2069,17 @@ public class CollectPCDAction { // If the token is not in database, create a new token instance and add // a usage instance into this token in database. // - token = new Token(pcdBuildData.getCName(), - translateSchemaStringToUUID(pcdBuildData.getTokenSpaceGuid())); + try { + tokenSpaceStrRet = GlobalData.getGuidInfoFromCname(pcdBuildData.getTokenSpaceGuidCName()); + } catch (Exception e) { + throw new EntityException("Fail to get token space guid for token " + token.cName); + } + + if (tokenSpaceStrRet == null) { + throw new EntityException("Fail to get token space guid for token " + token.cName); + } + + token = new Token(pcdBuildData.getCName(), tokenSpaceStrRet[1]); token.datumType = datumType; token.tokenNumber = tokenNumber; @@ -1675,47 +2113,54 @@ public class CollectPCDAction { // ------------------------------------------------ // usageInstance = new UsageInstance(token, - moduleName, - translateSchemaStringToUUID(modules.get(index).module.getModuleGuid()), - modules.get(index).module.getPackageName(), - translateSchemaStringToUUID(modules.get(index).module.getPackageGuid()), - modules.get(index).type, + modules.get(index).getModuleId().getModule(), pcdType, - modules.get(index).module.getArch().toString(), - null, + modules.get(index).getModuleId().getArch(), datum, maxDatumSize); token.addUsageInstance(usageInstance); } } + + // + // ------------------------------------------------ + // 3), Add unreference dynamic_Ex pcd token into Pcd database. + // ------------------------------------------------ + // + List tokenArray = getUnreferencedDynamicPcd(); + if (tokenArray != null) { + for (index = 0; index < tokenArray.size(); index ++) { + dbManager.addTokenToDatabase(tokenArray.get(index).getPrimaryKeyString(), + tokenArray.get(index)); + } + } } - /** - Get dynamic information for a dynamic PCD from seciton in FPD file. - - This function should be implemented in GlobalData in future. - - @param token The token instance which has hold module's PCD information - @param moduleName The name of module who will use this Dynamic PCD. - - @return DynamicPcdBuildDefinitions.PcdBuildData - */ - /***/ - private DynamicPcdBuildDefinitions.PcdBuildData getDynamicInfoFromFPD(Token token, - String moduleName) - throws EntityException { - int index = 0; - String exceptionString = null; - String dynamicPrimaryKey = null; + private List getUnreferencedDynamicPcd () throws EntityException { + List tokenArray = new ArrayList(); + Token token = null; DynamicPcdBuildDefinitions dynamicPcdBuildDefinitions = null; List dynamicPcdBuildDataArray = null; + DynamicPcdBuildDefinitions.PcdBuildData pcdBuildData = null; + List skuInfoList = null; + Token.PCD_TYPE pcdType; + SkuInstance skuInstance = null; + String primaryKey = null; + boolean hasSkuId0 = false; + int index, offset, index2; + String temp; + String exceptionString; + String hiiDefaultValue; + String tokenSpaceStrRet[]; + String variableGuidString[]; // - // If FPD document is not be opened, open and initialize it. + // Open fpd document to get Section. + // BUGBUG: the function should be move GlobalData in furture. // if (fpdDocInstance == null) { try { - fpdDocInstance = (FrameworkPlatformDescriptionDocument)XmlObject.Factory.parse(new File(fpdFilePath)); + fpdDocInstance = (PlatformSurfaceAreaDocument)XmlObject.Factory.parse(new File(fpdFilePath)); } catch(IOException ioE) { throw new EntityException("File IO error for xml file:" + fpdFilePath + "\n" + ioE.getMessage()); } catch(XmlException xmlE) { @@ -1723,46 +2168,232 @@ public class CollectPCDAction { } } - dynamicPcdBuildDefinitions = fpdDocInstance.getFrameworkPlatformDescription().getDynamicPcdBuildDefinitions(); + dynamicPcdBuildDefinitions = fpdDocInstance.getPlatformSurfaceArea().getDynamicPcdBuildDefinitions(); if (dynamicPcdBuildDefinitions == null) { - exceptionString = String.format("[FPD file error] There are no in FPD file but contains Dynamic type "+ - "PCD entry %s in module %s!", - token.cName, - moduleName); - throw new EntityException(exceptionString); + return null; } dynamicPcdBuildDataArray = dynamicPcdBuildDefinitions.getPcdBuildDataList(); - for (index = 0; index < dynamicPcdBuildDataArray.size(); index ++) { - dynamicPrimaryKey = Token.getPrimaryKeyString(dynamicPcdBuildDataArray.get(index).getCName(), - translateSchemaStringToUUID(dynamicPcdBuildDataArray.get(index).getTokenSpaceGuid())); - if (dynamicPrimaryKey.equalsIgnoreCase(token.getPrimaryKeyString())) { - return dynamicPcdBuildDataArray.get(index); + for (index2 = 0; index2 < dynamicPcdBuildDataArray.size(); index2 ++) { + pcdBuildData = dynamicPcdBuildDataArray.get(index2); + try { + tokenSpaceStrRet = GlobalData.getGuidInfoFromCname(pcdBuildData.getTokenSpaceGuidCName()); + } catch ( Exception e ) { + throw new EntityException ("Faile get Guid for token " + pcdBuildData.getCName() + ":" + e.getMessage()); + } + + if (tokenSpaceStrRet == null) { + throw new EntityException ("Fail to get Token space guid for token" + pcdBuildData.getCName()); + } + + primaryKey = Token.getPrimaryKeyString(pcdBuildData.getCName(), + tokenSpaceStrRet[1]); + + if (dbManager.isTokenInDatabase(primaryKey)) { + continue; + } + + pcdType = Token.getpcdTypeFromString(pcdBuildData.getItemType().toString()); + if (pcdType != Token.PCD_TYPE.DYNAMIC_EX) { + throw new EntityException (String.format("[FPD file error] It not allowed for DYNAMIC PCD %s who is no used by any module", + pcdBuildData.getCName())); + } + + // + // Create new token for unreference dynamic PCD token + // + token = new Token(pcdBuildData.getCName(), tokenSpaceStrRet[1]); + token.datumSize = pcdBuildData.getMaxDatumSize(); + + + token.datumType = Token.getdatumTypeFromString(pcdBuildData.getDatumType().toString()); + token.tokenNumber = Long.decode(pcdBuildData.getToken().toString()); + token.dynamicExTokenNumber = token.tokenNumber; + token.isDynamicPCD = true; + token.updateSupportPcdType(pcdType); + + exceptionString = verifyDatum(token.cName, + null, + null, + token.datumType, + token.datumSize); + if (exceptionString != null) { + throw new EntityException(exceptionString); + } + + skuInfoList = pcdBuildData.getSkuInfoList(); + + // + // Loop all sku data + // + for (index = 0; index < skuInfoList.size(); index ++) { + skuInstance = new SkuInstance(); + // + // Although SkuId in schema is BigInteger, but in fact, sku id is 32 bit value. + // + temp = skuInfoList.get(index).getSkuId().toString(); + skuInstance.id = Integer.decode(temp); + if (skuInstance.id == 0) { + hasSkuId0 = true; + } + // + // Judge whether is DefaultGroup at first, because most case is DefautlGroup. + // + if (skuInfoList.get(index).getValue() != null) { + skuInstance.value.setValue(skuInfoList.get(index).getValue().toString()); + if ((exceptionString = verifyDatum(token.cName, + null, + skuInfoList.get(index).getValue().toString(), + token.datumType, + token.datumSize)) != null) { + throw new EntityException(exceptionString); + } + + token.skuData.add(skuInstance); + + continue; + } + + // + // Judge whether is HII group case. + // + if (skuInfoList.get(index).getVariableName() != null) { + exceptionString = null; + if (skuInfoList.get(index).getVariableGuid() == null) { + exceptionString = String.format("[FPD file error] For dynamic PCD %s in section in FPD "+ + "file, who use HII, but there is no defined for Sku %d data!", + token.cName, + index); + if (exceptionString != null) { + throw new EntityException(exceptionString); + } + } + + if (skuInfoList.get(index).getVariableOffset() == null) { + exceptionString = String.format("[FPD file error] For dynamic PCD %s in section in FPD "+ + "file, who use HII, but there is no defined for Sku %d data!", + token.cName, + index); + if (exceptionString != null) { + throw new EntityException(exceptionString); + } + } + + if (skuInfoList.get(index).getHiiDefaultValue() == null) { + exceptionString = String.format("[FPD file error] For dynamic PCD %s in section in FPD "+ + "file, who use HII, but there is no defined for Sku %d data!", + token.cName, + index); + if (exceptionString != null) { + throw new EntityException(exceptionString); + } + } + + if (skuInfoList.get(index).getHiiDefaultValue() != null) { + hiiDefaultValue = skuInfoList.get(index).getHiiDefaultValue().toString(); + } else { + hiiDefaultValue = null; + } + + if ((exceptionString = verifyDatum(token.cName, + null, + hiiDefaultValue, + token.datumType, + token.datumSize)) != null) { + throw new EntityException(exceptionString); + } + + offset = Integer.decode(skuInfoList.get(index).getVariableOffset()); + if (offset > 0xFFFF) { + throw new EntityException(String.format("[FPD file error] For dynamic PCD %s , the variable offset defined in sku %d data "+ + "exceed 64K, it is not allowed!", + token.cName, + index)); + } + + // + // Get variable guid string according to the name of guid which will be mapped into a GUID in SPD file. + // + variableGuidString = GlobalData.getGuidInfoFromCname(skuInfoList.get(index).getVariableGuid().toString()); + if (variableGuidString == null) { + throw new EntityException(String.format("[GUID Error] For dynamic PCD %s, the variable guid %s can be found in all SPD file!", + token.cName, + skuInfoList.get(index).getVariableGuid().toString())); + } + String variableStr = skuInfoList.get(index).getVariableName(); + Pattern pattern = Pattern.compile("0x([a-fA-F0-9]){4}"); + Matcher matcher = pattern.matcher(variableStr); + List varNameList = new ArrayList(); + while (matcher.find()){ + String str = variableStr.substring(matcher.start(),matcher.end()); + varNameList.add(str); + } + + skuInstance.value.setHiiData(varNameList, + translateSchemaStringToUUID(variableGuidString[1]), + skuInfoList.get(index).getVariableOffset(), + skuInfoList.get(index).getHiiDefaultValue().toString()); + token.skuData.add(skuInstance); + continue; + } + + if (skuInfoList.get(index).getVpdOffset() != null) { + skuInstance.value.setVpdData(skuInfoList.get(index).getVpdOffset()); + token.skuData.add(skuInstance); + continue; + } + + exceptionString = String.format("[FPD file error] For dynamic PCD %s, the dynamic info must "+ + "be one of 'DefaultGroup', 'HIIGroup', 'VpdGroup'.", + token.cName); + throw new EntityException(exceptionString); } + + if (!hasSkuId0) { + exceptionString = String.format("[FPD file error] For dynamic PCD %s in , there are "+ + "no sku id = 0 data, which is required for every dynamic PCD", + token.cName); + throw new EntityException(exceptionString); + } + + tokenArray.add(token); } - return null; + return tokenArray; } /** - Verify the maxDatumSize for a PCD data is matched to Datum type. + Verify the datum value according its datum size and datum type, this + function maybe moved to FPD verification tools in future. - @param token The token instance - @param moduleName The module name who use this PCD data. - @param maxDatumSize The value of max datum size in FPD file - @param datumType The datum type + @param cName + @param moduleName + @param datum + @param datumType + @param maxDatumSize - @return String if is unmatched, set the exception information - as return value, otherwice is null. - **/ - private String verifyDatumSize(String cName, - String moduleName, - int maxDatumSize, - Token.DATUM_TYPE datumType) { - String exceptionString = null; + @return String + */ + /***/ + public String verifyDatum(String cName, + String moduleName, + String datum, + Token.DATUM_TYPE datumType, + int maxDatumSize) { + String exceptionString = null; + int value; + BigInteger value64; + String subStr; + int index; + + if (moduleName == null) { + moduleName = "section "; + } else { + moduleName = "module " + moduleName; + } if (maxDatumSize == 0) { - exceptionString = String.format("[FPD file error] You maybe miss for PCD %s in module %s", + exceptionString = String.format("[FPD file error] You maybe miss for PCD %s in %s", cName, moduleName); return exceptionString; @@ -1771,52 +2402,362 @@ public class CollectPCDAction { switch (datumType) { case UINT8: if (maxDatumSize != 1) { - exceptionString = String.format("[FPD file error] The datum type of PCD data %s in module %s "+ + exceptionString = String.format("[FPD file error] The datum type of PCD data %s in %s "+ "is UINT8, but datum size is %d, they are not matched!", - cName, - moduleName, - maxDatumSize); + cName, + moduleName, + maxDatumSize); + return exceptionString; + } + + if (datum != null) { + try { + value = Integer.decode(datum); + } catch (NumberFormatException nfeExp) { + exceptionString = String.format("[FPD file error] The datum for PCD %s in %s is not valid "+ + "digital format of UINT8", + cName, + moduleName); + return exceptionString; + } + if (value > 0xFF) { + exceptionString = String.format("[FPD file error] The datum for PCD %s in %s is %s exceed"+ + " the max size of UINT8 - 0xFF", + cName, + moduleName, + datum); + return exceptionString; + } } break; case UINT16: if (maxDatumSize != 2) { - exceptionString = String.format("[FPD file error] The datum type of PCD data %s in module %s "+ + exceptionString = String.format("[FPD file error] The datum type of PCD data %s in %s "+ "is UINT16, but datum size is %d, they are not matched!", - cName, - moduleName, - maxDatumSize); + cName, + moduleName, + maxDatumSize); + return exceptionString; + } + if (datum != null) { + try { + value = Integer.decode(datum); + } catch (NumberFormatException nfeExp) { + exceptionString = String.format("[FPD file error] The datum for PCD %s in %s is "+ + "not valid digital of UINT16", + cName, + moduleName); + return exceptionString; + } + if (value > 0xFFFF) { + exceptionString = String.format("[FPD file error] The datum for PCD %s in %s is %s "+ + "which exceed the range of UINT16 - 0xFFFF", + cName, + moduleName, + datum); + return exceptionString; + } } break; case UINT32: if (maxDatumSize != 4) { - exceptionString = String.format("[FPD file error] the datum type of PCD data %s in module %s "+ + exceptionString = String.format("[FPD file error] The datum type of PCD data %s in %s "+ "is UINT32, but datum size is %d, they are not matched!", - cName, - moduleName, - maxDatumSize); + cName, + moduleName, + maxDatumSize); + return exceptionString; + } + + if (datum != null) { + try { + if (datum.length() > 2) { + if ((datum.charAt(0) == '0') && + ((datum.charAt(1) == 'x') || (datum.charAt(1) == 'X'))){ + subStr = datum.substring(2, datum.length()); + value64 = new BigInteger(subStr, 16); + } else { + value64 = new BigInteger(datum); + } + } else { + value64 = new BigInteger(datum); + } + } catch (NumberFormatException nfeExp) { + exceptionString = String.format("[FPD file error] The datum for PCD %s in %s is not "+ + "valid digital of UINT32", + cName, + moduleName); + return exceptionString; + } + + if (value64.bitLength() > 32) { + exceptionString = String.format("[FPD file error] The datum for PCD %s in %s is %s which "+ + "exceed the range of UINT32 - 0xFFFFFFFF", + cName, + moduleName, + datum); + return exceptionString; + } } break; case UINT64: if (maxDatumSize != 8) { - exceptionString = String.format("[FPD file error] the datum type of PCD data %s in module %s "+ + exceptionString = String.format("[FPD file error] The datum type of PCD data %s in %s "+ "is UINT64, but datum size is %d, they are not matched!", - cName, - moduleName, - maxDatumSize); + cName, + moduleName, + maxDatumSize); + return exceptionString; + } + + if (datum != null) { + try { + if (datum.length() > 2) { + if ((datum.charAt(0) == '0') && + ((datum.charAt(1) == 'x') || (datum.charAt(1) == 'X'))){ + subStr = datum.substring(2, datum.length()); + value64 = new BigInteger(subStr, 16); + } else { + value64 = new BigInteger(datum); + } + } else { + value64 = new BigInteger(datum); + } + } catch (NumberFormatException nfeExp) { + exceptionString = String.format("[FPD file error] The datum for PCD %s in %s is not valid"+ + " digital of UINT64", + cName, + moduleName); + return exceptionString; + } + + if (value64.bitLength() > 64) { + exceptionString = String.format("[FPD file error] The datum for PCD %s in %s is %s "+ + "exceed the range of UINT64 - 0xFFFFFFFFFFFFFFFF", + cName, + moduleName, + datum); + return exceptionString; + } } break; case BOOLEAN: if (maxDatumSize != 1) { - exceptionString = String.format("[FPD file error] the datum type of PCD data %s in module %s "+ + exceptionString = String.format("[FPD file error] The datum type of PCD data %s in %s "+ "is BOOLEAN, but datum size is %d, they are not matched!", - cName, - moduleName, - maxDatumSize); + cName, + moduleName, + maxDatumSize); + return exceptionString; + } + + if (datum != null) { + if (!(datum.equalsIgnoreCase("TRUE") || + datum.equalsIgnoreCase("FALSE"))) { + exceptionString = String.format("[FPD file error] The datum type of PCD data %s in %s "+ + "is BOOELAN, but value is not 'true'/'TRUE' or 'FALSE'/'false'", + cName, + moduleName); + return exceptionString; + } + + } + break; + case POINTER: + if (datum == null) { + break; + } + + char ch = datum.charAt(0); + int start, end; + String strValue; + // + // For void* type PCD, only three datum is support: + // 1) Unicode: string with start char is "L" + // 2) Ansci: String start char is "" + // 3) byte array: String start char "{" + // + if (ch == 'L') { + start = datum.indexOf('\"'); + end = datum.lastIndexOf('\"'); + if ((start > end) || + (end > datum.length())|| + ((start == end) && (datum.length() > 0))) { + exceptionString = String.format("[FPD file error] The datum type of PCD %s in %s is VOID* and datum is "+ + "a UNICODE string because start with L\", but format maybe"+ + "is not right, correct UNICODE string is L\"...\"!", + cName, + moduleName); + return exceptionString; + } + + strValue = datum.substring(start + 1, end); + if ((strValue.length() * 2) > maxDatumSize) { + exceptionString = String.format("[FPD file error] The datum type of PCD %s in %s is VOID*, and datum is "+ + "a UNICODE string, but the datum size is %d exceed to : %d", + cName, + moduleName, + strValue.length() * 2, + maxDatumSize); + return exceptionString; + } + } else if (ch == '\"'){ + start = datum.indexOf('\"'); + end = datum.lastIndexOf('\"'); + if ((start > end) || + (end > datum.length())|| + ((start == end) && (datum.length() > 0))) { + exceptionString = String.format("[FPD file error] The datum type of PCD %s in %s is VOID* and datum is "+ + "a ANSCII string because start with \", but format maybe"+ + "is not right, correct ANSIC string is \"...\"!", + cName, + moduleName); + return exceptionString; + } + strValue = datum.substring(start + 1, end); + if ((strValue.length()) > maxDatumSize) { + exceptionString = String.format("[FPD file error] The datum type of PCD %s in %s is VOID*, and datum is "+ + "a ANSCI string, but the datum size is %d which exceed to : %d", + cName, + moduleName, + strValue.length(), + maxDatumSize); + return exceptionString; + } + } else if (ch =='{') { + String[] strValueArray; + + start = datum.indexOf('{'); + end = datum.lastIndexOf('}'); + strValue = datum.substring(start + 1, end); + strValue = strValue.trim(); + if (strValue.length() == 0) { + exceptionString = String.format ("[FPD file error] The datum type of PCD %s in %s is VOID*, and "+ + "it is byte array in fact, but '{}' is not valid for NULL datam but"+ + " need use '{0}'", + cName, + moduleName); + return exceptionString; + } + strValueArray = strValue.split(","); + for (index = 0; index < strValueArray.length; index ++) { + try{ + value = Integer.decode(strValueArray[index].trim()); + } catch (NumberFormatException nfeEx) { + exceptionString = String.format("[FPD file error] The datum type of PCD %s in %s is VOID*, and "+ + "it is byte array in fact. For every byte in array should be a valid"+ + "byte digital, but element %s is not a valid byte digital!", + cName, + moduleName, + strValueArray[index]); + return exceptionString; + } + if (value > 0xFF) { + exceptionString = String.format("[FPD file error] The datum type of PCD %s in %s is VOID*, "+ + "it is byte array in fact. But the element of %s exceed the byte range", + cName, + moduleName, + strValueArray[index]); + return exceptionString; + } + } + + if (strValueArray.length > maxDatumSize) { + exceptionString = String.format("[FPD file error] The datum type of PCD %s in %s is VOID*, and datum is byte"+ + "array, but the number of bytes is %d which exceed to : %d!", + cName, + moduleName, + strValueArray.length, + maxDatumSize); + return exceptionString; + } + } else { + exceptionString = String.format("[FPD file error] The datum type of PCD %s in %s is VOID*. For VOID* type, you have three format choise:\n "+ + "1) UNICODE string: like L\"xxxx\";\r\n"+ + "2) ANSIC string: like \"xxx\";\r\n"+ + "3) Byte array: like {0x2, 0x45, 0x23}\r\n"+ + "But the datum in seems does not following above format!", + cName, + moduleName); + return exceptionString; } break; + default: + exceptionString = String.format("[FPD file error] For PCD entry %s in %s, datum type is unknown, it should be one of "+ + "UINT8, UINT16, UINT32, UINT64, VOID*, BOOLEAN", + cName, + moduleName); + return exceptionString; + } + return null; + } + + /** + Get dynamic information for a dynamic PCD from seciton in FPD file. + + This function should be implemented in GlobalData in future. + + @param token The token instance which has hold module's PCD information + @param moduleName The name of module who will use this Dynamic PCD. + + @return DynamicPcdBuildDefinitions.PcdBuildData + */ + /***/ + private DynamicPcdBuildDefinitions.PcdBuildData getDynamicInfoFromFPD(Token token, + String moduleName) + throws EntityException { + int index = 0; + String exceptionString = null; + String dynamicPrimaryKey = null; + DynamicPcdBuildDefinitions dynamicPcdBuildDefinitions = null; + List dynamicPcdBuildDataArray = null; + String[] tokenSpaceStrRet = null; + + // + // If FPD document is not be opened, open and initialize it. + // BUGBUG: The code should be moved into GlobalData in future. + // + if (fpdDocInstance == null) { + try { + fpdDocInstance = (PlatformSurfaceAreaDocument)XmlObject.Factory.parse(new File(fpdFilePath)); + } catch(IOException ioE) { + throw new EntityException("File IO error for xml file:" + fpdFilePath + "\n" + ioE.getMessage()); + } catch(XmlException xmlE) { + throw new EntityException("Can't parse the FPD xml fle:" + fpdFilePath + "\n" + xmlE.getMessage()); + } + } + + dynamicPcdBuildDefinitions = fpdDocInstance.getPlatformSurfaceArea().getDynamicPcdBuildDefinitions(); + if (dynamicPcdBuildDefinitions == null) { + exceptionString = String.format("[FPD file error] There are no in FPD file but contains Dynamic type "+ + "PCD entry %s in module %s!", + token.cName, + moduleName); + throw new EntityException(exceptionString); + } + + dynamicPcdBuildDataArray = dynamicPcdBuildDefinitions.getPcdBuildDataList(); + for (index = 0; index < dynamicPcdBuildDataArray.size(); index ++) { + //String tokenSpaceGuidString = GlobalData.getGuidInfoFromCname(dynamicPcdBuildDataArray.get(index).getTokenSpaceGuidCName())[1]; + String tokenSpaceGuidString = null; + try { + tokenSpaceStrRet = GlobalData.getGuidInfoFromCname(dynamicPcdBuildDataArray.get(index).getTokenSpaceGuidCName()); + } catch (Exception e) { + throw new EntityException ("Fail to get token space guid for token " + dynamicPcdBuildDataArray.get(index).getCName()); + } + + if (tokenSpaceStrRet == null) { + throw new EntityException ("Fail to get token space guid for token " + dynamicPcdBuildDataArray.get(index).getCName()); + } + + dynamicPrimaryKey = Token.getPrimaryKeyString(dynamicPcdBuildDataArray.get(index).getCName(), + tokenSpaceStrRet[1]); + if (dynamicPrimaryKey.equalsIgnoreCase(token.getPrimaryKeyString())) { + return dynamicPcdBuildDataArray.get(index); + } } - return exceptionString; + return null; } /** @@ -1844,6 +2785,10 @@ public class CollectPCDAction { SkuInstance skuInstance = null; String temp; boolean hasSkuId0 = false; + Token.PCD_TYPE pcdType = Token.PCD_TYPE.UNKNOWN; + long tokenNumber = 0; + String hiiDefaultValue = null; + String[] variableGuidString = null; List skuInfoList = null; DynamicPcdBuildDefinitions.PcdBuildData dynamicInfo = null; @@ -1860,7 +2805,11 @@ public class CollectPCDAction { token.datumSize = dynamicInfo.getMaxDatumSize(); - exceptionString = verifyDatumSize(token.cName, moduleName, token.datumSize, token.datumType); + exceptionString = verifyDatum(token.cName, + moduleName, + null, + token.datumType, + token.datumSize); if (exceptionString != null) { throw new EntityException(exceptionString); } @@ -1875,6 +2824,19 @@ public class CollectPCDAction { dynamicInfo.getMaxDatumSize()); throw new EntityException(exceptionString); } + tokenNumber = Long.decode(dynamicInfo.getToken().toString()); + if (tokenNumber != token.tokenNumber) { + exceptionString = String.format("[FPD file error] For dynamic PCD %s, the token number in module %s is 0x%x, but"+ + "in , the token number is 0x%x, they are not match!", + token.cName, + moduleName, + token.tokenNumber, + tokenNumber); + throw new EntityException(exceptionString); + } + + pcdType = Token.getpcdTypeFromString(dynamicInfo.getItemType().toString()); + token.dynamicExTokenNumber = tokenNumber; skuInfoList = dynamicInfo.getSkuInfoList(); @@ -1895,7 +2857,15 @@ public class CollectPCDAction { // Judge whether is DefaultGroup at first, because most case is DefautlGroup. // if (skuInfoList.get(index).getValue() != null) { - skuInstance.value.setValue(skuInfoList.get(index).getValue()); + skuInstance.value.setValue(skuInfoList.get(index).getValue().toString()); + if ((exceptionString = verifyDatum(token.cName, + null, + skuInfoList.get(index).getValue().toString(), + token.datumType, + token.datumSize)) != null) { + throw new EntityException(exceptionString); + } + token.skuData.add(skuInstance); // @@ -1904,8 +2874,8 @@ public class CollectPCDAction { // if (datum != null) { if ((skuInstance.id == 0) && - !datum.equalsIgnoreCase(skuInfoList.get(index).getValue())) { - exceptionString = "[FPD file error] For dynamic PCD " + token.cName + ", the value in module is " + datum.toString() + " but the "+ + !datum.toString().equalsIgnoreCase(skuInfoList.get(index).getValue().toString())) { + exceptionString = "[FPD file error] For dynamic PCD " + token.cName + ", the value in module " + moduleName + " is " + datum.toString() + " but the "+ "value of sku 0 data in is " + skuInstance.value.value + ". They are must be same!"+ " or you could not define value for a dynamic PCD in every !"; throw new EntityException(exceptionString); @@ -1924,7 +2894,9 @@ public class CollectPCDAction { "file, who use HII, but there is no defined for Sku %d data!", token.cName, index); - + if (exceptionString != null) { + throw new EntityException(exceptionString); + } } if (skuInfoList.get(index).getVariableOffset() == null) { @@ -1932,6 +2904,9 @@ public class CollectPCDAction { "file, who use HII, but there is no defined for Sku %d data!", token.cName, index); + if (exceptionString != null) { + throw new EntityException(exceptionString); + } } if (skuInfoList.get(index).getHiiDefaultValue() == null) { @@ -1939,11 +2914,25 @@ public class CollectPCDAction { "file, who use HII, but there is no defined for Sku %d data!", token.cName, index); + if (exceptionString != null) { + throw new EntityException(exceptionString); + } + } + + if (skuInfoList.get(index).getHiiDefaultValue() != null) { + hiiDefaultValue = skuInfoList.get(index).getHiiDefaultValue().toString(); + } else { + hiiDefaultValue = null; } - if (exceptionString != null) { + if ((exceptionString = verifyDatum(token.cName, + null, + hiiDefaultValue, + token.datumType, + token.datumSize)) != null) { throw new EntityException(exceptionString); } + offset = Integer.decode(skuInfoList.get(index).getVariableOffset()); if (offset > 0xFFFF) { throw new EntityException(String.format("[FPD file error] For dynamic PCD %s , the variable offset defined in sku %d data "+ @@ -1952,10 +2941,28 @@ public class CollectPCDAction { index)); } - skuInstance.value.setHiiData(skuInfoList.get(index).getVariableName(), - translateSchemaStringToUUID(skuInfoList.get(index).getVariableGuid().toString()), + // + // Get variable guid string according to the name of guid which will be mapped into a GUID in SPD file. + // + variableGuidString = GlobalData.getGuidInfoFromCname(skuInfoList.get(index).getVariableGuid().toString()); + if (variableGuidString == null) { + throw new EntityException(String.format("[GUID Error] For dynamic PCD %s, the variable guid %s can be found in all SPD file!", + token.cName, + skuInfoList.get(index).getVariableGuid().toString())); + } + String variableStr = skuInfoList.get(index).getVariableName(); + Pattern pattern = Pattern.compile("0x([a-fA-F0-9]){4}"); + Matcher matcher = pattern.matcher(variableStr); + List varNameList = new ArrayList(); + while (matcher.find()){ + String str = variableStr.substring(matcher.start(),matcher.end()); + varNameList.add(str); + } + + skuInstance.value.setHiiData(varNameList, + translateSchemaStringToUUID(variableGuidString[1]), skuInfoList.get(index).getVariableOffset(), - skuInfoList.get(index).getHiiDefaultValue()); + skuInfoList.get(index).getHiiDefaultValue().toString()); token.skuData.add(skuInstance); continue; } @@ -2019,6 +3026,9 @@ public class CollectPCDAction { return new UUID(0, 0); } + uuidString = uuidString.replaceAll("\\{", ""); + uuidString = uuidString.replaceAll("\\}", ""); + // // If the UUID schema string is GuidArrayType type then need translate // to GuidNamingConvention type at first. @@ -2103,11 +3113,16 @@ public class CollectPCDAction { **/ public static void main(String argv[]) throws EntityException { CollectPCDAction ca = new CollectPCDAction(); - ca.setWorkspacePath("m:/tianocore/edk2"); - ca.setFPDFilePath("m:/tianocore/edk2/EdkNt32Pkg/Nt32.fpd"); + String projectDir = "x:/edk2"; + ca.setWorkspacePath(projectDir); + ca.setFPDFilePath(projectDir + "/EdkNt32Pkg/Nt32.fpd"); ca.setActionMessageLevel(ActionMessage.MAX_MESSAGE_LEVEL); GlobalData.initInfo("Tools" + File.separator + "Conf" + File.separator + "FrameworkDatabase.db", - "m:/tianocore/edk2"); + projectDir, + "tools_def.txt"); + System.out.println("After initInfo!"); + FpdParserTask fpt = new FpdParserTask(); + fpt.parseFpdFile(new File(projectDir + "/EdkNt32Pkg/Nt32.fpd")); ca.execute(); } }