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

Commit 33adb57a authored by Jorim Jaggi's avatar Jorim Jaggi
Browse files

Don't double lock when resetting jank tests 

Fixes: 180875541
Test: adb shell dumpsys com.android.systemui reset
Change-Id: Ie656730bbbbb249e6798617cf236b93332e79eb6
parent 6844338e

libs/hwui/JankTracker.cpp

+0 −1
Original line number Diff line number Diff line
@@ -236,7 +236,6 @@ void JankTracker::dumpFrames(int fd) { 
}



void JankTracker::reset() {

    std::lock_guard lock(mDataMutex);

    mFrames.clear();

    mData->reset();

    (*mGlobalData)->reset();

libs/hwui/renderthread/RenderProxy.cpp

+4 −1
Original line number Diff line number Diff line
@@ -230,7 +230,10 @@ void RenderProxy::dumpProfileInfo(int fd, int dumpFlags) { 
}



void RenderProxy::resetProfileInfo() {

    mRenderThread.queue().runSync([=]() { mContext->resetFrameStats(); });

    mRenderThread.queue().runSync([=]() {

        std::lock_guard lock(mRenderThread.getJankDataMutex());

        mContext->resetFrameStats();

    });

}



uint32_t RenderProxy::frameTimePercentile(int percentile) {