From ab49a0e74671fb13ce72f839403cfc82f76e4217 Mon Sep 17 00:00:00 2001 From: Erico Nunes Date: Thu, 19 Sep 2019 21:08:05 +0200 Subject: [PATCH] lima: remove partial clear support from pipe->clear() pipe->clear() is not called for partial clears, which mesa emulates by drawing a quad. Furthermore, drivers should not use rasterizer state information for scissor information (which was being used to handle the partial clears). So, remove the partial clear support since it was not supposed to be handled by pipe->clear() anyway. This fixes issues with clearing after switching to different sized framebuffers. Signed-off-by: Erico Nunes Reviewed-by: Vasily Khoruzhick Reviewed-by: Qiang Yu --- src/gallium/drivers/lima/lima_draw.c | 98 ++-------------------------- 1 file changed, 5 insertions(+), 93 deletions(-) diff --git a/src/gallium/drivers/lima/lima_draw.c b/src/gallium/drivers/lima/lima_draw.c index c635e5a87bf..8d306caf64a 100644 --- a/src/gallium/drivers/lima/lima_draw.c +++ b/src/gallium/drivers/lima/lima_draw.c @@ -339,74 +339,6 @@ lima_pack_reload_plbu_cmd(struct lima_context *ctx) PLBU_CMD_END(); } -static void -lima_pack_clear_plbu_cmd(struct lima_context *ctx) -{ - #define lima_clear_render_state_offset 0x0000 - #define lima_clear_shader_offset 0x0040 - #define lima_clear_buffer_size 0x0080 - - void *cpu; - unsigned offset; - struct pipe_resource *pres = NULL; - u_upload_alloc(ctx->uploader, 0, lima_clear_buffer_size, - 0x40, &offset, &pres, &cpu); - - struct lima_resource *res = lima_resource(pres); - uint32_t va = res->bo->va + offset; - - struct lima_screen *screen = lima_screen(ctx->base.screen); - uint32_t gl_pos_va = screen->pp_buffer->va + pp_clear_gl_pos_offset; - - /* const0 clear_color, mov.v1 $0 ^const0.xxxx, stop */ - uint32_t clear_shader[] = { - 0x00021025, 0x0000000c, - (ctx->clear.color_16pc << 12) | 0x000007cf, - ctx->clear.color_16pc >> 12, - ctx->clear.color_16pc >> 44, - }; - memcpy(cpu + lima_clear_shader_offset, &clear_shader, - sizeof(clear_shader)); - - uint32_t clear_shader_va = va + lima_clear_shader_offset; - uint32_t clear_shader_first_instr_size = clear_shader[0] & 0x1f; - - struct lima_render_state clear_render_state = { - .blend_color_bg = 0x00800080, - .blend_color_ra = 0x00ff0080, - .alpha_blend = 0xfc321892, - .depth_test = 0x0000003e, - .depth_range = 0xffff0000, - .stencil_front = 0x00000007, - .stencil_back = 0x00000007, - .multi_sample = 0x0000f007, - .shader_address = clear_shader_va | clear_shader_first_instr_size, - }; - memcpy(cpu + lima_clear_render_state_offset, &clear_render_state, - sizeof(clear_render_state)); - - PLBU_CMD_BEGIN(22); - - PLBU_CMD_VIEWPORT_LEFT(0); - PLBU_CMD_VIEWPORT_RIGHT(0x45800000); - PLBU_CMD_VIEWPORT_TOP(0); - PLBU_CMD_VIEWPORT_BOTTOM(0x45800000); - - struct pipe_scissor_state *scissor = &ctx->scissor; - PLBU_CMD_SCISSORS(scissor->minx, scissor->maxx, scissor->miny, scissor->maxy); - - PLBU_CMD_RSW_VERTEX_ARRAY(va + lima_clear_render_state_offset, gl_pos_va); - - PLBU_CMD_UNKNOWN2(); - PLBU_CMD_UNKNOWN1(); - - PLBU_CMD_INDICES(screen->pp_buffer->va + pp_shared_index_offset); - PLBU_CMD_INDEXED_DEST(gl_pos_va); - PLBU_CMD_DRAW_ELEMENTS(0xf, 0, 3); - - PLBU_CMD_END(); -} - static void lima_pack_head_plbu_cmd(struct lima_context *ctx) { @@ -445,19 +377,6 @@ lima_is_scissor_zero(struct lima_context *ctx) && scissor->miny == scissor->maxy; } -static bool -lima_is_scissor_full_fb(struct lima_context *ctx) -{ - if (!ctx->rasterizer || !ctx->rasterizer->base.scissor) - return true; - - struct pipe_scissor_state *scissor = &ctx->scissor; - struct lima_context_framebuffer *fb = &ctx->framebuffer; - return - scissor->minx == 0 && scissor->maxx == fb->base.width && - scissor->miny == 0 && scissor->maxy == fb->base.height; -} - static void hilbert_rotate(int n, int *x, int *y, int rx, int ry) { @@ -702,16 +621,13 @@ lima_clear(struct pipe_context *pctx, unsigned buffers, const union pipe_color_union *color, double depth, unsigned stencil) { struct lima_context *ctx = lima_context(pctx); - bool full_fb_clear = lima_is_scissor_full_fb(ctx); - if (full_fb_clear) { - lima_flush(ctx); + lima_flush(ctx); - /* no need to reload if cleared */ - if (ctx->framebuffer.base.nr_cbufs && (buffers & PIPE_CLEAR_COLOR0)) { - struct lima_surface *surf = lima_surface(ctx->framebuffer.base.cbufs[0]); - surf->reload = false; - } + /* no need to reload if cleared */ + if (ctx->framebuffer.base.nr_cbufs && (buffers & PIPE_CLEAR_COLOR0)) { + struct lima_surface *surf = lima_surface(ctx->framebuffer.base.cbufs[0]); + surf->reload = false; } struct lima_context_clear *clear = &ctx->clear; @@ -741,10 +657,6 @@ lima_clear(struct pipe_context *pctx, unsigned buffers, lima_pack_head_plbu_cmd(ctx); - /* partial clear */ - if (!full_fb_clear) - lima_pack_clear_plbu_cmd(ctx); - ctx->dirty |= LIMA_CONTEXT_DIRTY_CLEAR; } -- 2.30.2