[Bug 47419] New: Regression in 4.11, Dune 2000 crashes on mission start
https://bugs.winehq.org/show_bug.cgi?id=47419 Bug ID: 47419 Summary: Regression in 4.11, Dune 2000 crashes on mission start Product: Wine Version: 4.11 Hardware: x86 OS: Linux Status: NEW Severity: normal Priority: P2 Component: directx-d3d Assignee: wine-bugs(a)winehq.org Reporter: o.dierick(a)piezo-forte.be Distribution: --- Created attachment 64765 --> https://bugs.winehq.org/attachment.cgi?id=64765 Dune 2000 crash at mission start, wine 4.11 terminal output Hello, Dune 2000 crashes when starting a mission since wine 4.11. Regression test revealed that reverting the following commit on top of wine 4.11 works around the regression: Module: wine Branch: master Commit: 50dfd232a4f2052b767a96e7dd8ec186d15fbe69 URL: https://source.winehq.org/git/wine.git/?a=commit;h=50dfd232a4f2052b767a96e7d... Author: Michael Muller <michael at fds-team.de> Date: Mon Jun 10 23:54:22 2019 +0430 ddraw: Properly set DDCAPS.ddsOldCaps in ddraw7_GetCaps(). Signed-off-by: David Adam <david.adam.cnrs at gmail.com> Signed-off-by: Henri Verbeet <hverbeet at codeweavers.com> Signed-off-by: Alexandre Julliard <julliard at winehq.org> Notes: - Virtual desktop is enabled to 640x400 (retail game resolution is fixed). - DirectDrawRenderer is set to "gdi" in registry. - Game is patched to version 1.06. Regards. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=47419 Olivier F. R. Dierick <o.dierick(a)piezo-forte.be> changed: What |Removed |Added ---------------------------------------------------------------------------- Distribution|--- |Debian Regression SHA1| |50dfd232a4f2052b767a96e7dd8 | |ec186d15fbe69 Keywords| |regression Hardware|x86 |x86-64 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=47419 Olivier F. R. Dierick <o.dierick(a)piezo-forte.be> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |47410 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=47419 Olivier F. R. Dierick <o.dierick(a)piezo-forte.be> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |INVALID Status|NEW |RESOLVED --- Comment #1 from Olivier F. R. Dierick <o.dierick(a)piezo-forte.be> --- Hello, Removing DirectDrawRenderer override from the registry fixes the regression. 'gdi' rendering was necessary years ago to avoid a crash. It doesn't seem required anymore. Default renderer works fine. Resolving INVALID. Regards. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=47419 Bug 47419 depends on bug 47410, which changed state. Bug 47410 Summary: Regression in 4.10, Dune 2000 installer crashes in winevdm https://bugs.winehq.org/show_bug.cgi?id=47410 What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |FIXED -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=47419 --- Comment #2 from Henri Verbeet <hverbeet(a)gmail.com> --- (In reply to Olivier F. R. Dierick from comment #1)
Resolving INVALID.
Probably, but if you're up for it, it may still be interesting to figure out what the issue with the gdi/no3d renderer is. In particular, I guess the application doesn't like one of the caps set in "caps.ddsOldCaps.dwCaps". If you can figure out which one(s), perhaps we can just make sure they aren't set with the gdi/no3d renderer. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=47419 Olivier F. R. Dierick <o.dierick(a)piezo-forte.be> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|INVALID |--- Status|RESOLVED |REOPENED --- Comment #3 from Olivier F. R. Dierick <o.dierick(a)piezo-forte.be> --- (In reply to Henri Verbeet from comment #2)
(In reply to Olivier F. R. Dierick from comment #1)
Resolving INVALID.
Probably, but if you're up for it, it may still be interesting to figure out what the issue with the gdi/no3d renderer is. In particular, I guess the application doesn't like one of the caps set in "caps.ddsOldCaps.dwCaps". If you can figure out which one(s), perhaps we can just make sure they aren't set with the gdi/no3d renderer.
Hello, With a debugging FIXME and filtering the caps one by one the dwCaps that causes trouble to the gdi/no3d renderer is identified as DDCAPS_ALIGNSIZESRC. With this debugging patch the application fails: --- diff --- + FIXME("caps.ddsOldCaps.dwCaps was %#x.\n", caps.ddsOldCaps.dwCaps); caps.ddsOldCaps.dwCaps = caps.ddsCaps.dwCaps; + FIXME("caps.ddsOldCaps.dwCaps wants %#x.\n", caps.ddsOldCaps.dwCaps); + caps.ddsOldCaps.dwCaps &= ~DDCAPS_PALETTE; + caps.ddsOldCaps.dwCaps &= ~DDCAPS_OVERLAY; + caps.ddsOldCaps.dwCaps &= ~DDCAPS_BLTSTRETCH; + caps.ddsOldCaps.dwCaps &= ~DDCAPS_BLTFOURCC; + caps.ddsOldCaps.dwCaps &= ~DDCAPS_BLT; + caps.ddsOldCaps.dwCaps &= ~DDCAPS_ALIGNSTRIDE; + caps.ddsOldCaps.dwCaps &= ~DDCAPS_ALIGNSIZEDEST; + caps.ddsOldCaps.dwCaps &= ~DDCAPS_ALIGNBOUNDARYDEST; + FIXME("caps.ddsOldCaps.dwCaps got %#x.\n", caps.ddsOldCaps.dwCaps); --- end of diff --- --- output --- (...) 002d:fixme:ddraw:ddraw7_GetCaps caps.ddsOldCaps.dwCaps was 0. 002d:fixme:ddraw:ddraw7_GetCaps caps.ddsOldCaps.dwCaps wants 0x8b76. 002d:fixme:ddraw:ddraw7_GetCaps caps.ddsOldCaps.dwCaps got 0x10. (...) 002d:trace:ddraw:ddraw_surface1_Blt iface 0x15a760, dst_rect (0,0)-(640,400), src_surface (nil), src_rect (null), flags 0x1000400, fx 0xacfb00. 002d:trace:ddraw:ddraw_surface7_Blt iface 0x15a750, dst_rect (0,0)-(640,400), src_surface (nil), src_rect (null), flags 0x1000400, fx 0xacfb00. wine: Unhandled page fault on read access to 0x00000014 at address 0x44c5cc (thread 002d), starting debugger... Unhandled exception: page fault on read access to 0x00000014 in 32-bit code (0x0044c5cc). (...) --- end of output --- With this debugging patch the application works: --- diff --- + FIXME("caps.ddsOldCaps.dwCaps was %#x.\n", caps.ddsOldCaps.dwCaps); caps.ddsOldCaps.dwCaps = caps.ddsCaps.dwCaps; + FIXME("caps.ddsOldCaps.dwCaps wants %#x.\n", caps.ddsOldCaps.dwCaps); + caps.ddsOldCaps.dwCaps &= ~DDCAPS_ALIGNSIZESRC; + FIXME("caps.ddsOldCaps.dwCaps got %#x.\n", caps.ddsOldCaps.dwCaps); --- end of diff --- --- output --- (...) 002d:fixme:ddraw:ddraw7_GetCaps caps.ddsOldCaps.dwCaps was 0. 002d:fixme:ddraw:ddraw7_GetCaps caps.ddsOldCaps.dwCaps wants 0x8b76. 002d:fixme:ddraw:ddraw7_GetCaps caps.ddsOldCaps.dwCaps got 0x8b66. (...) 002d:trace:ddraw:ddraw_surface1_Blt iface 0x2d8a890, dst_rect (0,0)-(640,400), src_surface (nil), src_rect (null), flags 0x1000400, fx 0xacfb00. 002d:trace:ddraw:ddraw_surface7_Blt iface 0x2d8a880, dst_rect (0,0)-(640,400), src_surface (nil), src_rect (null), flags 0x1000400, fx 0xacfb00. 002d:trace:ddraw:ddraw_surface1_Restore iface 0x2d8a890. 002d:trace:ddraw:ddraw_surface7_Restore iface 0x2d8a880. (...) --- end of output --- Regards. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=47419 --- Comment #4 from Olivier F. R. Dierick <o.dierick(a)piezo-forte.be> --- Created attachment 64771 --> https://bugs.winehq.org/attachment.cgi?id=64771 WINEDEBUG=+ddraw full terminal output. This is the full terminal output with WINEDEBUG=+ddraw and the debugging patch that filters all caps but DDCAPS_ALIGNSIZESRC. 5.9MB uncompressed. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=47419 --- Comment #5 from Henri Verbeet <hverbeet(a)gmail.com> --- Created attachment 64772 --> https://bugs.winehq.org/attachment.cgi?id=64772 patch Actually, that's DDSCAPS_FLIP. In theory the attached patch should work. Perhaps it makes sense to commit that, I still have to check what native software devices return. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=47419 Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=47419 --- Comment #6 from Olivier F. R. Dierick <o.dierick(a)piezo-forte.be> --- (In reply to Henri Verbeet from comment #5)
Created attachment 64772 [details] patch
Actually, that's DDSCAPS_FLIP. In theory the attached patch should work. Perhaps it makes sense to commit that, I still have to check what native software devices return.
Hello, The attached patch works for Dune 2000 with GDI renderer. Regards. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=47419 Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Staged patchset| |https://github.com/wine-sta | |ging/wine-staging/tree/mast | |er/patches/wined3d-unset-fl | |ip-gdi Status|REOPENED |STAGED -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=47419 --- Comment #7 from Henri Verbeet <hverbeet(a)gmail.com> --- Created attachment 64872 --> https://bugs.winehq.org/attachment.cgi?id=64872 different patch It turns out that the previous patch doesn't quite line up with the caps that Windows reports. This patch should be closer, does it also work? -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=47419 --- Comment #8 from Olivier F. R. Dierick <o.dierick(a)piezo-forte.be> --- Created attachment 64880 --> https://bugs.winehq.org/attachment.cgi?id=64880 Dune 2000 still crashes unless WINEDDSCAPS_FLIP is filtered. (In reply to Henri Verbeet from comment #7)
Created attachment 64872 [details] different patch
It turns out that the previous patch doesn't quite line up with the caps that Windows reports. This patch should be closer, does it also work?
Hello, The game still crashes with the second patch unless the attached patch is applied on top of it. Other than that, it has no negative effect. Regards. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=47419 --- Comment #9 from Henri Verbeet <hverbeet(a)gmail.com> --- (In reply to Olivier F. R. Dierick from comment #8)
The game still crashes with the second patch unless the attached patch is applied on top of it. Other than that, it has no negative effect.
That's unfortunate. Tests on Windows show that WINEDDSCAPS_FLIP always seems to be included. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=47419 Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |leslie_alistair(a)hotmail.com --- Comment #10 from Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> --- Code has been merged which might fix this issue, can you please try again with plain wine? -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=47419 Olivier F. R. Dierick <o.dierick(a)piezo-forte.be> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #64771|0 |1 is obsolete| | --- Comment #11 from Olivier F. R. Dierick <o.dierick(a)piezo-forte.be> --- Created attachment 65039 --> https://bugs.winehq.org/attachment.cgi?id=65039 WINEDEBUG=+ddraw full terminal output. (In reply to Alistair Leslie-Hughes from comment #10)
Code has been merged which might fix this issue, can you please try again with plain wine?
Hello, Tested with current GIT (commit cf5ccf2), still crashing with GDI rendering. Regards. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=47419 Zebediah Figura <z.figura12(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |z.figura12(a)gmail.com --- Comment #12 from Zebediah Figura <z.figura12(a)gmail.com> --- Is there really a reason to keep this patch around? Is there something that's actually broken in Wine? Do we expect the no3d backend to work? (Does the game work without a functional 3d driver on Windows?) -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=47419 Alex Henrie <alexhenrie24@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |alexhenrie24@gmail.com -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=47419 Gijs Vermeulen <gijsvrm@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Regression in 4.11, Dune |Dune 2000 crashes on |2000 crashes on mission |mission start |start | -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=47419 --- Comment #13 from Alex Henrie <alexhenrie24@gmail.com> --- Created attachment 80056 --> http://bugs.winehq.org/attachment.cgi?id=80056 Program to print DirectDraw capabilities on Windows 95 (In reply to Zeb Figura from comment #12)
Do we expect the no3d backend to work? (Does the game work without a functional 3d driver on Windows?)
Dune 2000 runs slowly but without error on my old 486 desktop PC that runs Windows 95 and has Cirrus Logic CL-GD5434 graphics. I was able to start a mission in the game without any problems. Here's a small test program I wrote to confirm that my 486 does not support 3D acceleration. On that computer, it prints: Driver dwCaps=0x08000400 ddsOldCaps=0x40200250 Emulator dwCaps=0xf4408241 ddsOldCaps=0x00421350 DDSCAPS_3DDEVICE is 0x00002000, which is nowhere to be found. On the other hand, DDSCAPS_FLIP, which is 0x00000010, is present in ddsOldCaps in both the driver and the hardware emulation layer. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=47419 --- Comment #14 from Zeb Figura <z.figura12@gmail.com> --- I've looked further into it and done some disassembly. The game has what looks like a bug. It does something like this abbreviated C: static bool use_vidmem = true; /* address 4e6cd0 */ static IDirectDrawSurface *draw_surface; /* address 794190 */ create_primary() { DDSURFACEDESC desc = {sizeof(desc)}; if (use_vidmem) { desc.dwFlags = DDSD_CAPS | DDSD_BACKBUFFERCOUNT; desc.dwCaps = DDSCAPS_FLIP | DDSCAPS_PRIMARYSURFACE | DDSCAPS_COMPLEX; desc.dwBackBufferCount = 1; } IDirectDraw_CreateSurface(ddraw, &desc, &frontbuffer); if (use_vidmem) IDirectDrawSurface_GetAttachedSurface(frontbuffer, &backbuffer); } setup() { ... IDirectDraw2_GetCaps(ddraw, &caps); if (!(caps.ddsCaps.dwCaps & DDSCAPS_FLIP)) use_vidmem = false; create_primary(); IDirectDrawSurface_GetDesc(frontbuffer, &desc); if (!(desc.ddsCaps.dwCaps & DDSCAPS_VIDEOMEMORY)) use_vidmem = false; ... } recreate_stuff() { ... if (backbuffer) IDirectDrawSurface_Release(backbuffer); create_primary(); ... } Problem comes when something later tries to blit to the backbuffer, and checks "backbuffer" for NULL. In the renderer=gdi/no3d case, with DDSCAPS_FLIP, it was created and destroyed, but the pointer was never cleared, so we get a crash. Without DDSCAPS_FLIP, we never created a backbuffer in the first place, so the game handles NULL there. With GL, we have VIDEOMEMORY, so we never clear use_vidmem, so when the reset happens we create a new backbuffer. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=47419 --- Comment #15 from Zeb Figura <z.figura12@gmail.com> --- I asked Alex to run another test, and the Cirrus card puts the frontbuffer in video memory. If the no3d renderer is supposed to be emulating a 2D-only card, we should probably rethink how we report sysmem/vidmem. OTOH Windows 7's ddraw actually seems to also have 2D emulation even in the absence of a 3D card (at least it does so inside qemu without any sort of passthrough). That layer doesn't report DDSCAPS_FLIP from GetCaps() and it puts the frontbuffer in sysmem. So I guess we should probably pick one and stick with it? -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (3)
-
wine-bugs＠winehq.org -
WineHQ Bugzilla -
WineHQ Bugzilla