Bug 105379 - The Witcher: Enhanced Edition under Wine fails to launch
Summary: The Witcher: Enhanced Edition under Wine fails to launch
Status: RESOLVED FIXED
Alias: None
Product: Mesa
Classification: Unclassified
Component: Drivers/Gallium/radeonsi (show other bugs)
Version: git
Hardware: x86-64 (AMD64) Linux (All)
: medium normal
Assignee: Default DRI bug account
QA Contact: Default DRI bug account
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-03-07 09:31 UTC by Andrew Sheldon
Modified: 2018-09-03 13:40 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
Wine log with Mesa git (503 bytes, text/plain)
2018-03-07 09:31 UTC, Andrew Sheldon
Details
Wine log with mesa git and reverted commit (1.10 KB, text/plain)
2018-03-07 09:32 UTC, Andrew Sheldon
Details

Description Andrew Sheldon 2018-03-07 09:31:57 UTC
Created attachment 137850 [details]
Wine log with Mesa git

The Witcher run under Wine fails to launch. Essentially, it hangs indefinitely before any video output but the game can be closed with Ctrl-C, and no ill effects are seen otherwise. Reverting commit b650fc09c3a35ca624aad5fe4b5c34867708f116 in Mesa works around the issue. The game also works fine with a Nvidia 980ti and 390.25 binary drivers.

Hardware: Radeon RX 550
Software: 4.15.6 kernel [DC=0], Mesa git, Wine 3.3

Commit message:

commit b650fc09c3a35ca624aad5fe4b5c34867708f116                                                                                                                                                                                        
Author: Nicolai Hähnle <nicolai.haehnle@amd.com>                                                                                                                                                                                       
Date:   Sun Oct 22 17:38:42 2017 +0200

    radeonsi: fix potential use-after-free of debug callbacks
    
    Found by inspection.
    
    Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Comment 1 Andrew Sheldon 2018-03-07 09:32:29 UTC
Created attachment 137851 [details]
Wine log with mesa git and reverted commit
Comment 2 Matias N. Goldberg 2018-03-09 21:53:08 UTC
I'm no mesa dev but could you test with and before bc65dcab3bc48673ff6180afb036561a4b8b1119 ?

This might be a dupe of https://bugs.freedesktop.org/show_bug.cgi?id=105339

Unless you're sure your error started happening since b650fc09c3a35ca624aad5fe4b5c34867708f116
Comment 3 Andrew Sheldon 2018-03-10 03:58:20 UTC
(In reply to Matias N. Goldberg from comment #2)
> I'm no mesa dev but could you test with and before
> bc65dcab3bc48673ff6180afb036561a4b8b1119 ?
> 
> This might be a dupe of https://bugs.freedesktop.org/show_bug.cgi?id=105339
> 
> Unless you're sure your error started happening since
> b650fc09c3a35ca624aad5fe4b5c34867708f116

I'm certain it's that particular commit since I've been using a reverted patch of that commit with my system Mesa for the last few days.

I.e. this:

diff --git a/src/gallium/drivers/radeonsi/si_pipe.c b/src/gallium/drivers/radeonsi/si_pipe.c                      
index 7c50ffae6a..7940fc8d5b 100644
--- a/src/gallium/drivers/radeonsi/si_pipe.c
+++ b/src/gallium/drivers/radeonsi/si_pipe.c
@@ -158,10 +158,6 @@ static void si_set_debug_callback(struct pipe_context *ctx,
                                  const struct pipe_debug_callback *cb)
 {
        struct si_context *sctx = (struct si_context *)ctx;
-       struct si_screen *screen = sctx->screen;
-
-       util_queue_finish(&screen->shader_compiler_queue);
-       util_queue_finish(&screen->shader_compiler_queue_low_priority);
 
        if (cb)
                sctx->debug = *cb;
Comment 4 Timothy Arceri 2018-08-31 08:31:34 UTC
The game launches fine for me with mesa from git are you still having issues?
Comment 5 Timothy Arceri 2018-08-31 09:17:24 UTC
Yeah I suspect the following commit fixed it:

commit 7083ac7290a0c37a45494437a45441112f3cc36c
Author: Marek Olšák <marek.olsak@amd.com>
Date:   Tue Apr 24 17:01:35 2018 -0400

    util/u_queue: fix a deadlock in util_queue_finish
    
    Cc: 18.0 18.1 <mesa-stable@lists.freedesktop.org>
    Reviewed-by: Nicolai Hähnle <nicolai.haehnle@amd.com>

Feel free to reopen if you are still having problems.
Comment 6 Andrew Sheldon 2018-09-03 13:10:55 UTC
It's still broken for me actually, and reverting the commit original commit works around the issue. Maybe there is something that is specific to my system that is causing the problem.
Comment 7 Andrew Sheldon 2018-09-03 13:40:29 UTC
Scratch that; looks like a manual git build works, but not mesa-9999 in the Gentoo repository. I've tested with minimal cflags too and the problem remains, so I'm not sure what the difference is.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.