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

Commit be399a18 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

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



Change-Id: I5551ef6d363de44b584bf0371c71546ddb10ccb7
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 869c3a98 631f93bf

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