MiPad4/android_device_xiaomi_sdm660-common
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

sdm660-common: power-libperfmgr: Properly handle sustained performance mode

Browse source 
* when tested with [1], even after exiting the app, spm was
   still active in powerhal thereby keeping max freqs limited
   and blocking boosts

 * enable/disable it properly and end all other hints before
   enabling, to fix this issue

 * while we're at it, don't handle max freq for anything other
   than spm in powerhint

[1] https://android.googlesource.com/platform/cts/+/refs/heads/master/hostsidetests/sustainedperf

Signed-off-by: Adithya R <gh0strider.2k18.reborn@gmail.com>
This commit is contained in:
Adithya R 2021-09-05 18:15:55 +05:30 committed by CakesTwix
parent e8f300c3f5
commit 4c33049a26
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
power-libperfmgr/aidl/Power.cpp
View file

@ -91,14 +91,14 @@ ndk::ScopedAStatus Power::setMode(Mode type, bool enabled) {
break; break;
case Mode::SUSTAINED_PERFORMANCE: case Mode::SUSTAINED_PERFORMANCE:
if (enabled) { if (enabled) {
endAllHints();
HintManager::GetInstance()->DoHint("SUSTAINED_PERFORMANCE"); HintManager::GetInstance()->DoHint("SUSTAINED_PERFORMANCE");
} else {
HintManager::GetInstance()->EndHint("SUSTAINED_PERFORMANCE");
} }
mSustainedPerfModeOn = true; mSustainedPerfModeOn = enabled;
break; break;
case Mode::LAUNCH: case Mode::LAUNCH:
if (mSustainedPerfModeOn) {
break;
}
[[fallthrough]]; [[fallthrough]];
case Mode::FIXED_PERFORMANCE: case Mode::FIXED_PERFORMANCE:
[[fallthrough]]; [[fallthrough]];