Summary: | [Bisected ILK] Piglit bugs/point-sprite regressed | ||
---|---|---|---|
Product: | Mesa | Reporter: | lu hua <huax.lu> |
Component: | Drivers/DRI/i965 | Assignee: | Kenneth Graunke <kenneth> |
Status: | VERIFIED FIXED | QA Contact: | |
Severity: | major | ||
Priority: | high | CC: | eric, xunx.fang |
Version: | git | ||
Hardware: | All | ||
OS: | Linux (All) | ||
Whiteboard: | |||
i915 platform: | i915 features: |
Description
lu hua
2012-02-22 18:19:20 UTC
Ian, Eric? I can confirm that reverting this commit on top of latest master makes the test pass. Still not sure why. It turns out that allocating all your slots before computing the VUE map helps. :) Patch on mailing list: http://lists.freedesktop.org/archives/mesa-dev/2012-August/025184.html commit caa4ae5d7d864278ffbf5dbd9c25bb2932e91fc5 Author: Kenneth Graunke <kenneth@whitecape.org> Date: Sun Aug 5 02:30:38 2012 -0700 i965: Allocate dummy slots for point sprites before computing VUE map. Commit f0cecd43d6b6d moved the VUE map computation to be only once, at VS compile time. However, it did so in slightly the wrong place: it made the one call to brw_vue_compute_map happen right before the allocation of dummy slots for replaced point sprite coordinates, causing a different VUE map to be generated (at least on Ironlake). Fixes a regression in Piglit's point-sprite test on Ironlake. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=46489 Signed-off-by: Kenneth Graunke <kenneth@whitecape.org> Reviewed-by: Eric Anholt <eric@anholt.net> Verified. Fixed by commit caa4ae5d7d864278ffbf5dbd9c25bb2932e91fc5 |
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.