Bug 104197 - Some Wine programs fail to start after mesa commit af2c320190
Summary: Some Wine programs fail to start after mesa commit af2c320190
Status: RESOLVED FIXED
Alias: None
Product: Mesa
Classification: Unclassified
Component: Drivers/DRI/i965 (show other bugs)
Version: git
Hardware: Other All
: medium normal
Assignee: Francisco Jerez
QA Contact: Intel 3D Bugs Mailing List
URL:
Whiteboard:
Keywords: bisected, regression
Depends on:
Blocks:
 
Reported: 2017-12-10 18:25 UTC by aguertin+freedesktop
Modified: 2017-12-22 00:33 UTC (History)
3 users (show)

See Also:
i915 platform:
i915 features:


Attachments

Description aguertin+freedesktop 2017-12-10 18:25:12 UTC
After af2c320190, running either League of Legends or the Blizzard Launcher (through wine) causes those programs to crash almost immediately upon startup.

There may be other programs with problems, those are just the two I've noticed. It may not be restricted to wine, either.

Unfortunately I'm not good at getting debugging information out of wine (especially from programs that have their own crash handlers) so I don't have much information beyond the bisection.


Bisect output:
af2c320190f3c73180f1610c8df955a7fa2a4d09 is the first bad commit
commit af2c320190f3c73180f1610c8df955a7fa2a4d09
Author: Francisco Jerez <currojerez@riseup.net>
Date:   Thu Jun 15 15:23:57 2017 -0700

    intel/fs: Implement GRF bank conflict mitigation pass.
[...]
:040000 040000 f68427a37c7d2d4a8fa696449fa723c7995d2e68 169b720ac98eac61cef120e37212467fa689c938 M	src


System information:
Ivybridge (i7-3770)
Gentoo
kernel 4.13.4
libdrm 2.4.88
xorg-server 1.19.5
Comment 1 Francisco Jerez 2017-12-10 18:52:24 UTC
(In reply to bugzilla from comment #0)
> After af2c320190, running either League of Legends or the Blizzard Launcher
> (through wine) causes those programs to crash almost immediately upon
> startup.
> 

Can you get me an apitrace reproducing the issue?

Thanks.
Comment 2 aguertin+freedesktop 2017-12-10 21:11:03 UTC
(In reply to Francisco Jerez from comment #1)
> Can you get me an apitrace reproducing the issue?

I've tried but I haven't been able to so far. 

Running
LD_PRELOAD=/usr/lib32/apitrace/wrappers/glxtrace.so wine LeagueClient.exe
gives me an empty file for output. Trying with the 64 bit wrapper gives no output file at all.

apitrace works fine on glxgears.

I'm not very familiar with this so I may be doing something wrong.
Comment 3 Francisco Jerez 2017-12-11 07:00:44 UTC
(In reply to bugzilla from comment #2)
> (In reply to Francisco Jerez from comment #1)
> > Can you get me an apitrace reproducing the issue?
> 
> I've tried but I haven't been able to so far. 
> 

A shader dump (e.g. using the MESA_SHADER_CAPTURE_PATH environment variable) may be sufficient for me to reproduce the issue.
Comment 4 Eero Tamminen 2017-12-11 16:32:54 UTC
I saw this when using Valgrind for another program/problem:
------------------------------------------------------
==21860== Conditional jump or move depends on uninitialised value(s)
==21860==    at 0x6DDCB08: fs_visitor::bank_conflict_cycles(fs_inst const*) const (brw_fs_bank_conflicts.cpp:906)
==21860==    by 0x6E5981F: fs_instruction_scheduler::issue_time(backend_instruction*) (brw_schedule_instructions.cpp:1546)
==21860==    by 0x6E5F57B: instruction_scheduler::schedule_instructions(bblock_t*) (brw_schedule_instructions.cpp:1602)
==21860==    by 0x6E5F7B4: instruction_scheduler::run(cfg_t*) (brw_schedule_instructions.cpp:1704)
==21860==    by 0x6E5F9B0: fs_visitor::schedule_instructions(instruction_scheduler_mode) (brw_schedule_instructions.cpp:1730)
==21860==    by 0x6DFB8B5: fs_visitor::allocate_registers(unsigned int, bool) (brw_fs.cpp:6071)
==21860==    by 0x6DFE1BC: fs_visitor::run_fs(bool, bool) (brw_fs.cpp:6454)
==21860==    by 0x6DFF1CA: brw_compile_fs (brw_fs.cpp:6789)
==21860==    by 0x6EB170C: blorp_compile_fs (blorp.c:194)
==21860==    by 0x6EC179C: brw_blorp_get_blit_kernel (blorp_blit.c:1331)
==21860==    by 0x6EC179C: try_blorp_blit (blorp_blit.c:1859)
==21860==    by 0x6EC179C: do_blorp_blit (blorp_blit.c:2000)
==21860==    by 0x6EC4C12: blorp_copy (blorp_blit.c:2525)
==21860==    by 0x6B81E1E: copy_buffer_to_image (anv_blorp.c:391)
------------------------------------------------------
Comment 5 Francisco Jerez 2017-12-12 04:39:56 UTC
Can you give a try to this [1] patch I attached to bug #104199 to make sure this is not a duplicate?

[1] https://bugs.freedesktop.org/attachment.cgi?id=136094
Comment 6 aguertin+freedesktop 2017-12-13 15:38:06 UTC
I tried with git master as of just now (dc07bb5fd1), which looks like it should already include the patch from bug 104199. It does not fix my crash.

MESA_SHADER_CAPTURE_PATH gives only a single output file, with the following contents:

[require]
GL_ARB_vertex_program

[vertex program]
!!ARBvp1.0
PARAM C[66] = { program.env[0..65] };
ADDRESS A0;PARAM zero = {0.0, 0.0, 0.0, 0.0};
ARL A0.x, zero.x;
MOV result.position, C[A0.x + 65];
END
Comment 7 Francisco Jerez 2017-12-13 19:00:40 UTC
I can't reproduce the crash with that shader.  Can you get me your config.log, and a copy of your Mesa binary libraries? (including i965_dri.so)
Comment 8 Francisco Jerez 2017-12-17 22:28:40 UTC
Can you confirm whether the patch [1] I just attached to bug 104226 fixes the crash?  This is most likely the same issue.

[1] https://bugs.freedesktop.org/attachment.cgi?id=136240
Comment 9 Francisco Jerez 2017-12-22 00:33:45 UTC
Should be fixed in master assuming that this is the same problem described in bug #104226.  Please reopen if things still don't work for you.


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.