From 8dd85e63a902508b646b6cfa127cb7162b3dc751 Mon Sep 17 00:00:00 2001 From: Subhajeet Muhuri Date: Tue, 21 Jan 2020 20:43:22 +0530 Subject: [PATCH] sm6250-common: power-libperfmgr: Remove ATRACE instances Change-Id: I50eaf5fd6c3763ab005ec965c3dfa173808313e3 --- power-libperfmgr/InteractionHandler.cpp | 6 ------ power-libperfmgr/Power.cpp | 4 ---- 2 files changed, 10 deletions(-) diff --git a/power-libperfmgr/InteractionHandler.cpp b/power-libperfmgr/InteractionHandler.cpp index 032a470..2116b4a 100644 --- a/power-libperfmgr/InteractionHandler.cpp +++ b/power-libperfmgr/InteractionHandler.cpp @@ -15,7 +15,6 @@ */ #define LOG_TAG "android.hardware.power@1.3-service.xiaomi_sm6250-libperfmgr" -#define ATRACE_TAG (ATRACE_TAG_POWER | ATRACE_TAG_HAL) #include #include @@ -105,7 +104,6 @@ void InteractionHandler::PerfLock() { if (!mHintManager->DoHint("INTERACTION")) { ALOGE("%s: do hint INTERACTION failed", __func__); } - ATRACE_INT("interaction_lock", 1); } void InteractionHandler::PerfRel() { @@ -113,7 +111,6 @@ void InteractionHandler::PerfRel() { if (!mHintManager->EndHint("INTERACTION")) { ALOGE("%s: end hint INTERACTION failed", __func__); } - ATRACE_INT("interaction_lock", 0); } size_t InteractionHandler::CalcTimespecDiffMs(struct timespec start, struct timespec end) { @@ -124,7 +121,6 @@ size_t InteractionHandler::CalcTimespecDiffMs(struct timespec start, struct time } void InteractionHandler::Acquire(int32_t duration) { - ATRACE_CALL(); std::lock_guard lk(mLock); if (mState == INTERACTION_STATE_UNINITIALIZED) { @@ -170,7 +166,6 @@ void InteractionHandler::Acquire(int32_t duration) { void InteractionHandler::Release() { std::lock_guard lk(mLock); if (mState == INTERACTION_STATE_WAITING) { - ATRACE_CALL(); PerfRel(); mState = INTERACTION_STATE_IDLE; } else { @@ -195,7 +190,6 @@ void InteractionHandler::WaitForIdle(int32_t wait_ms, int32_t timeout_ms) { ssize_t ret; struct pollfd pfd[2]; - ATRACE_CALL(); ALOGV("%s: wait:%d timeout:%d", __func__, wait_ms, timeout_ms); diff --git a/power-libperfmgr/Power.cpp b/power-libperfmgr/Power.cpp index 51037ba..7ee82d7 100644 --- a/power-libperfmgr/Power.cpp +++ b/power-libperfmgr/Power.cpp @@ -14,7 +14,6 @@ * limitations under the License. */ -#define ATRACE_TAG (ATRACE_TAG_POWER | ATRACE_TAG_HAL) #define LOG_TAG "android.hardware.power@1.3-service.xiaomi_sm6250-libperfmgr" #include @@ -100,7 +99,6 @@ Return Power::powerHint(PowerHint_1_0 hint, int32_t data) { if (!mReady) { return Void(); } - ATRACE_INT(android::hardware::power::V1_0::toString(hint).c_str(), data); ALOGD_IF(hint != PowerHint_1_0::INTERACTION, "%s: %d", android::hardware::power::V1_0::toString(hint).c_str(), static_cast(data)); switch (hint) { @@ -218,7 +216,6 @@ Return Power::powerHintAsync_1_2(PowerHint_1_2 hint, int32_t data) { return Void(); } - ATRACE_INT(android::hardware::power::V1_2::toString(hint).c_str(), data); ALOGD_IF(hint >= PowerHint_1_2::AUDIO_STREAMING, "%s: %d", android::hardware::power::V1_2::toString(hint).c_str(), static_cast(data)); @@ -255,7 +252,6 @@ Return Power::powerHintAsync_1_3(PowerHint_1_3 hint, int32_t data) { } if (hint == PowerHint_1_3::EXPENSIVE_RENDERING) { - ATRACE_INT(android::hardware::power::V1_3::toString(hint).c_str(), data); if (mSustainedPerfModeOn) { ALOGV("%s: ignoring due to other active perf hints", __func__); } else {