commit 852c381dbd1a9f86e6759201b50ac8847a1c3e3c
author Eino-Ville Talvala <etalvala@google.com> Mon Sep 24 09:46:53 2012 -0700
committer Eino-Ville Talvala <etalvala@google.com> Tue Sep 25 10:35:28 2012 -0700
tree 5e88063cfbedff931e367ded1286ad12d5c78a15
parent 67d8ac3a8b49b114366684cd6310493423581056

Camera2: Debug and logging fixes.

- Fix LOG_TAGs to work with standard logcat filtering.
- Add ATRACE to Camera2Device
- Shorten thread names to fit in 16 characters
- Remove some ATRACEs to avoid flooding trace unnecessarily

Bug: 7220520
Change-Id: Ief8d677b5c8bb35d94e0f21b11b86505e83169ea

services/camera/libcameraservice/Camera2Client.cpp

diff --git a/services/camera/libcameraservice/Camera2Client.cpp b/services/camera/libcameraservice/Camera2Client.cpp
index 495feda..a83977f 100644
--- a/services/camera/libcameraservice/Camera2Client.cpp
+++ b/services/camera/libcameraservice/Camera2Client.cpp
@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-#define LOG_TAG "Camera2Client"
+#define LOG_TAG "Camera2"
 #define ATRACE_TAG ATRACE_TAG_CAMERA
 //#define LOG_NDEBUG 0
 
@@ -102,27 +102,27 @@
     String8 threadName;
 
     mFrameProcessor = new FrameProcessor(this);
-    threadName = String8::format("Camera2Client[%d]::FrameProcessor",
+    threadName = String8::format("C2-%d-FrameProc",
             mCameraId);
     mFrameProcessor->run(threadName.string());
 
     mCaptureSequencer = new CaptureSequencer(this);
-    threadName = String8::format("Camera2Client[%d]::CaptureSequencer",
+    threadName = String8::format("C2-%d-CaptureSeq",
             mCameraId);
     mCaptureSequencer->run(threadName.string());
 
     mJpegProcessor = new JpegProcessor(this, mCaptureSequencer);
-    threadName = String8::format("Camera2Client[%d]::JpegProcessor",
+    threadName = String8::format("C2-%d-JpegProc",
             mCameraId);
     mJpegProcessor->run(threadName.string());
 
     mZslProcessor = new ZslProcessor(this, mCaptureSequencer);
-    threadName = String8::format("Camera2Client[%d]::ZslProcessor",
+    threadName = String8::format("C2-%d-ZslProc",
             mCameraId);
     mZslProcessor->run(threadName.string());
 
     mCallbackProcessor = new CallbackProcessor(this);
-    threadName = String8::format("Camera2Client[%d]::CallbackProcessor",
+    threadName = String8::format("C2-%d-CallbkProc",
             mCameraId);
     mCallbackProcessor->run(threadName.string());