Dump systemUI and notification service in bugreport critical section
- modified publishBinderService to accept dump prioirty arguments
- registed NotificationManagerService with critical and normal dump support and
split dump into critical and normal sections
- ActivityManagerService critical section duration (manual runs): 0.024s, 0.050s, 0.038s
- NotificationManagerService critical section duration (manual runs): 0.007s,0.017s
Bug: 73958222
Test: Take bug report and verify contents
Test: mmm -j56 frameworks/native/cmds/dumpstate && \
adb sync data && adb shell /data/nativetest64/dumpstate_smoke_test/dumpstate_smoke_test && \
atest FrameworksServicesTests:PriorityDumpTest && \
printf "\n\n#### ALL TESTS PASSED ####\n"
Change-Id: I23cc9c056a550505393222f521cba5841782132e
diff --git a/services/core/java/com/android/server/SystemService.java b/services/core/java/com/android/server/SystemService.java
index 58731d2..8e9e74e 100644
--- a/services/core/java/com/android/server/SystemService.java
+++ b/services/core/java/com/android/server/SystemService.java
@@ -16,6 +16,8 @@
package com.android.server;
+import static android.os.IServiceManager.DUMP_FLAG_PRIORITY_DEFAULT;
+
import android.app.ActivityThread;
import android.content.Context;
import android.os.IBinder;
@@ -199,6 +201,9 @@
/**
* Publish the service so it is accessible to other services and apps.
+ *
+ * @param name the name of the new service
+ * @param service the service object
*/
protected final void publishBinderService(String name, IBinder service) {
publishBinderService(name, service, false);
@@ -206,10 +211,29 @@
/**
* Publish the service so it is accessible to other services and apps.
+ *
+ * @param name the name of the new service
+ * @param service the service object
+ * @param allowIsolated set to true to allow isolated sandboxed processes
+ * to access this service
*/
protected final void publishBinderService(String name, IBinder service,
boolean allowIsolated) {
- ServiceManager.addService(name, service, allowIsolated);
+ publishBinderService(name, service, allowIsolated, DUMP_FLAG_PRIORITY_DEFAULT);
+ }
+
+ /**
+ * Publish the service so it is accessible to other services and apps.
+ *
+ * @param name the name of the new service
+ * @param service the service object
+ * @param allowIsolated set to true to allow isolated sandboxed processes
+ * to access this service
+ * @param dumpPriority supported dump priority levels as a bitmask
+ */
+ protected final void publishBinderService(String name, IBinder service,
+ boolean allowIsolated, int dumpPriority) {
+ ServiceManager.addService(name, service, allowIsolated, dumpPriority);
}
/**