system/qemu
system/qemu
1
0
Fork 0
mirror of https://gitlab.com/qemu-project/qemu synced 2024-07-23 11:24:38 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

dsoundaudio: remove *_retries kludges

Browse source 
According to MSDN this may happen when the window is not in the foreground, but
the default is 1 since a long time (which means no retries), so it should be ok.
I've found no problems during testing it on Windows 7 and wine, so this was
probably only the case with some old Windows versions.

Signed-off-by: Kővágó, Zoltán <DirtY.iCE.hu@gmail.com>
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
This commit is contained in:
Kővágó, Zoltán 2015-06-12 14:33:04 +02:00 committed by Gerd Hoffmann
parent 73ad33ef7b
commit 2762955f72
2 changed files with 20 additions and 83 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

35
audio/dsound_template.h
View file

@ -72,48 +72,27 @@ static int glue (dsound_lock_, TYPE) (
) )
{ {
HRESULT hr; HRESULT hr;
int i;
LPVOID p1 = NULL, p2 = NULL; LPVOID p1 = NULL, p2 = NULL;
DWORD blen1 = 0, blen2 = 0; DWORD blen1 = 0, blen2 = 0;
DWORD flag; DWORD flag;
DSoundConf *conf = &s->conf;
#ifdef DSBTYPE_IN #ifdef DSBTYPE_IN
flag = entire ? DSCBLOCK_ENTIREBUFFER : 0; flag = entire ? DSCBLOCK_ENTIREBUFFER : 0;
#else #else
flag = entire ? DSBLOCK_ENTIREBUFFER : 0; flag = entire ? DSBLOCK_ENTIREBUFFER : 0;
#endif #endif
for (i = 0; i < conf->lock_retries; ++i) { hr = glue(IFACE, _Lock)(buf, pos, len, &p1, &blen1, &p2, &blen2, flag);
hr = glue (IFACE, _Lock) (
buf,
pos,
len,
&p1,
&blen1,
&p2,
&blen2,
flag
);
if (FAILED (hr)) { if (FAILED (hr)) {
#ifndef DSBTYPE_IN #ifndef DSBTYPE_IN
if (hr == DSERR_BUFFERLOST) { if (hr == DSERR_BUFFERLOST) {
if (glue (dsound_restore_, TYPE) (buf, s)) { if (glue (dsound_restore_, TYPE) (buf, s)) {
dsound_logerr (hr, "Could not lock " NAME "\n"); dsound_logerr (hr, "Could not lock " NAME "\n");
goto fail;
}
continue;
} }
#endif
dsound_logerr (hr, "Could not lock " NAME "\n");
goto fail; goto fail;
} }
#endif
break; dsound_logerr (hr, "Could not lock " NAME "\n");
}
if (i == conf->lock_retries) {
dolog ("%d attempts to lock " NAME " failed\n", i);
goto fail; goto fail;
} }

68
audio/dsoundaudio.c
View file

@ -42,9 +42,6 @@
/* #define DEBUG_DSOUND */ /* #define DEBUG_DSOUND */
typedef struct { typedef struct {
int lock_retries;
int restore_retries;
int getstatus_retries;
int set_primary; int set_primary;
int bufsize_in; int bufsize_in;
int bufsize_out; int bufsize_out;
@ -274,26 +271,14 @@ static void print_wave_format (WAVEFORMATEX *wfx)
static int dsound_restore_out (LPDIRECTSOUNDBUFFER dsb, dsound *s) static int dsound_restore_out (LPDIRECTSOUNDBUFFER dsb, dsound *s)
{ {
HRESULT hr; HRESULT hr;
int i;
for (i = 0; i < s->conf.restore_retries; ++i) { hr = IDirectSoundBuffer_Restore (dsb);
hr = IDirectSoundBuffer_Restore (dsb);
switch (hr) { if (hr != DS_OK) {
case DS_OK: dsound_logerr (hr, "Could not restore playback buffer\n");
return 0; return -1;
case DSERR_BUFFERLOST:
continue;
default:
dsound_logerr (hr, "Could not restore playback buffer\n");
return -1;
}
} }
return 0;
dolog ("%d attempts to restore playback buffer failed\n", i);
return -1;
} }
#include "dsound_template.h" #include "dsound_template.h"
@ -305,22 +290,16 @@ static int dsound_get_status_out (LPDIRECTSOUNDBUFFER dsb, DWORD *statusp,
dsound *s) dsound *s)
{ {
HRESULT hr; HRESULT hr;
int i;
for (i = 0; i < s->conf.getstatus_retries; ++i) { hr = IDirectSoundBuffer_GetStatus (dsb, statusp);
hr = IDirectSoundBuffer_GetStatus (dsb, statusp); if (FAILED (hr)) {
if (FAILED (hr)) { dsound_logerr (hr, "Could not get playback buffer status\n");
dsound_logerr (hr, "Could not get playback buffer status\n"); return -1;
return -1; }
}
if (*statusp & DSERR_BUFFERLOST) { if (*statusp & DSERR_BUFFERLOST) {
if (dsound_restore_out (dsb, s)) { dsound_restore_out(dsb, s);
return -1; return -1;
}
continue;
}
break;
} }
return 0; return 0;
@ -844,9 +823,6 @@ static int dsound_run_in (HWVoiceIn *hw)
} }
static DSoundConf glob_conf = { static DSoundConf glob_conf = {
.lock_retries = 1,
.restore_retries = 1,
.getstatus_retries = 1,
.set_primary = 0, .set_primary = 0,
.bufsize_in = 16384, .bufsize_in = 16384,
.bufsize_out = 16384, .bufsize_out = 16384,
@ -958,24 +934,6 @@ static void *dsound_audio_init (void)
} }
static struct audio_option dsound_options[] = { static struct audio_option dsound_options[] = {
{
.name = "LOCK_RETRIES",
.tag = AUD_OPT_INT,
.valp = &glob_conf.lock_retries,
.descr = "Number of times to attempt locking the buffer"
},
{
.name = "RESTOURE_RETRIES",
.tag = AUD_OPT_INT,
.valp = &glob_conf.restore_retries,
.descr = "Number of times to attempt restoring the buffer"
},
{
.name = "GETSTATUS_RETRIES",
.tag = AUD_OPT_INT,
.valp = &glob_conf.getstatus_retries,
.descr = "Number of times to attempt getting status of the buffer"
},
{ {
.name = "SET_PRIMARY", .name = "SET_PRIMARY",
.tag = AUD_OPT_BOOL, .tag = AUD_OPT_BOOL,