diff options
author | Topi Pohjolainen <topi.pohjolainen@intel.com> | 2016-05-15 08:48:13 +0300 |
---|---|---|
committer | Topi Pohjolainen <topi.pohjolainen@intel.com> | 2016-07-04 08:33:41 +0300 |
commit | bdab945edde9d54dbd57659bae5fa5519a4be21f (patch) | |
tree | 7a056abee4c7e912426169d1d3d7a402909e76f8 | |
parent | 7295428e4183cca1840e7ed02159519734fdab0a (diff) |
i965/draw: Expose vertex buffer state setup
Also change the interface to use start and end offsets.
Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
-rw-r--r-- | src/mesa/drivers/dri/i965/brw_draw.h | 13 | ||||
-rw-r--r-- | src/mesa/drivers/dri/i965/brw_draw_upload.c | 42 |
2 files changed, 37 insertions, 18 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_draw.h b/src/mesa/drivers/dri/i965/brw_draw.h index 23d98ef8b4b7..64ad9b5cd660 100644 --- a/src/mesa/drivers/dri/i965/brw_draw.h +++ b/src/mesa/drivers/dri/i965/brw_draw.h @@ -27,9 +27,22 @@ #define BRW_DRAW_H #include "main/mtypes.h" /* for struct gl_context... */ +#include "intel_bufmgr.h" struct brw_context; +uint32_t * +brw_emit_vertex_buffer_state(struct brw_context *brw, + unsigned buffer_nr, + drm_intel_bo *bo, + unsigned start_offset, + unsigned end_offset, + unsigned stride, + unsigned step_rate, + uint32_t *__map); + +#define EMIT_VERTEX_BUFFER_STATE(...) __map = \ + brw_emit_vertex_buffer_state(__VA_ARGS__, __map) void brw_draw_prims(struct gl_context *ctx, const struct _mesa_prim *prims, diff --git a/src/mesa/drivers/dri/i965/brw_draw_upload.c b/src/mesa/drivers/dri/i965/brw_draw_upload.c index bfe20c57324e..623e4ab77099 100644 --- a/src/mesa/drivers/dri/i965/brw_draw_upload.c +++ b/src/mesa/drivers/dri/i965/brw_draw_upload.c @@ -695,15 +695,15 @@ brw_prepare_shader_draw_parameters(struct brw_context *brw) /** * Emit a VERTEX_BUFFER_STATE entry (part of 3DSTATE_VERTEX_BUFFERS). */ -static uint32_t * -emit_vertex_buffer_state(struct brw_context *brw, - unsigned buffer_nr, - drm_intel_bo *bo, - unsigned bo_ending_address, - unsigned bo_offset, - unsigned stride, - unsigned step_rate, - uint32_t *__map) +uint32_t * +brw_emit_vertex_buffer_state(struct brw_context *brw, + unsigned buffer_nr, + drm_intel_bo *bo, + unsigned start_offset, + unsigned end_offset, + unsigned stride, + unsigned step_rate, + uint32_t *__map) { struct gl_context *ctx = &brw->ctx; uint32_t dw0; @@ -728,9 +728,17 @@ emit_vertex_buffer_state(struct brw_context *brw, "VBO stride %d too large, bad rendering may occur\n", stride); OUT_BATCH(dw0 | (stride << BRW_VB0_PITCH_SHIFT)); - OUT_RELOC(bo, I915_GEM_DOMAIN_VERTEX, 0, bo_offset); + OUT_RELOC(bo, I915_GEM_DOMAIN_VERTEX, 0, start_offset); if (brw->gen >= 5) { - OUT_RELOC(bo, I915_GEM_DOMAIN_VERTEX, 0, bo_ending_address); + /* From the BSpec: 3D Pipeline Stages - 3D Pipeline Geometry - + * Vertex Fetch (VF) Stage - State + * + * Instead of "VBState.StartingBufferAddress + VBState.MaxIndex x + * VBState.BufferPitch", the address of the byte immediately beyond the + * last valid byte of the buffer is determined by + * "VBState.EndAddress + 1". + */ + OUT_RELOC(bo, I915_GEM_DOMAIN_VERTEX, 0, end_offset - 1); } else { OUT_BATCH(0); } @@ -738,7 +746,6 @@ emit_vertex_buffer_state(struct brw_context *brw, return __map; } -#define EMIT_VERTEX_BUFFER_STATE(...) __map = emit_vertex_buffer_state(__VA_ARGS__, __map) static void brw_emit_vertices(struct brw_context *brw) @@ -813,18 +820,17 @@ brw_emit_vertices(struct brw_context *brw) */ unsigned padding = (brw->gen <= 7 && !brw->is_baytrail && !brw->is_haswell) * 2; - EMIT_VERTEX_BUFFER_STATE(brw, i, buffer->bo, - buffer->offset + buffer->size + padding - 1, - buffer->offset, buffer->stride, - buffer->step_rate); + EMIT_VERTEX_BUFFER_STATE(brw, i, buffer->bo, buffer->offset, + buffer->offset + buffer->size + padding, + buffer->stride, buffer->step_rate); } if (uses_draw_params) { EMIT_VERTEX_BUFFER_STATE(brw, brw->vb.nr_buffers, brw->draw.draw_params_bo, - brw->draw.draw_params_bo->size - 1, brw->draw.draw_params_offset, + brw->draw.draw_params_bo->size, 0, /* stride */ 0); /* step rate */ } @@ -832,8 +838,8 @@ brw_emit_vertices(struct brw_context *brw) if (brw->vs.prog_data->uses_drawid) { EMIT_VERTEX_BUFFER_STATE(brw, brw->vb.nr_buffers + 1, brw->draw.draw_id_bo, - brw->draw.draw_id_bo->size - 1, brw->draw.draw_id_offset, + brw->draw.draw_id_bo->size, 0, /* stride */ 0); /* step rate */ } |