diff options
| author | Kenneth Graunke <[email protected]> | 2013-11-26 16:32:15 -0800 |
|---|---|---|
| committer | Kenneth Graunke <[email protected]> | 2013-12-03 16:24:58 -0800 |
| commit | 26b7b50afeea8ff252501b9c9b10385960b236f0 (patch) | |
| tree | 9b5c36ff4baa99712fb1d7a6dbf0fc3a025fb0d1 | |
| parent | e03994bf47f9741a7a3a6181eb4f4d0cf286f70f (diff) | |
i965: Fix BRW_BATCH_STRUCT to specify RENDER_RING, not UNKNOWN_RING.
I missed this in the boolean -> enum conversion. C cheerfully casts
false -> 0 -> UNKNOWN_RING. On Gen4-5, this causes the render ring
prelude hook to get called in the middle of the batch, which is crazy.
BRW_BATCH_STRUCT is not used on Gen6+.
Fixes regressions since 395a32717df494353703f3581edcd3ba380f16d6
("i965: Introduce an UNKNOWN_RING state.").
Fixes "fips -v glxgears" on Ironlake.
Signed-off-by: Kenneth Graunke <[email protected]>
Reviewed-by: Eric Anholt <[email protected]>
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_state.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_state.h b/src/mesa/drivers/dri/i965/brw_state.h index 2a4b23ceed3..3f7831caebe 100644 --- a/src/mesa/drivers/dri/i965/brw_state.h +++ b/src/mesa/drivers/dri/i965/brw_state.h @@ -173,8 +173,8 @@ void brw_destroy_caches( struct brw_context *brw ); /*********************************************************************** * brw_state_batch.c */ -#define BRW_BATCH_STRUCT(brw, s) intel_batchbuffer_data(brw, (s), \ - sizeof(*(s)), false) +#define BRW_BATCH_STRUCT(brw, s) \ + intel_batchbuffer_data(brw, (s), sizeof(*(s)), RENDER_RING) void *brw_state_batch(struct brw_context *brw, enum state_struct_type type, |