diff --git a/services/camera/libcameraservice/device3/Camera3Device.cpp b/services/camera/libcameraservice/device3/Camera3Device.cpp index 8a5cea961f..3409b32cec 100644 --- a/services/camera/libcameraservice/device3/Camera3Device.cpp +++ b/services/camera/libcameraservice/device3/Camera3Device.cpp @@ -4873,6 +4873,15 @@ status_t Camera3Device::RequestThread::prepareHalRequests() { captureRequest->mOutputStreams.size()); halRequest->output_buffers = outputBuffers->array(); std::set requestedPhysicalCameras; + + sp parent = mParent.promote(); + if (parent == NULL) { + // Should not happen, and nowhere to send errors to, so just log it + CLOGE("RequestThread: Parent is gone"); + return INVALID_OPERATION; + } + nsecs_t waitDuration = kBaseGetBufferWait + parent->getExpectedInFlightDuration(); + for (size_t j = 0; j < captureRequest->mOutputStreams.size(); j++) { sp outputStream = captureRequest->mOutputStreams.editItemAt(j); @@ -4893,6 +4902,7 @@ status_t Camera3Device::RequestThread::prepareHalRequests() { } res = outputStream->getBuffer(&outputBuffers->editItemAt(j), + waitDuration, captureRequest->mOutputSurfaces[j]); if (res != OK) { // Can't get output buffer from gralloc queue - this could be due to @@ -4919,13 +4929,6 @@ status_t Camera3Device::RequestThread::prepareHalRequests() { totalNumBuffers += halRequest->num_output_buffers; // Log request in the in-flight queue - sp parent = mParent.promote(); - if (parent == NULL) { - // Should not happen, and nowhere to send errors to, so just log it - CLOGE("RequestThread: Parent is gone"); - return INVALID_OPERATION; - } - // If this request list is for constrained high speed recording (not // preview), and the current request is not the last one in the batch, // do not send callback to the app. diff --git a/services/camera/libcameraservice/device3/Camera3Device.h b/services/camera/libcameraservice/device3/Camera3Device.h index ef3cbc4e9c..131d62f5d6 100644 --- a/services/camera/libcameraservice/device3/Camera3Device.h +++ b/services/camera/libcameraservice/device3/Camera3Device.h @@ -221,6 +221,7 @@ class Camera3Device : static const size_t kInFlightWarnLimitHighSpeed = 256; // batch size 32 * pipe depth 8 static const nsecs_t kDefaultExpectedDuration = 100000000; // 100 ms static const nsecs_t kMinInflightDuration = 5000000000; // 5 s + static const nsecs_t kBaseGetBufferWait = 3000000000; // 3 sec. // SCHED_FIFO priority for request submission thread in HFR mode static const int kRequestThreadPriority = 1; diff --git a/services/camera/libcameraservice/device3/Camera3Stream.cpp b/services/camera/libcameraservice/device3/Camera3Stream.cpp index 6030d15f50..6154301b71 100644 --- a/services/camera/libcameraservice/device3/Camera3Stream.cpp +++ b/services/camera/libcameraservice/device3/Camera3Stream.cpp @@ -576,6 +576,7 @@ status_t Camera3Stream::tearDown() { } status_t Camera3Stream::getBuffer(camera3_stream_buffer *buffer, + nsecs_t waitBufferTimeout, const std::vector& surface_ids) { ATRACE_CALL(); Mutex::Autolock l(mLock); @@ -593,13 +594,16 @@ status_t Camera3Stream::getBuffer(camera3_stream_buffer *buffer, ALOGV("%s: Already dequeued max output buffers (%d), wait for next returned one.", __FUNCTION__, camera3_stream::max_buffers); nsecs_t waitStart = systemTime(SYSTEM_TIME_MONOTONIC); - res = mOutputBufferReturnedSignal.waitRelative(mLock, kWaitForBufferDuration); + if (waitBufferTimeout < kWaitForBufferDuration) { + waitBufferTimeout = kWaitForBufferDuration; + } + res = mOutputBufferReturnedSignal.waitRelative(mLock, waitBufferTimeout); nsecs_t waitEnd = systemTime(SYSTEM_TIME_MONOTONIC); mBufferLimitLatency.add(waitStart, waitEnd); if (res != OK) { if (res == TIMED_OUT) { ALOGE("%s: wait for output buffer return timed out after %lldms (max_buffers %d)", - __FUNCTION__, kWaitForBufferDuration / 1000000LL, + __FUNCTION__, waitBufferTimeout / 1000000LL, camera3_stream::max_buffers); } return res; diff --git a/services/camera/libcameraservice/device3/Camera3Stream.h b/services/camera/libcameraservice/device3/Camera3Stream.h index 4ddcf1ab85..1af939c0a0 100644 --- a/services/camera/libcameraservice/device3/Camera3Stream.h +++ b/services/camera/libcameraservice/device3/Camera3Stream.h @@ -311,6 +311,7 @@ class Camera3Stream : * */ status_t getBuffer(camera3_stream_buffer *buffer, + nsecs_t waitBufferTimeout, const std::vector& surface_ids = std::vector()); /** diff --git a/services/camera/libcameraservice/device3/Camera3StreamInterface.h b/services/camera/libcameraservice/device3/Camera3StreamInterface.h index 9ed7184203..2dde1c3258 100644 --- a/services/camera/libcameraservice/device3/Camera3StreamInterface.h +++ b/services/camera/libcameraservice/device3/Camera3StreamInterface.h @@ -237,6 +237,7 @@ class Camera3StreamInterface : public virtual RefBase { * */ virtual status_t getBuffer(camera3_stream_buffer *buffer, + nsecs_t waitBufferTimeout, const std::vector& surface_ids = std::vector()) = 0; /**