[Bug 91117] New: Nimbus (running in wine) has rendering issues, objects are semi-transparent

[Bug 91117] New: Nimbus (running in wine) has rendering issues, objects are semi-transparent

[bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ]

[-- Attachment #1.1: Type: text/plain, Size: 2305 bytes --]

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

            Bug ID: 91117
           Summary: Nimbus (running in wine) has rendering issues, objects
                    are semi-transparent
           Product: Mesa
           Version: 10.5
          Hardware: Other
               URL: http://store.steampowered.com/app/50000/
                OS: All
            Status: NEW
          Keywords: bisected, regression
          Severity: normal
          Priority: medium
         Component: Drivers/DRI/nouveau
          Assignee: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
          Reporter: gyebro69-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org
        QA Contact: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
                CC: imirkin-FrUbXkNCsVf2fBVCVOL8/A@public.gmane.org

The game is available on Steam and needs Wine to run.
The main menu and all the objects in the game appear transparently, see this
screenshot for comparison (images on the right show the correct behaviour,
images on the left is the current state): http://imgur.com/aDhYehq

The problem doesn't occur with the binary Nvidia drivers (340.76).

Regression introduced by

commit 9e94b87b6012450d714edc6d0c46b15a89d5ce61
Author: Ilia Mirkin <imirkin-FrUbXkNCsVf2fBVCVOL8/A@public.gmane.org>
Date:   Thu Jan 1 01:01:13 2015 -0500

    nv50/ir: fold MAD when one of the multiplicands is const

    Fold MAD dst, src0, immed, src2 (or src0/immed swapped) when
     - immed = 0 -> MOV dst, src2
     - immed = +/- 1 -> ADD dst, src0, src2

    These types of MAD patterns were observed in some st/nine shaders.

Reverting the commit on current git fixes the problem for me.

Trace file generated by apitrace:
https://drive.google.com/open?id=0B-tTbLKBl-tObmNTbmZLV08xMzA&authuser=0

To reproduce the problem in the demo version you need Wine (preferably the
latest development version) and a Steam account. The game needs native d3dx9_36
('winetricks d3dx9_36').

Fedora 22 32-bit
Mesa 10.6-branchpoint-676-g7de8569
VGA compatible controller: NVIDIA Corporation G92 [GeForce GTS 250] (rev a2)
(prog-if 00 [VGA controller])
Kernel 4.0.5-300.fc22.i686+PAE
xorg-x11-server-Xorg-1.17.2-1.fc22.i686

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.

[-- Attachment #1.2: Type: text/html, Size: 3878 bytes --]

[-- Attachment #2: Type: text/plain, Size: 153 bytes --]

_______________________________________________
Nouveau mailing list
Nouveau@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nouveau

[Bug 91117] Nimbus (running in wine) has rendering issues, objects are semi-transparent

[bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ]

[-- Attachment #1.1: Type: text/plain, Size: 556 bytes --]

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

--- Comment #1 from Ilia Mirkin <imirkin-FrUbXkNCsVf2fBVCVOL8/A@public.gmane.org> ---
The trace starts working when applying

http://patchwork.freedesktop.org/patch/53049/

Note that the bisected commit merely enables the code fixed by the above to
operate on the shaders in question rather than actually having a problem by
itself. The resulting code ended up as "b - a" instead of "a - b".

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.

[-- Attachment #1.2: Type: text/html, Size: 1445 bytes --]

[-- Attachment #2: Type: text/plain, Size: 153 bytes --]

_______________________________________________
Nouveau mailing list
Nouveau@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nouveau

[Bug 91117] Nimbus (running in wine) has rendering issues, objects are semi-transparent

[bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ]

[-- Attachment #1.1: Type: text/plain, Size: 1279 bytes --]

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

Ilia Mirkin <imirkin-FrUbXkNCsVf2fBVCVOL8/A@public.gmane.org> changed:

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

--- Comment #2 from Ilia Mirkin <imirkin-FrUbXkNCsVf2fBVCVOL8/A@public.gmane.org> ---
Fix pushed. If this trace still gives you issues, feel free to reopen.

commit ad62ec8316a926682958e7ab52639992867c3755
Author: Ilia Mirkin <imirkin-FrUbXkNCsVf2fBVCVOL8/A@public.gmane.org>
Date:   Fri Jun 26 15:01:22 2015 -0400

    nv50/ir: propagate modifier to right arg when const-folding mad

    An immediate has to be the second arg of an ADD operation. However we
    were mistakenly propagating the modifier of the non-folded value to the
    folded immediate argument.

    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91117
    Signed-off-by: Ilia Mirkin <imirkin-FrUbXkNCsVf2fBVCVOL8/A@public.gmane.org>
    Cc: "10.5 10.6" <mesa-stable-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.

[-- Attachment #1.2: Type: text/html, Size: 3098 bytes --]

[-- Attachment #2: Type: text/plain, Size: 153 bytes --]

_______________________________________________
Nouveau mailing list
Nouveau@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nouveau