From: Neil Roberts Date: Mon, 2 Feb 2015 17:47:23 +0000 (+0000) Subject: i965/skl: Implement WaDisable1DDepthStencil X-Git-Tag: upstream/17.1.0~20898 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5b29b2922afe2b8167a589fc2896a071fc85b693;p=platform%2Fupstream%2Fmesa.git i965/skl: Implement WaDisable1DDepthStencil Skylake+ doesn't support setting a depth buffer to a 1D surface but it does allow pretending it's a 2D texture with a height of 1 instead. This fixes the GL_DEPTH_COMPONENT_* tests of the copyteximage piglit test (and also seems to avoid a subsequent GPU hang). Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89037 Reviewed-by: Kenneth Graunke --- diff --git a/src/mesa/drivers/dri/i965/gen8_depth_state.c b/src/mesa/drivers/dri/i965/gen8_depth_state.c index e428089..b4eb6e1 100644 --- a/src/mesa/drivers/dri/i965/gen8_depth_state.c +++ b/src/mesa/drivers/dri/i965/gen8_depth_state.c @@ -190,6 +190,18 @@ gen8_emit_depth_stencil_hiz(struct brw_context *brw, case GL_TEXTURE_3D: assert(mt); depth = MAX2(mt->logical_depth0, 1); + surftype = translate_tex_target(gl_target); + break; + case GL_TEXTURE_1D_ARRAY: + case GL_TEXTURE_1D: + if (brw->gen >= 9) { + /* WaDisable1DDepthStencil. Skylake+ doesn't support 1D depth + * textures but it does allow pretending it's a 2D texture + * instead. + */ + surftype = BRW_SURFACE_2D; + break; + } /* fallthrough */ default: surftype = translate_tex_target(gl_target);