Summary: | piglit vp-max-array regression | ||
---|---|---|---|
Product: | Mesa | Reporter: | Vinson Lee <vlee> |
Component: | Mesa core | Assignee: | Brian Paul <brianp> |
Status: | CLOSED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | medium | CC: | jfonseca |
Version: | 7.11 | ||
Hardware: | x86 (IA32) | ||
OS: | Linux (All) | ||
Whiteboard: | |||
i915 platform: | i915 features: |
Description
Vinson Lee
2011-08-26 12:44:15 UTC
f9cc6acbfa6f6f91b29a0601436d91a330fdce79 is the first bad commit commit f9cc6acbfa6f6f91b29a0601436d91a330fdce79 Author: Brian Paul <brianp@vmware.com> Date: Wed Apr 27 09:31:30 2011 -0600 st/mesa: raise shader MaxParameters if driver supports more The default value is 64 but drivers usually advertise more, like 4096. Allows ARB vp/fp programs to use more parameters. Reviewed-by: Jose Fonseca <jfonseca@vmware.com> :040000 040000 812e681a5f4e6d7921f676d244b81ff79feeeb09 f18dfdba56466b0219c0ccd3d3139a059fa984ed M src bisect run success Looks like this is actually a bug in the piglit test. I've posted a patch for review to the piglit list. This should be fixed by the following piglit commit: commit c19366f77895608bc4cd0e6b3bbc92b6b17f2c29 Author: Ian Romanick <ian.d.romanick@intel.com> Date: Tue Aug 30 11:05:41 2011 -0700 vp-max-array: Exercise GL_MAX_PROGRAM_PARAMETERS_ARB with a contant array This eliminates the dependency on GL_MAX_PROGRAM_LOCAL_PARAMETERS_ARB. In particular, this makes the test run correctly on implementations where GL_MAX_PROGRAM_LOCAL_PARAMETERS_ARB is less than GL_MAX_PROGRAM_PARAMETERS_ARB. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=40405 Tested-by: Brian Paul <brianp@vmware.com> mesa: 00792e3586746c833ffc9bb65712e38038916e06 (master) piglit: 59502051469ba3648a537cf694b910d5b72c8161 (master) vp-max-array passes now. |
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.