From 57718cc5a7de0c4f12f14702b1c735b9eb133516 Mon Sep 17 00:00:00 2001 From: glx Date: Mon, 25 Jan 2010 00:43:51 +0000 Subject: (svn r18913) -Fix (r18892): the deadlock was still possible --- src/sound/win32_s.cpp | 20 ++++++++++++++------ 1 file changed, 14 insertions(+), 6 deletions(-) (limited to 'src/sound/win32_s.cpp') diff --git a/src/sound/win32_s.cpp b/src/sound/win32_s.cpp index 67f3b7584..3dc8fd4d0 100644 --- a/src/sound/win32_s.cpp +++ b/src/sound/win32_s.cpp @@ -38,16 +38,18 @@ static void PrepareHeader(WAVEHDR *hdr) static DWORD WINAPI SoundThread(LPVOID arg) { MSG msg; - WAVEHDR *hdr; while (_waveout != NULL) { - for (hdr = _wave_hdr; hdr != endof(_wave_hdr); hdr++) { + for (WAVEHDR *hdr = _wave_hdr; hdr != endof(_wave_hdr); hdr++) { if ((hdr->dwFlags & WHDR_INQUEUE) != 0) continue; MxMixSamples(hdr->lpData, hdr->dwBufferLength / 4); if (waveOutWrite(_waveout, hdr, sizeof(WAVEHDR)) != MMSYSERR_NOERROR) usererror("waveOutWrite failed"); } - GetMessage(&msg, NULL, MM_WOM_DONE, MM_WOM_DONE); + + /* Wait for the device to be closed or ready to play new data. */ + GetMessage(&msg, NULL, MM_WOM_CLOSE, MM_WOM_DONE); } + return 0; } @@ -63,8 +65,10 @@ const char *SoundDriver_Win32::Start(const char * const *parm) _bufsize = GetDriverParamInt(parm, "bufsize", (GB(GetVersion(), 0, 8) > 5) ? 8192 : 4096); + /* Create the sound thread in suspended state because we are not ready to play anything. */ if (NULL == (_thread = CreateThread(NULL, 8192, SoundThread, 0, CREATE_SUSPENDED, &_threadId))) return "Failed to create thread"; + /* Open the sound device, it will send messages to sound thread. */ if (waveOutOpen(&_waveout, WAVE_MAPPER, &wfex, (DWORD_PTR)_threadId, 0, CALLBACK_THREAD) != MMSYSERR_NOERROR) return "waveOutOpen failed"; MxInitialize(wfex.nSamplesPerSec); @@ -72,6 +76,7 @@ const char *SoundDriver_Win32::Start(const char * const *parm) PrepareHeader(&_wave_hdr[0]); PrepareHeader(&_wave_hdr[1]); + /* We are now ready to play sound, so resume the sound thread. */ ResumeThread(_thread); return NULL; @@ -81,13 +86,16 @@ void SoundDriver_Win32::Stop() { HWAVEOUT waveout = _waveout; + /* Break the sound thread loop, but the thread can still be waiting for a message. */ _waveout = NULL; - WaitForMultipleObjects(1, &_thread, true, INFINITE); - waveOutReset(waveout); + /* Stop the playback (if any) and close the device. This will unlock the thread if it's waiting for a message. */ + waveOutReset(waveout); // Triggers MM_WOM_DONE message if there were pending playbacks. waveOutUnprepareHeader(waveout, &_wave_hdr[0], sizeof(WAVEHDR)); waveOutUnprepareHeader(waveout, &_wave_hdr[1], sizeof(WAVEHDR)); - waveOutClose(waveout); + waveOutClose(waveout); // Triggers MM_WOM_CLOSE message. + /* Now we can wait for the sound thread to finish because we know it will. */ + WaitForMultipleObjects(1, &_thread, true, INFINITE); CloseHandle(_thread); } -- cgit v1.2.3-54-g00ecf