Commit 4949a793 authored Mar 17, 2023 by Jeff Sharkey

Kick off health checks with lock held.

We've seen evidence from Log.wtf() that health checks are racing with
other mutations because the AMS lock isn't held.  To mitigate this,
perform health checks through our Handler where the lock is acquired.

Bug: 267673062
Test: treehugger
Change-Id: I8c7411bc42475789373c29c57cbd384faaf7b9ee

parent 172cf2f2

services/core/java/com/android/server/am/BroadcastQueueModernImpl.java

+1 −1

Original line number	Diff line number	Diff line
		@@ -1339,7 +1339,7 @@ class BroadcastQueueModernImpl extends BroadcastQueue {
		}, ActivityManager.UID_OBSERVER_CACHED, 0, "android");

		// Kick off periodic health checks
		checkHealthLocked();
		mLocalHandler.sendEmptyMessage(MSG_CHECK_HEALTH);
		}

		@Override