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

Commit 3c74da9e authored by Yiwei Zhang's avatar Yiwei Zhang
Browse files

GpuStats: move the stats send at Activity launch off UI thread 

Prevously when App Activity launches, we send GpuStats on the main UI thread.
The async binder to send the stats only takes 30us. However, to get the
gpuservice, the sync binder takes around 1.5ms because of the massive binders
on CPU at Activity launch. Thus we just move the stats sending to a separate
thread off the main UI thread. Now it's just 50us cost on the main UI thread.

Bug: 136228448
Test: systrace and check
Change-Id: Iff47dbc6a26a9d62ad33e92f93c079ac74fb3ea2
parent 5e364566

libs/graphicsenv/GraphicsEnv.cpp

+15 −11
Original line number Diff line number Diff line
@@ -37,6 +37,7 @@ 


#include <memory>

#include <string>

#include <thread>



// TODO(b/37049319) Get this from a header once one exists

extern "C" {
@@ -163,6 +164,7 @@ void GraphicsEnv::setDriverPathAndSphalLibraries(const std::string path, 
void GraphicsEnv::hintActivityLaunch() {

    ATRACE_CALL();



    std::thread trySendGpuStatsThread([this]() {

        // If there's already graphics driver preloaded in the process, just send

        // the stats info to GpuStats directly through async binder.

        std::lock_guard<std::mutex> lock(mStatsLock);
@@ -174,6 +176,8 @@ void GraphicsEnv::hintActivityLaunch() { 
            mGpuStats.vkDriverToSend = false;

            sendGpuStatsLocked(GraphicsEnv::Api::API_VK, true, mGpuStats.vkDriverLoadingTime);

        }

    });

    trySendGpuStatsThread.detach();

}



void GraphicsEnv::setGpuStats(const std::string& driverPackageName,