mirror of https://github.com/acidanthera/audk.git
Support "Defining Module Sequence in FV.inf". Since modules sequences in Fv.inf can be customized explicitly in FPD.BuildOptions.UserExtensions[UserID='IMAGES' Identifier='1'] with WIZARD tool, now build tool will check it and use it if exists.
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@1605 6f19259b-4bc3-4df7-8a09-765794883524
This commit is contained in:
parent
b63cc1b617
commit
bb511931b1
|
@ -205,7 +205,7 @@ public class FpdParserForThread extends FpdParserTask {
|
|||
//
|
||||
// Exist ready thread
|
||||
//
|
||||
// EdkLog.log(this, EdkLog.EDK_ALWAYS, "Exist ready thread");
|
||||
EdkLog.log(this, EdkLog.EDK_DEBUG, "Exist ready thread");
|
||||
|
||||
} else if (existNoneReady && currentRunNumber == 0) {
|
||||
//
|
||||
|
@ -216,21 +216,21 @@ public class FpdParserForThread extends FpdParserTask {
|
|||
//
|
||||
// Current queue build finish, move to next
|
||||
//
|
||||
EdkLog.log(this, EdkLog.EDK_ALWAYS, "Current queue build finish, move to next");
|
||||
EdkLog.log(this, EdkLog.EDK_DEBUG, "Current queue build finish, move to next");
|
||||
++currentQueueCode;
|
||||
continue ;
|
||||
} else {
|
||||
//
|
||||
// active thread exist, but no ready thread
|
||||
//
|
||||
EdkLog.log(this, EdkLog.EDK_ALWAYS, "Active thread exist, but no ready thread. Current running number is " + currentRunNumber);
|
||||
EdkLog.log(this, EdkLog.EDK_DEBUG, "Active thread exist, but no ready thread. Current running number is " + currentRunNumber);
|
||||
}
|
||||
|
||||
try {
|
||||
deamonSemaphore.wait();
|
||||
|
||||
//
|
||||
// if find error. Let other threads to finish
|
||||
// if find error. Waiting running threads to finish
|
||||
//
|
||||
if (errorModule != null) {
|
||||
while (currentRunNumber > 0) {
|
||||
|
|
|
@ -36,6 +36,7 @@ import org.apache.xmlbeans.XmlObject;
|
|||
|
||||
import org.tianocore.common.definitions.EdkDefinitions;
|
||||
import org.tianocore.common.exception.EdkException;
|
||||
import org.tianocore.common.logger.EdkLog;
|
||||
import org.tianocore.pcd.action.ActionMessage;
|
||||
import org.tianocore.build.FrameworkBuildTask;
|
||||
import org.tianocore.build.global.GlobalData;
|
||||
|
@ -43,11 +44,15 @@ import org.tianocore.build.global.OutputManager;
|
|||
import org.tianocore.build.global.SurfaceAreaQuery;
|
||||
import org.tianocore.build.id.FpdModuleIdentification;
|
||||
import org.tianocore.build.id.ModuleIdentification;
|
||||
import org.tianocore.build.id.PackageIdentification;
|
||||
import org.tianocore.build.id.PlatformIdentification;
|
||||
import org.tianocore.build.pcd.action.PlatformPcdPreprocessActionForBuilding;
|
||||
import org.tianocore.build.toolchain.ToolChainAttribute;
|
||||
import org.tianocore.build.toolchain.ToolChainElement;
|
||||
import org.tianocore.build.toolchain.ToolChainMap;
|
||||
import org.w3c.dom.NamedNodeMap;
|
||||
import org.w3c.dom.Node;
|
||||
import org.w3c.dom.NodeList;
|
||||
|
||||
/**
|
||||
<code>FpdParserTask</code> is an ANT task. The main function is parsing Framework
|
||||
|
@ -117,6 +122,8 @@ public class FpdParserTask extends Task {
|
|||
Surface area is not valid.
|
||||
**/
|
||||
public void execute() throws BuildException {
|
||||
this.setTaskName("FpdParser");
|
||||
|
||||
//
|
||||
// Parse FPD file
|
||||
//
|
||||
|
@ -267,13 +274,45 @@ public class FpdParserTask extends Task {
|
|||
}
|
||||
bw.newLine();
|
||||
}
|
||||
|
||||
|
||||
//
|
||||
// Files
|
||||
//
|
||||
Set<FpdModuleIdentification> moduleSeqSet = getModuleSequenceForFv(validFv[i]);
|
||||
|
||||
Set<FpdModuleIdentification> filesSet = fvs.get(validFv[i]);
|
||||
if (filesSet != null) {
|
||||
FpdModuleIdentification[] files = filesSet.toArray(new FpdModuleIdentification[filesSet.size()]);
|
||||
|
||||
FpdModuleIdentification[] files = null;
|
||||
if (moduleSeqSet == null) {
|
||||
if (filesSet != null) {
|
||||
files = filesSet.toArray(new FpdModuleIdentification[filesSet.size()]);
|
||||
}
|
||||
} else {
|
||||
//
|
||||
// if moduleSeqSet and filesSet is inconsistent, report error
|
||||
//
|
||||
if (filesSet == null && moduleSeqSet.size() != 0) {
|
||||
throw new BuildException("Can not find any modules belongs to FV[" + validFv[i] + "], but listed some in BuildOptions.UserExtensions[@UserID='IMAGES' @Identifier='1']");
|
||||
} else if(moduleSeqSet.size() != filesSet.size()){
|
||||
throw new BuildException("Modules for FV[" + validFv[i] + "] defined in FrameworkModules and in BuildOptions.UserExtensions[@UserID='IMAGES' @Identifier='1'] are inconsistent. ");
|
||||
} else {
|
||||
//
|
||||
// whether all modules in moduleSeqSet listed in filesSet
|
||||
//
|
||||
Iterator<FpdModuleIdentification> iter = moduleSeqSet.iterator();
|
||||
while (iter.hasNext()) {
|
||||
FpdModuleIdentification item = iter.next();
|
||||
if (!filesSet.contains(item)) {
|
||||
throw new BuildException("Can not find " + item + " belongs to FV[" + validFv[i] + "]");
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
files = moduleSeqSet.toArray(new FpdModuleIdentification[moduleSeqSet.size()]);
|
||||
}
|
||||
|
||||
|
||||
if (files != null) {
|
||||
bw.write("[files]");
|
||||
bw.newLine();
|
||||
for (int j = 0; j < files.length; j++) {
|
||||
|
@ -289,6 +328,10 @@ public class FpdParserTask extends Task {
|
|||
BuildException buildException = new BuildException("Generation of the FV file [" + fvFile.getPath() + "] failed!\n" + ex.getMessage());
|
||||
buildException.setStackTrace(ex.getStackTrace());
|
||||
throw buildException;
|
||||
} catch (EdkException ex) {
|
||||
BuildException buildException = new BuildException("Generation of the FV file [" + fvFile.getPath() + "] failed!\n" + ex.getMessage());
|
||||
buildException.setStackTrace(ex.getStackTrace());
|
||||
throw buildException;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
@ -613,4 +656,99 @@ public class FpdParserTask extends Task {
|
|||
|
||||
return archs;
|
||||
}
|
||||
|
||||
private Set<FpdModuleIdentification> getModuleSequenceForFv(String fvName) throws EdkException {
|
||||
Node node = saq.getFpdModuleSequence(fvName);
|
||||
Set<FpdModuleIdentification> result = new LinkedHashSet<FpdModuleIdentification>();
|
||||
|
||||
if ( node == null) {
|
||||
EdkLog.log(this, EdkLog.EDK_WARNING, "FV[" + fvName + "] does not specify module sequence in FPD. Assuming present sequence as default sequence in FV. ");
|
||||
return null;
|
||||
} else {
|
||||
NodeList childNodes = node.getChildNodes();
|
||||
for (int i = 0; i < childNodes.getLength(); i++) {
|
||||
Node childItem = childNodes.item(i);
|
||||
if (childItem.getNodeType() == Node.ELEMENT_NODE) {
|
||||
//
|
||||
// Find child elements "IncludeModules"
|
||||
//
|
||||
if (childItem.getNodeName().compareTo("IncludeModules") == 0) {
|
||||
//
|
||||
// result will be updated
|
||||
//
|
||||
processNodes(childItem, result);
|
||||
} else if (childItem.getNodeName().compareTo("FvName") == 0) {
|
||||
|
||||
} else if (childItem.getNodeName().compareTo("InfFileName") == 0) {
|
||||
|
||||
} else {
|
||||
//
|
||||
// Report Warning
|
||||
//
|
||||
EdkLog.log(this, EdkLog.EDK_WARNING, "Unrecognised element " + childItem.getNodeName() + " under FPD.BuildOptions.UserExtensions[UserID='IMAGES' Identifier='1']");
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
return result;
|
||||
}
|
||||
|
||||
private void processNodes(Node node, Set<FpdModuleIdentification> result) throws EdkException {
|
||||
//
|
||||
// Found out all elements "Module"
|
||||
//
|
||||
NodeList childNodes = node.getChildNodes();
|
||||
for (int j = 0; j < childNodes.getLength(); j++) {
|
||||
Node childItem = childNodes.item(j);
|
||||
if (childItem.getNodeType() == Node.ELEMENT_NODE) {
|
||||
if (childItem.getNodeName().compareTo("Module") == 0) {
|
||||
String moduleGuid = null;
|
||||
String moduleVersion = null;
|
||||
String packageGuid = null;
|
||||
String packageVersion = null;
|
||||
String arch = null;
|
||||
|
||||
NamedNodeMap attr = childItem.getAttributes();
|
||||
for (int i = 0; i < attr.getLength(); i++) {
|
||||
Node attrItem = attr.item(i);
|
||||
if (attrItem.getNodeName().compareTo("ModuleGuid") == 0) {
|
||||
moduleGuid = attrItem.getNodeValue();
|
||||
} else if (attrItem.getNodeName().compareTo("ModuleVersion") == 0) {
|
||||
moduleVersion = attrItem.getNodeValue();
|
||||
} else if (attrItem.getNodeName().compareTo("PackageGuid") == 0) {
|
||||
packageGuid = attrItem.getNodeValue();
|
||||
} else if (attrItem.getNodeName().compareTo("PackageVersion") == 0) {
|
||||
packageVersion = attrItem.getNodeValue();
|
||||
} else if (attrItem.getNodeName().compareTo("Arch") == 0) {
|
||||
arch = attrItem.getNodeValue();
|
||||
} else {
|
||||
//
|
||||
// Report warning
|
||||
//
|
||||
EdkLog.log(this, EdkLog.EDK_WARNING, "Unrecognised attribute " + attrItem.getNodeName() + " under FPD.BuildOptions.UserExtensions[UserID='IMAGES' Identifier='1'].IncludeModules.Module");
|
||||
}
|
||||
}
|
||||
|
||||
PackageIdentification packageId = new PackageIdentification(packageGuid, packageVersion);
|
||||
GlobalData.refreshPackageIdentification(packageId);
|
||||
|
||||
ModuleIdentification moduleId = new ModuleIdentification(moduleGuid, moduleVersion);
|
||||
moduleId.setPackage(packageId);
|
||||
GlobalData.refreshModuleIdentification(moduleId);
|
||||
|
||||
if (arch == null) {
|
||||
throw new EdkException("Attribute [Arch] is required for element FPD.BuildOptions.UserExtensions[UserID='IMAGES' Identifier='1'].IncludeModules.Module. ");
|
||||
}
|
||||
|
||||
result.add(new FpdModuleIdentification(moduleId, arch));
|
||||
} else {
|
||||
//
|
||||
// Report Warning
|
||||
//
|
||||
EdkLog.log(this, EdkLog.EDK_WARNING, "Unrecognised element " + childItem.getNodeName() + " under FPD.BuildOptions.UserExtensions[UserID='IMAGES' Identifier='1'].IncludeModules");
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
|
@ -14,9 +14,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
|
|||
package org.tianocore.build.global;
|
||||
|
||||
import java.io.File;
|
||||
import java.util.HashMap;
|
||||
import java.util.Iterator;
|
||||
import java.util.Map;
|
||||
|
||||
import org.apache.tools.ant.BuildException;
|
||||
import org.apache.tools.ant.Task;
|
||||
|
|
|
@ -27,9 +27,7 @@ import org.tianocore.ExternsDocument.Externs.Extern;
|
|||
import org.apache.xmlbeans.XmlObject;
|
||||
import org.apache.xmlbeans.XmlString;
|
||||
import org.tianocore.*;
|
||||
import org.tianocore.ExternsDocument.Externs;
|
||||
import org.tianocore.FilenameDocument.Filename;
|
||||
import org.tianocore.ModuleSurfaceAreaDocument.ModuleSurfaceArea;
|
||||
import org.tianocore.MsaHeaderDocument.MsaHeader;
|
||||
import org.tianocore.ProtocolsDocument.Protocols.Protocol;
|
||||
import org.tianocore.ProtocolsDocument.Protocols.ProtocolNotify;
|
||||
|
@ -1993,4 +1991,17 @@ public class SurfaceAreaQuery {
|
|||
return false;
|
||||
}
|
||||
}
|
||||
|
||||
public Node getFpdModuleSequence(String fvName) {
|
||||
String[] xPath = new String[] { "/BuildOptions/UserExtensions[@UserID='IMAGES' and @Identifier='1' and ./FvName='" + fvName + "']" };
|
||||
Object[] result = get("PlatformSurfaceArea", xPath);
|
||||
|
||||
if (result == null || result.length == 0) {
|
||||
return null;
|
||||
}
|
||||
|
||||
UserExtensionsDocument.UserExtensions a = (UserExtensionsDocument.UserExtensions)result[0];
|
||||
|
||||
return a.getDomNode();
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue