drgreen/android_frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Ease contention on the mProcessStatsThread mutex in updateCpuStats." into froyo

Browse Source
This commit is contained in:
Brad Fitzpatrick
2010-04-19 11:50:53 -07:00
committed by Android (Google) Code Review
parent 1892013643 01fad4a526
commit afbf9f8d6d
1 changed files with 17 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
services/java/com/android/server/am/ActivityManagerService.java
View File

@ -123,6 +123,8 @@ import java.util.List;
import java.util.Locale;
import java.util.Map;
import java.util.Set;
import java.util.concurrent.atomic.AtomicBoolean;
import java.util.concurrent.atomic.AtomicLong;
public final class ActivityManagerService extends ActivityManagerNative implements Watchdog.Monitor {
static final String TAG = "ActivityManager";
@ -927,7 +929,9 @@ public final class ActivityManagerService extends ActivityManagerNative implemen
*/
final ProcessStats mProcessStats = new ProcessStats(
MONITOR_THREAD_CPU_USAGE);
long mLastCpuTime = 0;
final AtomicLong mLastCpuTime = new AtomicLong(0);
final AtomicBoolean mProcessStatsMutexFree = new AtomicBoolean(true);
long mLastWriteTime = 0;
long mInitialStartTime = 0;
@ -1430,7 +1434,7 @@ public final class ActivityManagerService extends ActivityManagerNative implemen
try {
synchronized(this) {
final long now = SystemClock.uptimeMillis();
long nextCpuDelay = (mLastCpuTime+MONITOR_CPU_MAX_TIME)-now;
long nextCpuDelay = (mLastCpuTime.get()+MONITOR_CPU_MAX_TIME)-now;
long nextWriteDelay = (mLastWriteTime+BATTERY_STATS_TIME)-now;
//Slog.i(TAG, "Cpu delay=" + nextCpuDelay
// + ", write delay=" + nextWriteDelay);
@ -1438,12 +1442,12 @@ public final class ActivityManagerService extends ActivityManagerNative implemen
nextCpuDelay = nextWriteDelay;
}
if (nextCpuDelay > 0) {
mProcessStatsMutexFree.set(true);
this.wait(nextCpuDelay);
}
}
} catch (InterruptedException e) {
}
updateCpuStatsNow();
} catch (Exception e) {
Slog.e(TAG, "Unexpected exception collecting process stats", e);
@ -1470,9 +1474,12 @@ public final class ActivityManagerService extends ActivityManagerNative implemen
}
void updateCpuStats() {
synchronized (mProcessStatsThread) {
final long now = SystemClock.uptimeMillis();
if (mLastCpuTime < (now-MONITOR_CPU_MIN_TIME)) {
if (mLastCpuTime.get() >= now - MONITOR_CPU_MIN_TIME) {
return;
}
if (mProcessStatsMutexFree.compareAndSet(true, false)) {
synchronized (mProcessStatsThread) {
mProcessStatsThread.notify();
}
}
@ -1480,12 +1487,13 @@ public final class ActivityManagerService extends ActivityManagerNative implemen
void updateCpuStatsNow() {
synchronized (mProcessStatsThread) {
mProcessStatsMutexFree.set(false);
final long now = SystemClock.uptimeMillis();
boolean haveNewCpuStats = false;
if (MONITOR_CPU_USAGE &&
mLastCpuTime < (now-MONITOR_CPU_MIN_TIME)) {
mLastCpuTime = now;
mLastCpuTime.get() < (now-MONITOR_CPU_MIN_TIME)) {
mLastCpuTime.set(now);
haveNewCpuStats = true;
mProcessStats.update();
//Slog.i(TAG, mProcessStats.printCurrentState());