Commit graph

142 commits

Author SHA1 Message Date
Treehugger Robot
aba5cf7aa4 Merge "Use auto prefer fit for launch" into main 2024-10-07 16:17:21 +00:00
Sam Ou
a3244ab2c8 Merge "thermal: Remove bcl related tzones" into main 2024-09-18 15:49:46 +00:00
Rick Yiu
827f321757 Use auto prefer fit for launch
Enable auto prefer fit for high prio tasks when app launch, and tune
uclmap min settings for different task prio.

Bug: 353885521
Flag: NONE powerhint change does not support flags
Test: build pass
Change-Id: Iad6f5d332e66dc09ed12d09e3e443144bc25c31d
2024-09-13 09:56:57 +00:00
Wei Wang
8c8f944c73 Merge "Enable auto prefer idle" into main 2024-08-29 23:51:42 +00:00
Ali K. Zadeh
f5d247a3f2 Revert^2 "Set max load balance to 4, reduce Downrate Limit for lit and mid cores"
This reverts commit e00b219010.

Reason for revert: Fix the tokay powerhint

Test: tokay device boots successfully
Bug: 354763942

Change-Id: I218bd68a770bf8af43f154a51cdc31521486950f
2024-08-19 16:50:57 +00:00
Maciej Żenczykowski
e00b219010 Revert "Set max load balance to 4, reduce Downrate Limit for lit and mid cores"
This reverts commit d1274379ad.

Reason for revert:

git_main tokay-trunk_staging-userdebug boot failure (something about failing to parse /vendor/etc/powerhint.json).

https://android-build.corp.google.com/test_investigate/?invocationId=I81200010307251269&testResultId=TR31029411252209740&legacy=&tab=changes

Confirmed locally a revert results in a phone that boots.

Change-Id: I134fc0926170940f374a04362b71f5b9c28188d4
2024-08-17 05:02:12 +00:00
Ali K. Zadeh
d1274379ad Set max load balance to 4, reduce Downrate Limit for lit and mid cores
Bug: 354763942
Test: b/354763942#comment29
Flag: EXEMPT resource file change only
Change-Id: I7683bd8847504b7db2fbd19f31ccb6af1b9037de
2024-08-08 22:25:08 +00:00
Paulsu Su
77224746ca powerhint: CPULittleClusterMinFreq default 820MHz
Change-Id: Ib546be142955d36eb417f34d6ab086833e380320
Test: boot to home
Bug: 348088963
Flag: NONE powerhint do not support flag
2024-08-07 07:27:06 +00:00
samou
d35eeb2c3d thermal: Remove bcl related tzones
Flag: EXEMPT tuning changes
Bug: 349935208
Test: Confirm boot
Change-Id: I2029cde12297d51ab14e336e3ce733cbed879f6c
Signed-off-by: samou <samou@google.com>
2024-08-06 03:55:18 +00:00
Martin Liu
ae6907b6b8 Boosting kswapd uclamp min value when the panel is on
Boosting kswapd's uclamp minimum value in the scheduler
allows kswapd more opportunities to remain in middle cores.
This change enhances kswapd's memory reclamation efficiency.

Bug: 351708752
Test: check kbnob value when the panel is on/off
Flag: EXEMPT device policy tuning
Change-Id: I4b75016aa3f2de22b0b9e46d8e594d4042124a04
Signed-off-by: Martin Liu <liumartin@google.com>
2024-08-01 14:29:57 +00:00
Rick Yiu
89d596b0f8 Enable auto prefer idle
Also disable prefer idle for TA and FG by default.

Bug: 236868627
Flag: NONE powerhint change does not support flags
Test: build pass
Change-Id: I0c09c4ef30e10d358a71de11283dda43b4f96837
2024-07-30 09:11:58 +00:00
Treehugger Robot
874d9c7a23 Merge "Adpf:caimito: switch to multitier heuristic boost." into main 2024-07-27 00:58:12 +00:00
guibing
ca3ac26d16 Adpf:caimito: switch to multitier heuristic boost.
Bug: 344685639
Test: Checked the trace file running on the device.
Flag: NONE powerhal doesn't have a flag yet
Change-Id: I550086e5c47991ed25c3b988ca11e8cf9df5d118
2024-07-26 23:21:38 +00:00
Ali K. Zadeh
fb22d799d6 [caimito] Enable PA_KILL during PA_KILL_BOOST
Currently, we enable PA_KILL always on camera launch. This CL puts this
feature behind a power boost profile which can only be triggered from
camera hal and is controlled by a mendel flag.

Bug: 350594261
Test: adb logcat
Flag: EXEMPT resource only update
Change-Id: Id227bd35fda97d5b0cbdfe9e4df39eff2e565b7d
2024-07-19 18:58:03 +00:00
Zorah Chen
0495e5cffe Merge "Powerhint tuning for photo capture" into main 2024-07-12 05:54:56 +00:00
zorahchen
d68bded2d6 Powerhint tuning for photo capture
UClamp.min of all threads in top app is set to mid core max capacity during photo capture, which will dispatch them to big core.
Critical threads of photo capture will be interrupted due to their lower priority, while other cores idle.

To deal with this situation,
prefer_idle mask to big/mid cores can reduce the idle in mid cores.
UClamp.min on nice can set UClamp.min of photo capture critical threads to mid core max capacity, while others as zero,
this can make critical threads run on big core without frequently interrupted.
By applying this, photo capture latency can be reduced with slight change of power consumption.

Bug:345601796
Performance Test:https://g3doc.corp.google.com/wireless/android/camera/camera_performance_analyzer/power_tuning/results/g3doc/pmu_retune_capture_enhanced/index.md?cl=alikz/1010
Kibble Power Test:https://g3doc.corp.google.com/wireless/android/camera/camera_performance_analyzer/power_tuning/results/g3doc/pmu_kibble_capture_rt_enhanced/index.md?cl=alikz/1044
Flag: NONE powerhint change does not support flags
Change-Id: I8683b50b212d50b85b22b5cb48d712fc8eb43444
2024-07-12 02:03:49 +00:00
Ali K. Zadeh
7c81eff0a1 Increase SPC thresholds based on the field data
Bug: 350111391
Test: Kibbles/Framedrops https://docs.google.com/document/d/1QvryQvAj_hjSjouG7P6MD1HwVB0n46GZNvmZgz6uQ9M/edit?resourcekey=0-rexjumgYT6SqyTbb0OvOEg&tab=t.0
Test: https://docs.google.com/spreadsheets/d/1e1pm7N1GBfYHU9tvG3gx9Z84tVYCokDlpSnHF2wlbbA/edit?gid=0#gid=0
Flag: EXEMPT bugfix

Change-Id: Iad09beafc595e89d86131b00a3a221af0688eb7a
2024-07-10 16:19:27 +00:00
Kurt Huang
0f93229c19 powerhint: set valid PMU_BIG_SPC_THRESHOLD for CAMERA_PROJECT11
http://ag/27702721 removed value 50 from PMU_BIG_SPC_THRESHOLD. Set the
value to 20 for CAMERA_PROJECT11.

Bug: 347647774
Test: Flash on ripcurrentpro board and boot
Change-Id: I20aa20119ab8317d3ba92b8118d34f3b9e884b66
2024-07-02 07:21:57 +00:00
Richard Chang
62e495a0dd Merge "Reapply "powerhint: add proactive kill hint for camera launch"" into main 2024-06-27 07:19:51 +00:00
Qais Yousef
176560b44a powerhint: Enable auto migration margins/dvfs headroom for game mode
Since the system is more reactive now when tasks are getting busier, set
DownRateLimitUs to 500us to avoid increasing power.

Also enable pertask iowait boost as util_avg will always start from
0 for forked tasks and this could impact io tasks that need additional
help to ensure to migrate to bigger core if they need more perf.

Bug: 339664411
Test: PTE A/B test
Flag: NONE powerhint change does not support flags
Signed-off-by: Qais Yousef <qyousef@google.com>
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:c09bc9d8b58cc4f9bba136a1dc97b5c0b7751dce)
Merged-In: Iecbbaccc81297ee0f66f8c45299846f2415ee29c
Change-Id: Iecbbaccc81297ee0f66f8c45299846f2415ee29c
2024-06-24 23:08:44 +00:00
Qais Yousef
0de9ecec3a 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: 339664411
Test: PTE A/B test
Flag: NONE no behavior change
Signed-off-by: Qais Yousef <qyousef@google.com>
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:6e7e510a8eeace8d37836d6e428e13a61d66f0c1)
Merged-In: Icbbfb2d332b412fa51d6d434d44d1ada8545338c
Change-Id: Icbbfb2d332b412fa51d6d434d44d1ada8545338c
2024-06-24 23:08:28 +00:00
Yvonne Yip
eb1267c4d3 powerhint: Add nodes required to enable auto migration margins/dvfs headroom
Everything should be disabled by default.

Make sure camera and game explicitly disable them by default too.

Ensure the feature is disable when screen is idle.

Bug: 339664411
Test: PTE A/B test
Flag: NONE no behavior change
Signed-off-by: Yvonne Yip <ykyyip@google.com>
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:a4523a214541f2cd3a079c010790c372b8058f31)
Merged-In: I3dd2bfe1824af489cd1ccc3a69557c180bf33ddf
Change-Id: I3dd2bfe1824af489cd1ccc3a69557c180bf33ddf
2024-06-24 23:05:07 +00:00
Qais Yousef
d92ae25af8 powerhint: Explicitly set DownRateLimitUs for camera
So that we can change the default values without impacting it.

Bug: 339664411
Test: PTE A/B test
Flag: NONE no behavior change
Signed-off-by: Qais Yousef <qyousef@google.com>
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:eb1962034f12f42907e5641a195f49456e951b31)
Merged-In: I8ca117bba2e0b59802de5fb19eae507548ff0a6e
Change-Id: I8ca117bba2e0b59802de5fb19eae507548ff0a6e
2024-06-24 22:58:21 +00:00
Richard Chang
4fe16883a8 Reapply "powerhint: add proactive kill hint for camera launch"
This reverts commit 09e63f9519.

Bug: 322916612
Test: local test
Flag: EXEMPT resource only update
Change-Id: Ida78f3e34cd7ac29b9d6311b6c1afb6f57bca224
2024-06-21 07:02:05 +00:00
Wei Wang
7427169bcb Merge "Set big core to group_cfs_skip_mask for sys group by default" into main 2024-06-17 17:08:07 +00:00
Lynn Yeh
7ebbdf444d Merge changes from topic "pmu_fix" into 24D1-dev am: 3a4927f4ec
Original change: https://googleplex-android-review.googlesource.com/c/device/google/caimito/+/27765730

Change-Id: Ic8e2ed0f2659a7eebc72448d8d7458910edead3f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-06-13 03:06:45 +00:00
Ali K. Zadeh
7f98995d82 Update PMU thresholds with PMU ignorance logic disabled am: 8a6924bd35
Original change: https://googleplex-android-review.googlesource.com/c/device/google/caimito/+/27702721

Change-Id: If07b24c018a4fab6fea5c0a70718de2546d0ad76
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-06-13 03:06:41 +00:00
Ali K. Zadeh
f1dfb794d4 Reduce CPUDVFSHeadroom for mid and big core (for 60fps and 240fps)
Reducing CPUDVFSHeadroom can help save power in 60fps and 240fps
use-cases once RT threads are enabled. Even without the RT patch
(ag/27725456), we can see savings without any impact to framedrop rate.

Bug: 341495529
Test: https://docs.google.com/document/d/14djeCdDIjDy9wqtYc4C9OPnp7sOx3HfbQz4dIZarvVE
Change-Id: Ib16a52d83669c5e9705d80c48b7097e102cc807f
2024-06-12 23:43:15 +00:00
Ali K. Zadeh
8a6924bd35 Update PMU thresholds with PMU ignorance logic disabled
pa/2830448 removes the core ignorance logic in PMU and causes the
behavior of PMU to change. With the new PMU logic, we need to re-tune
all the SPC thresholds for camera use-cases. This CL adjusts SPC
thresholds accordingly, so that we do not have power regression once the
PMU change in the kernel is merged.

Bug: 341495529
Test: https://docs.google.com/document/d/1IZDKtnyEnMZcakaPzQHtF23az6EjCTRGw3SdWt0Up68/edit?usp=sharing&resourcekey=0-ZAvo0M9HA-f-DY42vIatWA
Change-Id: Ifa10ebfa66da90832930487ef2d6b5610a7999bb
2024-06-11 00:06:52 +00:00
Michael Williams
bcd4392974 Merge "Switch Faceauth FACE_UNLOCK_BOOST Power Hint to Mid Cluster" into main 2024-06-06 15:19:29 +00:00
Treehugger Robot
2133c5133f Merge "Revert "powerhint: add proactive kill hint for camera launch"" into main 2024-05-30 03:39:47 +00:00
Minchan Kim
09e63f9519 Revert "powerhint: add proactive kill hint for camera launch"
Revert submission 27473137-P24_PA_KILL_HINT_24Q4

Reason for revert: will merge after mid of June

Reverted changes: /q/submissionid:27473137-P24_PA_KILL_HINT_24Q4

Change-Id: Ie6fadb9881a00486ff631ab286aca9ff062a7a40
Bug: 322916612
2024-05-30 00:58:50 +00:00
Sam Paradis
cb9eb99be0 Switch Faceauth FACE_UNLOCK_BOOST Power Hint to Mid Cluster
Faceauth is pinning to the mid-core for P24. Adjust the power hint to boost the appropriate cluster.

Bug:329893113

Change-Id: Ic13ad468a647c9088c03dd3d92e261f48944710d
2024-05-29 23:57:54 +00:00
Minchan Kim
b56cd60d53 Merge "powerhint: add proactive kill hint for camera launch" into main 2024-05-28 17:01:59 +00:00
Rick Yiu
8f43205282 Set big core to group_cfs_skip_mask for sys group by default
Let sys group tasks will run less in big core by default.

Bug: 328210236
Test: build pass
Change-Id: Ic4941e88138c8c836e1189ec89fd027b4d6bff90
2024-05-24 15:06:03 +00:00
Shijie “Jessie” Hu
d08c793986 Merge "[Project11] CPU scheduling update." into 24D1-dev am: 43ec216bd3
Original change: https://googleplex-android-review.googlesource.com/c/device/google/caimito/+/27473443

Change-Id: Id2ea8cb86d7a3d8adf2ad5b773da3e9884262bd4
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-05-24 03:11:05 +00:00
Shijie “Jessie” Hu
43ec216bd3 Merge "[Project11] CPU scheduling update." into 24D1-dev 2024-05-24 02:53:49 +00:00
Shijie (Jessie) Hu
061f44ae4f [Project11] CPU scheduling update.
Bug: 341968564
Test: power and perf test on Komodo.Test result show in b/341968564#comment27

- Use newly tuned STREAMING_STANDARD setting.

Change-Id: If64ad8148c4740fe5716381d21543bc9d26ab623
2024-05-23 22:53:23 +00:00
Richard Chang
15705aa358 powerhint: add proactive kill hint for camera launch
Bug: 322916612
Test: local test
Change-Id: I9f65cbbb7bcb4a85afee50d1a85a753deebd1386
2024-05-23 03:46:22 +00:00
Taeju Park
c3ffd4e875 powerhint: bug fix for PMU_FORCE_LIMIT am: 8a52ed0919
Original change: https://googleplex-android-review.googlesource.com/c/device/google/caimito/+/27441386

Change-Id: Ib2a7edbec896398ea7816989e76672a56651cd17
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-05-21 21:44:57 +00:00
Taeju Park
8a52ed0919 powerhint: bug fix for PMU_FORCE_LIMIT
PMU_FORCE_LIMIT node is used without declaration

Bug: 341151887
Change-Id: I6ca864d80e25a5c81f04606d0a3315edd1357386
Signed-off-by: Taeju Park <taeju@google.com>
2024-05-21 18:50:49 +00:00
Ali K. Zadeh
df45ec3772 60fps: change PMU max frequency for CL2 to 1164MHz am: c14ec8230a
Original change: https://googleplex-android-review.googlesource.com/c/device/google/caimito/+/27327800

Change-Id: I700e17de5b99ee53ad2b9335268d22aad13adc81
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-05-17 05:35:16 +00:00
Ali K. Zadeh
c14ec8230a 60fps: change PMU max frequency for CL2 to 1164MHz
Bug: 340289125
Test: clock residency
(https://docs.google.com/spreadsheets/d/1TQQU5TGHE6B0xOgd77tyUyGnU-oksXi1X0Bs_vCsENE/edit#gid=1243247080)
Test: Kibble (https://docs.google.com/spreadsheets/d/1WF9WVVeNvDSawzUtrq-L89jh1vuAootVET3vRXQTJR8/edit#gid=1987296588)

Change-Id: I18fc6a3ffe535278d95ea32e855d3b8b8ab12b4a
2024-05-16 22:18:54 +00:00
Taeju Park
70bbca81db Merge "powerhint: add PMU_FORCE_LIMIT control knob" into 24D1-dev am: ef1f5d317a
Original change: https://googleplex-android-review.googlesource.com/c/device/google/caimito/+/27215478

Change-Id: I6b5eeefd3448f5f5072053a8eae3ea6ca1c413b1
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-05-10 19:21:08 +00:00
Taeju Park
ef1f5d317a Merge "powerhint: add PMU_FORCE_LIMIT control knob" into 24D1-dev 2024-05-10 19:07:40 +00:00
Ali K. Zadeh
38b4b1562b Merge "Enable PMU during photo capture" into 24D1-dev am: 0508270ad8
Original change: https://googleplex-android-review.googlesource.com/c/device/google/caimito/+/27238472

Change-Id: Icb25839791a8bed17ee44af3de72f8bd2485e890
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-05-10 00:17:48 +00:00
Ali K. Zadeh
0508270ad8 Merge "Enable PMU during photo capture" into 24D1-dev 2024-05-09 23:58:52 +00:00
Ali K. Zadeh
730f445f01 Enable PMU during photo capture
Bug: 298227152
Test: Kibble 12MP - https://docs.google.com/spreadsheets/d/1Exxzy_7CHu4RQQ_-eR2s16Ob9BpxdQZFn6GIkUmoc4k/edit?resourcekey=0-CrG5HCXn9v7iR0PFk84j2g#gid=328258859
Test: Kibble 50MP - https://docs.google.com/spreadsheets/d/1qIdxci3qRwgM_XD5rRdmQPqDrsf8VHQIqQggDh2rgGY/edit?resourcekey=0-t25YyqKalsLbrCM7ADDHlQ#gid=2134912777
Test: Kibble 12MP - repeated capture - https://docs.google.com/spreadsheets/d/1XTwC5iCaobxwCcj5QH1f_34lJysvFDv53wt4K9l8DAM/edit#gid=1477598242
Test: Kibble 50MP - repeated capture - https://docs.google.com/spreadsheets/d/17wlxE2MsRx56Vhr9IE1iBYLz1Lee1s7Bb7E_GKaBJtc/edit?resourcekey=0-Vh8vf55xZHgYnWqDESm5lA#gid=1630323952
Test: Latency 12MP - https://docs.google.com/spreadsheets/d/1xUmS4H8SmwKzrL-CScIhZABsrlCq1CZsDT9ZSWFAMwU/edit#gid=466916271
Test: Latency 50MP - https://docs.google.com/spreadsheets/d/1GUs1TVCCOD6Bv73ilxrjg99tJVq6wQ5kJfYrJ1djY54/edit#gid=1885938822
Change-Id: I5083184f9e226d42584de5b6910994f1d5d3cc52
2024-05-09 22:44:32 +00:00
Rick Yiu
8f96f84997 Set big core to group_cfs_skip_mask for sys group am: d8c84c47ff
Original change: https://googleplex-android-review.googlesource.com/c/device/google/caimito/+/27280353

Change-Id: Ia3e88bd2b40c562e24e46adaab15baca7613025d
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-05-09 16:45:19 +00:00
Rick Yiu
d8c84c47ff Set big core to group_cfs_skip_mask for sys group
Set big core to group_cfs_skip_mask for sys group when screen is off,
so that sys group tasks will run less in big core.

Bug: 328210236
Test: suspend/resume test
Change-Id: I4868ba0303e981250105b1ff69a6db6458df844a
2024-05-09 16:06:24 +00:00