DirectSound, new patch available for testing

[Davin McCall]

Dang. Forgot to attach.

---
Here's another patch to fix the clicking noise. Once again apply over the top.

With this one, I think everything should be at least as good as it was with my very first patch. (Plus the full device buffer will be used if UseDirectHW="y", which should reduce underrun problems).

Davin


On Thu, 3 Nov 2005 18:42:50 +1100
Davin McCall <davmac at davmac.org> wrote:

> Thanks for that -
> 
> There was at least one problem with the patch. I've attached a fix for that (apply over the top of the previous one).
> 
> I don't get buffer underruns in my own testing, but I do get an annoying clicking noise every so often (seems to be when streaming buffers wrap from the end back to the start). I think that's a mixer issue rather than an ALSA driver issue however as it occurs with OSS as well as ALSA. At the moment, I'm focusing primarily on the ALSA driver issues.
> 
> I'd be grateful if you could try this modification (best results should be with full acceleration, and UseDirectHW set to "y").
> 
> Davin
> 
> 
> 
> On Thu, 03 Nov 2005 02:18:48 +0000
> Randall Walls <rwalls at gwi.net> wrote:
> 
> > Tested again with BattleZone2 and I'm still getting what sounds like 
> > serious buffer underruns with this new patch. This one seemed to 'skip' 
> > a bit more too, depending on action taking place on screen, but that 
> > could be due to other issues (system load etc...). Running with Alsa as 
> > the audio driver and full hardware acceleration, no emulation. Tested 
> > both with and without the registry key set, and stuttering and the 
> > looping sounds actually seemed to get worse WITH the key. Switched to 
> > OSS driver to test (this would be Alsa's OSS emulation) and I still get 
> > the repeating sounds but some of the other issues are better. 
> > BattleZone2 does have a demo that can still be found online, and there 
> > was a patch on the list not too long ago to get the demo up and running.
> > 
> > Again, let me know if any traces would be benneficial.
> > 
> > Randall Walls
> > 
> > 
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dsound2.patch
Type: text/x-patch
Size: 2092 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-devel/attachments/20051103/6c5e6baf/dsound2.patch