Merge "Frameworks/base: Add debug.gencfi property"

This commit is contained in:
Andreas Gampe
2015-04-27 15:06:46 +00:00
committed by Gerrit Code Review
5 changed files with 25 additions and 1 deletions

View File

@ -634,6 +634,9 @@ public class Process {
if ((debugFlags & Zygote.DEBUG_ENABLE_JIT) != 0) {
argsForZygote.add("--enable-jit");
}
if ((debugFlags & Zygote.DEBUG_GENERATE_CFI) != 0) {
argsForZygote.add("--generate-cfi");
}
if ((debugFlags & Zygote.DEBUG_ENABLE_ASSERT) != 0) {
argsForZygote.add("--enable-assert");
}

View File

@ -40,6 +40,8 @@ public final class Zygote {
public static final int DEBUG_ENABLE_JNI_LOGGING = 1 << 4;
/** enable the JIT compiler */
public static final int DEBUG_ENABLE_JIT = 1 << 5;
/** Force generation of CFI code */
public static final int DEBUG_GENERATE_CFI = 1 << 6;
/** No external storage should be mounted. */

View File

@ -322,7 +322,7 @@ class ZygoteConnection {
/**
* From --enable-debugger, --enable-checkjni, --enable-assert,
* --enable-safemode, --enable-jit, and --enable-jni-logging.
* --enable-safemode, --enable-jit, --generate-cfi and --enable-jni-logging.
*/
int debugFlags;
@ -434,6 +434,8 @@ class ZygoteConnection {
debugFlags |= Zygote.DEBUG_ENABLE_CHECKJNI;
} else if (arg.equals("--enable-jit")) {
debugFlags |= Zygote.DEBUG_ENABLE_JIT;
} else if (arg.equals("--generate-cfi")) {
debugFlags |= Zygote.DEBUG_GENERATE_CFI;
} else if (arg.equals("--enable-jni-logging")) {
debugFlags |= Zygote.DEBUG_ENABLE_JNI_LOGGING;
} else if (arg.equals("--enable-assert")) {

View File

@ -873,6 +873,19 @@ int AndroidRuntime::startVm(JavaVM** pJavaVM, JNIEnv** pEnv)
snprintf(cpuAbiListBuf, sizeof(cpuAbiListBuf), "--cpu-abilist=%s", propBuf);
addOption(cpuAbiListBuf);
/*
* When running with debug.gencfi, add --include-cfi to the compiler options so that the boot
* image, if it is compiled on device, will include CFI info, as well as other compilations
* started by the runtime.
*/
property_get("debug.gencfi", propBuf, "");
if (strcmp(propBuf, "true") == 0) {
addOption("-Xcompiler-option");
addOption("--include-cfi");
addOption("-Ximage-compiler-option");
addOption("--include-cfi");
}
initArgs.version = JNI_VERSION_1_4;
initArgs.options = mOptions.editArray();
initArgs.nOptions = mOptions.size();

View File

@ -3016,6 +3016,10 @@ public final class ActivityManagerService extends ActivityManagerNative
debugFlags |= Zygote.DEBUG_ENABLE_JIT;
}
}
String genCFIDebugProperty = SystemProperties.get("debug.gencfi");
if ("true".equals(genCFIDebugProperty)) {
debugFlags |= Zygote.DEBUG_GENERATE_CFI;
}
if ("1".equals(SystemProperties.get("debug.jni.logging"))) {
debugFlags |= Zygote.DEBUG_ENABLE_JNI_LOGGING;
}