summaryrefslogtreecommitdiffstats
path: root/services
diff options
context:
space:
mode:
authorEric Laurent <elaurent@google.com>2015-09-09 12:26:52 -0700
committerEric Laurent <elaurent@google.com>2015-09-09 12:41:10 -0700
commit564d144fc28917d42e3a67718ac51d61bfc36315 (patch)
tree6db8c93d002f5594bf80ee709706609a4df92fb4 /services
parent27e0350ae57184d5e300f96c8338379f5d3d0fbc (diff)
downloadframeworks_av-564d144fc28917d42e3a67718ac51d61bfc36315.zip
frameworks_av-564d144fc28917d42e3a67718ac51d61bfc36315.tar.gz
frameworks_av-564d144fc28917d42e3a67718ac51d61bfc36315.tar.bz2
Revert "Do not acknowledge flush() on start() for static tracks"
This reverts commit cafe86a9cb6625bb1ec6383e16e28e4c9e455f87. Bug: 23924093. Change-Id: I186d1013b06a286eca93c30bb9b3545dc36695ff
Diffstat (limited to 'services')
-rw-r--r--services/audioflinger/Tracks.cpp9
1 files changed, 3 insertions, 6 deletions
diff --git a/services/audioflinger/Tracks.cpp b/services/audioflinger/Tracks.cpp
index d058255..f7da209 100644
--- a/services/audioflinger/Tracks.cpp
+++ b/services/audioflinger/Tracks.cpp
@@ -712,12 +712,9 @@ status_t AudioFlinger::PlaybackThread::Track::start(AudioSystem::sync_event_t ev
// But in this case we know the mixer thread (whether normal mixer or fast mixer)
// isn't looking at this track yet: we still hold the normal mixer thread lock,
// and for fast tracks the track is not yet in the fast mixer thread's active set.
- // TODO: remove race condition on stop() followed by start().
- if (mSharedBuffer == 0) { // only streaming tracks use flush().
- ServerProxy::Buffer buffer;
- buffer.mFrameCount = 1;
- (void) mAudioTrackServerProxy->obtainBuffer(&buffer, true /*ackFlush*/);
- }
+ ServerProxy::Buffer buffer;
+ buffer.mFrameCount = 1;
+ (void) mAudioTrackServerProxy->obtainBuffer(&buffer, true /*ackFlush*/);
}
} else {
status = BAD_VALUE;