requesting review and/or testing for directsound changes

[Maarten Lankhorst]

Hi all,

with http://repo.or.cz/w/wine/multimedia.git and
http://repo.or.cz/w/openal-soft.git you should be able to get the 
benefits of openal-soft's mixer while continuing to use whatever driver 
you want for mmdevapi. As a result directsound will only support 
openal-soft, but multi channel is suddenly supported, as is support for 
the floating point format. 24-bits and 32-bits int are currently not, 
but as soon as that capability is added to openal-soft.git wine will 
have a nice working dsound implementation with support for a lot of 
things that are missing now. The resampling code will also be a lot 
better and you should expect a higher quality when resampling is used.

The capture changes should work nicely, and I don't expect a problem 
with that, since it's based on openal. The rendering code is essentially 
reviving the old dsound-openal, and outputting it to mmdevapi.

Note: 24-bits and 32-bits int are unsupported and will most likely cause 
a crash, disable the #if 1 in get_fmtstr_EXT to prevent lying about 
support. It was put in place so the wine dsound tests would pass.

A dsound.dll.so binary with openal-soft.git statically linked in can be 
found at: http://www.astro.rug.nl/~lankhorst/dsound.dll.so

If you're using pulseaudio, patch your 32-bits alsa-plugins with the 
attached patch, or silence may occur when using the new mmdevapi drivers.

Cheers,
Maarten

[Stefan Dösinger]

Hi,
I did some testing with the shiny new sound code. I tested with and without 
pulseaudio, with current dsound, native(win7) dsound and your precompiled 
binary.

Pulseaudio + current code: Sound effects are sometimes garbled or partially 
played, but are on time and not repeated

No Pulseaudio + current code: not tested

Pulseaudio + maarten's binary: Old sound effects played over and over again, 
after a few seconds the summed up effects morph into static

No Pulseaudio + maarten's binary: Crashed once, on a 2nd try it also loops 
effects all over, sums up to static after a few seconds

Pulseaudio + native dsound: Effects play OK, but sometimes the sound is 
interruped and afterwards delayed. Delays sum up

No pulseaudio + native: works perfectly.

BF1942 had a mouse regression recently. I tried to disable xinput(by 
commenting out the HAVE_XINPUT*_H defines in config.h) and recompiling but this 
did not help. I also tried native dinput(with and without xinput), but this 
did not change anything. The symptom is that the mouse is mostly moving at the 
edges of the screen and can be moved away from there only very carefully. This 
worked two weeks ago. I'll try to run a regression test.

On Tuesday 26 April 2011 12:50:52 Maarten Lankhorst wrote:
> Hi all,
>
> with http://repo.or.cz/w/wine/multimedia.git and
> http://repo.or.cz/w/openal-soft.git you should be able to get the
> benefits of openal-soft's mixer while continuing to use whatever driver
> you want for mmdevapi. As a result directsound will only support
> openal-soft, but multi channel is suddenly supported, as is support for
> the floating point format. 24-bits and 32-bits int are currently not,
> but as soon as that capability is added to openal-soft.git wine will
> have a nice working dsound implementation with support for a lot of
> things that are missing now. The resampling code will also be a lot
> better and you should expect a higher quality when resampling is used.
>
> The capture changes should work nicely, and I don't expect a problem
> with that, since it's based on openal. The rendering code is essentially
> reviving the old dsound-openal, and outputting it to mmdevapi.
>
> Note: 24-bits and 32-bits int are unsupported and will most likely cause
> a crash, disable the #if 1 in get_fmtstr_EXT to prevent lying about
> support. It was put in place so the wine dsound tests would pass.
>
> A dsound.dll.so binary with openal-soft.git statically linked in can be
> found at: http://www.astro.rug.nl/~lankhorst/dsound.dll.so
>
> If you're using pulseaudio, patch your 32-bits alsa-plugins with the
> attached patch, or silence may occur when using the new mmdevapi drivers.
>
> Cheers,
> Maarten