| Summary: | [i965] piglit glsl-vs-arrays failure | ||
|---|---|---|---|
| Product: | Mesa | Reporter: | Gordon Jin <gordon.jin> |
| Component: | Drivers/DRI/i965 | Assignee: | Eric Anholt <eric> |
| Status: | VERIFIED FIXED | QA Contact: | |
| Severity: | normal | ||
| Priority: | medium | CC: | idr |
| Version: | git | ||
| Hardware: | Other | ||
| OS: | All | ||
| Whiteboard: | |||
| i915 platform: | i915 features: | ||
| Bug Depends on: | |||
| Bug Blocks: | 29044 | ||
piglit shader/glsl-vs-mov-after-deref fails with the same problem:
Probe at (15,15)
Expected: 0.000000 1.000000 0.000000
Observed: 1.000000 1.000000 1.000000
PIGLIT: {'result': 'fail' }
The test was created to show this failure. It's not the uniforms that are the problem, but array access. glsl-vs-mov-after-deref failed with half of the problems that glsl-vs-arrays has, and is now fixed. commit c686ee0fa7e2298408259f5533b739c7d05c78b8 Author: Eric Anholt <eric@anholt.net> Date: Wed Jul 21 21:45:34 2010 -0700 i965: In the VS, multiply the address reg by the appropriate register size. commit 35bbbf47425244188334a89163191d9f00bdeced Author: Eric Anholt <eric@anholt.net> Date: Fri Jul 23 10:34:29 2010 -0700 i965: Add support for VS relative addressing of temporary arrays. Fixes glsl-vs-arrays. Bug #27388. It passed now. so verified. |
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.
piglit shader/glsl-vs-arrays fails with below output: Probe at (15,15) Expected: 0.000000 1.000000 0.000000 Observed: 0.000000 0.098039 0.000000 PIGLIT: {'result': 'fail' } It may be not regression, at least not recent regression. It passes with software rendering. It passes (with i965) if I change glsl-vs-arrays.vert as below: - temp[index2] = temp[index1]; + temp[2] = temp[3]; So it indicates the assumption of "index1=3, index2=2" are broken and the uniform vars are not passed through.