From: Thomas Hellstrom Date: Wed, 12 Apr 2017 07:28:49 +0000 (+0200) Subject: svga: Use utility functions to help determine whether we can use copy_region X-Git-Tag: upstream/18.1.0~8752 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=71f857d6ab504c4e355b2fcaf36b66426e6061ab;p=platform%2Fupstream%2Fmesa.git svga: Use utility functions to help determine whether we can use copy_region It seems like the SVGA tests are in general more stringent than the utility tests, but they also miss some blitter features like filters and window rectangles, and if new blitter features are added in the future, it might be possible that we forget adding tests for those. So in addition to the SVGA tests, use the utility tests to restrict the situations where we can use copy_region. Signed-off-by: Thomas Hellstrom Reviewed-by: Brian Paul Reviewed-by: Charmaine Lee --- diff --git a/src/gallium/drivers/svga/svga_pipe_blit.c b/src/gallium/drivers/svga/svga_pipe_blit.c index 677d78c..13788fd 100644 --- a/src/gallium/drivers/svga/svga_pipe_blit.c +++ b/src/gallium/drivers/svga/svga_pipe_blit.c @@ -275,9 +275,8 @@ static bool can_blit_via_svga_copy_region(struct svga_context *svga, const struct pipe_blit_info *blit_info) { - /* check that the blit src/dst regions are same size, no flipping, etc. */ - if (blit_info->src.box.width != blit_info->dst.box.width || - blit_info->src.box.height != blit_info->dst.box.height) + if (!util_can_blit_via_copy_region(blit_info, FALSE) && + !util_can_blit_via_copy_region(blit_info, TRUE)) return false; /* For depth+stencil formats, copy with mask != PIPE_MASK_ZS is not @@ -287,9 +286,7 @@ can_blit_via_svga_copy_region(struct svga_context *svga, blit_info->mask != (PIPE_MASK_ZS)) return false; - if (blit_info->alpha_blend || - (svga->render_condition && blit_info->render_condition_enable) || - blit_info->scissor_enable) + if (svga->render_condition && blit_info->render_condition_enable) return false; return check_blending_and_srgb_cond(svga, blit_info);