Revert "Camera2: Fix a potential race condition in the readout thread"
The last fix incorrectly added a mutex lock which did not guard the appropriate
variables.
In fact it actually *introduced* a new race condition in a HAL test.
This reverts commit 6edd7ab4a4.
This commit is contained in:
@@ -1135,7 +1135,6 @@ void EmulatedFakeCamera2::ReadoutThread::setNextOperation(
|
|||||||
camera_metadata_t *request,
|
camera_metadata_t *request,
|
||||||
Buffers *buffers) {
|
Buffers *buffers) {
|
||||||
Mutex::Autolock lock(mInputMutex);
|
Mutex::Autolock lock(mInputMutex);
|
||||||
Mutex::Autolock iLock(mInternalsMutex);
|
|
||||||
if ( !readyForNextCapture() ) {
|
if ( !readyForNextCapture() ) {
|
||||||
ALOGE("In flight queue full, dropping captures");
|
ALOGE("In flight queue full, dropping captures");
|
||||||
mParent->signalError();
|
mParent->signalError();
|
||||||
|
|||||||
Reference in New Issue
Block a user