Commit graph

85 commits

Author SHA1 Message Date
Treehugger Robot
14484bec22 Merge "powerhint: Raise minimum MIF frequency to 546 MHz during screen on" into main 2024-09-04 04:16:15 +00:00
Chungkai Mei
d37470ea6e powerhint: Raise minimum MIF frequency to 546 MHz during screen on
It improved 55% surfaceflinger janks in photos_playgca30fpsvideo cuj test case.

Test: Build and power measurement
Bug: 336872050
Flag: NONE powerhint change does not support flags
Change-Id: Idad3394e5ac9d1d275f144e2d1304a115b7dc3af
Signed-off-by: Chungkai Mei <chungkai@google.com>
2024-08-23 01:35:02 +00:00
Sam Ou
cc2b786053 Merge "thermal: Remove bcl related tzones" into main 2024-08-12 12:32:35 +00:00
samou
697c73328c thermal: Remove bcl related tzones
Flag: EXEMPT tuning changes
Bug: 349935208
Test: Confirm boot
Change-Id: I9162338b20afd9b5d4472d8a7176c69e04ce0175
Signed-off-by: samou <samou@google.com>
2024-08-06 07:40:49 +00:00
guibing
7eeceadfac Adpf:felix: 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: I1b37c1e71eb148a4dab454b7e7db9cf46f09d9a3
2024-07-26 23:19:57 +00:00
guibing
ff36604d18 Adpf:felix: enable heuristic boost.
Heuristic boost will adjust the ADPF boost profile when the
powerhal detects the power session is very janky, providing more
boost for deadline sensitive threads managered in the session.

Bug: 341111433
Test: adb shell dumpsys android.hardware.power.IPower/default;
      Check the device adpf sessions' traces.
Change-Id: I02ed6e281447a080346337554d9566e4173e57b7
2024-05-16 20:45:57 +00:00
Matt Buckley
d0669d782e Merge "Add POWER_EFFICIENCY uclamp max to adpf config" into main 2024-04-29 19:42:15 +00:00
Jimmy Shiu
1ab55d56bc powerhint: set ADPF CPU_LOAD_UP CPU_LOAD_RESET values
Bug: 329483662
Test: Manual Test
Change-Id: I9b748e2fb6110cac3d8589da2f3bb6d89a619ed3
2024-04-10 11:45:08 +00:00
Jimmy Shiu
8bed816740 powerhint: set uclampmin_init value for applying CPU Load boost
Bug: 329483662
Test: Manual Test
Change-Id: I9e3afbeb455391ab1493f184bbf27f3c8e6c8e5a
2024-04-03 01:04:47 +00:00
Sam Paradis
011bda50b8 Change FACE_UNLOCK_BOOST to Maximize CPU Frequency
This is part of an optimization that limits the length and prevelance of
this power hint. Strict, ultra-agression minimizes power and latency
versus always-on mid aggression.

Bug: 297964952
Change-Id: I6d6f3b6c8990bbe3c04575a60797a2aefa898172
2024-03-13 17:12:21 +00:00
Treehugger Robot
68451c06e0 Merge "Camera: Adjust to improve camera launch and capture performance" into main 2024-01-24 06:50:18 +00:00
Pointer Kung
d53fe60449 Camera: Adjust to improve camera launch and capture performance
Because F10 enabled PMU and EM for 30fps, it also needs to sync
ag/19286153 and ag/20084677 to enable CDPreferIdle for
CAMERA_STREAMING_STANDARD and disable PMU and EM during the capture
time (also increase the boost duration from 1000ms to 2500ms) to
avoid performance impact.

Bug: 321164832
Test: Build pass, gca_smoke_test, CTS
Change-Id: I5e40662f7e49bd381891f705621209a41ddc96d7
2024-01-22 05:39:44 +00:00
paulsu
2056632361 Disable PMU at the start of camera launch
Follow ag/20075269, ag/25261942 has caused regression in GCA launch
time due to activation of PMU at the camera launch time, and capping the
CPU frequency. This CL prevents PMU activation for as long as we are in
powerhints or power-boosts related to camera launch (LAUNCH, CAMERA_LAUNCH,
CAMERA_LAUNCH_EXTENDED).

Bug: 321164832
Test: perfetto trace

Change-Id: I54cb50259adc145847c96e75e2dd79f3eb3b3231
Signed-off-by: paulsu <paulsu@google.com>
2024-01-19 06:53:21 +00:00
David Chen
ec09d678cb Add zoom boost in powerhint for F10
This patch is meant to reduce frame drop rate in 60FPS video recording with zoom in/out.
The boost will be triggered at 60FPS video recording while zoom ratio change (for each request that contain zoom ratio control in metadata), and will last for 15 frames.

Test: gca_smoke_test
Bug: 305613607
Change-Id: Ia2a6a452cf2ac7b8ee7a1bd53e5085582d79a2be
2023-12-05 08:28:18 +00:00
paulsu
a227608517 Powerhint update containing PMU and EM tuning for 30FPS
Port from ag/20028064

Bug: 262668624
Test: Local test
Change-Id: Ia2cd6e1c55fdc2d2cefb507782208cd19dc651df
Signed-off-by: paulsu <paulsu@google.com>
2023-11-07 06:44:45 +00:00
guibing
806d6a25f1 powerhint: felix: update vendor_sched procfs paths
Vendor sched procfs path hierarchy gets updated in kernel.
Update the paths in powerhint json file to match the change.

Bug: 289151587
Test: UIbench ab test.
Change-Id: I3028f9f97d65f92fd43651995790d3902d0883f7
2023-10-11 00:00:05 +00:00
Jimmy Shiu
979e0ab288 Merge "powerhint: Add DISPLAY_CHANGE hint" into main 2023-10-03 14:57:42 +00:00
Rick Yiu
e952b1cf6f Disable limiting camera_hal for LAUNCH hint
If auto rotation is enabled, it will trigger LAUNCH hint while
camera is running if device rotates, which could limit camera_hal
tasks to little cores. So, disable it first until there is a new
hint for the rotation event.

Bug: 297273578
Test: build pass
Change-Id: I2902d23c9c0c0e31cd0a444d1e8b7a953c41a8d8
2023-09-19 02:51:40 +00:00
jimmyshiu
9901f4e467 powerhint: Add DISPLAY_CHANGE hint
Add DISPLAY_CHANGE hint for display layout change due to rotation or
switching between inner and outer panels.

Bug: 298150450
Test: build pass
Change-Id: I0f7667c3368843b12bbc0bd3151864bde6367a00
2023-09-14 20:45:57 +08:00
paulsu
71c09f7978 F10 camera pmu tuning with on 60fps case
P22 pmu settings caused power and frame drop regression. This patch lower the little frequency limit and lower the pmu threshold. And see lower power and no frame drop regression.

Bug: 262354426
Test: Camera team power and performance test, full result can be found in b/262354426#comment43
Change-Id: I5300b8f87565e3478ee1c97441d282e268935147
Signed-off-by: paulsu <paulsu@google.com>
2023-09-01 03:45:04 +00:00
Matt Buckley
8f82ebc72c Add POWER_EFFICIENCY uclamp max to adpf config
Add the "UclampMax_EfficientBase" value to the ADPF config in the
powerhint.json to specify efficient core scheduling in the static
case, and "UclampMax_EfficientOffset" for the dynamic case, for this
device.

Bug: 277285195
Test: manual
Change-Id: Ibeca70e171d8c15ec73eb57cd8dd7a5bf21eaea5
2023-08-19 05:25:10 +00:00
Treehugger Robot
d71ef7e1cb Merge "disable vendor idle balancer" into udc-qpr-dev 2023-07-12 14:48:38 +00:00
Chungkai Mei
c9d309c9f5 disable vendor idle balancer
Camera is highly optimized in launched devices, so that we disable idle balancer to avoid influencing previous optimization.

Bug: 281489720
Change-Id: I210f5c14ec589b06e8d8e80799cf7cb650baa9c5
Signed-off-by: Chungkai Mei <chungkai@google.com>
2023-06-21 02:43:50 +00:00
TeYuan Wang
be5d3b337b powerhint: remove the duplicate thermal powerhint settings
For the current design, thermalHAL will only enable one hint base on the current severity and disable the previous one.
As a result, the power hint of higher severity should maintain the actions from lower severity which cause the larger config size and maintenance effort.

The new thermalHAL design will keep the lower severity's hint enabled, so we could remove the duplicate policies from higher level.

Bug: 241737737
Test: verified felix with emul_temp
Change-Id: I5f265fc07bdf1071902071c252b6b78040dfff38
2023-06-16 13:23:46 -07:00
TreeHugger Robot
2b0862874e Merge "[Camera] Add CAMERA_MULTI_CAM_STREAMING power hint" into udc-dev am: fe04c75dd5 am: 616954570d
Original change: https://googleplex-android-review.googlesource.com/c/device/google/felix/+/23626525

Change-Id: I9bf71940987853488deb8bd6e38ea4f1b0e51089
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-06-15 07:16:58 +00:00
TreeHugger Robot
fe04c75dd5 Merge "[Camera] Add CAMERA_MULTI_CAM_STREAMING power hint" into udc-dev 2023-06-15 05:15:01 +00:00
paulsu
d7fcbb1933 Remove UgBgGroupThrottle node to keep ug_bg_group_throttle 308 by default am: 8a7c8edcca am: 66203c644c
Original change: https://googleplex-android-review.googlesource.com/c/device/google/felix/+/23686048

Change-Id: I90644fd802e02472c67a224c7aa1d7057a447fe5
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-06-14 09:50:47 +00:00
paulsu
8a7c8edcca Remove UgBgGroupThrottle node to keep ug_bg_group_throttle 308 by default
Bug: 280258727
Test: Found regression in b/286002292, local test.
Change-Id: I2f4225be0b98a11028edf1ba4271f2ce6da8c8db
Signed-off-by: paulsu <paulsu@google.com>
2023-06-14 05:25:17 +00:00
Ziyi Cui
f619bc1510 powerhint: Reduce StaleTimeFactor
Reduce StaleTimeFactor for power saving

Bug: 269523712
Test: boot and run uibench
Change-Id: I0c8811a78ca66c8f9ec2f9c40c83fd02655eb0f0
Signed-off-by: Ziyi Cui <ziyic@google.com>
2023-06-13 17:41:24 +00:00
Vince Chiu
7f4285b673 [Camera] Add CAMERA_MULTI_CAM_STREAMING power hint
Not limit max CPU frequency when running multicamera streaming on GCA photo mode.

Test: GCA
Bug: 286792574
Change-Id: Iecf36827a4f26af4a07b6f8d26cff8e942cd8098
2023-06-12 08:24:55 +00:00
paulsu
a7700b4795 Create group throttle powerhint
Test: Local test
Bug: 280258727
Change-Id: Icdb30bbcf4d673487848a288cd207cf7560fa996
Signed-off-by: paulsu <paulsu@google.com>
2023-06-08 10:50:47 +00:00
Rick Yiu
af6713737b Merge "The dvfs headroom was split into its own knob. Update to use it." into udc-dev 2023-05-12 01:02:24 +00:00
Qais Yousef
39f1bf7894 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.

Bug: 261705301
Signed-off-by: Qais Yousef <qyousef@google.com>
Change-Id: Id026048db7c9ec21fd843783b918201230774d2a
2023-05-10 10:30:11 +00:00
Mao Lee
1ff12806bb Camera: Add the property and create the power hint to adjust backend min clock for 1080p30fps video stream for front cameras
Bug: 279141417
Test: adb logcat, GCA, CTS
Change-Id: I0f2a11980d0db5893e5f235271d569184b11dd43
2023-05-08 03:28:11 +00:00
Rick Yiu
99bd5e7c56 Fix wrong setting of ML_ACC
Bug: 261695590
Test: device boots
Change-Id: I53ee01f334009c460eae1ea4faa7bdde094d22f4
2023-04-26 09:15:48 +00:00
Wei Wang
5f9e313f78 Merge "thermal: lower tj control temp during battery save" into udc-dev 2023-04-25 21:46:42 +00:00
Qais Yousef
777bbfe51a Merge "powerhint.json: Update uclamp_min hint to account for margin" into udc-dev 2023-04-25 21:27:51 +00:00
Peter (YM)
479ce68e37 thermal: lower tj control temp during battery save
Use LOW_POWER power hint to detect and lower the control temp
for junction control loops

Bug: 266011718
Test: Build and boot on device. Check thermal control temperatures
during battery save
Change-Id: Ic2f643a5c48aefdd7ea137674513cd57c82ab23d
Signed-off-by: Peter (YM) <peterym@google.com>
2023-04-24 13:31:54 +08:00
Jimmy Shiu
6e6e02f461 powerhint: Boost GPU(302000) by 1st frame
Use CPU_LOAD_RESET hint to boost the GPU freq to 302000

Bug: 243025173
Test: Manual
Change-Id: I370657c0e01baf936144024c12c1f8502624073f
2023-04-12 22:04:23 +08:00
Qais Yousef
98f52f4845 powerhint.json: Update uclamp_min hint to account for margin
Upstream has fixed relationship between uclamp_min and margin which
means uclamp_min value between 80-100% of capacity of the CPU is a valid
value and will not cause upmigration.

To keep the current hints in powerhint.json the same, we must update
uclamp_min to be capacity_orig_of(medium_cpu) + 1.

This change is dependent on uclamp vs fits margin in GKI, vendor
modules. And an improvement in sched_pixel to ignore the 25% boost if
the only reason to request the frequency is uclamp_min.

To cater for the later - we multiply each uclamp_min value with 1.25
- except ones that look intentionally set around capacity boundaries.

Bug: 261695590
Test: PtsUiBench
Signed-off-by: Qais Yousef <qyousef@google.com>
Change-Id: I60f4d7cd2cecf7ff4d73fa0cee1a09522b1d9bfd
2023-04-06 16:35:17 +00:00
TreeHugger Robot
4993a8c17a Merge "Camera: Lower CPU min frequency for GCA_CAMERA_SHOT_ALLCPU" into tm-qpr-dev am: 34e623f4c7 am: eed02aafd7
Original change: https://googleplex-android-review.googlesource.com/c/device/google/felix/+/21982323

Change-Id: I78cf5fb4617e367aca0f62434dc99fcb63fcbcc5
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-03-28 00:40:09 +00:00
pointerkung
dabbb43dbf Camera: Lower CPU min frequency for GCA_CAMERA_SHOT_ALLCPU
Bug: 272483004
Test: Build pass
Change-Id: I8f5a9c5459913de33dad957c939b8f51cc832119
2023-03-13 13:46:14 +08:00
Jimmy Shiu
4726366a11 Merge "powerhint: Boost GPU(302000) by 1st frame" into tm-qpr-dev 2023-03-08 18:17:03 +00:00
Jimmy Shiu
ee538e6ebc powerhint: Boost GPU(302000) by 1st frame
Take ADPF_FIRST_FRAME hint to boost the GPU freq to 302000

Bug: 243025173
Test: Manual
Change-Id: Iac5c503e6fd8f19f0252b65f9ef10ed82b3ed378
Merged-In: Iac5c503e6fd8f19f0252b65f9ef10ed82b3ed378
2023-03-07 01:16:29 +00:00
Jimmy Shiu
4e0918a63c powerhint: Boost GPU(302000) by 1st frame
Take ADPF_FIRST_FRAME hint to boost the GPU freq to 302000

Bug: 243025173
Test: Manual
Change-Id: Iac5c503e6fd8f19f0252b65f9ef10ed82b3ed378
2023-03-07 01:15:37 +08:00
Wei Wang
93c1a22615 Merge "thermal: more aggresive dex2oat throttling" into tm-qpr-dev am: ffa79ad7a6 am: 5077617ce8
Original change: https://googleplex-android-review.googlesource.com/c/device/google/felix/+/21475504

Change-Id: I1c4d932464db851d062997132d2a25d1e1048f84
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-02-24 23:48:51 +00:00
Wei Wang
ffa79ad7a6 Merge "thermal: more aggresive dex2oat throttling" into tm-qpr-dev 2023-02-24 22:26:30 +00:00
Rick Yiu
109fc044e1 Merge "Change CAMERA_CAPTURE_CPU_THROTTLE setting" into tm-qpr-dev am: 6011bed809 am: 26b5d65463
Original change: https://googleplex-android-review.googlesource.com/c/device/google/felix/+/21153641

Change-Id: Iea3e74528c79ae66f6d13ba993e9c90a23048b6d
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-02-17 09:56:38 +00:00
TeYuan Wang
9fab91e5ed thermal: more aggresive dex2oat throttling
Bug: 267841021
Test: Build and boot on device
Change-Id: Ic8356a391fc21af070c6700a310b0975b3ee5a56
2023-02-17 13:00:55 +08:00
Rick Yiu
99308c016c Change CAMERA_CAPTURE_CPU_THROTTLE setting
Sync the value to CAMERA_STREAMING_STANDARD.

Bug: 260372860
Test: build pass
Change-Id: Ia38516acfd8979a020051062d6fb83805af45aab
2023-01-31 06:09:49 +00:00