[Bug 89944] GPU crash in Civilization 5

[bugzilla-dae...@freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=89944

Marek Olšák  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #10 from Marek Olšák  ---
(In reply to Sami Liedes from comment #9)
> Tested; cannot reproduce problem with latest mesa (28d9e904) and llvm
> (r234939).

Ok. Closing then.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL: 

[Bug 89944] GPU crash in Civilization 5

[bugzilla-dae...@freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=89944

--- Comment #9 from Sami Liedes  ---
Tested; cannot reproduce problem with latest mesa (28d9e904) and llvm
(r234939).

-- 
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL: 

[Bug 89944] GPU crash in Civilization 5

[bugzilla-dae...@freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=89944

--- Comment #8 from Tom Stellard  ---
Can you pull the latest version of llvm and mesa and see if the problem still
exists?

-- 
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL: 

[Bug 89944] GPU crash in Civilization 5

[bugzilla-dae...@freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=89944

Sami Liedes  changed:

   What|Removed |Added

 Attachment #115008|R600_DEBUG=ps,vs,gs output  |R600_DEBUG=ps,vs,gs output
description|from first bad commit   |from first bad commit
   |(R600_DEBUG=ps,vs,gs output |(30f51f1a)
   |from last good commit   |
   |(R600_DEBUG=ps,vs,gs output |
   |from last good commit   |
   |(30f51f1a)  |

-- 
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL: 

[Bug 89944] GPU crash in Civilization 5

[bugzilla-dae...@freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=89944

--- Comment #7 from Sami Liedes  ---
Created attachment 115008
  --> https://bugs.freedesktop.org/attachment.cgi?id=115008=edit
R600_DEBUG=ps,vs,gs output from first bad commit (R600_DEBUG=ps,vs,gs output
from last good commit (R600_DEBUG=ps,vs,gs output from last good commit
(30f51f1a)

-- 
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL: 

[Bug 89944] GPU crash in Civilization 5

[bugzilla-dae...@freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=89944

Sami Liedes  changed:

   What|Removed |Added

 CC||sami.liedes at iki.fi

--- Comment #6 from Sami Liedes  ---
Created attachment 115007
  --> https://bugs.freedesktop.org/attachment.cgi?id=115007=edit
R600_DEBUG=ps,vs,gs output from last good commit (8eb6c109)

-- 
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL: 

[Bug 89944] GPU crash in Civilization 5

[bugzilla-dae...@freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=89944

--- Comment #5 from Tom Stellard  ---
(In reply to Sami Liedes from comment #3)
> I bisected this down to this commit:
> 
> 
> commit 30f51f1a1a70bc838d5bed449daff0dd9f2e8ef2
> Author: Kenneth Graunke 
> Date:   Wed Oct 22 20:48:21 2014 -0700
> 
> glsl: Optimize "if (cond) discard;" to a conditional discard.   
> 
> 
> 
> st_glsl_to_tgsi and ir_to_mesa have handled conditional discards for a  
> 
> long time; the previous patch added that capability to i965.
> 
> 
> 
> i965 (Haswell) shader-db stats: 
> 
> 
> 
> Without NIR:
> 
> total instructions in shared programs: 5792133 -> 5776360 (-0.27%)  
> 
> instructions in affected programs: 737585 -> 721812 (-2.14%)
> 
> helped:6300 
> 
> HURT:  68   
> 
> GAINED:2
> 
> 
> 
> With NIR:   
> 
> total instructions in shared programs: 5787538 -> 5769569 (-0.31%)  
> 
> instructions in affected programs: 767843 -> 749874 (-2.34%)
> helped:6522
> HURT:  35
> GAINED:6
> 
> Signed-off-by: Kenneth Graunke 
> Reviewed-by: Connor Abbott 
> Reviewed-by: Matt Turner 
> Reviewed-by: Eric Anholt 
> 
> 
> I can also confirm that reverting that commit on top of recent HEAD
> (4deca127) fixes the issue.
> 
> I can attach R600_DEBUG=ps,gs,vs output from the offending commit and its
> parent if you think comparing them is of any use

That would be useful.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL: 

[Bug 89944] GPU crash in Civilization 5

[bugzilla-dae...@freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=89944

--- Comment #4 from qaridarium  ---
I think I do have the same problem.

http://www.phoronix.com/forums/showthread.php?116700-HD7850-radeon-oibaf-ppa-Zivilisation-5-crash

I do have a hd7850 and I use the oibaf PPA to run the GIT code

-- 
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL: 

[Bug 89944] GPU crash in Civilization 5

[bugzilla-dae...@freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=89944

Sami Liedes  changed:

   What|Removed |Added

 CC||kenneth at whitecape.org

--- Comment #3 from Sami Liedes  ---
I bisected this down to this commit:


commit 30f51f1a1a70bc838d5bed449daff0dd9f2e8ef2
Author: Kenneth Graunke 
Date:   Wed Oct 22 20:48:21 2014 -0700

glsl: Optimize "if (cond) discard;" to a conditional discard.   

st_glsl_to_tgsi and ir_to_mesa have handled conditional discards for a  
long time; the previous patch added that capability to i965.

i965 (Haswell) shader-db stats: 

Without NIR:
total instructions in shared programs: 5792133 -> 5776360 (-0.27%)  
instructions in affected programs: 737585 -> 721812 (-2.14%)
helped:6300 
HURT:  68   
GAINED:2

With NIR:   
total instructions in shared programs: 5787538 -> 5769569 (-0.31%)  
instructions in affected programs: 767843 -> 749874 (-2.34%)
helped:6522
HURT:  35
GAINED:6

Signed-off-by: Kenneth Graunke 
Reviewed-by: Connor Abbott 
Reviewed-by: Matt Turner 
Reviewed-by: Eric Anholt 


I can also confirm that reverting that commit on top of recent HEAD (4deca127)
fixes the issue.

I can attach R600_DEBUG=ps,gs,vs output from the offending commit and its
parent if you think comparing them is of any use.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL: 

[Bug 89944] GPU crash in Civilization 5

[bugzilla-dae...@freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=89944

--- Comment #2 from Sami Liedes  ---
Created attachment 114937
  --> https://bugs.freedesktop.org/attachment.cgi?id=114937=edit
Non-crashing debug output from mesa 10.4.2, for comparison

-- 
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL: 

[Bug 89944] GPU crash in Civilization 5

[bugzilla-dae...@freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=89944

--- Comment #1 from Sami Liedes  ---
Created attachment 114936
  --> https://bugs.freedesktop.org/attachment.cgi?id=114936=edit
R600_DEBUG=ps,vs,gs output from crashing program

-- 
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL: 

[Bug 89944] GPU crash in Civilization 5

[bugzilla-dae...@freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=89944

Bug ID: 89944
   Summary: GPU crash in Civilization 5
   Product: Mesa
   Version: git
  Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: NEW
  Severity: normal
  Priority: medium
 Component: Drivers/Gallium/radeonsi
  Assignee: dri-devel at lists.freedesktop.org
  Reporter: sami.liedes at iki.fi
QA Contact: dri-devel at lists.freedesktop.org

Created attachment 114935
  --> https://bugs.freedesktop.org/attachment.cgi?id=114935=edit
dmesg

Civilization 5, with all DLC, causes a GPU crash on recent git (490621f) on
Radeon R9 270. The game works perfectly on mesa 10.4.2 from Debian unstable,
though. Hardware and kernel are x86-64, but the game (and obviously mesa) is
32-bit.

The crash happens when the game is loaded, probably where the map would first
get drawn.

Mesa was built in a 32-bit chroot and configured with

   ./configure --prefix=/home/sliedes/local/mesa32 --build=i686-pc-linux-gnu
--host=i686-pc-linux-gnu --with-gallium-drivers=radeonsi
--with-egl-platforms=drm,x11 --enable-driglx-direct --enable-gles1
--enable-gles2 --enable-glx-tls --enable-texture-float

LLVM version is 3.6.0, built in release mode.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL: