Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit cf7953f7 authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Revert "libbinder: double cost for incorrect wait config"" am: 1b5ab7cd am: 233a12d0 

Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/2562811



Change-Id: I5f1b33d8ed784f1dcd2dd3aae2b060eb0c0658e4
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 38d15544 233a12d0

libs/binder/IServiceManager.cpp

+4 −2
Original line number Diff line number Diff line
@@ -412,11 +412,13 @@ sp<IBinder> ServiceManagerShim::waitForService(const String16& name16) 
            // command, so we hang indefinitely.

            std::unique_lock<std::mutex> lock(waiter->mMutex);

            using std::literals::chrono_literals::operator""s;

            waiter->mCv.wait_for(lock, 2s, [&] { return waiter->mBinder != nullptr; });

            waiter->mCv.wait_for(lock, 1s, [&] {

                return waiter->mBinder != nullptr;

            });

            if (waiter->mBinder != nullptr) return waiter->mBinder;

        }



        ALOGW("Waited two seconds for %s (is service started? Number of threads started in the "

        ALOGW("Waited one second for %s (is service started? Number of threads started in the "

              "threadpool: %zu. Are binder threads started and available?)",

              name.c_str(), ProcessState::self()->getThreadPoolMaxTotalThreadCount());