Commit graph

58 commits

Author SHA1 Message Date
Rick Yiu
d46e930521 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: I104bdf8952da66346ecf73ed02245b8bc4be3ee8
2023-09-19 03:00:46 +00:00
Treehugger Robot
9c9700c239 Merge "disable vendor idle balancer" into udc-qpr-dev 2023-07-12 14:48:38 +00:00
Chungkai Mei
a135ce6a40 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
3e74bb58d8 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 T6pro with emul_temp
Change-Id: I82d84e1e085b1a22284aa913606b40f219583210
2023-06-15 15:42:33 -07:00
paulsu
677208c388 Remove UgBgGroupThrottle node to keep ug_bg_group_throttle 308 by default am: 2cd8ff3d6e am: 6516401c66
Original change: https://googleplex-android-review.googlesource.com/c/device/google/tangorpro/+/23680144

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

Bug: 269523712
Test: boot and run uibench
Change-Id: I106a4a2322a93147205e079c9461619967a33917
Signed-off-by: Ziyi Cui <ziyic@google.com>
2023-06-13 17:35:15 +00:00
paulsu
16d86f57d6 Create group throttle powerhint
Test: Local test
Bug: 280258727
Change-Id: Id481cc4434e5769cf227924eab8530a2e63841b9
Signed-off-by: paulsu <paulsu@google.com>
2023-06-08 11:15:52 +00:00
Rick Yiu
33750f3a2a 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
764bacd20a 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: I4223b3984003369e8fc7b2f0eac03bbe68569d6c
2023-05-10 10:26:35 +00:00
Mao Lee
87930f7a54 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: I8ce33dc7d749de4c25085ae845f2c47b50678211
2023-05-08 03:25:21 +00:00
Qais Yousef
6fca96663e Merge "powerhint.json: Update uclamp_min hint to account for margin" into udc-dev 2023-04-25 21:27:51 +00:00
Jimmy Shiu
826c53b945 powerhint: Boost GPU(302000) by 1st frame
Take CPU_LOAD_RESET hint to boost the GPU freq to 302000

Bug: 243025173
Test: Manual
Change-Id: I57da44fc7e7c11e84241cccf81f83b4ce22768fa
2023-04-12 22:08:54 +08:00
Qais Yousef
7a77f284ec 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: I1e6889ebea9877cb23708722a73e97177e6be185
2023-04-06 16:38:25 +00:00
Jimmy Shiu
e12a1c7cf9 Merge "powerhint: Boost GPU(302000) by 1st frame" into udc-dev 2023-03-07 01:13:53 +00:00
Jimmy Shiu
f55c796284 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: I53fabad1b16c14da7c0883bb4858b8622781b609
2023-03-07 01:15:02 +08:00
pointerkung
34b8126ebc Camera powerhint update
Sync PMU and EM_Profile tuning changes (ag/20028064, ag/20075269 and ag/20084677) for 30fps power and performance optimization.

Bug: 265749383
Test: Build pass, GCA
Change-Id: Id45edbf77de1435559cb1171e56bee9c5a2e80b7
2023-03-03 11:10:52 +08:00
sashwinbalaji
48f6675481 Thermal: Remove Camera Powerhint am: c6737051ac am: 78a4033590
Original change: https://googleplex-android-review.googlesource.com/c/device/google/tangorpro/+/20680256

Change-Id: Ia13e807c266d78b023cec58b01399917a87d2328
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-01-17 16:20:41 +00:00
sashwinbalaji
c6737051ac Thermal: Remove Camera Powerhint
Removing camera powerhint for verification

Bug: 206893843
Test: tested locally
Change-Id: I0e10d03cd6ee477620fc4c5075cbe07613d89a58
2023-01-17 10:57:53 +00:00
Luke Chang
12b6dd337e powerhint: add CPU_LOAD_RESET rule
Set minimum mif freq for 1st frame to avoid memlat can't reflect in
time.

Test: Build and check 1st frame boost
Bug: 259275437
Bug: 263383561

Change-Id: I495e36632aa1550021e9c2d8417acbd12fa90ea3
Signed-off-by: Luke Chang <lukechang@google.com>
2023-01-08 16:47:51 +00:00
sashwinbalaji
b137028ac1 thermal: Remove face unlock powerhint
- Remove powerhint as face auth not supported

Bug: 263690696
Test: Update local powerhint

Change-Id: I2160b2ef4b4136e2b2812a3051e958e837c1ea65
2022-12-26 17:39:45 +08:00
sashwinbalaji
28d3ef8f26 thermal: Update powerhint to use VIRTUAL-SKIN-HINT
Replace VIRTUAL-SKIN with VIRTUAL-SKIN-HINT

Bug: 262471610
Test: tested locally by emul_temp
Change-Id: I83f75e5f4b97bf47ac612c862063e8db5bde886f
2022-12-14 14:03:10 +08:00
Chungkai Mei
7661bc43df Remove UClampThreshold hint
the file node was removed so also remove these
dummy setting

Bug: 260806701
Change-Id: I23044f7f32cf4b2b2cc93205ec5751525f05ee7f
Signed-off-by: Chungkai Mei <chungkai@google.com>
2022-12-08 09:44:24 +00:00
TreeHugger Robot
cfc3593d31 Merge "Change the default cpuset of camera daemon" into tm-qpr-dev 2022-11-21 10:19:18 +00:00
TreeHugger Robot
2b3cb2a7d5 Merge "Don't set the min CPU frequency to max frequency for CAMERA_SHOT" into tm-qpr-dev 2022-11-18 07:32:02 +00:00
Chih-ren Yu (Swim)
8e4eccaad0 Don't set the min CPU frequency to max frequency for CAMERA_SHOT
Bug: 258442520
Test:
1. Camera check list
2. 3rd party APP snapshot, GCA live video shot
Change-Id: Ic8cf6129b5b549619f9a925a6b0e23992e23f537
2022-11-18 03:32:16 +00:00
Rick Yiu
a11b655e79 Change the default cpuset of camera daemon
Changing cpuset itself takes time, so we should reduce the cpuset
change as possible. Since all camera streaming mode change to the
same cpuset value, we could make it default setting.

Bug: 240889279
Test: build pass
Change-Id: I4ecce69a254bf30b128b25ab59b1b69a02bfb18d
2022-11-17 03:07:37 +00:00
Jimmy Shiu
7567ae2d98 powerhint: increase boost cap for large UI
Large screen have more elements to be drew and current ADPF boost cap is
not high enough.
Raise the uclamp.min Cap from BIG 1.42GHz to BIG 1.58GHz
Raise the uclamp.min Init from MID 553MHz to MID 696GHz or BIG 500MHz
Also remove ADPF 120/90FPS config since the display only supports 60FPS

Bug: 257170078
Test: PtsUiBench & Calendar
Change-Id: I6865dd7d1237a687f62400a9720624821f342402
2022-11-16 09:36:06 +08:00
Kyle Lin
a1e87a3f5c powerhint: relax the cap when camera exit
Camera hints was canceled at activityStop and applied the rule for
a while after LAUNCH hint, exten the LAUNCH HINT to fix it.

Bug: 255927500
Test: build and launch/exit camera
Change-Id: I86b19467f6038a7fde0935bf79f0a0e6bc76523b
2022-11-14 03:38:49 +00:00
Rick Yiu
6ddc5e88b1 Limit camera daemon cpuset to little cores when app launch
When exiting camera app, camera daemon is still busy in background,
which may affect the launch animation of launcher. So, limit the cpuset
of camera daemon to little cores for 1 second when there is app launch
hint to improve it. This only happens when camera is running. Also bypass this behavior when video recording is active.

Bug: 240889279
Test: camera daemon cpuset changed as expected
Change-Id: I130b1e4718f8e5ef47ad761a2a73d86697d0f8bc
2022-11-14 03:38:38 +00:00
Chungkai Mei
954576dbbd powerhint: synchronize and remove unused hints
Bug: 251357480
Test: Build pass
Test: pts-tradefed run commandAndExit pts -m PtsPerformanceTestCases -t com.google.android.perf.pts.PowerHintJsonVerifyTest
Signed-off-by: Chungkai Mei <chungkai@google.com>
Change-Id: Ia8cb9a2f1a7ee51ff5ea0fd86e5823036095ee67
2022-10-13 07:06:22 +00:00
Chung-Kai (Michael) Mei
0b7dcca4ad Merge "Update camera and face unlock related powerhint" into tm-qpr-dev 2022-10-07 04:51:37 +00:00
Chungkai Mei
fc75279aad Update camera and face unlock related powerhint
Boost big cpu cluster when face unlock and relax
CPU minimum frequency boost for LAUNCH_CAMERA

Bug: 248960565
Change-Id: I075b7f4447327be4ef601c531b43473f79dbc74a
2022-10-07 03:00:02 +00:00
Jay Chien
78e8b81514 [T6] Relax CPU minimum frequency boost for LAUNCH_CAMERA
Synchronizing solution from L10: ag/2002817

Bug: 251027883
Test: CTS testCameraLaunch, GCA
Change-Id: I4a3956d3be18e353bf883af79fafeec07e36cda9
2022-10-05 06:51:32 +00:00
Richard Chang
e93b23905f Merge "powerhint: clean up unused hints in T6p" into tm-qpr-dev 2022-09-19 05:12:37 +00:00
Richard Chang
54bda0f2d3 powerhint: clean up unused hints in T6p
LAUNCH_120FPS is removed but some actions still use it.

Bug: 246738037
Test: pts-tradefed run commandAndExit pts -m PtsPerformanceTestCases -t
com.google.android.perf.pts.PowerHintJsonVerifyTest

Change-Id: I3ab91cace52c617adf0ccadb95098de697efc5c0
2022-09-16 08:20:38 +00:00
TeYuan Wang
e657950726 Remove frequency boost of big cores for FACE_UNLOCK
Bug: 245619770
Test: build
Change-Id: I31b68a52e80c924bbecaa1b18a15c9ff4b2335ed
2022-09-15 23:38:11 +08:00
sashwinbalaji
e9f6f92eb6 thermal: update control temp in powerhint
1) Change the default trip_point_1_temp and trip_point_2_temp in accordance to new config provided by hardware (b/235166018#comment5).
2) Removing AUR nodes

Bug: 246546076
Test: Used local build to verify the trip_point_1_temp & trip_point_2_temp when VIRTUAL_SKIN
severity is changed on updating quiet_therm temp.

Change-Id: I240ba1b77706b6ce8991285dfa0ea6deae25f15b
2022-09-14 15:21:11 +08:00
Siddharth Kapoor
001814c4ac Replace GPUMinFreq 151000 with 202000
Bug: 240479599
Change-Id: If7a5970b4f6d6049fc0aee4fc7caeb99ccbbdf21
2022-09-05 22:22:16 -07:00
TreeHugger Robot
c305d4a222 Merge "Config dex2oat according to thermal status" into tm-qpr-dev 2022-09-02 06:04:31 +00:00
Rick Yiu
3ed689ec0d Merge "Remove RestrictedCpuset hint" into tm-qpr-dev 2022-09-02 04:59:42 +00:00
Rick Yiu
60f3c1e73e Config dex2oat according to thermal status
Config dex2oat cpuset and thread count according to thermal status.

LIGHT cpuset = 0,1,2,3,4,5 thread = 6
MODERATE cpuset = 0,1,2,3 thread = 4
SEVERE cpuset = 0,1,2,3 thread = 4
CRITICAL cpuset = 0,1,2,3 thread = 2
EMERGENCY cpuset = 0,1,2,3 thread = 1

Bug: 242948832
Test: build pass
Change-Id: If352833f9fe0f2593f5e98648587e2a99d5669c5
2022-09-01 08:37:24 +00:00
Rick Yiu
de3d6dd287 Remove RestrictedCpuset hint
With systemui now changed its cpuset in early stage, we no longer
need RestrictedCpuset hint as workaround.

Bug: 242775562
Test: build pass
Change-Id: I17603010b7070594f703e04b09cfb3a2a37ecac5
2022-08-30 06:48:24 +00:00
jimmyshiu
ba8ccf33f5 Enable ADPF CPU hints for SF
Allow these devices to send CPU hints to PowerHAL from SF to reduce
drops and save power.

Bug: b/241152899
Bug: b/195990840

Test: manual
Change-Id: I06a38eeade1aea7cb12bef079b555e6e34d2581c
(Port from commit 8a419466b86ab07a25fc18d7d6a2a9ee154f6839)
2022-08-29 12:22:09 +00:00
chungkai
987953c22c powerhal: update powerhint
Test: Build pass
Bug: 236819367
Signed-off-by: chungkai <chungkai@google.com>
Change-Id: I407d64f6e0a1dad86db08b70059faae3eb78b5e8
2022-07-13 02:22:29 +00:00
sashwinbalaji
e0e6499f0e thermal: update control temp in powerhint
1) Change the default trip_point_1_temp and trip_point_2_temp in accordance to new config provided by hardware.
2) Removing AUR nodes

Bug: 233329602
Test: Used local build to verify the trip_point_1_temp & trip_point_2_temp when VIRTUAL_SKIN
severity is changed on updating quiet_therm temp.

Change-Id: I179991c60b27edb6df518f387cbea1043548ea8d
2022-06-28 10:39:32 +00:00
chungkai
c2afe0fbf0 powerhal: correct the name of powertint "ReducePreferIdle"
Test: Build pass
Bug: 235887477
Signed-off-by: chungkai <chungkai@google.com>
Change-Id: I41d78202839d12aa0cba176f1e1b6233085aed9d
2022-06-22 09:40:53 +00:00
Jonglin Lee
68323f2e75 tangorpro: sync latest powerhint
Sync latest powerhint to unblock boot.

Bug: 235850212
Test: Boot to home
Signed-off-by: Jonglin Lee <jonglin@google.com>
Change-Id: I0369570176d962411ae1bc2b40396952e8ac205f
2022-06-13 15:40:19 -07:00
chungkai
03ce73f12f powerhal: sync latest powerhint
we need disable uclamp_threshold and reduce_prefer_idle, boost FG group when apps launch and remove FPS based hints for TA boost.

Bug: 233423618
Test: build pass
Signed-off-by: chungkai <chungkai@google.com>
Change-Id: I3fe9318d46426c1e5ef7defe85126501313d419f
2022-05-24 02:56:09 +00:00
Wei Wang
96e63735dc Merge "ADPF: remove UclampMin_Granularity" into tm-d1-dev 2022-05-16 16:33:41 +00:00