diff options
author | Paul Berry <[email protected]> | 2013-09-01 05:49:17 -0700 |
---|---|---|
committer | Paul Berry <[email protected]> | 2013-10-15 11:34:30 -0700 |
commit | caf9cef7eee77f736ff76a65f385bf718efd1dc1 (patch) | |
tree | 771e8a120c3e8436a5327ca7d12d3b89bdded863 /src/mesa/drivers/dri/i965/gen6_wm_state.c | |
parent | 2910a82eb43bafc73d542336c21f415f5f4a3bed (diff) |
i965/fs: Remove bogus field prog_data->dispatch_width.
Despite the name, this field wasn't being set to the dispatch width at
all; it was always 8. The only place it was used was that the
constant buffer read length was aligned to it, and as far as I can
tell from the docs, there is no need to align this value to the
dispatch width; aligning it to a multiple of 8 is sufficient. So I've
just replaced it with a hardcoded 8.
v2: In gen6_wm_state, use brw->wm.base.push_const_size for consistency
with VS and GS state upload.
Reviewed-by: Eric Anholt <[email protected]>
Diffstat (limited to 'src/mesa/drivers/dri/i965/gen6_wm_state.c')
-rw-r--r-- | src/mesa/drivers/dri/i965/gen6_wm_state.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/mesa/drivers/dri/i965/gen6_wm_state.c b/src/mesa/drivers/dri/i965/gen6_wm_state.c index c96a107d5b2..e3395ce422a 100644 --- a/src/mesa/drivers/dri/i965/gen6_wm_state.c +++ b/src/mesa/drivers/dri/i965/gen6_wm_state.c @@ -78,8 +78,7 @@ gen6_upload_wm_push_constants(struct brw_context *brw) printf("\n"); } - brw->wm.base.push_const_size = - ALIGN(prog_data->nr_params, prog_data->dispatch_width) / 8; + brw->wm.base.push_const_size = ALIGN(prog_data->nr_params, 8) / 8; } } @@ -123,8 +122,7 @@ upload_wm_state(struct brw_context *brw) * state flags from gen6_upload_wm_push_constants. */ OUT_BATCH(brw->wm.base.push_const_offset + - ALIGN(brw->wm.prog_data->nr_params, - brw->wm.prog_data->dispatch_width) / 8 - 1); + brw->wm.base.push_const_size - 1); OUT_BATCH(0); OUT_BATCH(0); OUT_BATCH(0); |