From 0f4f93aec642b4c56212c0ee29714965458b85f8 Mon Sep 17 00:00:00 2001 From: "Chung-Kai (Michael) Mei" Date: Tue, 24 Dec 2024 00:36:08 -0800 Subject: [PATCH 1/3] Revert "backport launch powerhint to felix" This reverts commit c0cc1cd8f151dbb1a5df6ab1c4b1e076354b5536. Reason for revert: camera has regression Change-Id: Ica39fe7d25325510689d1835cffdb34a558891af --- powerhint.json | 257 +++---------------------------------------------- 1 file changed, 15 insertions(+), 242 deletions(-) diff --git a/powerhint.json b/powerhint.json index af2ef5f..9c9e8a3 100644 --- a/powerhint.json +++ b/powerhint.json @@ -62,18 +62,6 @@ ], "ResetOnInit": true }, - { - "Name": "CPULittleClusterDownRateLimitUs", - "Path": "/sys/devices/system/cpu/cpu0/cpufreq/sched_pixel/down_rate_limit_us", - "Values": [ - "5000", - "2000", - "1000", - "500" - ], - "DefaultIndex": 0, - "ResetOnInit": true - }, { "Name": "CPUMidClusterMaxFreq", "Path": "/sys/devices/system/cpu/cpu4/cpufreq/scaling_max_freq", @@ -106,19 +94,6 @@ ], "ResetOnInit": true }, - { - "Name": "CPUMidClusterDownRateLimitUs", - "Path": "/sys/devices/system/cpu/cpu4/cpufreq/sched_pixel/down_rate_limit_us", - "Values": [ - "20000", - "5000", - "3000", - "2000", - "500" - ], - "DefaultIndex": 0, - "ResetOnInit": true - }, { "Name": "CPUBigClusterMaxFreq", "Path": "/sys/devices/system/cpu/cpu6/cpufreq/scaling_max_freq", @@ -153,18 +128,6 @@ ], "ResetOnInit": true }, - { - "Name": "CPUBigClusterDownRateLimitUs", - "Path": "/sys/devices/system/cpu/cpu6/cpufreq/sched_pixel/down_rate_limit_us", - "Values": [ - "20000", - "3000", - "2000", - "500" - ], - "DefaultIndex": 0, - "ResetOnInit": true - }, { "Name": "GPUDvfsPeriod", "Path": "/sys/devices/platform/28000000.mali/dvfs_period", @@ -183,15 +146,6 @@ ], "ResetOnInit": true }, - { - "Name": "AutoPreferIdle", - "Path": "/proc/vendor_sched/auto_prefer_idle", - "Values": [ - "0", - "1" - ], - "ResetOnInit": true - }, { "Name": "CPUUtilThreshold", "Path": "/proc/vendor_sched/util_threshold", @@ -212,16 +166,6 @@ "DefaultIndex": 0, "ResetOnInit": true }, - { - "Name": "CPUTaperedDVFSHeadroomEnable", - "Path": "/proc/vendor_sched/tapered_dvfs_headroom_enable", - "Values": [ - "0", - "1" - ], - "DefaultIndex": 0, - "ResetOnInit": true - }, { "Name": "MIFTargetLoad", "Path": "/sys/class/devfreq/17000010.devfreq_mif/interactive/target_load", @@ -250,54 +194,6 @@ ], "ResetOnInit": true }, - { - "Name": "TAUclampMinOnNiceEnable", - "Path": "/proc/vendor_sched/groups/ta/uclamp_min_on_nice_enable", - "Values": [ - "1", - "0" - ], - "ResetOnInit": true - }, - { - "Name": "TAUclampMinOnNiceHighValue", - "Path": "/proc/vendor_sched/groups/ta/uclamp_min_on_nice_high_value", - "Values": [ - "765", - "0" - ], - "ResetOnInit": true - }, - { - "Name": "TAUclampMinOnNiceMidValue", - "Path": "/proc/vendor_sched/groups/ta/uclamp_min_on_nice_mid_value", - "Values": [ - "600", - "590", - "0" - ], - "ResetOnInit": true - }, - { - "Name": "TAUclampMinOnNiceHighPrio", - "Path": "/proc/vendor_sched/groups/ta/uclamp_min_on_nice_high_prio", - "Values": [ - "110", - "119", - "120" - ], - "ResetOnInit": true - }, - { - "Name": "TAUclampMinOnNiceMidPrio", - "Path": "/proc/vendor_sched/groups/ta/uclamp_min_on_nice_mid_prio", - "Values": [ - "128", - "139", - "130" - ], - "ResetOnInit": true - }, { "Name": "FGUClampBoost", "Path": "/proc/vendor_sched/groups/fg/uclamp_min", @@ -390,25 +286,6 @@ ], "ResetOnInit": true }, - { - "Name": "TAAutoPreferFit", - "Path": "/proc/vendor_sched/groups/ta/auto_prefer_fit", - "Values": [ - "1", - "0" - ], - "ResetOnInit": true - }, - { - "Name": "TAPreferredIdleMaskHigh", - "Path": "/proc/vendor_sched/groups/ta/preferred_idle_mask_high", - "Values": [ - "0xff", - "0xf0", - "0x7f" - ], - "ResetOnInit": true - }, { "Name": "FGPreferIdle", "Path": "/proc/vendor_sched/groups/fg/prefer_idle", @@ -803,26 +680,6 @@ ], "DefaultIndex": 0, "ResetOnInit": true - }, - { - "Name": "CPUUClampMaxFilterEnable", - "Path": "/proc/vendor_sched/uclamp_max_filter_enable", - "Values": [ - "1", - "0" - ], - "DefaultIndex": 0, - "ResetOnInit": true - }, - { - "Name": "BGUClampMaxBoost", - "Path": "/proc/vendor_sched/groups/bg/uclamp_max", - "Values": [ - "130", - "512" - ], - "DefaultIndex": 0, - "ResetOnInit": true } ], "Actions": [ @@ -856,18 +713,18 @@ "Duration": 5000, "Value": "0" }, - { - "PowerHint": "LAUNCH", - "Node": "AutoPreferIdle", - "Duration": 5000, - "Value": "0" - }, { "PowerHint": "LAUNCH_GPU", "Node": "GPUDvfsPeriod", "Duration": 1000, "Value": "10" }, + { + "PowerHint": "LAUNCH", + "Node": "FGPreferIdle", + "Duration": 5000, + "Value": "1" + }, { "PowerHint": "LAUNCH", "Type": "DoHint", @@ -893,15 +750,9 @@ }, { "PowerHint": "LAUNCH", - "Node": "CPUUClampMaxFilterEnable", + "Node": "TAUClampBoost", "Duration": 5000, - "Value": "0" - }, - { - "PowerHint": "LAUNCH", - "Node": "BGUClampMaxBoost", - "Duration": 5000, - "Value": "512" + "Value": "764" }, { "PowerHint": "LAUNCH", @@ -917,13 +768,8 @@ }, { "PowerHint": "LAUNCH", - "Type": "DoHint", - "Value": "LAUNCH_PMU" - }, - { - "PowerHint": "LAUNCH_PMU", - "Node": "PMU_POLL", - "Duration": 3000, + "Node": "NPITaskPacking", + "Duration": 5000, "Value": "0" }, { @@ -933,88 +779,15 @@ }, { "PowerHint": "LAUNCH", - "Node": "CPUDVFSHeadroom", - "Duration": 5000, - "Value": "1280" + "Type": "DoHint", + "Value": "LAUNCH_PMU" }, { - "PowerHint": "LAUNCH", - "Node": "CPUTaperedDVFSHeadroomEnable", - "Duration": 5000, + "PowerHint": "LAUNCH_PMU", + "Node": "PMU_POLL", + "Duration": 3000, "Value": "0" }, - { - "PowerHint": "LAUNCH", - "Node": "CPULittleClusterDownRateLimitUs", - "Duration": 5000, - "Value": "5000" - }, - { - "PowerHint": "LAUNCH", - "Node": "CPUMidClusterDownRateLimitUs", - "Duration": 5000, - "Value": "20000" - }, - { - "PowerHint": "LAUNCH", - "Node": "CPUBigClusterDownRateLimitUs", - "Duration": 5000, - "Value": "20000" - }, - { - "PowerHint": "LAUNCH", - "Node": "TAPreferIdle", - "Duration": 5000, - "Value": "1" - }, - { - "PowerHint": "LAUNCH", - "Node": "TAAutoPreferFit", - "Duration": 5000, - "Value": "1" - }, - { - "PowerHint": "LAUNCH", - "Node": "TAPreferredIdleMaskHigh", - "Duration": 5000, - "Value": "0xff" - }, - { - "PowerHint": "LAUNCH", - "Node": "TAUclampMinOnNiceEnable", - "Duration": 5000, - "Value": "1" - }, - { - "PowerHint": "LAUNCH", - "Node": "TAUclampMinOnNiceHighValue", - "Duration": 5000, - "Value": "765" - }, - { - "PowerHint": "LAUNCH", - "Node": "TAUclampMinOnNiceHighPrio", - "Duration": 5000, - "Value": "110" - }, - { - "PowerHint": "LAUNCH", - "Node": "TAUclampMinOnNiceMidValue", - "Duration": 5000, - "Value": "590" - }, - { - "PowerHint": "LAUNCH", - "Node": "TAUclampMinOnNiceMidPrio", - "Duration": 5000, - "Value": "139" - }, - { - "PowerHint": "LAUNCH", - "Node": "FGPreferIdle", - "Duration": 5000, - "Value": "1" - }, { "PowerHint": "LAUNCH_EXTEND", "Node": "CPUBigClusterMaxFreq", From 5d7d292cd1ea60d010a3f99ae02e3173604893b4 Mon Sep 17 00:00:00 2001 From: Chungkai Mei Date: Tue, 24 Dec 2024 08:40:32 +0000 Subject: [PATCH 2/3] Backport autopreferfit for launch Bug: 379810138 Flag: EXEMPT add powerhint config Change-Id: I984252d8cd38cdd8f0e1837f991923fc93574cba Signed-off-by: Chungkai Mei --- powerhint.json | 93 ++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 93 insertions(+) diff --git a/powerhint.json b/powerhint.json index 9c9e8a3..1fbdf9e 100644 --- a/powerhint.json +++ b/powerhint.json @@ -194,6 +194,54 @@ ], "ResetOnInit": true }, + { + "Name": "TAUclampMinOnNiceEnable", + "Path": "/proc/vendor_sched/groups/ta/uclamp_min_on_nice_enable", + "Values": [ + "1", + "0" + ], + "ResetOnInit": true + }, + { + "Name": "TAUclampMinOnNiceHighValue", + "Path": "/proc/vendor_sched/groups/ta/uclamp_min_on_nice_high_value", + "Values": [ + "765", + "0" + ], + "ResetOnInit": true + }, + { + "Name": "TAUclampMinOnNiceMidValue", + "Path": "/proc/vendor_sched/groups/ta/uclamp_min_on_nice_mid_value", + "Values": [ + "600", + "590", + "0" + ], + "ResetOnInit": true + }, + { + "Name": "TAUclampMinOnNiceHighPrio", + "Path": "/proc/vendor_sched/groups/ta/uclamp_min_on_nice_high_prio", + "Values": [ + "110", + "119", + "120" + ], + "ResetOnInit": true + }, + { + "Name": "TAUclampMinOnNiceMidPrio", + "Path": "/proc/vendor_sched/groups/ta/uclamp_min_on_nice_mid_prio", + "Values": [ + "128", + "139", + "130" + ], + "ResetOnInit": true + }, { "Name": "FGUClampBoost", "Path": "/proc/vendor_sched/groups/fg/uclamp_min", @@ -286,6 +334,15 @@ ], "ResetOnInit": true }, + { + "Name": "TAAutoPreferFit", + "Path": "/proc/vendor_sched/groups/ta/auto_prefer_fit", + "Values": [ + "1", + "0" + ], + "ResetOnInit": true + }, { "Name": "FGPreferIdle", "Path": "/proc/vendor_sched/groups/fg/prefer_idle", @@ -772,6 +829,42 @@ "Duration": 5000, "Value": "0" }, + { + "PowerHint": "LAUNCH", + "Node": "TAAutoPreferFit", + "Duration": 5000, + "Value": "1" + }, + { + "PowerHint": "LAUNCH", + "Node": "TAUclampMinOnNiceEnable", + "Duration": 5000, + "Value": "1" + }, + { + "PowerHint": "LAUNCH", + "Node": "TAUclampMinOnNiceHighValue", + "Duration": 5000, + "Value": "765" + }, + { + "PowerHint": "LAUNCH", + "Node": "TAUclampMinOnNiceHighPrio", + "Duration": 5000, + "Value": "110" + }, + { + "PowerHint": "LAUNCH", + "Node": "TAUclampMinOnNiceMidValue", + "Duration": 5000, + "Value": "590" + }, + { + "PowerHint": "LAUNCH", + "Node": "TAUclampMinOnNiceMidPrio", + "Duration": 5000, + "Value": "139" + }, { "PowerHint": "LAUNCH", "Type": "DoHint", From 6e1fd03c381705e318822bc63bd08bb3fdfb6756 Mon Sep 17 00:00:00 2001 From: Chungkai Mei Date: Tue, 24 Dec 2024 09:14:52 +0000 Subject: [PATCH 3/3] Change set preferred idle mask to top-app Set value to 0x3f exclude big core for all low/mid priority when app launch. Bug: 379810138 Flag: EXEMPT add powerhint config Change-Id: Iaeb21216459a150107f44838aec5e410cb278a30 Signed-off-by: Chungkai Mei --- powerhint.json | 32 ++++++++++++++++++++++++++++++++ 1 file changed, 32 insertions(+) diff --git a/powerhint.json b/powerhint.json index 1fbdf9e..a827cdb 100644 --- a/powerhint.json +++ b/powerhint.json @@ -343,6 +343,26 @@ ], "ResetOnInit": true }, + { + "Name": "TAPreferredIdleMaskMid", + "Path": "/proc/vendor_sched/groups/ta/preferred_idle_mask_mid", + "Values": [ + "0x3f", + "0xf0", + "0xff" + ], + "ResetOnInit": true + }, + { + "Name": "TAPreferredIdleMaskLow", + "Path": "/proc/vendor_sched/groups/ta/preferred_idle_mask_low", + "Values": [ + "0x3f", + "0xf0", + "0xff" + ], + "ResetOnInit": true + }, { "Name": "FGPreferIdle", "Path": "/proc/vendor_sched/groups/fg/prefer_idle", @@ -835,6 +855,18 @@ "Duration": 5000, "Value": "1" }, + { + "PowerHint": "LAUNCH", + "Node": "TAPreferredIdleMaskMid", + "Duration": 5000, + "Value": "0x3f" + }, + { + "PowerHint": "LAUNCH", + "Node": "TAPreferredIdleMaskLow", + "Duration": 5000, + "Value": "0x3f" + }, { "PowerHint": "LAUNCH", "Node": "TAUclampMinOnNiceEnable",