From a35069a7b55bd2ca5cd16d7a7e67b46e92d7c7b1 Mon Sep 17 00:00:00 2001 From: Alyssa Rosenzweig Date: Fri, 7 Jun 2019 10:32:17 -0700 Subject: [PATCH] panfrost: Refactor blitting code We refactor the wallpaper rendering code to separate the wallpaper-specific bits from the general blitting capabilities. In the (hopefully near) future, we'll turn this on to implement real Gallium blits, e.g. for automatic mipmap generation. Signed-off-by: Alyssa Rosenzweig --- src/gallium/drivers/panfrost/meson.build | 1 + src/gallium/drivers/panfrost/pan_blit.c | 133 ++++++++++++++++++++++++++++ src/gallium/drivers/panfrost/pan_context.c | 45 +--------- src/gallium/drivers/panfrost/pan_resource.c | 61 ++++++------- src/gallium/drivers/panfrost/pan_resource.h | 9 ++ 5 files changed, 170 insertions(+), 79 deletions(-) create mode 100644 src/gallium/drivers/panfrost/pan_blit.c diff --git a/src/gallium/drivers/panfrost/meson.build b/src/gallium/drivers/panfrost/meson.build index eda7abb..d01c132 100644 --- a/src/gallium/drivers/panfrost/meson.build +++ b/src/gallium/drivers/panfrost/meson.build @@ -44,6 +44,7 @@ files_panfrost = files( 'pan_context.c', 'pan_afbc.c', + 'pan_blit.c', 'pan_job.c', 'pan_trace.c', 'pan_drm.c', diff --git a/src/gallium/drivers/panfrost/pan_blit.c b/src/gallium/drivers/panfrost/pan_blit.c new file mode 100644 index 0000000..5859f92 --- /dev/null +++ b/src/gallium/drivers/panfrost/pan_blit.c @@ -0,0 +1,133 @@ +/* + * Copyright (C) 2014 Broadcom + * Copyright (C) 2019 Collabora + * + * Permission is hereby granted, free of charge, to any person obtaining a + * copy of this software and associated documentation files (the "Software"), + * to deal in the Software without restriction, including without limitation + * the rights to use, copy, modify, merge, publish, distribute, sublicense, + * and/or sell copies of the Software, and to permit persons to whom the + * Software is furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice (including the next + * paragraph) shall be included in all copies or substantial portions of the + * Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE + * SOFTWARE. + * + * Authors (Collabora): + * Tomeu Vizoso + * Alyssa Rosenzweig + * + */ + +#include "pan_context.h" +#include "util/u_format.h" + +static void +panfrost_blitter_save(struct panfrost_context *ctx) +{ + + util_blitter_save_vertex_buffer_slot(ctx->blitter, ctx->vertex_buffers); + util_blitter_save_vertex_elements(ctx->blitter, ctx->vertex); + util_blitter_save_vertex_shader(ctx->blitter, ctx->vs); + util_blitter_save_rasterizer(ctx->blitter, ctx->rasterizer); + util_blitter_save_viewport(ctx->blitter, &ctx->pipe_viewport); + util_blitter_save_scissor(ctx->blitter, &ctx->scissor); + util_blitter_save_fragment_shader(ctx->blitter, ctx->fs); + util_blitter_save_blend(ctx->blitter, ctx->blend); + util_blitter_save_depth_stencil_alpha(ctx->blitter, ctx->depth_stencil); + util_blitter_save_stencil_ref(ctx->blitter, &ctx->stencil_ref); + util_blitter_save_so_targets(ctx->blitter, 0, NULL); + + /* For later */ +// util_blitter_save_sample_mask(ctx->blitter, ctx->sample_mask); + + util_blitter_save_framebuffer(ctx->blitter, &ctx->pipe_framebuffer); + util_blitter_save_fragment_sampler_states(ctx->blitter, + ctx->sampler_count[PIPE_SHADER_FRAGMENT], + (void **)(&ctx->samplers[PIPE_SHADER_FRAGMENT])); + util_blitter_save_fragment_sampler_views(ctx->blitter, + ctx->sampler_view_count[PIPE_SHADER_FRAGMENT], + (struct pipe_sampler_view **)&ctx->sampler_views[PIPE_SHADER_FRAGMENT]); +} + +static bool +panfrost_u_blitter_blit(struct pipe_context *pipe, + const struct pipe_blit_info *info) +{ + struct panfrost_context *ctx = pan_context(pipe); + + if (!util_blitter_is_blit_supported(ctx->blitter, info)) { + fprintf(stderr, "blit unsupported %s -> %s\n", + util_format_short_name(info->src.resource->format), + util_format_short_name(info->dst.resource->format)); + return false; + } + + /* TODO: Scissor */ + + panfrost_blitter_save(ctx); + util_blitter_blit(ctx->blitter, info); + + return true; +} + +void +panfrost_blit(struct pipe_context *pipe, + const struct pipe_blit_info *info) +{ + /* We don't have a hardware blit, so we just fake it with + * u_blitter. We could do a little better by culling + * vertex jobs, though. */ + + /* TODO: Implement blitting. Commented out because u_blitter is not + * fully integrated and creates bugs in other places. */ +#if 0 + if (panfrost_u_blitter_blit(pipe, info)) + return; + + fprintf(stderr, "Unhandled blit"); +#endif + + return; +} + +/* Blits a framebuffer to "itself". Mali is a tiler, so the + * framebuffer is implicitly cleared every frame, so if there is + * no actual glClear(), we have to blit it back ourselves. + */ + +void +panfrost_blit_wallpaper(struct panfrost_context *ctx) +{ + struct pipe_blit_info binfo = { }; + + panfrost_blitter_save(ctx); + + binfo.src.resource = binfo.dst.resource = ctx->pipe_framebuffer.cbufs[0]->texture; + binfo.src.level = binfo.dst.level = 0; + binfo.src.box.x = binfo.dst.box.x = 0; + binfo.src.box.y = binfo.dst.box.y = 0; + binfo.src.box.width = binfo.dst.box.width = ctx->pipe_framebuffer.width; + binfo.src.box.height = binfo.dst.box.height = ctx->pipe_framebuffer.height; + + /* This avoids an assert due to missing nir_texop_txb support */ + //binfo.src.box.depth = binfo.dst.box.depth = 1; + + binfo.src.format = binfo.dst.format = ctx->pipe_framebuffer.cbufs[0]->texture->format; + + assert(ctx->pipe_framebuffer.nr_cbufs == 1); + binfo.mask = PIPE_MASK_RGBA; + binfo.filter = PIPE_TEX_FILTER_LINEAR; + binfo.scissor_enable = FALSE; + + util_blitter_blit(ctx->blitter, &binfo); +} + diff --git a/src/gallium/drivers/panfrost/pan_context.c b/src/gallium/drivers/panfrost/pan_context.c index 21e56ef..94fcae5 100644 --- a/src/gallium/drivers/panfrost/pan_context.c +++ b/src/gallium/drivers/panfrost/pan_context.c @@ -1320,54 +1320,13 @@ static void panfrost_draw_wallpaper(struct pipe_context *pipe) { struct panfrost_context *ctx = pan_context(pipe); - struct pipe_blit_info binfo = { }; /* Nothing to reload? */ if (ctx->pipe_framebuffer.cbufs[0] == NULL) return; - util_blitter_save_vertex_buffer_slot(ctx->blitter, ctx->vertex_buffers); - util_blitter_save_vertex_elements(ctx->blitter, ctx->vertex); - util_blitter_save_vertex_shader(ctx->blitter, ctx->vs); - util_blitter_save_rasterizer(ctx->blitter, ctx->rasterizer); - util_blitter_save_viewport(ctx->blitter, &ctx->pipe_viewport); - util_blitter_save_scissor(ctx->blitter, &ctx->scissor); - util_blitter_save_fragment_shader(ctx->blitter, ctx->fs); - util_blitter_save_blend(ctx->blitter, ctx->blend); - util_blitter_save_depth_stencil_alpha(ctx->blitter, ctx->depth_stencil); - util_blitter_save_stencil_ref(ctx->blitter, &ctx->stencil_ref); - util_blitter_save_so_targets(ctx->blitter, 0, NULL); - - /* For later */ -// util_blitter_save_sample_mask(ctx->blitter, vc4->sample_mask); - - util_blitter_save_framebuffer(ctx->blitter, &ctx->pipe_framebuffer); - util_blitter_save_fragment_sampler_states(ctx->blitter, - ctx->sampler_count[PIPE_SHADER_FRAGMENT], - (void **)(&ctx->samplers[PIPE_SHADER_FRAGMENT])); - util_blitter_save_fragment_sampler_views(ctx->blitter, - ctx->sampler_view_count[PIPE_SHADER_FRAGMENT], - (struct pipe_sampler_view **)&ctx->sampler_views[PIPE_SHADER_FRAGMENT]); - - - binfo.src.resource = binfo.dst.resource = ctx->pipe_framebuffer.cbufs[0]->texture; - binfo.src.level = binfo.dst.level = 0; - binfo.src.box.x = binfo.dst.box.x = 0; - binfo.src.box.y = binfo.dst.box.y = 0; - binfo.src.box.width = binfo.dst.box.width = ctx->pipe_framebuffer.width; - binfo.src.box.height = binfo.dst.box.height = ctx->pipe_framebuffer.height; - - /* This avoids an assert due to missing nir_texop_txb support */ - //binfo.src.box.depth = binfo.dst.box.depth = 1; - - binfo.src.format = binfo.dst.format = ctx->pipe_framebuffer.cbufs[0]->texture->format; - - assert(ctx->pipe_framebuffer.nr_cbufs == 1); - binfo.mask = PIPE_MASK_RGBA; - binfo.filter = PIPE_TEX_FILTER_LINEAR; - binfo.scissor_enable = FALSE; - - util_blitter_blit(ctx->blitter, &binfo); + /* Blit the wallpaper in */ + panfrost_blit_wallpaper(ctx); /* We are flushing all queued draws and we know that no more jobs will * be added until the next frame. diff --git a/src/gallium/drivers/panfrost/pan_resource.c b/src/gallium/drivers/panfrost/pan_resource.c index 9e8f305..ef0a3ba 100644 --- a/src/gallium/drivers/panfrost/pan_resource.c +++ b/src/gallium/drivers/panfrost/pan_resource.c @@ -1,31 +1,33 @@ -/************************************************************************** - * - * Copyright 2008 VMware, Inc. - * Copyright 2014 Broadcom - * Copyright 2018 Alyssa Rosenzweig - * All Rights Reserved. +/* + * Copyright (C) 2008 VMware, Inc. + * Copyright (C) 2014 Broadcom + * Copyright (C) 2018-2019 Alyssa Rosenzweig + * Copyright (C) 2019 Collabora * * Permission is hereby granted, free of charge, to any person obtaining a - * copy of this software and associated documentation files (the - * "Software"), to deal in the Software without restriction, including - * without limitation the rights to use, copy, modify, merge, publish, - * distribute, sub license, and/or sell copies of the Software, and to - * permit persons to whom the Software is furnished to do so, subject to - * the following conditions: + * copy of this software and associated documentation files (the "Software"), + * to deal in the Software without restriction, including without limitation + * the rights to use, copy, modify, merge, publish, distribute, sublicense, + * and/or sell copies of the Software, and to permit persons to whom the + * Software is furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice (including the next + * paragraph) shall be included in all copies or substantial portions of the + * Software. * - * The above copyright notice and this permission notice (including the - * next paragraph) shall be included in all copies or substantial portions - * of the Software. + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE + * SOFTWARE. * - * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS - * OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF - * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NON-INFRINGEMENT. - * IN NO EVENT SHALL VMWARE AND/OR ITS SUPPLIERS BE LIABLE FOR - * ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, - * TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE - * SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. + * Authors (Collabora): + * Tomeu Vizoso + * Alyssa Rosenzweig * - **************************************************************************/ + */ #include #include @@ -129,19 +131,6 @@ panfrost_flush_resource(struct pipe_context *pctx, struct pipe_resource *prsc) //DBG("TODO %s\n", __func__); } -static void -panfrost_blit(struct pipe_context *pipe, - const struct pipe_blit_info *info) -{ - if (util_try_blit_via_copy_region(pipe, info)) - return; - - /* TODO */ - DBG("Unhandled blit.\n"); - - return; -} - static struct pipe_surface * panfrost_create_surface(struct pipe_context *pipe, struct pipe_resource *pt, diff --git a/src/gallium/drivers/panfrost/pan_resource.h b/src/gallium/drivers/panfrost/pan_resource.h index feb6da1..d75b68c 100644 --- a/src/gallium/drivers/panfrost/pan_resource.h +++ b/src/gallium/drivers/panfrost/pan_resource.h @@ -133,4 +133,13 @@ panfrost_format_supports_afbc(enum pipe_format format); void panfrost_enable_afbc(struct panfrost_context *ctx, struct panfrost_resource *rsrc, bool ds); +/* Blitting */ + +void +panfrost_blit(struct pipe_context *pipe, + const struct pipe_blit_info *info); + +void +panfrost_blit_wallpaper(struct panfrost_context *ctx); + #endif /* PAN_RESOURCE_H */ -- 2.7.4