http://bugs.winehq.org/show_bug.cgi?id=28723
Andrew Eikum aeikum@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #37437|0 |1 is obsolete| |
--- Comment #54 from Andrew Eikum aeikum@codeweavers.com 2011-11-28 14:57:40 CST --- Created attachment 37683 --> http://bugs.winehq.org/attachment.cgi?id=37683 ALSA fixes patchset
Here's another version, cleaned up to submission quality. It contains Alexey's work, as well as Jörg's GetPosition() reimplementation and some other miscellaneous fixes.
This patchset does not address Bug 29056, but I don't think it makes the situation any worse. That bug can be dealt with separately.
These patches also make no changes or improvements for applications that aren't picky about period sizes (Comment 44). But again I think the solution for that can come later, if one is needed.
The issue of patch ownership will also need to be looked at.
Jörg: I removed the upper bound (5 seconds) on delay_frames in GetPosition(). How strongly do you feel we need that type of logic?
Alexey: Does this patchset fix RAGE for you?
Does anybody object to submitting this? Do we think this is a complete enough fix that we should ask people to retest ALSA-related bugs?