[Bug 29243] New: Homefront: Choppy sound in menu and game play. (first appeared in wine-1.3.31)
http://bugs.winehq.org/show_bug.cgi?id=29243
Bug #: 29243 Summary: Homefront: Choppy sound in menu and game play. (first appeared in wine-1.3.31) Product: Wine Version: 1.3.31 Platform: x86 URL: steam://install/55130 OS/Version: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: winealsa.drv AssignedTo: wine-bugs@winehq.org ReportedBy: maverickplayer@googlemail.com Classification: Unclassified Regression SHA1: b0652dd8bdb144645be4a6bf77cbb68a8ade49d9
Created attachment 37795 --> http://bugs.winehq.org/attachment.cgi?id=37795 [PATCH] Revert "winealsa.drv: Don't try to control ALSA's behavior."
For the game Homefront, wine-1.3.31 introduces a bug that is not present in wine-1.3.30. The bug is choppy sound in the game menu and the game play. The intro credit videos do not have the issue. Using 'git bisect' discovered that
b0652dd8bdb144645be4a6bf77cbb68a8ade49d9 is the first bad commit
Attached is a patch to revert that 'bad' commit in wine-1.3.34 which still has the bug. This required some manual conflict resolution. The resulting version does not have the bug.
http://bugs.winehq.org/show_bug.cgi?id=29243
Maverick Player maverickplayer@googlemail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch, regression
http://bugs.winehq.org/show_bug.cgi?id=29243
--- Comment #1 from Maverick Player maverickplayer@googlemail.com 2011-12-04 16:48:01 CST --- Created attachment 37796 --> http://bugs.winehq.org/attachment.cgi?id=37796 Homefront: Log when running on wine-1.3.30 using WINEDEBUG="+tid,+mmdevapi,+winmm,+midi,+dsound,+dmusic,+mci,+oss,+alsa,+coreaudio"
In this wine version the bug is not present.
http://bugs.winehq.org/show_bug.cgi?id=29243
--- Comment #2 from Maverick Player maverickplayer@googlemail.com 2011-12-04 16:48:49 CST --- Created attachment 37797 --> http://bugs.winehq.org/attachment.cgi?id=37797 Homefront: Log when running on wine-1.3.31 using WINEDEBUG="+tid,+mmdevapi,+winmm,+midi,+dsound,+dmusic,+mci,+oss,+alsa,+coreaudio"
In this wine version the bug is present.
http://bugs.winehq.org/show_bug.cgi?id=29243
--- Comment #3 from Maverick Player maverickplayer@googlemail.com 2011-12-04 16:49:18 CST --- Created attachment 37798 --> http://bugs.winehq.org/attachment.cgi?id=37798 Homefront: Log when running on wine-1.3.34 using WINEDEBUG="+tid,+mmdevapi,+winmm,+midi,+dsound,+dmusic,+mci,+oss,+alsa,+coreaudio"
In this wine version the bug is present.
http://bugs.winehq.org/show_bug.cgi?id=29243
--- Comment #4 from Maverick Player maverickplayer@googlemail.com 2011-12-04 16:52:44 CST --- Created attachment 37799 --> http://bugs.winehq.org/attachment.cgi?id=37799 Homefront: Log when running on wine-1.3.34+fix-homefront-choppy-sound using WINEDEBUG="+tid,+mmdevapi,+winmm,+midi,+dsound,+dmusic,+mci,+oss,+alsa,+coreaudio"
In this wine version, which includes the patch [PATCH] Revert "winealsa.drv: Don't try to control ALSA's behavior." the bug is not present.
http://bugs.winehq.org/show_bug.cgi?id=29243
Dmitry Timoshkov dmitry@baikal.ru changed:
What |Removed |Added ---------------------------------------------------------------------------- URL|steam://install/55130 | Summary|Homefront: Choppy sound in |Homefront: Choppy sound in |menu and game play. (first |menu and game play |appeared in wine-1.3.31) |
http://bugs.winehq.org/show_bug.cgi?id=29243
Andrew Eikum aeikum@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |aeikum@codeweavers.com
--- Comment #5 from Andrew Eikum aeikum@codeweavers.com 2011-12-05 08:47:59 CST --- The ALSA period size is 940 frames, and the game feeds data in chunks of 480 frames, whenever there's less than 480 frames in the buffer. So the game has at most 960 frames of data into the buffer at any time. Since that's so close to the ALSA period size, lots of underruns occur.
We're working on large improvements to the ALSA driver in Bug 28723. It's likely that this bug will be fixed when the fixes for that bug are in.
http://bugs.winehq.org/show_bug.cgi?id=29243
Maverick Player maverickplayer@googlemail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- URL| |http://store.steampowered.c | |om/app/55100/
http://bugs.winehq.org/show_bug.cgi?id=29243
Maverick Player maverickplayer@googlemail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |54b24f39260a3780bdf97d810a6 | |9647d7357cb62 Status|UNCONFIRMED |RESOLVED Resolution| |FIXED
--- Comment #6 from Maverick Player maverickplayer@googlemail.com 2011-12-30 18:50:38 CST --- Bug fixed in wine-1.3.36.
Bug was still present in wine-1.3.35. A git bisect found the fixing commit to be:
commit 54b24f39260a3780bdf97d810a69647d7357cb62 Author: Andrew Eikum aeikum@codeweavers.com Date: Mon Dec 19 15:24:13 2011 -0600
winealsa.drv: Limit the data written to ALSA's buffer.
Based on an algorithm by Alexey Loukianov.
:040000 040000 3fc55174705a7f4edaf7c80b012b4420750038b1 50627baa8893c1d8bfdf4694c409cdf8e6ee0316 M dlls
I double checked by explicitly building and testing the previous commit (3082c1a9de14e9c21c4f93331b6d595bbca00e34) and this commit.
Thanks for your initial quick response to the bug report and your work on fixing the bug. Keep up the great work and thank you for wine.
http://bugs.winehq.org/show_bug.cgi?id=29243
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #7 from Alexandre Julliard julliard@winehq.org 2012-01-13 13:50:34 CST --- Closing bugs fixed in 1.3.37.
-
wine-bugs＠winehq.org