[Bug 13090] New: winecfg crashes if you click the "Test Sound" button while using the ALSA driver
http://bugs.winehq.org/show_bug.cgi?id=13090
Summary: winecfg crashes if you click the "Test Sound" button while using the ALSA driver Product: Wine Version: 1.0-rc1 Platform: PC-x86-64 OS/Version: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown AssignedTo: wine-bugs@winehq.org ReportedBy: andrebrait@gmail.com
By clicking the "Test Sound" button while using the ALSA driver, the sound is played, but winecfg crashes instantly after that. Work normally with OSS driver. Not tested with EsounD or Jack.
http://bugs.winehq.org/show_bug.cgi?id=13090
--- Comment #1 from Vitaliy Margolen vitaliy@kievinfo.com 2008-05-10 16:19:12 --- What distro?
http://bugs.winehq.org/show_bug.cgi?id=13090
--- Comment #2 from André Brait andrebrait@gmail.com 2008-05-10 19:57:06 --- (In reply to comment #1)
What distro?
Ubuntu Hardy x86-64 Tested with both pre-compiled package and the one I compiled myself.
http://bugs.winehq.org/show_bug.cgi?id=13090
--- Comment #3 from Vitaliy Margolen vitaliy@kievinfo.com 2008-05-10 19:59:55 --- Sounds like pulseaudio gets in the way. Kill it, it's not compatible with Wine.
http://bugs.winehq.org/show_bug.cgi?id=13090
--- Comment #4 from André Brait andrebrait@gmail.com 2008-05-10 23:51:10 --- I killed it and tried to use it's wrapper, no progress.
http://bugs.winehq.org/show_bug.cgi?id=13090
Louis Lenders xerox_xerox2000@yahoo.co.uk changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |xerox_xerox2000@yahoo.co.uk Status|UNCONFIRMED |NEW Ever Confirmed|0 |1 Keywords| |regression Target Milestone|--- |1.0.0
--- Comment #5 from Louis Lenders xerox_xerox2000@yahoo.co.uk 2008-05-11 03:38:15 --- I can confirm this bug. In wine-0.9.56 this all works fine, but in current git it hangs. I think this need to be fixed, before the final release. If winecfg just hangs in a final release it would be a shame, right?
http://bugs.winehq.org/show_bug.cgi?id=13090
--- Comment #6 from André Brait andrebrait@gmail.com 2008-05-11 06:08:27 --- Thank you. I'm happy to know I'm not the only one with this bug.
http://bugs.winehq.org/show_bug.cgi?id=13090
Louis Lenders xerox_xerox2000@yahoo.co.uk changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |m.b.lankhorst@gmail.com
--- Comment #7 from Louis Lenders xerox_xerox2000@yahoo.co.uk 2008-05-11 10:35:26 --- c8bdebde3dc94baf1439697306e6f797ca5c7ad2 is first bad commit commit c8bdebde3dc94baf1439697306e6f797ca5c7ad2 Author: Maarten Lankhorst m.b.lankhorst@gmail.com Date: Thu May 8 14:51:07 2008 -0700
winealsa: Remove calls to deprecated functions.
:040000 040000 f34f76e35d76f4a14b5d3c874d32bddd48bf807f f220197d6917c30cd1c9819e25b11fa98d345f4c M dlls
http://bugs.winehq.org/show_bug.cgi?id=13090
--- Comment #8 from André Brait andrebrait@gmail.com 2008-05-11 16:36:52 --- Sorry, I'm a newbie, but I just know how to recognize a bug. What does it mean?
http://bugs.winehq.org/show_bug.cgi?id=13090
--- Comment #9 from Louis Lenders xerox_xerox2000@yahoo.co.uk 2008-05-11 16:40:41 ---
What does it mean?
That means i did a regression test , to find the patch that caused the bug. Now waiting for response from the one who made the patch, and hopefully a fix ;)
http://bugs.winehq.org/show_bug.cgi?id=13090
--- Comment #10 from André Brait andrebrait@gmail.com 2008-05-11 16:43:03 --- Ok, Thank you =D
http://bugs.winehq.org/show_bug.cgi?id=13090
Neil Skrypuch ns03ja@brocku.ca changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |ns03ja@brocku.ca
--- Comment #11 from Neil Skrypuch ns03ja@brocku.ca 2008-05-11 19:31:12 --- I can confirm this too on Gentoo with ALSA (and x86-64).
http://bugs.winehq.org/show_bug.cgi?id=13090
--- Comment #12 from André Brait andrebrait@gmail.com 2008-05-11 19:38:59 --- Thanks It seems I really found a bug =D
http://bugs.winehq.org/show_bug.cgi?id=13090
Rafał Mużyło galtgendo@o2.pl changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |galtgendo@o2.pl
--- Comment #13 from Rafał Mużyło galtgendo@o2.pl 2008-05-12 10:33:14 --- Though I claim no real knowledge about ALSA, after googling and reading alsa-lib sources, I came to conclusion, that while tick_time seems unused in wine and snd_pcm_sw_params_set_xfer_align looks like a noop, snd_pcm_sw_params_set_xrun_mode is definitely not one. It looks like it should have been replaced by snd_pcm_sw_params_set_stop_threshold(pcm, sw_params, pcm->boundary). I will probably test it soon.
http://bugs.winehq.org/show_bug.cgi?id=13090
--- Comment #14 from Rafał Mużyło galtgendo@o2.pl 2008-05-12 12:19:29 --- Well, alsa-libsources suggest that following should work: snd_pcm_uframes_t boundary; snd_pcm_sw_params_get_boundary(sw_params, &boundary); EXIT_ON_ERROR( snd_pcm_sw_params_set_stop_threshold(pcm, sw_params, boundary), MMSYSERR_ERROR, "unable to set stop threshold"); but for some reason, it doesn't (only needed in waveout.c, cause others already do it).
http://bugs.winehq.org/show_bug.cgi?id=13090
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED
--- Comment #15 from Alexandre Julliard julliard@winehq.org 2008-05-12 13:45:35 --- Patch is reverted for now.
http://bugs.winehq.org/show_bug.cgi?id=13090
--- Comment #16 from Rafał Mużyło galtgendo@o2.pl 2008-05-12 14:51:35 --- Well, no. Reverting that patch doesn't work for me.
However, reverting this one (commit 0fe3a59b46a44b6d2d54a1afe1c4198c53d6c68c), does.
It prints then (when I run certain app and close it shortly after the sound starts playing) ALSA lib conf.c:3952:(snd_config_expand) Unknown parameters 0 ALSA lib pcm.c:2144:(snd_pcm_open_noupdate) Unknown PCM default:0 ALSA lib conf.c:3952:(snd_config_expand) Unknown parameters 0 ALSA lib pcm.c:2144:(snd_pcm_open_noupdate) Unknown PCM default:0 fixme:mixer:ALSA_MixerInit No master control found on MPU-401 UART, disabling mixer fixme:mixer:ALSA_MixerInit No master control found on VirMIDI, disabling mixer fixme:win:WINNLSEnableIME hUnknown1 0x10024 bUnknown2 0: stub! fixme:win:EnumDisplayDevicesW ((null),0,0x6ef8ac,0x00000000), stub! fixme:dmime:IDirectMusicPerformance8Impl_Init (iface = 0x145928, dmusic = 0x49c7f0, dsound = 0x144270, hwnd = (nil)) fixme:dmime:IDirectMusicPerformance8Impl_SetNotificationHandle (0x145928, 0xa4, 0x0): stub fixme:dmime:IDirectMusicPerformance8Impl_AddNotificationType (0x145928, GUID_NOTIFICATION_SEGMENT): stub err:dmloader:IDirectMusicLoaderImpl_IDirectMusicLoader_SetObject : could not attach stream to file fixme:dmime:IDirectMusicPerformance8Impl_AddPort (0x145928, 0x1a1ec8): stub fixme:dmime:IDirectMusicPerformance8Impl_AssignPChannelBlock (0x145928, 0, 0x1a1ec8, 0): semi-stub fixme:dmime:IDirectMusicPerformance8Impl_CloseDown (0x145928): stub fixme:winmm:MMDRV_Exit Closing while ll-driver open
But the sound does work then.
http://bugs.winehq.org/show_bug.cgi?id=13090
--- Comment #17 from Rafał Mużyło galtgendo@o2.pl 2008-05-12 14:54:13 --- What's more it works even with that deprecation patch applied, though it will probably need to be extended with snd_pcm_uframes_t boundary; snd_pcm_sw_params_get_boundary(sw_params, &boundary); EXIT_ON_ERROR( snd_pcm_sw_params_set_stop_threshold(pcm, sw_params, boundary), MMSYSERR_ERROR, "unable to set stop threshold"); in waveout.c.
http://bugs.winehq.org/show_bug.cgi?id=13090
--- Comment #18 from Rafał Mużyło galtgendo@o2.pl 2008-05-12 15:35:45 --- So the conclusion is that, for some reason it doesn't work with pcmname == default but works with pcmname == plughw:0,0
http://bugs.winehq.org/show_bug.cgi?id=13090
--- Comment #19 from Rafał Mużyło galtgendo@o2.pl 2008-05-12 15:51:30 --- It's probably a pulseaudio issue, but a one meaning you assume card can always do something, that can't be done with pulseaudio. My asound.conf is a standard one (for pulseaudio): pcm.pulse { type pulse }
ctl.pulse { type pulse } pcm.!default { type pulse } ctl.!default { type pulse }
http://bugs.winehq.org/show_bug.cgi?id=13090
--- Comment #20 from Rafał Mużyło galtgendo@o2.pl 2008-05-12 16:57:58 --- On an unrelated note, when sound work (well, even when it didn't) when sound is set to either midi or DirectSoundWave, game works, but the moment it's switched to DirectMusic, it crashes with following backtrace: Unhandled exception: page fault on read access to 0x00000000 in 32-bit code (0x00429c18). Register dump: CS:0073 SS:007b DS:007b ES:007b FS:0033 GS:003b EIP:00429c18 ESP:006ef5cc EBP:006ef710 EFLAGS:00010246( - 00 -RIZP1) EAX:00000000 EBX:00000001 ECX:00000000 EDX:00000065 ESI:006ef5f8 EDI:006ef609 Stack dump: 0x006ef5cc: 00000005 00429a68 006ef5f8 00002710 0x006ef5dc: 00463b65 004132dd 006ef5f8 00000001 0x006ef5ec: 00000001 00480ca0 00000000 4953554d 0x006ef5fc: 47425c43 31305f4d 64696d2e 00000000 0x006ef60c: 00000000 00000000 00000000 00000000 0x006ef61c: 00000000 00000000 00000000 00000000 Backtrace: =>1 0x00429c18 in <game name> (+0x29c18) (0x006ef710) 2 0x0041b782 in <game name> (+0x1b782) (0x006ef74c) 3 0x0041f2bc in <game name> (+0x1f2bc) (0x00009c8a) 4 0x00000000 (0x00000000) 0x00429c18: movl 0x0(%eax),%ecx Modules: Module Address Debug info Name (123 modules) PE 400000- 4e9000 Export discipline ELF 6a7ff000-6a82a000 Deferred dmloader<elf> -PE 6a810000-6a82a000 \ dmloader ELF 6afd6000-6b017000 Deferred dmime<elf> -PE 6afe0000-6b017000 \ dmime ELF 6b114000-6b133000 Deferred dmsynth<elf> -PE 6b120000-6b133000 \ dmsynth ELF 6b9d5000-6ba20000 Deferred dsound<elf> -PE 6b9e0000-6ba20000 \ dsound ELF 7bf00000-7bf04000 Deferred <wine-loader> ELF 7bf89000-7bfaf000 Deferred dmusic<elf> -PE 7bf90000-7bfaf000 \ dmusic ELF 7c42d000-7c447000 Deferred libjis.so ELF 7c447000-7c44c000 Deferred euc-jp.so ELF 7c44c000-7c46d000 Deferred libexpat.so.1 ELF 7c46d000-7c689000 Deferred r300_dri.so ELF 7c689000-7c69c000 Deferred libxcb-glx.so.0 ELF 7c69c000-7c6a7000 Deferred libdrm.so.2 ELF 7c6a7000-7c71d000 Deferred libgl.so.1 ELF 7c73d000-7c851000 Deferred wined3d<elf> -PE 7c750000-7c851000 \ wined3d ELF 7c851000-7c8a8000 Deferred ddraw<elf> -PE 7c860000-7c8a8000 \ ddraw ELF 7c8a8000-7c8bd000 Deferred winejoystick<elf> -PE 7c8b0000-7c8bd000 \ winejoystick ELF 7d101000-7d117000 Deferred midimap<elf> -PE 7d110000-7d117000 \ midimap ELF 7d117000-7d120000 Deferred librt.so.1 ELF 7d120000-7d1de000 Deferred libasound.so.2 ELF 7d1de000-7d215000 Deferred winealsa<elf> -PE 7d1f0000-7d215000 \ winealsa ELF 7d215000-7d247000 Deferred libhx509.so.1 ELF 7d247000-7d2b0000 Deferred libgcrypt.so.11 ELF 7d2b0000-7d2c1000 Deferred libtasn1.so.3 ELF 7d2c1000-7d2d4000 Deferred libresolv.so.2 ELF 7d2d4000-7d306000 Deferred libcrypt.so.1 ELF 7d306000-7d319000 Deferred libroken.so.18 ELF 7d319000-7d45d000 Deferred libcrypto.so.0.9.8 ELF 7d45d000-7d4d4000 Deferred libasn1.so.8 ELF 7d4d4000-7d530000 Deferred libkrb5.so.22 ELF 7d530000-7d5ad000 Deferred libgnutls.so.26 ELF 7d5ad000-7d5d6000 Deferred libgssapi.so.2 ELF 7d5d6000-7d60c000 Deferred libcups.so.2 ELF 7d613000-7d62c000 Deferred msacm32<elf> -PE 7d620000-7d62c000 \ msacm32 ELF 7de34000-7de67000 Deferred uxtheme<elf> -PE 7de40000-7de67000 \ uxtheme ELF 7de67000-7de71000 Deferred libxcursor.so.1 ELF 7de71000-7de77000 Deferred libxfixes.so.3 ELF 7de77000-7de7b000 Deferred libxcomposite.so.1 ELF 7de7b000-7de83000 Deferred libxrandr.so.2 ELF 7de83000-7de8c000 Deferred libxrender.so.1 ELF 7de8c000-7de90000 Deferred libxinerama.so.1 ELF 7de90000-7de96000 Deferred libxdmcp.so.6 ELF 7de96000-7deb0000 Deferred libxcb.so.1 ELF 7deb0000-7deb3000 Deferred libxcb-xlib.so.0 ELF 7deb3000-7deb7000 Deferred libxau.so.6 ELF 7deb7000-7ded0000 Deferred libice.so.6 ELF 7ded0000-7dfb9000 Deferred libx11.so.6 ELF 7dfb9000-7dfc8000 Deferred libxext.so.6 ELF 7dfc8000-7dfce000 Deferred libxxf86vm.so.1 ELF 7dfce000-7dfd7000 Deferred libsm.so.6 ELF 7dfd8000-7dfdb000 Deferred libx11-xcb.so.1 ELF 7dfdb000-7dfdf000 Deferred libxdamage.so.1 ELF 7dfdf000-7dfe4000 Deferred libgpg-error.so.0 ELF 7dfe4000-7dfea000 Deferred libheimntlm.so.0 ELF 7dff3000-7dff7000 Deferred libcom_err.so.2 ELF 7dff7000-7e091000 Deferred winex11<elf> -PE 7e010000-7e091000 \ winex11 ELF 7e10c000-7e22c000 Deferred libxml2.so.2 ELF 7e22c000-7e257000 Deferred libfontconfig.so.1 ELF 7e257000-7e26a000 Deferred libz.so.1 ELF 7e26a000-7e2e6000 Deferred libfreetype.so.6 ELF 7e306000-7e327000 Deferred imm32<elf> -PE 7e310000-7e327000 \ imm32 ELF 7e327000-7e346000 Deferred iphlpapi<elf> -PE 7e330000-7e346000 \ iphlpapi ELF 7e346000-7e3a9000 Deferred rpcrt4<elf> -PE 7e350000-7e3a9000 \ rpcrt4 ELF 7e3a9000-7e44e000 Deferred ole32<elf> -PE 7e3c0000-7e44e000 \ ole32 ELF 7e44e000-7e463000 Deferred lz32<elf> -PE 7e450000-7e463000 \ lz32 ELF 7e463000-7e48c000 Deferred msvfw32<elf> -PE 7e470000-7e48c000 \ msvfw32 ELF 7e48c000-7e4b4000 Deferred msacm32<elf> -PE 7e490000-7e4b4000 \ msacm32 ELF 7e4b4000-7e4f0000 Deferred avifil32<elf> -PE 7e4c0000-7e4f0000 \ avifil32 ELF 7e4f0000-7e585000 Deferred winmm<elf> -PE 7e500000-7e585000 \ winmm ELF 7e585000-7e5bd000 Deferred winspool<elf> -PE 7e590000-7e5bd000 \ winspool ELF 7e5bd000-7e681000 Deferred comctl32<elf> -PE 7e5d0000-7e681000 \ comctl32 ELF 7e681000-7e6dd000 Deferred shlwapi<elf> -PE 7e690000-7e6dd000 \ shlwapi ELF 7e6dd000-7e7ec000 Deferred shell32<elf> -PE 7e6f0000-7e7ec000 \ shell32 ELF 7e7ec000-7e898000 Deferred comdlg32<elf> -PE 7e7f0000-7e898000 \ comdlg32 ELF 7e898000-7e8eb000 Deferred advapi32<elf> -PE 7e8b0000-7e8eb000 \ advapi32 ELF 7e8eb000-7e989000 Deferred gdi32<elf> -PE 7e900000-7e989000 \ gdi32 ELF 7e989000-7ead4000 Deferred user32<elf> -PE 7e9a0000-7ead4000 \ user32 ELF 7edc2000-7eef1000 Deferred kernel32<elf> -PE 7ede0000-7eef1000 \ kernel32 ELF 7eef1000-7eefc000 Deferred libnss_files.so.2 ELF 7eefc000-7ef13000 Deferred libnsl.so.1 ELF 7ef13000-7ef38000 Deferred libm.so.6 ELF 7ef3e000-7ef58000 Deferred version<elf> -PE 7ef40000-7ef58000 \ version ELF 7ef58000-7f000000 Deferred ntdll<elf> -PE 7ef70000-7f000000 \ ntdll ELF b7ca3000-b7cad000 Deferred libnss_nis.so.2 ELF b7cae000-b7cb2000 Deferred libdl.so.2 ELF b7cb2000-b7def000 Deferred libc.so.6 ELF b7def000-b7e07000 Deferred libpthread.so.0 ELF b7e07000-b7e10000 Deferred libnss_compat.so.2 ELF b7e27000-b7f5e000 Deferred libwine.so.1 ELF b7f5f000-b7f7c000 Deferred ld-linux.so.2 Threads: process tid prio (all id:s are in hex) 0000000c 00000012 0 0000000e 0 0000000d 0 0000000f 00000015 0 00000014 0 00000011 0 00000010 0 0000001c (D) <full game path> 00000025 0 00000024 0 00000023 0 00000022 0 00000021 0 00000020 15 0000001d 0 <== 0000001e 0000001f 0 Backtrace: =>1 0x00429c18 in <game name> (+0x29c18) (0x006ef710) 2 0x0041b782 in <game name> (+0x1b782) (0x006ef74c) 3 0x0041f2bc in <game name>(+0x1f2bc) (0x00009c8a) 4 0x00000000 (0x00000000)
Should I open a new bug for it or is non-working DirectMusic common knowledge (I've seen a few similar open bugs) ?
http://bugs.winehq.org/show_bug.cgi?id=13090
--- Comment #21 from Rafał Mużyło galtgendo@o2.pl 2008-05-13 08:50:09 --- As default is hw:0, this simply means that for pulseaudio, direct hardware access is impossible while using alsa sound plugin, it has to be done via software.
http://bugs.winehq.org/show_bug.cgi?id=13090
--- Comment #22 from Rafał Mużyło galtgendo@o2.pl 2008-05-13 09:03:20 --- However UseDirectHW is useless here, cause the failure happens while it's already false.
http://bugs.winehq.org/show_bug.cgi?id=13090
--- Comment #23 from Rafał Mużyło galtgendo@o2.pl 2008-05-13 12:11:52 --- OK, finally googling payed off. Following the non-advice at https://bugtrack.alsa-project.org/alsa-bug/view.php?id=3943 it did following: in dlls/winealsa.drv/waveout.c inside wodUpdatePlayedTotal right before InterlockedExchange set delay=0; Yes it's a hack. Yes, it possibly breaks something. However, now sound works even with hw:0.
http://bugs.winehq.org/show_bug.cgi?id=13090
--- Comment #24 from Rafał Mużyło galtgendo@o2.pl 2008-05-13 17:43:25 --- Oh, and sorry for pasting the backtrace, I just now noticed the line about attaching them.
http://bugs.winehq.org/show_bug.cgi?id=13090
Vitaliy Margolen vitaliy@kievinfo.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |cplusplus328@gmail.com
--- Comment #25 from Vitaliy Margolen vitaliy@kievinfo.com 2008-05-17 22:08:48 --- *** Bug 13287 has been marked as a duplicate of this bug. ***
http://bugs.winehq.org/show_bug.cgi?id=13090
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #26 from Alexandre Julliard julliard@winehq.org 2008-05-23 10:37:14 --- Closing bugs fixed in 1.0-rc2.
-
wine-bugs＠winehq.org