See go/pixel-perf-amm-experiment for more info.
Bug: 365855872
Test: build/flash; set .Perf__amm_experiment=true, observe auto_migration_margins_enable value changing 1->0; set .Perf__amm_experiment="", observe the value changing 0->1
Flag: NONE controlled by Mendel instead
Change-Id: I77124ff55150075bb6d129c7c56ebed1a939ad59
Signed-off-by: Dmitry Skiba <dskiba@google.com>
This change introduces "is APF enabled" system properties, and
uses them as trigger conditions for APF nodes / pre-APF nodes.
By default APF nodes are active, and pre-APF nodes are inactive,
which maintains the situation before this change.
A future experiment will flip "is APF enabled" system properties,
deactivating APF nodes and activating pre-APF nodes. For more
info see go/pixel-perf-apf-experiment.
Bug: 365855872
Test: set vendor.powerhal.apf_enabled=false, vendor.powerhal.apf_disabled=true, observe auto_prefer_fit
procfs value staying "false" and instead preferred_idle_mask_mid
changing to 0xff during app launches
Flag: NONE powerhint change does not support flags
Change-Id: I45ba5311e46aa2c6b40705ff098889d47c6a2fa2
Signed-off-by: Dmitry Skiba <dskiba@google.com>
By default slow it down as auto margins can lead to faster ramping up in
low activity use cases.
Set it back to default in GAME mode to retain previous behavior. Set it
explicitly for DISPLAY_IDLE as a potential area for more tuning in the
future.
Bug: 315748906
Flag: EXEMPT powerhints are not compatible with flagging
Signed-off-by: Qais Yousef <qyousef@google.com>
Change-Id: I275545cc98714afd3efefb9f57e6bac55fcb6ea3
And reduce DownRateLimitUs to 500us as the system should be more
reactive now when something needs to rampup quickly.
Enable pertask iowait boost too as with auto dvfs headroom enabled all
tasks will start form 0 util_avg and this could impact iowait tasks
without this being enabled to help them migrate up to bigger cores.
Camera use cases should still use their own default DownRateLimitUs and
disable this feature to keep their dvfs_headroom tuning intact.
Bug: 315748906
Flag: EXEMPT powerhints are not compatible with flagging
Signed-off-by: Qais Yousef <qyousef@google.com>
Change-Id: Ifaee3631458e144270f41b66595c5e853546171c
we already saw some janks caused by longer runnable time.
so Enable TAPreferHighCap for first frame
Bug: 372168866
Flag: NONE powerhint do not support flag
Change-Id: I01f699abd1eaed5760789d6c0788cbe4791e2b13
Signed-off-by: Chungkai Mei <chungkai@google.com>
The node "/sys/kernel/vendor_mm/pa_kill/reclaim_kb" is write-only. So we need to add WriteOnly flag to prevent powerhal to read the node.
SELinux errors:
avc: denied { dac_read_search } for comm="android.hardwar" capability=2 scontext=u:r:hal_power_default:s0 tcontext=u:r:hal_power_default:s0 tclass=capability permissive=0
avc: denied { dac_override } for comm="android.hardwar" capability=1 scontext=u:r:hal_power_default:s0 tcontext=u:r:hal_power_default:s0 tclass=capability permissive=0
Bug: 350830411
Change-Id: I649e263a5371782184ce712fef3e8d42941806d7
Test: adb shell dumpsys android.hardware.power.IPower/default, check if any avc denied messages
Flag: EXEMPT config json change
These streams do not disable TA or FG prefer idle, so restore it.
Bug: 236868627
Flag: NONE powerhint change does not support flags
Test: build pass
Change-Id: Ia98132e94c6572165687cb4437492ab0669f9427
Enables a signal indicating that the app has janked as
detected by the ADPF API.
Flag: NONE align device tuning from P24.
Test: perf / power tests
Bug: 371094656
Change-Id: I19279e0f5a5fea59ffd0728128d6170f61e8a3e2
Signed-off-by: Chungkai Mei <chungkai@google.com>
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
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: I28d8023d3bc8852d6bcb771c0b2338a15f97ca37
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: I43f57177a7bd8dafc902063fd63d3bbb58ffff71
Let sys group tasks will run less in big core by default.
Bug: 328210236
Test: build pass
Flag: NONE powerhint change does not support flags
Change-Id: I9f71dc336792ad0b1add06a83d6261273313f9ef
Refer to ag/28864349
Bug: 354763942
Test: Build pass, Boot to home
Flag: EXEMPT resource file change only
Change-Id: I85c033e0fd309ff9b6fe8fb24f6c9e687e31f1c6
Port from ag/28607900
This CL defines CAMERA_MULTICAM_BOOST (with the same settings as the
existing CAMERA_ZOOMING_BOOST). Having both definitions is necessary to
be able to transition from zooming boost to multicam boost. Once the
multicam boost flag is fully rolled out, we can remove the
CAMERA_ZOOMING_BOOST.
Bug: 347829135
Test: adb logcat
Flag: EXEMPT resource file only change
Change-Id: I0a240e1b6c82b6b0e4577005c16b589f1865c154
sync with ag/28274083.
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: 356005553
Test: 1. adb logcat
2. gca_smoke.py test pass
Flag: EXEMPT resource only update
Change-Id: I5320d90072b92ebfb791ad50905667d764f7e3e8
Bug: 344685639
Test: Checked the trace file running on the device.
Flag: NONE powerhal doesn't have a flag yet
Change-Id: If4e4bc917252244c7460908f40d328a2e23182e1
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: 315748906
Test: build
Flag: NONE powerhint change does not support flags
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:61cf08366eb4e916beb1ad910754c8bc6c18bf14)
Merged-In: I03a83cbf7b0c6fdf1abd3c63ccb96a38350d17da
Change-Id: I03a83cbf7b0c6fdf1abd3c63ccb96a38350d17da
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: 340534339
Test: build
Flag: NONE powerhint change does not support flags
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:dad48beaeb9f9fce8acb67c06bfd022e86ce0e0d)
Merged-In: Ie9d20fe5ad6b423e8c26a36b7f01592040292ead
Change-Id: Ie9d20fe5ad6b423e8c26a36b7f01592040292ead
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: 315748906
Test: build
Flag: NONE powerhint change does not support flags
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:bb4df08c9ea617625d25cf12bc60c41c173d996b)
Merged-In: Ibd08cec303010e87057463f5aff737a5105e04b6
Change-Id: Ibd08cec303010e87057463f5aff737a5105e04b6
So that we can change the default values without impacting it.
Bug: 315748906powerhint: Explicitly set DownRateLimitUs for camera
So that we can change the default values without impacting it.
Bug: 315748906
Test: build
Flag: NONE powerhint change does not support flags
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:9084165d0d0f699f7cc3968dded4f71d4ec20a92)
Merged-In: Ic14d51f0b5b512f5557d904d682e9f934bbab87f
Change-Id: Ic14d51f0b5b512f5557d904d682e9f934bbab87f
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.
Refer to ag/28118299
Bug: 345601796
Test: Build pass, Boot to home
Flag: NONE powerhint change does not support flags
Change-Id: I7d6d0c5c23aaf7888a64ecbafde2275129fd22a1
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.
sync from ag/27765730
Bug: 341495529
Test: GCA
Change-Id: I8692df07c2a52115a497b7e71ccbe9638add1cc8
Faceauth is pinning to the mid-core for this device. Adjust the power hint to boost the appropriate cluster.
Bug: 329893113
Change-Id: Ifcb27e61752ef5ddb8484888adc77b894f5a5cf8
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: Ic7a87141616ea44c30cf88493d2d46dbee8da014
Bug: 322916612
Bug: 341968564
Test: power and performance test are updated to
b/341968564#comment38.
- Use newly tuned STREAMING_STANDARD setting.
Change-Id: Ida2e73b7d295bed53cff04cca0004fcaa78a696c
This CL enables PMU feature during photo capture. With the new settings, we can achieve up to 120ms saving in capture latency, while paying a penalty of 60mW. However, even with the added power, we are still below the F10 numbers (go/f10-camera-power-gtw).
Refer to ag/27238472
Bug: 340130143
Test: GCA photo capture
Change-Id: Ic1f4d5752a943a6898d92b0e44c78a85325be0a9