Snap for 6606167 from 2e2461eab59b9c93a6eb197aa0e30eedbc98a6e2 to rvc-release (a9bc5121) · Commits · e / devices / smdk4412 / android_frameworks_av

camera/CaptureResult.cpp

+6 −0

Original line number	Diff line number	Diff line
		@@ -49,6 +49,9 @@ status_t CaptureResultExtras::readFromParcel(const android::Parcel *parcel) {
		}
		errorPhysicalCameraId = cameraId;
		}
		parcel->readInt64(&lastCompletedRegularFrameNumber);
		parcel->readInt64(&lastCompletedReprocessFrameNumber);
		parcel->readInt64(&lastCompletedZslFrameNumber);

		return OK;
		}
		@@ -76,6 +79,9 @@ status_t CaptureResultExtras::writeToParcel(android::Parcel *parcel) const {
		} else {
		parcel->writeBool(false);
		}
		parcel->writeInt64(lastCompletedRegularFrameNumber);
		parcel->writeInt64(lastCompletedReprocessFrameNumber);
		parcel->writeInt64(lastCompletedZslFrameNumber);

		return OK;
		}

camera/include/camera/CaptureResult.h

+32 −1

Original line number	Diff line number	Diff line
		@@ -76,6 +76,34 @@ struct CaptureResultExtras : public android::Parcelable {
		*/
		String16 errorPhysicalCameraId;

		// The last completed frame numbers shouldn't be checked in onResultReceived() and notifyError()
		// because the output buffers could be arriving after onResultReceived() and
		// notifyError(). Given this constraint, we check it for each
		// onCaptureStarted, and if there is no further onCaptureStarted(),
		// check for onDeviceIdle() to clear out all pending frame numbers.

		/**
		* The latest regular request frameNumber for which all buffers and capture result have been
		* returned or notified as an BUFFER_ERROR/RESULT_ERROR/REQUEST_ERROR. -1 if
		* none has completed.
		*/
		int64_t lastCompletedRegularFrameNumber;

		/**
		* The latest reprocess request frameNumber for which all buffers and capture result have been
		* returned or notified as an BUFFER_ERROR/RESULT_ERROR/REQUEST_ERROR. -1 if
		* none has completed.
		*/
		int64_t lastCompletedReprocessFrameNumber;

		/**
		* The latest Zsl request frameNumber for which all buffers and capture result have been
		* returned or notified as an BUFFER_ERROR/RESULT_ERROR/REQUEST_ERROR. -1 if
		* none has completed.
		*/
		int64_t lastCompletedZslFrameNumber;


		/**
		* Constructor initializes object as invalid by setting requestId to be -1.
		*/
		@@ -87,7 +115,10 @@ struct CaptureResultExtras : public android::Parcelable {
		frameNumber(0),
		partialResultCount(0),
		errorStreamId(-1),
		errorPhysicalCameraId() {
		errorPhysicalCameraId(),
		lastCompletedRegularFrameNumber(-1),
		lastCompletedReprocessFrameNumber(-1),
		lastCompletedZslFrameNumber(-1) {
		}

		/**

camera/ndk/impl/ACameraDevice.cpp

+82 −35

Original line number	Diff line number	Diff line
		@@ -1336,19 +1336,19 @@ CameraDevice::checkRepeatingSequenceCompleteLocked(
		void
		CameraDevice::checkAndFireSequenceCompleteLocked() {
		int64_t completedFrameNumber = mFrameNumberTracker.getCompletedFrameNumber();
		//std::map<int, int64_t> mSequenceLastFrameNumberMap;
		auto it = mSequenceLastFrameNumberMap.begin();
		while (it != mSequenceLastFrameNumberMap.end()) {
		int sequenceId = it->first;
		int64_t lastFrameNumber = it->second;
		bool seqCompleted = false;
		int64_t lastFrameNumber = it->second.lastFrameNumber;
		bool hasCallback = true;

		if (mRemote == nullptr) {
		ALOGW("Camera %s closed while checking sequence complete", getId());
		return;
		}

		ALOGV("%s: seq %d's last frame number %" PRId64 ", completed %" PRId64,
		__FUNCTION__, sequenceId, lastFrameNumber, completedFrameNumber);
		if (!it->second.isSequenceCompleted) {
		// Check if there is callback for this sequence
		// This should not happen because we always register callback (with nullptr inside)
		if (mSequenceCallbackMap.count(sequenceId) == 0) {
		@@ -1357,16 +1357,14 @@ CameraDevice::checkAndFireSequenceCompleteLocked() {
		}

		if (lastFrameNumber <= completedFrameNumber) {
		ALOGV("seq %d reached last frame %" PRId64 ", completed %" PRId64,
		sequenceId, lastFrameNumber, completedFrameNumber);
		seqCompleted = true;
		ALOGV("Mark sequenceId %d as sequence completed", sequenceId);
		it->second.isSequenceCompleted = true;
		}

		if (seqCompleted && hasCallback) {
		// remove callback holder from callback map
		if (it->second.isSequenceCompleted && hasCallback) {
		auto cbIt = mSequenceCallbackMap.find(sequenceId);
		CallbackHolder cbh = cbIt->second;
		mSequenceCallbackMap.erase(cbIt);

		// send seq complete callback
		sp<AMessage> msg = new AMessage(kWhatCaptureSeqEnd, mHandler);
		msg->setPointer(kContextKey, cbh.mContext);
		@@ -1382,10 +1380,53 @@ CameraDevice::checkAndFireSequenceCompleteLocked() {
		cbh.mSession.clear();
		postSessionMsgAndCleanup(msg);
		}
		}

		// No need to track sequence complete if there is no callback registered
		if (seqCompleted \|\| !hasCallback) {
		if (it->second.isSequenceCompleted && it->second.isInflightCompleted) {
		if (mSequenceCallbackMap.find(sequenceId) != mSequenceCallbackMap.end()) {
		mSequenceCallbackMap.erase(sequenceId);
		}
		it = mSequenceLastFrameNumberMap.erase(it);
		ALOGV("%s: Remove holder for sequenceId %d", __FUNCTION__, sequenceId);
		} else {
		++it;
		}
		}
		}

		void
		CameraDevice::removeCompletedCallbackHolderLocked(int64_t lastCompletedRegularFrameNumber) {
		auto it = mSequenceLastFrameNumberMap.begin();
		while (it != mSequenceLastFrameNumberMap.end()) {
		int sequenceId = it->first;
		int64_t lastFrameNumber = it->second.lastFrameNumber;

		if (mRemote == nullptr) {
		ALOGW("Camera %s closed while checking sequence complete", getId());
		return;
		}

		ALOGV("%s: seq %d's last frame number %" PRId64
		", completed inflight frame number %" PRId64,
		__FUNCTION__, sequenceId, lastFrameNumber,
		lastCompletedRegularFrameNumber);
		if (lastFrameNumber <= lastCompletedRegularFrameNumber) {
		if (it->second.isSequenceCompleted) {
		// Check if there is callback for this sequence
		// This should not happen because we always register callback (with nullptr inside)
		if (mSequenceCallbackMap.count(sequenceId) == 0) {
		ALOGW("No callback found for sequenceId %d", sequenceId);
		} else {
		mSequenceCallbackMap.erase(sequenceId);
		}

		it = mSequenceLastFrameNumberMap.erase(it);
		ALOGV("%s: Remove holder for sequenceId %d", __FUNCTION__, sequenceId);
		} else {
		ALOGV("Mark sequenceId %d as inflight completed", sequenceId);
		it->second.isInflightCompleted = true;
		++it;
		}
		} else {
		++it;
		}
		@@ -1480,6 +1521,9 @@ CameraDevice::ServiceCallback::onDeviceIdle() {
		return ret;
		}

		dev->removeCompletedCallbackHolderLocked(
		std::numeric_limits<int64_t>::max()/lastCompletedRegularFrameNumber/);

		if (dev->mIdle) {
		// Already in idle state. Possibly other thread did waitUntilIdle
		return ret;
		@@ -1522,6 +1566,9 @@ CameraDevice::ServiceCallback::onCaptureStarted(
		return ret;
		}

		dev->removeCompletedCallbackHolderLocked(
		resultExtras.lastCompletedRegularFrameNumber);

		int sequenceId = resultExtras.requestId;
		int32_t burstId = resultExtras.burstId;

camera/ndk/impl/ACameraDevice.h

+18 −2

Original line number	Diff line number	Diff line
		@@ -267,8 +267,23 @@ class CameraDevice final : public RefBase {
		static const int REQUEST_ID_NONE = -1;
		int mRepeatingSequenceId = REQUEST_ID_NONE;

		// sequence id -> last frame number map
		std::map<int, int64_t> mSequenceLastFrameNumberMap;
		// sequence id -> last frame number holder map
		struct RequestLastFrameNumberHolder {
		int64_t lastFrameNumber;
		// Whether the current sequence is completed (capture results are
		// generated). May be set to true, but
		// not removed from the map if not all inflight requests in the sequence
		// have been completed.
		bool isSequenceCompleted = false;
		// Whether all inflight requests in the sequence are completed
		// (capture results and buffers are generated). May be
		// set to true, but not removed from the map yet if the capture results
		// haven't been delivered to the app yet.
		bool isInflightCompleted = false;
		RequestLastFrameNumberHolder(int64_t lastFN) :
		lastFrameNumber(lastFN) {}
		};
		std::map<int, RequestLastFrameNumberHolder> mSequenceLastFrameNumberMap;

		struct CallbackHolder {
		CallbackHolder(sp<ACameraCaptureSession> session,
		@@ -338,6 +353,7 @@ class CameraDevice final : public RefBase {

		void checkRepeatingSequenceCompleteLocked(const int sequenceId, const int64_t lastFrameNumber);
		void checkAndFireSequenceCompleteLocked();
		void removeCompletedCallbackHolderLocked(int64_t lastCompletedRegularFrameNumber);

		// Misc variables
		int32_t mShadingMapSize[2]; // const after constructor

services/audioflinger/Threads.cpp

+0 −4

Original line number	Diff line number	Diff line
		@@ -6062,10 +6062,6 @@ bool AudioFlinger::DirectOutputThread::shouldStandby_l()
		bool trackPaused = false;
		bool trackStopped = false;

		if ((mType == DIRECT) && audio_is_linear_pcm(mFormat) && !usesHwAvSync()) {
		return !mStandby;
		}

		// do not put the HAL in standby when paused. AwesomePlayer clear the offloaded AudioTrack
		// after a timeout and we will enter standby then.
		if (mTracks.size() > 0) {