From 018cd270b06fdab7afa10da0524c69fe1bb6bd34 Mon Sep 17 00:00:00 2001 From: Lingfeng Yang Date: Wed, 26 Dec 2018 16:43:40 -0800 Subject: [PATCH] pipeline timestamps test: fix too-large VkBufferCopy The pipeline timestamps test intends to copy the entire buffer contents from source to destination, but the copy is done for 512 bytes, which currently exceeds the size of the buffer, 256. That is not valid behavior. Components: Vulkan Affects: dEQP-VK.pipeline.timestamp.transfer_tests.transfer_stage_with_copy_buffer_method VK-GL-CTS Issue: 1535 Change-Id: I1ab1ef3a94f4d22702d37c86b6aa2c21c0919a7c --- .../vulkancts/modules/vulkan/pipeline/vktPipelineTimestampTests.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/external/vulkancts/modules/vulkan/pipeline/vktPipelineTimestampTests.cpp b/external/vulkancts/modules/vulkan/pipeline/vktPipelineTimestampTests.cpp index 94f2bab..7a4239b 100644 --- a/external/vulkancts/modules/vulkan/pipeline/vktPipelineTimestampTests.cpp +++ b/external/vulkancts/modules/vulkan/pipeline/vktPipelineTimestampTests.cpp @@ -1725,9 +1725,9 @@ void TransferTestInstance::configCommandBuffer(void) { const VkBufferCopy copyBufRegion = { - 0u, // VkDeviceSize srcOffset; - 0u, // VkDeviceSize destOffset; - 512u, // VkDeviceSize copySize; + 0u, // VkDeviceSize srcOffset; + 0u, // VkDeviceSize destOffset; + m_bufSize, // VkDeviceSize copySize; }; vk.cmdCopyBuffer(*m_cmdBuffer, *m_srcBuffer, *m_dstBuffer, 1u, ©BufRegion); break; -- 2.7.4