Summary: | VBO rendering using glDrawArrays causes program termination and "cs IB too big" message | ||
---|---|---|---|
Product: | Mesa | Reporter: | RCX Slinger <rcxslinger> |
Component: | Drivers/DRI/R600 | Assignee: | Default DRI bug account <dri-devel> |
Status: | RESOLVED FIXED | QA Contact: | |
Severity: | major | ||
Priority: | medium | ||
Version: | git | ||
Hardware: | x86 (IA32) | ||
OS: | Linux (All) | ||
Whiteboard: | |||
i915 platform: | i915 features: | ||
Attachments: |
NeHe lesson 45 source (SDL port), modified to demonstrate bug
try to use base_vtx as index offset for auto_index (no) changes between patched and unpatched mesa reported by piglit |
Description
RCX Slinger
2011-01-01 07:59:43 UTC
Created attachment 41554 [details]
NeHe lesson 45 source (SDL port), modified to demonstrate bug
Created attachment 42437 [details] [review] try to use base_vtx as index offset for auto_index Can you try if the attached patch works (In reply to comment #2) > Created an attachment (id=42437) [details] > try to use base_vtx as index offset for auto_index > > Can you try if the attached patch works Perfect, this solves the bug! Since the patch makes the instruction buffer stay at the same reliable size, it might even solve (any) similar bugs. (it also solves some rendering artefacts noticed but not mentioned) Closing The patch which solves the problem has not been applied to the mesa tree. Created attachment 48270 [details]
(no) changes between patched and unpatched mesa reported by piglit
No changes/regressions detected by piglit when running "r600.tests" and comparing the results between patched and unpatched mesa (from git).
Both mesa builds were compiled to use "mesa classic" instead of gallium (the patch only affects mesa classic, not gallium).
If anyone could merge the patch I would appreciate it.
Thank you.
The patch was merged by Andre Maasikas. Thank you!!! |
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.