From 4051835886e7a2056489309d0545e0fe1d084894 Mon Sep 17 00:00:00 2001 From: Vincent Palomares Date: Tue, 9 May 2023 19:53:52 -0700 Subject: [PATCH 1/3] shusky: powerhint: Update LITTLE CPU freq target Bug: 281138320 Test: build, camera Change-Id: I85983896cccaf9a31903c4ceeb905407e1d63d3b Signed-off-by: Vincent Palomares --- perf/powerhint-husky.json | 4 ++-- perf/powerhint-ripcurrent.json | 4 ++-- perf/powerhint-shiba.json | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/perf/powerhint-husky.json b/perf/powerhint-husky.json index 1075ef9..24b99b0 100644 --- a/perf/powerhint-husky.json +++ b/perf/powerhint-husky.json @@ -84,7 +84,7 @@ "1197000", "1328000", "1548000", - "1844000" + "1704000" ], "DefaultIndex": 0, "ResetOnInit": true @@ -1863,7 +1863,7 @@ "PowerHint": "LOW_POWER_LITTLE_CLUSTER_90", "Node": "CPULittleClusterMaxFreq", "Duration": 3600000, - "Value": "1844000" + "Value": "1704000" }, { "PowerHint": "LOW_POWER_MID_CLUSTER_50", diff --git a/perf/powerhint-ripcurrent.json b/perf/powerhint-ripcurrent.json index ee5209d..e03805f 100644 --- a/perf/powerhint-ripcurrent.json +++ b/perf/powerhint-ripcurrent.json @@ -84,7 +84,7 @@ "1197000", "1328000", "1548000", - "1844000" + "1704000" ], "DefaultIndex": 0, "ResetOnInit": true @@ -1751,7 +1751,7 @@ "PowerHint": "LOW_POWER_LITTLE_CLUSTER_90", "Node": "CPULittleClusterMaxFreq", "Duration": 3600000, - "Value": "1844000" + "Value": "1704000" }, { "PowerHint": "LOW_POWER_MID_CLUSTER_50", diff --git a/perf/powerhint-shiba.json b/perf/powerhint-shiba.json index 9afa6cb..47cd697 100644 --- a/perf/powerhint-shiba.json +++ b/perf/powerhint-shiba.json @@ -84,7 +84,7 @@ "1197000", "1328000", "1548000", - "1844000" + "1704000" ], "DefaultIndex": 0, "ResetOnInit": true @@ -1751,7 +1751,7 @@ "PowerHint": "LOW_POWER_LITTLE_CLUSTER_90", "Node": "CPULittleClusterMaxFreq", "Duration": 3600000, - "Value": "1844000" + "Value": "1704000" }, { "PowerHint": "LOW_POWER_MID_CLUSTER_50", From 9897a96f19387311a1767a5180e1d3656f51ea47 Mon Sep 17 00:00:00 2001 From: Suprabh Shukla Date: Wed, 10 May 2023 23:11:49 +0000 Subject: [PATCH 2/3] Adding interrupt mapping for sound_trigger wakeups Test: Manually inspect the output of `dumpsys batterystats --wakeups` Bug: 271922941 Change-Id: I1e537775db73b56df36515427117df2e21483dc6 --- .../frameworks/base/core/res/res/xml/irq_device_map.xml | 3 +++ .../frameworks/base/core/res/res/xml/irq_device_map.xml | 3 +++ 2 files changed, 6 insertions(+) diff --git a/husky/overlay/frameworks/base/core/res/res/xml/irq_device_map.xml b/husky/overlay/frameworks/base/core/res/res/xml/irq_device_map.xml index 99cd94f..edeba97 100644 --- a/husky/overlay/frameworks/base/core/res/res/xml/irq_device_map.xml +++ b/husky/overlay/frameworks/base/core/res/res/xml/irq_device_map.xml @@ -26,6 +26,9 @@ Cellular_data + + Sound_trigger + Sensor diff --git a/shiba/overlay/frameworks/base/core/res/res/xml/irq_device_map.xml b/shiba/overlay/frameworks/base/core/res/res/xml/irq_device_map.xml index 99cd94f..edeba97 100644 --- a/shiba/overlay/frameworks/base/core/res/res/xml/irq_device_map.xml +++ b/shiba/overlay/frameworks/base/core/res/res/xml/irq_device_map.xml @@ -26,6 +26,9 @@ Cellular_data + + Sound_trigger + Sensor From d429c9d0087281b98b4c5e4a085ccef6a2f25842 Mon Sep 17 00:00:00 2001 From: paulsu Date: Thu, 11 May 2023 07:32:31 +0000 Subject: [PATCH 3/3] The dvfs headroom was split into its own knob. Update to use it. util_threshold only controls migration margin now. dvfs headroom has its own new knob. Test: Local run Bug: 281792287 Change-Id: I828f943aab936b532f658dc6bf3acfc903b7fc02 Signed-off-by: paulsu --- perf/powerhint-husky.json | 22 ++++++++++++++++++++++ perf/powerhint-ripcurrent.json | 22 ++++++++++++++++++++++ perf/powerhint-shiba.json | 22 ++++++++++++++++++++++ 3 files changed, 66 insertions(+) diff --git a/perf/powerhint-husky.json b/perf/powerhint-husky.json index 60e4fb7..145d1df 100644 --- a/perf/powerhint-husky.json +++ b/perf/powerhint-husky.json @@ -187,6 +187,16 @@ "DefaultIndex": 0, "ResetOnInit": true }, + { + "Name": "CPUDVFSHeadroom", + "Path": "/proc/vendor_sched/dvfs_headroom", + "Values": [ + "1280", + "1100" + ], + "DefaultIndex": 0, + "ResetOnInit": true + }, { "Name": "CAMUClampBoost", "Path": "/proc/vendor_sched/cam_uclamp_min", @@ -1355,6 +1365,12 @@ "Duration": 0, "Value": "1100" }, + { + "PowerHint": "CAMERA_STREAMING_HIGH", + "Node": "CPUDVFSHeadroom", + "Duration": 0, + "Value": "1100" + }, { "PowerHint": "CAMERA_STREAMING_HIGH", "Node": "CDPreferHighCap", @@ -1775,6 +1791,12 @@ "Duration": 0, "Value": "1100" }, + { + "PowerHint": "CAMERA_STREAMING_VIDEO_CALL", + "Node": "CPUDVFSHeadroom", + "Duration": 0, + "Value": "1100" + }, { "PowerHint": "CAMERA_STREAMING_VIDEO_CALL", "Node": "NPITaskPacking", diff --git a/perf/powerhint-ripcurrent.json b/perf/powerhint-ripcurrent.json index f1e3bb9..2797170 100644 --- a/perf/powerhint-ripcurrent.json +++ b/perf/powerhint-ripcurrent.json @@ -177,6 +177,16 @@ "DefaultIndex": 0, "ResetOnInit": true }, + { + "Name": "CPUDVFSHeadroom", + "Path": "/proc/vendor_sched/dvfs_headroom", + "Values": [ + "1280", + "1100" + ], + "DefaultIndex": 0, + "ResetOnInit": true + }, { "Name": "CAMUClampBoost", "Path": "/proc/vendor_sched/cam_uclamp_min", @@ -1243,6 +1253,12 @@ "Duration": 0, "Value": "1100" }, + { + "PowerHint": "CAMERA_STREAMING_HIGH", + "Node": "CPUDVFSHeadroom", + "Duration": 0, + "Value": "1100" + }, { "PowerHint": "CAMERA_STREAMING_HIGH", "Node": "CDPreferHighCap", @@ -1663,6 +1679,12 @@ "Duration": 0, "Value": "1100" }, + { + "PowerHint": "CAMERA_STREAMING_VIDEO_CALL", + "Node": "CPUDVFSHeadroom", + "Duration": 0, + "Value": "1100" + }, { "PowerHint": "CAMERA_STREAMING_VIDEO_CALL", "Node": "NPITaskPacking", diff --git a/perf/powerhint-shiba.json b/perf/powerhint-shiba.json index 41391c7..44b0ec5 100644 --- a/perf/powerhint-shiba.json +++ b/perf/powerhint-shiba.json @@ -177,6 +177,16 @@ "DefaultIndex": 0, "ResetOnInit": true }, + { + "Name": "CPUDVFSHeadroom", + "Path": "/proc/vendor_sched/dvfs_headroom", + "Values": [ + "1280", + "1100" + ], + "DefaultIndex": 0, + "ResetOnInit": true + }, { "Name": "CAMUClampBoost", "Path": "/proc/vendor_sched/cam_uclamp_min", @@ -1243,6 +1253,12 @@ "Duration": 0, "Value": "1100" }, + { + "PowerHint": "CAMERA_STREAMING_HIGH", + "Node": "CPUDVFSHeadroom", + "Duration": 0, + "Value": "1100" + }, { "PowerHint": "CAMERA_STREAMING_HIGH", "Node": "CDPreferHighCap", @@ -1663,6 +1679,12 @@ "Duration": 0, "Value": "1100" }, + { + "PowerHint": "CAMERA_STREAMING_VIDEO_CALL", + "Node": "CPUDVFSHeadroom", + "Duration": 0, + "Value": "1100" + }, { "PowerHint": "CAMERA_STREAMING_VIDEO_CALL", "Node": "NPITaskPacking",