From 4c7f4c25d03cb1bd88fade526386e50e45e71e87 Mon Sep 17 00:00:00 2001 From: Chad Versace Date: Thu, 7 Jan 2016 13:46:20 -0800 Subject: [PATCH] anv/meta: Fix hardcoded format size in anv_CmdCopy* When looping through VkBufferImageCopy regions, for each region we incremented the offset into the VkBuffer assuming the format size was 4. Fixes CTS tests dEQP-VK.pipeline.image.view_type.cube_array.3d.* on Skylake. --- src/vulkan/anv_meta.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/vulkan/anv_meta.c b/src/vulkan/anv_meta.c index e351db08728..b61cda793d5 100644 --- a/src/vulkan/anv_meta.c +++ b/src/vulkan/anv_meta.c @@ -1234,9 +1234,9 @@ void anv_CmdCopyBufferToImage( * increment the offset directly in the image effectively * re-binding it to different backing memory. */ - /* XXX: Insert a real CPP */ src_image->offset += src_image->extent.width * - src_image->extent.height * 4; + src_image->extent.height * + src_image->format->isl_layout->bs; } anv_DestroyImage(vk_device, anv_image_to_handle(src_image), @@ -1336,9 +1336,9 @@ void anv_CmdCopyImageToBuffer( * increment the offset directly in the image effectively * re-binding it to different backing memory. */ - /* XXX: Insert a real CPP */ dest_image->offset += dest_image->extent.width * - dest_image->extent.height * 4; + dest_image->extent.height * + src_image->format->isl_layout->bs; } anv_DestroyImage(vk_device, anv_image_to_handle(dest_image),