diff options
author | Francisco Jerez <currojerez@riseup.net> | 2015-09-04 15:30:41 +0300 |
---|---|---|
committer | Francisco Jerez <currojerez@riseup.net> | 2015-11-17 15:22:44 +0200 |
commit | eb7503d454373df2e5138a3aa724fd58e31337ed (patch) | |
tree | b084cd1702ec6db9ab285e315f2f0d4d063cca2a | |
parent | 6994ca20aad110734f87ef5297fecd53e8f6e34e (diff) |
i965: Resolve color and flush for all active shader images in intel_update_state().image-load-store-flush
Fixes arb_shader_image_load_store/execution/load-from-cleared-image.shader_test.
Noticed no significant FPS regressions in CPU-bound benchmarks from the
Finnish benchmarking system on VLV, with n=6 and 95% confidence level.
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Jason Ekstrand <jason.ekstrand@intel.com>
Cc: mesa-stable@lists.freedesktop.org
Tested-by: Jordan Justen <jordan.l.justen@intel.com>
-rw-r--r-- | src/mesa/drivers/dri/i965/brw_context.c | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_context.c b/src/mesa/drivers/dri/i965/brw_context.c index 4d499295730..be3ad5c5230 100644 --- a/src/mesa/drivers/dri/i965/brw_context.c +++ b/src/mesa/drivers/dri/i965/brw_context.c @@ -189,6 +189,24 @@ intel_update_state(struct gl_context * ctx, GLuint new_state) brw_render_cache_set_check_flush(brw, tex_obj->mt->bo); } + /* Resolve color for each active shader image. */ + for (unsigned i = 0; i < MESA_SHADER_STAGES; i++) { + const struct gl_shader *shader = ctx->_Shader->CurrentProgram[i] ? + ctx->_Shader->CurrentProgram[i]->_LinkedShaders[i] : NULL; + + if (unlikely(shader && shader->NumImages)) { + for (unsigned j = 0; j < shader->NumImages; j++) { + struct gl_image_unit *u = &ctx->ImageUnits[shader->ImageUnits[j]]; + tex_obj = intel_texture_object(u->TexObj); + + if (tex_obj && tex_obj->mt) { + intel_miptree_resolve_color(brw, tex_obj->mt); + brw_render_cache_set_check_flush(brw, tex_obj->mt->bo); + } + } + } + } + _mesa_lock_context_textures(ctx); } |