diff options
| author | Chris Forbes <chrisforbes@google.com> | 2016-06-13 09:59:34 +1200 |
|---|---|---|
| committer | Tobin Ehlis <tobine@google.com> | 2016-06-13 14:56:31 -0600 |
| commit | dda1e861321cfda600472dd22df3a84bc7e57c15 (patch) | |
| tree | c3e1ea603d5d66147bd93eb14e3ba1f79baca413 /layers/core_validation.cpp | |
| parent | f2ccbd9add820c5cba7c5a2e1e5aa8c85997053d (diff) | |
| download | usermoji-dda1e861321cfda600472dd22df3a84bc7e57c15.tar.xz | |
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 <chrisforbes@google.com>
Diffstat (limited to 'layers/core_validation.cpp')
| -rw-r--r-- | layers/core_validation.cpp | 4 |
1 files changed, 4 insertions, 0 deletions
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); |