http://bugs.winehq.org/show_bug.cgi?id=14351
Summary: RtlpWaitForCriticalSection times out towards end of tests Product: Wine Version: CVS/GIT Platform: PC-x86-64 OS/Version: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: testcases AssignedTo: wine-bugs@winehq.org ReportedBy: austin.lund@gmail.com
I've been getting a crash all the time in winetest since 1.0. Seems there is not a problem with any of the tests themselves (i.e. running make test works).
The last few lines of output from winetest is:
fixme:xrandr:X11DRV_XRandR_SetCurrentMode Cannot change screen BPP from 32 to 8 fixme:xrandr:X11DRV_XRandR_SetCurrentMode Cannot change screen BPP from 32 to 16 err:x11settings:X11DRV_ChangeDisplaySettingsEx No matching mode found 1440x900x24 @0! (XRandR) err:ntdll:RtlpWaitForCriticalSection section 0x7bc91824 "loader.c: loader_section" wait timed out in thread 0039, blocked by 0036, retrying (60 sec) wine: Critical section 7bc91824 wait failed at address 0x7bc3ab60 (thread 0039), starting debugger... err:seh:raise_exception Unhandled exception code c0000194 flags 0 addr 0x7bc3ab60
I did a bisection and I got it to occur first at this commit:
commit 3434935f44eadecae1e8aa11d08142a40b671978 Author: Hans Leidekker hans@it.vu.nl Date: Mon Jun 23 20:57:46 2008 +0200
wininet: Clear last error upon successful return from HttpSendRequest.
But when reverting this commit, it still happens.
http://bugs.winehq.org/show_bug.cgi?id=14351
Lei Zhang thestig@google.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression
http://bugs.winehq.org/show_bug.cgi?id=14351
--- Comment #1 from Lei Zhang thestig@google.com 2008-07-09 15:18:11 --- Can you run the regression test again just to make sure? Please use a clean .wine directory each time.
http://bugs.winehq.org/show_bug.cgi?id=14351
--- Comment #2 from Austin Lund austin.lund@gmail.com 2008-07-09 20:53:25 ---
From wine-devel:
"Austin Lund" austin.lund@gmail.com writes:
2008/7/7 Austin Lund austin.lund@gmail.com:
2008/7/3 Austin Lund austin.lund@gmail.com:
I get a crash all the time in winetest since 1.0. Seems there is not a problem with any of the tests themselves (i.e. running make test).
The last few lines of output from winetest is:
fixme:xrandr:X11DRV_XRandR_SetCurrentMode Cannot change screen BPP from 32 to 8 fixme:xrandr:X11DRV_XRandR_SetCurrentMode Cannot change screen BPP from 32 to 16 err:x11settings:X11DRV_ChangeDisplaySettingsEx No matching mode found 1440x900x24 @0! (XRandR) err:ntdll:RtlpWaitForCriticalSection section 0x7bc91824 "loader.c: loader_section" wait timed out in thread 0039, blocked by 0036, retrying (60 sec) wine: Critical section 7bc91824 wait failed at address 0x7bc3ab60 (thread 0039), starting debugger... err:seh:raise_exception Unhandled exception code c0000194 flags 0 addr 0x7bc3ab60
The problem is the localspl/spoolss/winspool.drv dependency mess, trying to load one of these dlls causes a crash inside the loader, which then causes the loader section to not be released properly.
-- Alexandre Julliard julliard@winehq.org
http://bugs.winehq.org/show_bug.cgi?id=14351
--- Comment #3 from Austin Lund austin.lund@gmail.com 2008-07-09 21:03:19 --- I ran the bisection again and ended up at a completely different point.
http://bugs.winehq.org/show_bug.cgi?id=14351
--- Comment #4 from Austin Lund austin.lund@gmail.com 2008-07-09 21:34:25 --- But the critical information is that reverting it from the current head fixes it.
The first bad commit is:
commit 4f6f415aaa2dbc4a00b2b1b7dfe0ec5c5ed9a253 Author: Huw Davies huw@codeweavers.com Date: Mon Jun 23 17:12:12 2008 +0100
winspool.drv: Make sure the printer driver registry entry exists.
http://bugs.winehq.org/show_bug.cgi?id=14351
Detlef Riekenberg wine.dev@web.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |wine.dev@web.de, | |huw@codeweavers.com Status|UNCONFIRMED |NEW Ever Confirmed|0 |1
--- Comment #5 from Detlef Riekenberg wine.dev@web.de 2008-07-10 11:11:57 --- (In reply to comment #4)
But the critical information is that reverting it from the current head fixes it.
The first bad commit is:
commit 4f6f415aaa2dbc4a00b2b1b7dfe0ec5c5ed9a253 Author: Huw Davies huw@codeweavers.com Date: Mon Jun 23 17:12:12 2008 +0100
winspool.drv: Make sure the printer driver registry entry exists.
My Patch to revert that commit was not accepted, but a workaround is in the currents test. I'm working to move the related code (*Monitors / *Ports) to localspl, but i'm not sure, if it makes into the release tomorrow.
More informations are here: http://www.winehq.org/pipermail/wine-patches/2008-June/056544.html
http://bugs.winehq.org/show_bug.cgi?id=14351
Detlef Riekenberg wine.dev@web.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED
--- Comment #6 from Detlef Riekenberg wine.dev@web.de 2008-07-17 10:38:29 --- The Patch was reverted.
http://bugs.winehq.org/show_bug.cgi?id=14351
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #7 from Alexandre Julliard julliard@winehq.org 2008-07-25 13:21:59 --- Closing bugs fixed in 1.1.2.
http://bugs.winehq.org/show_bug.cgi?id=14351
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Version|CVS/GIT |unspecified