blob: 3616e2ed6b3a1cfe3bbb10c37f706a50315ddc44 [file] [log] [blame]
Jeremy Gebben170781d2020-11-19 16:21:21 -07001/*
sfricke-samsungae54c1e2022-01-21 05:35:21 -08002 * Copyright (c) 2015-2022 The Khronos Group Inc.
3 * Copyright (c) 2015-2022 Valve Corporation
4 * Copyright (c) 2015-2022 LunarG, Inc.
5 * Copyright (c) 2015-2022 Google, Inc.
Jeremy Gebben170781d2020-11-19 16:21:21 -07006 *
7 * Licensed under the Apache License, Version 2.0 (the "License");
8 * you may not use this file except in compliance with the License.
9 * You may obtain a copy of the License at
10 *
11 * http://www.apache.org/licenses/LICENSE-2.0
12 *
13 * Author: Chia-I Wu <olvaffe@gmail.com>
14 * Author: Chris Forbes <chrisf@ijw.co.nz>
15 * Author: Courtney Goeltzenleuchter <courtney@LunarG.com>
16 * Author: Mark Lobodzinski <mark@lunarg.com>
17 * Author: Mike Stroyan <mike@LunarG.com>
18 * Author: Tobin Ehlis <tobine@google.com>
19 * Author: Tony Barbour <tony@LunarG.com>
20 * Author: Cody Northrop <cnorthrop@google.com>
21 * Author: Dave Houlton <daveh@lunarg.com>
22 * Author: Jeremy Kniager <jeremyk@lunarg.com>
23 * Author: Shannon McPherson <shannon@lunarg.com>
24 * Author: John Zulauf <jzulauf@lunarg.com>
25 */
26#include <type_traits>
27
28#include "cast_utils.h"
29#include "layer_validation_tests.h"
30
31TEST_F(VkSyncValTest, SyncBufferCopyHazards) {
32 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
33 if (DeviceExtensionSupported(gpu(), nullptr, VK_AMD_BUFFER_MARKER_EXTENSION_NAME)) {
34 m_device_extension_names.push_back(VK_AMD_BUFFER_MARKER_EXTENSION_NAME);
35 }
36 ASSERT_NO_FATAL_FAILURE(InitState(nullptr, nullptr, VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT));
37 bool has_amd_buffer_maker = DeviceExtensionEnabled(VK_AMD_BUFFER_MARKER_EXTENSION_NAME);
38
39 VkBufferObj buffer_a;
40 VkBufferObj buffer_b;
41 VkBufferObj buffer_c;
42 VkMemoryPropertyFlags mem_prop = VK_MEMORY_PROPERTY_DEVICE_LOCAL_BIT;
43 buffer_a.init_as_src_and_dst(*m_device, 256, mem_prop);
44 buffer_b.init_as_src_and_dst(*m_device, 256, mem_prop);
45 buffer_c.init_as_src_and_dst(*m_device, 256, mem_prop);
46
47 VkBufferCopy region = {0, 0, 256};
48 VkBufferCopy front2front = {0, 0, 128};
49 VkBufferCopy front2back = {0, 128, 128};
50 VkBufferCopy back2back = {128, 128, 128};
51
52 auto cb = m_commandBuffer->handle();
53 m_commandBuffer->begin();
54
55 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &region);
56
57 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
58 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_a.handle(), 1, &region);
59 m_errorMonitor->VerifyFound();
60
61 // Use the barrier to clean up the WAW, and try again. (and show that validation is accounting for the barrier effect too.)
Mark Lobodzinski07d0a612020-12-30 15:42:31 -070062 auto buffer_barrier = LvlInitStruct<VkBufferMemoryBarrier>();
Jeremy Gebben170781d2020-11-19 16:21:21 -070063 buffer_barrier.srcAccessMask = VK_ACCESS_TRANSFER_READ_BIT;
64 buffer_barrier.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
65 buffer_barrier.buffer = buffer_a.handle();
66 buffer_barrier.offset = 0;
67 buffer_barrier.size = 256;
68 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 0, nullptr, 1, &buffer_barrier, 0,
69 nullptr);
70
71 m_errorMonitor->ExpectSuccess();
72 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_a.handle(), 1, &front2front);
73 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_a.handle(), 1, &back2back);
74 m_errorMonitor->VerifyNotFound();
75
76 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
77 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_a.handle(), 1, &front2back);
78 m_errorMonitor->VerifyFound();
79
80 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
81 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_b.handle(), 1, &region);
82 m_errorMonitor->VerifyFound();
83
84 // NOTE: Since the previous command skips in validation, the state update is never done, and the validation layer thus doesn't
85 // record the write operation to b. So we'll need to repeat it successfully to set up for the *next* test.
86
87 // Use the barrier to clean up the WAW, and try again. (and show that validation is accounting for the barrier effect too.)
Mark Lobodzinski07d0a612020-12-30 15:42:31 -070088 auto mem_barrier = LvlInitStruct<VkMemoryBarrier>();
Jeremy Gebben170781d2020-11-19 16:21:21 -070089 mem_barrier.srcAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
90 mem_barrier.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
91 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 1, &mem_barrier, 0, nullptr, 0,
92 nullptr);
93 m_errorMonitor->ExpectSuccess();
94
95 vk::CmdCopyBuffer(m_commandBuffer->handle(), buffer_c.handle(), buffer_b.handle(), 1, &region);
96 m_errorMonitor->VerifyNotFound();
97
98 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
99 mem_barrier.srcAccessMask = VK_ACCESS_TRANSFER_READ_BIT; // Protect C but not B
100 mem_barrier.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
101 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 1, &mem_barrier, 0, nullptr, 0,
102 nullptr);
103 vk::CmdCopyBuffer(m_commandBuffer->handle(), buffer_b.handle(), buffer_c.handle(), 1, &region);
104 m_errorMonitor->VerifyFound();
105
106 m_commandBuffer->end();
107
108 // CmdFillBuffer
109 m_errorMonitor->ExpectSuccess();
110 m_commandBuffer->reset();
111 m_commandBuffer->begin();
112 vk::CmdFillBuffer(m_commandBuffer->handle(), buffer_a.handle(), 0, 256, 1);
113 m_commandBuffer->end();
114 m_errorMonitor->VerifyNotFound();
115
116 m_commandBuffer->reset();
117 m_commandBuffer->begin();
118 vk::CmdCopyBuffer(cb, buffer_b.handle(), buffer_a.handle(), 1, &region);
119 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
120 vk::CmdFillBuffer(m_commandBuffer->handle(), buffer_a.handle(), 0, 256, 1);
121 m_errorMonitor->VerifyFound();
122 m_commandBuffer->end();
123
124 // CmdUpdateBuffer
125 int i = 10;
126 m_errorMonitor->ExpectSuccess();
127 m_commandBuffer->reset();
128 m_commandBuffer->begin();
129 vk::CmdUpdateBuffer(m_commandBuffer->handle(), buffer_a.handle(), 0, sizeof(i), &i);
130 m_commandBuffer->end();
131 m_errorMonitor->VerifyNotFound();
132
133 m_commandBuffer->reset();
134 m_commandBuffer->begin();
135 vk::CmdCopyBuffer(cb, buffer_b.handle(), buffer_a.handle(), 1, &region);
136 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
137 vk::CmdUpdateBuffer(m_commandBuffer->handle(), buffer_a.handle(), 0, sizeof(i), &i);
138 m_errorMonitor->VerifyFound();
139 m_commandBuffer->end();
140
John Zulaufcbf67cf2021-04-26 21:06:32 -0600141 // Create secondary buffers to use
John Zulaufee17cce2021-04-15 18:21:38 -0600142 m_errorMonitor->ExpectSuccess();
John Zulaufcbf67cf2021-04-26 21:06:32 -0600143 VkCommandBufferObj secondary_cb1(m_device, m_commandPool, VK_COMMAND_BUFFER_LEVEL_SECONDARY);
144 VkCommandBuffer scb1 = secondary_cb1.handle();
145 secondary_cb1.begin();
146 vk::CmdCopyBuffer(scb1, buffer_c.handle(), buffer_a.handle(), 1, &front2front);
147 secondary_cb1.end();
148 m_errorMonitor->VerifyNotFound();
149
150 m_errorMonitor->ExpectSuccess();
151 VkCommandBufferObj secondary_cb2(m_device, m_commandPool, VK_COMMAND_BUFFER_LEVEL_SECONDARY);
152 VkCommandBuffer scb2 = secondary_cb2.handle();
153 secondary_cb2.begin();
154 vk::CmdCopyBuffer(scb2, buffer_a.handle(), buffer_c.handle(), 1, &front2front);
155 secondary_cb2.end();
156 m_errorMonitor->VerifyNotFound();
157
158 m_errorMonitor->ExpectSuccess();
159 VkCommandBufferObj secondary_cb3(m_device, m_commandPool, VK_COMMAND_BUFFER_LEVEL_SECONDARY);
160 VkCommandBuffer scb3 = secondary_cb3.handle();
161 secondary_cb3.begin();
162 secondary_cb3.PipelineBarrier(VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 0, nullptr, 0, nullptr, 0,
163 nullptr);
164 secondary_cb3.end();
165 m_errorMonitor->VerifyNotFound();
166
167 m_errorMonitor->ExpectSuccess();
168 VkCommandBufferObj secondary_cb4(m_device, m_commandPool, VK_COMMAND_BUFFER_LEVEL_SECONDARY);
169 VkCommandBuffer scb4 = secondary_cb4.handle();
170 secondary_cb4.begin();
171 vk::CmdCopyBuffer(scb4, buffer_b.handle(), buffer_c.handle(), 1, &front2front);
172 secondary_cb4.end();
173 m_errorMonitor->VerifyNotFound();
174
175 // One secondary CB hazard with active command buffer
176 m_errorMonitor->ExpectSuccess();
John Zulaufee17cce2021-04-15 18:21:38 -0600177 m_commandBuffer->reset();
178 m_commandBuffer->begin();
179 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_a.handle(), 1, &front2front);
180 m_errorMonitor->VerifyNotFound();
181 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
John Zulaufcbf67cf2021-04-26 21:06:32 -0600182 vk::CmdExecuteCommands(cb, 1, &scb1);
John Zulaufee17cce2021-04-15 18:21:38 -0600183 m_errorMonitor->VerifyFound();
184 m_commandBuffer->end();
185
John Zulaufcbf67cf2021-04-26 21:06:32 -0600186 // Two secondary CB hazard with each other
John Zulaufee17cce2021-04-15 18:21:38 -0600187 m_commandBuffer->reset();
John Zulaufcbf67cf2021-04-26 21:06:32 -0600188 m_commandBuffer->begin();
189 m_errorMonitor->VerifyNotFound();
190 // This is also a "SYNC-HAZARD-WRITE_AFTER_WRITE" present, but only the first hazard is reported.
191 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
192 {
193 VkCommandBuffer two_cbs[2] = {scb1, scb2};
194 vk::CmdExecuteCommands(cb, 2, two_cbs);
195 }
196 m_errorMonitor->VerifyFound();
197 m_commandBuffer->end();
John Zulaufee17cce2021-04-15 18:21:38 -0600198
John Zulaufcbf67cf2021-04-26 21:06:32 -0600199 // Two secondary CB hazard with each other
200 m_commandBuffer->reset();
201 m_commandBuffer->begin();
202 m_errorMonitor->VerifyNotFound();
203 {
204 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
205 VkCommandBuffer two_cbs[2] = {scb1, scb4};
206 vk::CmdExecuteCommands(cb, 2, two_cbs);
207 m_errorMonitor->VerifyFound();
208 }
209 m_commandBuffer->end();
210
211 // Add a secondary CB with a barrier
212 m_commandBuffer->reset();
213 m_commandBuffer->begin();
214 {
215 m_errorMonitor->ExpectSuccess();
216 VkCommandBuffer three_cbs[3] = {scb1, scb3, scb4};
217 vk::CmdExecuteCommands(cb, 3, three_cbs);
218 m_errorMonitor->VerifyNotFound();
219 }
220 m_commandBuffer->end();
221
222 m_commandBuffer->reset();
Jeremy Gebben170781d2020-11-19 16:21:21 -0700223 // CmdWriteBufferMarkerAMD
224 if (has_amd_buffer_maker) {
225 auto fpCmdWriteBufferMarkerAMD =
226 (PFN_vkCmdWriteBufferMarkerAMD)vk::GetDeviceProcAddr(m_device->device(), "vkCmdWriteBufferMarkerAMD");
227 if (!fpCmdWriteBufferMarkerAMD) {
228 printf("%s Test requires unsupported vkCmdWriteBufferMarkerAMD feature. Skipped.\n", kSkipPrefix);
229 } else {
230 m_errorMonitor->ExpectSuccess();
231 m_commandBuffer->reset();
232 m_commandBuffer->begin();
233 fpCmdWriteBufferMarkerAMD(m_commandBuffer->handle(), VK_PIPELINE_STAGE_TRANSFER_BIT, buffer_a.handle(), 0, 1);
234 m_commandBuffer->end();
235 m_errorMonitor->VerifyNotFound();
236
237 m_commandBuffer->reset();
238 m_commandBuffer->begin();
239 vk::CmdCopyBuffer(cb, buffer_b.handle(), buffer_a.handle(), 1, &region);
240 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
241 fpCmdWriteBufferMarkerAMD(m_commandBuffer->handle(), VK_PIPELINE_STAGE_TRANSFER_BIT, buffer_a.handle(), 0, 1);
242 m_errorMonitor->VerifyFound();
243 m_commandBuffer->end();
244 }
245 } else {
246 printf("%s Test requires unsupported vkCmdWriteBufferMarkerAMD feature. Skipped.\n", kSkipPrefix);
247 }
248}
249
Jeremy Gebben5c1bb2d2021-02-15 08:58:04 -0700250TEST_F(VkSyncValTest, Sync2BufferCopyHazards) {
251 SetTargetApiVersion(VK_API_VERSION_1_2);
252 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
253 if (DeviceExtensionSupported(gpu(), nullptr, VK_KHR_SYNCHRONIZATION_2_EXTENSION_NAME)) {
254 m_device_extension_names.push_back(VK_KHR_SYNCHRONIZATION_2_EXTENSION_NAME);
255 } else {
256 printf("%s Synchronization2 not supported, skipping test\n", kSkipPrefix);
257 return;
258 }
259
260 if (!CheckSynchronization2SupportAndInitState(this)) {
261 printf("%s Synchronization2 not supported, skipping test\n", kSkipPrefix);
262 return;
263 }
264 auto fpCmdPipelineBarrier2KHR = (PFN_vkCmdPipelineBarrier2KHR)vk::GetDeviceProcAddr(m_device->device(), "vkCmdPipelineBarrier2KHR");
265
266 VkBufferObj buffer_a;
267 VkBufferObj buffer_b;
268 VkBufferObj buffer_c;
269 VkMemoryPropertyFlags mem_prop = VK_MEMORY_PROPERTY_DEVICE_LOCAL_BIT;
270 buffer_a.init_as_src_and_dst(*m_device, 256, mem_prop);
271 buffer_b.init_as_src_and_dst(*m_device, 256, mem_prop);
272 buffer_c.init_as_src_and_dst(*m_device, 256, mem_prop);
273
274 VkBufferCopy region = {0, 0, 256};
275 VkBufferCopy front2front = {0, 0, 128};
276 VkBufferCopy front2back = {0, 128, 128};
277 VkBufferCopy back2back = {128, 128, 128};
278
279 auto cb = m_commandBuffer->handle();
280 m_commandBuffer->begin();
281
282 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &region);
283
284 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
285 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_a.handle(), 1, &region);
286 m_errorMonitor->VerifyFound();
287
288 // Use the barrier to clean up the WAW, and try again. (and show that validation is accounting for the barrier effect too.)
289 {
290 auto buffer_barrier = lvl_init_struct<VkBufferMemoryBarrier2KHR>();
291 buffer_barrier.srcStageMask = VK_PIPELINE_STAGE_2_COPY_BIT_KHR;
292 buffer_barrier.dstStageMask = VK_PIPELINE_STAGE_2_COPY_BIT_KHR;
293 buffer_barrier.srcAccessMask = VK_ACCESS_2_TRANSFER_READ_BIT_KHR;
294 buffer_barrier.dstAccessMask = VK_ACCESS_2_TRANSFER_WRITE_BIT_KHR;
295 buffer_barrier.buffer = buffer_a.handle();
296 buffer_barrier.offset = 0;
297 buffer_barrier.size = 256;
298 auto dep_info = lvl_init_struct<VkDependencyInfoKHR>();
299 dep_info.bufferMemoryBarrierCount = 1;
300 dep_info.pBufferMemoryBarriers = &buffer_barrier;
301 fpCmdPipelineBarrier2KHR(cb, &dep_info);
302 }
303
304 m_errorMonitor->ExpectSuccess();
305 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_a.handle(), 1, &front2front);
306 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_a.handle(), 1, &back2back);
307 m_errorMonitor->VerifyNotFound();
308
309 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
310 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_a.handle(), 1, &front2back);
311 m_errorMonitor->VerifyFound();
312
313 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
314 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_b.handle(), 1, &region);
315 m_errorMonitor->VerifyFound();
316
317 // NOTE: Since the previous command skips in validation, the state update is never done, and the validation layer thus doesn't
318 // record the write operation to b. So we'll need to repeat it successfully to set up for the *next* test.
319
320 // Use the barrier to clean up the WAW, and try again. (and show that validation is accounting for the barrier effect too.)
321 {
322 auto mem_barrier = lvl_init_struct<VkMemoryBarrier2KHR>();
323 mem_barrier.srcStageMask = VK_PIPELINE_STAGE_2_COPY_BIT_KHR;
324 mem_barrier.dstStageMask = VK_PIPELINE_STAGE_2_COPY_BIT_KHR;
325 mem_barrier.srcAccessMask = VK_ACCESS_2_TRANSFER_WRITE_BIT_KHR;
326 mem_barrier.dstAccessMask = VK_ACCESS_2_TRANSFER_WRITE_BIT_KHR;
327 auto dep_info = lvl_init_struct<VkDependencyInfoKHR>();
328 dep_info.memoryBarrierCount = 1;
329 dep_info.pMemoryBarriers = &mem_barrier;
330 fpCmdPipelineBarrier2KHR(cb, &dep_info);
331 m_errorMonitor->ExpectSuccess();
332
333 vk::CmdCopyBuffer(m_commandBuffer->handle(), buffer_c.handle(), buffer_b.handle(), 1, &region);
334 m_errorMonitor->VerifyNotFound();
335
336 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
337 mem_barrier.srcAccessMask = VK_ACCESS_2_TRANSFER_READ_BIT_KHR; // Protect C but not B
338 mem_barrier.dstAccessMask = VK_ACCESS_2_TRANSFER_WRITE_BIT_KHR;
339 fpCmdPipelineBarrier2KHR(cb, &dep_info);
340 vk::CmdCopyBuffer(m_commandBuffer->handle(), buffer_b.handle(), buffer_c.handle(), 1, &region);
341 m_errorMonitor->VerifyFound();
342
343 m_commandBuffer->end();
344 }
345}
346
Jeremy Gebben170781d2020-11-19 16:21:21 -0700347TEST_F(VkSyncValTest, SyncCopyOptimalImageHazards) {
348 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
349 ASSERT_NO_FATAL_FAILURE(InitState(nullptr, nullptr, VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT));
350
351 VkImageUsageFlags usage = VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT;
352 VkFormat format = VK_FORMAT_R8G8B8A8_UNORM;
353 VkImageObj image_a(m_device);
354 auto image_ci = VkImageObj::ImageCreateInfo2D(128, 128, 1, 2, format, usage, VK_IMAGE_TILING_OPTIMAL);
355 image_a.Init(image_ci);
356 ASSERT_TRUE(image_a.initialized());
357
358 VkImageObj image_b(m_device);
359 image_b.Init(image_ci);
360 ASSERT_TRUE(image_b.initialized());
361
362 VkImageObj image_c(m_device);
363 image_c.Init(image_ci);
364 ASSERT_TRUE(image_c.initialized());
365
366 VkImageSubresourceLayers layers_all{VK_IMAGE_ASPECT_COLOR_BIT, 0, 0, 2};
367 VkImageSubresourceLayers layers_0{VK_IMAGE_ASPECT_COLOR_BIT, 0, 0, 1};
368 VkImageSubresourceLayers layers_1{VK_IMAGE_ASPECT_COLOR_BIT, 0, 1, 1};
369 VkImageSubresourceRange full_subresource_range{VK_IMAGE_ASPECT_COLOR_BIT, 0, 1, 0, 2};
370 VkOffset3D zero_offset{0, 0, 0};
371 VkOffset3D half_offset{64, 64, 0};
372 VkExtent3D full_extent{128, 128, 1}; // <-- image type is 2D
373 VkExtent3D half_extent{64, 64, 1}; // <-- image type is 2D
374
375 VkImageCopy full_region = {layers_all, zero_offset, layers_all, zero_offset, full_extent};
376 VkImageCopy region_0_to_0 = {layers_0, zero_offset, layers_0, zero_offset, full_extent};
377 VkImageCopy region_0_to_1 = {layers_0, zero_offset, layers_1, zero_offset, full_extent};
378 VkImageCopy region_1_to_1 = {layers_1, zero_offset, layers_1, zero_offset, full_extent};
379 VkImageCopy region_0_front = {layers_0, zero_offset, layers_0, zero_offset, half_extent};
380 VkImageCopy region_0_back = {layers_0, half_offset, layers_0, half_offset, half_extent};
381
382 m_commandBuffer->begin();
383
384 image_c.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
385 image_b.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
386 image_a.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
387
388 auto cb = m_commandBuffer->handle();
389
390 vk::CmdCopyImage(cb, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &full_region);
391
392 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
393 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &full_region);
394 m_errorMonitor->VerifyFound();
395
396 // Use the barrier to clean up the WAW, and try again. (and show that validation is accounting for the barrier effect too.)
Mark Lobodzinski07d0a612020-12-30 15:42:31 -0700397 auto image_barrier = LvlInitStruct<VkImageMemoryBarrier>();
Jeremy Gebben170781d2020-11-19 16:21:21 -0700398 image_barrier.srcAccessMask = VK_ACCESS_TRANSFER_READ_BIT;
399 image_barrier.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
400 image_barrier.image = image_a.handle();
401 image_barrier.subresourceRange = full_subresource_range;
402 image_barrier.oldLayout = VK_IMAGE_LAYOUT_GENERAL;
403 image_barrier.newLayout = VK_IMAGE_LAYOUT_GENERAL;
404 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 0, nullptr, 0, nullptr, 1,
405 &image_barrier);
406
407 m_errorMonitor->ExpectSuccess();
408 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region_0_to_0);
409 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region_1_to_1);
410 m_errorMonitor->VerifyNotFound();
411
412 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
413 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region_0_to_1);
414 m_errorMonitor->VerifyFound();
415
416 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
417 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &full_region);
418 m_errorMonitor->VerifyFound();
419
420 // NOTE: Since the previous command skips in validation, the state update is never done, and the validation layer thus doesn't
421 // record the write operation to b. So we'll need to repeat it successfully to set up for the *next* test.
422
423 // Use the barrier to clean up the WAW, and try again. (and show that validation is accounting for the barrier effect too.)
Mark Lobodzinski07d0a612020-12-30 15:42:31 -0700424 auto mem_barrier = LvlInitStruct<VkMemoryBarrier>();
Jeremy Gebben170781d2020-11-19 16:21:21 -0700425 mem_barrier.srcAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
426 mem_barrier.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
427 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 1, &mem_barrier, 0, nullptr, 0,
428 nullptr);
429 m_errorMonitor->ExpectSuccess();
430 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &full_region);
431 m_errorMonitor->VerifyNotFound();
432
433 // Use barrier to protect last reader, but not last writer...
434 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
435 mem_barrier.srcAccessMask = VK_ACCESS_TRANSFER_READ_BIT; // Protects C but not B
436 mem_barrier.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
437 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 1, &mem_barrier, 0, nullptr, 0,
438 nullptr);
439 vk::CmdCopyImage(cb, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &full_region);
440 m_errorMonitor->VerifyFound();
441
442 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region_0_front);
443 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
444 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region_0_front);
445 m_errorMonitor->VerifyFound();
446
447 m_errorMonitor->ExpectSuccess();
448 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region_0_back);
449 m_errorMonitor->VerifyNotFound();
450
451 m_commandBuffer->end();
452
John Zulaufe972b752021-05-04 15:47:17 -0600453 // Test secondary command buffers
454 // Create secondary buffers to use
455 m_errorMonitor->ExpectSuccess();
456 VkCommandBufferObj secondary_cb1(m_device, m_commandPool, VK_COMMAND_BUFFER_LEVEL_SECONDARY);
457 VkCommandBuffer scb1 = secondary_cb1.handle();
458 secondary_cb1.begin();
459 vk::CmdCopyImage(scb1, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &full_region);
460 secondary_cb1.end();
461 m_errorMonitor->VerifyNotFound();
462
463 auto record_primary = [&]() {
464 m_commandBuffer->reset();
465 m_commandBuffer->begin();
466 vk::CmdCopyImage(cb, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &full_region);
467 vk::CmdExecuteCommands(cb, 1, &scb1);
468 m_commandBuffer->end();
469 };
470
471 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
472 record_primary();
473 m_errorMonitor->VerifyFound();
474
475 m_errorMonitor->ExpectSuccess();
476 // With a barrier...
477 secondary_cb1.reset();
478 secondary_cb1.begin();
479 vk::CmdPipelineBarrier(scb1, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 1, &mem_barrier, 0, nullptr, 0,
480 nullptr);
481 vk::CmdCopyImage(scb1, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &full_region);
482 secondary_cb1.end();
483 record_primary();
484 m_errorMonitor->VerifyNotFound();
485
486 auto image_transition_barrier = image_barrier;
487 image_transition_barrier.image = image_a.handle();
488 image_transition_barrier.oldLayout = VK_IMAGE_LAYOUT_GENERAL;
489 image_transition_barrier.newLayout = VK_IMAGE_LAYOUT_TRANSFER_SRC_OPTIMAL;
490
491 m_errorMonitor->ExpectSuccess();
492 secondary_cb1.reset();
493 secondary_cb1.begin();
494 // Use the wrong stage, get an error
495 vk::CmdPipelineBarrier(scb1, VK_PIPELINE_STAGE_TOP_OF_PIPE_BIT, VK_PIPELINE_STAGE_TOP_OF_PIPE_BIT, 0, 0, nullptr, 0, nullptr, 1,
496 &image_transition_barrier);
497 secondary_cb1.end();
498 m_errorMonitor->VerifyNotFound();
499
500 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
501 record_primary();
502 m_errorMonitor->VerifyFound();
503
504 // CmdResolveImage hazard testing
Jeremy Gebben170781d2020-11-19 16:21:21 -0700505 VkImageFormatProperties formProps = {{0, 0, 0}, 0, 0, 0, 0};
506 vk::GetPhysicalDeviceImageFormatProperties(m_device->phy().handle(), VK_FORMAT_R8G8B8A8_UNORM, VK_IMAGE_TYPE_2D,
507 VK_IMAGE_TILING_OPTIMAL, VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT, 0, &formProps);
508
509 if (!(formProps.sampleCounts & VK_SAMPLE_COUNT_2_BIT)) {
510 printf("%s CmdResolveImage Test requires unsupported VK_SAMPLE_COUNT_2_BIT feature. Skipped.\n", kSkipPrefix);
511 } else {
512 m_errorMonitor->ExpectSuccess();
513 VkImageObj image_s2_a(m_device), image_s2_b(m_device);
514 image_ci.samples = VK_SAMPLE_COUNT_2_BIT;
515 image_s2_a.Init(image_ci);
516 ASSERT_TRUE(image_s2_a.initialized());
517
518 image_s2_b.Init(image_ci);
519 ASSERT_TRUE(image_s2_b.initialized());
520
521 VkImageResolve r_full_region = {layers_all, zero_offset, layers_all, zero_offset, full_extent};
522
523 m_commandBuffer->reset();
524 m_commandBuffer->begin();
525 image_s2_a.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
526 image_s2_b.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
527 vk::CmdResolveImage(cb, image_s2_a.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
528 &r_full_region);
529 m_commandBuffer->end();
530 m_errorMonitor->VerifyNotFound();
531
532 m_commandBuffer->reset();
533 m_commandBuffer->begin();
534 vk::CmdCopyImage(cb, image_s2_b.handle(), VK_IMAGE_LAYOUT_GENERAL, image_s2_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
535 &full_region);
536 vk::CmdCopyImage(cb, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &full_region);
537
538 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
539 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
540 vk::CmdResolveImage(cb, image_s2_a.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
541 &r_full_region);
542 m_errorMonitor->VerifyFound();
543
544 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
545 vk::CmdResolveImage(cb, image_s2_b.handle(), VK_IMAGE_LAYOUT_GENERAL, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
546 &r_full_region);
547 m_errorMonitor->VerifyFound();
548 m_commandBuffer->end();
549 }
550}
551
Jeremy Gebben5c1bb2d2021-02-15 08:58:04 -0700552TEST_F(VkSyncValTest, Sync2CopyOptimalImageHazards) {
553 SetTargetApiVersion(VK_API_VERSION_1_2);
554 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
555 if (DeviceExtensionSupported(gpu(), nullptr, VK_KHR_SYNCHRONIZATION_2_EXTENSION_NAME)) {
556 m_device_extension_names.push_back(VK_KHR_SYNCHRONIZATION_2_EXTENSION_NAME);
557 } else {
558 printf("%s Synchronization2 not supported, skipping test\n", kSkipPrefix);
559 return;
560 }
561
562 if (!CheckSynchronization2SupportAndInitState(this)) {
563 printf("%s Synchronization2 not supported, skipping test\n", kSkipPrefix);
564 return;
565 }
566 auto fpCmdPipelineBarrier2KHR = (PFN_vkCmdPipelineBarrier2KHR)vk::GetDeviceProcAddr(m_device->device(), "vkCmdPipelineBarrier2KHR");
567
568 VkImageUsageFlags usage = VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT;
569 VkFormat format = VK_FORMAT_R8G8B8A8_UNORM;
570 VkImageObj image_a(m_device);
571 auto image_ci = VkImageObj::ImageCreateInfo2D(128, 128, 1, 2, format, usage, VK_IMAGE_TILING_OPTIMAL);
572 image_a.Init(image_ci);
573 ASSERT_TRUE(image_a.initialized());
574
575 VkImageObj image_b(m_device);
576 image_b.Init(image_ci);
577 ASSERT_TRUE(image_b.initialized());
578
579 VkImageObj image_c(m_device);
580 image_c.Init(image_ci);
581 ASSERT_TRUE(image_c.initialized());
582
583 VkImageSubresourceLayers layers_all{VK_IMAGE_ASPECT_COLOR_BIT, 0, 0, 2};
584 VkImageSubresourceLayers layers_0{VK_IMAGE_ASPECT_COLOR_BIT, 0, 0, 1};
585 VkImageSubresourceLayers layers_1{VK_IMAGE_ASPECT_COLOR_BIT, 0, 1, 1};
586 VkImageSubresourceRange full_subresource_range{VK_IMAGE_ASPECT_COLOR_BIT, 0, 1, 0, 2};
587 VkOffset3D zero_offset{0, 0, 0};
588 VkOffset3D half_offset{64, 64, 0};
589 VkExtent3D full_extent{128, 128, 1}; // <-- image type is 2D
590 VkExtent3D half_extent{64, 64, 1}; // <-- image type is 2D
591
592 VkImageCopy full_region = {layers_all, zero_offset, layers_all, zero_offset, full_extent};
593 VkImageCopy region_0_to_0 = {layers_0, zero_offset, layers_0, zero_offset, full_extent};
594 VkImageCopy region_0_to_1 = {layers_0, zero_offset, layers_1, zero_offset, full_extent};
595 VkImageCopy region_1_to_1 = {layers_1, zero_offset, layers_1, zero_offset, full_extent};
596 VkImageCopy region_0_front = {layers_0, zero_offset, layers_0, zero_offset, half_extent};
597 VkImageCopy region_0_back = {layers_0, half_offset, layers_0, half_offset, half_extent};
598
599 m_commandBuffer->begin();
600
601 image_c.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
602 image_b.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
603 image_a.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
604
605 auto cb = m_commandBuffer->handle();
606
607 vk::CmdCopyImage(cb, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &full_region);
608
609 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
610 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &full_region);
611 m_errorMonitor->VerifyFound();
612
613 // Use the barrier to clean up the WAW, and try again. (and show that validation is accounting for the barrier effect too.)
614 {
615 auto image_barrier = lvl_init_struct<VkImageMemoryBarrier2KHR>();
616 image_barrier.srcStageMask = VK_PIPELINE_STAGE_2_COPY_BIT_KHR;
617 image_barrier.dstStageMask = VK_PIPELINE_STAGE_2_COPY_BIT_KHR;
618 image_barrier.srcAccessMask = VK_ACCESS_2_TRANSFER_READ_BIT_KHR;
619 image_barrier.dstAccessMask = VK_ACCESS_2_TRANSFER_WRITE_BIT_KHR;
620 image_barrier.image = image_a.handle();
621 image_barrier.subresourceRange = full_subresource_range;
622 image_barrier.oldLayout = VK_IMAGE_LAYOUT_GENERAL;
623 image_barrier.newLayout = VK_IMAGE_LAYOUT_GENERAL;
624 auto dep_info = lvl_init_struct<VkDependencyInfoKHR>();
625 dep_info.imageMemoryBarrierCount = 1;
626 dep_info.pImageMemoryBarriers = &image_barrier;
627 fpCmdPipelineBarrier2KHR(cb, &dep_info);
628 }
629
630 m_errorMonitor->ExpectSuccess();
631 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region_0_to_0);
632 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region_1_to_1);
633 m_errorMonitor->VerifyNotFound();
634
635 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
636 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region_0_to_1);
637 m_errorMonitor->VerifyFound();
638
639 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
640 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &full_region);
641 m_errorMonitor->VerifyFound();
642
643 // NOTE: Since the previous command skips in validation, the state update is never done, and the validation layer thus doesn't
644 // record the write operation to b. So we'll need to repeat it successfully to set up for the *next* test.
645
646 // Use the barrier to clean up the WAW, and try again. (and show that validation is accounting for the barrier effect too.)
647 {
648 auto mem_barrier = lvl_init_struct<VkMemoryBarrier2KHR>();
649 mem_barrier.srcStageMask = VK_PIPELINE_STAGE_2_COPY_BIT_KHR;
650 mem_barrier.dstStageMask = VK_PIPELINE_STAGE_2_COPY_BIT_KHR;
651 mem_barrier.srcAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
652 mem_barrier.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
653 auto dep_info = lvl_init_struct<VkDependencyInfoKHR>();
654 dep_info.memoryBarrierCount = 1;
655 dep_info.pMemoryBarriers = &mem_barrier;
656 fpCmdPipelineBarrier2KHR(cb, &dep_info);
657 m_errorMonitor->ExpectSuccess();
658 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &full_region);
659 m_errorMonitor->VerifyNotFound();
660
661 // Use barrier to protect last reader, but not last writer...
662 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
663 mem_barrier.srcAccessMask = VK_ACCESS_2_TRANSFER_READ_BIT_KHR; // Protects C but not B
664 mem_barrier.dstAccessMask = VK_ACCESS_2_TRANSFER_WRITE_BIT_KHR;
665 fpCmdPipelineBarrier2KHR(cb, &dep_info);
666 vk::CmdCopyImage(cb, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &full_region);
667 m_errorMonitor->VerifyFound();
668 }
669
670 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region_0_front);
671 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
672 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region_0_front);
673 m_errorMonitor->VerifyFound();
674
675 m_errorMonitor->ExpectSuccess();
676 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region_0_back);
677 m_errorMonitor->VerifyNotFound();
678
679 m_commandBuffer->end();
680}
681
Jeremy Gebben170781d2020-11-19 16:21:21 -0700682TEST_F(VkSyncValTest, SyncCopyOptimalMultiPlanarHazards) {
683 // TODO: Add code to enable sync validation
684 // Enable KHR multiplane req'd extensions
685 bool mp_extensions = InstanceExtensionSupported(VK_KHR_GET_PHYSICAL_DEVICE_PROPERTIES_2_EXTENSION_NAME,
686 VK_KHR_GET_MEMORY_REQUIREMENTS_2_SPEC_VERSION);
687 if (mp_extensions) {
688 m_instance_extension_names.push_back(VK_KHR_GET_PHYSICAL_DEVICE_PROPERTIES_2_EXTENSION_NAME);
689 }
690 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
Mike Schuchardt7cc57842021-09-15 10:49:59 -0700691 mp_extensions = mp_extensions && DeviceExtensionSupported(gpu(), nullptr, VK_KHR_MAINTENANCE_1_EXTENSION_NAME);
Jeremy Gebben170781d2020-11-19 16:21:21 -0700692 mp_extensions = mp_extensions && DeviceExtensionSupported(gpu(), nullptr, VK_KHR_GET_MEMORY_REQUIREMENTS_2_EXTENSION_NAME);
693 mp_extensions = mp_extensions && DeviceExtensionSupported(gpu(), nullptr, VK_KHR_BIND_MEMORY_2_EXTENSION_NAME);
694 mp_extensions = mp_extensions && DeviceExtensionSupported(gpu(), nullptr, VK_KHR_SAMPLER_YCBCR_CONVERSION_EXTENSION_NAME);
695 if (mp_extensions) {
Mike Schuchardt7cc57842021-09-15 10:49:59 -0700696 m_device_extension_names.push_back(VK_KHR_MAINTENANCE_1_EXTENSION_NAME);
Jeremy Gebben170781d2020-11-19 16:21:21 -0700697 m_device_extension_names.push_back(VK_KHR_GET_MEMORY_REQUIREMENTS_2_EXTENSION_NAME);
698 m_device_extension_names.push_back(VK_KHR_BIND_MEMORY_2_EXTENSION_NAME);
699 m_device_extension_names.push_back(VK_KHR_SAMPLER_YCBCR_CONVERSION_EXTENSION_NAME);
700 } else {
701 printf("%s test requires KHR multiplane extensions, not available. Skipping.\n", kSkipPrefix);
702 return;
703 }
704
705 ASSERT_NO_FATAL_FAILURE(InitState());
706
707 VkImageUsageFlags usage = VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT;
708 VkFormat format = VK_FORMAT_G8_B8_R8_3PLANE_420_UNORM;
709 VkImageObj image_a(m_device);
710 const auto image_ci = VkImageObj::ImageCreateInfo2D(128, 128, 1, 2, format, usage, VK_IMAGE_TILING_OPTIMAL);
711 // Verify format
712 bool supported = ImageFormatAndFeaturesSupported(instance(), gpu(), image_ci,
713 VK_FORMAT_FEATURE_TRANSFER_SRC_BIT | VK_FORMAT_FEATURE_TRANSFER_DST_BIT);
714 if (!supported) {
715 printf("%s Multiplane image format not supported. Skipping test.\n", kSkipPrefix);
716 return; // Assume there's low ROI on searching for different mp formats
717 }
718
719 image_a.Init(image_ci);
720 VkImageObj image_b(m_device);
721 image_b.Init(image_ci);
722 VkImageObj image_c(m_device);
723 image_c.Init(image_ci);
724
725 VkImageSubresourceLayers layer_all_plane0{VK_IMAGE_ASPECT_PLANE_0_BIT_KHR, 0, 0, 2};
726 VkImageSubresourceLayers layer0_plane0{VK_IMAGE_ASPECT_PLANE_0_BIT_KHR, 0, 0, 1};
727 VkImageSubresourceLayers layer0_plane1{VK_IMAGE_ASPECT_PLANE_1_BIT_KHR, 0, 0, 1};
728 VkImageSubresourceLayers layer1_plane1{VK_IMAGE_ASPECT_PLANE_1_BIT_KHR, 0, 1, 1};
729 VkImageSubresourceRange full_subresource_range{
730 VK_IMAGE_ASPECT_PLANE_0_BIT_KHR | VK_IMAGE_ASPECT_PLANE_1_BIT_KHR | VK_IMAGE_ASPECT_PLANE_2_BIT_KHR, 0, 1, 0, 2};
731 VkOffset3D zero_offset{0, 0, 0};
732 VkOffset3D one_four_offset{32, 32, 0};
733 VkExtent3D full_extent{128, 128, 1}; // <-- image type is 2D
734 VkExtent3D half_extent{64, 64, 1}; // <-- image type is 2D
735 VkExtent3D one_four_extent{32, 32, 1}; // <-- image type is 2D
736
737 VkImageCopy region_all_plane0_to_all_plane0 = {layer_all_plane0, zero_offset, layer_all_plane0, zero_offset, full_extent};
738 VkImageCopy region_layer0_plane0_to_layer0_plane0 = {layer0_plane0, zero_offset, layer0_plane0, zero_offset, full_extent};
739 VkImageCopy region_layer0_plane0_to_layer0_plane1 = {layer0_plane0, zero_offset, layer0_plane1, zero_offset, half_extent};
740 VkImageCopy region_layer1_plane1_to_layer1_plane1_front = {layer1_plane1, zero_offset, layer1_plane1, zero_offset,
741 one_four_extent};
742 VkImageCopy region_layer1_plane1_to_layer1_plane1_back = {layer1_plane1, one_four_offset, layer1_plane1, one_four_offset,
743 one_four_extent};
744
745 m_commandBuffer->begin();
746
747 image_c.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
748 image_b.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
749 image_a.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
750
751 auto cb = m_commandBuffer->handle();
752
753 vk::CmdCopyImage(cb, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
754 &region_all_plane0_to_all_plane0);
755
756 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
757 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
758 &region_all_plane0_to_all_plane0);
759 m_errorMonitor->VerifyFound();
760
761 // Use the barrier to clean up the WAW, and try again. (and show that validation is accounting for the barrier effect too.)
Mark Lobodzinski07d0a612020-12-30 15:42:31 -0700762 auto image_barrier = LvlInitStruct<VkImageMemoryBarrier>();
Jeremy Gebben170781d2020-11-19 16:21:21 -0700763 image_barrier.srcAccessMask = VK_ACCESS_TRANSFER_READ_BIT;
764 image_barrier.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
765 image_barrier.image = image_a.handle();
766 image_barrier.subresourceRange = full_subresource_range;
767 image_barrier.oldLayout = VK_IMAGE_LAYOUT_GENERAL;
768 image_barrier.newLayout = VK_IMAGE_LAYOUT_GENERAL;
769 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 0, nullptr, 0, nullptr, 1,
770 &image_barrier);
771
772 m_errorMonitor->ExpectSuccess();
773 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
774 &region_layer0_plane0_to_layer0_plane0);
775 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
776 &region_layer0_plane0_to_layer0_plane1);
777 m_errorMonitor->VerifyNotFound();
778
779 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
780 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
781 &region_layer0_plane0_to_layer0_plane1);
782 m_errorMonitor->VerifyFound();
783
784 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
785 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
786 &region_all_plane0_to_all_plane0);
787 m_errorMonitor->VerifyFound();
788
789 // NOTE: Since the previous command skips in validation, the state update is never done, and the validation layer thus doesn't
790 // record the write operation to b. So we'll need to repeat it successfully to set up for the *next* test.
791
792 // Use the barrier to clean up the WAW, and try again. (and show that validation is accounting for the barrier effect too.)
Mark Lobodzinski07d0a612020-12-30 15:42:31 -0700793 auto mem_barrier = LvlInitStruct<VkMemoryBarrier>();
Jeremy Gebben170781d2020-11-19 16:21:21 -0700794 mem_barrier.srcAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
795 mem_barrier.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
796 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 1, &mem_barrier, 0, nullptr, 0,
797 nullptr);
798 m_errorMonitor->ExpectSuccess();
799 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
800 &region_all_plane0_to_all_plane0);
801 m_errorMonitor->VerifyNotFound();
802
803 // Use barrier to protect last reader, but not last writer...
804 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
805 mem_barrier.srcAccessMask = VK_ACCESS_TRANSFER_READ_BIT; // Protects C but not B
806 mem_barrier.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
807 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 1, &mem_barrier, 0, nullptr, 0,
808 nullptr);
809 vk::CmdCopyImage(cb, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
810 &region_all_plane0_to_all_plane0);
811 m_errorMonitor->VerifyFound();
812
813 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
814 &region_layer1_plane1_to_layer1_plane1_front);
815 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
816 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
817 &region_layer1_plane1_to_layer1_plane1_front);
818 m_errorMonitor->VerifyFound();
819
820 m_errorMonitor->ExpectSuccess();
821 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
822 &region_layer1_plane1_to_layer1_plane1_back);
823 m_errorMonitor->VerifyNotFound();
824
825 m_commandBuffer->end();
826}
827
828TEST_F(VkSyncValTest, SyncCopyLinearImageHazards) {
829 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
830 ASSERT_NO_FATAL_FAILURE(InitState());
831
832 VkImageUsageFlags usage = VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT;
833 VkFormat format = VK_FORMAT_R8G8B8A8_UNORM;
834 VkImageObj image_a(m_device);
835 const auto image_ci = VkImageObj::ImageCreateInfo2D(128, 128, 1, 1, format, usage, VK_IMAGE_TILING_LINEAR);
836 image_a.Init(image_ci);
837 VkImageObj image_b(m_device);
838 image_b.Init(image_ci);
839 VkImageObj image_c(m_device);
840 image_c.Init(image_ci);
841
842 VkImageSubresourceLayers layers_all{VK_IMAGE_ASPECT_COLOR_BIT, 0, 0, 1};
843 VkImageSubresourceRange full_subresource_range{VK_IMAGE_ASPECT_COLOR_BIT, 0, 1, 0, 1};
844 VkOffset3D zero_offset{0, 0, 0};
845 VkOffset3D half_offset{64, 64, 0};
846 VkExtent3D full_extent{128, 128, 1}; // <-- image type is 2D
847 VkExtent3D half_extent{64, 64, 1}; // <-- image type is 2D
848
849 VkImageCopy full_region = {layers_all, zero_offset, layers_all, zero_offset, full_extent};
850 VkImageCopy region_front = {layers_all, zero_offset, layers_all, zero_offset, half_extent};
851 VkImageCopy region_back = {layers_all, half_offset, layers_all, half_offset, half_extent};
852
853 m_commandBuffer->begin();
854
855 image_c.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
856 image_b.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
857 image_a.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
858
859 auto cb = m_commandBuffer->handle();
860
861 vk::CmdCopyImage(cb, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &full_region);
862
863 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
864 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &full_region);
865 m_errorMonitor->VerifyFound();
866
867 // Use the barrier to clean up the WAW, and try again. (and show that validation is accounting for the barrier effect too.)
Mark Lobodzinski07d0a612020-12-30 15:42:31 -0700868 auto image_barrier = LvlInitStruct<VkImageMemoryBarrier>();
Jeremy Gebben170781d2020-11-19 16:21:21 -0700869 image_barrier.srcAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
870 image_barrier.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
871 image_barrier.image = image_b.handle();
872 image_barrier.subresourceRange = full_subresource_range;
873 image_barrier.oldLayout = VK_IMAGE_LAYOUT_GENERAL;
874 image_barrier.newLayout = VK_IMAGE_LAYOUT_GENERAL;
875 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 0, nullptr, 0, nullptr, 1,
876 &image_barrier);
877
878 m_errorMonitor->ExpectSuccess();
879 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &full_region);
880 m_errorMonitor->VerifyNotFound();
881
882 // Use barrier to protect last reader, but not last writer...
883 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
884 image_barrier.srcAccessMask = VK_ACCESS_TRANSFER_READ_BIT; // Protects C but not B
885 image_barrier.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
886 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 0, nullptr, 0, nullptr, 1,
887 &image_barrier);
888 vk::CmdCopyImage(cb, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &full_region);
889 m_errorMonitor->VerifyFound();
890
891 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region_front);
892 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
893 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region_front);
894 m_errorMonitor->VerifyFound();
895
896 m_errorMonitor->ExpectSuccess();
897 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region_back);
898 m_errorMonitor->VerifyNotFound();
899}
900
901TEST_F(VkSyncValTest, SyncCopyLinearMultiPlanarHazards) {
902 // TODO: Add code to enable sync validation
903 // Enable KHR multiplane req'd extensions
904 bool mp_extensions = InstanceExtensionSupported(VK_KHR_GET_PHYSICAL_DEVICE_PROPERTIES_2_EXTENSION_NAME,
905 VK_KHR_GET_MEMORY_REQUIREMENTS_2_SPEC_VERSION);
906 if (mp_extensions) {
907 m_instance_extension_names.push_back(VK_KHR_GET_PHYSICAL_DEVICE_PROPERTIES_2_EXTENSION_NAME);
908 }
909 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
Mike Schuchardt7cc57842021-09-15 10:49:59 -0700910 mp_extensions = mp_extensions && DeviceExtensionSupported(gpu(), nullptr, VK_KHR_MAINTENANCE_1_EXTENSION_NAME);
Jeremy Gebben170781d2020-11-19 16:21:21 -0700911 mp_extensions = mp_extensions && DeviceExtensionSupported(gpu(), nullptr, VK_KHR_GET_MEMORY_REQUIREMENTS_2_EXTENSION_NAME);
912 mp_extensions = mp_extensions && DeviceExtensionSupported(gpu(), nullptr, VK_KHR_BIND_MEMORY_2_EXTENSION_NAME);
913 mp_extensions = mp_extensions && DeviceExtensionSupported(gpu(), nullptr, VK_KHR_SAMPLER_YCBCR_CONVERSION_EXTENSION_NAME);
914 if (mp_extensions) {
Mike Schuchardt7cc57842021-09-15 10:49:59 -0700915 m_device_extension_names.push_back(VK_KHR_MAINTENANCE_1_EXTENSION_NAME);
Jeremy Gebben170781d2020-11-19 16:21:21 -0700916 m_device_extension_names.push_back(VK_KHR_GET_MEMORY_REQUIREMENTS_2_EXTENSION_NAME);
917 m_device_extension_names.push_back(VK_KHR_BIND_MEMORY_2_EXTENSION_NAME);
918 m_device_extension_names.push_back(VK_KHR_SAMPLER_YCBCR_CONVERSION_EXTENSION_NAME);
919 } else {
920 printf("%s test requires KHR multiplane extensions, not available. Skipping.\n", kSkipPrefix);
921 return;
922 }
923
924 ASSERT_NO_FATAL_FAILURE(InitState());
925
926 VkImageUsageFlags usage = VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT;
927 VkFormat format = VK_FORMAT_G8_B8_R8_3PLANE_420_UNORM;
928 VkImageObj image_a(m_device);
929 const auto image_ci = VkImageObj::ImageCreateInfo2D(128, 128, 1, 1, format, usage, VK_IMAGE_TILING_LINEAR);
930 // Verify format
931 bool supported = ImageFormatAndFeaturesSupported(instance(), gpu(), image_ci,
932 VK_FORMAT_FEATURE_TRANSFER_SRC_BIT | VK_FORMAT_FEATURE_TRANSFER_DST_BIT);
933 if (!supported) {
934 printf("%s Multiplane image format not supported. Skipping test.\n", kSkipPrefix);
935 return; // Assume there's low ROI on searching for different mp formats
936 }
937
938 image_a.Init(image_ci);
939 VkImageObj image_b(m_device);
940 image_b.Init(image_ci);
941 VkImageObj image_c(m_device);
942 image_c.Init(image_ci);
943
944 VkImageSubresourceLayers layer_all_plane0{VK_IMAGE_ASPECT_PLANE_0_BIT_KHR, 0, 0, 1};
945 VkImageSubresourceLayers layer_all_plane1{VK_IMAGE_ASPECT_PLANE_1_BIT_KHR, 0, 0, 1};
946 VkImageSubresourceRange full_subresource_range{
947 VK_IMAGE_ASPECT_PLANE_0_BIT_KHR | VK_IMAGE_ASPECT_PLANE_1_BIT_KHR | VK_IMAGE_ASPECT_PLANE_2_BIT_KHR, 0, 1, 0, 1};
948 VkOffset3D zero_offset{0, 0, 0};
949 VkOffset3D one_four_offset{32, 32, 0};
950 VkExtent3D full_extent{128, 128, 1}; // <-- image type is 2D
951 VkExtent3D half_extent{64, 64, 1}; // <-- image type is 2D
952 VkExtent3D one_four_extent{32, 32, 1}; // <-- image type is 2D
953
954 VkImageCopy region_plane0_to_plane0 = {layer_all_plane0, zero_offset, layer_all_plane0, zero_offset, full_extent};
955 VkImageCopy region_plane0_to_plane1 = {layer_all_plane0, zero_offset, layer_all_plane1, zero_offset, half_extent};
956 VkImageCopy region_plane1_to_plane1_front = {layer_all_plane1, zero_offset, layer_all_plane1, zero_offset, one_four_extent};
957 VkImageCopy region_plane1_to_plane1_back = {layer_all_plane1, one_four_offset, layer_all_plane1, one_four_offset,
958 one_four_extent};
959
960 m_commandBuffer->begin();
961
962 image_c.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
963 image_b.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
964 image_a.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
965
966 auto cb = m_commandBuffer->handle();
967
968 vk::CmdCopyImage(cb, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
969 &region_plane0_to_plane0);
970
971 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
972 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
973 &region_plane0_to_plane0);
974 m_errorMonitor->VerifyFound();
975
976 // Use the barrier to clean up the WAW, and try again. (and show that validation is accounting for the barrier effect too.)
Mark Lobodzinski07d0a612020-12-30 15:42:31 -0700977 auto image_barrier = LvlInitStruct<VkImageMemoryBarrier>();
Jeremy Gebben170781d2020-11-19 16:21:21 -0700978 image_barrier.srcAccessMask = VK_ACCESS_TRANSFER_READ_BIT;
979 image_barrier.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
980 image_barrier.image = image_a.handle();
981 image_barrier.subresourceRange = full_subresource_range;
982 image_barrier.oldLayout = VK_IMAGE_LAYOUT_GENERAL;
983 image_barrier.newLayout = VK_IMAGE_LAYOUT_GENERAL;
984 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 0, nullptr, 0, nullptr, 1,
985 &image_barrier);
986
987 m_errorMonitor->ExpectSuccess();
988 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
989 &region_plane0_to_plane0);
990 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
991 &region_plane0_to_plane1);
992 m_errorMonitor->VerifyNotFound();
993
994 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
995 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
996 &region_plane0_to_plane1);
997 m_errorMonitor->VerifyFound();
998
999 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
1000 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
1001 &region_plane0_to_plane0);
1002 m_errorMonitor->VerifyFound();
1003
1004 // NOTE: Since the previous command skips in validation, the state update is never done, and the validation layer thus doesn't
1005 // record the write operation to b. So we'll need to repeat it successfully to set up for the *next* test.
1006
1007 // Use the barrier to clean up the WAW, and try again. (and show that validation is accounting for the barrier effect too.)
Mark Lobodzinski07d0a612020-12-30 15:42:31 -07001008 auto mem_barrier = LvlInitStruct<VkMemoryBarrier>();
Jeremy Gebben170781d2020-11-19 16:21:21 -07001009 mem_barrier.srcAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
1010 mem_barrier.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
1011 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 1, &mem_barrier, 0, nullptr, 0,
1012 nullptr);
1013 m_errorMonitor->ExpectSuccess();
1014 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
1015 &region_plane0_to_plane0);
1016 m_errorMonitor->VerifyNotFound();
1017
1018 // Use barrier to protect last reader, but not last writer...
1019 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
1020 mem_barrier.srcAccessMask = VK_ACCESS_TRANSFER_READ_BIT; // Protects C but not B
1021 mem_barrier.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
1022 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 1, &mem_barrier, 0, nullptr, 0,
1023 nullptr);
1024 vk::CmdCopyImage(cb, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
1025 &region_plane0_to_plane0);
1026 m_errorMonitor->VerifyFound();
1027
1028 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
1029 &region_plane1_to_plane1_front);
1030 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
1031 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
1032 &region_plane1_to_plane1_front);
1033 m_errorMonitor->VerifyFound();
1034
1035 m_errorMonitor->ExpectSuccess();
1036 vk::CmdCopyImage(cb, image_c.handle(), VK_IMAGE_LAYOUT_GENERAL, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
1037 &region_plane1_to_plane1_back);
1038 m_errorMonitor->VerifyNotFound();
1039
1040 m_commandBuffer->end();
1041}
1042
1043TEST_F(VkSyncValTest, SyncCopyBufferImageHazards) {
1044 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
1045 ASSERT_NO_FATAL_FAILURE(InitState());
1046
1047 VkBufferObj buffer_a, buffer_b;
1048 VkMemoryPropertyFlags mem_prop = VK_MEMORY_PROPERTY_DEVICE_LOCAL_BIT;
1049 buffer_a.init_as_src_and_dst(*m_device, 2048, mem_prop);
1050 buffer_b.init_as_src_and_dst(*m_device, 2048, mem_prop);
1051
1052 VkImageUsageFlags usage = VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT;
1053 VkFormat format = VK_FORMAT_R8G8B8A8_UNORM;
1054 VkImageObj image_a(m_device), image_b(m_device);
1055 const auto image_ci = VkImageObj::ImageCreateInfo2D(32, 32, 1, 2, format, usage, VK_IMAGE_TILING_OPTIMAL);
1056 image_a.Init(image_ci);
1057 image_b.Init(image_ci);
1058
1059 VkImageSubresourceLayers layers_0{VK_IMAGE_ASPECT_COLOR_BIT, 0, 0, 1};
1060 VkImageSubresourceLayers layers_1{VK_IMAGE_ASPECT_COLOR_BIT, 0, 1, 1};
1061 VkOffset3D zero_offset{0, 0, 0};
1062 VkOffset3D half_offset{16, 16, 0};
1063 VkExtent3D half_extent{16, 16, 1}; // <-- image type is 2D
1064
1065 VkBufferImageCopy region_buffer_front_image_0_front = {0, 16, 16, layers_0, zero_offset, half_extent};
1066 VkBufferImageCopy region_buffer_front_image_1_front = {0, 16, 16, layers_1, zero_offset, half_extent};
1067 VkBufferImageCopy region_buffer_front_image_1_back = {0, 16, 16, layers_1, half_offset, half_extent};
1068 VkBufferImageCopy region_buffer_back_image_0_front = {1024, 16, 16, layers_0, zero_offset, half_extent};
1069 VkBufferImageCopy region_buffer_back_image_0_back = {1024, 16, 16, layers_0, half_offset, half_extent};
1070 VkBufferImageCopy region_buffer_back_image_1_front = {1024, 16, 16, layers_1, zero_offset, half_extent};
1071 VkBufferImageCopy region_buffer_back_image_1_back = {1024, 16, 16, layers_1, half_offset, half_extent};
1072
1073 m_commandBuffer->begin();
1074 image_b.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
1075 image_a.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
1076
1077 auto cb = m_commandBuffer->handle();
1078 vk::CmdCopyBufferToImage(cb, buffer_a.handle(), image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
1079 &region_buffer_front_image_0_front);
1080
1081 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
1082 vk::CmdCopyBufferToImage(cb, buffer_a.handle(), image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
1083 &region_buffer_front_image_0_front);
1084 m_errorMonitor->VerifyFound();
1085
1086 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
1087 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
1088 vk::CmdCopyImageToBuffer(cb, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, buffer_a.handle(), 1,
1089 &region_buffer_front_image_0_front);
1090 m_errorMonitor->VerifyFound();
1091
1092 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
1093 vk::CmdCopyImageToBuffer(cb, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, buffer_a.handle(), 1,
1094 &region_buffer_back_image_0_front);
1095 m_errorMonitor->VerifyFound();
1096
1097 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
1098 vk::CmdCopyImageToBuffer(cb, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, buffer_a.handle(), 1,
1099 &region_buffer_front_image_1_front);
1100 m_errorMonitor->VerifyFound();
1101
1102 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
1103 vk::CmdCopyImageToBuffer(cb, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, buffer_a.handle(), 1,
1104 &region_buffer_front_image_1_back);
1105 m_errorMonitor->VerifyFound();
1106
1107 m_errorMonitor->ExpectSuccess();
1108 vk::CmdCopyImageToBuffer(cb, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, buffer_a.handle(), 1, &region_buffer_back_image_0_back);
1109 m_errorMonitor->VerifyNotFound();
1110
Mark Lobodzinski07d0a612020-12-30 15:42:31 -07001111 auto buffer_barrier = LvlInitStruct<VkBufferMemoryBarrier>();
Jeremy Gebben170781d2020-11-19 16:21:21 -07001112 buffer_barrier.srcAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
1113 buffer_barrier.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
1114 buffer_barrier.buffer = buffer_a.handle();
1115 buffer_barrier.offset = 1024;
1116 buffer_barrier.size = 2048;
1117 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 0, nullptr, 1, &buffer_barrier, 0,
1118 nullptr);
1119
1120 m_errorMonitor->ExpectSuccess();
1121 vk::CmdCopyImageToBuffer(cb, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, buffer_a.handle(), 1,
1122 &region_buffer_back_image_1_front);
1123 m_errorMonitor->VerifyNotFound();
1124
1125 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 0, nullptr, 1, &buffer_barrier, 0,
1126 nullptr);
1127
1128 m_errorMonitor->ExpectSuccess();
1129 vk::CmdCopyImageToBuffer(cb, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, buffer_a.handle(), 1, &region_buffer_back_image_1_back);
1130 m_errorMonitor->VerifyNotFound();
1131
1132 vk::CmdCopyImageToBuffer(cb, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, buffer_b.handle(), 1,
1133 &region_buffer_front_image_0_front);
1134
1135 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
1136 vk::CmdCopyImageToBuffer(cb, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, buffer_b.handle(), 1,
1137 &region_buffer_front_image_0_front);
1138 m_errorMonitor->VerifyFound();
1139
1140 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
1141 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
1142 vk::CmdCopyBufferToImage(cb, buffer_b.handle(), image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
1143 &region_buffer_front_image_0_front);
1144 m_errorMonitor->VerifyFound();
1145
1146 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
1147 vk::CmdCopyBufferToImage(cb, buffer_b.handle(), image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
1148 &region_buffer_back_image_0_front);
1149 m_errorMonitor->VerifyFound();
1150
1151 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
1152 vk::CmdCopyBufferToImage(cb, buffer_b.handle(), image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
1153 &region_buffer_front_image_1_front);
1154 m_errorMonitor->VerifyFound();
1155
1156 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
1157 vk::CmdCopyBufferToImage(cb, buffer_b.handle(), image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
1158 &region_buffer_front_image_1_back);
1159 m_errorMonitor->VerifyFound();
1160
1161 m_errorMonitor->ExpectSuccess();
1162 vk::CmdCopyBufferToImage(cb, buffer_b.handle(), image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region_buffer_back_image_0_back);
1163 m_errorMonitor->VerifyNotFound();
1164
1165 buffer_barrier.srcAccessMask = VK_ACCESS_TRANSFER_READ_BIT;
1166 buffer_barrier.dstAccessMask = VK_ACCESS_TRANSFER_READ_BIT;
1167 buffer_barrier.buffer = buffer_b.handle();
1168 buffer_barrier.offset = 1024;
1169 buffer_barrier.size = 2048;
1170 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 0, nullptr, 1, &buffer_barrier, 0,
1171 nullptr);
1172
1173 m_errorMonitor->ExpectSuccess();
1174 vk::CmdCopyBufferToImage(cb, buffer_b.handle(), image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
1175 &region_buffer_back_image_1_front);
1176 m_errorMonitor->VerifyNotFound();
1177
1178 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 0, nullptr, 1, &buffer_barrier, 0,
1179 nullptr);
1180
1181 m_errorMonitor->ExpectSuccess();
1182 vk::CmdCopyBufferToImage(cb, buffer_b.handle(), image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region_buffer_back_image_1_back);
1183 m_errorMonitor->VerifyNotFound();
1184
1185 m_commandBuffer->end();
1186}
1187
1188TEST_F(VkSyncValTest, SyncBlitImageHazards) {
1189 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
1190 ASSERT_NO_FATAL_FAILURE(InitState());
1191
1192 VkImageUsageFlags usage = VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT;
1193 VkFormat format = VK_FORMAT_R8G8B8A8_UNORM;
1194 VkImageObj image_a(m_device), image_b(m_device);
1195 const auto image_ci = VkImageObj::ImageCreateInfo2D(32, 32, 1, 2, format, usage, VK_IMAGE_TILING_OPTIMAL);
1196 image_a.Init(image_ci);
1197 image_b.Init(image_ci);
1198
1199 VkImageSubresourceLayers layers_0{VK_IMAGE_ASPECT_COLOR_BIT, 0, 0, 1};
1200 VkImageSubresourceLayers layers_1{VK_IMAGE_ASPECT_COLOR_BIT, 0, 1, 1};
1201 VkOffset3D zero_offset{0, 0, 0};
1202 VkOffset3D half_0_offset{16, 16, 0};
1203 VkOffset3D half_1_offset{16, 16, 1};
1204 VkOffset3D full_offset{32, 32, 1};
1205 VkImageBlit region_0_front_1_front = {layers_0, {zero_offset, half_1_offset}, layers_1, {zero_offset, half_1_offset}};
1206 VkImageBlit region_1_front_0_front = {layers_1, {zero_offset, half_1_offset}, layers_0, {zero_offset, half_1_offset}};
1207 VkImageBlit region_1_back_0_back = {layers_1, {half_0_offset, full_offset}, layers_0, {half_0_offset, full_offset}};
1208
1209 m_commandBuffer->begin();
1210 image_b.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
1211 image_a.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
1212
1213 auto cb = m_commandBuffer->handle();
1214
1215 vk::CmdBlitImage(cb, image_a.image(), VK_IMAGE_LAYOUT_GENERAL, image_b.image(), VK_IMAGE_LAYOUT_GENERAL, 1,
1216 &region_0_front_1_front, VK_FILTER_NEAREST);
1217
1218 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
1219 vk::CmdBlitImage(cb, image_a.image(), VK_IMAGE_LAYOUT_GENERAL, image_b.image(), VK_IMAGE_LAYOUT_GENERAL, 1,
1220 &region_0_front_1_front, VK_FILTER_NEAREST);
1221 m_errorMonitor->VerifyFound();
1222
1223 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
1224 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
1225 vk::CmdBlitImage(cb, image_b.image(), VK_IMAGE_LAYOUT_GENERAL, image_a.image(), VK_IMAGE_LAYOUT_GENERAL, 1,
1226 &region_1_front_0_front, VK_FILTER_NEAREST);
1227 m_errorMonitor->VerifyFound();
1228
1229 m_errorMonitor->ExpectSuccess();
1230 vk::CmdBlitImage(cb, image_b.image(), VK_IMAGE_LAYOUT_GENERAL, image_a.image(), VK_IMAGE_LAYOUT_GENERAL, 1,
1231 &region_1_back_0_back, VK_FILTER_NEAREST);
1232 m_errorMonitor->VerifyNotFound();
1233
1234 m_commandBuffer->end();
1235}
1236
1237TEST_F(VkSyncValTest, SyncRenderPassBeginTransitionHazard) {
1238 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
1239 ASSERT_NO_FATAL_FAILURE(InitState());
John Zulaufbb373682021-10-05 17:21:40 -06001240 const VkSubpassDependency external_subpass_dependency = {VK_SUBPASS_EXTERNAL,
1241 0,
1242 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT,
1243 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT,
1244 VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT,
1245 VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT,
1246 VK_DEPENDENCY_BY_REGION_BIT};
1247 m_additionalSubpassDependencies.push_back(external_subpass_dependency);
Jeremy Gebben170781d2020-11-19 16:21:21 -07001248 ASSERT_NO_FATAL_FAILURE(InitRenderTarget(2));
1249
1250 // Render Target Information
1251 auto width = static_cast<uint32_t>(m_width);
1252 auto height = static_cast<uint32_t>(m_height);
1253 auto *rt_0 = m_renderTargets[0].get();
1254 auto *rt_1 = m_renderTargets[1].get();
1255
1256 // Other buffers with which to interact
1257 VkImageUsageFlags usage = VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT;
1258 VkFormat format = VK_FORMAT_R8G8B8A8_UNORM;
1259 VkImageObj image_a(m_device), image_b(m_device);
1260 const auto image_ci = VkImageObj::ImageCreateInfo2D(width, height, 1, 1, format, usage, VK_IMAGE_TILING_OPTIMAL);
1261 image_a.Init(image_ci);
1262 image_b.Init(image_ci);
1263
1264 VkOffset3D zero_offset{0, 0, 0};
1265 VkExtent3D full_extent{width, height, 1}; // <-- image type is 2D
1266 VkImageSubresourceLayers layer_color{VK_IMAGE_ASPECT_COLOR_BIT, 0, 0, 1};
1267 VkImageCopy region_to_copy = {layer_color, zero_offset, layer_color, zero_offset, full_extent};
1268
1269 auto cb = m_commandBuffer->handle();
1270
1271 m_errorMonitor->ExpectSuccess();
1272 m_commandBuffer->begin();
1273 image_b.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
1274 image_a.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
1275 rt_0->SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
1276 rt_1->SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
1277
1278 rt_0->SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
1279 vk::CmdCopyImage(cb, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, rt_0->handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region_to_copy);
1280 m_errorMonitor->VerifyNotFound();
1281
1282 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
1283 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo); // This fails so the driver call is skip and no end is valid
1284 m_errorMonitor->VerifyFound();
1285
1286 m_errorMonitor->ExpectSuccess();
1287 // Use the barrier to clean up the WAW, and try again. (and show that validation is accounting for the barrier effect too.)
1288 VkImageSubresourceRange rt_full_subresource_range{VK_IMAGE_ASPECT_COLOR_BIT, 0, 1, 0, 1};
Mark Lobodzinski07d0a612020-12-30 15:42:31 -07001289 auto image_barrier = LvlInitStruct<VkImageMemoryBarrier>();
Jeremy Gebben170781d2020-11-19 16:21:21 -07001290 image_barrier.srcAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
John Zulaufbb373682021-10-05 17:21:40 -06001291 image_barrier.dstAccessMask = VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT;
Jeremy Gebben170781d2020-11-19 16:21:21 -07001292 image_barrier.image = rt_0->handle();
1293 image_barrier.subresourceRange = rt_full_subresource_range;
1294 image_barrier.oldLayout = VK_IMAGE_LAYOUT_GENERAL;
1295 image_barrier.newLayout = VK_IMAGE_LAYOUT_GENERAL;
John Zulaufbb373682021-10-05 17:21:40 -06001296 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT, 0, 0, nullptr, 0,
1297 nullptr, 1, &image_barrier);
Jeremy Gebben170781d2020-11-19 16:21:21 -07001298 vk::CmdCopyImage(cb, rt_1->handle(), VK_IMAGE_LAYOUT_GENERAL, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region_to_copy);
1299 m_errorMonitor->VerifyNotFound();
1300
1301 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
1302 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo); // This fails so the driver call is skip and no end is valid
1303 m_errorMonitor->VerifyFound();
1304
1305 m_errorMonitor->ExpectSuccess();
1306 // A global execution barrier that the implict external dependency can chain with should work...
1307 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TOP_OF_PIPE_BIT, 0, 0, nullptr, 0, nullptr, 0,
1308 nullptr);
1309
1310 // With the barrier above, the layout transition has a chained execution sync operation, and the default
1311 // implict VkSubpassDependency safes the load op clear vs. the layout transition...
1312 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
1313 m_commandBuffer->EndRenderPass();
1314 m_errorMonitor->VerifyNotFound();
1315}
1316
1317TEST_F(VkSyncValTest, SyncCmdDispatchDrawHazards) {
1318 // TODO: Add code to enable sync validation
1319 SetTargetApiVersion(VK_API_VERSION_1_2);
1320
1321 // Enable VK_KHR_draw_indirect_count for KHR variants
1322 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
sfricke-samsung6fc3e322022-02-15 22:41:29 -08001323 VkPhysicalDeviceVulkan12Features features12 = LvlInitStruct<VkPhysicalDeviceVulkan12Features>();
Jeremy Gebben170781d2020-11-19 16:21:21 -07001324 if (DeviceExtensionSupported(gpu(), nullptr, VK_KHR_DRAW_INDIRECT_COUNT_EXTENSION_NAME)) {
1325 m_device_extension_names.push_back(VK_KHR_DRAW_INDIRECT_COUNT_EXTENSION_NAME);
1326 if (DeviceValidationVersion() >= VK_API_VERSION_1_2) {
1327 features12.drawIndirectCount = VK_TRUE;
1328 }
1329 }
1330 ASSERT_NO_FATAL_FAILURE(InitState(nullptr, &features12, VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT));
1331 bool has_khr_indirect = DeviceExtensionEnabled(VK_KHR_DRAW_INDIRECT_COUNT_EXTENSION_NAME);
1332 ASSERT_NO_FATAL_FAILURE(InitRenderTarget());
1333
1334 VkImageUsageFlags image_usage_combine = VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT | VK_IMAGE_USAGE_SAMPLED_BIT |
1335 VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT;
1336 VkFormat format = VK_FORMAT_R8G8B8A8_UNORM;
1337 VkImageObj image_c_a(m_device), image_c_b(m_device);
1338 const auto image_c_ci = VkImageObj::ImageCreateInfo2D(16, 16, 1, 1, format, image_usage_combine, VK_IMAGE_TILING_OPTIMAL);
1339 image_c_a.Init(image_c_ci);
1340 image_c_b.Init(image_c_ci);
1341
1342 VkImageView imageview_c = image_c_a.targetView(format);
1343 VkImageUsageFlags image_usage_storage =
1344 VK_IMAGE_USAGE_STORAGE_BIT | VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT;
1345 VkImageObj image_s_a(m_device), image_s_b(m_device);
1346 const auto image_s_ci = VkImageObj::ImageCreateInfo2D(16, 16, 1, 1, format, image_usage_storage, VK_IMAGE_TILING_OPTIMAL);
1347 image_s_a.Init(image_s_ci);
1348 image_s_b.Init(image_s_ci);
1349 image_s_a.SetLayout(VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
1350 image_s_b.SetLayout(VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
1351
1352 VkImageView imageview_s = image_s_a.targetView(format);
1353
Jeremy Gebben18ac1052021-08-12 11:07:32 -06001354 vk_testing::Sampler sampler_s, sampler_c;
Jeremy Gebben170781d2020-11-19 16:21:21 -07001355 VkSamplerCreateInfo sampler_ci = SafeSaneSamplerCreateInfo();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06001356 sampler_s.init(*m_device, sampler_ci);
1357 sampler_c.init(*m_device, sampler_ci);
Jeremy Gebben170781d2020-11-19 16:21:21 -07001358
1359 VkBufferObj buffer_a, buffer_b;
1360 VkMemoryPropertyFlags mem_prop = VK_MEMORY_PROPERTY_DEVICE_LOCAL_BIT;
1361 VkBufferUsageFlags buffer_usage = VK_BUFFER_USAGE_UNIFORM_BUFFER_BIT | VK_BUFFER_USAGE_STORAGE_TEXEL_BUFFER_BIT |
1362 VK_BUFFER_USAGE_TRANSFER_SRC_BIT | VK_BUFFER_USAGE_TRANSFER_DST_BIT;
1363 buffer_a.init(*m_device, buffer_a.create_info(2048, buffer_usage, nullptr), mem_prop);
1364 buffer_b.init(*m_device, buffer_b.create_info(2048, buffer_usage, nullptr), mem_prop);
1365
Jeremy Gebben18ac1052021-08-12 11:07:32 -06001366 vk_testing::BufferView bufferview;
Mark Lobodzinski07d0a612020-12-30 15:42:31 -07001367 auto bvci = LvlInitStruct<VkBufferViewCreateInfo>();
Jeremy Gebben170781d2020-11-19 16:21:21 -07001368 bvci.buffer = buffer_a.handle();
1369 bvci.format = VK_FORMAT_R32_SFLOAT;
1370 bvci.offset = 0;
1371 bvci.range = VK_WHOLE_SIZE;
1372
Jeremy Gebben18ac1052021-08-12 11:07:32 -06001373 bufferview.init(*m_device, bvci);
Jeremy Gebben170781d2020-11-19 16:21:21 -07001374
1375 OneOffDescriptorSet descriptor_set(m_device,
1376 {
1377 {0, VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER, 1, VK_SHADER_STAGE_ALL, nullptr},
1378 {1, VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER, 1, VK_SHADER_STAGE_ALL, nullptr},
1379 {2, VK_DESCRIPTOR_TYPE_STORAGE_IMAGE, 1, VK_SHADER_STAGE_ALL, nullptr},
1380 {3, VK_DESCRIPTOR_TYPE_STORAGE_TEXEL_BUFFER, 1, VK_SHADER_STAGE_ALL, nullptr},
1381 });
1382
sfricke-samsung36428462021-02-10 01:23:34 -08001383 descriptor_set.WriteDescriptorBufferInfo(0, buffer_a.handle(), 0, 2048);
Jeremy Gebben18ac1052021-08-12 11:07:32 -06001384 descriptor_set.WriteDescriptorImageInfo(1, imageview_c, sampler_c.handle(), VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER,
Jeremy Gebben170781d2020-11-19 16:21:21 -07001385 VK_IMAGE_LAYOUT_GENERAL);
Jeremy Gebben18ac1052021-08-12 11:07:32 -06001386 descriptor_set.WriteDescriptorImageInfo(2, imageview_s, sampler_s.handle(), VK_DESCRIPTOR_TYPE_STORAGE_IMAGE, VK_IMAGE_LAYOUT_GENERAL);
1387 descriptor_set.WriteDescriptorBufferView(3, bufferview.handle());
Jeremy Gebben170781d2020-11-19 16:21:21 -07001388 descriptor_set.UpdateDescriptorSets();
1389
1390 // Dispatch
sfricke-samsung1c0b96a2021-07-08 22:24:09 -07001391 std::string csSource = R"glsl(
1392 #version 450
1393 layout(set=0, binding=0) uniform foo { float x; } ub0;
1394 layout(set=0, binding=1) uniform sampler2D cis1;
1395 layout(set=0, binding=2, rgba8) uniform readonly image2D si2;
1396 layout(set=0, binding=3, r32f) uniform readonly imageBuffer stb3;
1397 void main(){
1398 vec4 vColor4;
1399 vColor4.x = ub0.x;
1400 vColor4 = texture(cis1, vec2(0));
1401 vColor4 = imageLoad(si2, ivec2(0));
1402 vColor4 = imageLoad(stb3, 0);
1403 }
1404 )glsl";
Jeremy Gebben170781d2020-11-19 16:21:21 -07001405
John Zulaufbe8562b2020-12-15 14:21:01 -07001406 VkEventObj event;
1407 event.init(*m_device, VkEventObj::create_info(0));
1408 VkEvent event_handle = event.handle();
1409
Jeremy Gebben170781d2020-11-19 16:21:21 -07001410 CreateComputePipelineHelper pipe(*this);
1411 pipe.InitInfo();
sfricke-samsungae54c1e2022-01-21 05:35:21 -08001412 pipe.cs_.reset(new VkShaderObj(this, csSource, VK_SHADER_STAGE_COMPUTE_BIT));
Jeremy Gebben170781d2020-11-19 16:21:21 -07001413 pipe.InitState();
1414 pipe.pipeline_layout_ = VkPipelineLayoutObj(m_device, {&descriptor_set.layout_});
1415 pipe.CreateComputePipeline();
1416
1417 m_commandBuffer->begin();
1418
1419 VkBufferCopy buffer_region = {0, 0, 2048};
1420 vk::CmdCopyBuffer(m_commandBuffer->handle(), buffer_b.handle(), buffer_a.handle(), 1, &buffer_region);
1421
1422 VkImageSubresourceLayers layer{VK_IMAGE_ASPECT_COLOR_BIT, 0, 0, 1};
1423 VkOffset3D zero_offset{0, 0, 0};
1424 VkExtent3D full_extent{16, 16, 1};
1425 VkImageCopy image_region = {layer, zero_offset, layer, zero_offset, full_extent};
1426 vk::CmdCopyImage(m_commandBuffer->handle(), image_c_b.handle(), VK_IMAGE_LAYOUT_GENERAL, image_c_a.handle(),
1427 VK_IMAGE_LAYOUT_GENERAL, 1, &image_region);
1428 vk::CmdCopyImage(m_commandBuffer->handle(), image_s_b.handle(), VK_IMAGE_LAYOUT_GENERAL, image_s_a.handle(),
1429 VK_IMAGE_LAYOUT_GENERAL, 1, &image_region);
1430
1431 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_COMPUTE, pipe.pipeline_);
1432 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_COMPUTE, pipe.pipeline_layout_.handle(), 0, 1,
1433 &descriptor_set.set_, 0, nullptr);
1434
1435 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
1436 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
1437 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
1438 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
1439 vk::CmdDispatch(m_commandBuffer->handle(), 1, 1, 1);
1440 m_errorMonitor->VerifyFound();
1441
1442 m_commandBuffer->end();
1443 m_commandBuffer->reset();
1444 m_commandBuffer->begin();
1445
1446 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_COMPUTE, pipe.pipeline_);
1447 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_COMPUTE, pipe.pipeline_layout_.handle(), 0, 1,
1448 &descriptor_set.set_, 0, nullptr);
1449 vk::CmdDispatch(m_commandBuffer->handle(), 1, 1, 1);
1450
1451 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
1452 vk::CmdCopyBuffer(m_commandBuffer->handle(), buffer_b.handle(), buffer_a.handle(), 1, &buffer_region);
1453 m_errorMonitor->VerifyFound();
1454
1455 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
1456 vk::CmdCopyImage(m_commandBuffer->handle(), image_c_b.handle(), VK_IMAGE_LAYOUT_GENERAL, image_c_a.handle(),
1457 VK_IMAGE_LAYOUT_GENERAL, 1, &image_region);
1458 m_errorMonitor->VerifyFound();
1459
1460 m_commandBuffer->end();
1461 m_commandBuffer->reset();
1462
1463 // DispatchIndirect
1464 m_errorMonitor->ExpectSuccess();
1465 VkBufferObj buffer_dispatchIndirect, buffer_dispatchIndirect2;
1466 buffer_usage = VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT | VK_BUFFER_USAGE_TRANSFER_SRC_BIT | VK_BUFFER_USAGE_TRANSFER_DST_BIT;
1467 buffer_dispatchIndirect.init(
1468 *m_device, buffer_dispatchIndirect.create_info(sizeof(VkDispatchIndirectCommand), buffer_usage, nullptr), mem_prop);
1469 buffer_dispatchIndirect2.init(
1470 *m_device, buffer_dispatchIndirect2.create_info(sizeof(VkDispatchIndirectCommand), buffer_usage, nullptr), mem_prop);
1471 m_commandBuffer->begin();
1472 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_COMPUTE, pipe.pipeline_);
1473 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_COMPUTE, pipe.pipeline_layout_.handle(), 0, 1,
1474 &descriptor_set.set_, 0, nullptr);
1475 vk::CmdDispatchIndirect(m_commandBuffer->handle(), buffer_dispatchIndirect.handle(), 0);
1476 m_commandBuffer->end();
1477 m_errorMonitor->VerifyNotFound();
1478
1479 m_commandBuffer->reset();
1480 m_commandBuffer->begin();
1481
1482 buffer_region = {0, 0, sizeof(VkDispatchIndirectCommand)};
1483 vk::CmdCopyBuffer(m_commandBuffer->handle(), buffer_dispatchIndirect2.handle(), buffer_dispatchIndirect.handle(), 1,
1484 &buffer_region);
1485 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_COMPUTE, pipe.pipeline_);
1486 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_COMPUTE, pipe.pipeline_layout_.handle(), 0, 1,
1487 &descriptor_set.set_, 0, nullptr);
1488 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
1489 vk::CmdDispatchIndirect(m_commandBuffer->handle(), buffer_dispatchIndirect.handle(), 0);
1490 m_errorMonitor->VerifyFound();
1491 m_commandBuffer->end();
1492
1493 // Draw
1494 m_errorMonitor->ExpectSuccess();
1495 const float vbo_data[3] = {1.f, 0.f, 1.f};
1496 VkVertexInputAttributeDescription VertexInputAttributeDescription = {0, 0, VK_FORMAT_R32G32B32_SFLOAT, sizeof(vbo_data)};
1497 VkVertexInputBindingDescription VertexInputBindingDescription = {0, sizeof(vbo_data), VK_VERTEX_INPUT_RATE_VERTEX};
1498 VkBufferObj vbo, vbo2;
1499 buffer_usage = VK_BUFFER_USAGE_VERTEX_BUFFER_BIT | VK_BUFFER_USAGE_TRANSFER_SRC_BIT | VK_BUFFER_USAGE_TRANSFER_DST_BIT;
1500 vbo.init(*m_device, vbo.create_info(sizeof(vbo_data), buffer_usage, nullptr), mem_prop);
1501 vbo2.init(*m_device, vbo2.create_info(sizeof(vbo_data), buffer_usage, nullptr), mem_prop);
1502
sfricke-samsungae54c1e2022-01-21 05:35:21 -08001503 VkShaderObj vs(this, bindStateVertShaderText, VK_SHADER_STAGE_VERTEX_BIT);
1504 VkShaderObj fs(this, csSource, VK_SHADER_STAGE_FRAGMENT_BIT);
Jeremy Gebben170781d2020-11-19 16:21:21 -07001505
1506 CreatePipelineHelper g_pipe(*this);
1507 g_pipe.InitInfo();
1508 g_pipe.InitState();
1509 g_pipe.vi_ci_.pVertexBindingDescriptions = &VertexInputBindingDescription;
1510 g_pipe.vi_ci_.vertexBindingDescriptionCount = 1;
1511 g_pipe.vi_ci_.pVertexAttributeDescriptions = &VertexInputAttributeDescription;
1512 g_pipe.vi_ci_.vertexAttributeDescriptionCount = 1;
1513 g_pipe.shader_stages_ = {vs.GetStageCreateInfo(), fs.GetStageCreateInfo()};
1514 g_pipe.pipeline_layout_ = VkPipelineLayoutObj(m_device, {&descriptor_set.layout_});
1515 ASSERT_VK_SUCCESS(g_pipe.CreateGraphicsPipeline());
1516
1517 m_commandBuffer->reset();
1518 m_commandBuffer->begin();
1519 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
1520 VkDeviceSize offset = 0;
1521 vk::CmdBindVertexBuffers(m_commandBuffer->handle(), 0, 1, &vbo.handle(), &offset);
1522
1523 VkViewport viewport = {0, 0, 16, 16, 0, 1};
1524 vk::CmdSetViewport(m_commandBuffer->handle(), 0, 1, &viewport);
1525 VkRect2D scissor = {{0, 0}, {16, 16}};
1526 vk::CmdSetScissor(m_commandBuffer->handle(), 0, 1, &scissor);
1527
1528 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_);
1529 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_layout_.handle(), 0, 1,
1530 &descriptor_set.set_, 0, nullptr);
1531 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
1532 m_commandBuffer->EndRenderPass();
1533 m_commandBuffer->end();
1534 m_errorMonitor->VerifyNotFound();
1535
1536 m_commandBuffer->reset();
1537 m_commandBuffer->begin();
1538
1539 buffer_region = {0, 0, sizeof(vbo_data)};
1540 vk::CmdCopyBuffer(m_commandBuffer->handle(), vbo2.handle(), vbo.handle(), 1, &buffer_region);
1541
1542 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
1543 vk::CmdBindVertexBuffers(m_commandBuffer->handle(), 0, 1, &vbo.handle(), &offset);
1544 vk::CmdSetViewport(m_commandBuffer->handle(), 0, 1, &viewport);
1545 vk::CmdSetScissor(m_commandBuffer->handle(), 0, 1, &scissor);
1546 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_);
1547 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_layout_.handle(), 0, 1,
1548 &descriptor_set.set_, 0, nullptr);
1549
1550 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
1551 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
1552 m_errorMonitor->VerifyFound();
1553
1554 m_commandBuffer->EndRenderPass();
1555 m_commandBuffer->end();
1556
John Zulaufbe8562b2020-12-15 14:21:01 -07001557 // Repeat the draw test with a WaitEvent to protect it.
1558 m_errorMonitor->ExpectSuccess();
1559 m_commandBuffer->reset();
1560 m_commandBuffer->begin();
1561
1562 vk::CmdCopyBuffer(m_commandBuffer->handle(), vbo2.handle(), vbo.handle(), 1, &buffer_region);
1563
Mark Lobodzinski07d0a612020-12-30 15:42:31 -07001564 auto vbo_barrier = LvlInitStruct<VkBufferMemoryBarrier>();
John Zulaufbe8562b2020-12-15 14:21:01 -07001565 vbo_barrier.srcAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
1566 vbo_barrier.dstAccessMask = VK_ACCESS_VERTEX_ATTRIBUTE_READ_BIT;
1567 vbo_barrier.srcQueueFamilyIndex = VK_QUEUE_FAMILY_IGNORED;
1568 vbo_barrier.dstQueueFamilyIndex = VK_QUEUE_FAMILY_IGNORED;
1569 vbo_barrier.buffer = vbo.handle();
1570 vbo_barrier.offset = buffer_region.dstOffset;
1571 vbo_barrier.size = buffer_region.size;
1572
1573 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
1574
1575 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
1576 vk::CmdBindVertexBuffers(m_commandBuffer->handle(), 0, 1, &vbo.handle(), &offset);
1577 vk::CmdSetViewport(m_commandBuffer->handle(), 0, 1, &viewport);
1578 vk::CmdSetScissor(m_commandBuffer->handle(), 0, 1, &scissor);
1579 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_);
1580 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_layout_.handle(), 0, 1,
1581 &descriptor_set.set_, 0, nullptr);
1582
1583 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_VERTEX_INPUT_BIT, 0, nullptr, 1,
1584 &vbo_barrier, 0, nullptr);
1585 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
1586
1587 m_commandBuffer->EndRenderPass();
1588 m_commandBuffer->end();
1589 m_errorMonitor->VerifyNotFound();
1590
Jeremy Gebben170781d2020-11-19 16:21:21 -07001591 // DrawIndexed
1592 m_errorMonitor->ExpectSuccess();
1593 const float ibo_data[3] = {0.f, 0.f, 0.f};
1594 VkBufferObj ibo, ibo2;
1595 buffer_usage = VK_BUFFER_USAGE_INDEX_BUFFER_BIT | VK_BUFFER_USAGE_TRANSFER_SRC_BIT | VK_BUFFER_USAGE_TRANSFER_DST_BIT;
1596 ibo.init(*m_device, ibo.create_info(sizeof(ibo_data), buffer_usage, nullptr), mem_prop);
1597 ibo2.init(*m_device, ibo2.create_info(sizeof(ibo_data), buffer_usage, nullptr), mem_prop);
1598
1599 m_commandBuffer->reset();
1600 m_commandBuffer->begin();
1601 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
1602 vk::CmdBindVertexBuffers(m_commandBuffer->handle(), 0, 1, &vbo.handle(), &offset);
1603 vk::CmdBindIndexBuffer(m_commandBuffer->handle(), ibo.handle(), 0, VK_INDEX_TYPE_UINT16);
1604 vk::CmdSetViewport(m_commandBuffer->handle(), 0, 1, &viewport);
1605 vk::CmdSetScissor(m_commandBuffer->handle(), 0, 1, &scissor);
1606
1607 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_);
1608 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_layout_.handle(), 0, 1,
1609 &descriptor_set.set_, 0, nullptr);
1610 m_commandBuffer->DrawIndexed(3, 1, 0, 0, 0);
1611 m_commandBuffer->EndRenderPass();
1612 m_commandBuffer->end();
1613 m_errorMonitor->VerifyNotFound();
1614
1615 m_commandBuffer->reset();
1616 m_commandBuffer->begin();
1617
1618 buffer_region = {0, 0, sizeof(ibo_data)};
1619 vk::CmdCopyBuffer(m_commandBuffer->handle(), ibo2.handle(), ibo.handle(), 1, &buffer_region);
1620
1621 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
1622 vk::CmdBindVertexBuffers(m_commandBuffer->handle(), 0, 1, &vbo.handle(), &offset);
1623 vk::CmdBindIndexBuffer(m_commandBuffer->handle(), ibo.handle(), 0, VK_INDEX_TYPE_UINT16);
1624 vk::CmdSetViewport(m_commandBuffer->handle(), 0, 1, &viewport);
1625 vk::CmdSetScissor(m_commandBuffer->handle(), 0, 1, &scissor);
1626 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_);
1627 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_layout_.handle(), 0, 1,
1628 &descriptor_set.set_, 0, nullptr);
1629
1630 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
1631 m_commandBuffer->DrawIndexed(3, 1, 0, 0, 0);
1632 m_errorMonitor->VerifyFound();
1633
1634 m_commandBuffer->EndRenderPass();
1635 m_commandBuffer->end();
1636
1637 // DrawIndirect
1638 m_errorMonitor->ExpectSuccess();
1639 VkBufferObj buffer_drawIndirect, buffer_drawIndirect2;
1640 buffer_usage = VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT | VK_BUFFER_USAGE_TRANSFER_SRC_BIT | VK_BUFFER_USAGE_TRANSFER_DST_BIT;
1641 buffer_drawIndirect.init(*m_device, buffer_drawIndirect.create_info(sizeof(VkDrawIndirectCommand), buffer_usage, nullptr),
1642 mem_prop);
1643 buffer_drawIndirect2.init(*m_device, buffer_drawIndirect2.create_info(sizeof(VkDrawIndirectCommand), buffer_usage, nullptr),
1644 mem_prop);
1645
1646 m_commandBuffer->reset();
1647 m_commandBuffer->begin();
1648 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
1649 vk::CmdBindVertexBuffers(m_commandBuffer->handle(), 0, 1, &vbo.handle(), &offset);
1650 vk::CmdSetViewport(m_commandBuffer->handle(), 0, 1, &viewport);
1651 vk::CmdSetScissor(m_commandBuffer->handle(), 0, 1, &scissor);
1652
1653 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_);
1654 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_layout_.handle(), 0, 1,
1655 &descriptor_set.set_, 0, nullptr);
1656 vk::CmdDrawIndirect(m_commandBuffer->handle(), buffer_drawIndirect.handle(), 0, 1, sizeof(VkDrawIndirectCommand));
1657 m_commandBuffer->EndRenderPass();
1658 m_commandBuffer->end();
1659 m_errorMonitor->VerifyNotFound();
1660
1661 m_commandBuffer->reset();
1662 m_commandBuffer->begin();
1663
1664 buffer_region = {0, 0, sizeof(VkDrawIndirectCommand)};
1665 vk::CmdCopyBuffer(m_commandBuffer->handle(), buffer_drawIndirect2.handle(), buffer_drawIndirect.handle(), 1, &buffer_region);
1666
1667 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
1668 vk::CmdBindVertexBuffers(m_commandBuffer->handle(), 0, 1, &vbo.handle(), &offset);
1669 vk::CmdSetViewport(m_commandBuffer->handle(), 0, 1, &viewport);
1670 vk::CmdSetScissor(m_commandBuffer->handle(), 0, 1, &scissor);
1671 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_);
1672 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_layout_.handle(), 0, 1,
1673 &descriptor_set.set_, 0, nullptr);
1674
1675 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
1676 vk::CmdDrawIndirect(m_commandBuffer->handle(), buffer_drawIndirect.handle(), 0, 1, sizeof(VkDrawIndirectCommand));
1677 m_errorMonitor->VerifyFound();
1678
1679 m_commandBuffer->EndRenderPass();
1680 m_commandBuffer->end();
1681
1682 // DrawIndexedIndirect
1683 m_errorMonitor->ExpectSuccess();
1684 VkBufferObj buffer_drawIndexedIndirect, buffer_drawIndexedIndirect2;
1685 buffer_usage = VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT | VK_BUFFER_USAGE_TRANSFER_SRC_BIT | VK_BUFFER_USAGE_TRANSFER_DST_BIT;
1686 buffer_drawIndexedIndirect.init(
1687 *m_device, buffer_drawIndexedIndirect.create_info(sizeof(VkDrawIndexedIndirectCommand), buffer_usage, nullptr), mem_prop);
1688 buffer_drawIndexedIndirect2.init(
1689 *m_device, buffer_drawIndexedIndirect2.create_info(sizeof(VkDrawIndexedIndirectCommand), buffer_usage, nullptr), mem_prop);
1690
1691 m_commandBuffer->reset();
1692 m_commandBuffer->begin();
1693 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
1694 vk::CmdBindVertexBuffers(m_commandBuffer->handle(), 0, 1, &vbo.handle(), &offset);
1695 vk::CmdBindIndexBuffer(m_commandBuffer->handle(), ibo.handle(), 0, VK_INDEX_TYPE_UINT16);
1696 vk::CmdSetViewport(m_commandBuffer->handle(), 0, 1, &viewport);
1697 vk::CmdSetScissor(m_commandBuffer->handle(), 0, 1, &scissor);
1698
1699 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_);
1700 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_layout_.handle(), 0, 1,
1701 &descriptor_set.set_, 0, nullptr);
1702 vk::CmdDrawIndexedIndirect(m_commandBuffer->handle(), buffer_drawIndirect.handle(), 0, 1, sizeof(VkDrawIndexedIndirectCommand));
1703 m_commandBuffer->EndRenderPass();
1704 m_commandBuffer->end();
1705 m_errorMonitor->VerifyNotFound();
1706
1707 m_commandBuffer->reset();
1708 m_commandBuffer->begin();
1709
1710 buffer_region = {0, 0, sizeof(VkDrawIndexedIndirectCommand)};
1711 vk::CmdCopyBuffer(m_commandBuffer->handle(), buffer_drawIndexedIndirect2.handle(), buffer_drawIndexedIndirect.handle(), 1,
1712 &buffer_region);
1713
1714 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
1715 vk::CmdBindVertexBuffers(m_commandBuffer->handle(), 0, 1, &vbo.handle(), &offset);
1716 vk::CmdBindIndexBuffer(m_commandBuffer->handle(), ibo.handle(), 0, VK_INDEX_TYPE_UINT16);
1717 vk::CmdSetViewport(m_commandBuffer->handle(), 0, 1, &viewport);
1718 vk::CmdSetScissor(m_commandBuffer->handle(), 0, 1, &scissor);
1719 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_);
1720 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_layout_.handle(), 0, 1,
1721 &descriptor_set.set_, 0, nullptr);
1722
1723 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
1724 vk::CmdDrawIndexedIndirect(m_commandBuffer->handle(), buffer_drawIndexedIndirect.handle(), 0, 1,
1725 sizeof(VkDrawIndexedIndirectCommand));
1726 m_errorMonitor->VerifyFound();
1727
1728 m_commandBuffer->EndRenderPass();
1729 m_commandBuffer->end();
1730
1731 if (has_khr_indirect) {
1732 // DrawIndirectCount
1733 auto fpCmdDrawIndirectCountKHR =
1734 (PFN_vkCmdDrawIndirectCount)vk::GetDeviceProcAddr(m_device->device(), "vkCmdDrawIndirectCountKHR");
1735 if (!fpCmdDrawIndirectCountKHR) {
1736 printf("%s Test requires unsupported vkCmdDrawIndirectCountKHR feature. Skipped.\n", kSkipPrefix);
1737 } else {
1738 m_errorMonitor->ExpectSuccess();
1739 VkBufferObj buffer_count, buffer_count2;
1740 buffer_usage =
1741 VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT | VK_BUFFER_USAGE_TRANSFER_SRC_BIT | VK_BUFFER_USAGE_TRANSFER_DST_BIT;
1742 buffer_count.init(*m_device, buffer_count.create_info(sizeof(uint32_t), buffer_usage, nullptr), mem_prop);
1743 buffer_count2.init(*m_device, buffer_count2.create_info(sizeof(uint32_t), buffer_usage, nullptr), mem_prop);
1744
1745 m_commandBuffer->reset();
1746 m_commandBuffer->begin();
1747 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
1748 vk::CmdBindVertexBuffers(m_commandBuffer->handle(), 0, 1, &vbo.handle(), &offset);
1749 vk::CmdSetViewport(m_commandBuffer->handle(), 0, 1, &viewport);
1750 vk::CmdSetScissor(m_commandBuffer->handle(), 0, 1, &scissor);
1751
1752 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_);
1753 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_layout_.handle(),
1754 0, 1, &descriptor_set.set_, 0, nullptr);
1755 fpCmdDrawIndirectCountKHR(m_commandBuffer->handle(), buffer_drawIndirect.handle(), 0, buffer_count.handle(), 0, 1,
1756 sizeof(VkDrawIndirectCommand));
1757 m_commandBuffer->EndRenderPass();
1758 m_commandBuffer->end();
1759 m_errorMonitor->VerifyNotFound();
1760
1761 m_commandBuffer->reset();
1762 m_commandBuffer->begin();
1763
1764 buffer_region = {0, 0, sizeof(uint32_t)};
1765 vk::CmdCopyBuffer(m_commandBuffer->handle(), buffer_count2.handle(), buffer_count.handle(), 1, &buffer_region);
1766
1767 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
1768 vk::CmdBindVertexBuffers(m_commandBuffer->handle(), 0, 1, &vbo.handle(), &offset);
1769 vk::CmdSetViewport(m_commandBuffer->handle(), 0, 1, &viewport);
1770 vk::CmdSetScissor(m_commandBuffer->handle(), 0, 1, &scissor);
1771 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_);
1772 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_layout_.handle(),
1773 0, 1, &descriptor_set.set_, 0, nullptr);
1774
1775 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
1776 fpCmdDrawIndirectCountKHR(m_commandBuffer->handle(), buffer_drawIndirect.handle(), 0, buffer_count.handle(), 0, 1,
1777 sizeof(VkDrawIndirectCommand));
1778 m_errorMonitor->VerifyFound();
1779
1780 m_commandBuffer->EndRenderPass();
1781 m_commandBuffer->end();
1782 }
1783
1784 // DrawIndexedIndirectCount
1785 auto fpCmdDrawIndexIndirectCountKHR =
1786 (PFN_vkCmdDrawIndirectCount)vk::GetDeviceProcAddr(m_device->device(), "vkCmdDrawIndexedIndirectCountKHR");
1787 if (!fpCmdDrawIndexIndirectCountKHR) {
1788 printf("%s Test requires unsupported vkCmdDrawIndexedIndirectCountKHR feature. Skipped.\n", kSkipPrefix);
1789 } else {
1790 m_errorMonitor->ExpectSuccess();
1791 VkBufferObj buffer_count, buffer_count2;
1792 buffer_usage =
1793 VK_BUFFER_USAGE_INDIRECT_BUFFER_BIT | VK_BUFFER_USAGE_TRANSFER_SRC_BIT | VK_BUFFER_USAGE_TRANSFER_DST_BIT;
1794 buffer_count.init(*m_device, buffer_count.create_info(sizeof(uint32_t), buffer_usage, nullptr), mem_prop);
1795 buffer_count2.init(*m_device, buffer_count2.create_info(sizeof(uint32_t), buffer_usage, nullptr), mem_prop);
1796
1797 m_commandBuffer->reset();
1798 m_commandBuffer->begin();
1799 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
1800 vk::CmdBindVertexBuffers(m_commandBuffer->handle(), 0, 1, &vbo.handle(), &offset);
1801 vk::CmdBindIndexBuffer(m_commandBuffer->handle(), ibo.handle(), 0, VK_INDEX_TYPE_UINT16);
1802 vk::CmdSetViewport(m_commandBuffer->handle(), 0, 1, &viewport);
1803 vk::CmdSetScissor(m_commandBuffer->handle(), 0, 1, &scissor);
1804
1805 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_);
1806 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_layout_.handle(),
1807 0, 1, &descriptor_set.set_, 0, nullptr);
1808 fpCmdDrawIndexIndirectCountKHR(m_commandBuffer->handle(), buffer_drawIndexedIndirect.handle(), 0, buffer_count.handle(),
1809 0, 1, sizeof(VkDrawIndexedIndirectCommand));
1810 m_commandBuffer->EndRenderPass();
1811 m_commandBuffer->end();
1812 m_errorMonitor->VerifyNotFound();
1813
1814 m_commandBuffer->reset();
1815 m_commandBuffer->begin();
1816
1817 buffer_region = {0, 0, sizeof(uint32_t)};
1818 vk::CmdCopyBuffer(m_commandBuffer->handle(), buffer_count2.handle(), buffer_count.handle(), 1, &buffer_region);
1819
1820 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
1821 vk::CmdBindVertexBuffers(m_commandBuffer->handle(), 0, 1, &vbo.handle(), &offset);
1822 vk::CmdBindIndexBuffer(m_commandBuffer->handle(), ibo.handle(), 0, VK_INDEX_TYPE_UINT16);
1823 vk::CmdSetViewport(m_commandBuffer->handle(), 0, 1, &viewport);
1824 vk::CmdSetScissor(m_commandBuffer->handle(), 0, 1, &scissor);
1825 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_);
1826 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_layout_.handle(),
1827 0, 1, &descriptor_set.set_, 0, nullptr);
1828
1829 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
1830 fpCmdDrawIndexIndirectCountKHR(m_commandBuffer->handle(), buffer_drawIndexedIndirect.handle(), 0, buffer_count.handle(),
1831 0, 1, sizeof(VkDrawIndexedIndirectCommand));
1832 m_errorMonitor->VerifyFound();
1833
1834 m_commandBuffer->EndRenderPass();
1835 m_commandBuffer->end();
1836 }
1837 } else {
1838 printf("%s Test requires unsupported vkCmdDrawIndirectCountKHR & vkDrawIndexedIndirectCountKHR feature. Skipped.\n",
1839 kSkipPrefix);
1840 }
1841}
1842
1843TEST_F(VkSyncValTest, SyncCmdClear) {
1844 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
1845 ASSERT_NO_FATAL_FAILURE(InitState(nullptr, nullptr, VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT));
1846 // CmdClearColorImage
1847 m_errorMonitor->ExpectSuccess();
1848 VkImageUsageFlags usage = VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT;
1849 VkFormat format = VK_FORMAT_R8G8B8A8_UNORM;
1850 VkImageObj image_a(m_device), image_b(m_device);
1851 auto image_ci = VkImageObj::ImageCreateInfo2D(128, 128, 1, 1, format, usage, VK_IMAGE_TILING_OPTIMAL);
1852 image_a.Init(image_ci);
1853 image_b.Init(image_ci);
1854
1855 VkImageSubresourceLayers layers_all{VK_IMAGE_ASPECT_COLOR_BIT, 0, 0, 1};
1856 VkOffset3D zero_offset{0, 0, 0};
1857 VkExtent3D full_extent{128, 128, 1}; // <-- image type is 2D
1858 VkImageSubresourceRange full_subresource_range{VK_IMAGE_ASPECT_COLOR_BIT, 0, 1, 0, 1};
1859
1860 VkImageCopy full_region = {layers_all, zero_offset, layers_all, zero_offset, full_extent};
1861
1862 m_commandBuffer->begin();
1863
1864 image_b.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
1865 image_a.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
1866
1867 auto cb = m_commandBuffer->handle();
1868 VkClearColorValue ccv = {};
1869 vk::CmdClearColorImage(m_commandBuffer->handle(), image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, &ccv, 1, &full_subresource_range);
1870 m_commandBuffer->end();
1871 m_errorMonitor->VerifyNotFound();
1872
1873 m_commandBuffer->reset();
1874 m_commandBuffer->begin();
1875 vk::CmdCopyImage(cb, image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &full_region);
1876
1877 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
1878 vk::CmdClearColorImage(m_commandBuffer->handle(), image_a.handle(), VK_IMAGE_LAYOUT_GENERAL, &ccv, 1, &full_subresource_range);
1879 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
1880 vk::CmdClearColorImage(m_commandBuffer->handle(), image_b.handle(), VK_IMAGE_LAYOUT_GENERAL, &ccv, 1, &full_subresource_range);
1881 m_errorMonitor->VerifyFound();
1882
1883 m_commandBuffer->end();
1884
1885 // CmdClearDepthStencilImage
1886 format = FindSupportedDepthStencilFormat(gpu());
1887 if (!format) {
1888 printf("%s No Depth + Stencil format found. Skipped.\n", kSkipPrefix);
1889 return;
1890 }
1891 m_errorMonitor->ExpectSuccess();
1892 VkImageObj image_ds_a(m_device), image_ds_b(m_device);
1893 image_ci = VkImageObj::ImageCreateInfo2D(128, 128, 1, 1, format, usage, VK_IMAGE_TILING_OPTIMAL);
1894 image_ds_a.Init(image_ci);
1895 image_ds_b.Init(image_ci);
1896
1897 const VkImageAspectFlags ds_aspect = VK_IMAGE_ASPECT_DEPTH_BIT | VK_IMAGE_ASPECT_STENCIL_BIT;
1898 image_ds_a.SetLayout(ds_aspect, VK_IMAGE_LAYOUT_GENERAL);
1899 image_ds_b.SetLayout(ds_aspect, VK_IMAGE_LAYOUT_GENERAL);
1900
1901 m_commandBuffer->begin();
1902 const VkClearDepthStencilValue clear_value = {};
1903 VkImageSubresourceRange ds_range = {ds_aspect, 0, 1, 0, 1};
1904
1905 vk::CmdClearDepthStencilImage(cb, image_ds_a.handle(), VK_IMAGE_LAYOUT_GENERAL, &clear_value, 1, &ds_range);
1906 m_commandBuffer->end();
1907 m_errorMonitor->VerifyNotFound();
1908
1909 VkImageSubresourceLayers ds_layers_all{ds_aspect, 0, 0, 1};
1910 VkImageCopy ds_full_region = {ds_layers_all, zero_offset, ds_layers_all, zero_offset, full_extent};
1911
1912 m_commandBuffer->reset();
1913 m_commandBuffer->begin();
1914 vk::CmdCopyImage(cb, image_ds_a.handle(), VK_IMAGE_LAYOUT_GENERAL, image_ds_b.handle(), VK_IMAGE_LAYOUT_GENERAL, 1,
1915 &ds_full_region);
1916
1917 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
1918 vk::CmdClearDepthStencilImage(m_commandBuffer->handle(), image_ds_a.handle(), VK_IMAGE_LAYOUT_GENERAL, &clear_value, 1,
1919 &ds_range);
1920 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
1921 vk::CmdClearDepthStencilImage(m_commandBuffer->handle(), image_ds_b.handle(), VK_IMAGE_LAYOUT_GENERAL, &clear_value, 1,
1922 &ds_range);
1923 m_errorMonitor->VerifyFound();
1924
1925 m_commandBuffer->end();
1926}
1927
1928TEST_F(VkSyncValTest, SyncCmdQuery) {
1929 // CmdCopyQueryPoolResults
1930 m_errorMonitor->ExpectSuccess();
1931 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
1932 ASSERT_NO_FATAL_FAILURE(InitState(nullptr, nullptr, VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT));
1933 if (IsPlatform(kNexusPlayer)) {
1934 printf("%s This test should not run on Nexus Player\n", kSkipPrefix);
1935 return;
1936 }
1937 if ((m_device->queue_props.empty()) || (m_device->queue_props[0].queueCount < 2)) {
1938 printf("%s Queue family needs to have multiple queues to run this test.\n", kSkipPrefix);
1939 return;
1940 }
1941 uint32_t queue_count;
1942 vk::GetPhysicalDeviceQueueFamilyProperties(gpu(), &queue_count, NULL);
Jeremy Gebbend2573fc2021-05-12 17:17:38 -06001943 std::vector<VkQueueFamilyProperties> queue_props(queue_count);
1944 vk::GetPhysicalDeviceQueueFamilyProperties(gpu(), &queue_count, queue_props.data());
Jeremy Gebben170781d2020-11-19 16:21:21 -07001945 if (queue_props[m_device->graphics_queue_node_index_].timestampValidBits == 0) {
1946 printf("%s Device graphic queue has timestampValidBits of 0, skipping.\n", kSkipPrefix);
1947 return;
1948 }
1949
Jeremy Gebben18ac1052021-08-12 11:07:32 -06001950 vk_testing::QueryPool query_pool;
sfricke-samsung6fc3e322022-02-15 22:41:29 -08001951 VkQueryPoolCreateInfo query_pool_create_info = LvlInitStruct<VkQueryPoolCreateInfo>();
Jeremy Gebben170781d2020-11-19 16:21:21 -07001952 query_pool_create_info.queryType = VK_QUERY_TYPE_TIMESTAMP;
1953 query_pool_create_info.queryCount = 1;
Jeremy Gebben18ac1052021-08-12 11:07:32 -06001954 query_pool.init(*m_device, query_pool_create_info);
Jeremy Gebben170781d2020-11-19 16:21:21 -07001955
1956 VkBufferObj buffer_a, buffer_b;
1957 VkMemoryPropertyFlags mem_prop = VK_MEMORY_PROPERTY_DEVICE_LOCAL_BIT;
1958 buffer_a.init_as_src_and_dst(*m_device, 256, mem_prop);
1959 buffer_b.init_as_src_and_dst(*m_device, 256, mem_prop);
1960
1961 VkBufferCopy region = {0, 0, 256};
1962
1963 auto cb = m_commandBuffer->handle();
1964 m_commandBuffer->begin();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06001965 vk::CmdResetQueryPool(cb, query_pool.handle(), 0, 1);
1966 vk::CmdWriteTimestamp(cb, VK_PIPELINE_STAGE_ALL_GRAPHICS_BIT, query_pool.handle(), 0);
1967 vk::CmdCopyQueryPoolResults(cb, query_pool.handle(), 0, 1, buffer_a.handle(), 0, 0, VK_QUERY_RESULT_WAIT_BIT);
Jeremy Gebben170781d2020-11-19 16:21:21 -07001968 m_commandBuffer->end();
1969 m_errorMonitor->VerifyNotFound();
1970
1971 m_commandBuffer->reset();
1972 m_commandBuffer->begin();
1973 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &region);
Jeremy Gebben18ac1052021-08-12 11:07:32 -06001974 vk::CmdResetQueryPool(cb, query_pool.handle(), 0, 1);
1975 vk::CmdWriteTimestamp(cb, VK_PIPELINE_STAGE_ALL_GRAPHICS_BIT, query_pool.handle(), 0);
Jeremy Gebben170781d2020-11-19 16:21:21 -07001976 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
Jeremy Gebben18ac1052021-08-12 11:07:32 -06001977 vk::CmdCopyQueryPoolResults(cb, query_pool.handle(), 0, 1, buffer_a.handle(), 0, 256, VK_QUERY_RESULT_WAIT_BIT);
Jeremy Gebben170781d2020-11-19 16:21:21 -07001978 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
Jeremy Gebben18ac1052021-08-12 11:07:32 -06001979 vk::CmdCopyQueryPoolResults(cb, query_pool.handle(), 0, 1, buffer_b.handle(), 0, 256, VK_QUERY_RESULT_WAIT_BIT);
Jeremy Gebben170781d2020-11-19 16:21:21 -07001980 m_commandBuffer->end();
1981 m_errorMonitor->VerifyFound();
1982
1983 // TODO:Track VkQueryPool
1984 // TODO:CmdWriteTimestamp
Jeremy Gebben170781d2020-11-19 16:21:21 -07001985}
1986
1987TEST_F(VkSyncValTest, SyncCmdDrawDepthStencil) {
1988 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
1989 ASSERT_NO_FATAL_FAILURE(InitState(nullptr, nullptr, VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT));
1990 m_errorMonitor->ExpectSuccess();
1991
1992 const auto format_ds = FindSupportedDepthStencilFormat(gpu());
1993 if (!format_ds) {
1994 printf("%s No Depth + Stencil format found. Skipped.\n", kSkipPrefix);
1995 return;
1996 }
1997 const auto format_dp = FindSupportedDepthOnlyFormat(gpu());
1998 if (!format_dp) {
1999 printf("%s No only Depth format found. Skipped.\n", kSkipPrefix);
2000 return;
2001 }
2002 const auto format_st = FindSupportedStencilOnlyFormat(gpu());
2003 if (!format_st) {
2004 printf("%s No only Stencil format found. Skipped.\n", kSkipPrefix);
2005 return;
2006 }
2007
2008 VkDepthStencilObj image_ds(m_device), image_dp(m_device), image_st(m_device);
2009 image_ds.Init(m_device, 16, 16, format_ds);
2010 image_dp.Init(m_device, 16, 16, format_dp);
2011 image_st.Init(m_device, 16, 16, format_st);
2012
2013 VkRenderpassObj rp_ds(m_device, format_ds, true), rp_dp(m_device, format_dp, true), rp_st(m_device, format_st, true);
2014
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002015 vk_testing::Framebuffer fb_ds, fb_dp, fb_st;
Jeremy Gebben170781d2020-11-19 16:21:21 -07002016 VkFramebufferCreateInfo fbci = {
2017 VK_STRUCTURE_TYPE_FRAMEBUFFER_CREATE_INFO, nullptr, 0, rp_ds.handle(), 1, image_ds.BindInfo(), 16, 16, 1};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002018 fb_ds.init(*m_device, fbci);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002019 fbci = {VK_STRUCTURE_TYPE_FRAMEBUFFER_CREATE_INFO, nullptr, 0, rp_dp.handle(), 1, image_dp.BindInfo(), 16, 16, 1};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002020 fb_dp.init(*m_device, fbci);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002021 fbci = {VK_STRUCTURE_TYPE_FRAMEBUFFER_CREATE_INFO, nullptr, 0, rp_st.handle(), 1, image_st.BindInfo(), 16, 16, 1};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002022 fb_st.init(*m_device, fbci);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002023
2024 VkStencilOpState stencil = {};
2025 stencil.failOp = VK_STENCIL_OP_KEEP;
2026 stencil.passOp = VK_STENCIL_OP_KEEP;
2027 stencil.depthFailOp = VK_STENCIL_OP_KEEP;
2028 stencil.compareOp = VK_COMPARE_OP_NEVER;
2029
Mark Lobodzinski07d0a612020-12-30 15:42:31 -07002030 auto ds_ci = LvlInitStruct<VkPipelineDepthStencilStateCreateInfo>();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002031 ds_ci.depthTestEnable = VK_TRUE;
2032 ds_ci.depthWriteEnable = VK_TRUE;
2033 ds_ci.depthCompareOp = VK_COMPARE_OP_NEVER;
2034 ds_ci.stencilTestEnable = VK_TRUE;
2035 ds_ci.front = stencil;
2036 ds_ci.back = stencil;
2037
2038 CreatePipelineHelper g_pipe_ds(*this), g_pipe_dp(*this), g_pipe_st(*this);
2039 g_pipe_ds.InitInfo();
2040 g_pipe_ds.gp_ci_.renderPass = rp_ds.handle();
2041 g_pipe_ds.gp_ci_.pDepthStencilState = &ds_ci;
2042 g_pipe_ds.InitState();
2043 ASSERT_VK_SUCCESS(g_pipe_ds.CreateGraphicsPipeline());
2044 g_pipe_dp.InitInfo();
2045 g_pipe_dp.gp_ci_.renderPass = rp_dp.handle();
2046 ds_ci.stencilTestEnable = VK_FALSE;
2047 g_pipe_dp.gp_ci_.pDepthStencilState = &ds_ci;
2048 g_pipe_dp.InitState();
2049 ASSERT_VK_SUCCESS(g_pipe_dp.CreateGraphicsPipeline());
2050 g_pipe_st.InitInfo();
2051 g_pipe_st.gp_ci_.renderPass = rp_st.handle();
2052 ds_ci.depthTestEnable = VK_FALSE;
2053 ds_ci.stencilTestEnable = VK_TRUE;
2054 g_pipe_st.gp_ci_.pDepthStencilState = &ds_ci;
2055 g_pipe_st.InitState();
2056 ASSERT_VK_SUCCESS(g_pipe_st.CreateGraphicsPipeline());
2057
2058 m_commandBuffer->begin();
2059 m_renderPassBeginInfo.renderArea = {{0, 0}, {16, 16}};
2060 m_renderPassBeginInfo.pClearValues = nullptr;
2061 m_renderPassBeginInfo.clearValueCount = 0;
2062
2063 m_renderPassBeginInfo.renderPass = rp_ds.handle();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002064 m_renderPassBeginInfo.framebuffer = fb_ds.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002065 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2066 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_ds.pipeline_);
2067 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
2068 m_commandBuffer->EndRenderPass();
2069
2070 m_renderPassBeginInfo.renderPass = rp_dp.handle();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002071 m_renderPassBeginInfo.framebuffer = fb_dp.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002072 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2073 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_dp.pipeline_);
2074 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
2075 m_commandBuffer->EndRenderPass();
2076
2077 m_renderPassBeginInfo.renderPass = rp_st.handle();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002078 m_renderPassBeginInfo.framebuffer = fb_st.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002079 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2080 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_st.pipeline_);
2081 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
2082 m_commandBuffer->EndRenderPass();
2083
2084 m_commandBuffer->end();
2085 m_errorMonitor->VerifyNotFound();
2086
2087 m_commandBuffer->reset();
2088 m_commandBuffer->begin();
2089
2090 VkImageCopy copyRegion;
2091 copyRegion.srcSubresource.aspectMask = VK_IMAGE_ASPECT_DEPTH_BIT;
2092 copyRegion.srcSubresource.mipLevel = 0;
2093 copyRegion.srcSubresource.baseArrayLayer = 0;
2094 copyRegion.srcSubresource.layerCount = 1;
2095 copyRegion.srcOffset = {0, 0, 0};
2096 copyRegion.dstSubresource.aspectMask = VK_IMAGE_ASPECT_DEPTH_BIT;
2097 copyRegion.dstSubresource.mipLevel = 0;
2098 copyRegion.dstSubresource.baseArrayLayer = 0;
2099 copyRegion.dstSubresource.layerCount = 1;
2100 copyRegion.dstOffset = {0, 0, 0};
2101 copyRegion.extent = {16, 16, 1};
2102
2103 m_commandBuffer->CopyImage(image_ds.handle(), VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL, image_dp.handle(),
2104 VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL, 1, &copyRegion);
2105
2106 copyRegion.srcSubresource.aspectMask = VK_IMAGE_ASPECT_STENCIL_BIT;
2107 copyRegion.dstSubresource.aspectMask = VK_IMAGE_ASPECT_STENCIL_BIT;
2108 m_commandBuffer->CopyImage(image_ds.handle(), VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL, image_st.handle(),
2109 VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL, 1, &copyRegion);
2110 m_renderPassBeginInfo.renderPass = rp_ds.handle();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002111 m_renderPassBeginInfo.framebuffer = fb_ds.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002112 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
2113 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2114 m_errorMonitor->VerifyFound();
2115 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2116 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_ds.pipeline_);
2117 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
2118 m_commandBuffer->EndRenderPass();
2119
2120 m_renderPassBeginInfo.renderPass = rp_dp.handle();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002121 m_renderPassBeginInfo.framebuffer = fb_dp.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002122 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
2123 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2124 m_errorMonitor->VerifyFound();
2125 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2126 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_dp.pipeline_);
2127 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
2128 m_commandBuffer->EndRenderPass();
2129
2130 m_renderPassBeginInfo.renderPass = rp_st.handle();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002131 m_renderPassBeginInfo.framebuffer = fb_st.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002132 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
2133 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2134 m_errorMonitor->VerifyFound();
2135 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2136 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_st.pipeline_);
2137 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
2138 m_commandBuffer->EndRenderPass();
2139
2140 m_commandBuffer->end();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002141}
2142
John Zulaufd57a36b2021-08-16 10:34:44 -06002143
Jeremy Gebben170781d2020-11-19 16:21:21 -07002144TEST_F(VkSyncValTest, RenderPassLoadHazardVsInitialLayout) {
2145 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
John Zulaufd57a36b2021-08-16 10:34:44 -06002146 bool do_none_load_op_test = false;
2147 if (DeviceExtensionSupported(gpu(), nullptr, VK_EXT_LOAD_STORE_OP_NONE_EXTENSION_NAME)) {
2148 m_device_extension_names.push_back(VK_EXT_LOAD_STORE_OP_NONE_EXTENSION_NAME);
2149 do_none_load_op_test = true;
2150 }
2151
Jeremy Gebben170781d2020-11-19 16:21:21 -07002152 ASSERT_NO_FATAL_FAILURE(InitState());
2153 ASSERT_NO_FATAL_FAILURE(InitRenderTarget());
2154
2155 VkImageUsageFlags usage_color = VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT;
2156 VkImageUsageFlags usage_input = VK_IMAGE_USAGE_TRANSFER_DST_BIT | VK_IMAGE_USAGE_INPUT_ATTACHMENT_BIT;
2157 VkFormat format = VK_FORMAT_R8G8B8A8_UNORM;
2158 VkImageObj image_color(m_device), image_input(m_device);
2159 auto image_ci = VkImageObj::ImageCreateInfo2D(32, 32, 1, 1, format, usage_color, VK_IMAGE_TILING_OPTIMAL);
2160 image_color.Init(image_ci);
2161 image_ci.usage = usage_input;
2162 image_input.Init(image_ci);
2163 VkImageView attachments[] = {image_color.targetView(format), image_input.targetView(format)};
2164
John Zulaufd57a36b2021-08-16 10:34:44 -06002165 VkAttachmentDescription attachmentDescriptions[] = {
Jeremy Gebben170781d2020-11-19 16:21:21 -07002166 // Result attachment
2167 {(VkAttachmentDescriptionFlags)0, VK_FORMAT_R8G8B8A8_UNORM, VK_SAMPLE_COUNT_1_BIT, VK_ATTACHMENT_LOAD_OP_CLEAR,
2168 VK_ATTACHMENT_STORE_OP_STORE, VK_ATTACHMENT_LOAD_OP_DONT_CARE, VK_ATTACHMENT_STORE_OP_DONT_CARE,
2169 VK_IMAGE_LAYOUT_UNDEFINED, // Here causes DesiredError that SYNC-HAZARD-NONE in BeginRenderPass.
2170 // It should be VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL
2171 VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL},
2172 // Input attachment
2173 {(VkAttachmentDescriptionFlags)0, VK_FORMAT_R8G8B8A8_UNORM, VK_SAMPLE_COUNT_1_BIT, VK_ATTACHMENT_LOAD_OP_LOAD,
2174 VK_ATTACHMENT_STORE_OP_STORE, VK_ATTACHMENT_LOAD_OP_DONT_CARE, VK_ATTACHMENT_STORE_OP_DONT_CARE,
2175 VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL, VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL}};
2176
2177 const VkAttachmentReference resultAttachmentRef = {0u, VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL};
2178 const VkAttachmentReference inputAttachmentRef = {1u, VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL};
2179
2180 const VkSubpassDescription subpassDescription = {(VkSubpassDescriptionFlags)0,
2181 VK_PIPELINE_BIND_POINT_GRAPHICS,
2182 1u,
2183 &inputAttachmentRef,
2184 1u,
2185 &resultAttachmentRef,
2186 0,
2187 0,
2188 0u,
2189 0};
2190
2191 const VkSubpassDependency subpassDependency = {VK_SUBPASS_EXTERNAL,
2192 0,
2193 VK_PIPELINE_STAGE_TRANSFER_BIT,
2194 VK_PIPELINE_STAGE_FRAGMENT_SHADER_BIT,
2195 VK_ACCESS_TRANSFER_WRITE_BIT,
2196 VK_ACCESS_INPUT_ATTACHMENT_READ_BIT | VK_ACCESS_SHADER_READ_BIT,
2197 VK_DEPENDENCY_BY_REGION_BIT};
2198
2199 const VkRenderPassCreateInfo renderPassInfo = {VK_STRUCTURE_TYPE_RENDER_PASS_CREATE_INFO,
2200 0,
2201 (VkRenderPassCreateFlags)0,
2202 2u,
2203 attachmentDescriptions,
2204 1u,
2205 &subpassDescription,
2206 1u,
2207 &subpassDependency};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002208 vk_testing::RenderPass rp;
2209 rp.init(*m_device, renderPassInfo);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002210
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002211 vk_testing::Framebuffer fb;
2212 VkFramebufferCreateInfo fbci = {VK_STRUCTURE_TYPE_FRAMEBUFFER_CREATE_INFO, nullptr, 0, rp.handle(), 2, attachments, 32, 32, 1};
2213 fb.init(*m_device, fbci);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002214
2215 image_input.SetLayout(VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL);
2216
2217 m_commandBuffer->begin();
2218
2219 m_renderPassBeginInfo.renderArea = {{0, 0}, {32, 32}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002220 m_renderPassBeginInfo.renderPass = rp.handle();
2221 m_renderPassBeginInfo.framebuffer = fb.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002222
2223 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
2224 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
2225 // Even though we have no accesses prior, the layout transition *is* an access, so load can be validated vs. layout transition
2226 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2227 m_errorMonitor->VerifyFound();
John Zulaufd57a36b2021-08-16 10:34:44 -06002228
2229 vk_testing::RenderPass rp_no_load_store;
2230 if (do_none_load_op_test) {
2231 m_errorMonitor->ExpectSuccess();
2232 attachmentDescriptions[0].loadOp = VK_ATTACHMENT_LOAD_OP_NONE_EXT;
2233 attachmentDescriptions[0].storeOp = VK_ATTACHMENT_STORE_OP_NONE_EXT;
2234 attachmentDescriptions[1].loadOp = VK_ATTACHMENT_LOAD_OP_NONE_EXT;
2235 attachmentDescriptions[1].storeOp = VK_ATTACHMENT_STORE_OP_NONE_EXT;
2236 rp_no_load_store.init(*m_device, renderPassInfo);
2237 m_renderPassBeginInfo.renderPass = rp_no_load_store.handle();
2238 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2239 m_commandBuffer->EndRenderPass();
2240 m_errorMonitor->VerifyNotFound();
2241 } else {
2242 printf("%s VK_EXT_load_store_op_none not supported, skipping sub-test\n", kSkipPrefix);
2243 }
Jeremy Gebben170781d2020-11-19 16:21:21 -07002244}
2245
2246TEST_F(VkSyncValTest, SyncRenderPassWithWrongDepthStencilInitialLayout) {
2247 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
2248 ASSERT_NO_FATAL_FAILURE(InitState());
2249 if (IsPlatform(kNexusPlayer)) {
2250 printf("%s This test should not run on Nexus Player\n", kSkipPrefix);
2251 return;
2252 }
2253
2254 ASSERT_NO_FATAL_FAILURE(InitRenderTarget());
2255
2256 VkFormat color_format = VK_FORMAT_R8G8B8A8_UNORM;
2257 VkFormat ds_format = FindSupportedDepthStencilFormat(gpu());
2258 if (!ds_format) {
2259 printf("%s No Depth + Stencil format found. Skipped.\n", kSkipPrefix);
2260 return;
2261 }
2262 VkImageUsageFlags usage_color = VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT;
2263 VkImageUsageFlags usage_ds = VK_IMAGE_USAGE_DEPTH_STENCIL_ATTACHMENT_BIT;
2264 VkImageObj image_color(m_device), image_color2(m_device);
2265 auto image_ci = VkImageObj::ImageCreateInfo2D(32, 32, 1, 1, color_format, usage_color, VK_IMAGE_TILING_OPTIMAL);
2266 image_color.Init(image_ci);
2267 image_color2.Init(image_ci);
2268 VkDepthStencilObj image_ds(m_device);
2269 image_ds.Init(m_device, 32, 32, ds_format, usage_ds);
2270
2271 const VkAttachmentDescription colorAttachmentDescription = {(VkAttachmentDescriptionFlags)0,
2272 color_format,
2273 VK_SAMPLE_COUNT_1_BIT,
2274 VK_ATTACHMENT_LOAD_OP_CLEAR,
2275 VK_ATTACHMENT_STORE_OP_STORE,
2276 VK_ATTACHMENT_LOAD_OP_DONT_CARE,
2277 VK_ATTACHMENT_STORE_OP_DONT_CARE,
2278 VK_IMAGE_LAYOUT_UNDEFINED,
2279 VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL};
2280
2281 const VkAttachmentDescription depthStencilAttachmentDescription = {
2282 (VkAttachmentDescriptionFlags)0, ds_format, VK_SAMPLE_COUNT_1_BIT, VK_ATTACHMENT_LOAD_OP_CLEAR,
2283 VK_ATTACHMENT_STORE_OP_STORE, VK_ATTACHMENT_LOAD_OP_CLEAR, VK_ATTACHMENT_STORE_OP_STORE,
2284 VK_IMAGE_LAYOUT_UNDEFINED, // Here causes DesiredError that SYNC-HAZARD-WRITE_AFTER_WRITE in BeginRenderPass.
2285 // It should be VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL
2286 VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL};
2287
2288 std::vector<VkAttachmentDescription> attachmentDescriptions;
2289 attachmentDescriptions.push_back(colorAttachmentDescription);
2290 attachmentDescriptions.push_back(depthStencilAttachmentDescription);
2291
2292 const VkAttachmentReference colorAttachmentRef = {0u, VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL};
2293
2294 const VkAttachmentReference depthStencilAttachmentRef = {1u, VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL};
2295
2296 const VkSubpassDescription subpassDescription = {(VkSubpassDescriptionFlags)0,
2297 VK_PIPELINE_BIND_POINT_GRAPHICS,
2298 0u,
2299 0,
2300 1u,
2301 &colorAttachmentRef,
2302 0,
2303 &depthStencilAttachmentRef,
2304 0u,
2305 0};
2306
2307 const VkRenderPassCreateInfo renderPassInfo = {VK_STRUCTURE_TYPE_RENDER_PASS_CREATE_INFO,
2308 0,
2309 (VkRenderPassCreateFlags)0,
2310 (uint32_t)attachmentDescriptions.size(),
2311 &attachmentDescriptions[0],
2312 1u,
2313 &subpassDescription,
2314 0u,
2315 0};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002316 vk_testing::RenderPass rp;
2317 rp.init(*m_device, renderPassInfo);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002318
2319 VkImageView fb_attachments[] = {image_color.targetView(color_format),
2320 image_ds.targetView(ds_format, VK_IMAGE_ASPECT_DEPTH_BIT | VK_IMAGE_ASPECT_STENCIL_BIT)};
2321 const VkFramebufferCreateInfo fbci = {
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002322 VK_STRUCTURE_TYPE_FRAMEBUFFER_CREATE_INFO, 0, 0u, rp.handle(), 2u, fb_attachments, 32, 32, 1u,
Jeremy Gebben170781d2020-11-19 16:21:21 -07002323 };
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002324 vk_testing::Framebuffer fb;
2325 fb.init(*m_device, fbci);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002326 fb_attachments[0] = image_color2.targetView(color_format);
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002327 vk_testing::Framebuffer fb1;
2328 fb1.init(*m_device, fbci);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002329
2330 CreatePipelineHelper g_pipe(*this);
2331 g_pipe.InitInfo();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002332 g_pipe.gp_ci_.renderPass = rp.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002333
2334 VkStencilOpState stencil = {};
2335 stencil.failOp = VK_STENCIL_OP_KEEP;
2336 stencil.passOp = VK_STENCIL_OP_KEEP;
2337 stencil.depthFailOp = VK_STENCIL_OP_KEEP;
2338 stencil.compareOp = VK_COMPARE_OP_NEVER;
2339
Mark Lobodzinski07d0a612020-12-30 15:42:31 -07002340 auto ds_ci = LvlInitStruct<VkPipelineDepthStencilStateCreateInfo>();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002341 ds_ci.depthTestEnable = VK_TRUE;
2342 ds_ci.depthWriteEnable = VK_TRUE;
2343 ds_ci.depthCompareOp = VK_COMPARE_OP_NEVER;
2344 ds_ci.stencilTestEnable = VK_TRUE;
2345 ds_ci.front = stencil;
2346 ds_ci.back = stencil;
2347
2348 g_pipe.gp_ci_.pDepthStencilState = &ds_ci;
2349 g_pipe.InitState();
2350 ASSERT_VK_SUCCESS(g_pipe.CreateGraphicsPipeline());
2351
2352 m_commandBuffer->begin();
Tony-LunarG73f37032021-06-07 11:47:03 -06002353 VkClearValue clear = {};
2354 std::array<VkClearValue, 2> clear_values = { {clear, clear} };
2355 m_renderPassBeginInfo.pClearValues = clear_values.data();
2356 m_renderPassBeginInfo.clearValueCount = clear_values.size();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002357 m_renderPassBeginInfo.renderArea = {{0, 0}, {32, 32}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002358 m_renderPassBeginInfo.renderPass = rp.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002359
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002360 m_renderPassBeginInfo.framebuffer = fb.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002361 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2362 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_);
2363 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
2364 m_commandBuffer->EndRenderPass();
2365
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002366 m_renderPassBeginInfo.framebuffer = fb1.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002367
2368 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
2369 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2370 m_errorMonitor->VerifyFound();
2371}
2372
2373TEST_F(VkSyncValTest, SyncLayoutTransition) {
2374 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
2375 ASSERT_NO_FATAL_FAILURE(InitState());
2376 if (IsPlatform(kNexusPlayer)) {
2377 printf("%s This test should not run on Nexus Player\n", kSkipPrefix);
2378 return;
2379 }
2380
2381 ASSERT_NO_FATAL_FAILURE(InitRenderTarget());
2382
2383 VkImageUsageFlags usage_color = VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT | VK_IMAGE_USAGE_TRANSFER_SRC_BIT;
2384 VkImageUsageFlags usage_input =
2385 VK_IMAGE_USAGE_INPUT_ATTACHMENT_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT | VK_IMAGE_USAGE_SAMPLED_BIT;
2386 VkFormat format = VK_FORMAT_R8G8B8A8_UNORM;
2387 VkImageObj image_color(m_device), image_input(m_device);
2388 auto image_ci = VkImageObj::ImageCreateInfo2D(64, 64, 1, 1, format, usage_input, VK_IMAGE_TILING_OPTIMAL);
2389 image_input.InitNoLayout(image_ci);
2390 image_ci.usage = usage_color;
2391 image_color.InitNoLayout(image_ci);
2392 VkImageView view_input = image_input.targetView(format);
2393 VkImageView view_color = image_color.targetView(format);
2394 VkImageView attachments[] = {view_color, view_input};
2395
2396 const VkAttachmentDescription fbAttachment = {
2397 0u,
2398 format,
2399 VK_SAMPLE_COUNT_1_BIT,
2400 VK_ATTACHMENT_LOAD_OP_CLEAR,
2401 VK_ATTACHMENT_STORE_OP_STORE,
2402 VK_ATTACHMENT_LOAD_OP_DONT_CARE,
2403 VK_ATTACHMENT_STORE_OP_DONT_CARE,
2404 VK_IMAGE_LAYOUT_UNDEFINED,
2405 VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL,
2406 };
2407
2408 std::vector<VkAttachmentDescription> attachmentDescs;
2409 attachmentDescs.push_back(fbAttachment);
2410
2411 // Add it as a frame buffer attachment.
2412 const VkAttachmentDescription inputAttachment = {
2413 0u,
2414 format,
2415 VK_SAMPLE_COUNT_1_BIT,
2416 VK_ATTACHMENT_LOAD_OP_LOAD,
2417 VK_ATTACHMENT_STORE_OP_DONT_CARE,
2418 VK_ATTACHMENT_LOAD_OP_DONT_CARE,
2419 VK_ATTACHMENT_STORE_OP_DONT_CARE,
2420 VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL,
2421 VK_IMAGE_LAYOUT_GENERAL,
2422 };
2423 attachmentDescs.push_back(inputAttachment);
2424
2425 std::vector<VkAttachmentReference> inputAttachments;
2426 const VkAttachmentReference inputRef = {
2427 1u,
2428 VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL,
2429 };
2430 inputAttachments.push_back(inputRef);
2431
2432 const VkAttachmentReference colorRef = {
2433 0u,
2434 VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL,
2435 };
2436 const std::vector<VkAttachmentReference> colorAttachments(1u, colorRef);
2437
2438 const VkSubpassDescription subpass = {
2439 0u,
2440 VK_PIPELINE_BIND_POINT_GRAPHICS,
2441 static_cast<uint32_t>(inputAttachments.size()),
2442 inputAttachments.data(),
2443 static_cast<uint32_t>(colorAttachments.size()),
2444 colorAttachments.data(),
2445 0u,
2446 nullptr,
2447 0u,
2448 nullptr,
2449 };
2450 const std::vector<VkSubpassDescription> subpasses(1u, subpass);
2451
2452 const VkRenderPassCreateInfo renderPassInfo = {
2453 VK_STRUCTURE_TYPE_RENDER_PASS_CREATE_INFO,
2454 nullptr,
2455 0u,
2456 static_cast<uint32_t>(attachmentDescs.size()),
2457 attachmentDescs.data(),
2458 static_cast<uint32_t>(subpasses.size()),
2459 subpasses.data(),
2460 0u,
2461 nullptr,
2462 };
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002463 vk_testing::RenderPass rp;
2464 rp.init(*m_device, renderPassInfo);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002465
2466 const VkFramebufferCreateInfo fbci = {
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002467 VK_STRUCTURE_TYPE_FRAMEBUFFER_CREATE_INFO, 0, 0u, rp.handle(), 2u, attachments, 64, 64, 1u,
Jeremy Gebben170781d2020-11-19 16:21:21 -07002468 };
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002469 vk_testing::Framebuffer fb;
2470 fb.init(*m_device, fbci);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002471
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002472 vk_testing::Sampler sampler;
Jeremy Gebben170781d2020-11-19 16:21:21 -07002473 VkSamplerCreateInfo sampler_info = SafeSaneSamplerCreateInfo();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002474 sampler.init(*m_device, sampler_info);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002475
sfricke-samsung1c0b96a2021-07-08 22:24:09 -07002476 char const *fsSource = R"glsl(
2477 #version 450
2478 layout(input_attachment_index=0, set=0, binding=0) uniform subpassInput x;
2479 void main() {
2480 vec4 color = subpassLoad(x);
2481 }
2482 )glsl";
Jeremy Gebben170781d2020-11-19 16:21:21 -07002483
sfricke-samsungae54c1e2022-01-21 05:35:21 -08002484 VkShaderObj vs(this, bindStateVertShaderText, VK_SHADER_STAGE_VERTEX_BIT);
2485 VkShaderObj fs(this, fsSource, VK_SHADER_STAGE_FRAGMENT_BIT);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002486
2487 CreatePipelineHelper g_pipe(*this);
2488 g_pipe.InitInfo();
2489 g_pipe.shader_stages_ = {vs.GetStageCreateInfo(), fs.GetStageCreateInfo()};
2490 g_pipe.dsl_bindings_ = {{0, VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT, 1, VK_SHADER_STAGE_FRAGMENT_BIT, nullptr}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002491 g_pipe.gp_ci_.renderPass = rp.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002492 g_pipe.InitState();
2493 ASSERT_VK_SUCCESS(g_pipe.CreateGraphicsPipeline());
2494
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002495 g_pipe.descriptor_set_->WriteDescriptorImageInfo(0, view_input, sampler.handle(), VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002496 g_pipe.descriptor_set_->UpdateDescriptorSets();
2497
2498 m_commandBuffer->begin();
2499 auto cb = m_commandBuffer->handle();
2500 VkClearColorValue ccv = {};
2501 VkImageSubresourceRange full_subresource_range{VK_IMAGE_ASPECT_COLOR_BIT, 0, 1, 0, 1};
2502
2503 const VkImageMemoryBarrier preClearBarrier = {
2504 VK_STRUCTURE_TYPE_IMAGE_MEMORY_BARRIER, 0, 0, VK_ACCESS_TRANSFER_WRITE_BIT, VK_IMAGE_LAYOUT_UNDEFINED,
2505 VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL, 0, 0, image_input.handle(), full_subresource_range,
2506 };
2507 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TOP_OF_PIPE_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0u, 0u, nullptr, 0u, nullptr, 1u,
2508 &preClearBarrier);
2509
2510 vk::CmdClearColorImage(m_commandBuffer->handle(), image_input.handle(), VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL, &ccv, 1,
2511 &full_subresource_range);
2512
2513 const VkImageMemoryBarrier postClearBarrier = {
2514 VK_STRUCTURE_TYPE_IMAGE_MEMORY_BARRIER,
2515 0,
2516 VK_ACCESS_TRANSFER_WRITE_BIT,
2517 VK_ACCESS_SHADER_READ_BIT,
2518 VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL,
2519 VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL,
2520 0,
2521 0,
2522 image_input.handle(),
2523 full_subresource_range,
2524 };
2525 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_FRAGMENT_SHADER_BIT, 0u, 0u, nullptr, 0u, nullptr,
2526 1u, &postClearBarrier);
2527
2528 m_renderPassBeginInfo.renderArea = {{0, 0}, {64, 64}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002529 m_renderPassBeginInfo.renderPass = rp.handle();
2530 m_renderPassBeginInfo.framebuffer = fb.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002531
2532 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2533 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_);
2534 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_layout_.handle(), 0, 1,
2535 &g_pipe.descriptor_set_->set_, 0, nullptr);
2536
2537 // Positive test for ordering rules between load and input attachment usage
2538 m_errorMonitor->ExpectSuccess();
2539 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
2540
2541 // Positive test for store ordering vs. input attachment and dependency *to* external for layout transition
2542 m_commandBuffer->EndRenderPass();
2543 m_errorMonitor->VerifyNotFound();
2544
2545 // Catch a conflict with the input attachment final layout transition
2546 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
2547 vk::CmdClearColorImage(m_commandBuffer->handle(), image_input.handle(), VK_IMAGE_LAYOUT_GENERAL, &ccv, 1,
2548 &full_subresource_range);
2549 m_errorMonitor->VerifyFound();
John Zulaufe972b752021-05-04 15:47:17 -06002550
2551 // There should be no hazard for ILT after ILT
2552 m_errorMonitor->ExpectSuccess();
2553 m_commandBuffer->end();
2554 m_commandBuffer->reset();
2555 m_commandBuffer->begin();
2556 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TOP_OF_PIPE_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0u, 0u, nullptr, 0u, nullptr, 1u,
2557 &preClearBarrier);
2558 const VkImageMemoryBarrier wawBarrier = {
2559 VK_STRUCTURE_TYPE_IMAGE_MEMORY_BARRIER,
2560 0,
2561 VK_ACCESS_SHADER_READ_BIT,
2562 VK_ACCESS_SHADER_READ_BIT,
2563 VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL,
2564 VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL,
2565 0,
2566 0,
2567 image_input.handle(),
2568 full_subresource_range,
2569 };
2570 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "SYNC-HAZARD-WRITE_AFTER_WRITE");
2571 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_VERTEX_SHADER_BIT, VK_PIPELINE_STAGE_FRAGMENT_SHADER_BIT, 0u, 0u, nullptr, 0u,
2572 nullptr, 1u, &wawBarrier);
2573 m_errorMonitor->VerifyNotFound();
2574 m_commandBuffer->end();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002575}
2576
2577TEST_F(VkSyncValTest, SyncSubpassMultiDep) {
2578 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
2579 ASSERT_NO_FATAL_FAILURE(InitState());
2580 if (IsPlatform(kNexusPlayer)) {
2581 printf("%s This test should not run on Nexus Player\n", kSkipPrefix);
2582 return;
2583 }
2584
2585 ASSERT_NO_FATAL_FAILURE(InitRenderTarget());
2586
2587 VkImageUsageFlags usage_color = VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT | VK_IMAGE_USAGE_TRANSFER_SRC_BIT;
2588 VkImageUsageFlags usage_input =
2589 VK_IMAGE_USAGE_INPUT_ATTACHMENT_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT | VK_IMAGE_USAGE_SAMPLED_BIT;
2590 VkFormat format = VK_FORMAT_R8G8B8A8_UNORM;
2591 VkImageObj image_color(m_device), image_input(m_device);
2592 auto image_ci = VkImageObj::ImageCreateInfo2D(64, 64, 1, 1, format, usage_input, VK_IMAGE_TILING_OPTIMAL);
2593 image_input.InitNoLayout(image_ci);
2594 image_ci.usage = usage_color;
2595 image_color.InitNoLayout(image_ci);
2596 VkImageView view_input = image_input.targetView(format);
2597 VkImageView view_color = image_color.targetView(format);
2598 VkImageView attachments[] = {view_color, view_input};
2599 VkImageSubresourceRange full_subresource_range{VK_IMAGE_ASPECT_COLOR_BIT, 0, 1, 0, 1};
2600 VkImageSubresourceLayers mip_0_layer_0{VK_IMAGE_ASPECT_COLOR_BIT, 0, 0, 1};
2601 VkOffset3D image_zero{0, 0, 0};
2602 VkExtent3D image_size{64, 64, 1};
2603 VkImageCopy full_region{mip_0_layer_0, image_zero, mip_0_layer_0, image_zero, image_size};
2604
2605 const VkAttachmentDescription fbAttachment = {
2606 0u,
2607 format,
2608 VK_SAMPLE_COUNT_1_BIT,
2609 VK_ATTACHMENT_LOAD_OP_CLEAR,
2610 VK_ATTACHMENT_STORE_OP_STORE,
2611 VK_ATTACHMENT_LOAD_OP_DONT_CARE,
2612 VK_ATTACHMENT_STORE_OP_STORE,
2613 VK_IMAGE_LAYOUT_GENERAL,
2614 VK_IMAGE_LAYOUT_GENERAL,
2615 };
2616
2617 std::vector<VkAttachmentDescription> attachmentDescs;
2618 attachmentDescs.push_back(fbAttachment);
2619
2620 // Add it as a frame buffer attachment.
2621 const VkAttachmentDescription inputAttachment = {
2622 0u,
2623 format,
2624 VK_SAMPLE_COUNT_1_BIT,
2625 VK_ATTACHMENT_LOAD_OP_LOAD,
2626 VK_ATTACHMENT_STORE_OP_DONT_CARE,
2627 VK_ATTACHMENT_LOAD_OP_DONT_CARE,
2628 VK_ATTACHMENT_STORE_OP_DONT_CARE,
2629 VK_IMAGE_LAYOUT_GENERAL,
2630 VK_IMAGE_LAYOUT_GENERAL,
2631 };
2632 attachmentDescs.push_back(inputAttachment);
2633
2634 std::vector<VkAttachmentReference> inputAttachments;
2635 const VkAttachmentReference inputRef = {
2636 1u,
2637 VK_IMAGE_LAYOUT_GENERAL,
2638 };
2639 inputAttachments.push_back(inputRef);
2640
2641 const VkAttachmentReference colorRef = {
2642 0u,
2643 VK_IMAGE_LAYOUT_GENERAL,
2644 };
2645 const std::vector<VkAttachmentReference> colorAttachments(1u, colorRef);
2646
2647 const VkSubpassDescription subpass = {
2648 0u,
2649 VK_PIPELINE_BIND_POINT_GRAPHICS,
2650 static_cast<uint32_t>(inputAttachments.size()),
2651 inputAttachments.data(),
2652 static_cast<uint32_t>(colorAttachments.size()),
2653 colorAttachments.data(),
2654 0u,
2655 nullptr,
2656 0u,
2657 nullptr,
2658 };
2659 const std::vector<VkSubpassDescription> subpasses(1u, subpass);
2660
2661 std::vector<VkSubpassDependency> subpass_dep_postive;
2662 subpass_dep_postive.push_back({VK_SUBPASS_EXTERNAL, 0, VK_PIPELINE_STAGE_TRANSFER_BIT,
2663 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT, 0, VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT,
2664 VK_DEPENDENCY_VIEW_LOCAL_BIT});
2665 subpass_dep_postive.push_back({VK_SUBPASS_EXTERNAL, 0, VK_PIPELINE_STAGE_TRANSFER_BIT,
2666 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT, VK_ACCESS_TRANSFER_WRITE_BIT,
2667 VK_ACCESS_COLOR_ATTACHMENT_READ_BIT, VK_DEPENDENCY_VIEW_LOCAL_BIT});
2668 subpass_dep_postive.push_back({0, VK_SUBPASS_EXTERNAL, VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT,
2669 VK_PIPELINE_STAGE_TRANSFER_BIT, VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT,
2670 VK_ACCESS_TRANSFER_READ_BIT, VK_DEPENDENCY_VIEW_LOCAL_BIT});
2671 subpass_dep_postive.push_back({0, VK_SUBPASS_EXTERNAL, VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT,
2672 VK_PIPELINE_STAGE_TRANSFER_BIT, VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT,
2673 VK_ACCESS_TRANSFER_WRITE_BIT, VK_DEPENDENCY_VIEW_LOCAL_BIT});
2674
2675 VkRenderPassCreateInfo renderPassInfo = {
2676 VK_STRUCTURE_TYPE_RENDER_PASS_CREATE_INFO,
2677 nullptr,
2678 0u,
2679 static_cast<uint32_t>(attachmentDescs.size()),
2680 attachmentDescs.data(),
2681 static_cast<uint32_t>(subpasses.size()),
2682 subpasses.data(),
2683 static_cast<uint32_t>(subpass_dep_postive.size()),
2684 subpass_dep_postive.data(),
2685 };
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002686 vk_testing::RenderPass rp_positive;
2687 rp_positive.init(*m_device, renderPassInfo);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002688
2689 std::vector<VkSubpassDependency> subpass_dep_negative;
2690 subpass_dep_negative.push_back({VK_SUBPASS_EXTERNAL, 0, VK_PIPELINE_STAGE_TRANSFER_BIT,
2691 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT, 0, VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT,
2692 VK_DEPENDENCY_VIEW_LOCAL_BIT});
2693 // Show that the two barriers do *not* chain by breaking the positive barrier into two bits.
2694 subpass_dep_negative.push_back({VK_SUBPASS_EXTERNAL, 0, VK_PIPELINE_STAGE_TRANSFER_BIT,
2695 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT, VK_ACCESS_TRANSFER_WRITE_BIT, 0,
2696 VK_DEPENDENCY_VIEW_LOCAL_BIT});
2697 subpass_dep_negative.push_back({VK_SUBPASS_EXTERNAL, 0, VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT,
2698 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT, 0, VK_ACCESS_COLOR_ATTACHMENT_READ_BIT,
2699 VK_DEPENDENCY_VIEW_LOCAL_BIT});
2700
2701 renderPassInfo.dependencyCount = static_cast<uint32_t>(subpass_dep_negative.size());
2702 renderPassInfo.pDependencies = subpass_dep_negative.data();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002703 vk_testing::RenderPass rp_negative;
2704 rp_negative.init(*m_device, renderPassInfo);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002705
2706 // rp_postive and rp_negative should be compatible for the same fb object
2707 const VkFramebufferCreateInfo fbci = {
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002708 VK_STRUCTURE_TYPE_FRAMEBUFFER_CREATE_INFO, 0, 0u, rp_positive.handle(), 2u, attachments, 64, 64, 1u,
Jeremy Gebben170781d2020-11-19 16:21:21 -07002709 };
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002710 vk_testing::Framebuffer fb;
2711 fb.init(*m_device, fbci);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002712
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002713 vk_testing::Sampler sampler;
Jeremy Gebben170781d2020-11-19 16:21:21 -07002714 VkSamplerCreateInfo sampler_info = SafeSaneSamplerCreateInfo();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002715 sampler.init(*m_device, sampler_info);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002716
sfricke-samsung1c0b96a2021-07-08 22:24:09 -07002717 char const *fsSource = R"glsl(
2718 #version 450
2719 layout(input_attachment_index=0, set=0, binding=0) uniform subpassInput x;
2720 void main() {
2721 vec4 color = subpassLoad(x);
2722 }
2723 )glsl";
Jeremy Gebben170781d2020-11-19 16:21:21 -07002724
sfricke-samsungae54c1e2022-01-21 05:35:21 -08002725 VkShaderObj vs(this, bindStateVertShaderText, VK_SHADER_STAGE_VERTEX_BIT);
2726 VkShaderObj fs(this, fsSource, VK_SHADER_STAGE_FRAGMENT_BIT);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002727
2728 CreatePipelineHelper g_pipe(*this);
2729 g_pipe.InitInfo();
2730 g_pipe.shader_stages_ = {vs.GetStageCreateInfo(), fs.GetStageCreateInfo()};
2731 g_pipe.dsl_bindings_ = {{0, VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT, 1, VK_SHADER_STAGE_FRAGMENT_BIT, nullptr}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002732 g_pipe.gp_ci_.renderPass = rp_positive.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002733 g_pipe.InitState();
2734 ASSERT_VK_SUCCESS(g_pipe.CreateGraphicsPipeline());
2735
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002736 g_pipe.descriptor_set_->WriteDescriptorImageInfo(0, view_input, sampler.handle(), VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002737 g_pipe.descriptor_set_->UpdateDescriptorSets();
2738
2739 m_commandBuffer->begin();
2740 auto cb = m_commandBuffer->handle();
2741 VkClearColorValue ccv = {};
2742
2743 const VkImageMemoryBarrier xferDestBarrier = {VK_STRUCTURE_TYPE_IMAGE_MEMORY_BARRIER,
2744 nullptr,
2745 VK_ACCESS_MEMORY_READ_BIT | VK_ACCESS_MEMORY_WRITE_BIT,
2746 VK_ACCESS_TRANSFER_WRITE_BIT,
2747 VK_IMAGE_LAYOUT_GENERAL,
2748 VK_IMAGE_LAYOUT_GENERAL,
2749 VK_QUEUE_FAMILY_IGNORED,
2750 VK_QUEUE_FAMILY_IGNORED,
2751 VK_NULL_HANDLE,
2752 full_subresource_range};
2753 const VkImageMemoryBarrier xferDestToSrcBarrier = {
2754 VK_STRUCTURE_TYPE_IMAGE_MEMORY_BARRIER,
2755 nullptr,
2756 VK_ACCESS_TRANSFER_WRITE_BIT,
2757 VK_ACCESS_TRANSFER_READ_BIT,
2758 VK_IMAGE_LAYOUT_GENERAL,
2759 VK_IMAGE_LAYOUT_GENERAL,
2760 VK_QUEUE_FAMILY_IGNORED,
2761 VK_QUEUE_FAMILY_IGNORED,
2762 VK_NULL_HANDLE,
2763 full_subresource_range,
2764 };
2765
2766 VkImageMemoryBarrier preClearBarrier = xferDestBarrier;
2767 preClearBarrier.image = image_color.handle();
2768
2769 VkImageMemoryBarrier preCopyBarriers[2] = {xferDestToSrcBarrier, xferDestBarrier};
2770 preCopyBarriers[0].image = image_color.handle();
2771 preCopyBarriers[1].image = image_input.handle();
2772 // Positive test for ordering rules between load and input attachment usage
2773 m_errorMonitor->ExpectSuccess();
2774
2775 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TOP_OF_PIPE_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0u, 0u, nullptr, 0u, nullptr, 1u,
2776 &preClearBarrier);
2777
2778 vk::CmdClearColorImage(m_commandBuffer->handle(), image_color.handle(), VK_IMAGE_LAYOUT_GENERAL, &ccv, 1,
2779 &full_subresource_range);
2780
2781 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0u, 0u, nullptr, 0u, nullptr, 2u,
2782 preCopyBarriers);
2783
2784 vk::CmdCopyImage(m_commandBuffer->handle(), image_color.handle(), VK_IMAGE_LAYOUT_GENERAL, image_input.handle(),
2785 VK_IMAGE_LAYOUT_GENERAL, 1u, &full_region);
2786
2787 // No post copy image barrier, we are testing the subpass dependencies
2788
2789 m_renderPassBeginInfo.renderArea = {{0, 0}, {64, 64}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002790 m_renderPassBeginInfo.renderPass = rp_positive.handle();
2791 m_renderPassBeginInfo.framebuffer = fb.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002792
2793 // Postive renderpass multidependency test
2794 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2795 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_);
2796 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_layout_.handle(), 0, 1,
2797 &g_pipe.descriptor_set_->set_, 0, nullptr);
2798
2799 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
2800
2801 // Positive test for store ordering vs. input attachment and dependency *to* external for layout transition
2802 m_commandBuffer->EndRenderPass();
2803 // m_errorMonitor->VerifyNotFound();
2804
2805 vk::CmdCopyImage(m_commandBuffer->handle(), image_color.handle(), VK_IMAGE_LAYOUT_GENERAL, image_input.handle(),
2806 VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL, 1u, &full_region);
2807 m_errorMonitor->VerifyNotFound();
2808
2809 m_renderPassBeginInfo.renderArea = {{0, 0}, {64, 64}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002810 m_renderPassBeginInfo.renderPass = rp_negative.handle();
2811 m_renderPassBeginInfo.framebuffer = fb.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002812
2813 // Postive renderpass multidependency test, will fail IFF the dependencies are acting indepently.
2814 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "SYNC-HAZARD-READ_AFTER_WRITE");
2815 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2816 m_errorMonitor->VerifyFound();
2817}
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002818
2819TEST_F(VkSyncValTest, RenderPassAsyncHazard) {
2820 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
2821 ASSERT_NO_FATAL_FAILURE(InitState());
2822
Nathaniel Cesariof9cd1a82021-07-24 08:48:55 -06002823 if (IsPlatform(kPixel3) || IsPlatform(kPixel3aXL)) {
2824 printf("%s Temporarily disabling on Pixel 3 and Pixel 3a XL due to driver crash\n", kSkipPrefix);
2825 return;
2826 }
2827
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002828 // overall set up:
2829 // subpass 0:
2830 // write image 0
2831 // subpass 1:
2832 // read image 0
2833 // write image 1
2834 // subpass 2:
2835 // read image 0
2836 // write image 2
2837 // subpass 3:
2838 // read image 0
2839 // write image 3
2840 //
2841 // subpasses 1 & 2 can run in parallel but both should depend on 0
2842 // subpass 3 must run after 1 & 2 because otherwise the store operation will
2843 // race with the reads in the other subpasses.
2844
2845 constexpr VkFormat kFormat = VK_FORMAT_R8G8B8A8_UNORM;
2846 constexpr uint32_t kWidth = 32, kHeight = 32;
2847 constexpr uint32_t kNumImages = 4;
2848
sfricke-samsung6fc3e322022-02-15 22:41:29 -08002849 VkImageCreateInfo src_img_info = LvlInitStruct<VkImageCreateInfo>();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002850 src_img_info.flags = 0;
2851 src_img_info.imageType = VK_IMAGE_TYPE_2D;
2852 src_img_info.format = kFormat;
2853 src_img_info.extent = {kWidth, kHeight, 1};
2854 src_img_info.mipLevels = 1;
2855 src_img_info.arrayLayers = 1;
2856 src_img_info.samples = VK_SAMPLE_COUNT_2_BIT;
2857 src_img_info.tiling = VK_IMAGE_TILING_OPTIMAL;
2858 src_img_info.usage = VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT | VK_IMAGE_USAGE_INPUT_ATTACHMENT_BIT;
2859 src_img_info.sharingMode = VK_SHARING_MODE_EXCLUSIVE;
2860 src_img_info.queueFamilyIndexCount = 0;
2861 src_img_info.pQueueFamilyIndices = nullptr;
2862 src_img_info.initialLayout = VK_IMAGE_LAYOUT_UNDEFINED;
2863
sfricke-samsung6fc3e322022-02-15 22:41:29 -08002864 VkImageCreateInfo dst_img_info = LvlInitStruct<VkImageCreateInfo>();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002865 dst_img_info.flags = 0;
2866 dst_img_info.imageType = VK_IMAGE_TYPE_2D;
2867 dst_img_info.format = kFormat;
2868 dst_img_info.extent = {kWidth, kHeight, 1};
2869 dst_img_info.mipLevels = 1;
2870 dst_img_info.arrayLayers = 1;
2871 dst_img_info.samples = VK_SAMPLE_COUNT_1_BIT;
2872 dst_img_info.tiling = VK_IMAGE_TILING_OPTIMAL;
2873 dst_img_info.usage = VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT;
2874 dst_img_info.sharingMode = VK_SHARING_MODE_EXCLUSIVE;
2875 dst_img_info.queueFamilyIndexCount = 0;
2876 dst_img_info.pQueueFamilyIndices = nullptr;
2877 dst_img_info.initialLayout = VK_IMAGE_LAYOUT_UNDEFINED;
2878
2879 std::vector<std::unique_ptr<VkImageObj>> images;
2880 for (uint32_t i = 0; i < kNumImages; i++) {
2881 images.emplace_back(new VkImageObj(m_device));
2882 }
2883 images[0]->Init(src_img_info);
2884 for (uint32_t i = 1; i < images.size(); i++) {
2885 images[i]->Init(dst_img_info);
2886 }
2887
2888 std::array<VkImageView, kNumImages> attachments{};
2889 std::array<VkAttachmentDescription, kNumImages> attachment_descriptions{};
2890 std::array<VkAttachmentReference, kNumImages> color_refs{};
2891 std::array<VkImageMemoryBarrier, kNumImages> img_barriers{};
2892
2893 for (uint32_t i = 0; i < attachments.size(); i++) {
2894 attachments[i] = images[i]->targetView(kFormat);
2895 attachment_descriptions[i] = {};
2896 attachment_descriptions[i].flags = 0;
2897 attachment_descriptions[i].format = kFormat;
2898 attachment_descriptions[i].samples = VK_SAMPLE_COUNT_1_BIT;
2899 attachment_descriptions[i].loadOp = VK_ATTACHMENT_LOAD_OP_CLEAR;
2900 attachment_descriptions[i].storeOp = VK_ATTACHMENT_STORE_OP_STORE;
2901 attachment_descriptions[i].stencilLoadOp = VK_ATTACHMENT_LOAD_OP_DONT_CARE;
2902 attachment_descriptions[i].stencilStoreOp = VK_ATTACHMENT_STORE_OP_DONT_CARE;
2903 attachment_descriptions[i].initialLayout = VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL;
2904 attachment_descriptions[i].finalLayout =
2905 (i == 0) ? VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL : VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL;
2906
2907 color_refs[i] = {i, VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL};
2908
sfricke-samsung6fc3e322022-02-15 22:41:29 -08002909 img_barriers[i] = LvlInitStruct<VkImageMemoryBarrier>();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002910 img_barriers[i].srcAccessMask = 0;
2911 img_barriers[i].dstAccessMask = VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT;
2912 img_barriers[i].oldLayout = VK_IMAGE_LAYOUT_UNDEFINED;
2913 img_barriers[i].newLayout = VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL;
2914 img_barriers[i].srcQueueFamilyIndex = VK_QUEUE_FAMILY_IGNORED;
2915 img_barriers[i].dstQueueFamilyIndex = VK_QUEUE_FAMILY_IGNORED;
2916 img_barriers[i].image = images[i]->handle();
2917 img_barriers[i].subresourceRange = {VK_IMAGE_ASPECT_COLOR_BIT, 0, VK_REMAINING_MIP_LEVELS, 0, VK_REMAINING_ARRAY_LAYERS};
2918 }
2919
2920 const VkAttachmentReference input_ref{0u, VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL};
2921
2922 std::array<std::array<uint32_t, 2>, kNumImages - 1> preserve_subpass{{{2, 3}, {1, 3}, {1, 2}}};
2923
2924 std::array<VkSubpassDescription, kNumImages> subpasses{};
2925
2926 subpasses[0].pipelineBindPoint = VK_PIPELINE_BIND_POINT_GRAPHICS;
2927 subpasses[0].inputAttachmentCount = 0;
2928 subpasses[0].pInputAttachments = nullptr;
2929 subpasses[0].colorAttachmentCount = 1;
2930 subpasses[0].pColorAttachments = &color_refs[0];
2931
2932 for (uint32_t i = 1; i < subpasses.size(); i++) {
2933 subpasses[i].pipelineBindPoint = VK_PIPELINE_BIND_POINT_GRAPHICS;
2934 subpasses[i].inputAttachmentCount = 1;
2935 subpasses[i].pInputAttachments = &input_ref;
2936 subpasses[i].colorAttachmentCount = 1;
2937 subpasses[i].pColorAttachments = &color_refs[1];
2938 subpasses[i].preserveAttachmentCount = preserve_subpass[i - 1].size();
2939 subpasses[i].pPreserveAttachments = preserve_subpass[i - 1].data();
2940 }
2941
sfricke-samsung6fc3e322022-02-15 22:41:29 -08002942 VkRenderPassCreateInfo renderpass_info = LvlInitStruct<VkRenderPassCreateInfo>();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002943 renderpass_info.flags = 0;
2944 renderpass_info.attachmentCount = attachment_descriptions.size();
2945 renderpass_info.pAttachments = attachment_descriptions.data();
2946 renderpass_info.subpassCount = subpasses.size();
2947 renderpass_info.pSubpasses = subpasses.data();
2948 renderpass_info.dependencyCount = 0;
2949 renderpass_info.pDependencies = nullptr;
2950
sfricke-samsung6fc3e322022-02-15 22:41:29 -08002951 VkFramebufferCreateInfo fbci = LvlInitStruct<VkFramebufferCreateInfo>();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002952 fbci.flags = 0;
2953 fbci.attachmentCount = attachments.size();
2954 fbci.pAttachments = attachments.data();
2955 fbci.width = kWidth;
2956 fbci.height = kHeight;
2957 fbci.layers = 1;
2958
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002959 vk_testing::Sampler sampler;
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002960 VkSamplerCreateInfo sampler_info = SafeSaneSamplerCreateInfo();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002961 sampler.init(*m_device, sampler_info);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002962
sfricke-samsung1c0b96a2021-07-08 22:24:09 -07002963 char const *fsSource = R"glsl(
2964 #version 450
2965 layout(input_attachment_index=0, set=0, binding=0) uniform subpassInput x;
2966 void main() {
2967 vec4 color = subpassLoad(x);
2968 }
2969 )glsl";
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002970
sfricke-samsungae54c1e2022-01-21 05:35:21 -08002971 VkShaderObj vs(this, bindStateVertShaderText, VK_SHADER_STAGE_VERTEX_BIT);
2972 VkShaderObj fs(this, fsSource, VK_SHADER_STAGE_FRAGMENT_BIT);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002973
2974 VkClearValue clear = {};
2975 clear.color = m_clear_color;
Tony-LunarG73f37032021-06-07 11:47:03 -06002976 std::array<VkClearValue, 4> clear_values = {{clear, clear, clear, clear}};
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002977
2978 // run the renderpass with no dependencies
2979 {
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002980 vk_testing::RenderPass rp;
2981 vk_testing::Framebuffer fb;
2982 rp.init(*m_device, renderpass_info);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002983
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002984 fbci.renderPass = rp.handle();
2985 fb.init(*m_device, fbci);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002986
2987 CreatePipelineHelper g_pipe_0(*this);
2988 g_pipe_0.InitInfo();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002989 g_pipe_0.gp_ci_.renderPass = rp.handle();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002990 g_pipe_0.InitState();
2991 ASSERT_VK_SUCCESS(g_pipe_0.CreateGraphicsPipeline());
2992
2993 CreatePipelineHelper g_pipe_12(*this);
2994 g_pipe_12.InitInfo();
2995 g_pipe_12.shader_stages_ = {vs.GetStageCreateInfo(), fs.GetStageCreateInfo()};
2996 g_pipe_12.dsl_bindings_ = {{0, VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT, 1, VK_SHADER_STAGE_FRAGMENT_BIT, nullptr}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002997 g_pipe_12.gp_ci_.renderPass = rp.handle();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002998 g_pipe_12.InitState();
2999 ASSERT_VK_SUCCESS(g_pipe_12.CreateGraphicsPipeline());
3000
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003001 g_pipe_12.descriptor_set_->WriteDescriptorImageInfo(0, attachments[0], sampler.handle(), VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003002 g_pipe_12.descriptor_set_->UpdateDescriptorSets();
3003
3004 m_commandBuffer->begin();
3005
3006 vk::CmdPipelineBarrier(m_commandBuffer->handle(), VK_PIPELINE_STAGE_TOP_OF_PIPE_BIT,
3007 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT, 0, 0, nullptr, 0, nullptr, img_barriers.size(),
3008 img_barriers.data());
3009
3010 m_renderPassBeginInfo.renderArea = {{0, 0}, {16, 16}};
3011 m_renderPassBeginInfo.pClearValues = clear_values.data();
3012 m_renderPassBeginInfo.clearValueCount = clear_values.size();
3013
3014 m_renderPassBeginInfo.renderArea = {{0, 0}, {kWidth, kHeight}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003015 m_renderPassBeginInfo.renderPass = rp.handle();
3016 m_renderPassBeginInfo.framebuffer = fb.handle();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003017
3018 vk::CmdBeginRenderPass(m_commandBuffer->handle(), &m_renderPassBeginInfo, VK_SUBPASS_CONTENTS_INLINE);
3019 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_0.pipeline_);
3020 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_0.pipeline_layout_.handle(), 0,
3021 1, &g_pipe_0.descriptor_set_->set_, 0, NULL);
3022
3023 vk::CmdDraw(m_commandBuffer->handle(), 3, 1, 0, 0);
3024
3025 for (uint32_t i = 1; i < subpasses.size(); i++) {
3026 vk::CmdNextSubpass(m_commandBuffer->handle(), VK_SUBPASS_CONTENTS_INLINE);
3027 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_12.pipeline_);
3028 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS,
3029 g_pipe_12.pipeline_layout_.handle(), 0, 1, &g_pipe_12.descriptor_set_->set_, 0, NULL);
3030
3031 // we're racing the writes from subpass 0 with our shader reads
3032 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ-RACING-WRITE");
3033 vk::CmdDraw(m_commandBuffer->handle(), 3, 1, 0, 0);
3034 m_errorMonitor->VerifyFound();
3035 }
3036
3037 // we should get an error from async checking in both subpasses 2 & 3
3038 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE-RACING-WRITE");
3039 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE-RACING-WRITE");
3040 vk::CmdEndRenderPass(m_commandBuffer->handle());
3041 m_errorMonitor->VerifyFound();
3042
3043 m_commandBuffer->end();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003044 }
3045
3046 // add dependencies from subpass 0 to the others, which are necessary but not sufficient
3047 std::vector<VkSubpassDependency> subpass_dependencies;
3048 for (uint32_t i = 1; i < subpasses.size(); i++) {
3049 VkSubpassDependency dep{0,
3050 i,
3051 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT,
3052 VK_PIPELINE_STAGE_FRAGMENT_SHADER_BIT,
3053 VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT,
3054 VK_ACCESS_INPUT_ATTACHMENT_READ_BIT,
3055 0};
3056 subpass_dependencies.push_back(dep);
3057 }
3058 renderpass_info.dependencyCount = subpass_dependencies.size();
3059 renderpass_info.pDependencies = subpass_dependencies.data();
3060
3061 {
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003062 vk_testing::RenderPass rp;
3063 vk_testing::Framebuffer fb;
3064 rp.init(*m_device, renderpass_info);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003065
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003066 fbci.renderPass = rp.handle();
3067 fb.init(*m_device, fbci);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003068
3069 CreatePipelineHelper g_pipe_0(*this);
3070 g_pipe_0.InitInfo();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003071 g_pipe_0.gp_ci_.renderPass = rp.handle();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003072 g_pipe_0.InitState();
3073 ASSERT_VK_SUCCESS(g_pipe_0.CreateGraphicsPipeline());
3074
3075 CreatePipelineHelper g_pipe_12(*this);
3076 g_pipe_12.InitInfo();
3077 g_pipe_12.shader_stages_ = {vs.GetStageCreateInfo(), fs.GetStageCreateInfo()};
3078 g_pipe_12.dsl_bindings_ = {{0, VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT, 1, VK_SHADER_STAGE_FRAGMENT_BIT, nullptr}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003079 g_pipe_12.gp_ci_.renderPass = rp.handle();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003080 g_pipe_12.InitState();
3081 ASSERT_VK_SUCCESS(g_pipe_12.CreateGraphicsPipeline());
3082
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003083 g_pipe_12.descriptor_set_->WriteDescriptorImageInfo(0, attachments[0], sampler.handle(), VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003084 g_pipe_12.descriptor_set_->UpdateDescriptorSets();
3085
3086 m_commandBuffer->begin();
3087
3088 vk::CmdPipelineBarrier(m_commandBuffer->handle(), VK_PIPELINE_STAGE_TOP_OF_PIPE_BIT,
3089 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT, 0, 0, nullptr, 0, nullptr, img_barriers.size(),
3090 img_barriers.data());
3091
3092 m_renderPassBeginInfo.renderArea = {{0, 0}, {16, 16}};
3093 m_renderPassBeginInfo.pClearValues = clear_values.data();
3094 m_renderPassBeginInfo.clearValueCount = clear_values.size();
3095
3096 m_renderPassBeginInfo.renderArea = {{0, 0}, {kWidth, kHeight}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003097 m_renderPassBeginInfo.renderPass = rp.handle();
3098 m_renderPassBeginInfo.framebuffer = fb.handle();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003099
3100 vk::CmdBeginRenderPass(m_commandBuffer->handle(), &m_renderPassBeginInfo, VK_SUBPASS_CONTENTS_INLINE);
3101 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_0.pipeline_);
3102 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_0.pipeline_layout_.handle(), 0,
3103 1, &g_pipe_0.descriptor_set_->set_, 0, NULL);
3104
3105 vk::CmdDraw(m_commandBuffer->handle(), 3, 1, 0, 0);
3106
3107 m_errorMonitor->ExpectSuccess();
3108 for (uint32_t i = 1; i < subpasses.size(); i++) {
3109 vk::CmdNextSubpass(m_commandBuffer->handle(), VK_SUBPASS_CONTENTS_INLINE);
3110 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_12.pipeline_);
3111 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS,
3112 g_pipe_12.pipeline_layout_.handle(), 0, 1, &g_pipe_12.descriptor_set_->set_, 0, NULL);
3113 vk::CmdDraw(m_commandBuffer->handle(), 3, 1, 0, 0);
3114 }
3115 m_errorMonitor->VerifyNotFound();
3116 // expect this error because 2 subpasses could try to do the store operation
3117 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE-RACING-WRITE");
3118 // ... and this one because the store could happen during a shader read from another subpass
3119 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE-RACING-READ");
3120 vk::CmdEndRenderPass(m_commandBuffer->handle());
3121 m_errorMonitor->VerifyFound();
3122
3123 m_commandBuffer->end();
3124
3125 m_errorMonitor->VerifyFound();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003126 }
3127
3128 // try again with correct dependencies to make subpass 3 depend on 1 & 2
3129 for (uint32_t i = 1; i < (subpasses.size() - 1); i++) {
3130 VkSubpassDependency dep{i,
3131 static_cast<uint32_t>(subpasses.size() - 1),
3132 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT,
3133 VK_PIPELINE_STAGE_FRAGMENT_SHADER_BIT,
3134 VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT,
3135 VK_ACCESS_INPUT_ATTACHMENT_READ_BIT,
3136 0};
3137 subpass_dependencies.push_back(dep);
3138 }
3139 renderpass_info.dependencyCount = subpass_dependencies.size();
3140 renderpass_info.pDependencies = subpass_dependencies.data();
3141 {
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003142 vk_testing::RenderPass rp;
3143 vk_testing::Framebuffer fb;
3144 rp.init(*m_device, renderpass_info);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003145
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003146 fbci.renderPass = rp.handle();
3147 fb.init(*m_device, fbci);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003148
3149 CreatePipelineHelper g_pipe_0(*this);
3150 g_pipe_0.InitInfo();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003151 g_pipe_0.gp_ci_.renderPass = rp.handle();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003152 g_pipe_0.InitState();
3153 ASSERT_VK_SUCCESS(g_pipe_0.CreateGraphicsPipeline());
3154
3155 CreatePipelineHelper g_pipe_12(*this);
3156 g_pipe_12.InitInfo();
3157 g_pipe_12.shader_stages_ = {vs.GetStageCreateInfo(), fs.GetStageCreateInfo()};
3158 g_pipe_12.dsl_bindings_ = {{0, VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT, 1, VK_SHADER_STAGE_FRAGMENT_BIT, nullptr}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003159 g_pipe_12.gp_ci_.renderPass = rp.handle();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003160 g_pipe_12.InitState();
3161 ASSERT_VK_SUCCESS(g_pipe_12.CreateGraphicsPipeline());
3162
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003163 g_pipe_12.descriptor_set_->WriteDescriptorImageInfo(0, attachments[0], sampler.handle(), VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003164 g_pipe_12.descriptor_set_->UpdateDescriptorSets();
3165
3166 m_errorMonitor->ExpectSuccess();
3167 m_commandBuffer->begin();
3168 vk::CmdPipelineBarrier(m_commandBuffer->handle(), VK_PIPELINE_STAGE_TOP_OF_PIPE_BIT,
3169 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT, 0, 0, nullptr, 0, nullptr, img_barriers.size(),
3170 img_barriers.data());
3171
3172 m_renderPassBeginInfo.renderArea = {{0, 0}, {16, 16}};
3173 m_renderPassBeginInfo.pClearValues = clear_values.data();
3174 m_renderPassBeginInfo.clearValueCount = clear_values.size();
3175
3176 m_renderPassBeginInfo.renderArea = {{0, 0}, {kWidth, kHeight}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003177 m_renderPassBeginInfo.renderPass = rp.handle();
3178 m_renderPassBeginInfo.framebuffer = fb.handle();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003179
3180 vk::CmdBeginRenderPass(m_commandBuffer->handle(), &m_renderPassBeginInfo, VK_SUBPASS_CONTENTS_INLINE);
3181 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_0.pipeline_);
3182 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_0.pipeline_layout_.handle(), 0,
3183 1, &g_pipe_0.descriptor_set_->set_, 0, NULL);
3184
3185 vk::CmdDraw(m_commandBuffer->handle(), 3, 1, 0, 0);
3186
3187 for (uint32_t i = 1; i < subpasses.size(); i++) {
3188 vk::CmdNextSubpass(m_commandBuffer->handle(), VK_SUBPASS_CONTENTS_INLINE);
3189 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_12.pipeline_);
3190 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS,
3191 g_pipe_12.pipeline_layout_.handle(), 0, 1, &g_pipe_12.descriptor_set_->set_, 0, NULL);
3192 vk::CmdDraw(m_commandBuffer->handle(), 3, 1, 0, 0);
3193 }
3194
3195 vk::CmdEndRenderPass(m_commandBuffer->handle());
3196
3197 m_commandBuffer->end();
3198
3199 m_errorMonitor->VerifyNotFound();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003200 }
3201}
John Zulauf025ee442020-12-15 11:44:19 -07003202
3203TEST_F(VkSyncValTest, SyncEventsBufferCopy) {
3204 TEST_DESCRIPTION("Check Set/Wait protection for a variety of use cases using buffer copies");
3205 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
3206 ASSERT_NO_FATAL_FAILURE(InitState(nullptr, nullptr, VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT));
3207
3208 VkBufferObj buffer_a;
3209 VkBufferObj buffer_b;
3210 VkBufferObj buffer_c;
3211 VkMemoryPropertyFlags mem_prop = VK_MEMORY_PROPERTY_DEVICE_LOCAL_BIT;
3212 buffer_a.init_as_src_and_dst(*m_device, 256, mem_prop);
3213 buffer_b.init_as_src_and_dst(*m_device, 256, mem_prop);
3214 buffer_c.init_as_src_and_dst(*m_device, 256, mem_prop);
3215
3216 VkBufferCopy region = {0, 0, 256};
3217 VkBufferCopy front2front = {0, 0, 128};
3218 VkBufferCopy front2back = {0, 128, 128};
3219 VkBufferCopy back2back = {128, 128, 128};
3220
3221 VkEventObj event;
3222 event.init(*m_device, VkEventObj::create_info(0));
3223 VkEvent event_handle = event.handle();
3224
3225 auto cb = m_commandBuffer->handle();
3226 m_commandBuffer->begin();
3227
3228 // Copy after set for WAR (note we are writing to the back half of c but only reading from the front
3229 m_errorMonitor->ExpectSuccess();
3230 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &region);
3231 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3232 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_c.handle(), 1, &back2back);
3233 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, nullptr, 0,
3234 nullptr, 0, nullptr);
3235 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_a.handle(), 1, &front2front);
3236 m_errorMonitor->VerifyNotFound();
3237 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
3238 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_a.handle(), 1, &front2back);
3239 m_errorMonitor->VerifyFound();
3240 m_commandBuffer->end();
3241
3242 // WAR prevented
3243 m_commandBuffer->reset();
3244 m_commandBuffer->begin();
3245 m_errorMonitor->ExpectSuccess();
3246 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &region);
3247 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3248 // Just protect against WAR, only need a sync barrier.
3249 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, nullptr, 0,
3250 nullptr, 0, nullptr);
3251 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_a.handle(), 1, &region);
3252 m_errorMonitor->VerifyNotFound();
3253
3254 // Wait shouldn't prevent this WAW though, as it's only a synchronization barrier
3255 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
3256 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_b.handle(), 1, &region);
3257 m_errorMonitor->VerifyFound();
3258 m_commandBuffer->end();
3259
3260 // Prevent WAR and WAW
3261 m_commandBuffer->reset();
3262 m_commandBuffer->begin();
3263 m_errorMonitor->ExpectSuccess();
3264 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &region);
3265 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
Mark Lobodzinski07d0a612020-12-30 15:42:31 -07003266 auto mem_barrier_waw = LvlInitStruct<VkMemoryBarrier>();
John Zulauf025ee442020-12-15 11:44:19 -07003267 mem_barrier_waw.srcAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3268 mem_barrier_waw.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3269 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 1,
3270 &mem_barrier_waw, 0, nullptr, 0, nullptr);
3271 // The WAW should be safe (on a memory barrier)
3272 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_b.handle(), 1, &region);
3273 // The WAR should also be safe (on a sync barrier)
3274 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_a.handle(), 1, &region);
3275 m_errorMonitor->VerifyNotFound();
3276 m_commandBuffer->end();
3277
3278 // Barrier range check for WAW
Mark Lobodzinski07d0a612020-12-30 15:42:31 -07003279 auto buffer_barrier_front_waw = LvlInitStruct<VkBufferMemoryBarrier>();
John Zulauf025ee442020-12-15 11:44:19 -07003280 buffer_barrier_front_waw.srcAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3281 buffer_barrier_front_waw.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3282 buffer_barrier_front_waw.buffer = buffer_b.handle();
3283 buffer_barrier_front_waw.offset = front2front.dstOffset;
3284 buffer_barrier_front_waw.size = front2front.size;
3285
3286 // Front safe, back WAW
3287 m_commandBuffer->reset();
3288 m_commandBuffer->begin();
3289 m_errorMonitor->ExpectSuccess();
3290 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &region);
3291 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3292 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, nullptr, 1,
3293 &buffer_barrier_front_waw, 0, nullptr);
3294 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &front2front);
3295 m_errorMonitor->VerifyNotFound();
3296 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
3297 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &back2back);
3298 m_errorMonitor->VerifyFound();
3299 m_commandBuffer->end();
3300}
3301
3302TEST_F(VkSyncValTest, SyncEventsCopyImageHazards) {
3303 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
3304 ASSERT_NO_FATAL_FAILURE(InitState(nullptr, nullptr, VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT));
3305
3306 VkImageUsageFlags usage = VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT;
3307 VkFormat format = VK_FORMAT_R8G8B8A8_UNORM;
3308 VkImageObj image_a(m_device);
3309 auto image_ci = VkImageObj::ImageCreateInfo2D(128, 128, 1, 2, format, usage, VK_IMAGE_TILING_OPTIMAL);
3310 image_a.Init(image_ci);
3311 ASSERT_TRUE(image_a.initialized());
3312
3313 VkImageObj image_b(m_device);
3314 image_b.Init(image_ci);
3315 ASSERT_TRUE(image_b.initialized());
3316
3317 VkImageObj image_c(m_device);
3318 image_c.Init(image_ci);
3319 ASSERT_TRUE(image_c.initialized());
3320
3321 VkEventObj event;
3322 event.init(*m_device, VkEventObj::create_info(0));
3323 VkEvent event_handle = event.handle();
3324
3325 VkImageSubresourceLayers layers_all{VK_IMAGE_ASPECT_COLOR_BIT, 0, 0, 2};
3326 VkImageSubresourceLayers layers_0{VK_IMAGE_ASPECT_COLOR_BIT, 0, 0, 1};
3327 VkImageSubresourceLayers layers_1{VK_IMAGE_ASPECT_COLOR_BIT, 0, 1, 1};
3328 VkImageSubresourceRange layers_0_subresource_range{VK_IMAGE_ASPECT_COLOR_BIT, 0, 1, 0, 1};
3329 VkOffset3D zero_offset{0, 0, 0};
3330 VkOffset3D half_offset{64, 64, 0};
3331 VkExtent3D full_extent{128, 128, 1}; // <-- image type is 2D
3332 VkExtent3D half_extent{64, 64, 1}; // <-- image type is 2D
3333
3334 VkImageCopy full_region = {layers_all, zero_offset, layers_all, zero_offset, full_extent};
3335 VkImageCopy region_0_to_0 = {layers_0, zero_offset, layers_0, zero_offset, full_extent};
3336 VkImageCopy region_1_to_1 = {layers_1, zero_offset, layers_1, zero_offset, full_extent};
3337 VkImageCopy region_0_q0toq0 = {layers_0, zero_offset, layers_0, zero_offset, half_extent};
3338 VkImageCopy region_0_q0toq3 = {layers_0, zero_offset, layers_0, half_offset, half_extent};
3339 VkImageCopy region_0_q3toq3 = {layers_0, half_offset, layers_0, half_offset, half_extent};
3340
3341 auto cb = m_commandBuffer->handle();
3342 auto copy_general = [cb](const VkImageObj &from, const VkImageObj &to, const VkImageCopy &region) {
3343 vk::CmdCopyImage(cb, from.handle(), VK_IMAGE_LAYOUT_GENERAL, to.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region);
3344 };
3345
3346 auto set_layouts = [this, &image_a, &image_b, &image_c]() {
3347 image_c.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
3348 image_b.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
3349 image_a.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
3350 };
3351
John Zulaufdd462092020-12-18 12:00:35 -07003352 // Scope check. One access in, one access not
John Zulauf025ee442020-12-15 11:44:19 -07003353 m_commandBuffer->begin();
3354 set_layouts();
3355 m_errorMonitor->ExpectSuccess();
3356 copy_general(image_a, image_b, full_region);
3357 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3358 copy_general(image_a, image_c, region_0_q3toq3);
3359 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, nullptr, 0,
3360 nullptr, 0, nullptr);
3361 copy_general(image_c, image_a, region_0_q0toq0);
3362 m_errorMonitor->VerifyNotFound();
3363 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
3364 copy_general(image_c, image_a, region_0_q0toq3);
3365 m_errorMonitor->VerifyFound();
3366 m_commandBuffer->end();
3367
3368 // WAR prevented
3369 m_commandBuffer->reset();
3370 m_commandBuffer->begin();
3371 set_layouts();
3372 m_errorMonitor->ExpectSuccess();
3373 copy_general(image_a, image_b, full_region);
3374 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3375 // Just protect against WAR, only need a sync barrier.
3376 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, nullptr, 0,
3377 nullptr, 0, nullptr);
3378 copy_general(image_c, image_a, full_region);
3379 m_errorMonitor->VerifyNotFound();
3380
3381 // Wait shouldn't prevent this WAW though, as it's only a synchronization barrier
3382 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
3383 copy_general(image_c, image_b, full_region);
3384 m_errorMonitor->VerifyFound();
3385 m_commandBuffer->end();
3386
3387 // Prevent WAR and WAW
3388 m_commandBuffer->reset();
3389 m_commandBuffer->begin();
3390 m_errorMonitor->ExpectSuccess();
3391 set_layouts();
3392 copy_general(image_a, image_b, full_region);
3393 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
Mark Lobodzinski07d0a612020-12-30 15:42:31 -07003394 auto mem_barrier_waw = LvlInitStruct<VkMemoryBarrier>();
John Zulauf025ee442020-12-15 11:44:19 -07003395 mem_barrier_waw.srcAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3396 mem_barrier_waw.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3397 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 1,
3398 &mem_barrier_waw, 0, nullptr, 0, nullptr);
3399 // The WAW should be safe (on a memory barrier)
3400 copy_general(image_c, image_b, full_region);
3401 // The WAR should also be safe (on a sync barrier)
3402 copy_general(image_c, image_a, full_region);
3403 m_errorMonitor->VerifyNotFound();
3404 m_commandBuffer->end();
3405
3406 // Barrier range check for WAW
Mark Lobodzinski07d0a612020-12-30 15:42:31 -07003407 auto image_barrier_region0_waw = LvlInitStruct<VkImageMemoryBarrier>();
John Zulauf025ee442020-12-15 11:44:19 -07003408 image_barrier_region0_waw.srcAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3409 image_barrier_region0_waw.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3410 image_barrier_region0_waw.oldLayout = VK_IMAGE_LAYOUT_GENERAL;
3411 image_barrier_region0_waw.newLayout = VK_IMAGE_LAYOUT_GENERAL;
3412 image_barrier_region0_waw.srcQueueFamilyIndex = VK_QUEUE_FAMILY_IGNORED;
3413 image_barrier_region0_waw.dstQueueFamilyIndex = VK_QUEUE_FAMILY_IGNORED;
3414 image_barrier_region0_waw.image = image_b.handle();
3415 image_barrier_region0_waw.subresourceRange = layers_0_subresource_range;
3416
3417 // Region 0 safe, back WAW
3418 m_commandBuffer->reset();
3419 m_commandBuffer->begin();
3420 set_layouts();
3421 m_errorMonitor->ExpectSuccess();
3422 copy_general(image_a, image_b, full_region);
3423 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3424 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, nullptr, 0,
3425 nullptr, 1, &image_barrier_region0_waw);
3426 copy_general(image_a, image_b, region_0_to_0);
3427 m_errorMonitor->VerifyNotFound();
3428 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
3429 copy_general(image_a, image_b, region_1_to_1);
3430 m_errorMonitor->VerifyFound();
3431 m_commandBuffer->end();
3432}
John Zulauf4b5e4632020-12-15 11:48:59 -07003433
3434TEST_F(VkSyncValTest, SyncEventsCommandHazards) {
3435 TEST_DESCRIPTION("Check Set/Reset/Wait command hazard checking");
3436 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
3437 ASSERT_NO_FATAL_FAILURE(InitState(nullptr, nullptr, VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT));
3438
3439 VkEventObj event;
3440 event.init(*m_device, VkEventObj::create_info(0));
3441
3442 const VkEvent event_handle = event.handle();
3443
3444 m_commandBuffer->begin();
3445 m_errorMonitor->ExpectSuccess();
3446 m_commandBuffer->ResetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3447 m_errorMonitor->VerifyNotFound();
3448
John Zulauf4edde622021-02-15 08:54:50 -07003449 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "VUID-vkCmdResetEvent-event-03834");
John Zulauf4b5e4632020-12-15 11:48:59 -07003450 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, nullptr, 0,
3451 nullptr, 0, nullptr);
3452 m_errorMonitor->VerifyFound();
3453 m_errorMonitor->ExpectSuccess();
3454 m_commandBuffer->end();
3455
3456 m_commandBuffer->begin();
3457 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3458 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_FRAGMENT_SHADER_BIT, 0, nullptr,
3459 0, nullptr, 0, nullptr);
3460 m_errorMonitor->VerifyNotFound();
3461 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-vkCmdResetEvent-missingbarrier-wait");
3462 m_commandBuffer->ResetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3463 m_errorMonitor->VerifyFound();
3464 m_errorMonitor->ExpectSuccess();
3465 m_commandBuffer->end();
3466
3467 m_commandBuffer->begin();
3468 m_commandBuffer->ResetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3469 m_errorMonitor->VerifyNotFound();
3470 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-vkCmdSetEvent-missingbarrier-reset");
3471 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3472 m_errorMonitor->VerifyFound();
3473
3474 m_errorMonitor->ExpectSuccess();
3475 m_commandBuffer->PipelineBarrier(VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, 0U, 0, nullptr, 0,
3476 nullptr, 0, nullptr);
3477 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3478 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, nullptr, 0,
3479 nullptr, 0, nullptr);
3480 m_commandBuffer->ResetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3481 m_commandBuffer->PipelineBarrier(VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, 0U, 0, nullptr, 0,
3482 nullptr, 0, nullptr);
3483 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3484 m_errorMonitor->VerifyNotFound();
3485
3486 // Need a barrier between set and a reset
3487 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-vkCmdResetEvent-missingbarrier-set");
3488 m_commandBuffer->ResetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3489 m_errorMonitor->VerifyFound();
3490 m_errorMonitor->ExpectSuccess();
3491 m_commandBuffer->end();
3492
3493 m_commandBuffer->begin();
3494 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3495 m_errorMonitor->VerifyNotFound();
3496 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-vkCmdSetEvent-missingbarrier-set");
3497 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3498 m_errorMonitor->VerifyFound();
3499
3500 m_commandBuffer->end();
John Zulaufb0b6e9b2021-08-20 09:22:45 -06003501
3502 // Secondary command buffer events tests
3503 const auto cb = m_commandBuffer->handle();
3504 VkBufferObj buffer_a;
3505 VkBufferObj buffer_b;
3506 VkMemoryPropertyFlags mem_prop = VK_MEMORY_PROPERTY_DEVICE_LOCAL_BIT;
3507 buffer_a.init_as_src_and_dst(*m_device, 256, mem_prop);
3508 buffer_b.init_as_src_and_dst(*m_device, 256, mem_prop);
3509
3510 VkBufferCopy front2front = {0, 0, 128};
3511
3512 // Barrier range check for WAW
3513 auto buffer_barrier_front_waw = LvlInitStruct<VkBufferMemoryBarrier>();
3514 buffer_barrier_front_waw.srcAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3515 buffer_barrier_front_waw.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3516 buffer_barrier_front_waw.buffer = buffer_b.handle();
3517 buffer_barrier_front_waw.offset = front2front.dstOffset;
3518 buffer_barrier_front_waw.size = front2front.size;
3519
3520 m_errorMonitor->ExpectSuccess();
3521 VkCommandBufferObj secondary_cb1(m_device, m_commandPool, VK_COMMAND_BUFFER_LEVEL_SECONDARY);
3522 VkCommandBuffer scb1 = secondary_cb1.handle();
3523 secondary_cb1.begin();
3524 secondary_cb1.WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, nullptr, 1,
3525 &buffer_barrier_front_waw, 0, nullptr);
3526 vk::CmdCopyBuffer(scb1, buffer_a.handle(), buffer_b.handle(), 1, &front2front);
3527 secondary_cb1.end();
3528 m_errorMonitor->VerifyNotFound();
3529
3530 // One secondary cb hazarding with primary
3531 m_errorMonitor->ExpectSuccess();
3532 m_commandBuffer->reset();
3533 m_commandBuffer->begin();
3534 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &front2front);
3535 m_errorMonitor->VerifyNotFound();
3536 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
3537 vk::CmdExecuteCommands(cb, 1, &scb1);
3538 m_errorMonitor->VerifyFound();
3539 m_commandBuffer->end();
3540
3541 // One secondary cb sharing event with primary
3542 m_errorMonitor->ExpectSuccess();
3543 m_commandBuffer->reset();
3544 m_commandBuffer->begin();
3545 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &front2front);
3546 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3547 vk::CmdExecuteCommands(cb, 1, &scb1);
3548 m_commandBuffer->end();
3549 m_errorMonitor->VerifyNotFound();
John Zulauf4b5e4632020-12-15 11:48:59 -07003550}
ziga-lunarg3a16ff12021-07-30 12:09:55 +02003551
3552TEST_F(VkLayerTest, CmdWaitEvents2KHRUsedButSynchronizaion2Disabled) {
3553 TEST_DESCRIPTION("Using CmdWaitEvents2KHR when synchronization2 is not enabled");
Tony-LunarG53b72e52021-11-19 10:04:40 -07003554 SetTargetApiVersion(VK_API_VERSION_1_3);
sfricke-samsung6fc3e322022-02-15 22:41:29 -08003555
Tony-LunarGdf960d42022-01-27 16:13:34 -07003556 AddRequiredExtensions(VK_KHR_GET_PHYSICAL_DEVICE_PROPERTIES_2_EXTENSION_NAME);
3557 AddRequiredExtensions(VK_KHR_SYNCHRONIZATION_2_EXTENSION_NAME);
ziga-lunarg3a16ff12021-07-30 12:09:55 +02003558 ASSERT_NO_FATAL_FAILURE(InitFramework());
sjfricked700bc02022-05-30 16:35:06 +09003559 if (!AreRequiredExtensionsEnabled()) {
3560 GTEST_SKIP() << RequiredExtensionsNotSupported() << " not supported";
ziga-lunarg3a16ff12021-07-30 12:09:55 +02003561 }
ziga-lunarg3a16ff12021-07-30 12:09:55 +02003562 InitState();
3563
Tony-LunarG53b72e52021-11-19 10:04:40 -07003564 bool vulkan_13 = (DeviceValidationVersion() >= VK_API_VERSION_1_3);
ziga-lunarg3a16ff12021-07-30 12:09:55 +02003565 auto fpCmdWaitEvents2KHR = (PFN_vkCmdWaitEvents2KHR)vk::GetDeviceProcAddr(m_device->device(), "vkCmdWaitEvents2KHR");
3566
3567 VkEventObj event;
3568 event.init(*m_device, VkEventObj::create_info(0));
3569 VkEvent event_handle = event.handle();
3570
3571 VkDependencyInfoKHR dependency_info = LvlInitStruct<VkDependencyInfoKHR>();
3572
3573 m_commandBuffer->begin();
Tony-LunarG279601c2021-11-16 10:50:51 -07003574 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkCmdWaitEvents2-synchronization2-03836");
ziga-lunarg3a16ff12021-07-30 12:09:55 +02003575 fpCmdWaitEvents2KHR(m_commandBuffer->handle(), 1, &event_handle, &dependency_info);
3576 m_errorMonitor->VerifyFound();
Tony-LunarG53b72e52021-11-19 10:04:40 -07003577 if (vulkan_13) {
3578 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkCmdWaitEvents2-synchronization2-03836");
3579 vk::CmdWaitEvents2(m_commandBuffer->handle(), 1, &event_handle, &dependency_info);
3580 m_errorMonitor->VerifyFound();
3581 }
ziga-lunarg3a16ff12021-07-30 12:09:55 +02003582 m_commandBuffer->end();
3583}
ziga-lunarg15f450d2021-08-26 23:10:05 +02003584
3585TEST_F(VkLayerTest, Sync2FeatureDisabled) {
3586 TEST_DESCRIPTION("Call sync2 functions when the feature is disabled");
3587
Tony-LunarG53b72e52021-11-19 10:04:40 -07003588 SetTargetApiVersion(VK_API_VERSION_1_3);
Tony-LunarGdf960d42022-01-27 16:13:34 -07003589 AddRequiredExtensions(VK_KHR_GET_PHYSICAL_DEVICE_PROPERTIES_2_EXTENSION_NAME);
3590 AddRequiredExtensions(VK_KHR_SYNCHRONIZATION_2_EXTENSION_NAME);
ziga-lunarg15f450d2021-08-26 23:10:05 +02003591 ASSERT_NO_FATAL_FAILURE(InitFramework());
sjfricked700bc02022-05-30 16:35:06 +09003592 if (!AreRequiredExtensionsEnabled()) {
3593 GTEST_SKIP() << RequiredExtensionsNotSupported() << " not supported";
ziga-lunarg15f450d2021-08-26 23:10:05 +02003594 }
3595
3596 ASSERT_NO_FATAL_FAILURE(InitState());
3597
Tony-LunarG53b72e52021-11-19 10:04:40 -07003598 bool vulkan_13 = (DeviceValidationVersion() >= VK_API_VERSION_1_3);
ziga-lunarg15f450d2021-08-26 23:10:05 +02003599 VkPhysicalDeviceSynchronization2FeaturesKHR synchronization2 = LvlInitStruct<VkPhysicalDeviceSynchronization2FeaturesKHR>();
3600 synchronization2.synchronization2 = VK_FALSE; // Invalid
3601 auto features2 = LvlInitStruct<VkPhysicalDeviceFeatures2KHR>(&synchronization2);
3602 vk::GetPhysicalDeviceFeatures2(gpu(), &features2);
3603
3604 auto vkCmdPipelineBarrier2KHR =
3605 (PFN_vkCmdPipelineBarrier2KHR)vk::GetDeviceProcAddr(m_device->device(), "vkCmdPipelineBarrier2KHR");
3606 auto vkCmdResetEvent2KHR = (PFN_vkCmdResetEvent2KHR)vk::GetDeviceProcAddr(m_device->device(), "vkCmdResetEvent2KHR");
3607 auto vkCmdSetEvent2KHR = (PFN_vkCmdSetEvent2KHR)vk::GetDeviceProcAddr(m_device->device(), "vkCmdSetEvent2KHR");
3608 auto vkCmdWriteTimestamp2KHR =
3609 (PFN_vkCmdWriteTimestamp2KHR)vk::GetDeviceProcAddr(m_device->device(), "vkCmdWriteTimestamp2KHR");
3610
3611 bool timestamp = false;
3612
3613 uint32_t queue_count;
3614 vk::GetPhysicalDeviceQueueFamilyProperties(gpu(), &queue_count, NULL);
3615 std::vector<VkQueueFamilyProperties> queue_props(queue_count);
3616 vk::GetPhysicalDeviceQueueFamilyProperties(gpu(), &queue_count, queue_props.data());
3617 if (queue_props[m_device->graphics_queue_node_index_].timestampValidBits > 0) {
3618 timestamp = true;
3619 }
3620
3621 m_commandBuffer->begin();
3622
3623 VkDependencyInfoKHR dependency_info = LvlInitStruct<VkDependencyInfoKHR>();
3624
Tony-LunarG279601c2021-11-16 10:50:51 -07003625 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkCmdPipelineBarrier2-synchronization2-03848");
ziga-lunarg15f450d2021-08-26 23:10:05 +02003626 vkCmdPipelineBarrier2KHR(m_commandBuffer->handle(), &dependency_info);
3627 m_errorMonitor->VerifyFound();
3628
3629 VkEventCreateInfo eci = LvlInitStruct<VkEventCreateInfo>();
3630 vk_testing::Event event;
3631 event.init(*m_device, eci);
3632
3633 VkPipelineStageFlagBits2KHR stage = VK_PIPELINE_STAGE_2_TOP_OF_PIPE_BIT_KHR;
3634
Tony-LunarG279601c2021-11-16 10:50:51 -07003635 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkCmdResetEvent2-synchronization2-03829");
ziga-lunarg15f450d2021-08-26 23:10:05 +02003636 vkCmdResetEvent2KHR(m_commandBuffer->handle(), event.handle(), stage);
3637 m_errorMonitor->VerifyFound();
3638
Tony-LunarG279601c2021-11-16 10:50:51 -07003639 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkCmdSetEvent2-synchronization2-03824");
ziga-lunarg15f450d2021-08-26 23:10:05 +02003640 vkCmdSetEvent2KHR(m_commandBuffer->handle(), event.handle(), &dependency_info);
3641 m_errorMonitor->VerifyFound();
3642
3643 if (timestamp) {
3644 VkQueryPoolCreateInfo qpci = LvlInitStruct<VkQueryPoolCreateInfo>();
3645 qpci.queryType = VK_QUERY_TYPE_TIMESTAMP;
3646 qpci.queryCount = 1;
3647
3648 vk_testing::QueryPool query_pool;
3649 query_pool.init(*m_device, qpci);
3650
Tony-LunarG279601c2021-11-16 10:50:51 -07003651 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkCmdWriteTimestamp2-synchronization2-03858");
ziga-lunarg15f450d2021-08-26 23:10:05 +02003652 vkCmdWriteTimestamp2KHR(m_commandBuffer->handle(), stage, query_pool.handle(), 0);
3653 m_errorMonitor->VerifyFound();
Tony-LunarG53b72e52021-11-19 10:04:40 -07003654 if (vulkan_13) {
3655 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkCmdWriteTimestamp2-synchronization2-03858");
3656 vk::CmdWriteTimestamp2(m_commandBuffer->handle(), stage, query_pool.handle(), 0);
3657 m_errorMonitor->VerifyFound();
3658 }
3659 }
3660 if (vulkan_13) {
3661 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkCmdPipelineBarrier2-synchronization2-03848");
3662 vk::CmdPipelineBarrier2(m_commandBuffer->handle(), &dependency_info);
3663 m_errorMonitor->VerifyFound();
3664
3665 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkCmdResetEvent2-synchronization2-03829");
3666 vk::CmdResetEvent2(m_commandBuffer->handle(), event.handle(), stage);
3667 m_errorMonitor->VerifyFound();
3668
3669 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkCmdSetEvent2-synchronization2-03824");
3670 vk::CmdSetEvent2(m_commandBuffer->handle(), event.handle(), &dependency_info);
3671 m_errorMonitor->VerifyFound();
ziga-lunarg15f450d2021-08-26 23:10:05 +02003672 }
3673
3674 m_commandBuffer->end();
3675}
Jeremy Gebben9b6f0532022-02-02 11:10:31 -07003676
3677TEST_F(VkSyncValTest, DestroyedUnusedDescriptors) {
3678 TEST_DESCRIPTION("Verify unused descriptors are ignored and don't crash syncval if they've been destroyed.");
3679 SetTargetApiVersion(VK_API_VERSION_1_1);
sjfricked700bc02022-05-30 16:35:06 +09003680 AddRequiredExtensions(VK_KHR_GET_PHYSICAL_DEVICE_PROPERTIES_2_EXTENSION_NAME);
Jeremy Gebben9b6f0532022-02-02 11:10:31 -07003681 AddRequiredExtensions(VK_KHR_MAINTENANCE_3_EXTENSION_NAME);
3682 AddRequiredExtensions(VK_EXT_DESCRIPTOR_INDEXING_EXTENSION_NAME);
3683
3684 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
3685
sjfricked700bc02022-05-30 16:35:06 +09003686 if (!AreRequiredExtensionsEnabled()) {
3687 GTEST_SKIP() << RequiredExtensionsNotSupported() << " not supported";
Jeremy Gebben9b6f0532022-02-02 11:10:31 -07003688 }
3689
3690 auto indexing_features = LvlInitStruct<VkPhysicalDeviceDescriptorIndexingFeaturesEXT>();
3691 auto features2 = LvlInitStruct<VkPhysicalDeviceFeatures2KHR>();
3692 features2.pNext = &indexing_features;
3693
3694 auto vkGetPhysicalDeviceFeatures2KHR = reinterpret_cast<PFN_vkGetPhysicalDeviceFeatures2KHR>(
3695 vk::GetInstanceProcAddr(instance(), "vkGetPhysicalDeviceFeatures2KHR"));
3696 ASSERT_TRUE(vkGetPhysicalDeviceFeatures2KHR != nullptr);
3697
3698 vkGetPhysicalDeviceFeatures2KHR(gpu(), &features2);
3699 ASSERT_NO_FATAL_FAILURE(InitState(nullptr, &features2));
3700 if (!indexing_features.descriptorBindingPartiallyBound) {
3701 printf("%s Partially bound bindings not supported, skipping test\n", kSkipPrefix);
3702 return;
3703 }
3704 if (!indexing_features.descriptorBindingUpdateUnusedWhilePending) {
3705 printf("%s Updating unused while pending is not supported, skipping test\n", kSkipPrefix);
3706 return;
3707 }
3708
3709 ASSERT_NO_FATAL_FAILURE(InitViewport());
3710 ASSERT_NO_FATAL_FAILURE(InitRenderTarget());
3711 m_errorMonitor->ExpectSuccess();
3712
sfricke-samsung6fc3e322022-02-15 22:41:29 -08003713 VkDescriptorSetLayoutBindingFlagsCreateInfoEXT layout_createinfo_binding_flags =
3714 LvlInitStruct<VkDescriptorSetLayoutBindingFlagsCreateInfoEXT>();
Jeremy Gebben9b6f0532022-02-02 11:10:31 -07003715 constexpr size_t kNumDescriptors = 6;
3716
3717 std::array<VkDescriptorBindingFlagsEXT, kNumDescriptors> ds_binding_flags;
3718 for (auto &elem : ds_binding_flags) {
3719 elem = VK_DESCRIPTOR_BINDING_PARTIALLY_BOUND_BIT_EXT | VK_DESCRIPTOR_BINDING_UPDATE_UNUSED_WHILE_PENDING_BIT_EXT;
3720 }
3721
Jeremy Gebben9b6f0532022-02-02 11:10:31 -07003722 layout_createinfo_binding_flags.bindingCount = ds_binding_flags.size();
3723 layout_createinfo_binding_flags.pBindingFlags = ds_binding_flags.data();
3724
3725 // Prepare descriptors
3726 OneOffDescriptorSet descriptor_set(m_device,
3727 {
3728 {0, VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER, 1, VK_SHADER_STAGE_ALL, nullptr},
3729 {1, VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER, 1, VK_SHADER_STAGE_ALL, nullptr},
3730 {2, VK_DESCRIPTOR_TYPE_UNIFORM_TEXEL_BUFFER, 1, VK_SHADER_STAGE_ALL, nullptr},
3731 {3, VK_DESCRIPTOR_TYPE_SAMPLER, 1, VK_SHADER_STAGE_ALL, nullptr},
3732 {4, VK_DESCRIPTOR_TYPE_SAMPLED_IMAGE, 1, VK_SHADER_STAGE_ALL, nullptr},
3733 {5, VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER, 1, VK_SHADER_STAGE_ALL, nullptr},
3734 },
3735 0, &layout_createinfo_binding_flags, 0);
3736 const VkPipelineLayoutObj pipeline_layout(m_device, {&descriptor_set.layout_});
3737 uint32_t qfi = 0;
3738 auto buffer_create_info = LvlInitStruct<VkBufferCreateInfo>();
3739 buffer_create_info.size = 32;
3740 buffer_create_info.usage = VK_BUFFER_USAGE_UNIFORM_BUFFER_BIT;
3741 buffer_create_info.queueFamilyIndexCount = 1;
3742 buffer_create_info.pQueueFamilyIndices = &qfi;
3743
3744 VkBufferObj doit_buffer;
3745 doit_buffer.init(*m_device, buffer_create_info);
3746
3747 auto buffer = layer_data::make_unique<VkBufferObj>();
3748 buffer->init(*m_device, buffer_create_info);
3749
3750 VkDescriptorBufferInfo buffer_info[2] = {};
3751 buffer_info[0].buffer = doit_buffer.handle();
3752 buffer_info[0].offset = 0;
3753 buffer_info[0].range = sizeof(uint32_t);
3754 buffer_info[1].buffer = buffer->handle();
3755 buffer_info[1].offset = 0;
3756 buffer_info[1].range = sizeof(uint32_t);
3757
3758 VkBufferObj texel_buffer;
3759 buffer_create_info.usage = VK_BUFFER_USAGE_UNIFORM_TEXEL_BUFFER_BIT;
3760 texel_buffer.init(*m_device, buffer_create_info);
3761
3762 auto bvci = LvlInitStruct<VkBufferViewCreateInfo>();
3763 bvci.buffer = texel_buffer.handle();
3764 bvci.format = VK_FORMAT_R32_SFLOAT;
3765 bvci.offset = 0;
3766 bvci.range = VK_WHOLE_SIZE;
3767
3768 auto texel_bufferview = layer_data::make_unique<vk_testing::BufferView>();
3769 texel_bufferview->init(*m_device, bvci);
3770
3771 auto index_buffer_create_info = LvlInitStruct<VkBufferCreateInfo>();
3772 index_buffer_create_info.size = sizeof(uint32_t);
3773 index_buffer_create_info.usage = VK_BUFFER_USAGE_INDEX_BUFFER_BIT;
3774 VkBufferObj index_buffer;
3775 index_buffer.init(*m_device, index_buffer_create_info);
3776
3777 VkFormat format = VK_FORMAT_R8G8B8A8_UNORM;
3778 VkImageObj sampled_image(m_device);
3779 auto image_ci = VkImageObj::ImageCreateInfo2D(128, 128, 1, 1, format, VK_IMAGE_USAGE_SAMPLED_BIT, VK_IMAGE_TILING_OPTIMAL);
3780 sampled_image.Init(image_ci);
3781 auto sampled_view = layer_data::make_unique<vk_testing::ImageView>();
3782 auto imageview_ci = SafeSaneImageViewCreateInfo(sampled_image, format, VK_IMAGE_ASPECT_COLOR_BIT);
3783 sampled_view->init(*m_device, imageview_ci);
3784
3785 VkImageObj combined_image(m_device);
3786 image_ci = VkImageObj::ImageCreateInfo2D(128, 128, 1, 1, format, VK_IMAGE_USAGE_SAMPLED_BIT, VK_IMAGE_TILING_OPTIMAL);
3787 combined_image.Init(image_ci);
3788 imageview_ci = SafeSaneImageViewCreateInfo(combined_image, format, VK_IMAGE_ASPECT_COLOR_BIT);
3789 auto combined_view = layer_data::make_unique<vk_testing::ImageView>();
3790 combined_view->init(*m_device, imageview_ci);
3791
3792 vk_testing::Sampler sampler;
3793 VkSamplerCreateInfo sampler_ci = SafeSaneSamplerCreateInfo();
3794 sampler.init(*m_device, sampler_ci);
3795
3796 VkDescriptorImageInfo image_info[3] = {};
3797 image_info[0].sampler = sampler.handle();
3798 image_info[0].imageView = VK_NULL_HANDLE;
3799 image_info[0].imageLayout = VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL;
3800 image_info[1].sampler = VK_NULL_HANDLE;
3801 image_info[1].imageView = sampled_view->handle();
3802 image_info[1].imageLayout = VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL;
3803 image_info[2].sampler = sampler.handle();
3804 image_info[2].imageView = combined_view->handle();
3805 image_info[2].imageLayout = VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL;
3806
3807 // Update all descriptors
3808 std::array<VkWriteDescriptorSet, kNumDescriptors> descriptor_writes;
3809 descriptor_writes[0] = LvlInitStruct<VkWriteDescriptorSet>();
3810 descriptor_writes[0].dstSet = descriptor_set.set_;
3811 descriptor_writes[0].dstBinding = 0;
3812 descriptor_writes[0].descriptorCount = 1;
3813 descriptor_writes[0].descriptorType = VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER;
3814 descriptor_writes[0].pBufferInfo = &buffer_info[0];
3815
3816 descriptor_writes[1] = LvlInitStruct<VkWriteDescriptorSet>();
3817 descriptor_writes[1].dstSet = descriptor_set.set_;
3818 descriptor_writes[1].dstBinding = 1;
3819 descriptor_writes[1].descriptorCount = 1;
3820 descriptor_writes[1].descriptorType = VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER;
3821 descriptor_writes[1].pBufferInfo = &buffer_info[1];
3822
3823 descriptor_writes[2] = LvlInitStruct<VkWriteDescriptorSet>();
3824 descriptor_writes[2].dstSet = descriptor_set.set_;
3825 descriptor_writes[2].dstBinding = 2;
3826 descriptor_writes[2].descriptorCount = 1;
3827 descriptor_writes[2].descriptorType = VK_DESCRIPTOR_TYPE_UNIFORM_TEXEL_BUFFER;
3828 descriptor_writes[2].pTexelBufferView = &texel_bufferview->handle();
3829
3830 descriptor_writes[3] = LvlInitStruct<VkWriteDescriptorSet>();
3831 descriptor_writes[3].dstSet = descriptor_set.set_;
3832 descriptor_writes[3].dstBinding = 3;
3833 descriptor_writes[3].descriptorCount = 1;
3834 descriptor_writes[3].descriptorType = VK_DESCRIPTOR_TYPE_SAMPLER;
3835 descriptor_writes[3].pImageInfo = &image_info[0];
3836
3837 descriptor_writes[4] = LvlInitStruct<VkWriteDescriptorSet>();
3838 descriptor_writes[4].dstSet = descriptor_set.set_;
3839 descriptor_writes[4].dstBinding = 4;
3840 descriptor_writes[4].descriptorCount = 1;
3841 descriptor_writes[4].descriptorType = VK_DESCRIPTOR_TYPE_SAMPLED_IMAGE;
3842 descriptor_writes[4].pImageInfo = &image_info[1];
3843
3844 descriptor_writes[5] = LvlInitStruct<VkWriteDescriptorSet>();
3845 descriptor_writes[5].dstSet = descriptor_set.set_;
3846 descriptor_writes[5].dstBinding = 5;
3847 descriptor_writes[5].descriptorCount = 1;
3848 descriptor_writes[5].descriptorType = VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER;
3849 descriptor_writes[5].pImageInfo = &image_info[2];
3850
3851 vk::UpdateDescriptorSets(m_device->device(), descriptor_writes.size(), descriptor_writes.data(), 0, NULL);
3852
3853 // only descriptor 0 is used, the rest are going to get destroyed
3854 char const *shader_source = R"glsl(
3855 #version 450
3856 layout(set = 0, binding = 0) uniform foo_0 { int val; } doit;
3857 layout(set = 0, binding = 1) uniform foo_1 { int val; } readit;
3858 layout(set = 0, binding = 2) uniform samplerBuffer texels;
3859 layout(set = 0, binding = 3) uniform sampler samp;
3860 layout(set = 0, binding = 4) uniform texture2D img;
3861 layout(set = 0, binding = 5) uniform sampler2D sampled_image;
3862
3863 void main() {
3864 vec4 x;
3865 vec4 y;
3866 vec4 z;
3867 if (doit.val == 0) {
3868 gl_Position = vec4(0.0);
3869 x = vec4(0.0);
3870 y = vec4(0.0);
3871 z = vec4(0.0);
3872 } else {
3873 gl_Position = vec4(readit.val);
3874 x = texelFetch(texels, 5);
3875 y = texture(sampler2D(img, samp), vec2(0));
3876 z = texture(sampled_image, vec2(0));
3877 }
3878 }
3879 )glsl";
3880
3881 VkShaderObj vs(this, shader_source, VK_SHADER_STAGE_VERTEX_BIT);
3882 VkPipelineObj pipe(m_device);
3883 pipe.AddShader(&vs);
3884 pipe.AddDefaultColorAttachment();
3885 pipe.CreateVKPipeline(pipeline_layout.handle(), m_renderPass);
sfricke-samsung6fc3e322022-02-15 22:41:29 -08003886 VkCommandBufferBeginInfo begin_info = LvlInitStruct<VkCommandBufferBeginInfo>();
Jeremy Gebben9b6f0532022-02-02 11:10:31 -07003887 m_commandBuffer->begin(&begin_info);
3888 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, pipe.handle());
3889 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
3890 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, pipeline_layout.handle(), 0, 1,
3891 &descriptor_set.set_, 0, nullptr);
3892
3893 // destroy resources for the unused descriptors
3894 buffer.reset();
3895 texel_bufferview.reset();
3896 sampled_view.reset();
3897 combined_view.reset();
3898
3899 vk::CmdBindIndexBuffer(m_commandBuffer->handle(), index_buffer.handle(), 0, VK_INDEX_TYPE_UINT32);
3900 VkViewport viewport = {0, 0, 16, 16, 0, 1};
3901 vk::CmdSetViewport(m_commandBuffer->handle(), 0, 1, &viewport);
3902 VkRect2D scissor = {{0, 0}, {16, 16}};
3903 vk::CmdSetScissor(m_commandBuffer->handle(), 0, 1, &scissor);
3904 vk::CmdDrawIndexed(m_commandBuffer->handle(), 1, 1, 0, 0, 0);
3905 vk::CmdEndRenderPass(m_commandBuffer->handle());
3906 m_commandBuffer->end();
3907 m_commandBuffer->QueueCommandBuffer();
3908 vk::QueueWaitIdle(m_device->m_queue);
3909 m_errorMonitor->VerifyNotFound();
3910}
ziga-lunargc71f1a92022-03-23 23:08:35 +01003911
ziga-lunarg26ba4b92022-03-24 16:43:03 +01003912TEST_F(VkSyncValTest, TestInvalidExternalSubpassDependency) {
3913 TEST_DESCRIPTION("Test write after write hazard with invalid external subpass dependency");
3914
3915 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
3916 ASSERT_NO_FATAL_FAILURE(InitState());
3917
3918 VkSubpassDependency subpass_dependency = {};
3919 subpass_dependency.srcSubpass = 0;
3920 subpass_dependency.dstSubpass = VK_SUBPASS_EXTERNAL;
3921 subpass_dependency.srcStageMask = 0;
3922 subpass_dependency.dstStageMask = 0;
3923 subpass_dependency.srcAccessMask = 0;
3924 subpass_dependency.dstAccessMask = 0;
3925 subpass_dependency.dependencyFlags = 0;
3926
3927 VkAttachmentReference attach_ref1 = {};
3928 attach_ref1.attachment = 0;
3929 attach_ref1.layout = VK_IMAGE_LAYOUT_DEPTH_ATTACHMENT_OPTIMAL;
3930 VkAttachmentReference attach_ref2 = {};
3931 attach_ref2.attachment = 0;
3932 attach_ref2.layout = VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL;
3933
3934 VkSubpassDescription subpass_descriptions[2] = {};
3935 subpass_descriptions[0].pipelineBindPoint = VK_PIPELINE_BIND_POINT_GRAPHICS;
3936 subpass_descriptions[0].pDepthStencilAttachment = &attach_ref1;
3937 subpass_descriptions[1].pipelineBindPoint = VK_PIPELINE_BIND_POINT_GRAPHICS;
3938 subpass_descriptions[1].pDepthStencilAttachment = &attach_ref2;
3939
3940 VkAttachmentDescription attachment_description = {};
3941 attachment_description.format = VK_FORMAT_D32_SFLOAT;
3942 attachment_description.samples = VK_SAMPLE_COUNT_1_BIT;
3943 attachment_description.loadOp = VK_ATTACHMENT_LOAD_OP_LOAD;
3944 attachment_description.storeOp = VK_ATTACHMENT_STORE_OP_STORE;
3945 attachment_description.stencilLoadOp = VK_ATTACHMENT_LOAD_OP_DONT_CARE;
3946 attachment_description.stencilStoreOp = VK_ATTACHMENT_STORE_OP_DONT_CARE;
3947 attachment_description.initialLayout = VK_IMAGE_LAYOUT_DEPTH_ATTACHMENT_OPTIMAL;
3948 attachment_description.finalLayout = VK_IMAGE_LAYOUT_GENERAL;
3949
3950 auto rp_ci = LvlInitStruct<VkRenderPassCreateInfo>();
3951 rp_ci.subpassCount = 1;
3952 rp_ci.pSubpasses = subpass_descriptions;
3953 rp_ci.attachmentCount = 1;
3954 rp_ci.pAttachments = &attachment_description;
3955 rp_ci.dependencyCount = 1;
3956 rp_ci.pDependencies = &subpass_dependency;
3957
3958 vk_testing::RenderPass render_pass;
3959 render_pass.init(*m_device, rp_ci);
3960
3961 VkClearValue clear_value = {};
3962 clear_value.color = {{0, 0, 0, 0}};
3963
3964 VkImageCreateInfo image_ci = LvlInitStruct<VkImageCreateInfo>();
3965 image_ci.imageType = VK_IMAGE_TYPE_2D;
3966 image_ci.format = VK_FORMAT_D32_SFLOAT;
3967 image_ci.extent.width = 32;
3968 image_ci.extent.height = 32;
3969 image_ci.extent.depth = 1;
3970 image_ci.mipLevels = 1;
3971 image_ci.arrayLayers = 1;
3972 image_ci.samples = VK_SAMPLE_COUNT_1_BIT;
3973 image_ci.tiling = VK_IMAGE_TILING_OPTIMAL;
3974 image_ci.usage = VK_IMAGE_USAGE_DEPTH_STENCIL_ATTACHMENT_BIT;
3975
3976 VkImageObj image1(m_device);
3977 image1.init(&image_ci);
3978 ASSERT_TRUE(image1.initialized());
3979
3980 vk_testing::ImageView image_view1;
3981 VkImageViewCreateInfo iv_ci = LvlInitStruct<VkImageViewCreateInfo>();
3982 iv_ci.image = image1.handle();
3983 iv_ci.viewType = VK_IMAGE_VIEW_TYPE_2D;
3984 iv_ci.format = VK_FORMAT_D32_SFLOAT;
3985 iv_ci.subresourceRange.aspectMask = VK_IMAGE_ASPECT_DEPTH_BIT;
3986 iv_ci.subresourceRange.baseMipLevel = 0;
3987 iv_ci.subresourceRange.levelCount = 1;
3988 iv_ci.subresourceRange.baseArrayLayer = 0;
3989 iv_ci.subresourceRange.layerCount = 1;
3990 image_view1.init(*m_device, iv_ci);
3991
3992 VkImageView framebuffer_attachments[1] = {image_view1.handle()};
3993
3994 auto fb_ci = LvlInitStruct<VkFramebufferCreateInfo>();
3995 fb_ci.renderPass = render_pass.handle();
3996 fb_ci.attachmentCount = 1;
3997 fb_ci.pAttachments = framebuffer_attachments;
3998 fb_ci.width = 32;
3999 fb_ci.height = 32;
4000 fb_ci.layers = 1;
4001
4002 vk_testing::Framebuffer framebuffer;
4003 framebuffer.init(*m_device, fb_ci);
4004
4005 auto rp_bi = LvlInitStruct<VkRenderPassBeginInfo>();
4006 rp_bi.renderPass = render_pass.handle();
4007 rp_bi.framebuffer = framebuffer.handle();
4008 rp_bi.renderArea.extent.width = 32;
4009 rp_bi.renderArea.extent.height = 32;
4010 rp_bi.clearValueCount = 1;
4011 rp_bi.pClearValues = &clear_value;
4012
4013 auto ds_ci = LvlInitStruct<VkPipelineDepthStencilStateCreateInfo>();
4014 ds_ci.depthTestEnable = VK_FALSE;
4015 ds_ci.depthWriteEnable = VK_FALSE;
4016 ds_ci.depthCompareOp = VK_COMPARE_OP_NEVER;
4017
4018 CreatePipelineHelper pipe(*this);
4019 pipe.InitInfo();
4020 pipe.gp_ci_.renderPass = render_pass.handle();
4021 pipe.gp_ci_.pDepthStencilState = &ds_ci;
4022 pipe.InitState();
4023 ASSERT_VK_SUCCESS(pipe.CreateGraphicsPipeline());
4024
4025 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
4026
4027 m_commandBuffer->begin();
4028 m_commandBuffer->BeginRenderPass(rp_bi);
4029 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, pipe.pipeline_);
4030 vk::CmdDraw(m_commandBuffer->handle(), 3, 1, 0, 0);
4031 m_commandBuffer->EndRenderPass();
4032 m_commandBuffer->end();
4033
4034 m_errorMonitor->VerifyFound();
4035}
4036
ziga-lunargc71f1a92022-03-23 23:08:35 +01004037TEST_F(VkSyncValTest, TestCopyingToCompressedImage) {
4038 TEST_DESCRIPTION("Copy from uncompressed to compressed image with and without overlap.");
4039
4040 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
4041 bool copy_commands_2 = false;
4042 if (DeviceExtensionSupported(gpu(), nullptr, VK_KHR_COPY_COMMANDS_2_EXTENSION_NAME)) {
4043 m_device_extension_names.push_back(VK_KHR_COPY_COMMANDS_2_EXTENSION_NAME);
4044 copy_commands_2 = true;
4045 }
4046 ASSERT_NO_FATAL_FAILURE(InitState(nullptr, nullptr, VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT));
4047
4048 VkFormatProperties format_properties;
4049 VkFormat mp_format = VK_FORMAT_BC1_RGBA_UNORM_BLOCK;
4050 vk::GetPhysicalDeviceFormatProperties(gpu(), mp_format, &format_properties);
4051 if ((format_properties.linearTilingFeatures & VK_FORMAT_FEATURE_TRANSFER_DST_BIT) == 0) {
4052 printf(
4053 "%s Device does not support VK_FORMAT_FEATURE_STORAGE_IMAGE_BIT for VK_FORMAT_BC1_RGBA_UNORM_BLOCK, skipping test.\n",
4054 kSkipPrefix);
4055 return;
4056 }
4057
4058 VkImageObj src_image(m_device);
4059 src_image.Init(1, 1, 1, VK_FORMAT_R32G32_UINT, VK_IMAGE_USAGE_TRANSFER_SRC_BIT, VK_IMAGE_TILING_LINEAR);
4060 VkImageObj dst_image(m_device);
4061 dst_image.Init(12, 4, 1, VK_FORMAT_BC1_RGBA_UNORM_BLOCK, VK_IMAGE_USAGE_TRANSFER_DST_BIT, VK_IMAGE_TILING_LINEAR);
4062
4063 VkImageCopy copy_regions[2] = {};
4064 copy_regions[0].srcSubresource.aspectMask = VK_IMAGE_ASPECT_COLOR_BIT;
4065 copy_regions[0].srcSubresource.mipLevel = 0;
4066 copy_regions[0].srcSubresource.baseArrayLayer = 0;
4067 copy_regions[0].srcSubresource.layerCount = 1;
4068 copy_regions[0].srcOffset = {0, 0, 0};
4069 copy_regions[0].dstSubresource.aspectMask = VK_IMAGE_ASPECT_COLOR_BIT;
4070 copy_regions[0].dstSubresource.mipLevel = 0;
4071 copy_regions[0].dstSubresource.baseArrayLayer = 0;
4072 copy_regions[0].dstSubresource.layerCount = 1;
4073 copy_regions[0].dstOffset = {0, 0, 0};
4074 copy_regions[0].extent = {1, 1, 1};
4075 copy_regions[1].srcSubresource.aspectMask = VK_IMAGE_ASPECT_COLOR_BIT;
4076 copy_regions[1].srcSubresource.mipLevel = 0;
4077 copy_regions[1].srcSubresource.baseArrayLayer = 0;
4078 copy_regions[1].srcSubresource.layerCount = 1;
4079 copy_regions[1].srcOffset = {0, 0, 0};
4080 copy_regions[1].dstSubresource.aspectMask = VK_IMAGE_ASPECT_COLOR_BIT;
4081 copy_regions[1].dstSubresource.mipLevel = 0;
4082 copy_regions[1].dstSubresource.baseArrayLayer = 0;
4083 copy_regions[1].dstSubresource.layerCount = 1;
4084 copy_regions[1].dstOffset = {4, 0, 0};
4085 copy_regions[1].extent = {1, 1, 1};
4086
4087 m_commandBuffer->begin();
4088
4089 m_errorMonitor->ExpectSuccess();
4090 vk::CmdCopyImage(m_commandBuffer->handle(), src_image.handle(), VK_IMAGE_LAYOUT_GENERAL, dst_image.handle(),
4091 VK_IMAGE_LAYOUT_GENERAL, 1, &copy_regions[0]);
4092 vk::CmdCopyImage(m_commandBuffer->handle(), src_image.handle(), VK_IMAGE_LAYOUT_GENERAL, dst_image.handle(),
4093 VK_IMAGE_LAYOUT_GENERAL, 1, &copy_regions[1]);
4094 m_errorMonitor->VerifyNotFound();
4095 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "SYNC-HAZARD-WRITE_AFTER_WRITE");
4096 copy_regions[1].dstOffset = {7, 0, 0};
4097 vk::CmdCopyImage(m_commandBuffer->handle(), src_image.handle(), VK_IMAGE_LAYOUT_GENERAL, dst_image.handle(),
4098 VK_IMAGE_LAYOUT_GENERAL, 1, &copy_regions[1]);
4099 m_errorMonitor->VerifyFound();
4100
4101 m_commandBuffer->end();
4102
4103 if (copy_commands_2) {
4104 auto vkCmdCopyImage2KHR =
4105 reinterpret_cast<PFN_vkCmdCopyImage2KHR>(vk::GetInstanceProcAddr(instance(), "vkCmdCopyImage2KHR"));
4106 assert(vkCmdCopyImage2KHR != nullptr);
4107
4108 m_commandBuffer->reset();
4109
4110 VkImageCopy2KHR copy_regions2[2];
4111 copy_regions2[0] = LvlInitStruct<VkImageCopy2KHR>();
4112 copy_regions2[0].srcSubresource.aspectMask = VK_IMAGE_ASPECT_COLOR_BIT;
4113 copy_regions2[0].srcSubresource.mipLevel = 0;
4114 copy_regions2[0].srcSubresource.baseArrayLayer = 0;
4115 copy_regions2[0].srcSubresource.layerCount = 1;
4116 copy_regions2[0].srcOffset = {0, 0, 0};
4117 copy_regions2[0].dstSubresource.aspectMask = VK_IMAGE_ASPECT_COLOR_BIT;
4118 copy_regions2[0].dstSubresource.mipLevel = 0;
4119 copy_regions2[0].dstSubresource.baseArrayLayer = 0;
4120 copy_regions2[0].dstSubresource.layerCount = 1;
4121 copy_regions2[0].dstOffset = {0, 0, 0};
4122 copy_regions2[0].extent = {1, 1, 1};
4123 copy_regions2[1] = LvlInitStruct<VkImageCopy2KHR>();
4124 copy_regions2[1].srcSubresource.aspectMask = VK_IMAGE_ASPECT_COLOR_BIT;
4125 copy_regions2[1].srcSubresource.mipLevel = 0;
4126 copy_regions2[1].srcSubresource.baseArrayLayer = 0;
4127 copy_regions2[1].srcSubresource.layerCount = 1;
4128 copy_regions2[1].srcOffset = {0, 0, 0};
4129 copy_regions2[1].dstSubresource.aspectMask = VK_IMAGE_ASPECT_COLOR_BIT;
4130 copy_regions2[1].dstSubresource.mipLevel = 0;
4131 copy_regions2[1].dstSubresource.baseArrayLayer = 0;
4132 copy_regions2[1].dstSubresource.layerCount = 1;
4133 copy_regions2[1].dstOffset = {4, 0, 0};
4134 copy_regions2[1].extent = {1, 1, 1};
4135
4136 auto copy_image_info = LvlInitStruct<VkCopyImageInfo2KHR>();
4137 copy_image_info.srcImage = src_image.handle();
4138 copy_image_info.srcImageLayout = VK_IMAGE_LAYOUT_GENERAL;
4139 copy_image_info.dstImage = dst_image.handle();
4140 copy_image_info.dstImageLayout = VK_IMAGE_LAYOUT_GENERAL;
4141 copy_image_info.regionCount = 2;
4142 copy_image_info.pRegions = copy_regions2;
4143
4144 m_commandBuffer->begin();
4145
4146 m_errorMonitor->ExpectSuccess();
4147 vkCmdCopyImage2KHR(m_commandBuffer->handle(), &copy_image_info);
4148 m_errorMonitor->VerifyNotFound();
4149 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "SYNC-HAZARD-WRITE_AFTER_WRITE");
4150 copy_image_info.regionCount = 1;
4151 copy_image_info.pRegions = &copy_regions2[1];
4152 copy_regions[1].dstOffset = {7, 0, 0};
4153 vkCmdCopyImage2KHR(m_commandBuffer->handle(), &copy_image_info);
4154 m_errorMonitor->VerifyFound();
4155
4156 m_commandBuffer->end();
4157 }
4158}
John Zulaufd79e34f2022-04-20 16:39:59 -06004159
4160TEST_F(VkSyncValTest, SyncQSBufferCopyHazards) {
4161 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework(true)); // Enable QueueSubmit validation
4162 ASSERT_NO_FATAL_FAILURE(InitState(nullptr, nullptr, VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT));
4163
4164 VkBufferObj buffer_a;
4165 VkBufferObj buffer_b;
4166 VkBufferObj buffer_c;
4167 VkMemoryPropertyFlags mem_prop = VK_MEMORY_PROPERTY_DEVICE_LOCAL_BIT;
4168 buffer_a.init_as_src_and_dst(*m_device, 256, mem_prop);
4169 buffer_b.init_as_src_and_dst(*m_device, 256, mem_prop);
4170 buffer_c.init_as_src_and_dst(*m_device, 256, mem_prop);
4171
4172 VkBufferCopy region = {0, 0, 256};
4173
4174 VkCommandBufferObj cba(m_device, m_commandPool);
4175 VkCommandBufferObj cbb(m_device, m_commandPool);
4176
4177 cba.begin();
4178 const VkCommandBuffer h_cba = cba.handle();
4179 vk::CmdCopyBuffer(h_cba, buffer_a.handle(), buffer_b.handle(), 1, &region);
4180 cba.end();
4181
4182 const VkCommandBuffer h_cbb = cbb.handle();
4183 cbb.begin();
4184 vk::CmdCopyBuffer(h_cbb, buffer_c.handle(), buffer_a.handle(), 1, &region);
4185 cbb.end();
4186
4187 auto submit1 = lvl_init_struct<VkSubmitInfo>();
4188 submit1.commandBufferCount = 2;
4189 VkCommandBuffer two_cbs[2] = {h_cba, h_cbb};
4190 submit1.pCommandBuffers = two_cbs;
4191
4192 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
4193 vk::QueueSubmit(m_device->m_queue, 1, &submit1, VK_NULL_HANDLE);
4194 m_errorMonitor->VerifyFound();
4195
4196 vk::DeviceWaitIdle(m_device->device());
4197
4198 VkSubmitInfo submit2[2] = {lvl_init_struct<VkSubmitInfo>(), lvl_init_struct<VkSubmitInfo>()};
4199 submit2[0].commandBufferCount = 1;
4200 submit2[0].pCommandBuffers = &h_cba;
4201 submit2[1].commandBufferCount = 1;
4202 submit2[1].pCommandBuffers = &h_cbb;
4203 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
4204 vk::QueueSubmit(m_device->m_queue, 2, submit2, VK_NULL_HANDLE);
4205 m_errorMonitor->VerifyFound();
4206
4207 // With the skip settings, the above QueueSubmit's didn't record, so we can treat the global queue contexts as empty
4208 submit1.commandBufferCount = 1;
4209 submit1.pCommandBuffers = &h_cba;
4210 // Submit A
4211 m_errorMonitor->ExpectSuccess();
4212 vk::QueueSubmit(m_device->m_queue, 1, &submit1, VK_NULL_HANDLE);
4213 m_errorMonitor->VerifyNotFound();
4214
4215 submit1.pCommandBuffers = &h_cbb;
4216 // Submit B -- which should conflict via the queue's "last batch"
4217 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
4218 vk::QueueSubmit(m_device->m_queue, 1, &submit1, VK_NULL_HANDLE);
4219 m_errorMonitor->VerifyFound();
4220}
Nathaniel Cesariofaa36042022-06-14 13:47:26 -06004221
John Zulauf833ed032022-05-13 14:54:43 -06004222TEST_F(VkSyncValTest, SyncQSBufferCopyVsIdle) {
Nathaniel Cesariofaa36042022-06-14 13:47:26 -06004223 // TODO (jzulauf)
4224 GTEST_SKIP() << "this test is causing a sporadic crash on nvidia 32b release. Skip until further investigation";
4225
John Zulauf833ed032022-05-13 14:54:43 -06004226 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework(true)); // Enable QueueSubmit validation
4227 ASSERT_NO_FATAL_FAILURE(InitState(nullptr, nullptr, VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT));
4228
4229 VkBufferObj buffer_a;
4230 VkBufferObj buffer_b;
4231 VkBufferObj buffer_c;
4232 VkMemoryPropertyFlags mem_prop = VK_MEMORY_PROPERTY_DEVICE_LOCAL_BIT;
4233 buffer_a.init_as_src_and_dst(*m_device, 256, mem_prop);
4234 buffer_b.init_as_src_and_dst(*m_device, 256, mem_prop);
4235 buffer_c.init_as_src_and_dst(*m_device, 256, mem_prop);
4236
4237 VkBufferCopy region = {0, 0, 256};
4238
4239 m_errorMonitor->ExpectSuccess();
4240 VkCommandBufferObj cba(m_device, m_commandPool);
4241 VkCommandBufferObj cbb(m_device, m_commandPool);
4242
4243 cba.begin();
4244 const VkCommandBuffer h_cba = cba.handle();
4245 vk::CmdCopyBuffer(h_cba, buffer_a.handle(), buffer_b.handle(), 1, &region);
4246 cba.end();
4247
4248 const VkCommandBuffer h_cbb = cbb.handle();
4249 cbb.begin();
4250 vk::CmdCopyBuffer(h_cbb, buffer_c.handle(), buffer_a.handle(), 1, &region);
4251 cbb.end();
4252
4253 // Submit A
4254 auto submit1 = lvl_init_struct<VkSubmitInfo>();
4255 submit1.commandBufferCount = 1;
4256 submit1.pCommandBuffers = &h_cba;
4257 vk::QueueSubmit(m_device->m_queue, 1, &submit1, VK_NULL_HANDLE);
4258 m_errorMonitor->VerifyNotFound();
4259
4260 // Submit B which hazards vs. A
4261 submit1.pCommandBuffers = &h_cbb;
4262 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
4263 vk::QueueSubmit(m_device->m_queue, 1, &submit1, VK_NULL_HANDLE);
4264 m_errorMonitor->VerifyFound();
4265
4266 // With the skip settings, the above QueueSubmit's didn't record, so we can treat the previous submit as not
4267 // having happened. So we'll try again with a device wait idle
4268 // Submit B again, but after idling, which should remove the hazard
4269 m_errorMonitor->ExpectSuccess();
4270 vk::DeviceWaitIdle(m_device->device());
4271 vk::QueueSubmit(m_device->m_queue, 1, &submit1, VK_NULL_HANDLE);
4272 m_errorMonitor->VerifyNotFound();
4273
4274 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
4275 vk::QueueSubmit(m_device->m_queue, 1, &submit1, VK_NULL_HANDLE);
4276 m_errorMonitor->VerifyFound();
4277
4278 // With the skip settings, the above QueueSubmit's didn't record, so we can treat the previous submit as not
4279 // having happened. So we'll try again with a queue wait idle
4280 // Submit B again, but after idling, which should remove the hazard
4281 m_errorMonitor->ExpectSuccess();
4282 vk::QueueWaitIdle(m_device->m_queue);
4283 vk::QueueSubmit(m_device->m_queue, 1, &submit1, VK_NULL_HANDLE);
4284 m_errorMonitor->VerifyNotFound();
4285}
John Zulauf6df2d5c2022-05-28 13:02:21 -06004286
4287TEST_F(VkSyncValTest, SyncQSBufferCopyQSORules) {
4288 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework(true)); // Enable QueueSubmit validation
4289 ASSERT_NO_FATAL_FAILURE(InitState(nullptr, nullptr, VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT));
4290
4291 const auto &queues = m_device->dma_queues();
4292 VkQueue q0 = VK_NULL_HANDLE;
4293 VkQueue q1 = VK_NULL_HANDLE;
4294 uint32_t q_fam = ~0U;
4295
4296 const uint32_t q_count = static_cast<uint32_t>(queues.size());
4297 for (uint32_t q0_index = 0; q0_index < q_count; ++q0_index) {
4298 const auto *q0_entry = queues[q0_index];
4299 q0 = q0_entry->handle();
4300 q_fam = q0_entry->get_family_index();
4301 for (uint32_t q1_index = (q0_index + 1); q1_index < q_count; ++q1_index) {
4302 const auto *q1_entry = queues[q1_index];
4303 if (q_fam == q1_entry->get_family_index()) {
4304 q1 = q1_entry->handle();
4305 break;
4306 }
4307 }
4308 if (q1 != VK_NULL_HANDLE) {
4309 break;
4310 }
4311 }
4312 if (q1 == VK_NULL_HANDLE) {
4313 printf("%s Test requires at least 2 TRANSFER capable queues in the same queue_family. Skipped.\n", kSkipPrefix);
4314 return;
4315 }
4316
4317 VkBufferObj buffer_a;
4318 VkBufferObj buffer_b;
4319 VkBufferObj buffer_c;
4320 VkMemoryPropertyFlags mem_prop = VK_MEMORY_PROPERTY_DEVICE_LOCAL_BIT;
4321 buffer_a.init_as_src_and_dst(*m_device, 256, mem_prop);
4322 buffer_b.init_as_src_and_dst(*m_device, 256, mem_prop);
4323 buffer_c.init_as_src_and_dst(*m_device, 256, mem_prop);
4324
4325 VkBufferCopy region = {0, 0, 256};
4326
4327 m_errorMonitor->ExpectSuccess();
4328 VkCommandPoolObj pool(m_device, q_fam, VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT);
4329 VkCommandBufferObj cba(m_device, &pool);
4330 VkCommandBufferObj cbb(m_device, &pool);
John Zulaufd060c3f2022-06-08 16:00:46 -06004331 VkCommandBufferObj cbc(m_device, &pool);
John Zulauf6df2d5c2022-05-28 13:02:21 -06004332
4333 // Command Buffer A reads froms buffer A and writes to buffer B
4334 cba.begin();
4335 const VkCommandBuffer h_cba = cba.handle();
4336 vk::CmdCopyBuffer(h_cba, buffer_a.handle(), buffer_b.handle(), 1, &region);
4337 cba.end();
4338
4339 // Command Buffer B reads froms buffer C and writes to buffer A, but has a barrier to protect the write to A when
4340 // executed on the same queue, given that commands in "queue submission order" are within the first scope of the barrier.
4341 const VkCommandBuffer h_cbb = cbb.handle();
4342 cbb.begin();
4343
4344 // Use the barrier to clean up the WAR, which will work for command buffers ealier in queue submission order, or with
4345 // correct semaphore operations between queues.
4346 auto buffer_barrier = LvlInitStruct<VkBufferMemoryBarrier>();
4347 buffer_barrier.srcAccessMask = VK_ACCESS_TRANSFER_READ_BIT;
4348 buffer_barrier.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
4349 buffer_barrier.buffer = buffer_a.handle();
4350 buffer_barrier.offset = 0;
4351 buffer_barrier.size = 256;
4352 vk::CmdPipelineBarrier(h_cbb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, 0, nullptr, 1, &buffer_barrier,
4353 0, nullptr);
4354 vk::CmdCopyBuffer(h_cbb, buffer_c.handle(), buffer_a.handle(), 1, &region);
4355 cbb.end();
4356
John Zulaufd060c3f2022-06-08 16:00:46 -06004357 // Command Buffer C does the same copy as B but without the barrier.
4358 cbc.begin();
4359 const VkCommandBuffer h_cbc = cbc.handle();
4360 vk::CmdCopyBuffer(h_cbc, buffer_c.handle(), buffer_a.handle(), 1, &region);
4361 cbc.end();
4362
John Zulauf6df2d5c2022-05-28 13:02:21 -06004363 // Submit A and B on the same queue, to assure us the barrier *would* be sufficient given QSO
4364 // This is included in a "Sucess" section, just to verify CBA and CBB are set up correctly.
4365 auto submit1 = lvl_init_struct<VkSubmitInfo>();
4366 submit1.commandBufferCount = 1;
4367 submit1.pCommandBuffers = &h_cba;
4368 vk::QueueSubmit(q0, 1, &submit1, VK_NULL_HANDLE);
4369 submit1.pCommandBuffers = &h_cbb;
4370 vk::QueueSubmit(q0, 1, &submit1, VK_NULL_HANDLE);
4371 m_device->wait(); // DeviceWaitIdle, clearing the field for the next subcase
4372 m_errorMonitor->VerifyNotFound();
4373
4374 // Submit A and B on the different queues. Since no semaphore is used between the queues, CB B hazards asynchronously with,
4375 // CB A with A being read and written on independent queues.
4376 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE-RACING-READ");
4377 submit1.pCommandBuffers = &h_cba;
4378 vk::QueueSubmit(q0, 1, &submit1, VK_NULL_HANDLE);
4379 submit1.pCommandBuffers = &h_cbb;
4380 vk::QueueSubmit(q1, 1, &submit1, VK_NULL_HANDLE);
4381 m_errorMonitor->VerifyFound();
4382
4383 // Set up the semaphore for the next two cases
4384 m_errorMonitor->ExpectSuccess();
4385 auto semaphore_ci = LvlInitStruct<VkSemaphoreCreateInfo>();
4386 vk_testing::Semaphore semaphore;
4387 semaphore.init(*m_device, semaphore_ci);
4388 VkSemaphore h_semaphore = semaphore.handle();
4389
4390 VkSubmitInfo submit_signal = submit1;
4391 submit_signal.pCommandBuffers = &h_cba;
4392 submit_signal.signalSemaphoreCount = 1;
4393 submit_signal.pSignalSemaphores = &h_semaphore;
4394
4395 VkSubmitInfo submit_wait = submit1;
4396 VkPipelineStageFlags wait_mask = 0U; // for the next case, there is no mask to chain the semaphore to the barrier
4397 submit_wait.pCommandBuffers = &h_cbb;
4398 submit_wait.pWaitDstStageMask = &wait_mask;
4399 submit_wait.waitSemaphoreCount = 1;
4400 submit_wait.pWaitSemaphores = &h_semaphore;
4401
4402 m_device->wait();
4403 m_errorMonitor->VerifyNotFound();
4404
4405 // Submit A and B on the different queues, with an ineffectual semaphore. The wait mask is empty, thus nothing in CB B is in
4406 // the second excution scope of the waited signal.
4407 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
John Zulauf6df2d5c2022-05-28 13:02:21 -06004408 vk::QueueSubmit(q0, 1, &submit_signal, VK_NULL_HANDLE);
John Zulauf6df2d5c2022-05-28 13:02:21 -06004409 vk::QueueSubmit(q1, 1, &submit_wait, VK_NULL_HANDLE);
4410 m_errorMonitor->VerifyFound();
4411
4412 // The since second submit failed, it was skipped. So we can try again, without having to WaitDeviceIdle
4413 m_errorMonitor->ExpectSuccess();
4414 // Include transfers in the second execution scope of the waited signal, s.t. the PipelineBarrier in CB B can can with it.
4415 wait_mask = VK_PIPELINE_STAGE_TRANSFER_BIT;
4416 vk::QueueSubmit(q1, 1, &submit_wait, VK_NULL_HANDLE);
John Zulaufd060c3f2022-06-08 16:00:46 -06004417
4418 m_device->wait();
4419
4420 // Draw A and then C to verify the second access scope of the signal
4421 vk::QueueSubmit(q0, 1, &submit_signal, VK_NULL_HANDLE);
4422 submit_wait.pCommandBuffers = &h_cbc;
4423 vk::QueueSubmit(q1, 1, &submit_wait, VK_NULL_HANDLE);
4424
4425 m_device->wait();
4426
4427 // ... and again on the same queue
4428 vk::QueueSubmit(q0, 1, &submit_signal, VK_NULL_HANDLE);
4429 vk::QueueSubmit(q0, 1, &submit_wait, VK_NULL_HANDLE);
4430
John Zulauf6df2d5c2022-05-28 13:02:21 -06004431 m_errorMonitor->VerifyNotFound();
4432}