From 75fbe089a6a29b01c2b3425b1c0fc2ee43bc5ff8 Mon Sep 17 00:00:00 2001 From: Jonathan Marek Date: Mon, 10 Feb 2020 13:51:36 -0500 Subject: [PATCH] freedreno: name sysmem color/depth flush events Signed-off-by: Jonathan Marek Reviewed-by: Connor Abbott Reviewed-by: Rob Clark Part-of: --- src/freedreno/registers/adreno_pm4.xml | 5 +++-- src/freedreno/vulkan/tu_blit.c | 8 ++++---- src/freedreno/vulkan/tu_cmd_buffer.c | 10 +++++----- src/gallium/drivers/freedreno/a5xx/fd5_gmem.c | 2 +- src/gallium/drivers/freedreno/a6xx/fd6_blitter.c | 8 ++++---- src/gallium/drivers/freedreno/a6xx/fd6_draw.c | 6 +++--- src/gallium/drivers/freedreno/a6xx/fd6_emit.c | 4 ++-- src/gallium/drivers/freedreno/a6xx/fd6_gmem.c | 4 ++-- 8 files changed, 24 insertions(+), 23 deletions(-) diff --git a/src/freedreno/registers/adreno_pm4.xml b/src/freedreno/registers/adreno_pm4.xml index c5f3f0a62f3..09ecd0c3349 100644 --- a/src/freedreno/registers/adreno_pm4.xml +++ b/src/freedreno/registers/adreno_pm4.xml @@ -37,8 +37,9 @@ xsi:schemaLocation="http://nouveau.freedesktop.org/ rules-ng.xsd"> - - + + + diff --git a/src/freedreno/vulkan/tu_blit.c b/src/freedreno/vulkan/tu_blit.c index 9baca8e582f..b71effb62ed 100644 --- a/src/freedreno/vulkan/tu_blit.c +++ b/src/freedreno/vulkan/tu_blit.c @@ -283,8 +283,8 @@ void tu_blit(struct tu_cmd_buffer *cmdbuf, struct tu_cs *cs, tu_cs_reserve_space(cmdbuf->device, cs, 18); tu6_emit_event_write(cmdbuf, cs, LRZ_FLUSH, false); - tu6_emit_event_write(cmdbuf, cs, 0x1d, true); - tu6_emit_event_write(cmdbuf, cs, FACENESS_FLUSH, true); + tu6_emit_event_write(cmdbuf, cs, PC_CCU_FLUSH_COLOR_TS, true); + tu6_emit_event_write(cmdbuf, cs, PC_CCU_FLUSH_DEPTH_TS, true); tu6_emit_event_write(cmdbuf, cs, PC_CCU_INVALIDATE_COLOR, false); tu6_emit_event_write(cmdbuf, cs, PC_CCU_INVALIDATE_DEPTH, false); @@ -367,8 +367,8 @@ void tu_blit(struct tu_cmd_buffer *cmdbuf, struct tu_cs *cs, tu_cs_reserve_space(cmdbuf->device, cs, 17); - tu6_emit_event_write(cmdbuf, cs, 0x1d, true); - tu6_emit_event_write(cmdbuf, cs, FACENESS_FLUSH, true); + tu6_emit_event_write(cmdbuf, cs, PC_CCU_FLUSH_COLOR_TS, true); + tu6_emit_event_write(cmdbuf, cs, PC_CCU_FLUSH_DEPTH_TS, true); tu6_emit_event_write(cmdbuf, cs, CACHE_FLUSH_TS, true); tu6_emit_event_write(cmdbuf, cs, CACHE_INVALIDATE, false); } diff --git a/src/freedreno/vulkan/tu_cmd_buffer.c b/src/freedreno/vulkan/tu_cmd_buffer.c index d54891971eb..4f7eb2b7eb6 100644 --- a/src/freedreno/vulkan/tu_cmd_buffer.c +++ b/src/freedreno/vulkan/tu_cmd_buffer.c @@ -1493,7 +1493,7 @@ tu_cmd_prepare_sysmem_clear_ib(struct tu_cmd_buffer *cmd, * when nothing clears which we currently can't handle. */ tu_cs_reserve_space(cmd->device, &sub_cs, 5); - tu6_emit_event_write(cmd, &sub_cs, UNK_1D, true); + tu6_emit_event_write(cmd, &sub_cs, PC_CCU_FLUSH_COLOR_TS, true); cmd->state.sysmem_clear_ib = tu_cs_end_sub_stream(&cmd->sub_cs, &sub_cs); } @@ -1584,8 +1584,8 @@ tu6_sysmem_render_end(struct tu_cmd_buffer *cmd, struct tu_cs *cs) tu6_emit_lrz_flush(cmd, cs); - tu6_emit_event_write(cmd, cs, UNK_1C, true); - tu6_emit_event_write(cmd, cs, UNK_1D, true); + tu6_emit_event_write(cmd, cs, PC_CCU_FLUSH_COLOR_TS, true); + tu6_emit_event_write(cmd, cs, PC_CCU_FLUSH_DEPTH_TS, true); tu_cs_sanity_check(cs); } @@ -2733,8 +2733,8 @@ tu_CmdNextSubpass(VkCommandBuffer commandBuffer, VkSubpassContents contents) /* Emit flushes so that input attachments will read the correct value. This * is for sysmem only, although it shouldn't do much harm on gmem. */ - tu6_emit_event_write(cmd, cs, UNK_1C, true); - tu6_emit_event_write(cmd, cs, UNK_1D, true); + tu6_emit_event_write(cmd, cs, PC_CCU_FLUSH_COLOR_TS, true); + tu6_emit_event_write(cmd, cs, PC_CCU_FLUSH_DEPTH_TS, true); /* TODO: * since we don't know how to do GMEM->GMEM resolve, diff --git a/src/gallium/drivers/freedreno/a5xx/fd5_gmem.c b/src/gallium/drivers/freedreno/a5xx/fd5_gmem.c index d2d53832748..c6564669e46 100644 --- a/src/gallium/drivers/freedreno/a5xx/fd5_gmem.c +++ b/src/gallium/drivers/freedreno/a5xx/fd5_gmem.c @@ -775,7 +775,7 @@ fd5_emit_sysmem_fini(struct fd_batch *batch) fd5_emit_lrz_flush(ring); OUT_PKT7(ring, CP_EVENT_WRITE, 4); - OUT_RING(ring, UNK_1D); + OUT_RING(ring, PC_CCU_FLUSH_COLOR_TS); OUT_RELOCW(ring, fd5_ctx->blit_mem, 0, 0, 0); /* ADDR_LO/HI */ OUT_RING(ring, 0x00000000); } diff --git a/src/gallium/drivers/freedreno/a6xx/fd6_blitter.c b/src/gallium/drivers/freedreno/a6xx/fd6_blitter.c index 9ef8be6b5a0..24f4e46b89d 100644 --- a/src/gallium/drivers/freedreno/a6xx/fd6_blitter.c +++ b/src/gallium/drivers/freedreno/a6xx/fd6_blitter.c @@ -152,8 +152,8 @@ emit_setup(struct fd_batch *batch) { struct fd_ringbuffer *ring = batch->draw; - fd6_event_write(batch, ring, 0x1d, true); - fd6_event_write(batch, ring, FACENESS_FLUSH, true); + fd6_event_write(batch, ring, PC_CCU_FLUSH_COLOR_TS, true); + fd6_event_write(batch, ring, PC_CCU_FLUSH_DEPTH_TS, true); fd6_event_write(batch, ring, PC_CCU_INVALIDATE_COLOR, false); fd6_event_write(batch, ring, PC_CCU_INVALIDATE_DEPTH, false); } @@ -660,8 +660,8 @@ handle_rgba_blit(struct fd_context *ctx, const struct pipe_blit_info *info) emit_blit_or_clear_texture(ctx, batch->draw, info, NULL); } - fd6_event_write(batch, batch->draw, 0x1d, true); - fd6_event_write(batch, batch->draw, FACENESS_FLUSH, true); + fd6_event_write(batch, batch->draw, PC_CCU_FLUSH_COLOR_TS, true); + fd6_event_write(batch, batch->draw, PC_CCU_FLUSH_DEPTH_TS, true); fd6_event_write(batch, batch->draw, CACHE_FLUSH_TS, true); fd6_cache_inv(batch, batch->draw); diff --git a/src/gallium/drivers/freedreno/a6xx/fd6_draw.c b/src/gallium/drivers/freedreno/a6xx/fd6_draw.c index f2d26a0eadf..4385964ba68 100644 --- a/src/gallium/drivers/freedreno/a6xx/fd6_draw.c +++ b/src/gallium/drivers/freedreno/a6xx/fd6_draw.c @@ -404,7 +404,7 @@ fd6_clear_lrz(struct fd_batch *batch, struct fd_resource *zsbuf, double depth) OUT_RING(ring, A6XX_RB_2D_BLIT_CNTL_COLOR_FORMAT(FMT6_16_UNORM) | 0x4f00080); - fd6_event_write(batch, ring, UNK_1D, true); + fd6_event_write(batch, ring, PC_CCU_FLUSH_COLOR_TS, true); fd6_event_write(batch, ring, PC_CCU_INVALIDATE_COLOR, false); OUT_PKT4(ring, REG_A6XX_RB_2D_SRC_SOLID_C0, 4); @@ -452,8 +452,8 @@ fd6_clear_lrz(struct fd_batch *batch, struct fd_resource *zsbuf, double depth) OUT_PKT4(ring, REG_A6XX_RB_UNKNOWN_8E04, 1); OUT_RING(ring, 0x0); /* RB_UNKNOWN_8E04 */ - fd6_event_write(batch, ring, UNK_1D, true); - fd6_event_write(batch, ring, FACENESS_FLUSH, true); + fd6_event_write(batch, ring, PC_CCU_FLUSH_COLOR_TS, true); + fd6_event_write(batch, ring, PC_CCU_FLUSH_DEPTH_TS, true); fd6_event_write(batch, ring, CACHE_FLUSH_TS, true); fd6_cache_inv(batch, ring); diff --git a/src/gallium/drivers/freedreno/a6xx/fd6_emit.c b/src/gallium/drivers/freedreno/a6xx/fd6_emit.c index 8a72ff07025..0a73a931794 100644 --- a/src/gallium/drivers/freedreno/a6xx/fd6_emit.c +++ b/src/gallium/drivers/freedreno/a6xx/fd6_emit.c @@ -1404,8 +1404,8 @@ fd6_framebuffer_barrier(struct fd_context *ctx) OUT_RING(ring, CP_WAIT_REG_MEM_4_MASK(~0)); OUT_RING(ring, CP_WAIT_REG_MEM_5_DELAY_LOOP_CYCLES(16)); - fd6_event_write(batch, ring, UNK_1D, true); - fd6_event_write(batch, ring, UNK_1C, true); + fd6_event_write(batch, ring, PC_CCU_FLUSH_COLOR_TS, true); + fd6_event_write(batch, ring, PC_CCU_FLUSH_DEPTH_TS, true); seqno = fd6_event_write(batch, ring, CACHE_FLUSH_TS, true); diff --git a/src/gallium/drivers/freedreno/a6xx/fd6_gmem.c b/src/gallium/drivers/freedreno/a6xx/fd6_gmem.c index 37820614da3..09463a7f8d1 100644 --- a/src/gallium/drivers/freedreno/a6xx/fd6_gmem.c +++ b/src/gallium/drivers/freedreno/a6xx/fd6_gmem.c @@ -1426,7 +1426,7 @@ emit_sysmem_clears(struct fd_batch *batch, struct fd_ringbuffer *ring) } } - fd6_event_write(batch, ring, UNK_1D, true); + fd6_event_write(batch, ring, PC_CCU_FLUSH_COLOR_TS, true); } static void @@ -1513,7 +1513,7 @@ fd6_emit_sysmem_fini(struct fd_batch *batch) fd6_emit_lrz_flush(ring); - fd6_event_write(batch, ring, UNK_1D, true); + fd6_event_write(batch, ring, PC_CCU_FLUSH_COLOR_TS, true); } void -- 2.30.2