nv50: fix constbuf validation
authorChristoph Bumiller <e0425955@student.tuwien.ac.at>
Sat, 31 Jul 2010 19:30:35 +0000 (21:30 +0200)
committerChristoph Bumiller <e0425955@student.tuwien.ac.at>
Wed, 4 Aug 2010 22:50:00 +0000 (00:50 +0200)
We only uploaded up to the highest offset a program would use,
and if the constant buffer isn't changed when a new program is
used, the new program is missing the rest of them.

Might want to introduce a "fill state" for user mem constbufs.

src/gallium/drivers/nv50/nv50_shader_state.c

index f7e635528673bede8612f653cd02740aacd8b9bc..3d5df596ef796c1a1591de14bad1bc6735685249 100644 (file)
@@ -44,7 +44,7 @@ nv50_transfer_constbuf(struct nv50_context *nv50,
    if (!map)
       return;
 
-   count = MIN2(buf->width0, size);
+   count = buf->width0; /* MIN2(buf->width0, size); */
    start = 0;
 
    while (count) {
@@ -92,8 +92,13 @@ nv50_program_validate_data(struct nv50_context *nv50, struct nv50_program *p)
       }
    }
 
+   /* If the state tracker doesn't change the constbuf, and it is first
+    * validated with a program that doesn't use it, this check prevents
+    * it from even being uploaded. */
+   /*
    if (p->parm_size == 0)
       return;
+   */
 
    switch (p->type) {
    case PIPE_SHADER_VERTEX: