From c9e0f795936b2000f39a0731afd54b088acf75d5 Mon Sep 17 00:00:00 2001 From: Victor Hsieh Date: Thu, 12 May 2022 11:52:14 -0700 Subject: [PATCH] Make VMCompilationPerformance an aggregated profile with default The VMCompilationPerformance profile is used to run Isolated Compilation in a Protected VM, normally during the device idle, where relevant APEXes are staged. The original VMCompilationPerformance introduced in aosp/2060891 does not have any specific definition and requires vendors to customize it. This change re-defines it as an aggregated profile with a default set of existing profiles, so that performance can be reasonable by default. This profile may be renamed to a more generic name later, e.g. "SCHED_SP_COMPUTE". Bug: 231437770 Test: Run `composd_cmd test-compile` on a local device. Before: 1m50s +/- 10s (with whatever that's default) After: 1m25s +/- 5s Change-Id: Ib8cd65782c818474fb129efbd9ef9a3e23ad1eb3 --- libprocessgroup/profiles/task_profiles.json | 17 ++++------------- 1 file changed, 4 insertions(+), 13 deletions(-) diff --git a/libprocessgroup/profiles/task_profiles.json b/libprocessgroup/profiles/task_profiles.json index f5533c262..4092c1a37 100644 --- a/libprocessgroup/profiles/task_profiles.json +++ b/libprocessgroup/profiles/task_profiles.json @@ -223,19 +223,6 @@ } ] }, - { - "Name": "VMCompilationPerformance", - "Actions": [ - { - "Name": "JoinCgroup", - "Params": - { - "Controller": "cpu", - "Path": "system" - } - } - ] - }, { "Name": "CpuPolicySpread", "Actions": [ @@ -659,6 +646,10 @@ "Name": "SCHED_SP_SYSTEM", "Profiles": [ "ServicePerformance", "LowIoPriority", "TimerSlackNormal" ] }, + { + "Name": "VMCompilationPerformance", + "Profiles": [ "HighPerformance", "ProcessCapacityHigh", "LowIoPriority", "TimerSlackNormal" ] + }, { "Name": "SCHED_SP_RT_APP", "Profiles": [ "RealtimePerformance", "MaxIoPriority", "TimerSlackNormal" ]