Bug 27060

Summary: [965] piglit glsl-fs-raytrace failure due to lack of function calls.
Product: Mesa Reporter: Kristof Ralovich <kristof.ralovich>
Component: Drivers/DRI/i965Assignee: Eric Anholt <eric>
Status: RESOLVED FIXED QA Contact:
Severity: major    
Priority: medium CC: kristof.ralovich
Version: git   
Hardware: x86-64 (AMD64)   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:
Bug Depends on:    
Bug Blocks: 29044    
Attachments: misrendered GLSL testcase
updated test case

Description Kristof Ralovich 2010-03-13 10:28:31 UTC
Created attachment 34029 [details]
misrendered GLSL testcase

The attached testcase doesn't render correctly on a GM45.
Comment 1 Kristof Ralovich 2010-03-13 10:31:34 UTC
If you are here, please have a look at https://bugs.freedesktop.org/show_bug.cgi?id=26691 too.
Comment 2 Kristof Ralovich 2010-03-13 10:42:27 UTC
Created attachment 34030 [details]
updated test case

The previous test-case forced SW rendering which hides the bug by rendering correctly.
Comment 3 Eric Anholt 2010-03-14 10:41:58 UTC
commit 48dca99feb394febc3af44e14f23fb12a9cc9204
Author: Eric Anholt <eric@anholt.net>
Date:   Wed Mar 10 11:00:40 2010 -0800

    i965: Add support for the CMP opcode in the GLSL path.
    
    This would be triggered by use of sqrt() along with control flow.
    Fixes piglit-fs-sqrt-branch and a bug in Yo Frankie!.
Comment 4 Kristof Ralovich 2010-03-18 19:41:22 UTC
(In reply to comment #3)
> commit 48dca99feb394febc3af44e14f23fb12a9cc9204
> Author: Eric Anholt <eric@anholt.net>
> Date:   Wed Mar 10 11:00:40 2010 -0800
> 
>     i965: Add support for the CMP opcode in the GLSL path.
> 
>     This would be triggered by use of sqrt() along with control flow.
>     Fixes piglit-fs-sqrt-branch and a bug in Yo Frankie!.
> 

I have re-run the test case with today's Mesa master 8df65e98998b4c104db30cbba8a38be7eb2a9acd (including the above referred patch) and drm master c1c8bbf80b1f734e23996bf805dc78f32ebaf56f and the misrendering still exists. The warning has gone away for the sqrt instruction though. Please run the test case with LIBGL_ALWAYS_SOFTWARE=1 ./fsraytrace to what the normal rendering should be.
Comment 5 Eric Anholt 2010-08-16 23:41:51 UTC
This is fixed with the merge of GLSL2.
Comment 6 Kristof Ralovich 2010-08-18 21:12:21 UTC
I can confirm, it is working on GM45 with upstream mesa. Nice work! Thanks!

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.