gallium/radeon: move last_gfx_fence from radeonsi to common code

Reviewed-by: Nicolai Hähnle <nicolai.haehnle@amd.com>

diff --git a/src/gallium/drivers/radeon/r600_pipe_common.c b/src/gallium/drivers/radeon/r600_pipe_common.c
index caf2552bcda447a5ffd8010ed6cd2f2dfb4a8f8d..6f4fc98271b42134941552e089716bf8c759af2d 100644 (file)
--- a/src/gallium/drivers/radeon/r600_pipe_common.c
+++ b/src/gallium/drivers/radeon/r600_pipe_common.c
@@ -510,6 +510,7 @@ void r600_common_context_cleanup(struct r600_common_context *rctx)
        if (rctx->allocator_zeroed_memory) {
                u_suballocator_destroy(rctx->allocator_zeroed_memory);
        }
+       rctx->ws->fence_reference(&rctx->last_gfx_fence, NULL);
        rctx->ws->fence_reference(&rctx->last_sdma_fence, NULL);
 }
 

diff --git a/src/gallium/drivers/radeon/r600_pipe_common.h b/src/gallium/drivers/radeon/r600_pipe_common.h
index 7851a8624f9cd8237dc53f3cbce334cfb7729c0b..3bebd00a56ab11f65bc5225ad9a591deed493ade 100644 (file)
--- a/src/gallium/drivers/radeon/r600_pipe_common.h
+++ b/src/gallium/drivers/radeon/r600_pipe_common.h
@@ -509,6 +509,7 @@ struct r600_common_context {
        enum chip_class                 chip_class;
        struct r600_ring                gfx;
        struct r600_ring                dma;
+       struct pipe_fence_handle        *last_gfx_fence;
        struct pipe_fence_handle        *last_sdma_fence;
        unsigned                        initial_gfx_cs_size;
        unsigned                        gpu_reset_counter;

diff --git a/src/gallium/drivers/radeonsi/si_hw_context.c b/src/gallium/drivers/radeonsi/si_hw_context.c
index e2e7afd1834dc8b70e3737074a1a0dee02c0dcf6..91c985f3675c84b93f506da87ba5ebed53b06f51 100644 (file)
--- a/src/gallium/drivers/radeonsi/si_hw_context.c
+++ b/src/gallium/drivers/radeonsi/si_hw_context.c
@@ -102,9 +102,9 @@ void si_context_gfx_flush(void *context, unsigned flags,
        ctx->gfx_flush_in_progress = true;
 
        if (!radeon_emitted(cs, ctx->b.initial_gfx_cs_size) &&
-           (!fence || ctx->last_gfx_fence)) {
+           (!fence || ctx->b.last_gfx_fence)) {
                if (fence)
-                       ws->fence_reference(fence, ctx->last_gfx_fence);
+                       ws->fence_reference(fence, ctx->b.last_gfx_fence);
                if (!(flags & RADEON_FLUSH_ASYNC))
                        ws->cs_sync_flush(cs);
                ctx->gfx_flush_in_progress = false;
@@ -135,17 +135,17 @@ void si_context_gfx_flush(void *context, unsigned flags,
        }
 
        /* Flush the CS. */
-       ws->cs_flush(cs, flags, &ctx->last_gfx_fence);
+       ws->cs_flush(cs, flags, &ctx->b.last_gfx_fence);
 
        if (fence)
-               ws->fence_reference(fence, ctx->last_gfx_fence);
+               ws->fence_reference(fence, ctx->b.last_gfx_fence);
 
        /* Check VM faults if needed. */
        if (ctx->screen->b.debug_flags & DBG_CHECK_VM) {
                /* Use conservative timeout 800ms, after which we won't wait any
                 * longer and assume the GPU is hung.
                 */
-               ctx->b.ws->fence_wait(ctx->b.ws, ctx->last_gfx_fence, 800*1000*1000);
+               ctx->b.ws->fence_wait(ctx->b.ws, ctx->b.last_gfx_fence, 800*1000*1000);
 
                si_check_vm_faults(&ctx->b, &ctx->last_gfx, RING_GFX);
        }

diff --git a/src/gallium/drivers/radeonsi/si_pipe.c b/src/gallium/drivers/radeonsi/si_pipe.c
index 9304e5c594399155272419c051124d627485ea4c..c540cfdfab5c54f228f2cd279b90991e4ee0b13f 100644 (file)
--- a/src/gallium/drivers/radeonsi/si_pipe.c
+++ b/src/gallium/drivers/radeonsi/si_pipe.c
@@ -64,7 +64,6 @@ static void si_destroy_context(struct pipe_context *context)
        free(sctx->border_color_table);
        r600_resource_reference(&sctx->scratch_buffer, NULL);
        r600_resource_reference(&sctx->compute_scratch_buffer, NULL);
-       sctx->b.ws->fence_reference(&sctx->last_gfx_fence, NULL);
 
        si_pm4_free_state(sctx, sctx->init_config, ~0);
        if (sctx->init_config_gs_rings)

diff --git a/src/gallium/drivers/radeonsi/si_pipe.h b/src/gallium/drivers/radeonsi/si_pipe.h
index 41e94487c8ae262abc8abb561c940a4f1f819793..55f8965d2d4b1293e7a4cde1f987c13947cb3e4d 100644 (file)
--- a/src/gallium/drivers/radeonsi/si_pipe.h
+++ b/src/gallium/drivers/radeonsi/si_pipe.h
@@ -215,7 +215,6 @@ struct si_context {
        bool                            ce_need_synchronization;
        struct u_suballocator           *ce_suballocator;
 
-       struct pipe_fence_handle        *last_gfx_fence;
        struct si_shader_ctx_state      fixed_func_tcs_shader;
        LLVMTargetMachineRef            tm; /* only non-threaded compilation */
        bool                            gfx_flush_in_progress;