i915g: Call draw_set_mapped_vertex_buffer from draw_vbo
authorStéphane Marchesin <marcheu@chromium.org>
Sat, 6 Oct 2012 20:08:36 +0000 (13:08 -0700)
committerStéphane Marchesin <marcheu@chromium.org>
Sat, 6 Oct 2012 20:15:04 +0000 (13:15 -0700)
This regressed with the draw rework. Fixes glest and vdrift crash.

src/gallium/drivers/i915/i915_context.c
src/gallium/drivers/i915/i915_state.c

index 42d90ffc0c6a99969c215e69d9b3cbd16c8abeff..0ee6c0eb89d8fcc5ed7a82702a1e3f597e037ff4 100644 (file)
@@ -54,7 +54,7 @@ i915_draw_vbo(struct pipe_context *pipe, const struct pipe_draw_info *info)
    struct i915_context *i915 = i915_context(pipe);
    struct draw_context *draw = i915->draw;
    const void *mapped_indices = NULL;
-
+   unsigned i;
 
    /*
     * Ack vs contants here, helps ipers a lot.
@@ -64,6 +64,16 @@ i915_draw_vbo(struct pipe_context *pipe, const struct pipe_draw_info *info)
    if (i915->dirty)
       i915_update_derived(i915);
 
+   /*
+    * Map vertex buffers
+    */
+   for (i = 0; i < i915->nr_vertex_buffers; i++) {
+      const void *buf = i915->vertex_buffers[i].user_buffer;
+      if (!buf)
+            buf = i915_buffer(i915->vertex_buffers[i].buffer)->data;
+      draw_set_mapped_vertex_buffer(draw, i, buf);
+   }
+
    /*
     * Map index buffer, if present
     */
@@ -92,6 +102,12 @@ i915_draw_vbo(struct pipe_context *pipe, const struct pipe_draw_info *info)
     */
    draw_vbo(i915->draw, info);
 
+   /*
+    * unmap vertex/index buffers
+    */
+   for (i = 0; i < i915->nr_vertex_buffers; i++) {
+      draw_set_mapped_vertex_buffer(i915->draw, i, NULL);
+   }
    if (mapped_indices)
       draw_set_indexes(draw, NULL, 0);
 
index 74a5df484b4612e6085c1db5262ed7f5f24e6f78..aa2971c59cd46c8b1f39cb1b3ba8d177ac757c30 100644 (file)
@@ -936,24 +936,10 @@ static void i915_set_vertex_buffers(struct pipe_context *pipe,
    util_copy_vertex_buffers(i915->vertex_buffers,
                             &i915->nr_vertex_buffers,
                             buffers, count);
-#if 0
-   /* XXX doesn't look like this is needed */
-   /* unmap old */
-   for (i = 0; i < i915->num_vertex_buffers; i++) {
-      draw_set_mapped_vertex_buffer(draw, i, NULL);
-   }
-#endif
 
    /* pass-through to draw module */
    draw_set_vertex_buffers(draw, count, buffers);
 
-   /* map new */
-   for (i = 0; i < count; i++) {
-      const void *buf = buffers[i].user_buffer;
-      if (!buf)
-            buf = i915_buffer(buffers[i].buffer)->data;
-      draw_set_mapped_vertex_buffer(draw, i, buf);
-   }
 }
 
 static void *