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

Commit d5646aca authored by Alec Mouri's avatar Alec Mouri Committed by Automerger Merge Worker
Browse files

Merge "[GpuStats] Lazily-register statsd callbacks" into rvc-dev am: a13a1f96 

Change-Id: Ibdd25eecab43759d827f88fbb5c612896b75571c
parents 46f773d0 a13a1f96

services/gpuservice/gpustats/GpuStats.cpp

+16 −9
Original line number Diff line number Diff line
@@ -30,17 +30,12 @@ 


namespace android {



GpuStats::GpuStats() {

    AStatsManager_registerPullAtomCallback(android::util::GPU_STATS_GLOBAL_INFO,

                                           GpuStats::pullAtomCallback, nullptr, this);

    AStatsManager_registerPullAtomCallback(android::util::GPU_STATS_APP_INFO,

                                           GpuStats::pullAtomCallback, nullptr, this);

}



GpuStats::~GpuStats() {

    if (mStatsdRegistered) {

        AStatsManager_unregisterPullAtomCallback(android::util::GPU_STATS_GLOBAL_INFO);

        AStatsManager_unregisterPullAtomCallback(android::util::GPU_STATS_APP_INFO);

    }

}



static void addLoadingCount(GpuStatsInfo::Driver driver, bool isDriverLoaded,

                            GpuStatsGlobalInfo* const outGlobalInfo) {
@@ -97,6 +92,7 @@ void GpuStats::insertDriverStats(const std::string& driverPackageName, 
    ATRACE_CALL();



    std::lock_guard<std::mutex> lock(mLock);

    registerStatsdCallbacksIfNeeded();

    ALOGV("Received:\n"

          "\tdriverPackageName[%s]\n"

          "\tdriverVersionName[%s]\n"
@@ -150,6 +146,7 @@ void GpuStats::insertTargetStats(const std::string& appPackageName, 
    const std::string appStatsKey = appPackageName + std::to_string(driverVersionCode);



    std::lock_guard<std::mutex> lock(mLock);

    registerStatsdCallbacksIfNeeded();

    if (!mAppStats.count(appStatsKey)) {

        return;

    }
@@ -179,6 +176,16 @@ void GpuStats::interceptSystemDriverStatsLocked() { 
    mGlobalStats[0].glesVersion = property_get_int32("ro.opengles.version", 0);

}



void GpuStats::registerStatsdCallbacksIfNeeded() {

    if (!mStatsdRegistered) {

        AStatsManager_registerPullAtomCallback(android::util::GPU_STATS_GLOBAL_INFO,

                                               GpuStats::pullAtomCallback, nullptr, this);

        AStatsManager_registerPullAtomCallback(android::util::GPU_STATS_APP_INFO,

                                               GpuStats::pullAtomCallback, nullptr, this);

        mStatsdRegistered = true;

    }

}



void GpuStats::dump(const Vector<String16>& args, std::string* result) {

    ATRACE_CALL();

services/gpuservice/gpustats/include/gpustats/GpuStats.h

+4 −1
Original line number Diff line number Diff line
@@ -30,7 +30,6 @@ namespace android { 


class GpuStats {

public:

    GpuStats();

    ~GpuStats();



    // Insert new gpu driver stats into global stats and app stats.
@@ -66,12 +65,16 @@ private: 
    void dumpAppLocked(std::string* result);

    // Append cpuVulkanVersion and glesVersion to system driver stats

    void interceptSystemDriverStatsLocked();

    // Registers statsd callbacks if they have not already been registered

    void registerStatsdCallbacksIfNeeded();



    // Below limits the memory usage of GpuStats to be less than 10KB. This is

    // the preferred number for statsd while maintaining nice data quality.

    static const size_t MAX_NUM_APP_RECORDS = 100;

    // GpuStats access should be guarded by mLock.

    std::mutex mLock;

    // True if statsd callbacks have been registered.

    bool mStatsdRegistered = false;

    // Key is driver version code.

    std::unordered_map<uint64_t, GpuStatsGlobalInfo> mGlobalStats;

    // Key is <app package name>+<driver version code>.