http://bugs.winehq.org/show_bug.cgi?id=28214
Summary: Eurofighter Typhoon crashes after starting Product: Wine Version: 1.3.26 Platform: x86 OS/Version: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: directx-d3d AssignedTo: wine-bugs@winehq.org ReportedBy: gyebro69@gmail.com CC: hverbeet@gmail.com Regression SHA1: ace241ad595d1da0f0fc384a7065683fa00cac45
Created an attachment (id=36133) --> (http://bugs.winehq.org/attachment.cgi?id=36133) terminal output
The game is an old (2001) arcade-ish flight sim, utilizing DirectX 7. It has been recently re-released by DotEmu.
The problem: the game crashes right after starting in Wine-1.3.26. After crashing, the process typhoon.exe doesn't exit but it still remains in memory, consuming 100% cpu power until I kill it. The game starts correctly in Wine-1.3.25.
The result of the regression test:
ace241ad595d1da0f0fc384a7065683fa00cac45 is the first bad commit commit ace241ad595d1da0f0fc384a7065683fa00cac45 Author: Henri Verbeet hverbeet@codeweavers.com Date: Mon Aug 1 21:35:41 2011 +0200
wined3d: Resolve source and destination rectangles in wined3d_surface_blt().
:040000 040000 29aac6db06ac6dba74c9b8a12e39c859cf80d483 8076af6d121572a5003492a25753da1d0c237d23 M dlls
The patch cannot be reverted cleanly, git checkout ace241ad595d1da0f0fc384a7065683fa00cac45 >> the game crashes git reset --hard HEAD^ >> the game loads to the main menu
Note: there might be another regression somewhere between 1.3.26 and 1.3.27. If I start the game in 1.3.27 I get an error message: 'Fatal error: failed to create material x_efa_1'. This error message has never seen before.
No demo available. Please let me know if you need a log with specific debug channels enabled.
Fedora 15 x86 Nvidia gfx card / driver 280.13
http://bugs.winehq.org/show_bug.cgi?id=28214
GyB gyebro69@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression
http://bugs.winehq.org/show_bug.cgi?id=28214
--- Comment #1 from Henri Verbeet hverbeet@gmail.com 2011-08-28 09:33:35 CDT --- The log is from 1.3.26, right? It looks like this may be similar to the issue fixed by e1ab5f6e6bf251566289ccf3177ebaf8102cb72b. Does that patch make any difference on top of 1.3.26? If it doesn't, please create a +ddraw,+d3d,+d3d_surface log. If the "failed to create material" message doesn't happen in 1.3.26 you'll want to bisect that to see where it was introduced, it's likely an independent regression.
http://bugs.winehq.org/show_bug.cgi?id=28214
--- Comment #2 from GyB gyebro69@gmail.com 2011-08-28 10:06:53 CDT --- Created an attachment (id=36134) --> (http://bugs.winehq.org/attachment.cgi?id=36134) +ddraw,+d3d,+d3d_surface log (Wine-1.3.26)
The previous log was created with wine-1.3.25-304-gace241a (where the regression appeared). The current debug log was created with 1.3.26.
The patch in bug #28137 didn't resolve the crash (applied on 1.3.26).
I still have to find out the other commit which prevents the game from starting in 1.3.27.
http://bugs.winehq.org/show_bug.cgi?id=28214
--- Comment #3 from GyB gyebro69@gmail.com 2011-08-28 11:21:33 CDT --- I found the other commit causing that error message: http://source.winehq.org/git/wine.git/commit/156d24c63466479e72811ce0bddef1d...
If I revert it on 1.3.27, the game crashes again as it did in 1.3.26.
http://bugs.winehq.org/show_bug.cgi?id=28214
--- Comment #4 from Henri Verbeet hverbeet@gmail.com 2011-08-28 11:32:59 CDT --- That makes some sense. The log suggests it's trying to do a color-keyed blit between two DXT3 surfaces. I'm not sure how that could ever have worked, but in that case commit 156d24c63466479e72811ce0bddef1d5ec754998 should replace the crash with a FIXME in the console. ("Only plain blits supported on compressed surfaces.")
This probably requires some tests to see if this should actually be supported, or if e.g. just the return value is wrong. As a quick test you could try replacing the "if (flags || src_surface == dst_surface)" line with just "if (src_surface == dst_surface)" though.
http://bugs.winehq.org/show_bug.cgi?id=28214
--- Comment #5 from GyB gyebro69@gmail.com 2011-08-28 12:00:04 CDT --- (In reply to comment #4)
This probably requires some tests to see if this should actually be supported, or if e.g. just the return value is wrong. As a quick test you could try replacing the "if (flags || src_surface == dst_surface)" line with just "if (src_surface == dst_surface)" though.
With the modification in wind3d/surface.c the game starts fine and loads to the menu correctly.
http://bugs.winehq.org/show_bug.cgi?id=28214
GyB gyebro69@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download URL| |http://www.fileplanet.com/5 | |8346/50000/fileinfo/Eurofig | |hter-Typhoon-Demo-%28Non-Pl | |ayable%29
--- Comment #6 from GyB gyebro69@gmail.com 2011-08-29 09:07:18 CDT --- After a bit of more googling I found a non-playable demo which is in fact only a demonstration video of game, but the initial startup problem is reproducible in it (download link added to the URL field).
You need to start the demo like this (provided you installed the demo in the default directory): wine typhoon.exe HOVAPPDATA=C:/Program Files/Rage Software/Eurofighter Typhoon Demo
http://bugs.winehq.org/show_bug.cgi?id=28214
--- Comment #7 from Henri Verbeet hverbeet@gmail.com 2011-09-07 12:41:15 CDT --- Created an attachment (id=36277) --> (http://bugs.winehq.org/attachment.cgi?id=36277) patch
Could you give this patch a try?
http://bugs.winehq.org/show_bug.cgi?id=28214
--- Comment #8 from GyB gyebro69@gmail.com 2011-09-07 13:57:51 CDT --- (In reply to comment #7)
Created an attachment (id=36277)
--> (http://bugs.winehq.org/attachment.cgi?id=36277) [details]
patch
Could you give this patch a try?
The patch works and the game loads to the menu correctly.
http://bugs.winehq.org/show_bug.cgi?id=28214
--- Comment #9 from Henri Verbeet hverbeet@gmail.com 2011-09-09 11:05:37 CDT --- Should be fixed by bbb77cba74e98c36cf24b371939dd0e15a3b4c00.
http://bugs.winehq.org/show_bug.cgi?id=28214
GyB gyebro69@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution| |FIXED
--- Comment #10 from GyB gyebro69@gmail.com 2011-09-09 11:34:52 CDT --- The game starts normally in wine-1.3.27-481-gcb55d2a.
Thank you
http://bugs.winehq.org/show_bug.cgi?id=28214
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #11 from Alexandre Julliard julliard@winehq.org 2011-09-09 12:54:57 CDT --- Closing bugs fixed in 1.3.28.
http://bugs.winehq.org/show_bug.cgi?id=28214
GyB gyebro69@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |bbb77cba74e98c36cf24b371939 | |dd0e15a3b4c00