diff options
author | Kenneth Graunke <[email protected]> | 2013-06-25 22:14:04 -0700 |
---|---|---|
committer | Kenneth Graunke <[email protected]> | 2013-08-19 13:16:59 -0700 |
commit | 7e01af662ad12bd2b27034f3ca7687e2986b5dbd (patch) | |
tree | 14e9e1e6c658ed909ccf99beef13cc04b9b5f8c4 /src/mesa/drivers | |
parent | 2b7f876a6ad62ad9a93c0df15cb4be1fcc61d380 (diff) |
i965: Upload separate VS and FS sampler state tables.
Now, each shader stage has a sampler state table that only refers to the
samplers actually used by that problem. This should make the VS table
non-existant or very small.
Signed-off-by: Kenneth Graunke <[email protected]>
Reviewed-by: Paul Berry <[email protected]>
Diffstat (limited to 'src/mesa/drivers')
-rw-r--r-- | src/mesa/drivers/dri/i965/brw_context.h | 1 | ||||
-rw-r--r-- | src/mesa/drivers/dri/i965/brw_wm_sampler_state.c | 20 | ||||
-rw-r--r-- | src/mesa/drivers/dri/i965/gen7_sampler_state.c | 12 |
3 files changed, 15 insertions, 18 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_context.h b/src/mesa/drivers/dri/i965/brw_context.h index 5280b8831fe..fbe00b12fb8 100644 --- a/src/mesa/drivers/dri/i965/brw_context.h +++ b/src/mesa/drivers/dri/i965/brw_context.h @@ -817,6 +817,7 @@ struct brw_context /** Upload a SAMPLER_STATE table. */ void (*upload_sampler_state_table)(struct brw_context *brw, + struct gl_program *prog, uint32_t *sampler_count, uint32_t *sst_offset, uint32_t *sdc_offset); diff --git a/src/mesa/drivers/dri/i965/brw_wm_sampler_state.c b/src/mesa/drivers/dri/i965/brw_wm_sampler_state.c index 78366ca19f4..e8d47d06e9c 100644 --- a/src/mesa/drivers/dri/i965/brw_wm_sampler_state.c +++ b/src/mesa/drivers/dri/i965/brw_wm_sampler_state.c @@ -368,6 +368,7 @@ static void brw_update_sampler_state(struct brw_context *brw, static void brw_upload_sampler_state_table(struct brw_context *brw, + struct gl_program *prog, uint32_t *sampler_count, uint32_t *sst_offset, uint32_t *sdc_offset) @@ -375,11 +376,7 @@ brw_upload_sampler_state_table(struct brw_context *brw, struct gl_context *ctx = &brw->ctx; struct brw_sampler_state *samplers; - /* BRW_NEW_VERTEX_PROGRAM and BRW_NEW_FRAGMENT_PROGRAM */ - struct gl_program *vs = (struct gl_program *) brw->vertex_program; - struct gl_program *fs = (struct gl_program *) brw->fragment_program; - - GLbitfield SamplersUsed = vs->SamplersUsed | fs->SamplersUsed; + GLbitfield SamplersUsed = prog->SamplersUsed; /* ARB programs use the texture unit number as the sampler index, so we * need to find the highest unit used. A bit-count will not work. @@ -394,10 +391,9 @@ brw_upload_sampler_state_table(struct brw_context *brw, 32, sst_offset); memset(samplers, 0, *sampler_count * sizeof(*samplers)); - for (unsigned s = 0; s < brw->wm.sampler_count; s++) { + for (unsigned s = 0; s < *sampler_count; s++) { if (SamplersUsed & (1 << s)) { - const unsigned unit = (fs->SamplersUsed & (1 << s)) ? - fs->SamplerUnits[s] : vs->SamplerUnits[s]; + const unsigned unit = prog->SamplerUnits[s]; if (ctx->Texture.Unit[unit]._ReallyEnabled) brw_update_sampler_state(brw, unit, s, &samplers[s], *sst_offset, &sdc_offset[s]); @@ -410,12 +406,16 @@ brw_upload_sampler_state_table(struct brw_context *brw, static void brw_upload_samplers(struct brw_context *brw) { - brw->vtbl.upload_sampler_state_table(brw, + /* BRW_NEW_FRAGMENT_PROGRAM */ + struct gl_program *fs = (struct gl_program *) brw->fragment_program; + brw->vtbl.upload_sampler_state_table(brw, fs, &brw->wm.sampler_count, &brw->wm.sampler_offset, brw->wm.sdc_offset); - brw->vtbl.upload_sampler_state_table(brw, + /* BRW_NEW_VERTEX_PROGRAM */ + struct gl_program *vs = (struct gl_program *) brw->vertex_program; + brw->vtbl.upload_sampler_state_table(brw, vs, &brw->vs.sampler_count, &brw->vs.sampler_offset, brw->vs.sdc_offset); diff --git a/src/mesa/drivers/dri/i965/gen7_sampler_state.c b/src/mesa/drivers/dri/i965/gen7_sampler_state.c index c7b69d5d6c6..5701f4d8f04 100644 --- a/src/mesa/drivers/dri/i965/gen7_sampler_state.c +++ b/src/mesa/drivers/dri/i965/gen7_sampler_state.c @@ -186,6 +186,7 @@ gen7_update_sampler_state(struct brw_context *brw, int unit, int ss_index, static void gen7_upload_sampler_state_table(struct brw_context *brw, + struct gl_program *prog, uint32_t *sampler_count, uint32_t *sst_offset, uint32_t *sdc_offset) @@ -193,11 +194,7 @@ gen7_upload_sampler_state_table(struct brw_context *brw, struct gl_context *ctx = &brw->ctx; struct gen7_sampler_state *samplers; - /* BRW_NEW_VERTEX_PROGRAM and BRW_NEW_FRAGMENT_PROGRAM */ - struct gl_program *vs = (struct gl_program *) brw->vertex_program; - struct gl_program *fs = (struct gl_program *) brw->fragment_program; - - GLbitfield SamplersUsed = vs->SamplersUsed | fs->SamplersUsed; + GLbitfield SamplersUsed = prog->SamplersUsed; *sampler_count = _mesa_fls(SamplersUsed); @@ -209,10 +206,9 @@ gen7_upload_sampler_state_table(struct brw_context *brw, 32, sst_offset); memset(samplers, 0, *sampler_count * sizeof(*samplers)); - for (unsigned s = 0; s < brw->wm.sampler_count; s++) { + for (unsigned s = 0; s < *sampler_count; s++) { if (SamplersUsed & (1 << s)) { - const unsigned unit = (fs->SamplersUsed & (1 << s)) ? - fs->SamplerUnits[s] : vs->SamplerUnits[s]; + const unsigned unit = prog->SamplerUnits[s]; if (ctx->Texture.Unit[unit]._ReallyEnabled) gen7_update_sampler_state(brw, unit, s, &samplers[s], &sdc_offset[s]); |