Summary: | [softpipe] piglit fbo-generatemipmap-array regression | ||
---|---|---|---|
Product: | Mesa | Reporter: | Vinson Lee <vlee> |
Component: | Other | Assignee: | mesa-dev |
Status: | RESOLVED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | medium | CC: | brianp |
Version: | git | ||
Hardware: | x86-64 (AMD64) | ||
OS: | Linux (All) | ||
Whiteboard: | |||
i915 platform: | i915 features: |
Description
Vinson Lee
2012-03-22 21:51:39 UTC
d335e69b9ab8b3068dc402e22e49483f9caab1df is the first bad commit commit d335e69b9ab8b3068dc402e22e49483f9caab1df Author: Brian Paul <brianp@vmware.com> Date: Sat Mar 17 16:30:03 2012 -0600 mesa: rewrite/consolidate code in _mesa_test_texobj_completeness() Merge the mipmap level checking code that was separate cases for 1D, 2D, 3D and CUBE before. Reviewed-by: José Fonseca <jfonseca@vmware.com> Reviewed-by: Eric Anholt <eric@anholt.net> I took another look at the commit, and now noticed numFaces is never set to 6 for cube maps. Perhaps the test or mipmap generation is relying on texture to not be complete. (In reply to comment #2) > I took another look at the commit, and now noticed numFaces is never set to 6 > for cube maps. Good catch. I guess we don't have any tests that exercise incomplete cube maps. I'll fix that. > Perhaps the test or mipmap generation is relying on texture to not be complete. It looks like the bug is in the state tracker's gen-mipmap code. I don't know why this wasn't a problem sooner. I'll have a patch in a bit. Fixed with commit 318669f196ca922337da02af9d72773e76e70b45 |
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.