Snap for 7069213 from 2cffede25703367337461fa633b19ebfb69837f2 to rvc-d2-release
Change-Id: I53f00160b5f3965c2c3ef25d65269c323273225f
diff --git a/common/hal/hidl_service/hidl_profiler.cc b/common/hal/hidl_service/hidl_profiler.cc
index 3c7e40d..e7e9ab9 100644
--- a/common/hal/hidl_service/hidl_profiler.cc
+++ b/common/hal/hidl_service/hidl_profiler.cc
@@ -17,6 +17,7 @@
//#define LOG_NDEBUG 0
#define LOG_TAG "GCH_HidlProfiler"
#include <log/log.h>
+#include <mutex>
#include <utility>
#include "hidl_profiler.h"
@@ -57,6 +58,8 @@
};
std::unique_ptr<HidlProfiler> gHidlProfiler = nullptr;
+// Mutex to make all API functions mutually exclusive.
+std::mutex api_mutex;
void StartNewConnector() {
if (gHidlProfiler != nullptr && gHidlProfiler->profiler != nullptr) {
@@ -80,6 +83,7 @@
} // anonymous namespace
std::unique_ptr<HidlProfilerItem> OnCameraOpen() {
+ std::lock_guard lock(api_mutex);
gHidlProfiler = std::make_unique<HidlProfiler>();
if (gHidlProfiler == nullptr || gHidlProfiler->profiler == nullptr) {
ALOGE("%s: gHidlProfiler or profiler is nullptr.", __FUNCTION__);
@@ -94,6 +98,7 @@
}
std::unique_ptr<HidlProfilerItem> OnCameraFlush() {
+ std::lock_guard lock(api_mutex);
EndConnector();
if (gHidlProfiler == nullptr) {
gHidlProfiler = std::make_unique<HidlProfiler>();
@@ -110,6 +115,7 @@
}
std::unique_ptr<HidlProfilerItem> OnCameraClose() {
+ std::lock_guard lock(api_mutex);
EndConnector();
if (gHidlProfiler == nullptr) {
gHidlProfiler = std::make_unique<HidlProfiler>();
@@ -126,6 +132,7 @@
}
std::unique_ptr<HidlProfilerItem> OnCameraStreamConfigure() {
+ std::lock_guard lock(api_mutex);
EndConnector();
if (gHidlProfiler == nullptr) {
gHidlProfiler = std::make_unique<HidlProfiler>();
@@ -146,6 +153,7 @@
}
void OnFirstFrameRequest() {
+ std::lock_guard lock(api_mutex);
EndConnector();
if (gHidlProfiler != nullptr && gHidlProfiler->profiler != nullptr) {
gHidlProfiler->profiler->Start(
@@ -156,6 +164,7 @@
}
void OnFirstFrameResult() {
+ std::lock_guard lock(api_mutex);
if (gHidlProfiler != nullptr && gHidlProfiler->profiler != nullptr) {
gHidlProfiler->profiler->End(
"First frame", google::camera_common::Profiler::kInvalidRequestId);