Bug 21816 - glean/vertProg1 test fails on some r300 and higher cards
Summary: glean/vertProg1 test fails on some r300 and higher cards
Status: RESOLVED WONTFIX
Alias: None
Product: Mesa
Classification: Unclassified
Component: Drivers/DRI/r300 (show other bugs)
Version: git
Hardware: Other All
: medium normal
Assignee: Default DRI bug account
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-05-19 09:04 UTC by Maciej Cencora
Modified: 2014-07-07 16:43 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
LIT 0^0 fix (5.12 KB, patch)
2009-05-19 09:05 UTC, Maciej Cencora
Details | Splinter Review

Description Maciej Cencora 2009-05-19 09:04:26 UTC
The failing test is "LIT test 2 (degenerate case: 0 ^ 0 -> 1)". The z component of output vector should contain 1.0 value and gives us 0.0.

The attached patch fixes it, but it looks like not all cards are affected.

As it was discussed on IRC it is probably not a bug, but intentional change in HW to conform to DirectX which requires 0.0 result in this case.

This bug was created to collect some test results from different GPUs to track down which GPUs conform to DirectX/OpenGL in this case.

Currently checked cards

OpenGL conformant:
RV350

DirectX conformant:
RV380
RV570
Comment 1 Maciej Cencora 2009-05-19 09:05:18 UTC
Created attachment 26003 [details] [review]
LIT 0^0 fix
Comment 2 Brian Paul 2009-05-19 09:16:24 UTC
Just FYI: I'm thinking of adding a -pedantic flag to glean and to make this sub-test (and some others) depend on that flag.

The 0^0 case is unlikely to come up very often in practice and it's hard to justify the cost of all the extra instructions for LIT just to satisfy this corner case.
Comment 3 Maciej Cencora 2009-05-19 09:53:17 UTC
(In reply to comment #2)
> Just FYI: I'm thinking of adding a -pedantic flag to glean and to make this
> sub-test (and some others) depend on that flag.
> 
> The 0^0 case is unlikely to come up very often in practice and it's hard to
> justify the cost of all the extra instructions for LIT just to satisfy this
> corner case.
> 

I agree, I was also wondering if this new path should be enabled by default or only in full conformance mode.
Comment 4 Adam Jackson 2009-08-24 12:32:15 UTC
Mass version move, cvs -> git
Comment 5 Andreas Boll 2014-07-07 16:43:43 UTC
The classic r300 driver has been abandoned long ago.
It was replaced by the Gallium driver r300g.

If you have issues with r300g please file a new bug report with component Drivers/Gallium/r300

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.