From c811af767e61a3df7bd844b49812d85a9b910aa3 Mon Sep 17 00:00:00 2001 From: Jason Ekstrand Date: Mon, 10 Sep 2018 16:36:10 -0500 Subject: [PATCH] anv/so_memcpy: Don't consider src/dst_offset when computing block size The only thing that matters is the size since we never specify any offsets in terms of blocks. Reviewed-by: Lionel Landwerlin --- src/intel/vulkan/genX_gpu_memcpy.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/intel/vulkan/genX_gpu_memcpy.c b/src/intel/vulkan/genX_gpu_memcpy.c index 11ffd6fb436..fd78f4d125b 100644 --- a/src/intel/vulkan/genX_gpu_memcpy.c +++ b/src/intel/vulkan/genX_gpu_memcpy.c @@ -137,10 +137,8 @@ genX(cmd_buffer_so_memcpy)(struct anv_cmd_buffer *cmd_buffer, assert(src.offset + size <= src.bo->size); /* The maximum copy block size is 4 32-bit components at a time. */ - unsigned bs = 16; - bs = gcd_pow2_u64(bs, src.offset); - bs = gcd_pow2_u64(bs, dst.offset); - bs = gcd_pow2_u64(bs, size); + assert(size % 4 == 0); + unsigned bs = gcd_pow2_u64(16, size); enum isl_format format; switch (bs) {