diff options
author | Chris Forbes <[email protected]> | 2014-05-08 16:02:16 +1200 |
---|---|---|
committer | Chris Forbes <[email protected]> | 2014-05-09 09:46:19 +1200 |
commit | dd43900b7bb3bec0b136bb244a04841123575467 (patch) | |
tree | 00afb61f1c1f35cfb5c5bdb29dc6be64dfc064b0 /src | |
parent | cc8c00da885ba1446e73b77d9171965d1299ad37 (diff) |
i965/Gen7: Set up layer constraints properly for renderbuffers
There were a few problems here, which mostly just broke layered
rendering into a view:
- Render target view extent was always set to be == depth. This is
benign for non-layered-rendering, but allows writes off the end of the
render target for layered rendering, which ends badly.
- Layered rendering did not honor the mt_layer setting, so would not
properly handle MinLayer being set on a view.
V2: Restore the MAX2() to account for zero depth/layer_count.
Signed-off-by: Chris Forbes <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
Diffstat (limited to 'src')
-rw-r--r-- | src/mesa/drivers/dri/i965/gen7_wm_surface_state.c | 17 |
1 files changed, 7 insertions, 10 deletions
diff --git a/src/mesa/drivers/dri/i965/gen7_wm_surface_state.c b/src/mesa/drivers/dri/i965/gen7_wm_surface_state.c index f051024b1ab..819835d0f70 100644 --- a/src/mesa/drivers/dri/i965/gen7_wm_surface_state.c +++ b/src/mesa/drivers/dri/i965/gen7_wm_surface_state.c @@ -454,9 +454,11 @@ gen7_update_renderbuffer_surface(struct brw_context *brw, mesa_format rb_format = _mesa_get_render_format(ctx, intel_rb_format(irb)); uint32_t surftype; bool is_array = false; - int depth = MAX2(rb->Depth, 1); - int min_array_element; + int depth = MAX2(irb->layer_count, 1); const uint8_t mocs = GEN7_MOCS_L3; + + int min_array_element = irb->mt_layer / MAX2(mt->num_samples, 1); + GLenum gl_target = rb->TexImage ? rb->TexImage->TexObject->Target : GL_TEXTURE_2D; if (gl_target == GL_TEXTURE_1D_ARRAY) @@ -488,20 +490,15 @@ gen7_update_renderbuffer_surface(struct brw_context *brw, is_array = true; depth *= 6; break; + case GL_TEXTURE_3D: + depth = MAX2(rb->Depth, 1); + /* fallthrough */ default: surftype = translate_tex_target(gl_target); is_array = _mesa_tex_target_is_array(gl_target); break; } - if (layered) { - min_array_element = 0; - } else if (irb->mt->num_samples > 1) { - min_array_element = irb->mt_layer / irb->mt->num_samples; - } else { - min_array_element = irb->mt_layer; - } - surf[0] = surftype << BRW_SURFACE_TYPE_SHIFT | format << BRW_SURFACE_FORMAT_SHIFT | (irb->mt->array_spacing_lod0 ? GEN7_SURFACE_ARYSPC_LOD0 |