From 51140ac43c0f5bd26357e6a26ba2f4914c70a2fe Mon Sep 17 00:00:00 2001 From: Yvonne Yip Date: Mon, 17 Jun 2024 16:41:14 -0700 Subject: [PATCH] powerhint: Add nodes to control per-task iowait boost It's disabled by default. Ensure camera and game mode explicitly disable it too as it is only required when auto margins are enabled for now. Ensure it is disabled when screen is off too as We don't want io activities to cause wake up on bigger cores while screen is off. Bug: 340534339 Test: build Flag: NONE powerhint change does not support flags (cherry picked from https://googleplex-android-review.googlesource.com/q/commit:dad48beaeb9f9fce8acb67c06bfd022e86ce0e0d) Merged-In: Ie9d20fe5ad6b423e8c26a36b7f01592040292ead Change-Id: Ie9d20fe5ad6b423e8c26a36b7f01592040292ead --- powerhint-comet.json | 58 ++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 58 insertions(+) diff --git a/powerhint-comet.json b/powerhint-comet.json index cf435b8..48eba13 100644 --- a/powerhint-comet.json +++ b/powerhint-comet.json @@ -259,6 +259,16 @@ "DefaultIndex": 2, "ResetOnInit": true }, + { + "Name": "PerTaskMaxIowaitBoost", + "Path": "/proc/vendor_sched/per_task_iowait_boost_max_value", + "Values": [ + "0", + "512" + ], + "DefaultIndex": 0, + "ResetOnInit": true + }, { "Name": "CPUUClampMaxFilterEnable", "Path": "/proc/vendor_sched/uclamp_max_filter_enable", @@ -1413,6 +1423,12 @@ "Duration": 0, "Value": "0x80" }, + { + "PowerHint": "DISPLAY_INACTIVE", + "Node": "PerTaskMaxIowaitBoost", + "Duration": 0, + "Value": "0" + }, { "PowerHint": "CAMERA_LAUNCH", "Node": "PA_KILL", @@ -2002,6 +2018,12 @@ "Duration": 0, "Value": "0" }, + { + "PowerHint": "CAMERA_STREAMING_EXTREME", + "Node": "PerTaskMaxIowaitBoost", + "Duration": 0, + "Value": "0" + }, { "PowerHint": "CAMERA_STREAMING_EXTREME", "Node": "CPULittleClusterDownRateLimitUs", @@ -2170,6 +2192,12 @@ "Duration": 0, "Value": "0" }, + { + "PowerHint": "CAMERA_STREAMING_HIGH", + "Node": "PerTaskMaxIowaitBoost", + "Duration": 0, + "Value": "0" + }, { "PowerHint": "CAMERA_STREAMING_HIGH", "Node": "CPULittleClusterDownRateLimitUs", @@ -2362,6 +2390,12 @@ "Duration": 0, "Value": "0" }, + { + "PowerHint": "CAMERA_STREAMING_STANDARD", + "Node": "PerTaskMaxIowaitBoost", + "Duration": 0, + "Value": "0" + }, { "PowerHint": "CAMERA_STREAMING_STANDARD", "Node": "CPULittleClusterDownRateLimitUs", @@ -2470,6 +2504,12 @@ "Duration": 0, "Value": "0" }, + { + "PowerHint": "CAMERA_STREAMING_LOW", + "Node": "PerTaskMaxIowaitBoost", + "Duration": 0, + "Value": "0" + }, { "PowerHint": "CAMERA_STREAMING_LOW", "Node": "CPULittleClusterDownRateLimitUs", @@ -2698,6 +2738,12 @@ "Duration": 0, "Value": "0" }, + { + "PowerHint": "CAMERA_STREAMING_VIDEO_CALL", + "Node": "PerTaskMaxIowaitBoost", + "Duration": 0, + "Value": "0" + }, { "PowerHint": "CAMERA_STREAMING_VIDEO_CALL", "Node": "CPUDVFSHeadroom", @@ -3251,6 +3297,12 @@ "Duration": 0, "Value": "2" }, + { + "PowerHint": "GAME", + "Node": "PerTaskMaxIowaitBoost", + "Duration": 0, + "Value": "0" + }, { "PowerHint": "FACE_UNLOCK_BOOST", "Node": "CPUMidClusterMaxFreq", @@ -3359,6 +3411,12 @@ "Duration": 0, "Value": "0" }, + { + "PowerHint": "CAMERA_PROJECT11", + "Node": "PerTaskMaxIowaitBoost", + "Duration": 0, + "Value": "0" + }, { "PowerHint": "CAMERA_PROJECT11", "Node": "CPULittleClusterDownRateLimitUs",