From dda1e861321cfda600472dd22df3a84bc7e57c15 Mon Sep 17 00:00:00 2001 From: Chris Forbes Date: Mon, 13 Jun 2016 09:59:34 +1200 Subject: layers: Fix test fallout in CommandBufferTwoSubmits This test was accidentally triggering the fence reuse case as well, and blindly continuing past that error. It doesn't need to use the fence at all, so don't supply one. Also tweak the layer to not run into the assert if the caller elected to skip -- QueueSubmit still has validation and updates tangled together, but this improves it slightly. Signed-off-by: Chris Forbes --- layers/core_validation.cpp | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'layers/core_validation.cpp') diff --git a/layers/core_validation.cpp b/layers/core_validation.cpp index bfb934d4..8d97dc21 100644 --- a/layers/core_validation.cpp +++ b/layers/core_validation.cpp @@ -4535,6 +4535,10 @@ QueueSubmit(VkQueue queue, uint32_t submitCount, const VkSubmitInfo *pSubmits, V auto pFence = getFenceNode(dev_data, fence); skipCall |= ValidateFenceForSubmit(dev_data, pFence); + if (skipCall) { + return VK_ERROR_VALIDATION_FAILED_EXT; + } + // TODO : Review these old print functions and clean up as appropriate print_mem_list(dev_data); printCBList(dev_data); -- cgit v1.2.3