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

Commit c0d95276 authored by Steven Moreland's avatar Steven Moreland Committed by Automerger Merge Worker
Browse files

Merge "libbinder: log when threads requested, not started" into main am: 38a4ab9f 

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



Change-Id: I3d26045f6cd031a755e59c032db4c938419ca24a
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 8c82acab 38a4ab9f

libs/binder/IPCThreadState.cpp

+1 −0
Original line number Diff line number Diff line
@@ -775,6 +775,7 @@ void IPCThreadState::joinThreadPool(bool isMain) 
{

    LOG_THREADPOOL("**** THREAD %p (PID %d) IS JOINING THE THREAD POOL\n", (void*)pthread_self(),

                   getpid());

    mProcess->checkExpectingThreadPoolStart();

    mProcess->mCurrentThreads++;

    mOut.writeInt32(isMain ? BC_ENTER_LOOPER : BC_REGISTER_LOOPER);

libs/binder/ProcessState.cpp

+15 −0
Original line number Diff line number Diff line
@@ -501,6 +501,21 @@ bool ProcessState::isThreadPoolStarted() const { 
    return mThreadPoolStarted;

}



void ProcessState::checkExpectingThreadPoolStart() const {

    if (mThreadPoolStarted) return;



    // this is also racey, but you should setup the threadpool in the main thread. If that is an

    // issue, we can check if we are the process leader, but haven't seen the issue in practice.

    size_t requestedThreads = mMaxThreads.load();



    // if it's manually set to the default, we do ignore it here...

    if (requestedThreads == DEFAULT_MAX_BINDER_THREADS) return;

    if (requestedThreads == 0) return;



    ALOGW("Thread pool configuration of size %zu requested, but startThreadPool was not called.",

          requestedThreads);

}



#define DRIVER_FEATURES_PATH "/dev/binderfs/features/"

bool ProcessState::isDriverFeatureEnabled(const DriverFeature feature) {

    // Use static variable to cache the results.

libs/binder/include/binder/ProcessState.h

+2 −0
Original line number Diff line number Diff line
@@ -141,6 +141,8 @@ public: 
private:

    static sp<ProcessState> init(const char* defaultDriver, bool requireDefault);



    void checkExpectingThreadPoolStart() const;



    static void onFork();

    static void parentPostFork();

    static void childPostFork();