[Openal] OpenAL Soft 1.10 will be sooner than expected

[Ludwig Nussel]

Chris Robinson wrote:
> On Friday 30 October 2009 11:39:49 am Ludwig Nussel wrote:
> > No, still no sound with that patch. I've added additional printfs in
> > stream_write_callback() and it looks like it's never called. Whereas I
> >  suppose it should be called all the time as long as the app plays sound?
> 
> Yeah. PulseAudio is supposed to call it regularly as more sound data is needed 
> to fill the buffer, but it doesn't seem to call it properly/at all. Haven't 
> been able to find out why since everything returns success, and the change 
> that broke it should be inconsequential.

Problems identified:
- 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.
- 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.

cu
Ludwig

-- 
 (o_   Ludwig Nussel
 //\   SUSE LINUX Products GmbH, Development
 V_/_  http://www.suse.de/



-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-revert-locking-changes.diff
Type: text/x-diff
Size: 7162 bytes
Desc: not available
Url : http://opensource.creative.com/pipermail/openal/attachments/20091031/4a355ac7/0001-revert-locking-changes.bin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-fix-write_callback-function.diff
Type: text/x-diff
Size: 1113 bytes
Desc: not available
Url : http://opensource.creative.com/pipermail/openal/attachments/20091031/4a355ac7/0002-fix-write_callback-function.bin