[Bug 29657] winecoreaudio memory management and locking seems to be broken

wine-bugs at winehq.org wine-bugs at winehq.org
Sun Jan 29 14:15:07 CST 2012


--- Comment #3 from Jörg Höhle <hoehle at users.sourceforge.net> 2012-01-29 14:15:07 CST ---
Created attachment 38597
  --> http://bugs.winehq.org/attachment.cgi?id=38597
lockless callback design for winecoreaudio

Here are the 3 parts.  The wineoss part needs to be completed.

Obviously the code is begging for merging the third queue from
get_current_aqbuffer_position, perhaps via resolution of bug #29602. There
should be 2 queues only, and AQBuffer integrate sample time data.

Now you can run hundreds of test iterations without dead-lock and with little
CPU usage.

I've had one test run where EnqueueBuffer failed in render.c.
AudioRenderClient_ReleaseBuffer (0x149480)->(20671, 2)
er::AudioRenderClient_ReleaseBuffer Unable to enqueue buffer: fffefb8e
That was before I merged Andrew's GetPosition patch from bug #29602.

Also, WaitForSingleObject(event, 70) fails with rc=102 in rare cases.  That
means that there's either a flaw in the logic, or even on a dual core MacOS,
the CreateTimerQueue events can be very far from 10ms apart... I can't remember
whether that happened before I introduced EXECUTEINTIMERTHREAD though.

Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
Do not reply to this email, post in Bugzilla using the
above URL to reply.
------- You are receiving this mail because: -------
You are watching all bug changes.

More information about the wine-bugs mailing list