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

Commit bc21bd58 authored by Jing Ji's avatar Jing Ji
Browse files

Move the binder proxy accounting to libbinder 

So the android.os.Debug#getBinderProxyObjectCount will include
the binder proxies from both of the Java and native layers.

Bug: 298314844
Test: dumpsys meminfo
Merged-In: Ifb57eeacc0a4b6ee6e164e83f6f843bfe36f2b15
Change-Id: I4fc8f48e0055544c32d9cf597d3573f0a48a13e4
parent 7039b4f7

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) {