commit 48cca92c5e6301a09e7f829abcf7533e214f2d14
author Peter Kalauskas <peskal@google.com> Thu Feb 08 18:24:51 2024 +0000
committer Wei Wang <wvw@google.com> Fri Feb 09 00:28:51 2024 +0000
tree 851560b683f7b6e262f5a40a66931f8a4c45075e
parent ba180cc1c6ced287e491462d7a1fce8813b30123

Revert "Update SF to use createHintSessionWithConfig"

This reverts commit ba180cc1c6ced287e491462d7a1fce8813b30123.

Reason for revert: b/324164941, regression to perfetto_ft_systemui-missed_sf_frames-mean

Bug: 324484643
Bug: 324164941
Change-Id: I728ceba0bcad8f5e9a2f866e27e137e1395ae72f

services/surfaceflinger/DisplayHardware/PowerAdvisor.cpp

diff --git a/services/surfaceflinger/DisplayHardware/PowerAdvisor.cpp b/services/surfaceflinger/DisplayHardware/PowerAdvisor.cpp
index ee6d37b..a0c943b 100644
--- a/services/surfaceflinger/DisplayHardware/PowerAdvisor.cpp
+++ b/services/surfaceflinger/DisplayHardware/PowerAdvisor.cpp
@@ -48,7 +48,6 @@
 using aidl::android::hardware::power::Boost;
 using aidl::android::hardware::power::Mode;
 using aidl::android::hardware::power::SessionHint;
-using aidl::android::hardware::power::SessionTag;
 using aidl::android::hardware::power::WorkDuration;
 
 PowerAdvisor::~PowerAdvisor() = default;
@@ -207,12 +206,9 @@
 
 bool PowerAdvisor::ensurePowerHintSessionRunning() {
     if (mHintSession == nullptr && !mHintSessionThreadIds.empty() && usePowerHintSession()) {
-        auto ret =
-                getPowerHal().createHintSessionWithConfig(getpid(), static_cast<int32_t>(getuid()),
-                                                          mHintSessionThreadIds,
-                                                          mTargetDuration.ns(),
-                                                          SessionTag::SURFACEFLINGER,
-                                                          &mSessionConfig);
+        auto ret = getPowerHal().createHintSession(getpid(), static_cast<int32_t>(getuid()),
+                                                   mHintSessionThreadIds, mTargetDuration.ns());
+
         if (ret.isOk()) {
             mHintSession = ret.value();
         }