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

Commit 34d73860 authored by Jing Ji's avatar Jing Ji Committed by Automerger Merge Worker
Browse files

Merge "Move the binder proxy accounting to libbinder" into main am: 8fa25802... 

Merge "Move the binder proxy accounting to libbinder" into main am: 8fa25802 am: 3d2b2c63 am: 631f93bf am: be399a18

Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/2757285



Change-Id: Ib4c95ab2f7a98e21734652bb99e0fbc333fd6b13
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 77a947f0 be399a18

core/jni/android_util_Binder.cpp

+2 −19
Original line number Diff line number Diff line
@@ -157,12 +157,8 @@ static struct thread_dispatch_offsets_t 
// ****************************************************************************

// ****************************************************************************



static constexpr int32_t PROXY_WARN_INTERVAL = 5000;

static constexpr uint32_t GC_INTERVAL = 1000;



static std::atomic<uint32_t> gNumProxies(0);

static std::atomic<uint32_t> gProxiesWarned(0);



// Number of GlobalRefs held by JavaBBinders.

static std::atomic<uint32_t> gNumLocalRefsCreated(0);

static std::atomic<uint32_t> gNumLocalRefsDeleted(0);
@@ -842,19 +838,7 @@ jobject javaObjectForIBinder(JNIEnv* env, const sp<IBinder>& val) 
        return NULL;

    }

    BinderProxyNativeData* actualNativeData = getBPNativeData(env, object);

    if (actualNativeData == nativeData) {

        // Created a new Proxy

        uint32_t numProxies = gNumProxies.fetch_add(1, std::memory_order_relaxed);

        uint32_t numLastWarned = gProxiesWarned.load(std::memory_order_relaxed);

        if (numProxies >= numLastWarned + PROXY_WARN_INTERVAL) {

            // Multiple threads can get here, make sure only one of them gets to

            // update the warn counter.

            if (gProxiesWarned.compare_exchange_strong(numLastWarned,

                        numLastWarned + PROXY_WARN_INTERVAL, std::memory_order_relaxed)) {

                ALOGW("Unexpectedly many live BinderProxies: %d\n", numProxies);

            }

        }

    } else {

    if (actualNativeData != nativeData) {

        delete nativeData;

    }
@@ -1209,7 +1193,7 @@ jint android_os_Debug_getLocalObjectCount(JNIEnv* env, jobject clazz) 


jint android_os_Debug_getProxyObjectCount(JNIEnv* env, jobject clazz)

{

    return gNumProxies.load();

    return BpBinder::getBinderProxyCount();

}



jint android_os_Debug_getDeathObjectCount(JNIEnv* env, jobject clazz)
@@ -1494,7 +1478,6 @@ static void BinderProxy_destroy(void* rawNativeData) 
            nativeData->mObject.get(), nativeData->mOrgue.get());

    delete nativeData;

    IPCThreadState::self()->flushCommands();

    --gNumProxies;

}



JNIEXPORT jlong JNICALL android_os_BinderProxy_getNativeFinalizer(JNIEnv*, jclass) {