commit 1df1f88b8391fa67cea417b74dc724fb8ed3005f
author locke-lunarg <locke@lunarg.com> Mon Mar 02 16:42:08 2020 -0700
committer John Zulauf <32470354+jzulauf-lunarg@users.noreply.github.com> Mon Jul 13 10:05:48 2020 -0600
tree a49cbc7dc088dbd57be1664a5ed868ff2ed4d1e3
parent 5faaff57e7833bd9d01917dab74bf060611eb8a7

syncval: Add dest image adjustment for compressed

Add correct extent adjustments for destination images during detect
and update operations.

Change-Id: I005e2e7227105ab9340b7c34d6953d54ab9b3a8a

layers/synchronization_validation.cpp

diff --git a/layers/synchronization_validation.cpp b/layers/synchronization_validation.cpp
index 2faf6c3..eef434a 100644
--- a/layers/synchronization_validation.cpp
+++ b/layers/synchronization_validation.cpp
@@ -858,8 +858,10 @@
         }
 
         if (dst_image) {
+            VkExtent3D dst_copy_extent =
+                GetAdjustedDestImageExtent(src_image->createInfo.format, dst_image->createInfo.format, copy_region.extent);
             auto hazard = context->DetectHazard(*dst_image, SYNC_TRANSFER_TRANSFER_WRITE, copy_region.dstSubresource,
-                                                copy_region.dstOffset, copy_region.extent);
+                                                copy_region.dstOffset, dst_copy_extent);
             if (hazard.hazard) {
                 skip |= LogError(dstImage, string_SyncHazardVUID(hazard.hazard), "Hazard %s for dstImage %s, region %" PRIu32,
                                  string_SyncHazard(hazard.hazard), report_data->FormatHandle(dstImage).c_str(), region);
@@ -890,8 +892,10 @@
                                            copy_region.srcOffset, copy_region.extent, tag);
         }
         if (dst_image) {
+            VkExtent3D dst_copy_extent =
+                GetAdjustedDestImageExtent(src_image->createInfo.format, dst_image->createInfo.format, copy_region.extent);
             dst_tracker->UpdateAccessState(*dst_image, SYNC_TRANSFER_TRANSFER_WRITE, copy_region.dstSubresource,
-                                           copy_region.dstOffset, copy_region.extent, tag);
+                                           copy_region.dstOffset, dst_copy_extent, tag);
         }
     }
 }