| Bug ID | 86089 |
|---|---|
| Summary | [r600g] shader failure - r600_sb::bc_finalizer::cf_peephole() when starting Second Life |
| Product | Mesa |
| Version | unspecified |
| Hardware | Other |
| OS | All |
| Status | NEW |
| Severity | normal |
| Priority | medium |
| Component | Drivers/Gallium/r600 |
| Assignee | dri-devel@lists.freedesktop.org |
| Reporter | shawn.starr@rogers.com |
Compiling November 10th build of git master for rv6xx: kernel: 3.18.0-0.rc3.git2.2.fc22.1.x86_64 mesa-dri-drivers-10.4-0.devel.100.20141110.fc21.x86_64 (custom using Fedora's .spec file) I am using R600_DEBUG=hyperz I get the following crash: Program received signal SIGSEGV, Segmentation fault. 0x00007fffe916bb4e in r600_sb::bc_finalizer::cf_peephole (this=this@entry=0x7fffffffb350) at sb/sb_bc_finalize.cpp:877 877 c->jump_target = static_cast<cf_node*>(c->jump_target->next); Full stack dump attached
Created attachment 109185 [details]
Crash from gdb stacktrace
Bisecting came up with this as bad commit: 73dd50acf6d244979c2a657906aa56d3ac60d550 is the first bad commit commit 73dd50acf6d244979c2a657906aa56d3ac60d550 Author: Tapani P=C3=A4lli <tapani.palli@intel.com> Date: Wed Aug 6 09:46:54 2014 +0300 glsl: implement switch flow control using a loop Patch removes old variable based logic for handling a break inside switch. Switch is put inside a loop so that existing infrastructure for loop flow control can be used for the switch, now also dead code elimination works properly. Possible 'continue' call inside a switch needs now special handling which is taken care of by detecting continue, breaking out and calling continue for the outside loop. v2: remove one unnecessary ir_expression (Curro) Fixes following Piglit tests: fs-exec-after-break.shader_test fs-conditional-break.shader_test No Piglit or es3conform regressions. Signed-off-by: Tapani P=C3=A4lli <tapani.palli@intel.com> Reviewed-by: Francisco Jerez <currojerez@riseup.net> :040000 040000 eac8a47066723ff804317ef11bb006dd0bb80d0f b73ed6cc78a66c54d684635f69a358fd54f65929 M src
A second bisect, repeated this. So this is the commit that breaks r600 SB compiler.
| What | Removed | Added |
|---|---|---|
| Summary | [r600g] shader failure - r600_sb::bc_finalizer::cf_peephole() when starting Second Life | [r600g][mesa 10.4.0-dev] shader failure - r600_sb::bc_finalizer::cf_peephole() when starting Second Life |
| What | Removed | Added |
|---|---|---|
| CC | sonichedgehog_hyperblast00@yahoo.com |
*** Bug 86418 has been marked as a duplicate of this bug. ***
From bug 86418: I tried launching the Second Life viewer (Kokua 3.7.8 x64) using the latest GIT version of MESA, and noticed the viewer crashes as soon as it's time to render the world. The crash only happens when Basic Shaders are enabled.
As mentioned in the related bug report, I can confirm this. I did a GIT bisect and the same commit causes this crash for me too. I'm ready to test the fix once one is in place.
Created attachment 110153 [details] [review] fix loops with no repeat.
Patch causes lockup, discounting the other issues now fixed. This looks to be missing something else.
| What | Removed | Added |
|---|---|---|
| Status | NEW | RESOLVED |
| Resolution | --- | FIXED |
Confirming that it's fixed, the SL viewer works again with latest MESA GIT master.
| What | Removed | Added |
|---|---|---|
| Status | RESOLVED | CLOSED |