[Mesa-dev] [PATCH 5/9] radeonsi: remove a workaround for inexact *8_SNORM blits

Marek Olšák maraeo at gmail.com
Wed Mar 29 17:58:50 UTC 2017


From: Marek Olšák <marek.olsak at amd.com>

All tests pass on Fiji now. This prevents DCC disablement due to
incompatible DCC formats due to the fallback.
---
 src/gallium/drivers/radeonsi/si_blit.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/gallium/drivers/radeonsi/si_blit.c b/src/gallium/drivers/radeonsi/si_blit.c
index 0466f19..bc5c2d6 100644
--- a/src/gallium/drivers/radeonsi/si_blit.c
+++ b/src/gallium/drivers/radeonsi/si_blit.c
@@ -888,23 +888,21 @@ void si_resource_copy_region(struct pipe_context *ctx,
 
 		sbox.x = util_format_get_nblocksx(src->format, src_box->x);
 		sbox.y = util_format_get_nblocksy(src->format, src_box->y);
 		sbox.z = src_box->z;
 		sbox.width = util_format_get_nblocksx(src->format, src_box->width);
 		sbox.height = util_format_get_nblocksy(src->format, src_box->height);
 		sbox.depth = src_box->depth;
 		src_box = &sbox;
 
 		src_force_level = src_level;
-	} else if (!util_blitter_is_copy_supported(sctx->blitter, dst, src) ||
-		   /* also *8_SNORM has precision issues, use UNORM instead */
-		   util_format_is_snorm8(src->format)) {
+	} else if (!util_blitter_is_copy_supported(sctx->blitter, dst, src)) {
 		if (util_format_is_subsampled_422(src->format)) {
 			src_templ.format = PIPE_FORMAT_R8G8B8A8_UINT;
 			dst_templ.format = PIPE_FORMAT_R8G8B8A8_UINT;
 
 			dst_width = util_format_get_nblocksx(dst->format, dst_width);
 			src_width0 = util_format_get_nblocksx(src->format, src_width0);
 
 			dstx = util_format_get_nblocksx(dst->format, dstx);
 
 			sbox = *src_box;
-- 
2.7.4



More information about the mesa-dev mailing list