From 3d2f62fda87584c34f40d65857ba862405481094 Mon Sep 17 00:00:00 2001 From: Orion Hodson Date: Tue, 16 Nov 2021 09:07:14 +0000 Subject: [PATCH] AndroidRuntime: when CheckJNI is enabled log a message This matches behaviour described in: https://developer.android.com/training/articles/perf-jni#extended-checking All output from CheckJNI state was disabled as part of quietening logspam, but now we log a message only if CheckJNI is enabled. CheckJNI is off by default and only used as a developer correctness tool. Bug: N/A Test: adb root; adb shell stop; \ adb shell setprop dalvik.vm.checkjni 1; \ adb shell start; adb logcat -d | grep CheckJNI Change-Id: I93bc2d097a8769b7768c633b705b29fc1aac634a --- core/jni/AndroidRuntime.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/core/jni/AndroidRuntime.cpp b/core/jni/AndroidRuntime.cpp index 97cac291630cd..a13111143b0e2 100644 --- a/core/jni/AndroidRuntime.cpp +++ b/core/jni/AndroidRuntime.cpp @@ -746,8 +746,9 @@ int AndroidRuntime::startVm(JavaVM** pJavaVM, JNIEnv** pEnv, bool zygote, bool p } const bool checkJni = GetBoolProperty("dalvik.vm.checkjni", false); - ALOGV("CheckJNI is %s\n", checkJni ? "ON" : "OFF"); if (checkJni) { + ALOGD("CheckJNI is ON"); + /* extended JNI checking */ addOption("-Xcheck:jni");