Merge "fix race in ProcessState::getThreadPoolMaxTotalThreadCount" into main am: f0063973

}

size_t ProcessState::getThreadPoolMaxTotalThreadCount() const {

    if (mThreadPoolStarted) {

    // Need to read `mKernelStartedThreads` before `mThreadPoolStarted` (with

    // non-relaxed memory ordering) to avoid a race like the following:

    //

    // thread A: if (mThreadPoolStarted) { // evaluates false

    // thread B: mThreadPoolStarted = true;

    // thread B: mKernelStartedThreads++;

    // thread A: size_t kernelStarted = mKernelStartedThreads;

    // thread A: LOG_ALWAYS_FATAL_IF(kernelStarted != 0, ...);

    size_t kernelStarted = mKernelStartedThreads;

    if (mThreadPoolStarted) {

        size_t max = mMaxThreads;

        size_t current = mCurrentThreads;

    // must not be initialized or maybe has poll thread setup, we

    // currently don't track this in libbinder

    size_t kernelStarted = mKernelStartedThreads;

    LOG_ALWAYS_FATAL_IF(kernelStarted != 0, "Expecting 0 kernel started threads but have %zu",

                        kernelStarted);

    return mCurrentThreads;

    Vector<handle_entry> mHandleToObject;

    bool mForked;

    bool mThreadPoolStarted;

    std::atomic_bool mThreadPoolStarted;

    volatile int32_t mThreadPoolSeq;

    CallRestriction mCallRestriction;