power-libperfmgr: Remove cpu_load_up relative boost to PID
Increasingly, cpu_load_up exists as a tool to boost stray expensive
frames rather than to increase the PID setpoint, so this change reflects
that and seeks to prevent power regressions in its use towards this goal
Bug: 261130508
Test: manual
Change-Id: I6608f92decbdfc59d92b4e5716a75b323c1d2c3e
diff --git a/power-libperfmgr/aidl/PowerHintSession.cpp b/power-libperfmgr/aidl/PowerHintSession.cpp
index 4debfce..81f2f90 100644
--- a/power-libperfmgr/aidl/PowerHintSession.cpp
+++ b/power-libperfmgr/aidl/PowerHintSession.cpp
@@ -338,15 +338,10 @@
return ndk::ScopedAStatus::fromExceptionCode(EX_ILLEGAL_STATE);
}
disableTemporaryBoost();
- // The amount we boost threads that have unexpected workloads
- // Consider adding this value to the powerhint.json and using that value directly
- constexpr int kRelativeBoost = 150;
std::shared_ptr<AdpfConfig> adpfConfig = HintManager::GetInstance()->GetAdpfProfile();
switch (hint) {
case SessionHint::CPU_LOAD_UP:
- mNextUclampMin.store(
- std::min(adpfConfig->mUclampMinHigh,
- static_cast<uint32_t>(mDescriptor->current_min + kRelativeBoost)));
+ mNextUclampMin.store(mDescriptor->current_min);
mBoostTimerHandler->updateTimer(mDescriptor->duration * 2);
setSessionUclampMin(adpfConfig->mUclampMinHigh);
break;