blob: 54ac45819cbb38feeab5c7e61b9161116e418390 [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());
1323 VkPhysicalDeviceVulkan12Features features12 = {VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_VULKAN_1_2_FEATURES, nullptr};
1324 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;
Jeremy Gebben170781d2020-11-19 16:21:21 -07001951 VkQueryPoolCreateInfo query_pool_create_info{};
1952 query_pool_create_info.sType = VK_STRUCTURE_TYPE_QUERY_POOL_CREATE_INFO;
1953 query_pool_create_info.queryType = VK_QUERY_TYPE_TIMESTAMP;
1954 query_pool_create_info.queryCount = 1;
Jeremy Gebben18ac1052021-08-12 11:07:32 -06001955 query_pool.init(*m_device, query_pool_create_info);
Jeremy Gebben170781d2020-11-19 16:21:21 -07001956
1957 VkBufferObj buffer_a, buffer_b;
1958 VkMemoryPropertyFlags mem_prop = VK_MEMORY_PROPERTY_DEVICE_LOCAL_BIT;
1959 buffer_a.init_as_src_and_dst(*m_device, 256, mem_prop);
1960 buffer_b.init_as_src_and_dst(*m_device, 256, mem_prop);
1961
1962 VkBufferCopy region = {0, 0, 256};
1963
1964 auto cb = m_commandBuffer->handle();
1965 m_commandBuffer->begin();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06001966 vk::CmdResetQueryPool(cb, query_pool.handle(), 0, 1);
1967 vk::CmdWriteTimestamp(cb, VK_PIPELINE_STAGE_ALL_GRAPHICS_BIT, query_pool.handle(), 0);
1968 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 -07001969 m_commandBuffer->end();
1970 m_errorMonitor->VerifyNotFound();
1971
1972 m_commandBuffer->reset();
1973 m_commandBuffer->begin();
1974 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &region);
Jeremy Gebben18ac1052021-08-12 11:07:32 -06001975 vk::CmdResetQueryPool(cb, query_pool.handle(), 0, 1);
1976 vk::CmdWriteTimestamp(cb, VK_PIPELINE_STAGE_ALL_GRAPHICS_BIT, query_pool.handle(), 0);
Jeremy Gebben170781d2020-11-19 16:21:21 -07001977 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
Jeremy Gebben18ac1052021-08-12 11:07:32 -06001978 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 -07001979 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
Jeremy Gebben18ac1052021-08-12 11:07:32 -06001980 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 -07001981 m_commandBuffer->end();
1982 m_errorMonitor->VerifyFound();
1983
1984 // TODO:Track VkQueryPool
1985 // TODO:CmdWriteTimestamp
Jeremy Gebben170781d2020-11-19 16:21:21 -07001986}
1987
1988TEST_F(VkSyncValTest, SyncCmdDrawDepthStencil) {
1989 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
1990 ASSERT_NO_FATAL_FAILURE(InitState(nullptr, nullptr, VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT));
1991 m_errorMonitor->ExpectSuccess();
1992
1993 const auto format_ds = FindSupportedDepthStencilFormat(gpu());
1994 if (!format_ds) {
1995 printf("%s No Depth + Stencil format found. Skipped.\n", kSkipPrefix);
1996 return;
1997 }
1998 const auto format_dp = FindSupportedDepthOnlyFormat(gpu());
1999 if (!format_dp) {
2000 printf("%s No only Depth format found. Skipped.\n", kSkipPrefix);
2001 return;
2002 }
2003 const auto format_st = FindSupportedStencilOnlyFormat(gpu());
2004 if (!format_st) {
2005 printf("%s No only Stencil format found. Skipped.\n", kSkipPrefix);
2006 return;
2007 }
2008
2009 VkDepthStencilObj image_ds(m_device), image_dp(m_device), image_st(m_device);
2010 image_ds.Init(m_device, 16, 16, format_ds);
2011 image_dp.Init(m_device, 16, 16, format_dp);
2012 image_st.Init(m_device, 16, 16, format_st);
2013
2014 VkRenderpassObj rp_ds(m_device, format_ds, true), rp_dp(m_device, format_dp, true), rp_st(m_device, format_st, true);
2015
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002016 vk_testing::Framebuffer fb_ds, fb_dp, fb_st;
Jeremy Gebben170781d2020-11-19 16:21:21 -07002017 VkFramebufferCreateInfo fbci = {
2018 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 -06002019 fb_ds.init(*m_device, fbci);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002020 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 -06002021 fb_dp.init(*m_device, fbci);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002022 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 -06002023 fb_st.init(*m_device, fbci);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002024
2025 VkStencilOpState stencil = {};
2026 stencil.failOp = VK_STENCIL_OP_KEEP;
2027 stencil.passOp = VK_STENCIL_OP_KEEP;
2028 stencil.depthFailOp = VK_STENCIL_OP_KEEP;
2029 stencil.compareOp = VK_COMPARE_OP_NEVER;
2030
Mark Lobodzinski07d0a612020-12-30 15:42:31 -07002031 auto ds_ci = LvlInitStruct<VkPipelineDepthStencilStateCreateInfo>();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002032 ds_ci.depthTestEnable = VK_TRUE;
2033 ds_ci.depthWriteEnable = VK_TRUE;
2034 ds_ci.depthCompareOp = VK_COMPARE_OP_NEVER;
2035 ds_ci.stencilTestEnable = VK_TRUE;
2036 ds_ci.front = stencil;
2037 ds_ci.back = stencil;
2038
2039 CreatePipelineHelper g_pipe_ds(*this), g_pipe_dp(*this), g_pipe_st(*this);
2040 g_pipe_ds.InitInfo();
2041 g_pipe_ds.gp_ci_.renderPass = rp_ds.handle();
2042 g_pipe_ds.gp_ci_.pDepthStencilState = &ds_ci;
2043 g_pipe_ds.InitState();
2044 ASSERT_VK_SUCCESS(g_pipe_ds.CreateGraphicsPipeline());
2045 g_pipe_dp.InitInfo();
2046 g_pipe_dp.gp_ci_.renderPass = rp_dp.handle();
2047 ds_ci.stencilTestEnable = VK_FALSE;
2048 g_pipe_dp.gp_ci_.pDepthStencilState = &ds_ci;
2049 g_pipe_dp.InitState();
2050 ASSERT_VK_SUCCESS(g_pipe_dp.CreateGraphicsPipeline());
2051 g_pipe_st.InitInfo();
2052 g_pipe_st.gp_ci_.renderPass = rp_st.handle();
2053 ds_ci.depthTestEnable = VK_FALSE;
2054 ds_ci.stencilTestEnable = VK_TRUE;
2055 g_pipe_st.gp_ci_.pDepthStencilState = &ds_ci;
2056 g_pipe_st.InitState();
2057 ASSERT_VK_SUCCESS(g_pipe_st.CreateGraphicsPipeline());
2058
2059 m_commandBuffer->begin();
2060 m_renderPassBeginInfo.renderArea = {{0, 0}, {16, 16}};
2061 m_renderPassBeginInfo.pClearValues = nullptr;
2062 m_renderPassBeginInfo.clearValueCount = 0;
2063
2064 m_renderPassBeginInfo.renderPass = rp_ds.handle();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002065 m_renderPassBeginInfo.framebuffer = fb_ds.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002066 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2067 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_ds.pipeline_);
2068 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
2069 m_commandBuffer->EndRenderPass();
2070
2071 m_renderPassBeginInfo.renderPass = rp_dp.handle();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002072 m_renderPassBeginInfo.framebuffer = fb_dp.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002073 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2074 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_dp.pipeline_);
2075 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
2076 m_commandBuffer->EndRenderPass();
2077
2078 m_renderPassBeginInfo.renderPass = rp_st.handle();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002079 m_renderPassBeginInfo.framebuffer = fb_st.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002080 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2081 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_st.pipeline_);
2082 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
2083 m_commandBuffer->EndRenderPass();
2084
2085 m_commandBuffer->end();
2086 m_errorMonitor->VerifyNotFound();
2087
2088 m_commandBuffer->reset();
2089 m_commandBuffer->begin();
2090
2091 VkImageCopy copyRegion;
2092 copyRegion.srcSubresource.aspectMask = VK_IMAGE_ASPECT_DEPTH_BIT;
2093 copyRegion.srcSubresource.mipLevel = 0;
2094 copyRegion.srcSubresource.baseArrayLayer = 0;
2095 copyRegion.srcSubresource.layerCount = 1;
2096 copyRegion.srcOffset = {0, 0, 0};
2097 copyRegion.dstSubresource.aspectMask = VK_IMAGE_ASPECT_DEPTH_BIT;
2098 copyRegion.dstSubresource.mipLevel = 0;
2099 copyRegion.dstSubresource.baseArrayLayer = 0;
2100 copyRegion.dstSubresource.layerCount = 1;
2101 copyRegion.dstOffset = {0, 0, 0};
2102 copyRegion.extent = {16, 16, 1};
2103
2104 m_commandBuffer->CopyImage(image_ds.handle(), VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL, image_dp.handle(),
2105 VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL, 1, &copyRegion);
2106
2107 copyRegion.srcSubresource.aspectMask = VK_IMAGE_ASPECT_STENCIL_BIT;
2108 copyRegion.dstSubresource.aspectMask = VK_IMAGE_ASPECT_STENCIL_BIT;
2109 m_commandBuffer->CopyImage(image_ds.handle(), VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL, image_st.handle(),
2110 VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL, 1, &copyRegion);
2111 m_renderPassBeginInfo.renderPass = rp_ds.handle();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002112 m_renderPassBeginInfo.framebuffer = fb_ds.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002113 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
2114 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2115 m_errorMonitor->VerifyFound();
2116 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2117 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_ds.pipeline_);
2118 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
2119 m_commandBuffer->EndRenderPass();
2120
2121 m_renderPassBeginInfo.renderPass = rp_dp.handle();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002122 m_renderPassBeginInfo.framebuffer = fb_dp.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002123 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
2124 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2125 m_errorMonitor->VerifyFound();
2126 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2127 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_dp.pipeline_);
2128 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
2129 m_commandBuffer->EndRenderPass();
2130
2131 m_renderPassBeginInfo.renderPass = rp_st.handle();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002132 m_renderPassBeginInfo.framebuffer = fb_st.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002133 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
2134 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2135 m_errorMonitor->VerifyFound();
2136 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2137 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_st.pipeline_);
2138 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
2139 m_commandBuffer->EndRenderPass();
2140
2141 m_commandBuffer->end();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002142}
2143
John Zulaufd57a36b2021-08-16 10:34:44 -06002144
Jeremy Gebben170781d2020-11-19 16:21:21 -07002145TEST_F(VkSyncValTest, RenderPassLoadHazardVsInitialLayout) {
2146 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
John Zulaufd57a36b2021-08-16 10:34:44 -06002147 bool do_none_load_op_test = false;
2148 if (DeviceExtensionSupported(gpu(), nullptr, VK_EXT_LOAD_STORE_OP_NONE_EXTENSION_NAME)) {
2149 m_device_extension_names.push_back(VK_EXT_LOAD_STORE_OP_NONE_EXTENSION_NAME);
2150 do_none_load_op_test = true;
2151 }
2152
Jeremy Gebben170781d2020-11-19 16:21:21 -07002153 ASSERT_NO_FATAL_FAILURE(InitState());
2154 ASSERT_NO_FATAL_FAILURE(InitRenderTarget());
2155
2156 VkImageUsageFlags usage_color = VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT;
2157 VkImageUsageFlags usage_input = VK_IMAGE_USAGE_TRANSFER_DST_BIT | VK_IMAGE_USAGE_INPUT_ATTACHMENT_BIT;
2158 VkFormat format = VK_FORMAT_R8G8B8A8_UNORM;
2159 VkImageObj image_color(m_device), image_input(m_device);
2160 auto image_ci = VkImageObj::ImageCreateInfo2D(32, 32, 1, 1, format, usage_color, VK_IMAGE_TILING_OPTIMAL);
2161 image_color.Init(image_ci);
2162 image_ci.usage = usage_input;
2163 image_input.Init(image_ci);
2164 VkImageView attachments[] = {image_color.targetView(format), image_input.targetView(format)};
2165
John Zulaufd57a36b2021-08-16 10:34:44 -06002166 VkAttachmentDescription attachmentDescriptions[] = {
Jeremy Gebben170781d2020-11-19 16:21:21 -07002167 // Result attachment
2168 {(VkAttachmentDescriptionFlags)0, VK_FORMAT_R8G8B8A8_UNORM, VK_SAMPLE_COUNT_1_BIT, VK_ATTACHMENT_LOAD_OP_CLEAR,
2169 VK_ATTACHMENT_STORE_OP_STORE, VK_ATTACHMENT_LOAD_OP_DONT_CARE, VK_ATTACHMENT_STORE_OP_DONT_CARE,
2170 VK_IMAGE_LAYOUT_UNDEFINED, // Here causes DesiredError that SYNC-HAZARD-NONE in BeginRenderPass.
2171 // It should be VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL
2172 VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL},
2173 // Input attachment
2174 {(VkAttachmentDescriptionFlags)0, VK_FORMAT_R8G8B8A8_UNORM, VK_SAMPLE_COUNT_1_BIT, VK_ATTACHMENT_LOAD_OP_LOAD,
2175 VK_ATTACHMENT_STORE_OP_STORE, VK_ATTACHMENT_LOAD_OP_DONT_CARE, VK_ATTACHMENT_STORE_OP_DONT_CARE,
2176 VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL, VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL}};
2177
2178 const VkAttachmentReference resultAttachmentRef = {0u, VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL};
2179 const VkAttachmentReference inputAttachmentRef = {1u, VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL};
2180
2181 const VkSubpassDescription subpassDescription = {(VkSubpassDescriptionFlags)0,
2182 VK_PIPELINE_BIND_POINT_GRAPHICS,
2183 1u,
2184 &inputAttachmentRef,
2185 1u,
2186 &resultAttachmentRef,
2187 0,
2188 0,
2189 0u,
2190 0};
2191
2192 const VkSubpassDependency subpassDependency = {VK_SUBPASS_EXTERNAL,
2193 0,
2194 VK_PIPELINE_STAGE_TRANSFER_BIT,
2195 VK_PIPELINE_STAGE_FRAGMENT_SHADER_BIT,
2196 VK_ACCESS_TRANSFER_WRITE_BIT,
2197 VK_ACCESS_INPUT_ATTACHMENT_READ_BIT | VK_ACCESS_SHADER_READ_BIT,
2198 VK_DEPENDENCY_BY_REGION_BIT};
2199
2200 const VkRenderPassCreateInfo renderPassInfo = {VK_STRUCTURE_TYPE_RENDER_PASS_CREATE_INFO,
2201 0,
2202 (VkRenderPassCreateFlags)0,
2203 2u,
2204 attachmentDescriptions,
2205 1u,
2206 &subpassDescription,
2207 1u,
2208 &subpassDependency};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002209 vk_testing::RenderPass rp;
2210 rp.init(*m_device, renderPassInfo);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002211
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002212 vk_testing::Framebuffer fb;
2213 VkFramebufferCreateInfo fbci = {VK_STRUCTURE_TYPE_FRAMEBUFFER_CREATE_INFO, nullptr, 0, rp.handle(), 2, attachments, 32, 32, 1};
2214 fb.init(*m_device, fbci);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002215
2216 image_input.SetLayout(VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL);
2217
2218 m_commandBuffer->begin();
2219
2220 m_renderPassBeginInfo.renderArea = {{0, 0}, {32, 32}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002221 m_renderPassBeginInfo.renderPass = rp.handle();
2222 m_renderPassBeginInfo.framebuffer = fb.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002223
2224 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ_AFTER_WRITE");
2225 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
2226 // Even though we have no accesses prior, the layout transition *is* an access, so load can be validated vs. layout transition
2227 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2228 m_errorMonitor->VerifyFound();
John Zulaufd57a36b2021-08-16 10:34:44 -06002229
2230 vk_testing::RenderPass rp_no_load_store;
2231 if (do_none_load_op_test) {
2232 m_errorMonitor->ExpectSuccess();
2233 attachmentDescriptions[0].loadOp = VK_ATTACHMENT_LOAD_OP_NONE_EXT;
2234 attachmentDescriptions[0].storeOp = VK_ATTACHMENT_STORE_OP_NONE_EXT;
2235 attachmentDescriptions[1].loadOp = VK_ATTACHMENT_LOAD_OP_NONE_EXT;
2236 attachmentDescriptions[1].storeOp = VK_ATTACHMENT_STORE_OP_NONE_EXT;
2237 rp_no_load_store.init(*m_device, renderPassInfo);
2238 m_renderPassBeginInfo.renderPass = rp_no_load_store.handle();
2239 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2240 m_commandBuffer->EndRenderPass();
2241 m_errorMonitor->VerifyNotFound();
2242 } else {
2243 printf("%s VK_EXT_load_store_op_none not supported, skipping sub-test\n", kSkipPrefix);
2244 }
Jeremy Gebben170781d2020-11-19 16:21:21 -07002245}
2246
2247TEST_F(VkSyncValTest, SyncRenderPassWithWrongDepthStencilInitialLayout) {
2248 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
2249 ASSERT_NO_FATAL_FAILURE(InitState());
2250 if (IsPlatform(kNexusPlayer)) {
2251 printf("%s This test should not run on Nexus Player\n", kSkipPrefix);
2252 return;
2253 }
2254
2255 ASSERT_NO_FATAL_FAILURE(InitRenderTarget());
2256
2257 VkFormat color_format = VK_FORMAT_R8G8B8A8_UNORM;
2258 VkFormat ds_format = FindSupportedDepthStencilFormat(gpu());
2259 if (!ds_format) {
2260 printf("%s No Depth + Stencil format found. Skipped.\n", kSkipPrefix);
2261 return;
2262 }
2263 VkImageUsageFlags usage_color = VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT;
2264 VkImageUsageFlags usage_ds = VK_IMAGE_USAGE_DEPTH_STENCIL_ATTACHMENT_BIT;
2265 VkImageObj image_color(m_device), image_color2(m_device);
2266 auto image_ci = VkImageObj::ImageCreateInfo2D(32, 32, 1, 1, color_format, usage_color, VK_IMAGE_TILING_OPTIMAL);
2267 image_color.Init(image_ci);
2268 image_color2.Init(image_ci);
2269 VkDepthStencilObj image_ds(m_device);
2270 image_ds.Init(m_device, 32, 32, ds_format, usage_ds);
2271
2272 const VkAttachmentDescription colorAttachmentDescription = {(VkAttachmentDescriptionFlags)0,
2273 color_format,
2274 VK_SAMPLE_COUNT_1_BIT,
2275 VK_ATTACHMENT_LOAD_OP_CLEAR,
2276 VK_ATTACHMENT_STORE_OP_STORE,
2277 VK_ATTACHMENT_LOAD_OP_DONT_CARE,
2278 VK_ATTACHMENT_STORE_OP_DONT_CARE,
2279 VK_IMAGE_LAYOUT_UNDEFINED,
2280 VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL};
2281
2282 const VkAttachmentDescription depthStencilAttachmentDescription = {
2283 (VkAttachmentDescriptionFlags)0, ds_format, VK_SAMPLE_COUNT_1_BIT, VK_ATTACHMENT_LOAD_OP_CLEAR,
2284 VK_ATTACHMENT_STORE_OP_STORE, VK_ATTACHMENT_LOAD_OP_CLEAR, VK_ATTACHMENT_STORE_OP_STORE,
2285 VK_IMAGE_LAYOUT_UNDEFINED, // Here causes DesiredError that SYNC-HAZARD-WRITE_AFTER_WRITE in BeginRenderPass.
2286 // It should be VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL
2287 VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL};
2288
2289 std::vector<VkAttachmentDescription> attachmentDescriptions;
2290 attachmentDescriptions.push_back(colorAttachmentDescription);
2291 attachmentDescriptions.push_back(depthStencilAttachmentDescription);
2292
2293 const VkAttachmentReference colorAttachmentRef = {0u, VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL};
2294
2295 const VkAttachmentReference depthStencilAttachmentRef = {1u, VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL};
2296
2297 const VkSubpassDescription subpassDescription = {(VkSubpassDescriptionFlags)0,
2298 VK_PIPELINE_BIND_POINT_GRAPHICS,
2299 0u,
2300 0,
2301 1u,
2302 &colorAttachmentRef,
2303 0,
2304 &depthStencilAttachmentRef,
2305 0u,
2306 0};
2307
2308 const VkRenderPassCreateInfo renderPassInfo = {VK_STRUCTURE_TYPE_RENDER_PASS_CREATE_INFO,
2309 0,
2310 (VkRenderPassCreateFlags)0,
2311 (uint32_t)attachmentDescriptions.size(),
2312 &attachmentDescriptions[0],
2313 1u,
2314 &subpassDescription,
2315 0u,
2316 0};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002317 vk_testing::RenderPass rp;
2318 rp.init(*m_device, renderPassInfo);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002319
2320 VkImageView fb_attachments[] = {image_color.targetView(color_format),
2321 image_ds.targetView(ds_format, VK_IMAGE_ASPECT_DEPTH_BIT | VK_IMAGE_ASPECT_STENCIL_BIT)};
2322 const VkFramebufferCreateInfo fbci = {
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002323 VK_STRUCTURE_TYPE_FRAMEBUFFER_CREATE_INFO, 0, 0u, rp.handle(), 2u, fb_attachments, 32, 32, 1u,
Jeremy Gebben170781d2020-11-19 16:21:21 -07002324 };
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002325 vk_testing::Framebuffer fb;
2326 fb.init(*m_device, fbci);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002327 fb_attachments[0] = image_color2.targetView(color_format);
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002328 vk_testing::Framebuffer fb1;
2329 fb1.init(*m_device, fbci);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002330
2331 CreatePipelineHelper g_pipe(*this);
2332 g_pipe.InitInfo();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002333 g_pipe.gp_ci_.renderPass = rp.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002334
2335 VkStencilOpState stencil = {};
2336 stencil.failOp = VK_STENCIL_OP_KEEP;
2337 stencil.passOp = VK_STENCIL_OP_KEEP;
2338 stencil.depthFailOp = VK_STENCIL_OP_KEEP;
2339 stencil.compareOp = VK_COMPARE_OP_NEVER;
2340
Mark Lobodzinski07d0a612020-12-30 15:42:31 -07002341 auto ds_ci = LvlInitStruct<VkPipelineDepthStencilStateCreateInfo>();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002342 ds_ci.depthTestEnable = VK_TRUE;
2343 ds_ci.depthWriteEnable = VK_TRUE;
2344 ds_ci.depthCompareOp = VK_COMPARE_OP_NEVER;
2345 ds_ci.stencilTestEnable = VK_TRUE;
2346 ds_ci.front = stencil;
2347 ds_ci.back = stencil;
2348
2349 g_pipe.gp_ci_.pDepthStencilState = &ds_ci;
2350 g_pipe.InitState();
2351 ASSERT_VK_SUCCESS(g_pipe.CreateGraphicsPipeline());
2352
2353 m_commandBuffer->begin();
Tony-LunarG73f37032021-06-07 11:47:03 -06002354 VkClearValue clear = {};
2355 std::array<VkClearValue, 2> clear_values = { {clear, clear} };
2356 m_renderPassBeginInfo.pClearValues = clear_values.data();
2357 m_renderPassBeginInfo.clearValueCount = clear_values.size();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002358 m_renderPassBeginInfo.renderArea = {{0, 0}, {32, 32}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002359 m_renderPassBeginInfo.renderPass = rp.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002360
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002361 m_renderPassBeginInfo.framebuffer = fb.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002362 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2363 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_);
2364 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
2365 m_commandBuffer->EndRenderPass();
2366
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002367 m_renderPassBeginInfo.framebuffer = fb1.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002368
2369 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
2370 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2371 m_errorMonitor->VerifyFound();
2372}
2373
2374TEST_F(VkSyncValTest, SyncLayoutTransition) {
2375 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
2376 ASSERT_NO_FATAL_FAILURE(InitState());
2377 if (IsPlatform(kNexusPlayer)) {
2378 printf("%s This test should not run on Nexus Player\n", kSkipPrefix);
2379 return;
2380 }
2381
2382 ASSERT_NO_FATAL_FAILURE(InitRenderTarget());
2383
2384 VkImageUsageFlags usage_color = VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT | VK_IMAGE_USAGE_TRANSFER_SRC_BIT;
2385 VkImageUsageFlags usage_input =
2386 VK_IMAGE_USAGE_INPUT_ATTACHMENT_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT | VK_IMAGE_USAGE_SAMPLED_BIT;
2387 VkFormat format = VK_FORMAT_R8G8B8A8_UNORM;
2388 VkImageObj image_color(m_device), image_input(m_device);
2389 auto image_ci = VkImageObj::ImageCreateInfo2D(64, 64, 1, 1, format, usage_input, VK_IMAGE_TILING_OPTIMAL);
2390 image_input.InitNoLayout(image_ci);
2391 image_ci.usage = usage_color;
2392 image_color.InitNoLayout(image_ci);
2393 VkImageView view_input = image_input.targetView(format);
2394 VkImageView view_color = image_color.targetView(format);
2395 VkImageView attachments[] = {view_color, view_input};
2396
2397 const VkAttachmentDescription fbAttachment = {
2398 0u,
2399 format,
2400 VK_SAMPLE_COUNT_1_BIT,
2401 VK_ATTACHMENT_LOAD_OP_CLEAR,
2402 VK_ATTACHMENT_STORE_OP_STORE,
2403 VK_ATTACHMENT_LOAD_OP_DONT_CARE,
2404 VK_ATTACHMENT_STORE_OP_DONT_CARE,
2405 VK_IMAGE_LAYOUT_UNDEFINED,
2406 VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL,
2407 };
2408
2409 std::vector<VkAttachmentDescription> attachmentDescs;
2410 attachmentDescs.push_back(fbAttachment);
2411
2412 // Add it as a frame buffer attachment.
2413 const VkAttachmentDescription inputAttachment = {
2414 0u,
2415 format,
2416 VK_SAMPLE_COUNT_1_BIT,
2417 VK_ATTACHMENT_LOAD_OP_LOAD,
2418 VK_ATTACHMENT_STORE_OP_DONT_CARE,
2419 VK_ATTACHMENT_LOAD_OP_DONT_CARE,
2420 VK_ATTACHMENT_STORE_OP_DONT_CARE,
2421 VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL,
2422 VK_IMAGE_LAYOUT_GENERAL,
2423 };
2424 attachmentDescs.push_back(inputAttachment);
2425
2426 std::vector<VkAttachmentReference> inputAttachments;
2427 const VkAttachmentReference inputRef = {
2428 1u,
2429 VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL,
2430 };
2431 inputAttachments.push_back(inputRef);
2432
2433 const VkAttachmentReference colorRef = {
2434 0u,
2435 VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL,
2436 };
2437 const std::vector<VkAttachmentReference> colorAttachments(1u, colorRef);
2438
2439 const VkSubpassDescription subpass = {
2440 0u,
2441 VK_PIPELINE_BIND_POINT_GRAPHICS,
2442 static_cast<uint32_t>(inputAttachments.size()),
2443 inputAttachments.data(),
2444 static_cast<uint32_t>(colorAttachments.size()),
2445 colorAttachments.data(),
2446 0u,
2447 nullptr,
2448 0u,
2449 nullptr,
2450 };
2451 const std::vector<VkSubpassDescription> subpasses(1u, subpass);
2452
2453 const VkRenderPassCreateInfo renderPassInfo = {
2454 VK_STRUCTURE_TYPE_RENDER_PASS_CREATE_INFO,
2455 nullptr,
2456 0u,
2457 static_cast<uint32_t>(attachmentDescs.size()),
2458 attachmentDescs.data(),
2459 static_cast<uint32_t>(subpasses.size()),
2460 subpasses.data(),
2461 0u,
2462 nullptr,
2463 };
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002464 vk_testing::RenderPass rp;
2465 rp.init(*m_device, renderPassInfo);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002466
2467 const VkFramebufferCreateInfo fbci = {
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002468 VK_STRUCTURE_TYPE_FRAMEBUFFER_CREATE_INFO, 0, 0u, rp.handle(), 2u, attachments, 64, 64, 1u,
Jeremy Gebben170781d2020-11-19 16:21:21 -07002469 };
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002470 vk_testing::Framebuffer fb;
2471 fb.init(*m_device, fbci);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002472
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002473 vk_testing::Sampler sampler;
Jeremy Gebben170781d2020-11-19 16:21:21 -07002474 VkSamplerCreateInfo sampler_info = SafeSaneSamplerCreateInfo();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002475 sampler.init(*m_device, sampler_info);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002476
sfricke-samsung1c0b96a2021-07-08 22:24:09 -07002477 char const *fsSource = R"glsl(
2478 #version 450
2479 layout(input_attachment_index=0, set=0, binding=0) uniform subpassInput x;
2480 void main() {
2481 vec4 color = subpassLoad(x);
2482 }
2483 )glsl";
Jeremy Gebben170781d2020-11-19 16:21:21 -07002484
sfricke-samsungae54c1e2022-01-21 05:35:21 -08002485 VkShaderObj vs(this, bindStateVertShaderText, VK_SHADER_STAGE_VERTEX_BIT);
2486 VkShaderObj fs(this, fsSource, VK_SHADER_STAGE_FRAGMENT_BIT);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002487
2488 CreatePipelineHelper g_pipe(*this);
2489 g_pipe.InitInfo();
2490 g_pipe.shader_stages_ = {vs.GetStageCreateInfo(), fs.GetStageCreateInfo()};
2491 g_pipe.dsl_bindings_ = {{0, VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT, 1, VK_SHADER_STAGE_FRAGMENT_BIT, nullptr}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002492 g_pipe.gp_ci_.renderPass = rp.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002493 g_pipe.InitState();
2494 ASSERT_VK_SUCCESS(g_pipe.CreateGraphicsPipeline());
2495
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002496 g_pipe.descriptor_set_->WriteDescriptorImageInfo(0, view_input, sampler.handle(), VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002497 g_pipe.descriptor_set_->UpdateDescriptorSets();
2498
2499 m_commandBuffer->begin();
2500 auto cb = m_commandBuffer->handle();
2501 VkClearColorValue ccv = {};
2502 VkImageSubresourceRange full_subresource_range{VK_IMAGE_ASPECT_COLOR_BIT, 0, 1, 0, 1};
2503
2504 const VkImageMemoryBarrier preClearBarrier = {
2505 VK_STRUCTURE_TYPE_IMAGE_MEMORY_BARRIER, 0, 0, VK_ACCESS_TRANSFER_WRITE_BIT, VK_IMAGE_LAYOUT_UNDEFINED,
2506 VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL, 0, 0, image_input.handle(), full_subresource_range,
2507 };
2508 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TOP_OF_PIPE_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0u, 0u, nullptr, 0u, nullptr, 1u,
2509 &preClearBarrier);
2510
2511 vk::CmdClearColorImage(m_commandBuffer->handle(), image_input.handle(), VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL, &ccv, 1,
2512 &full_subresource_range);
2513
2514 const VkImageMemoryBarrier postClearBarrier = {
2515 VK_STRUCTURE_TYPE_IMAGE_MEMORY_BARRIER,
2516 0,
2517 VK_ACCESS_TRANSFER_WRITE_BIT,
2518 VK_ACCESS_SHADER_READ_BIT,
2519 VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL,
2520 VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL,
2521 0,
2522 0,
2523 image_input.handle(),
2524 full_subresource_range,
2525 };
2526 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_FRAGMENT_SHADER_BIT, 0u, 0u, nullptr, 0u, nullptr,
2527 1u, &postClearBarrier);
2528
2529 m_renderPassBeginInfo.renderArea = {{0, 0}, {64, 64}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002530 m_renderPassBeginInfo.renderPass = rp.handle();
2531 m_renderPassBeginInfo.framebuffer = fb.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002532
2533 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2534 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_);
2535 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_layout_.handle(), 0, 1,
2536 &g_pipe.descriptor_set_->set_, 0, nullptr);
2537
2538 // Positive test for ordering rules between load and input attachment usage
2539 m_errorMonitor->ExpectSuccess();
2540 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
2541
2542 // Positive test for store ordering vs. input attachment and dependency *to* external for layout transition
2543 m_commandBuffer->EndRenderPass();
2544 m_errorMonitor->VerifyNotFound();
2545
2546 // Catch a conflict with the input attachment final layout transition
2547 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
2548 vk::CmdClearColorImage(m_commandBuffer->handle(), image_input.handle(), VK_IMAGE_LAYOUT_GENERAL, &ccv, 1,
2549 &full_subresource_range);
2550 m_errorMonitor->VerifyFound();
John Zulaufe972b752021-05-04 15:47:17 -06002551
2552 // There should be no hazard for ILT after ILT
2553 m_errorMonitor->ExpectSuccess();
2554 m_commandBuffer->end();
2555 m_commandBuffer->reset();
2556 m_commandBuffer->begin();
2557 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TOP_OF_PIPE_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0u, 0u, nullptr, 0u, nullptr, 1u,
2558 &preClearBarrier);
2559 const VkImageMemoryBarrier wawBarrier = {
2560 VK_STRUCTURE_TYPE_IMAGE_MEMORY_BARRIER,
2561 0,
2562 VK_ACCESS_SHADER_READ_BIT,
2563 VK_ACCESS_SHADER_READ_BIT,
2564 VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL,
2565 VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL,
2566 0,
2567 0,
2568 image_input.handle(),
2569 full_subresource_range,
2570 };
2571 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "SYNC-HAZARD-WRITE_AFTER_WRITE");
2572 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_VERTEX_SHADER_BIT, VK_PIPELINE_STAGE_FRAGMENT_SHADER_BIT, 0u, 0u, nullptr, 0u,
2573 nullptr, 1u, &wawBarrier);
2574 m_errorMonitor->VerifyNotFound();
2575 m_commandBuffer->end();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002576}
2577
2578TEST_F(VkSyncValTest, SyncSubpassMultiDep) {
2579 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
2580 ASSERT_NO_FATAL_FAILURE(InitState());
2581 if (IsPlatform(kNexusPlayer)) {
2582 printf("%s This test should not run on Nexus Player\n", kSkipPrefix);
2583 return;
2584 }
2585
2586 ASSERT_NO_FATAL_FAILURE(InitRenderTarget());
2587
2588 VkImageUsageFlags usage_color = VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT | VK_IMAGE_USAGE_TRANSFER_SRC_BIT;
2589 VkImageUsageFlags usage_input =
2590 VK_IMAGE_USAGE_INPUT_ATTACHMENT_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT | VK_IMAGE_USAGE_SAMPLED_BIT;
2591 VkFormat format = VK_FORMAT_R8G8B8A8_UNORM;
2592 VkImageObj image_color(m_device), image_input(m_device);
2593 auto image_ci = VkImageObj::ImageCreateInfo2D(64, 64, 1, 1, format, usage_input, VK_IMAGE_TILING_OPTIMAL);
2594 image_input.InitNoLayout(image_ci);
2595 image_ci.usage = usage_color;
2596 image_color.InitNoLayout(image_ci);
2597 VkImageView view_input = image_input.targetView(format);
2598 VkImageView view_color = image_color.targetView(format);
2599 VkImageView attachments[] = {view_color, view_input};
2600 VkImageSubresourceRange full_subresource_range{VK_IMAGE_ASPECT_COLOR_BIT, 0, 1, 0, 1};
2601 VkImageSubresourceLayers mip_0_layer_0{VK_IMAGE_ASPECT_COLOR_BIT, 0, 0, 1};
2602 VkOffset3D image_zero{0, 0, 0};
2603 VkExtent3D image_size{64, 64, 1};
2604 VkImageCopy full_region{mip_0_layer_0, image_zero, mip_0_layer_0, image_zero, image_size};
2605
2606 const VkAttachmentDescription fbAttachment = {
2607 0u,
2608 format,
2609 VK_SAMPLE_COUNT_1_BIT,
2610 VK_ATTACHMENT_LOAD_OP_CLEAR,
2611 VK_ATTACHMENT_STORE_OP_STORE,
2612 VK_ATTACHMENT_LOAD_OP_DONT_CARE,
2613 VK_ATTACHMENT_STORE_OP_STORE,
2614 VK_IMAGE_LAYOUT_GENERAL,
2615 VK_IMAGE_LAYOUT_GENERAL,
2616 };
2617
2618 std::vector<VkAttachmentDescription> attachmentDescs;
2619 attachmentDescs.push_back(fbAttachment);
2620
2621 // Add it as a frame buffer attachment.
2622 const VkAttachmentDescription inputAttachment = {
2623 0u,
2624 format,
2625 VK_SAMPLE_COUNT_1_BIT,
2626 VK_ATTACHMENT_LOAD_OP_LOAD,
2627 VK_ATTACHMENT_STORE_OP_DONT_CARE,
2628 VK_ATTACHMENT_LOAD_OP_DONT_CARE,
2629 VK_ATTACHMENT_STORE_OP_DONT_CARE,
2630 VK_IMAGE_LAYOUT_GENERAL,
2631 VK_IMAGE_LAYOUT_GENERAL,
2632 };
2633 attachmentDescs.push_back(inputAttachment);
2634
2635 std::vector<VkAttachmentReference> inputAttachments;
2636 const VkAttachmentReference inputRef = {
2637 1u,
2638 VK_IMAGE_LAYOUT_GENERAL,
2639 };
2640 inputAttachments.push_back(inputRef);
2641
2642 const VkAttachmentReference colorRef = {
2643 0u,
2644 VK_IMAGE_LAYOUT_GENERAL,
2645 };
2646 const std::vector<VkAttachmentReference> colorAttachments(1u, colorRef);
2647
2648 const VkSubpassDescription subpass = {
2649 0u,
2650 VK_PIPELINE_BIND_POINT_GRAPHICS,
2651 static_cast<uint32_t>(inputAttachments.size()),
2652 inputAttachments.data(),
2653 static_cast<uint32_t>(colorAttachments.size()),
2654 colorAttachments.data(),
2655 0u,
2656 nullptr,
2657 0u,
2658 nullptr,
2659 };
2660 const std::vector<VkSubpassDescription> subpasses(1u, subpass);
2661
2662 std::vector<VkSubpassDependency> subpass_dep_postive;
2663 subpass_dep_postive.push_back({VK_SUBPASS_EXTERNAL, 0, VK_PIPELINE_STAGE_TRANSFER_BIT,
2664 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT, 0, VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT,
2665 VK_DEPENDENCY_VIEW_LOCAL_BIT});
2666 subpass_dep_postive.push_back({VK_SUBPASS_EXTERNAL, 0, VK_PIPELINE_STAGE_TRANSFER_BIT,
2667 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT, VK_ACCESS_TRANSFER_WRITE_BIT,
2668 VK_ACCESS_COLOR_ATTACHMENT_READ_BIT, VK_DEPENDENCY_VIEW_LOCAL_BIT});
2669 subpass_dep_postive.push_back({0, VK_SUBPASS_EXTERNAL, VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT,
2670 VK_PIPELINE_STAGE_TRANSFER_BIT, VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT,
2671 VK_ACCESS_TRANSFER_READ_BIT, VK_DEPENDENCY_VIEW_LOCAL_BIT});
2672 subpass_dep_postive.push_back({0, VK_SUBPASS_EXTERNAL, VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT,
2673 VK_PIPELINE_STAGE_TRANSFER_BIT, VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT,
2674 VK_ACCESS_TRANSFER_WRITE_BIT, VK_DEPENDENCY_VIEW_LOCAL_BIT});
2675
2676 VkRenderPassCreateInfo renderPassInfo = {
2677 VK_STRUCTURE_TYPE_RENDER_PASS_CREATE_INFO,
2678 nullptr,
2679 0u,
2680 static_cast<uint32_t>(attachmentDescs.size()),
2681 attachmentDescs.data(),
2682 static_cast<uint32_t>(subpasses.size()),
2683 subpasses.data(),
2684 static_cast<uint32_t>(subpass_dep_postive.size()),
2685 subpass_dep_postive.data(),
2686 };
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002687 vk_testing::RenderPass rp_positive;
2688 rp_positive.init(*m_device, renderPassInfo);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002689
2690 std::vector<VkSubpassDependency> subpass_dep_negative;
2691 subpass_dep_negative.push_back({VK_SUBPASS_EXTERNAL, 0, VK_PIPELINE_STAGE_TRANSFER_BIT,
2692 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT, 0, VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT,
2693 VK_DEPENDENCY_VIEW_LOCAL_BIT});
2694 // Show that the two barriers do *not* chain by breaking the positive barrier into two bits.
2695 subpass_dep_negative.push_back({VK_SUBPASS_EXTERNAL, 0, VK_PIPELINE_STAGE_TRANSFER_BIT,
2696 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT, VK_ACCESS_TRANSFER_WRITE_BIT, 0,
2697 VK_DEPENDENCY_VIEW_LOCAL_BIT});
2698 subpass_dep_negative.push_back({VK_SUBPASS_EXTERNAL, 0, VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT,
2699 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT, 0, VK_ACCESS_COLOR_ATTACHMENT_READ_BIT,
2700 VK_DEPENDENCY_VIEW_LOCAL_BIT});
2701
2702 renderPassInfo.dependencyCount = static_cast<uint32_t>(subpass_dep_negative.size());
2703 renderPassInfo.pDependencies = subpass_dep_negative.data();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002704 vk_testing::RenderPass rp_negative;
2705 rp_negative.init(*m_device, renderPassInfo);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002706
2707 // rp_postive and rp_negative should be compatible for the same fb object
2708 const VkFramebufferCreateInfo fbci = {
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002709 VK_STRUCTURE_TYPE_FRAMEBUFFER_CREATE_INFO, 0, 0u, rp_positive.handle(), 2u, attachments, 64, 64, 1u,
Jeremy Gebben170781d2020-11-19 16:21:21 -07002710 };
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002711 vk_testing::Framebuffer fb;
2712 fb.init(*m_device, fbci);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002713
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002714 vk_testing::Sampler sampler;
Jeremy Gebben170781d2020-11-19 16:21:21 -07002715 VkSamplerCreateInfo sampler_info = SafeSaneSamplerCreateInfo();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002716 sampler.init(*m_device, sampler_info);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002717
sfricke-samsung1c0b96a2021-07-08 22:24:09 -07002718 char const *fsSource = R"glsl(
2719 #version 450
2720 layout(input_attachment_index=0, set=0, binding=0) uniform subpassInput x;
2721 void main() {
2722 vec4 color = subpassLoad(x);
2723 }
2724 )glsl";
Jeremy Gebben170781d2020-11-19 16:21:21 -07002725
sfricke-samsungae54c1e2022-01-21 05:35:21 -08002726 VkShaderObj vs(this, bindStateVertShaderText, VK_SHADER_STAGE_VERTEX_BIT);
2727 VkShaderObj fs(this, fsSource, VK_SHADER_STAGE_FRAGMENT_BIT);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002728
2729 CreatePipelineHelper g_pipe(*this);
2730 g_pipe.InitInfo();
2731 g_pipe.shader_stages_ = {vs.GetStageCreateInfo(), fs.GetStageCreateInfo()};
2732 g_pipe.dsl_bindings_ = {{0, VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT, 1, VK_SHADER_STAGE_FRAGMENT_BIT, nullptr}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002733 g_pipe.gp_ci_.renderPass = rp_positive.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002734 g_pipe.InitState();
2735 ASSERT_VK_SUCCESS(g_pipe.CreateGraphicsPipeline());
2736
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002737 g_pipe.descriptor_set_->WriteDescriptorImageInfo(0, view_input, sampler.handle(), VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT);
Jeremy Gebben170781d2020-11-19 16:21:21 -07002738 g_pipe.descriptor_set_->UpdateDescriptorSets();
2739
2740 m_commandBuffer->begin();
2741 auto cb = m_commandBuffer->handle();
2742 VkClearColorValue ccv = {};
2743
2744 const VkImageMemoryBarrier xferDestBarrier = {VK_STRUCTURE_TYPE_IMAGE_MEMORY_BARRIER,
2745 nullptr,
2746 VK_ACCESS_MEMORY_READ_BIT | VK_ACCESS_MEMORY_WRITE_BIT,
2747 VK_ACCESS_TRANSFER_WRITE_BIT,
2748 VK_IMAGE_LAYOUT_GENERAL,
2749 VK_IMAGE_LAYOUT_GENERAL,
2750 VK_QUEUE_FAMILY_IGNORED,
2751 VK_QUEUE_FAMILY_IGNORED,
2752 VK_NULL_HANDLE,
2753 full_subresource_range};
2754 const VkImageMemoryBarrier xferDestToSrcBarrier = {
2755 VK_STRUCTURE_TYPE_IMAGE_MEMORY_BARRIER,
2756 nullptr,
2757 VK_ACCESS_TRANSFER_WRITE_BIT,
2758 VK_ACCESS_TRANSFER_READ_BIT,
2759 VK_IMAGE_LAYOUT_GENERAL,
2760 VK_IMAGE_LAYOUT_GENERAL,
2761 VK_QUEUE_FAMILY_IGNORED,
2762 VK_QUEUE_FAMILY_IGNORED,
2763 VK_NULL_HANDLE,
2764 full_subresource_range,
2765 };
2766
2767 VkImageMemoryBarrier preClearBarrier = xferDestBarrier;
2768 preClearBarrier.image = image_color.handle();
2769
2770 VkImageMemoryBarrier preCopyBarriers[2] = {xferDestToSrcBarrier, xferDestBarrier};
2771 preCopyBarriers[0].image = image_color.handle();
2772 preCopyBarriers[1].image = image_input.handle();
2773 // Positive test for ordering rules between load and input attachment usage
2774 m_errorMonitor->ExpectSuccess();
2775
2776 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TOP_OF_PIPE_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0u, 0u, nullptr, 0u, nullptr, 1u,
2777 &preClearBarrier);
2778
2779 vk::CmdClearColorImage(m_commandBuffer->handle(), image_color.handle(), VK_IMAGE_LAYOUT_GENERAL, &ccv, 1,
2780 &full_subresource_range);
2781
2782 vk::CmdPipelineBarrier(cb, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0u, 0u, nullptr, 0u, nullptr, 2u,
2783 preCopyBarriers);
2784
2785 vk::CmdCopyImage(m_commandBuffer->handle(), image_color.handle(), VK_IMAGE_LAYOUT_GENERAL, image_input.handle(),
2786 VK_IMAGE_LAYOUT_GENERAL, 1u, &full_region);
2787
2788 // No post copy image barrier, we are testing the subpass dependencies
2789
2790 m_renderPassBeginInfo.renderArea = {{0, 0}, {64, 64}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002791 m_renderPassBeginInfo.renderPass = rp_positive.handle();
2792 m_renderPassBeginInfo.framebuffer = fb.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002793
2794 // Postive renderpass multidependency test
2795 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2796 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_);
2797 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe.pipeline_layout_.handle(), 0, 1,
2798 &g_pipe.descriptor_set_->set_, 0, nullptr);
2799
2800 vk::CmdDraw(m_commandBuffer->handle(), 1, 0, 0, 0);
2801
2802 // Positive test for store ordering vs. input attachment and dependency *to* external for layout transition
2803 m_commandBuffer->EndRenderPass();
2804 // m_errorMonitor->VerifyNotFound();
2805
2806 vk::CmdCopyImage(m_commandBuffer->handle(), image_color.handle(), VK_IMAGE_LAYOUT_GENERAL, image_input.handle(),
2807 VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL, 1u, &full_region);
2808 m_errorMonitor->VerifyNotFound();
2809
2810 m_renderPassBeginInfo.renderArea = {{0, 0}, {64, 64}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002811 m_renderPassBeginInfo.renderPass = rp_negative.handle();
2812 m_renderPassBeginInfo.framebuffer = fb.handle();
Jeremy Gebben170781d2020-11-19 16:21:21 -07002813
2814 // Postive renderpass multidependency test, will fail IFF the dependencies are acting indepently.
2815 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "SYNC-HAZARD-READ_AFTER_WRITE");
2816 m_commandBuffer->BeginRenderPass(m_renderPassBeginInfo);
2817 m_errorMonitor->VerifyFound();
2818}
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002819
2820TEST_F(VkSyncValTest, RenderPassAsyncHazard) {
2821 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
2822 ASSERT_NO_FATAL_FAILURE(InitState());
2823
Nathaniel Cesariof9cd1a82021-07-24 08:48:55 -06002824 if (IsPlatform(kPixel3) || IsPlatform(kPixel3aXL)) {
2825 printf("%s Temporarily disabling on Pixel 3 and Pixel 3a XL due to driver crash\n", kSkipPrefix);
2826 return;
2827 }
2828
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002829 // overall set up:
2830 // subpass 0:
2831 // write image 0
2832 // subpass 1:
2833 // read image 0
2834 // write image 1
2835 // subpass 2:
2836 // read image 0
2837 // write image 2
2838 // subpass 3:
2839 // read image 0
2840 // write image 3
2841 //
2842 // subpasses 1 & 2 can run in parallel but both should depend on 0
2843 // subpass 3 must run after 1 & 2 because otherwise the store operation will
2844 // race with the reads in the other subpasses.
2845
2846 constexpr VkFormat kFormat = VK_FORMAT_R8G8B8A8_UNORM;
2847 constexpr uint32_t kWidth = 32, kHeight = 32;
2848 constexpr uint32_t kNumImages = 4;
2849
2850 VkImageCreateInfo src_img_info = {};
2851 src_img_info.sType = VK_STRUCTURE_TYPE_IMAGE_CREATE_INFO;
2852 src_img_info.pNext = NULL;
2853 src_img_info.flags = 0;
2854 src_img_info.imageType = VK_IMAGE_TYPE_2D;
2855 src_img_info.format = kFormat;
2856 src_img_info.extent = {kWidth, kHeight, 1};
2857 src_img_info.mipLevels = 1;
2858 src_img_info.arrayLayers = 1;
2859 src_img_info.samples = VK_SAMPLE_COUNT_2_BIT;
2860 src_img_info.tiling = VK_IMAGE_TILING_OPTIMAL;
2861 src_img_info.usage = VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT | VK_IMAGE_USAGE_INPUT_ATTACHMENT_BIT;
2862 src_img_info.sharingMode = VK_SHARING_MODE_EXCLUSIVE;
2863 src_img_info.queueFamilyIndexCount = 0;
2864 src_img_info.pQueueFamilyIndices = nullptr;
2865 src_img_info.initialLayout = VK_IMAGE_LAYOUT_UNDEFINED;
2866
2867 VkImageCreateInfo dst_img_info = {};
2868 dst_img_info.sType = VK_STRUCTURE_TYPE_IMAGE_CREATE_INFO;
2869 dst_img_info.pNext = nullptr;
2870 dst_img_info.flags = 0;
2871 dst_img_info.imageType = VK_IMAGE_TYPE_2D;
2872 dst_img_info.format = kFormat;
2873 dst_img_info.extent = {kWidth, kHeight, 1};
2874 dst_img_info.mipLevels = 1;
2875 dst_img_info.arrayLayers = 1;
2876 dst_img_info.samples = VK_SAMPLE_COUNT_1_BIT;
2877 dst_img_info.tiling = VK_IMAGE_TILING_OPTIMAL;
2878 dst_img_info.usage = VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT;
2879 dst_img_info.sharingMode = VK_SHARING_MODE_EXCLUSIVE;
2880 dst_img_info.queueFamilyIndexCount = 0;
2881 dst_img_info.pQueueFamilyIndices = nullptr;
2882 dst_img_info.initialLayout = VK_IMAGE_LAYOUT_UNDEFINED;
2883
2884 std::vector<std::unique_ptr<VkImageObj>> images;
2885 for (uint32_t i = 0; i < kNumImages; i++) {
2886 images.emplace_back(new VkImageObj(m_device));
2887 }
2888 images[0]->Init(src_img_info);
2889 for (uint32_t i = 1; i < images.size(); i++) {
2890 images[i]->Init(dst_img_info);
2891 }
2892
2893 std::array<VkImageView, kNumImages> attachments{};
2894 std::array<VkAttachmentDescription, kNumImages> attachment_descriptions{};
2895 std::array<VkAttachmentReference, kNumImages> color_refs{};
2896 std::array<VkImageMemoryBarrier, kNumImages> img_barriers{};
2897
2898 for (uint32_t i = 0; i < attachments.size(); i++) {
2899 attachments[i] = images[i]->targetView(kFormat);
2900 attachment_descriptions[i] = {};
2901 attachment_descriptions[i].flags = 0;
2902 attachment_descriptions[i].format = kFormat;
2903 attachment_descriptions[i].samples = VK_SAMPLE_COUNT_1_BIT;
2904 attachment_descriptions[i].loadOp = VK_ATTACHMENT_LOAD_OP_CLEAR;
2905 attachment_descriptions[i].storeOp = VK_ATTACHMENT_STORE_OP_STORE;
2906 attachment_descriptions[i].stencilLoadOp = VK_ATTACHMENT_LOAD_OP_DONT_CARE;
2907 attachment_descriptions[i].stencilStoreOp = VK_ATTACHMENT_STORE_OP_DONT_CARE;
2908 attachment_descriptions[i].initialLayout = VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL;
2909 attachment_descriptions[i].finalLayout =
2910 (i == 0) ? VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL : VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL;
2911
2912 color_refs[i] = {i, VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL};
2913
2914 img_barriers[i].sType = VK_STRUCTURE_TYPE_IMAGE_MEMORY_BARRIER;
2915 img_barriers[i].srcAccessMask = 0;
2916 img_barriers[i].dstAccessMask = VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT;
2917 img_barriers[i].oldLayout = VK_IMAGE_LAYOUT_UNDEFINED;
2918 img_barriers[i].newLayout = VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL;
2919 img_barriers[i].srcQueueFamilyIndex = VK_QUEUE_FAMILY_IGNORED;
2920 img_barriers[i].dstQueueFamilyIndex = VK_QUEUE_FAMILY_IGNORED;
2921 img_barriers[i].image = images[i]->handle();
2922 img_barriers[i].subresourceRange = {VK_IMAGE_ASPECT_COLOR_BIT, 0, VK_REMAINING_MIP_LEVELS, 0, VK_REMAINING_ARRAY_LAYERS};
2923 }
2924
2925 const VkAttachmentReference input_ref{0u, VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL};
2926
2927 std::array<std::array<uint32_t, 2>, kNumImages - 1> preserve_subpass{{{2, 3}, {1, 3}, {1, 2}}};
2928
2929 std::array<VkSubpassDescription, kNumImages> subpasses{};
2930
2931 subpasses[0].pipelineBindPoint = VK_PIPELINE_BIND_POINT_GRAPHICS;
2932 subpasses[0].inputAttachmentCount = 0;
2933 subpasses[0].pInputAttachments = nullptr;
2934 subpasses[0].colorAttachmentCount = 1;
2935 subpasses[0].pColorAttachments = &color_refs[0];
2936
2937 for (uint32_t i = 1; i < subpasses.size(); i++) {
2938 subpasses[i].pipelineBindPoint = VK_PIPELINE_BIND_POINT_GRAPHICS;
2939 subpasses[i].inputAttachmentCount = 1;
2940 subpasses[i].pInputAttachments = &input_ref;
2941 subpasses[i].colorAttachmentCount = 1;
2942 subpasses[i].pColorAttachments = &color_refs[1];
2943 subpasses[i].preserveAttachmentCount = preserve_subpass[i - 1].size();
2944 subpasses[i].pPreserveAttachments = preserve_subpass[i - 1].data();
2945 }
2946
2947 VkRenderPassCreateInfo renderpass_info = {};
2948 renderpass_info.sType = VK_STRUCTURE_TYPE_RENDER_PASS_CREATE_INFO;
2949 renderpass_info.pNext = nullptr;
2950 renderpass_info.flags = 0;
2951 renderpass_info.attachmentCount = attachment_descriptions.size();
2952 renderpass_info.pAttachments = attachment_descriptions.data();
2953 renderpass_info.subpassCount = subpasses.size();
2954 renderpass_info.pSubpasses = subpasses.data();
2955 renderpass_info.dependencyCount = 0;
2956 renderpass_info.pDependencies = nullptr;
2957
2958 VkFramebufferCreateInfo fbci = {};
2959 fbci.sType = VK_STRUCTURE_TYPE_FRAMEBUFFER_CREATE_INFO;
2960 fbci.pNext = nullptr;
2961 fbci.flags = 0;
2962 fbci.attachmentCount = attachments.size();
2963 fbci.pAttachments = attachments.data();
2964 fbci.width = kWidth;
2965 fbci.height = kHeight;
2966 fbci.layers = 1;
2967
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002968 vk_testing::Sampler sampler;
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002969 VkSamplerCreateInfo sampler_info = SafeSaneSamplerCreateInfo();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002970 sampler.init(*m_device, sampler_info);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002971
sfricke-samsung1c0b96a2021-07-08 22:24:09 -07002972 char const *fsSource = R"glsl(
2973 #version 450
2974 layout(input_attachment_index=0, set=0, binding=0) uniform subpassInput x;
2975 void main() {
2976 vec4 color = subpassLoad(x);
2977 }
2978 )glsl";
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002979
sfricke-samsungae54c1e2022-01-21 05:35:21 -08002980 VkShaderObj vs(this, bindStateVertShaderText, VK_SHADER_STAGE_VERTEX_BIT);
2981 VkShaderObj fs(this, fsSource, VK_SHADER_STAGE_FRAGMENT_BIT);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002982
2983 VkClearValue clear = {};
2984 clear.color = m_clear_color;
Tony-LunarG73f37032021-06-07 11:47:03 -06002985 std::array<VkClearValue, 4> clear_values = {{clear, clear, clear, clear}};
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002986
2987 // run the renderpass with no dependencies
2988 {
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002989 vk_testing::RenderPass rp;
2990 vk_testing::Framebuffer fb;
2991 rp.init(*m_device, renderpass_info);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002992
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002993 fbci.renderPass = rp.handle();
2994 fb.init(*m_device, fbci);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002995
2996 CreatePipelineHelper g_pipe_0(*this);
2997 g_pipe_0.InitInfo();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06002998 g_pipe_0.gp_ci_.renderPass = rp.handle();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07002999 g_pipe_0.InitState();
3000 ASSERT_VK_SUCCESS(g_pipe_0.CreateGraphicsPipeline());
3001
3002 CreatePipelineHelper g_pipe_12(*this);
3003 g_pipe_12.InitInfo();
3004 g_pipe_12.shader_stages_ = {vs.GetStageCreateInfo(), fs.GetStageCreateInfo()};
3005 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 -06003006 g_pipe_12.gp_ci_.renderPass = rp.handle();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003007 g_pipe_12.InitState();
3008 ASSERT_VK_SUCCESS(g_pipe_12.CreateGraphicsPipeline());
3009
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003010 g_pipe_12.descriptor_set_->WriteDescriptorImageInfo(0, attachments[0], sampler.handle(), VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003011 g_pipe_12.descriptor_set_->UpdateDescriptorSets();
3012
3013 m_commandBuffer->begin();
3014
3015 vk::CmdPipelineBarrier(m_commandBuffer->handle(), VK_PIPELINE_STAGE_TOP_OF_PIPE_BIT,
3016 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT, 0, 0, nullptr, 0, nullptr, img_barriers.size(),
3017 img_barriers.data());
3018
3019 m_renderPassBeginInfo.renderArea = {{0, 0}, {16, 16}};
3020 m_renderPassBeginInfo.pClearValues = clear_values.data();
3021 m_renderPassBeginInfo.clearValueCount = clear_values.size();
3022
3023 m_renderPassBeginInfo.renderArea = {{0, 0}, {kWidth, kHeight}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003024 m_renderPassBeginInfo.renderPass = rp.handle();
3025 m_renderPassBeginInfo.framebuffer = fb.handle();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003026
3027 vk::CmdBeginRenderPass(m_commandBuffer->handle(), &m_renderPassBeginInfo, VK_SUBPASS_CONTENTS_INLINE);
3028 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_0.pipeline_);
3029 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_0.pipeline_layout_.handle(), 0,
3030 1, &g_pipe_0.descriptor_set_->set_, 0, NULL);
3031
3032 vk::CmdDraw(m_commandBuffer->handle(), 3, 1, 0, 0);
3033
3034 for (uint32_t i = 1; i < subpasses.size(); i++) {
3035 vk::CmdNextSubpass(m_commandBuffer->handle(), VK_SUBPASS_CONTENTS_INLINE);
3036 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_12.pipeline_);
3037 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS,
3038 g_pipe_12.pipeline_layout_.handle(), 0, 1, &g_pipe_12.descriptor_set_->set_, 0, NULL);
3039
3040 // we're racing the writes from subpass 0 with our shader reads
3041 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-READ-RACING-WRITE");
3042 vk::CmdDraw(m_commandBuffer->handle(), 3, 1, 0, 0);
3043 m_errorMonitor->VerifyFound();
3044 }
3045
3046 // we should get an error from async checking in both subpasses 2 & 3
3047 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE-RACING-WRITE");
3048 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE-RACING-WRITE");
3049 vk::CmdEndRenderPass(m_commandBuffer->handle());
3050 m_errorMonitor->VerifyFound();
3051
3052 m_commandBuffer->end();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003053 }
3054
3055 // add dependencies from subpass 0 to the others, which are necessary but not sufficient
3056 std::vector<VkSubpassDependency> subpass_dependencies;
3057 for (uint32_t i = 1; i < subpasses.size(); i++) {
3058 VkSubpassDependency dep{0,
3059 i,
3060 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT,
3061 VK_PIPELINE_STAGE_FRAGMENT_SHADER_BIT,
3062 VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT,
3063 VK_ACCESS_INPUT_ATTACHMENT_READ_BIT,
3064 0};
3065 subpass_dependencies.push_back(dep);
3066 }
3067 renderpass_info.dependencyCount = subpass_dependencies.size();
3068 renderpass_info.pDependencies = subpass_dependencies.data();
3069
3070 {
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003071 vk_testing::RenderPass rp;
3072 vk_testing::Framebuffer fb;
3073 rp.init(*m_device, renderpass_info);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003074
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003075 fbci.renderPass = rp.handle();
3076 fb.init(*m_device, fbci);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003077
3078 CreatePipelineHelper g_pipe_0(*this);
3079 g_pipe_0.InitInfo();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003080 g_pipe_0.gp_ci_.renderPass = rp.handle();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003081 g_pipe_0.InitState();
3082 ASSERT_VK_SUCCESS(g_pipe_0.CreateGraphicsPipeline());
3083
3084 CreatePipelineHelper g_pipe_12(*this);
3085 g_pipe_12.InitInfo();
3086 g_pipe_12.shader_stages_ = {vs.GetStageCreateInfo(), fs.GetStageCreateInfo()};
3087 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 -06003088 g_pipe_12.gp_ci_.renderPass = rp.handle();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003089 g_pipe_12.InitState();
3090 ASSERT_VK_SUCCESS(g_pipe_12.CreateGraphicsPipeline());
3091
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003092 g_pipe_12.descriptor_set_->WriteDescriptorImageInfo(0, attachments[0], sampler.handle(), VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003093 g_pipe_12.descriptor_set_->UpdateDescriptorSets();
3094
3095 m_commandBuffer->begin();
3096
3097 vk::CmdPipelineBarrier(m_commandBuffer->handle(), VK_PIPELINE_STAGE_TOP_OF_PIPE_BIT,
3098 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT, 0, 0, nullptr, 0, nullptr, img_barriers.size(),
3099 img_barriers.data());
3100
3101 m_renderPassBeginInfo.renderArea = {{0, 0}, {16, 16}};
3102 m_renderPassBeginInfo.pClearValues = clear_values.data();
3103 m_renderPassBeginInfo.clearValueCount = clear_values.size();
3104
3105 m_renderPassBeginInfo.renderArea = {{0, 0}, {kWidth, kHeight}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003106 m_renderPassBeginInfo.renderPass = rp.handle();
3107 m_renderPassBeginInfo.framebuffer = fb.handle();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003108
3109 vk::CmdBeginRenderPass(m_commandBuffer->handle(), &m_renderPassBeginInfo, VK_SUBPASS_CONTENTS_INLINE);
3110 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_0.pipeline_);
3111 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_0.pipeline_layout_.handle(), 0,
3112 1, &g_pipe_0.descriptor_set_->set_, 0, NULL);
3113
3114 vk::CmdDraw(m_commandBuffer->handle(), 3, 1, 0, 0);
3115
3116 m_errorMonitor->ExpectSuccess();
3117 for (uint32_t i = 1; i < subpasses.size(); i++) {
3118 vk::CmdNextSubpass(m_commandBuffer->handle(), VK_SUBPASS_CONTENTS_INLINE);
3119 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_12.pipeline_);
3120 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS,
3121 g_pipe_12.pipeline_layout_.handle(), 0, 1, &g_pipe_12.descriptor_set_->set_, 0, NULL);
3122 vk::CmdDraw(m_commandBuffer->handle(), 3, 1, 0, 0);
3123 }
3124 m_errorMonitor->VerifyNotFound();
3125 // expect this error because 2 subpasses could try to do the store operation
3126 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE-RACING-WRITE");
3127 // ... and this one because the store could happen during a shader read from another subpass
3128 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE-RACING-READ");
3129 vk::CmdEndRenderPass(m_commandBuffer->handle());
3130 m_errorMonitor->VerifyFound();
3131
3132 m_commandBuffer->end();
3133
3134 m_errorMonitor->VerifyFound();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003135 }
3136
3137 // try again with correct dependencies to make subpass 3 depend on 1 & 2
3138 for (uint32_t i = 1; i < (subpasses.size() - 1); i++) {
3139 VkSubpassDependency dep{i,
3140 static_cast<uint32_t>(subpasses.size() - 1),
3141 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT,
3142 VK_PIPELINE_STAGE_FRAGMENT_SHADER_BIT,
3143 VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT,
3144 VK_ACCESS_INPUT_ATTACHMENT_READ_BIT,
3145 0};
3146 subpass_dependencies.push_back(dep);
3147 }
3148 renderpass_info.dependencyCount = subpass_dependencies.size();
3149 renderpass_info.pDependencies = subpass_dependencies.data();
3150 {
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003151 vk_testing::RenderPass rp;
3152 vk_testing::Framebuffer fb;
3153 rp.init(*m_device, renderpass_info);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003154
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003155 fbci.renderPass = rp.handle();
3156 fb.init(*m_device, fbci);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003157
3158 CreatePipelineHelper g_pipe_0(*this);
3159 g_pipe_0.InitInfo();
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003160 g_pipe_0.gp_ci_.renderPass = rp.handle();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003161 g_pipe_0.InitState();
3162 ASSERT_VK_SUCCESS(g_pipe_0.CreateGraphicsPipeline());
3163
3164 CreatePipelineHelper g_pipe_12(*this);
3165 g_pipe_12.InitInfo();
3166 g_pipe_12.shader_stages_ = {vs.GetStageCreateInfo(), fs.GetStageCreateInfo()};
3167 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 -06003168 g_pipe_12.gp_ci_.renderPass = rp.handle();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003169 g_pipe_12.InitState();
3170 ASSERT_VK_SUCCESS(g_pipe_12.CreateGraphicsPipeline());
3171
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003172 g_pipe_12.descriptor_set_->WriteDescriptorImageInfo(0, attachments[0], sampler.handle(), VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT);
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003173 g_pipe_12.descriptor_set_->UpdateDescriptorSets();
3174
3175 m_errorMonitor->ExpectSuccess();
3176 m_commandBuffer->begin();
3177 vk::CmdPipelineBarrier(m_commandBuffer->handle(), VK_PIPELINE_STAGE_TOP_OF_PIPE_BIT,
3178 VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT, 0, 0, nullptr, 0, nullptr, img_barriers.size(),
3179 img_barriers.data());
3180
3181 m_renderPassBeginInfo.renderArea = {{0, 0}, {16, 16}};
3182 m_renderPassBeginInfo.pClearValues = clear_values.data();
3183 m_renderPassBeginInfo.clearValueCount = clear_values.size();
3184
3185 m_renderPassBeginInfo.renderArea = {{0, 0}, {kWidth, kHeight}};
Jeremy Gebben18ac1052021-08-12 11:07:32 -06003186 m_renderPassBeginInfo.renderPass = rp.handle();
3187 m_renderPassBeginInfo.framebuffer = fb.handle();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003188
3189 vk::CmdBeginRenderPass(m_commandBuffer->handle(), &m_renderPassBeginInfo, VK_SUBPASS_CONTENTS_INLINE);
3190 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_0.pipeline_);
3191 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_0.pipeline_layout_.handle(), 0,
3192 1, &g_pipe_0.descriptor_set_->set_, 0, NULL);
3193
3194 vk::CmdDraw(m_commandBuffer->handle(), 3, 1, 0, 0);
3195
3196 for (uint32_t i = 1; i < subpasses.size(); i++) {
3197 vk::CmdNextSubpass(m_commandBuffer->handle(), VK_SUBPASS_CONTENTS_INLINE);
3198 vk::CmdBindPipeline(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS, g_pipe_12.pipeline_);
3199 vk::CmdBindDescriptorSets(m_commandBuffer->handle(), VK_PIPELINE_BIND_POINT_GRAPHICS,
3200 g_pipe_12.pipeline_layout_.handle(), 0, 1, &g_pipe_12.descriptor_set_->set_, 0, NULL);
3201 vk::CmdDraw(m_commandBuffer->handle(), 3, 1, 0, 0);
3202 }
3203
3204 vk::CmdEndRenderPass(m_commandBuffer->handle());
3205
3206 m_commandBuffer->end();
3207
3208 m_errorMonitor->VerifyNotFound();
Jeremy Gebben6feafd42020-11-30 09:11:38 -07003209 }
3210}
John Zulauf025ee442020-12-15 11:44:19 -07003211
3212TEST_F(VkSyncValTest, SyncEventsBufferCopy) {
3213 TEST_DESCRIPTION("Check Set/Wait protection for a variety of use cases using buffer copies");
3214 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
3215 ASSERT_NO_FATAL_FAILURE(InitState(nullptr, nullptr, VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT));
3216
3217 VkBufferObj buffer_a;
3218 VkBufferObj buffer_b;
3219 VkBufferObj buffer_c;
3220 VkMemoryPropertyFlags mem_prop = VK_MEMORY_PROPERTY_DEVICE_LOCAL_BIT;
3221 buffer_a.init_as_src_and_dst(*m_device, 256, mem_prop);
3222 buffer_b.init_as_src_and_dst(*m_device, 256, mem_prop);
3223 buffer_c.init_as_src_and_dst(*m_device, 256, mem_prop);
3224
3225 VkBufferCopy region = {0, 0, 256};
3226 VkBufferCopy front2front = {0, 0, 128};
3227 VkBufferCopy front2back = {0, 128, 128};
3228 VkBufferCopy back2back = {128, 128, 128};
3229
3230 VkEventObj event;
3231 event.init(*m_device, VkEventObj::create_info(0));
3232 VkEvent event_handle = event.handle();
3233
3234 auto cb = m_commandBuffer->handle();
3235 m_commandBuffer->begin();
3236
3237 // Copy after set for WAR (note we are writing to the back half of c but only reading from the front
3238 m_errorMonitor->ExpectSuccess();
3239 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &region);
3240 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3241 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_c.handle(), 1, &back2back);
3242 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, nullptr, 0,
3243 nullptr, 0, nullptr);
3244 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_a.handle(), 1, &front2front);
3245 m_errorMonitor->VerifyNotFound();
3246 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
3247 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_a.handle(), 1, &front2back);
3248 m_errorMonitor->VerifyFound();
3249 m_commandBuffer->end();
3250
3251 // WAR prevented
3252 m_commandBuffer->reset();
3253 m_commandBuffer->begin();
3254 m_errorMonitor->ExpectSuccess();
3255 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &region);
3256 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3257 // Just protect against WAR, only need a sync barrier.
3258 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, nullptr, 0,
3259 nullptr, 0, nullptr);
3260 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_a.handle(), 1, &region);
3261 m_errorMonitor->VerifyNotFound();
3262
3263 // Wait shouldn't prevent this WAW though, as it's only a synchronization barrier
3264 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
3265 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_b.handle(), 1, &region);
3266 m_errorMonitor->VerifyFound();
3267 m_commandBuffer->end();
3268
3269 // Prevent WAR and WAW
3270 m_commandBuffer->reset();
3271 m_commandBuffer->begin();
3272 m_errorMonitor->ExpectSuccess();
3273 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &region);
3274 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
Mark Lobodzinski07d0a612020-12-30 15:42:31 -07003275 auto mem_barrier_waw = LvlInitStruct<VkMemoryBarrier>();
John Zulauf025ee442020-12-15 11:44:19 -07003276 mem_barrier_waw.srcAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3277 mem_barrier_waw.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3278 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 1,
3279 &mem_barrier_waw, 0, nullptr, 0, nullptr);
3280 // The WAW should be safe (on a memory barrier)
3281 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_b.handle(), 1, &region);
3282 // The WAR should also be safe (on a sync barrier)
3283 vk::CmdCopyBuffer(cb, buffer_c.handle(), buffer_a.handle(), 1, &region);
3284 m_errorMonitor->VerifyNotFound();
3285 m_commandBuffer->end();
3286
3287 // Barrier range check for WAW
Mark Lobodzinski07d0a612020-12-30 15:42:31 -07003288 auto buffer_barrier_front_waw = LvlInitStruct<VkBufferMemoryBarrier>();
John Zulauf025ee442020-12-15 11:44:19 -07003289 buffer_barrier_front_waw.srcAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3290 buffer_barrier_front_waw.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3291 buffer_barrier_front_waw.buffer = buffer_b.handle();
3292 buffer_barrier_front_waw.offset = front2front.dstOffset;
3293 buffer_barrier_front_waw.size = front2front.size;
3294
3295 // Front safe, back WAW
3296 m_commandBuffer->reset();
3297 m_commandBuffer->begin();
3298 m_errorMonitor->ExpectSuccess();
3299 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &region);
3300 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3301 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, nullptr, 1,
3302 &buffer_barrier_front_waw, 0, nullptr);
3303 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &front2front);
3304 m_errorMonitor->VerifyNotFound();
3305 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
3306 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &back2back);
3307 m_errorMonitor->VerifyFound();
3308 m_commandBuffer->end();
3309}
3310
3311TEST_F(VkSyncValTest, SyncEventsCopyImageHazards) {
3312 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
3313 ASSERT_NO_FATAL_FAILURE(InitState(nullptr, nullptr, VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT));
3314
3315 VkImageUsageFlags usage = VK_IMAGE_USAGE_TRANSFER_SRC_BIT | VK_IMAGE_USAGE_TRANSFER_DST_BIT;
3316 VkFormat format = VK_FORMAT_R8G8B8A8_UNORM;
3317 VkImageObj image_a(m_device);
3318 auto image_ci = VkImageObj::ImageCreateInfo2D(128, 128, 1, 2, format, usage, VK_IMAGE_TILING_OPTIMAL);
3319 image_a.Init(image_ci);
3320 ASSERT_TRUE(image_a.initialized());
3321
3322 VkImageObj image_b(m_device);
3323 image_b.Init(image_ci);
3324 ASSERT_TRUE(image_b.initialized());
3325
3326 VkImageObj image_c(m_device);
3327 image_c.Init(image_ci);
3328 ASSERT_TRUE(image_c.initialized());
3329
3330 VkEventObj event;
3331 event.init(*m_device, VkEventObj::create_info(0));
3332 VkEvent event_handle = event.handle();
3333
3334 VkImageSubresourceLayers layers_all{VK_IMAGE_ASPECT_COLOR_BIT, 0, 0, 2};
3335 VkImageSubresourceLayers layers_0{VK_IMAGE_ASPECT_COLOR_BIT, 0, 0, 1};
3336 VkImageSubresourceLayers layers_1{VK_IMAGE_ASPECT_COLOR_BIT, 0, 1, 1};
3337 VkImageSubresourceRange layers_0_subresource_range{VK_IMAGE_ASPECT_COLOR_BIT, 0, 1, 0, 1};
3338 VkOffset3D zero_offset{0, 0, 0};
3339 VkOffset3D half_offset{64, 64, 0};
3340 VkExtent3D full_extent{128, 128, 1}; // <-- image type is 2D
3341 VkExtent3D half_extent{64, 64, 1}; // <-- image type is 2D
3342
3343 VkImageCopy full_region = {layers_all, zero_offset, layers_all, zero_offset, full_extent};
3344 VkImageCopy region_0_to_0 = {layers_0, zero_offset, layers_0, zero_offset, full_extent};
3345 VkImageCopy region_1_to_1 = {layers_1, zero_offset, layers_1, zero_offset, full_extent};
3346 VkImageCopy region_0_q0toq0 = {layers_0, zero_offset, layers_0, zero_offset, half_extent};
3347 VkImageCopy region_0_q0toq3 = {layers_0, zero_offset, layers_0, half_offset, half_extent};
3348 VkImageCopy region_0_q3toq3 = {layers_0, half_offset, layers_0, half_offset, half_extent};
3349
3350 auto cb = m_commandBuffer->handle();
3351 auto copy_general = [cb](const VkImageObj &from, const VkImageObj &to, const VkImageCopy &region) {
3352 vk::CmdCopyImage(cb, from.handle(), VK_IMAGE_LAYOUT_GENERAL, to.handle(), VK_IMAGE_LAYOUT_GENERAL, 1, &region);
3353 };
3354
3355 auto set_layouts = [this, &image_a, &image_b, &image_c]() {
3356 image_c.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
3357 image_b.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
3358 image_a.SetLayout(m_commandBuffer, VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL);
3359 };
3360
John Zulaufdd462092020-12-18 12:00:35 -07003361 // Scope check. One access in, one access not
John Zulauf025ee442020-12-15 11:44:19 -07003362 m_commandBuffer->begin();
3363 set_layouts();
3364 m_errorMonitor->ExpectSuccess();
3365 copy_general(image_a, image_b, full_region);
3366 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3367 copy_general(image_a, image_c, region_0_q3toq3);
3368 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, nullptr, 0,
3369 nullptr, 0, nullptr);
3370 copy_general(image_c, image_a, region_0_q0toq0);
3371 m_errorMonitor->VerifyNotFound();
3372 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_READ");
3373 copy_general(image_c, image_a, region_0_q0toq3);
3374 m_errorMonitor->VerifyFound();
3375 m_commandBuffer->end();
3376
3377 // WAR prevented
3378 m_commandBuffer->reset();
3379 m_commandBuffer->begin();
3380 set_layouts();
3381 m_errorMonitor->ExpectSuccess();
3382 copy_general(image_a, image_b, full_region);
3383 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3384 // Just protect against WAR, only need a sync barrier.
3385 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, nullptr, 0,
3386 nullptr, 0, nullptr);
3387 copy_general(image_c, image_a, full_region);
3388 m_errorMonitor->VerifyNotFound();
3389
3390 // Wait shouldn't prevent this WAW though, as it's only a synchronization barrier
3391 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
3392 copy_general(image_c, image_b, full_region);
3393 m_errorMonitor->VerifyFound();
3394 m_commandBuffer->end();
3395
3396 // Prevent WAR and WAW
3397 m_commandBuffer->reset();
3398 m_commandBuffer->begin();
3399 m_errorMonitor->ExpectSuccess();
3400 set_layouts();
3401 copy_general(image_a, image_b, full_region);
3402 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
Mark Lobodzinski07d0a612020-12-30 15:42:31 -07003403 auto mem_barrier_waw = LvlInitStruct<VkMemoryBarrier>();
John Zulauf025ee442020-12-15 11:44:19 -07003404 mem_barrier_waw.srcAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3405 mem_barrier_waw.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3406 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 1,
3407 &mem_barrier_waw, 0, nullptr, 0, nullptr);
3408 // The WAW should be safe (on a memory barrier)
3409 copy_general(image_c, image_b, full_region);
3410 // The WAR should also be safe (on a sync barrier)
3411 copy_general(image_c, image_a, full_region);
3412 m_errorMonitor->VerifyNotFound();
3413 m_commandBuffer->end();
3414
3415 // Barrier range check for WAW
Mark Lobodzinski07d0a612020-12-30 15:42:31 -07003416 auto image_barrier_region0_waw = LvlInitStruct<VkImageMemoryBarrier>();
John Zulauf025ee442020-12-15 11:44:19 -07003417 image_barrier_region0_waw.srcAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3418 image_barrier_region0_waw.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3419 image_barrier_region0_waw.oldLayout = VK_IMAGE_LAYOUT_GENERAL;
3420 image_barrier_region0_waw.newLayout = VK_IMAGE_LAYOUT_GENERAL;
3421 image_barrier_region0_waw.srcQueueFamilyIndex = VK_QUEUE_FAMILY_IGNORED;
3422 image_barrier_region0_waw.dstQueueFamilyIndex = VK_QUEUE_FAMILY_IGNORED;
3423 image_barrier_region0_waw.image = image_b.handle();
3424 image_barrier_region0_waw.subresourceRange = layers_0_subresource_range;
3425
3426 // Region 0 safe, back WAW
3427 m_commandBuffer->reset();
3428 m_commandBuffer->begin();
3429 set_layouts();
3430 m_errorMonitor->ExpectSuccess();
3431 copy_general(image_a, image_b, full_region);
3432 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3433 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, nullptr, 0,
3434 nullptr, 1, &image_barrier_region0_waw);
3435 copy_general(image_a, image_b, region_0_to_0);
3436 m_errorMonitor->VerifyNotFound();
3437 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
3438 copy_general(image_a, image_b, region_1_to_1);
3439 m_errorMonitor->VerifyFound();
3440 m_commandBuffer->end();
3441}
John Zulauf4b5e4632020-12-15 11:48:59 -07003442
3443TEST_F(VkSyncValTest, SyncEventsCommandHazards) {
3444 TEST_DESCRIPTION("Check Set/Reset/Wait command hazard checking");
3445 ASSERT_NO_FATAL_FAILURE(InitSyncValFramework());
3446 ASSERT_NO_FATAL_FAILURE(InitState(nullptr, nullptr, VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT));
3447
3448 VkEventObj event;
3449 event.init(*m_device, VkEventObj::create_info(0));
3450
3451 const VkEvent event_handle = event.handle();
3452
3453 m_commandBuffer->begin();
3454 m_errorMonitor->ExpectSuccess();
3455 m_commandBuffer->ResetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3456 m_errorMonitor->VerifyNotFound();
3457
John Zulauf4edde622021-02-15 08:54:50 -07003458 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "VUID-vkCmdResetEvent-event-03834");
John Zulauf4b5e4632020-12-15 11:48:59 -07003459 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, nullptr, 0,
3460 nullptr, 0, nullptr);
3461 m_errorMonitor->VerifyFound();
3462 m_errorMonitor->ExpectSuccess();
3463 m_commandBuffer->end();
3464
3465 m_commandBuffer->begin();
3466 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3467 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_FRAGMENT_SHADER_BIT, 0, nullptr,
3468 0, nullptr, 0, nullptr);
3469 m_errorMonitor->VerifyNotFound();
3470 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-vkCmdResetEvent-missingbarrier-wait");
3471 m_commandBuffer->ResetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3472 m_errorMonitor->VerifyFound();
3473 m_errorMonitor->ExpectSuccess();
3474 m_commandBuffer->end();
3475
3476 m_commandBuffer->begin();
3477 m_commandBuffer->ResetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3478 m_errorMonitor->VerifyNotFound();
3479 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-vkCmdSetEvent-missingbarrier-reset");
3480 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3481 m_errorMonitor->VerifyFound();
3482
3483 m_errorMonitor->ExpectSuccess();
3484 m_commandBuffer->PipelineBarrier(VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, 0U, 0, nullptr, 0,
3485 nullptr, 0, nullptr);
3486 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3487 m_commandBuffer->WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, nullptr, 0,
3488 nullptr, 0, nullptr);
3489 m_commandBuffer->ResetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3490 m_commandBuffer->PipelineBarrier(VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, 0U, 0, nullptr, 0,
3491 nullptr, 0, nullptr);
3492 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3493 m_errorMonitor->VerifyNotFound();
3494
3495 // Need a barrier between set and a reset
3496 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-vkCmdResetEvent-missingbarrier-set");
3497 m_commandBuffer->ResetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3498 m_errorMonitor->VerifyFound();
3499 m_errorMonitor->ExpectSuccess();
3500 m_commandBuffer->end();
3501
3502 m_commandBuffer->begin();
3503 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3504 m_errorMonitor->VerifyNotFound();
3505 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-vkCmdSetEvent-missingbarrier-set");
3506 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3507 m_errorMonitor->VerifyFound();
3508
3509 m_commandBuffer->end();
John Zulaufb0b6e9b2021-08-20 09:22:45 -06003510
3511 // Secondary command buffer events tests
3512 const auto cb = m_commandBuffer->handle();
3513 VkBufferObj buffer_a;
3514 VkBufferObj buffer_b;
3515 VkMemoryPropertyFlags mem_prop = VK_MEMORY_PROPERTY_DEVICE_LOCAL_BIT;
3516 buffer_a.init_as_src_and_dst(*m_device, 256, mem_prop);
3517 buffer_b.init_as_src_and_dst(*m_device, 256, mem_prop);
3518
3519 VkBufferCopy front2front = {0, 0, 128};
3520
3521 // Barrier range check for WAW
3522 auto buffer_barrier_front_waw = LvlInitStruct<VkBufferMemoryBarrier>();
3523 buffer_barrier_front_waw.srcAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3524 buffer_barrier_front_waw.dstAccessMask = VK_ACCESS_TRANSFER_WRITE_BIT;
3525 buffer_barrier_front_waw.buffer = buffer_b.handle();
3526 buffer_barrier_front_waw.offset = front2front.dstOffset;
3527 buffer_barrier_front_waw.size = front2front.size;
3528
3529 m_errorMonitor->ExpectSuccess();
3530 VkCommandBufferObj secondary_cb1(m_device, m_commandPool, VK_COMMAND_BUFFER_LEVEL_SECONDARY);
3531 VkCommandBuffer scb1 = secondary_cb1.handle();
3532 secondary_cb1.begin();
3533 secondary_cb1.WaitEvents(1, &event_handle, VK_PIPELINE_STAGE_TRANSFER_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, 0, nullptr, 1,
3534 &buffer_barrier_front_waw, 0, nullptr);
3535 vk::CmdCopyBuffer(scb1, buffer_a.handle(), buffer_b.handle(), 1, &front2front);
3536 secondary_cb1.end();
3537 m_errorMonitor->VerifyNotFound();
3538
3539 // One secondary cb hazarding with primary
3540 m_errorMonitor->ExpectSuccess();
3541 m_commandBuffer->reset();
3542 m_commandBuffer->begin();
3543 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &front2front);
3544 m_errorMonitor->VerifyNotFound();
3545 m_errorMonitor->SetDesiredFailureMsg(VK_DEBUG_REPORT_ERROR_BIT_EXT, "SYNC-HAZARD-WRITE_AFTER_WRITE");
3546 vk::CmdExecuteCommands(cb, 1, &scb1);
3547 m_errorMonitor->VerifyFound();
3548 m_commandBuffer->end();
3549
3550 // One secondary cb sharing event with primary
3551 m_errorMonitor->ExpectSuccess();
3552 m_commandBuffer->reset();
3553 m_commandBuffer->begin();
3554 vk::CmdCopyBuffer(cb, buffer_a.handle(), buffer_b.handle(), 1, &front2front);
3555 m_commandBuffer->SetEvent(event, VK_PIPELINE_STAGE_TRANSFER_BIT);
3556 vk::CmdExecuteCommands(cb, 1, &scb1);
3557 m_commandBuffer->end();
3558 m_errorMonitor->VerifyNotFound();
John Zulauf4b5e4632020-12-15 11:48:59 -07003559}
ziga-lunarg3a16ff12021-07-30 12:09:55 +02003560
3561TEST_F(VkLayerTest, CmdWaitEvents2KHRUsedButSynchronizaion2Disabled) {
3562 TEST_DESCRIPTION("Using CmdWaitEvents2KHR when synchronization2 is not enabled");
3563
3564 ASSERT_NO_FATAL_FAILURE(InitFramework());
3565 if (!DeviceExtensionSupported(gpu(), nullptr, VK_KHR_SYNCHRONIZATION_2_EXTENSION_NAME)) {
3566 printf("%s Synchronization2 not supported, skipping test\n", kSkipPrefix);
3567 return;
3568 }
3569 if (!DeviceExtensionSupported(gpu(), nullptr, VK_KHR_GET_PHYSICAL_DEVICE_PROPERTIES_2_EXTENSION_NAME)) {
3570 printf("%s %s not supported, skipping test\n", kSkipPrefix, VK_KHR_GET_PHYSICAL_DEVICE_PROPERTIES_2_EXTENSION_NAME);
3571 return;
3572 }
3573 m_device_extension_names.push_back(VK_KHR_SYNCHRONIZATION_2_EXTENSION_NAME);
3574 m_device_extension_names.push_back(VK_KHR_GET_PHYSICAL_DEVICE_PROPERTIES_2_EXTENSION_NAME);
3575 InitState();
3576
3577 auto fpCmdWaitEvents2KHR = (PFN_vkCmdWaitEvents2KHR)vk::GetDeviceProcAddr(m_device->device(), "vkCmdWaitEvents2KHR");
3578
3579 VkEventObj event;
3580 event.init(*m_device, VkEventObj::create_info(0));
3581 VkEvent event_handle = event.handle();
3582
3583 VkDependencyInfoKHR dependency_info = LvlInitStruct<VkDependencyInfoKHR>();
3584
3585 m_commandBuffer->begin();
Tony-LunarG279601c2021-11-16 10:50:51 -07003586 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkCmdWaitEvents2-synchronization2-03836");
ziga-lunarg3a16ff12021-07-30 12:09:55 +02003587 fpCmdWaitEvents2KHR(m_commandBuffer->handle(), 1, &event_handle, &dependency_info);
3588 m_errorMonitor->VerifyFound();
3589 m_commandBuffer->end();
3590}
ziga-lunarg15f450d2021-08-26 23:10:05 +02003591
3592TEST_F(VkLayerTest, Sync2FeatureDisabled) {
3593 TEST_DESCRIPTION("Call sync2 functions when the feature is disabled");
3594
3595 if (InstanceExtensionSupported(VK_KHR_GET_PHYSICAL_DEVICE_PROPERTIES_2_EXTENSION_NAME)) {
3596 m_instance_extension_names.push_back(VK_KHR_GET_PHYSICAL_DEVICE_PROPERTIES_2_EXTENSION_NAME);
3597 } else {
3598 printf("%s Did not find required instance extension %s; skipped.\n", kSkipPrefix,
3599 VK_KHR_GET_PHYSICAL_DEVICE_PROPERTIES_2_EXTENSION_NAME);
3600 return;
3601 }
3602 ASSERT_NO_FATAL_FAILURE(InitFramework());
3603 if (DeviceExtensionSupported(gpu(), nullptr, VK_KHR_SYNCHRONIZATION_2_EXTENSION_NAME)) {
3604 m_device_extension_names.push_back(VK_KHR_SYNCHRONIZATION_2_EXTENSION_NAME);
3605 } else {
3606 printf("%s Synchronization2 not supported, skipping test\n", kSkipPrefix);
3607 return;
3608 }
3609
3610 ASSERT_NO_FATAL_FAILURE(InitState());
3611
3612 VkPhysicalDeviceSynchronization2FeaturesKHR synchronization2 = LvlInitStruct<VkPhysicalDeviceSynchronization2FeaturesKHR>();
3613 synchronization2.synchronization2 = VK_FALSE; // Invalid
3614 auto features2 = LvlInitStruct<VkPhysicalDeviceFeatures2KHR>(&synchronization2);
3615 vk::GetPhysicalDeviceFeatures2(gpu(), &features2);
3616
3617 auto vkCmdPipelineBarrier2KHR =
3618 (PFN_vkCmdPipelineBarrier2KHR)vk::GetDeviceProcAddr(m_device->device(), "vkCmdPipelineBarrier2KHR");
3619 auto vkCmdResetEvent2KHR = (PFN_vkCmdResetEvent2KHR)vk::GetDeviceProcAddr(m_device->device(), "vkCmdResetEvent2KHR");
3620 auto vkCmdSetEvent2KHR = (PFN_vkCmdSetEvent2KHR)vk::GetDeviceProcAddr(m_device->device(), "vkCmdSetEvent2KHR");
3621 auto vkCmdWriteTimestamp2KHR =
3622 (PFN_vkCmdWriteTimestamp2KHR)vk::GetDeviceProcAddr(m_device->device(), "vkCmdWriteTimestamp2KHR");
3623
3624 bool timestamp = false;
3625
3626 uint32_t queue_count;
3627 vk::GetPhysicalDeviceQueueFamilyProperties(gpu(), &queue_count, NULL);
3628 std::vector<VkQueueFamilyProperties> queue_props(queue_count);
3629 vk::GetPhysicalDeviceQueueFamilyProperties(gpu(), &queue_count, queue_props.data());
3630 if (queue_props[m_device->graphics_queue_node_index_].timestampValidBits > 0) {
3631 timestamp = true;
3632 }
3633
3634 m_commandBuffer->begin();
3635
3636 VkDependencyInfoKHR dependency_info = LvlInitStruct<VkDependencyInfoKHR>();
3637
Tony-LunarG279601c2021-11-16 10:50:51 -07003638 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkCmdPipelineBarrier2-synchronization2-03848");
ziga-lunarg15f450d2021-08-26 23:10:05 +02003639 vkCmdPipelineBarrier2KHR(m_commandBuffer->handle(), &dependency_info);
3640 m_errorMonitor->VerifyFound();
3641
3642 VkEventCreateInfo eci = LvlInitStruct<VkEventCreateInfo>();
3643 vk_testing::Event event;
3644 event.init(*m_device, eci);
3645
3646 VkPipelineStageFlagBits2KHR stage = VK_PIPELINE_STAGE_2_TOP_OF_PIPE_BIT_KHR;
3647
Tony-LunarG279601c2021-11-16 10:50:51 -07003648 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkCmdResetEvent2-synchronization2-03829");
ziga-lunarg15f450d2021-08-26 23:10:05 +02003649 vkCmdResetEvent2KHR(m_commandBuffer->handle(), event.handle(), stage);
3650 m_errorMonitor->VerifyFound();
3651
Tony-LunarG279601c2021-11-16 10:50:51 -07003652 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkCmdSetEvent2-synchronization2-03824");
ziga-lunarg15f450d2021-08-26 23:10:05 +02003653 vkCmdSetEvent2KHR(m_commandBuffer->handle(), event.handle(), &dependency_info);
3654 m_errorMonitor->VerifyFound();
3655
3656 if (timestamp) {
3657 VkQueryPoolCreateInfo qpci = LvlInitStruct<VkQueryPoolCreateInfo>();
3658 qpci.queryType = VK_QUERY_TYPE_TIMESTAMP;
3659 qpci.queryCount = 1;
3660
3661 vk_testing::QueryPool query_pool;
3662 query_pool.init(*m_device, qpci);
3663
Tony-LunarG279601c2021-11-16 10:50:51 -07003664 m_errorMonitor->SetDesiredFailureMsg(kErrorBit, "VUID-vkCmdWriteTimestamp2-synchronization2-03858");
ziga-lunarg15f450d2021-08-26 23:10:05 +02003665 vkCmdWriteTimestamp2KHR(m_commandBuffer->handle(), stage, query_pool.handle(), 0);
3666 m_errorMonitor->VerifyFound();
3667 }
3668
3669 m_commandBuffer->end();
3670}