diff options
author | Ian Romanick <[email protected]> | 2012-10-19 14:04:08 -0700 |
---|---|---|
committer | Ian Romanick <[email protected]> | 2012-10-24 11:35:07 -0700 |
commit | 5fe5aa8e55a8db0b80f6ff9838bad47ce0406fd0 (patch) | |
tree | 20efaa8fc0f3cbbc38ca1a02fb2d7522439ee714 /src/mesa | |
parent | 8e019506257b720a1a6ff668aef57bff445a1725 (diff) |
i965: Fix regression in depth texture rendering on pre-SNB
This was introduced by commit 24db6d6 (cherry-picked from a683012). The
original patch fixed potential GPU hangs on SNB, and it caused some
rendering regressions there. The benefits outweigh the costs.
However, the work-around is not necessary for pre-SNB chipsets.
Applying the work-around there gives rendering regressions with no
benefit. This patch disables the work-around on pre-SNB chipsets.
Without the original patch, the piglit test
depthstencil-render-miplevels would reliably hang an SNB GPU. On ILK
this test would not hang, and it does not hang with this patch.
NOTE: This is a candidate for the 8.0 and 9.0 branches
Signed-off-by: Ian Romanick <[email protected]>
Acked-by: Kenneth Graunke <[email protected]>
Acked-by: Paul Berry <[email protected]>
Diffstat (limited to 'src/mesa')
-rw-r--r-- | src/mesa/drivers/dri/i965/brw_misc_state.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_misc_state.c b/src/mesa/drivers/dri/i965/brw_misc_state.c index 6dfa08e01af..4799af26b5f 100644 --- a/src/mesa/drivers/dri/i965/brw_misc_state.c +++ b/src/mesa/drivers/dri/i965/brw_misc_state.c @@ -413,8 +413,10 @@ static void emit_depthbuffer(struct brw_context *brw) * tile_x and tile_y to 0. This is a temporary workaround until we come * up with a better solution. */ - tile_x &= ~7; - tile_y &= ~7; + if (intel->gen >= 6) { + tile_x &= ~7; + tile_y &= ~7; + } BEGIN_BATCH(len); OUT_BATCH(_3DSTATE_DEPTH_BUFFER << 16 | (len - 2)); |