[Openal] OpenAL Soft 1.10 will be sooner than expected
ludwig.nussel at suse.de
Sun Nov 1 03:42:44 PST 2009
Chris Robinson wrote:
> An update on things.
> On Saturday 31 October 2009 1:02:04 pm Ludwig Nussel wrote:
> > - the locking change moved the call to
> > ppa_stream_set_write_callback() downwards. I guess it needs to be
> > done before ppa_stream_connect_playback() though as pulseaudio
> > doesn't call the callback otherwise. However, with that locking
> > change openal runs in a deadlock if
> > ppa_stream_set_write_callback() is called earlier. So the locking
> > change was either incomplete or wrong :-). Which in turn brings
> > back the question whether the behavior change in pulseaudio
> > was correct. I've reverted it locally and ioquake3 works fine now.
> I reverted the locking changes, but I redid it in a way that should be better
> compatible with newer, unpatched Pulse libs.
The workaround works. Still that pulseaudio change looks wrong to me.
> > - The change to stream_write_callback() was wrong, it didn't pass
> > 'len' to ppa_stream_write() but rather data->attr.minreq only. I
> > really don't know the first thing about that stuff but I guess
> > pulseaudio didn't play because it waited for more data.
> I changed the callback to do all full-sized updates given the amount of data
> PulseAudio wants. This seemed to help fix the stalls I get during playback.
> Both ioquake3 and Chromium are working with the PulseAudio backend for me,
Yes, great! I can can confirm that. Thanks!
(o_ Ludwig Nussel
//\ SUSE LINUX Products GmbH, Development
More information about the Openal