OSDN Git Service

Fix issue 2428563: Camera rendered inoperable by voice call interruption.
authorEric Laurent <elaurent@google.com>
Wed, 3 Mar 2010 02:38:06 +0000 (18:38 -0800)
committerEric Laurent <elaurent@google.com>
Fri, 5 Mar 2010 19:54:23 +0000 (11:54 -0800)
commit4f1fcc2890f7db5509740c3daedf51ee3a38869e
treee4d043825e143afdfb9f6283404a40396512b0f3
parent5500621e72abff8bd753fdb781dbc423978bca13
Fix issue 2428563: Camera rendered inoperable by voice call interruption.

The problem is that AudioRecord never exits read() when a timeout occurs while trying
to get new PCM data from audio hardware input buffer: it just keeps waiting and retrying until stop() is called.
In the same time, opencore AndroidAudioInput::audin_thread_func() loop cannot be exited when stuck
in AudioRecord::read() because the iExitAudioThread flag can only be sampled when AudioRecord::read()
returns. We remain stuck with the audio input thread running.

The fix consists in modifying AudioRecord behavior in case of timeout when getting new PCM samples.
We now wait only one timeout period and try to restart audio record, in case the problem is due to a media_server
process crash. If this fails, we exit read() with a number of bytes read equals to 0 so that
AndroidAudioInput::audin_thread_func() loop can exit.

Also modified Audioflinger::RecordThread() loop so that we attempt to recover from HAL read errors.
In case of read error, the input stream is forced to standby so that next read attempt does a
reconfiguration and restart of the audio input device.
libs/audioflinger/AudioFlinger.cpp