Camera: Removing cachedump call to miminimize delay during close.

- The cacheDump() call was added for better debugging by always dumping the state of the last camera session. Since LT/VT chipsets it is adding significant delay during camera close. Hence we are removing cachedump call during disconnect. https://partnerissuetracker.corp.google.com/issues/237992867 CRs-Fixed: 3228616 Change-Id: Ibba2a51f08c592bf15b046642101e11d890ed248 Signed-off-by: Pranav Vashi <[email protected]>
DerpFest-AOSP · Apr 1, 2024 · 0c31ac2 · 0c31ac2
1 parent cceaa8c
commit 0c31ac2
Showing 1 changed file with 0 additions and 8 deletions.
diff --git a/services/camera/libcameraservice/common/Camera2ClientBase.cpp b/services/camera/libcameraservice/common/Camera2ClientBase.cpp
@@ -272,14 +272,6 @@ binder::Status Camera2ClientBase<TClientBase>::disconnectImpl() {
 
     ALOGD("Camera %s: Shutting down", TClientBase::mCameraIdStr.c_str());
 
-    // Before detaching the device, cache the info from current open session.
-    // The disconnected check avoids duplication of info and also prevents
-    // deadlock while acquiring service lock in cacheDump.
-    if (!TClientBase::mDisconnected) {
-        ALOGD("Camera %s: start to cacheDump", TClientBase::mCameraIdStr.c_str());
-        Camera2ClientBase::getCameraService()->cacheDump();
-    }
-
     detachDevice();
 
     CameraService::BasicClient::disconnect();