Get vertex programs supplied by the user work properly with immediate mode - vertex...

Fix alu_offset in SINGLE_TEXTURE_PIPELINE.

diff --git a/src/mesa/drivers/dri/r300/Makefile b/src/mesa/drivers/dri/r300/Makefile
index 1ea07584aa5c5dcf24f15930e26e9865d5b4a626..f76ffd974de3d3a6ed39af1a52f3b455a8b1e499 100644 (file)
--- a/src/mesa/drivers/dri/r300/Makefile
+++ b/src/mesa/drivers/dri/r300/Makefile
@@ -5,7 +5,7 @@ TOP = ../../../../..
 include $(TOP)/configs/current
 
 LIBNAME = r300_dri.so
-DEFINES += -DCOMPILE_R300
+DEFINES += -DCOMPILE_R300 -DGLX_DIRECT_RENDERING
 
 MINIGLX_SOURCES = server/radeon_dri.c
 

diff --git a/src/mesa/drivers/dri/r300/r300_context.h b/src/mesa/drivers/dri/r300/r300_context.h
index a5d034e0cd494782f680996bb57e090758a12f68..37c58a0f2fd362970fb770a2d21cb762b59d77db 100644 (file)
--- a/src/mesa/drivers/dri/r300/r300_context.h
+++ b/src/mesa/drivers/dri/r300/r300_context.h
@@ -642,6 +642,8 @@ struct r300_state {
        struct r300_pixel_shader_state pixel_shader;
        struct r300_aos_rec aos[R300_MAX_AOS_ARRAYS];
        int aos_count;
+       GLuint render_inputs; /* actual render inputs that R300 was configured for. 
+                                They are the same as tnl->render_inputs for fixed pipeline */
 
        int hw_stencil;
 };

diff --git a/src/mesa/drivers/dri/r300/r300_fixed_pipelines.h b/src/mesa/drivers/dri/r300/r300_fixed_pipelines.h
index 0b74ed82e232e6e322cdf182e3a74454d5ebce63..e10318f954dd831ce799ef10e1a7c88116e705ea 100644 (file)
--- a/src/mesa/drivers/dri/r300/r300_fixed_pipelines.h
+++ b/src/mesa/drivers/dri/r300/r300_fixed_pipelines.h
@@ -263,7 +263,7 @@ static struct r300_pixel_shader_state SINGLE_TEXTURE_PIXEL_SHADER={
                        
                        tex_offset: 0,
                        tex_end: 0,
-                       alu_offset: 1,
+                       alu_offset: 0,
                        alu_end: 0
                        },
                        

diff --git a/src/mesa/drivers/dri/r300/r300_render.c b/src/mesa/drivers/dri/r300/r300_render.c
index 16b7258481b64641d29a88920ef7e16e4e57adbb..7092842e9d68080426e83de24fb10faecc211260 100644 (file)
--- a/src/mesa/drivers/dri/r300/r300_render.c
+++ b/src/mesa/drivers/dri/r300/r300_render.c
@@ -190,7 +190,7 @@ static void r300_render_immediate_primitive(r300ContextPtr rmesa,
 {
    TNLcontext *tnl = TNL_CONTEXT(ctx);
    struct vertex_buffer *VB = &tnl->vb;
-   GLuint i;
+   GLuint i, render_inputs;
    int k, type;
    LOCAL_VARS
                   
@@ -224,6 +224,13 @@ static void r300_render_immediate_primitive(r300ContextPtr rmesa,
        return;
        }
    
+   render_inputs = rmesa->state.render_inputs;
+
+   if(!render_inputs){
+       WARN_ONCE("Aeiee ! render_inputs==0. Skipping rendering.\n");
+       return;
+       }
+
    start_immediate_packet(end-start, type, 4*rmesa->state.aos_count);
 
        for(i=start;i<end;i++){
@@ -243,28 +250,28 @@ static void r300_render_immediate_primitive(r300ContextPtr rmesa,
                
                
                /* coordinates */
-               if(tnl->render_inputs & _TNL_BIT_POS)
+               if(render_inputs & _TNL_BIT_POS)
                        output_vector(VB->ObjPtr, i);
-               if(tnl->render_inputs & _TNL_BIT_NORMAL)
+               if(render_inputs & _TNL_BIT_NORMAL)
                        output_vector(VB->NormalPtr, i);
                
                /* color components */
-               if(tnl->render_inputs & _TNL_BIT_COLOR0)
+               if(render_inputs & _TNL_BIT_COLOR0)
                        output_vector(VB->ColorPtr[0], i);
-               if(tnl->render_inputs & _TNL_BIT_COLOR1)
+               if(render_inputs & _TNL_BIT_COLOR1)
                        output_vector(VB->SecondaryColorPtr[0], i);
 
-               if(tnl->render_inputs & _TNL_BIT_FOG)
+               if(render_inputs & _TNL_BIT_FOG)
                        output_vector(VB->FogCoordPtr, i);
                                        
                /* texture coordinates */
                for(k=0;k < ctx->Const.MaxTextureUnits;k++)
-                       if(tnl->render_inputs & (_TNL_BIT_TEX0<<k))
+                       if(render_inputs & (_TNL_BIT_TEX0<<k))
                                output_vector(VB->TexCoordPtr[k], i);
                
-               if(tnl->render_inputs & _TNL_BIT_INDEX)
+               if(render_inputs & _TNL_BIT_INDEX)
                        output_vector(VB->IndexPtr[0], i);
-               if(tnl->render_inputs & _TNL_BIT_POINTSIZE)
+               if(render_inputs & _TNL_BIT_POINTSIZE)
                        output_vector(VB->PointSizePtr, i);
                }
 
@@ -290,12 +297,12 @@ static GLboolean r300_run_immediate_render(GLcontext *ctx,
 
        if (RADEON_DEBUG == DEBUG_PRIMS)
                fprintf(stderr, "%s\n", __FUNCTION__);
-
+               
    #if 1 /* we need this, somehow */
    /* Flush state - make sure command buffer is nice and large */
    r300Flush(ctx);
    /* Make sure we have enough space */
-   #else 0
+   #else
    /* Count is very imprecize, but should be good upper bound */
    r300EnsureCmdBufSpace(rmesa, rmesa->hw.max_state_size + 4+2+30
        +VB->PrimitiveCount*(1+8)+VB->Count*4*rmesa->state.texture.tc_count+4, __FUNCTION__);

diff --git a/src/mesa/drivers/dri/r300/r300_state.c b/src/mesa/drivers/dri/r300/r300_state.c
index abd87728c942d534446c21b78cf1b35fc4b4d946..5a577b2ce31d63cf57530d8e42add94882e56419 100644 (file)
--- a/src/mesa/drivers/dri/r300/r300_state.c
+++ b/src/mesa/drivers/dri/r300/r300_state.c
@@ -864,47 +864,42 @@ void r300_setup_routing(GLcontext *ctx, GLboolean immediate)
 
                /* All offsets are 0 - for use by immediate mode.
                Should change later to handle vertex buffers */
-       if(r300->current_vp){
+       if(r300->current_vp!=NULL){
 
        /* VERT_ATTRIB_WEIGHT, VERT_ATTRIB_SIX, VERT_ATTRIB_SEVEN, VERT_ATTRIB_GENERIC0,
           VERT_ATTRIB_GENERIC1, VERT_ATTRIB_GENERIC2, VERT_ATTRIB_GENERIC3 */
-       
+       r300->state.render_inputs = 0;
+          
        if(r300->current_vp->inputs[VERT_ATTRIB_POS] != -1){
-               if(tnl->render_inputs & _TNL_BIT_POS){
-                       reg=r300->current_vp->inputs[VERT_ATTRIB_POS];
-                       CONFIGURE_AOS(VB->ObjPtr, 0, i_coords, AOS_FORMAT_FLOAT);
-               }else WARN_ONCE("vp expects pos but none was given\n");
+               reg=r300->current_vp->inputs[VERT_ATTRIB_POS];
+               CONFIGURE_AOS(VB->ObjPtr, 0, i_coords, AOS_FORMAT_FLOAT);
+               r300->state.render_inputs |= _TNL_BIT_POS;
        }
        if(r300->current_vp->inputs[VERT_ATTRIB_NORMAL] != -1){
-               if(tnl->render_inputs & _TNL_BIT_NORMAL){
-                       reg=r300->current_vp->inputs[VERT_ATTRIB_NORMAL];
-                       CONFIGURE_AOS(VB->NormalPtr, 0, i_normal, AOS_FORMAT_FLOAT);
-               }else WARN_ONCE("vp expects normal but none was given\n");
+               reg=r300->current_vp->inputs[VERT_ATTRIB_NORMAL];
+               CONFIGURE_AOS(VB->NormalPtr, 0, i_normal, AOS_FORMAT_FLOAT);
+               r300->state.render_inputs |= _TNL_BIT_NORMAL;
        }
        if(r300->current_vp->inputs[VERT_ATTRIB_COLOR0] != -1){
-               if(tnl->render_inputs & _TNL_BIT_COLOR0){
-                       reg=r300->current_vp->inputs[VERT_ATTRIB_COLOR0];
-                       CONFIGURE_AOS(VB->ColorPtr[0], 0, i_color[0], AOS_FORMAT_FLOAT_COLOR);
-               }else WARN_ONCE("vp expects primary color but none was given\n");
+               reg=r300->current_vp->inputs[VERT_ATTRIB_COLOR0];
+               CONFIGURE_AOS(VB->ColorPtr[0], 0, i_color[0], AOS_FORMAT_FLOAT_COLOR);
+               r300->state.render_inputs |= _TNL_BIT_COLOR0;
        }
        if(r300->current_vp->inputs[VERT_ATTRIB_COLOR1] != -1){
-               if(tnl->render_inputs & _TNL_BIT_COLOR1){
-                       reg=r300->current_vp->inputs[VERT_ATTRIB_COLOR1];
-                       CONFIGURE_AOS(VB->SecondaryColorPtr[0], 0, i_color[1], AOS_FORMAT_FLOAT_COLOR);
-               }else WARN_ONCE("vp expects secondary color but none was given\n");
+               reg=r300->current_vp->inputs[VERT_ATTRIB_COLOR1];
+               CONFIGURE_AOS(VB->SecondaryColorPtr[0], 0, i_color[1], AOS_FORMAT_FLOAT_COLOR);
+               r300->state.render_inputs |= _TNL_BIT_COLOR1;
        }
        if(r300->current_vp->inputs[VERT_ATTRIB_FOG] != -1){
-               if(tnl->render_inputs & _TNL_BIT_FOG){
-                       reg=r300->current_vp->inputs[VERT_ATTRIB_FOG];
-                       CONFIGURE_AOS(VB->FogCoordPtr, 0, i_fog, AOS_FORMAT_FLOAT);
-               }else WARN_ONCE("vp expects fog but none was given\n");
+               reg=r300->current_vp->inputs[VERT_ATTRIB_FOG];
+               CONFIGURE_AOS(VB->FogCoordPtr, 0, i_fog, AOS_FORMAT_FLOAT);
+               r300->state.render_inputs |= _TNL_BIT_FOG;
        }
        for(i=0;i < ctx->Const.MaxTextureUnits;i++) // tex 7 is last 
                if(r300->current_vp->inputs[VERT_ATTRIB_TEX0+i] != -1){
-                       if(tnl->render_inputs & (_TNL_BIT_TEX0<<i)){
-                               reg=r300->current_vp->inputs[VERT_ATTRIB_TEX0+i];
-                               CONFIGURE_AOS(VB->TexCoordPtr[i], 0, i_tex[i], AOS_FORMAT_FLOAT);
-                       }else fprintf(stderr, "vp expects tex%d but none was given\n", i);
+                       reg=r300->current_vp->inputs[VERT_ATTRIB_TEX0+i];
+                       CONFIGURE_AOS(VB->TexCoordPtr[i], 0, i_tex[i], AOS_FORMAT_FLOAT);
+                       r300->state.render_inputs |= _TNL_BIT_TEX0<<i;
                }
 #if 0
        if((tnl->render_inputs & _TNL_BIT_INDEX))
@@ -915,6 +910,8 @@ void r300_setup_routing(GLcontext *ctx, GLboolean immediate)
 #endif 
        }else{
        
+       r300->state.render_inputs = tnl->render_inputs;
+       
        if(tnl->render_inputs & _TNL_BIT_POS)
                CONFIGURE_AOS(VB->ObjPtr, 0, i_coords, AOS_FORMAT_FLOAT);
        if(tnl->render_inputs & _TNL_BIT_NORMAL)
@@ -941,7 +938,8 @@ void r300_setup_routing(GLcontext *ctx, GLboolean immediate)
        r300->state.aos_count=count;
 
        if (RADEON_DEBUG & DEBUG_STATE)
-               fprintf(stderr, "aos_count=%d\n", count);
+               fprintf(stderr, "aos_count=%d render_inputs=%08x\n", count, r300->state.render_inputs);
+               
 
        if(count>R300_MAX_AOS_ARRAYS){
                fprintf(stderr, "Aieee ! AOS array count exceeded !\n");
@@ -1025,11 +1023,19 @@ void r300_setup_routing(GLcontext *ctx, GLboolean immediate)
        R300_STATECHANGE(r300, vic);
        r300->hw.vic.cmd[R300_VIC_CNTL_0]=0x5555;  /* Hard coded value, no idea what it means */
 
-       r300->hw.vic.cmd[R300_VIC_CNTL_1]=R300_INPUT_CNTL_POS
-                                       | R300_INPUT_CNTL_COLOR;
+       r300->hw.vic.cmd[R300_VIC_CNTL_1]=0;
+       
+       if(r300->state.render_inputs & _TNL_BIT_POS)
+               r300->hw.vic.cmd[R300_VIC_CNTL_1]|=R300_INPUT_CNTL_POS;
+       
+       if(r300->state.render_inputs & _TNL_BIT_NORMAL)
+               r300->hw.vic.cmd[R300_VIC_CNTL_1]|=R300_INPUT_CNTL_NORMAL;
+       
+       if(r300->state.render_inputs & _TNL_BIT_COLOR0)
+               r300->hw.vic.cmd[R300_VIC_CNTL_1]|=R300_INPUT_CNTL_COLOR;
 
        for(i=0;i < ctx->Const.MaxTextureUnits;i++)
-               if(ctx->Texture.Unit[i].Enabled)
+               if(r300->state.render_inputs & (_TNL_BIT_TEX0<<i))
                        r300->hw.vic.cmd[R300_VIC_CNTL_1]|=(R300_INPUT_CNTL_TC0<<i);
 
        /* Stage 3: VAP output */
@@ -1039,7 +1045,7 @@ void r300_setup_routing(GLcontext *ctx, GLboolean immediate)
 
        r300->hw.vof.cmd[R300_VOF_CNTL_1]=0;
        for(i=0;i < ctx->Const.MaxTextureUnits;i++)
-               if(ctx->Texture.Unit[i].Enabled)
+               if(r300->state.render_inputs & (_TNL_BIT_TEX0<<i))
                        r300->hw.vof.cmd[R300_VOF_CNTL_1]|=(4<<(3*i));
 
 }
@@ -1181,7 +1187,10 @@ void r300_setup_textures(GLcontext *ctx)
                exit(-1);
                }
        for(i=0;i<mtu;i++){
-               if(ctx->Texture.Unit[i].Enabled){
+               if( ((r300->state.render_inputs & (_TNL_BIT_TEX0<<i))!=0) != ((ctx->Texture.Unit[i].Enabled)!=0) ) {
+                       WARN_ONCE("Mismatch between render_inputs and ctx->Texture.Unit[i].Enabled value.\n");
+                       }
+               if(r300->state.render_inputs & (_TNL_BIT_TEX0<<i)){
                        t=r300->state.texture.unit[i].texobj;
                        //fprintf(stderr, "format=%08x\n", r300->state.texture.unit[i].format);
                        r300->state.texture.tc_count++;
@@ -1323,10 +1332,11 @@ void r300SetupVertexShader(r300ContextPtr rmesa)
 {
        GLcontext* ctx = rmesa->radeon.glCtx;
        
-       if(rmesa->current_vp){
+       if(rmesa->current_vp != NULL){
                r300SetupVertexProgram(rmesa);
                return ;
        }       
+       
        /* Reset state, in case we don't use something */
        ((drm_r300_cmd_header_t*)rmesa->hw.vpp.cmd)->vpu.count = 0;
        ((drm_r300_cmd_header_t*)rmesa->hw.vpi.cmd)->vpu.count = 0;
@@ -1540,9 +1550,9 @@ void r300ResetHwState(r300ContextPtr r300)
 
        r300UpdateCulling(ctx);
 
-       r300_setup_routing(ctx, GL_TRUE);
-
        r300UpdateTextureState(ctx);
+       
+       r300_setup_routing(ctx, GL_TRUE);
        r300_setup_textures(ctx);
        r300_setup_rs_unit(ctx);
 

diff --git a/src/mesa/drivers/dri/r300/r300_texmem.c b/src/mesa/drivers/dri/r300/r300_texmem.c
index fd66c3e0af9a48e3996e836e12cdc3625feb54e9..3cfa4793ebe602ab2825942821bff6d601236502 100644 (file)
--- a/src/mesa/drivers/dri/r300/r300_texmem.c
+++ b/src/mesa/drivers/dri/r300/r300_texmem.c
@@ -47,6 +47,7 @@ SOFTWARE.
 #include "radeon_reg.h"                /* gets definition for usleep */
 #include "r300_context.h"
 #include "r300_state.h"
+#include "r300_cmdbuf.h"
 #include "radeon_ioctl.h"
 /*
 #include "r300_swtcl.h"

diff --git a/src/mesa/drivers/dri/r300/r300_texstate.c b/src/mesa/drivers/dri/r300/r300_texstate.c
index 09864ccad8e99c02bfef29a8f9270fa6147c05dd..76be5c649aaf547493d9bdf622edda0fac0f83cd 100644 (file)
--- a/src/mesa/drivers/dri/r300/r300_texstate.c
+++ b/src/mesa/drivers/dri/r300/r300_texstate.c
@@ -42,6 +42,7 @@ WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 
 #include "r300_context.h"
 #include "r300_state.h"
+#include "r300_ioctl.h"
 #include "radeon_ioctl.h"
 //#include "r300_swtcl.h"
 #include "r300_tex.h"