i965: be clear that the Fallback field is a boolean, not a bitfield
authorBrian Paul <brianp@vmware.com>
Tue, 27 Oct 2009 15:31:31 +0000 (09:31 -0600)
committerBrian Paul <brianp@vmware.com>
Tue, 27 Oct 2009 22:59:44 +0000 (16:59 -0600)
src/mesa/drivers/dri/i965/brw_draw_upload.c
src/mesa/drivers/dri/i965/brw_fallback.c
src/mesa/drivers/dri/i965/brw_state_upload.c

index a3ff6c58d89867205f5b133b5a58fc794c96d0bf..0fefbd9d8142a58075c9fd5a51378f48cfa1a651 100644 (file)
@@ -375,7 +375,7 @@ static void brw_prepare_vertices(struct brw_context *brw)
     * isn't an issue at this point.
     */
    if (brw->vb.nr_enabled >= BRW_VEP_MAX) {
-      intel->Fallback = 1;
+      intel->Fallback = GL_TRUE; /* boolean, not bitfield */
       return;
    }
 
@@ -427,7 +427,7 @@ static void brw_prepare_vertices(struct brw_context *brw)
            /* Position array not properly enabled:
             */
             if (input->glarray->StrideB == 0) {
-               intel->Fallback = 1;
+               intel->Fallback = GL_TRUE; /* boolean, not bitfield */
                return;
             }
 
index d27c6c24ca50258d647484846fc1f4a6d62891fa..562a17844b43ed4793d3682bdbdc0d5a094dcb6a 100644 (file)
@@ -133,7 +133,11 @@ const struct brw_tracked_state brw_check_fallback = {
 
 
 
-/* Not used:
+/**
+ * Called by the INTEL_FALLBACK() macro.
+ * NOTE: this is a no-op for the i965 driver.  The brw->intel.Fallback
+ * field is treated as a boolean, not a bitmask.  It's only set in a
+ * couple of places.
  */
 void intelFallback( struct intel_context *intel, GLuint bit, GLboolean mode )
 {
index b817b741e77fd745b79e76b118f0cecbbbc5d88c..ee447afa62471f56bbe53b96ecbc509cd488b579 100644 (file)
@@ -308,7 +308,7 @@ void brw_validate_state( struct brw_context *brw )
    if (brw->state.dirty.brw & BRW_NEW_CONTEXT)
       brw_clear_batch_cache(brw);
 
-   brw->intel.Fallback = 0;
+   brw->intel.Fallback = GL_FALSE; /* boolean, not bitfield */
 
    /* do prepare stage for all atoms */
    for (i = 0; i < Elements(atoms); i++) {