Nouveau - Mar 2015 - [Bug 89455] New: Unigine Heaven black and white boxes

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

            Bug ID: 89455
           Summary: Unigine Heaven black and white boxes
           Product: Mesa
           Version: git
          Hardware: Other
                OS: All
            Status: NEW
          Severity: normal
          Priority: medium
         Component: Drivers/DRI/nouveau
          Assignee: nouveau at lists.freedesktop.org
          Reporter: lukebenes at hotmail.com
        QA Contact: nouveau at lists.freedesktop.org

Created attachment 114066
  --> https://bugs.freedesktop.org/attachment.cgi?id=114066&action=edit
Example of the rendering issue

With a GeForce GTX 650 and Mesa 10.4, 10.5, and 10.6, Unigine Heaven 4 has
black and white boxes that pop up randomly. 

https://unigine.com/products/heaven/download/

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/nouveau/attachments/20150306/cec1e700/attachment.html>

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

Luke <lukebenes at hotmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #114066|text/plain                  |image/jpeg
          mime type|                            |

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/nouveau/attachments/20150306/d93a4b85/attachment.html>

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

Ilia Mirkin <imirkin at alum.mit.edu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Unigine Heaven black and    |[NVC0/Gallium] Unigine
                   |white boxes                 |Heaven black and white
                   |                            |boxes

--- Comment #1 from Ilia Mirkin <imirkin at alum.mit.edu> ---
Yeah... I thought it was my bad at one point, but I was able to repro the issue
going back to mesa 9.1 or 9.0 iirc (which predates any of my changes). Note
that this does not affect NV50, only NVC0.

The one issue on NVC0 that I'm aware of is _some_ sort of constbuf flushing
problem... might be what's going on here.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/nouveau/attachments/20150306/9f07d4b3/attachment.html>

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

--- Comment #2 from Ilia Mirkin <imirkin at alum.mit.edu> ---
This patch seems to improve things for me (on a GF108). Pretty sure this is
incomplete though.

diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp
index 248f98d..0420bf2 100644
--- a/src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp
+++ b/src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp
@@ -2433,6 +2433,8 @@ Converter::handleInstruction(const struct
tgsi_full_instruction *insn)
          src1 = fetchSrc(1, c);
          geni = mkOp2(op, dstTy, dst0[c], src0, src1);
          geni->subOp = tgsi::opcodeToSubOp(tgsi.getOpcode());
+         if (dstTy == TYPE_F32)
+            geni->ftz = true;
       }
       break;
    case TGSI_OPCODE_MAD:
@@ -2442,7 +2444,9 @@ Converter::handleInstruction(const struct
tgsi_full_instruction *insn)
          src0 = fetchSrc(0, c);
          src1 = fetchSrc(1, c);
          src2 = fetchSrc(2, c);
-         mkOp3(op, dstTy, dst0[c], src0, src1, src2);
+         geni = mkOp3(op, dstTy, dst0[c], src0, src1, src2);
+         if (dstTy == TYPE_F32)
+            geni->ftz = true;
       }
       break;
    case TGSI_OPCODE_MOV:

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/nouveau/attachments/20150428/378b3286/attachment.html>

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

--- Comment #3 from Ilia Mirkin <imirkin at alum.mit.edu> ---
I posted a more complete approach at
http://patchwork.freedesktop.org/patch/48063/

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/nouveau/attachments/20150428/7217e4aa/attachment.html>

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

Ilia Mirkin <imirkin at alum.mit.edu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #4 from Ilia Mirkin <imirkin at alum.mit.edu> ---
This should be fixed by:

commit 6fe0d4f0354418c6e68dd352996e9891ddd4dfd6
Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Tue Apr 28 03:30:08 2015 -0400

    nvc0/ir: flush denorms to zero in non-compute shaders

    This will set the FTZ flag (flush denorms to zero) on all opcodes that
    can take it.

    This resolves issues in Unigine Heaven 4.0 where there were solid-filled
    boxes popping up.

    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89455
    Cc: "10.4 10.5" <mesa-stable at lists.freedesktop.org>
    Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/nouveau/attachments/20150429/cd4493be/attachment.html>

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

Luke <lukebenes at hotmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |VERIFIED

--- Comment #5 from Luke <lukebenes at hotmail.com> ---
Verified Fixed. Thanks for taking care of this Ilia!

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/nouveau/attachments/20150430/9583f094/attachment.html>