summaryrefslogtreecommitdiffstats
path: root/src/mesa/vbo
Commit message (Collapse)AuthorAgeFilesLines
* mesa: remove //-style commentsBrian Paul2009-12-161-1/+1
|
* vbo: fix array index out of bounds error, and fix evaluator prioritiesBrian Paul2009-12-081-12/+21
| | | | | | | Fixes bug 25525. Plus, the GL_NV_vertex_program evaluators alias and override the convential evaluator maps, so set their state after the conventional maps.
* vbo: fix out-of-bounds array accessBrian Paul2009-11-043-1/+11
| | | | | | | | | | | | The exec->vtx.inputs[] array was being written past its end. This was clobbering the following vbo_exec_context::eval state. Probably not noticed since evaluators and immediate mode rendering don't happen at the same time. Fixed the loop in vbo_exec_vtx_init(). Changed the size of the vbo_exec_context::vtx.arrays[] array. Added a bunch of debug-build assertions. Issue found by Vinson Lee.
* vbo: limit number of warnings to 10Brian Paul2009-09-241-18/+25
| | | | Otherwise some apps will emit tons of warnings.
* mesa: refine the error checking vbo_exec_DrawRangeElements()Brian Paul2009-09-211-4/+31
| | | | | | | | | | If the 'end' index is out of bounds issue a warning as before. But instead of just no-op'ing the draw call, examine the actual array indices to see if they're OK. If the max array index is out of bounds, issue another warning and no-op the draw call. Otherwise, draw normally. This is a debug build-only feature since it could impact performance. This "fixes" the missing torus in the OGL Distilled / Picking demo.
* mesa: Make MultiDrawElements submit multiple primitives at once.Eric Anholt2009-09-012-0/+162
| | | | | | | | | Previously, MultiDrawElements just called DrawElements a bunch of times. By sending several primitives down the pipeline at once, we avoid a bunch of validation. On my GL demo, this improves fps by 2.5% (+/- .41%) and reduces CPU usage by 70.5% (+/- 2.9%) (n=3). Reviewed by: Ian Romanick <[email protected]>
* vbo: Fix array pointer calculation for MapBufferRange-mapped vertex data.Eric Anholt2009-08-281-1/+3
| | | | | We would end up with the offset from the start of the mapping rather than the offset from the start of the buffer.
* Merge branch 'mesa_7_5_branch'Brian Paul2009-08-261-0/+1
|\
| * vbo: fix divide by zero exceptionBrian Paul2009-08-241-1/+1
| | | | | | | | Fixes bug 23489.
* | vbo: fix divide by zero exceptionBrian Paul2009-08-241-1/+1
| | | | | | | | Fixes bug 23489.
* | vbo: call _mesa_valid_to_render()Brian Paul2009-08-141-4/+2
| |
* | vbo: call _mesa_valid_to_render()Brian Paul2009-08-141-20/+1
| |
* | vbo: use _mesa_is_bufferobj()Brian Paul2009-08-124-18/+21
| |
* | vbo: fix incorrect pointerBrian Paul2009-08-121-1/+1
| |
* | vbo: Avoid extra validation of DrawElements.Eric Anholt2009-08-127-75/+94
| | | | | | | | | | | | | | | | | | | | | | | | | | This saves mapping the index buffer to get a bounds on the indices that drivers just drop on the floor in the VBO case (cache win), saves a bonus walk of the indices in the CheckArrayBounds case, and other miscellaneous validation. On intel it's a particularly a large win (50-100% in my app) because even though we let the indices stay in both CPU and GPU caches, we still end up waiting for the GPU to be done with the buffer before reading from it. Drivers that want the min/max_index fields must now check index_bounds_valid and use vbo_get_minmax_index before using them.
* | vbo: Fix build on windows.Michal Krol2009-07-301-1/+1
| |
* | Merge branch 'mesa_7_5_branch'Brian Paul2009-07-131-0/+5
|\|
| * vbo: fix vbo/dlist memory leakBrian Paul2009-07-101-0/+5
| | | | | | | | Based on a patch by [email protected]
* | mesa/vbo: always recalculate min_index and max_index when splitting verticesMaciej Cencora2009-07-081-26/+10
| |
* | Merge branch 'mesa_7_5_branch'Jakob Bornecrantz2009-07-034-24/+38
|\| | | | | | | | | | | Conflicts: src/mesa/main/dlist.c src/mesa/vbo/vbo_save_api.c
| * mesa/vbo: fix compile and replay of nodes ending in a FALLBACKKeith Whitwell2009-06-301-13/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Where vbo save nodes are terminated with a call to DO_FALLBACK(), as in the case of a recursive CallList which is itself within a Begin/End pair, there two problems: 1) The display list node's primitive information was incorrect, stating the cut-off prim had zero vertices 2) On replay, we would get confused by a primitive that started in a node, but was terminated by individual opcodes. This change fixes the first problem by correctly terminating the last primitive on fallback, and the second by forcing the display list to use the Loopback path, converting all nodes into immediate-mode rendering. The loopback fix is a performance hit, but avoiding this would require a fairly large rework of this code.
| * Merge branch 'mesa_7_5_branch' into dlist-statechange-shortcircuitKeith Whitwell2009-06-301-9/+19
| |\ | | | | | | | | | Need this to pick up fixes for per-vertex materials.
| * | mesa/vbo: use _lookup_prim_by_nr for debuggingKeith Whitwell2009-06-304-4/+4
| | | | | | | | | | | | Switch over to specialized enum lookup for primitives
* | | vbo: If MapBufferRange fails, try MapBuffer instead.Corbin Simpson2009-07-021-1/+2
| | | | | | | | | | | | | | | Fixes segfaults with radeon winsys. (Probably libdrm_radeon doing something that it shouldn't.)
* | | Merge branch 'mesa_7_5_branch'Brian Paul2009-06-301-9/+19
|\ \ \ | | |/ | |/| | | | | | | | | | Conflicts: src/mesa/vbo/vbo_exec_draw.c
| * | mesa: Set FLUSH_EXPLICIT_BIT flags when calling FlushMappedBufferRange.José Fonseca2009-06-301-9/+19
| |/ | | | | | | As prescribed by ARB_map_buffer_range.
| * mesa: rename MAX_VERTEX_ATTRIBS to MAX_VERTEX_GENERIC_ATTRIBSBrian Paul2009-06-222-10/+10
| | | | | | | | | | | | Be clearer that this is the number of generic vertex program/shader attributes, not counting the legacy attributes (pos, normal, color, etc). (cherry picked from commit 4a95185c9f30c2de7a03bb1a0653f51b53b1111d)
| * vbo: return VP_NONE from get_program_mode() if running fixed-func vertex programBrian Paul2009-06-221-0/+2
| | | | | | | | | | | | | | | | If we're running a vertex program to emulated fixed-function, we still need to treat vertex arrays/attributes as if we're in fixed-function mode. This should probably be back-ported to Mesa 7.5 after a bit more testing. (cherry picked from commit dda82137d28aba846dda73da230871c115e30aaf)
* | mesa: Make VBO dlist printing use the same path as other dlist printing.Eric Anholt2009-06-191-12/+12
| | | | | | | | | | | | I was rather confused when mesa_print_display_list didn't show any of my glBegin()..glEnd(). Nothing but print_list appears to call this function, so matching its behavior seems like a good idea.
* | mesa: revert some recent VBO buffer object refcounting changesBrian Paul2009-06-152-8/+3
| | | | | | | | | | | | | | | | Reverts part of commit d7ea9ddf5824556e47decac7ba200f37cf1e552f. We were calling _mesa_reference_buffer_object() on some heap-allocated memory that was uninitialized and could trigger an assertion. We can actually go back to "looser" ref counting of the Null/default buffer object in these cases.
* | mesa: use _mesa_reference_buffer_object() in a few placesBrian Paul2009-06-123-4/+10
| |
* | Merge branch 'mesa_7_5_branch'José Fonseca2009-06-111-2/+7
|\|
| * vbo: fix assertion, #define IMM_BUFFER_NAMEBrian Paul2009-06-111-2/+7
| | | | | | | | | | This was sometimes seen when Glean exited upon test failure when using Gallium.
* | Merge branch 'mesa_7_5_branch'Jakob Bornecrantz2009-06-091-8/+21
|\| | | | | | | | | | | | | | | | | Conflicts: Makefile src/mesa/main/version.h src/mesa/shader/slang/slang_preprocess.c src/mesa/state_tracker/st_cb_bufferobjects.c
| * mesa/vbo: drop all references to vbo on destroyKeith Whitwell2009-06-081-8/+21
| | | | | | | | | | | | We were adding references to the input arrays, but failing to drop them on destruction. This could lead to a 64kb buffer being leaked each context destruction.
* | vbo: more glDrawElements debug code (disabled)Brian Paul2009-06-091-3/+74
| |
* | vbo: minor reformattingBrian Paul2009-06-031-16/+23
| |
* | vbo: move/refactor debug codeBrian Paul2009-06-031-38/+49
| |
* | vbo: added debug code to check array data validity (disabled)Brian Paul2009-06-031-0/+156
| |
* | vbo: new debug/dump code (disabled)Brian Paul2009-06-031-0/+58
| |
* | vbo: tweak out-of-bounds VBO access error messageBrian Paul2009-06-031-1/+1
| | | | | | | | Subtract 1 from _MaxElement to be clearer.
* | mesa: fix warning message in vbo_exec_DrawRangeElements()Brian Paul2009-05-221-1/+2
| |
* | mesa: use Elements() for loop limitBrian Paul2009-05-221-3/+2
| |
* | mesa: use Elements() for loop limitBrian Paul2009-05-221-1/+1
| |
* | vbo: s/32/VERT_ATTRIB_MAX/Brian Paul2009-05-221-2/+2
| |
* | vbo: asst. reformatting, clean-upsBrian Paul2009-05-221-34/+46
| |
* | Merge branch 'mesa_7_5_branch'Brian Paul2009-05-221-0/+1
|\|
| * vbo: fix crash in vbo_exec_bind_arrays()Brian Paul2009-05-221-0/+1
| | | | | | | | | | | | | | | | | | When a vertex shader uses generic vertex attribute 0, but not gl_Vertex, we need to set attribute[16] to point to attribute[0]. We were setting the attribute size, but not the pointer. Fixes crash in glsl/multitex.c when using the VertCoord attribute instead of gl_Vertex.
* | mesa: rename MAX_VERTEX_ATTRIBS to MAX_VERTEX_GENERIC_ATTRIBSBrian Paul2009-05-222-10/+10
| | | | | | | | | | Be clearer that this is the number of generic vertex program/shader attributes, not counting the legacy attributes (pos, normal, color, etc).
* | vbo: fix build breakage...oopsBrian Paul2009-05-211-1/+1
| |