commit e16ad28186952f323d7b099bba3e2c9fa6372272
author Xiang Wang <xwxw@google.com> Wed Jun 26 14:31:34 2024 -0700
committer Xiang Wang <xwxw@google.com> Thu Jun 27 17:08:59 2024 +0000
tree 304087345b49aa9e112f1942ec59d193a8608050
parent 1754db7dc35db7635e2bfd4f806f8319d25fe9d8

Skip binder call if it's the same target

Bug: 349632685
Flag: EXEMPT NDK
Test: atest PerformanceHintTest
Change-Id: I0b3bd63da15ec8690cd86806cbe86fce31b299bd

native/android/performance_hint.cpp

diff --git a/native/android/performance_hint.cpp b/native/android/performance_hint.cpp
index 7e5bef1..e91c7a9 100644
--- a/native/android/performance_hint.cpp
+++ b/native/android/performance_hint.cpp
@@ -244,6 +244,12 @@
         ALOGE("%s: targetDurationNanos must be positive", __FUNCTION__);
         return EINVAL;
     }
+    {
+        std::scoped_lock lock(sHintMutex);
+        if (mTargetDurationNanos == targetDurationNanos) {
+            return 0;
+        }
+    }
     ndk::ScopedAStatus ret = mHintSession->updateTargetWorkDuration(targetDurationNanos);
     if (!ret.isOk()) {
         ALOGE("%s: HintSession updateTargetWorkDuration failed: %s", __FUNCTION__,