summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVladimir Dergachev <volodya@freedesktop.org>2005-01-10 05:24:28 +0000
committerVladimir Dergachev <volodya@freedesktop.org>2005-01-10 05:24:28 +0000
commit8c231d2e28fde9e9218da3da74dc431075e6fd3f (patch)
treede2135367bbd8f469ec31c25651419075f8f5e27
parent6867117b589d46e496886b7cca99c37e6ec5b36b (diff)
Bring vertex buffer code up to date..
-rw-r--r--src/mesa/drivers/dri/r300/r300_cmdbuf.c38
-rw-r--r--src/mesa/drivers/dri/r300/r300_render.c79
2 files changed, 65 insertions, 52 deletions
diff --git a/src/mesa/drivers/dri/r300/r300_cmdbuf.c b/src/mesa/drivers/dri/r300/r300_cmdbuf.c
index 7c57d128ae..dd3389a60d 100644
--- a/src/mesa/drivers/dri/r300/r300_cmdbuf.c
+++ b/src/mesa/drivers/dri/r300/r300_cmdbuf.c
@@ -59,7 +59,7 @@ WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
/**
* Send the current command buffer via ioctl to the hardware.
*/
-int r300FlushCmdBuf(r300ContextPtr r300, const char* caller)
+int r300FlushCmdBufLocked(r300ContextPtr r300, const char* caller)
{
int ret;
int i;
@@ -81,8 +81,6 @@ int r300FlushCmdBuf(r300ContextPtr r300, const char* caller)
r300->cmdbuf.cmd_buf[i]);
}
- LOCK_HARDWARE(&r300->radeon);
-
cmd.buf = (char*)(r300->cmdbuf.cmd_buf + start);
cmd.bufsz = (r300->cmdbuf.count_used - start) * 4;
@@ -97,27 +95,43 @@ int r300FlushCmdBuf(r300ContextPtr r300, const char* caller)
if (cmd.nbox) {
ret = drmCommandWrite(r300->radeon.dri.fd,
DRM_RADEON_CMDBUF, &cmd, sizeof(cmd));
- if (ret) {
- UNLOCK_HARDWARE(&r300->radeon);
- fprintf(stderr, "drmCommandWrite: %d\n", ret);
- exit(-1);
- }
-
+
if (RADEON_DEBUG & DEBUG_SYNC) {
fprintf(stderr, "Syncing in %s\n\n", __FUNCTION__);
radeonWaitForIdleLocked(&r300->radeon);
}
} else {
+ ret = 0;
if (RADEON_DEBUG & DEBUG_IOCTL)
fprintf(stderr, "%s: No cliprects\n", __FUNCTION__);
}
- UNLOCK_HARDWARE(&r300->radeon);
-
r300->cmdbuf.count_used = 0;
r300->cmdbuf.count_reemit = 0;
- return 0;
+ return ret;
+}
+
+
+int r300FlushCmdBuf(r300ContextPtr r300, const char* caller)
+{
+ int ret;
+ int i;
+ drm_radeon_cmd_buffer_t cmd;
+ int start;
+
+ LOCK_HARDWARE(&r300->radeon);
+
+ ret=r300FlushCmdBufLocked(r300, caller);
+
+ UNLOCK_HARDWARE(&r300->radeon);
+
+ if (ret) {
+ fprintf(stderr, "drmRadeonCmdBuffer: %d (exiting)\n", ret);
+ exit(ret);
+ }
+
+ return ret;
}
diff --git a/src/mesa/drivers/dri/r300/r300_render.c b/src/mesa/drivers/dri/r300/r300_render.c
index d7ce53ed38..93cef4eb6d 100644
--- a/src/mesa/drivers/dri/r300/r300_render.c
+++ b/src/mesa/drivers/dri/r300/r300_render.c
@@ -340,9 +340,17 @@ static void upload_vertex_buffer(r300ContextPtr rmesa,
{
TNLcontext *tnl = TNL_CONTEXT(ctx);
struct vertex_buffer *VB = &tnl->vb;
- int offset=0, idx=0;
- int i,j;
+ int idx=0;
+ int i,j,k;
radeonScreenPtr rsp=rmesa->radeon.radeonScreen;
+
+ /* A hack - we don't want to overwrite vertex buffers, so we
+ just use AGP space for them.. Fix me ! */
+ static int offset=0;
+ if(offset>2*1024*1024){
+ //fprintf(stderr, "Wrapping agp vertex buffer offset\n");
+ offset=0;
+ }
/* Not the most efficient implementation, but, for now, I just want something that
works */
/* to do - make single memcpy per column (is it possible ?) */
@@ -351,7 +359,7 @@ static void upload_vertex_buffer(r300ContextPtr rmesa,
{ \
/* Is the data dirty ? */ \
if (v->flags & ((1<<v->size)-1)) { \
- fprintf(stderr, "size=%d vs stride=%d\n", v->size, v->stride); \
+ /* fprintf(stderr, "size=%d vs stride=%d\n", v->size, v->stride); */ \
if(v->size*4==v->stride){\
/* fast path */ \
memcpy(rsp->gartTextures.map+offset, v->data, v->stride*VB->Count); \
@@ -375,6 +383,10 @@ static void upload_vertex_buffer(r300ContextPtr rmesa,
UPLOAD_VECTOR(VB->ObjPtr, REG_COORDS, AOS_FORMAT_FLOAT);
UPLOAD_VECTOR(VB->ColorPtr[0], REG_COLOR0, AOS_FORMAT_FLOAT_COLOR);
+ /* texture coordinates */
+ for(k=0;k < ctx->Const.MaxTextureUnits;k++)
+ if(ctx->Texture.Unit[k].Enabled)
+ UPLOAD_VECTOR(VB->TexCoordPtr[k], REG_TEX0+i, AOS_FORMAT_FLOAT);
*n_arrays=idx;
if(idx>=R300_MAX_AOS_ARRAYS){
@@ -383,16 +395,13 @@ if(idx>=R300_MAX_AOS_ARRAYS){
}
}
-static void r300_render_vb_flat_primitive(r300ContextPtr rmesa,
+static void r300_render_vb_primitive(r300ContextPtr rmesa,
GLcontext *ctx,
int start,
int end,
int prim)
{
- TNLcontext *tnl = TNL_CONTEXT(ctx);
- struct vertex_buffer *VB = &tnl->vb;
- GLuint i;
- int k, type, n_arrays;
+ int type;
LOCAL_VARS
if(end<=start)return; /* do we need to watch for this ? */
@@ -403,7 +412,7 @@ static void r300_render_vb_flat_primitive(r300ContextPtr rmesa,
fire_AOS(PASS_PREFIX end-start, type);
}
-static GLboolean r300_run_vb_flat_render(GLcontext *ctx,
+static GLboolean r300_run_vb_render(GLcontext *ctx,
struct tnl_pipeline_stage *stage)
{
r300ContextPtr rmesa = R300_CONTEXT(ctx);
@@ -411,16 +420,11 @@ static GLboolean r300_run_vb_flat_render(GLcontext *ctx,
struct vertex_buffer *VB = &tnl->vb;
int i, j, n_arrays;
AOS_DATA vb_arrays[R300_MAX_AOS_ARRAYS];
- AOS_DATA vb_arrays2[R300_MAX_AOS_ARRAYS];
LOCAL_VARS
if (RADEON_DEBUG == DEBUG_PRIMS)
fprintf(stderr, "%s\n", __FUNCTION__);
- /* setup array of structures data */
-
- upload_vertex_buffer(rmesa, ctx, vb_arrays, &n_arrays);
- fprintf(stderr, "Using %d AOS arrays\n", n_arrays);
reg_start(R300_RB3D_DSTCACHE_CTLSTAT,0);
e32(0x0000000a);
@@ -428,42 +432,31 @@ static GLboolean r300_run_vb_flat_render(GLcontext *ctx,
reg_start(0x4f18,0);
e32(0x00000003);
- r300_setup_routing(rmesa, ctx, GL_FALSE);
-
+ r300_setup_routing(ctx, GL_FALSE);
+
r300EmitState(rmesa);
-
- FLAT_COLOR_PIPELINE.vertex_shader.matrix[0].length=16;
- memcpy(FLAT_COLOR_PIPELINE.vertex_shader.matrix[0].body.f, ctx->_ModelProjectMatrix.m, 16*4);
- FLAT_COLOR_PIPELINE.vertex_shader.unknown2.length=4;
- FLAT_COLOR_PIPELINE.vertex_shader.unknown2.body.f[0]=0.0;
- FLAT_COLOR_PIPELINE.vertex_shader.unknown2.body.f[1]=0.0;
- FLAT_COLOR_PIPELINE.vertex_shader.unknown2.body.f[2]=1.0;
- FLAT_COLOR_PIPELINE.vertex_shader.unknown2.body.f[3]=0.0;
+ /* setup array of structures data */
+ LOCK_HARDWARE(&(rmesa->radeon));
- program_pipeline(PASS_PREFIX &FLAT_COLOR_PIPELINE);
-
- set_quad0(PASS_PREFIX 1.0,1.0,1.0,1.0);
- set_init21(PASS_PREFIX 0.0,1.0);
+ upload_vertex_buffer(rmesa, ctx, vb_arrays, &n_arrays);
+ //fprintf(stderr, "Using %d AOS arrays\n", n_arrays);
+ for(i=0;i<n_arrays;i++)
+ rmesa->state.aos[i].offset=vb_arrays[i].offset;
for(i=0; i < VB->PrimitiveCount; i++){
GLuint prim = VB->Primitive[i].mode;
GLuint start = VB->Primitive[i].start;
GLuint length = VB->Primitive[i].count;
+
+ /* We need LOAD_VBPNTR to setup AOS_ATTR fields.. */
+ r300EmitLOAD_VBPNTR(rmesa, start);
- /* copy arrays */
- memcpy(vb_arrays2, vb_arrays, sizeof(AOS_DATA)*n_arrays);
- for(j=0;j<n_arrays;j++){
- vb_arrays2[j].offset+=vb_arrays2[j].stride*start*4;
- }
-
- setup_AOS(PASS_PREFIX vb_arrays2, n_arrays);
-
- r300_render_vb_flat_primitive(rmesa, ctx, start, start + length, prim);
+ r300_render_vb_primitive(rmesa, ctx, start, start + length, prim);
}
/* This sequence is required after any 3d drawing packet
- I suspect it work arounds a bug (or deficiency) in hardware */
+ I suspect it works around a bug (or deficiency) in hardware */
reg_start(R300_RB3D_DSTCACHE_CTLSTAT,0);
e32(0x0000000a);
@@ -474,8 +467,10 @@ static GLboolean r300_run_vb_flat_render(GLcontext *ctx,
end_3d(PASS_PREFIX_VOID);
/* Flush state - we are done drawing.. */
- r300Flush(ctx);
- fprintf(stderr, "\n");
+ r300FlushCmdBufLocked(ctx, __FUNCTION__);
+ radeonWaitForIdleLocked(&(rmesa->radeon));
+
+ UNLOCK_HARDWARE(&(rmesa->radeon));
return GL_FALSE;
}
@@ -500,7 +495,11 @@ static GLboolean r300_run_render(GLcontext *ctx,
#if 1
+ #if 0
return r300_run_immediate_render(ctx, stage);
+ #else
+ return r300_run_vb_render(ctx, stage);
+ #endif
#else
return GL_TRUE;
#endif