nvfx: fix GPU hardlocks when depth buffer is absent
authorLuca Barbieri <luca@luca-barbieri.com>
Thu, 19 Aug 2010 10:58:14 +0000 (12:58 +0200)
committerLuca Barbieri <luca@luca-barbieri.com>
Sat, 21 Aug 2010 18:42:14 +0000 (20:42 +0200)
src/gallium/drivers/nvfx/nvfx_context.h
src/gallium/drivers/nvfx/nvfx_state.c
src/gallium/drivers/nvfx/nvfx_state_emit.c
src/gallium/drivers/nvfx/nvfx_state_fb.c

index 2eec5a11a6dbc25e5306c1d6327ac3b700a1d44e..04447da3e18d4e39625e45ea8c23449911132d4c 100644 (file)
@@ -56,7 +56,7 @@ struct nvfx_rasterizer_state {
 struct nvfx_zsa_state {
        struct pipe_depth_stencil_alpha_state pipe;
        unsigned sb_len;
-       uint32_t sb[26];
+       uint32_t sb[24];
 };
 
 struct nvfx_blend_state {
index f529fb2a2766e59ce072bc26728457e87b7ab2c3..c8431641d3749371f639e7166f6914f865e954d9 100644 (file)
@@ -207,10 +207,8 @@ nvfx_depth_stencil_alpha_state_create(struct pipe_context *pipe,
        struct nvfx_zsa_state *zsaso = CALLOC(1, sizeof(*zsaso));
        struct nouveau_statebuf_builder sb = sb_init(zsaso->sb);
 
-       sb_method(sb, NV34TCL_DEPTH_FUNC, 3);
+       sb_method(sb, NV34TCL_DEPTH_FUNC, 1);
        sb_data  (sb, nvgl_comparison_op(cso->depth.func));
-       sb_data  (sb, cso->depth.writemask ? 1 : 0);
-       sb_data  (sb, cso->depth.enabled ? 1 : 0);
 
        sb_method(sb, NV34TCL_ALPHA_FUNC_ENABLE, 3);
        sb_data  (sb, cso->alpha.enabled ? 1 : 0);
index 7a706ea6d77a80b283391e3e9b2e5fbd3fc323fc..19512194be6073de68152c1ace618edb0ae408df 100644 (file)
@@ -113,11 +113,19 @@ nvfx_state_validate_common(struct nvfx_context *nvfx)
    etracer, neverball, foobillard, glest totally misrender
    TODO: find the right fix
 */
-       if(dirty & (NVFX_NEW_VIEWPORT | NVFX_NEW_RAST | NVFX_NEW_ZSA) || (all_swizzled > 0))
+       if(dirty & (NVFX_NEW_VIEWPORT | NVFX_NEW_RAST | NVFX_NEW_ZSA) || (all_swizzled >= 0))
        {
                nvfx_state_viewport_validate(nvfx);
        }
 
+       if(dirty & NVFX_NEW_ZSA || (all_swizzled >= 0))
+       {
+               WAIT_RING(chan, 3);
+               OUT_RING(chan, RING_3D(NV34TCL_DEPTH_WRITE_ENABLE, 2));
+               OUT_RING(chan, nvfx->framebuffer.zsbuf && nvfx->zsa->pipe.depth.writemask);
+               OUT_RING(chan, nvfx->framebuffer.zsbuf && nvfx->zsa->pipe.depth.enabled);
+       }
+
        if(flush_tex_cache)
        {
                // TODO: what about nv30?
index 28bbd36c2e81ae0d0936b6f3014ea85f6360e51f..3db9cec905454b150f3d30b33c8e8749f1fe8025 100644 (file)
@@ -252,6 +252,10 @@ nvfx_framebuffer_validate(struct nvfx_context *nvfx, unsigned prepare_result)
                        OUT_RING(chan, nvfx->hw_zeta.pitch);
                }
        }
+       else if(nvfx->is_nv4x) {
+               OUT_RING(chan, RING_3D(NV40TCL_ZETA_PITCH, 1));
+               OUT_RING(chan, 64);
+       }
 
        OUT_RING(chan, RING_3D(NV34TCL_RT_ENABLE, 1));
        OUT_RING(chan, rt_enable);