Revert "layers: Update docs for shared utils DLL"

This reverts commit 94dd6081959103c1c9b0bcfda173d4149d5555b9.

Change-Id: Ie5b437d822c5efe6baf0baf2830591f50bf3ee4d

demos: LX535 Remove deprecated device layers.

demos: Use XSync to fix intermittent xlib crash.

layers: Fix CmdBeginRenderPass after non-inline renderpass in same CB

Fixes #667.

Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: GH669, Fix clearValueCount errors

Validation was assuming LOAD_OP_CLEAR for some attachments resulting
in bad validation messages.

Change-Id: I8ff75ca63f591e3002b1ee2b61a6844d0cf1c07d

demos: Fix intermittent crash on xlib

smoke: Remove references to device layers

Change-Id: I2399cdefb76a086525fea29e71a7c321b5cfecdd

windowsbuild: +x of update_external_sources.bat

layers: Fix Windows crash in Dota2

In core_validation attachment_references_compatible(), having array
elements set to VK_ATTACHMENT_UNUSED was not handled resulting in
invalid memory references.

Change-Id: If78aea1467d191db905272bece16cc9f56643136

demos: remove deprecated device layer code

Change-Id: I916f30da7e1b2d43826f3d2080fed9721b1796d6

loader: update so version

layers: Get rid of fence->swapchain link

Nothing used this after rationalizing the state tracking.

Change-Id: I67cc267336b5d64bea9645a8e0a072885db58c26
Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Untangle validate+update in QueuePresentKHR

Change-Id: I8f719c6e57cbab836b90f30414e6494a0643580e
Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Fix AcquireNextImageKHR handling of sync objects

Change-Id: I3e5a5d08763c0bf26c6c69aecf4bcef17d055a48
Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Add getSemaphoreNode helper

Change-Id: Iceda3b28474e289189ef1f97b271f9e9bb7b2083
Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Rework fence state tracking

Instead of having a combination of firstTimeFlag, in_use, and
needsSignaled, let's just have an enum with the three states a fence can
actually be in.

Change-Id: I6a40cd709717c75740c9977b149c4f71a02a27ab
Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Tidy decrementResources a bit

Change-Id: Ib8f9a917115323d8d43b971593b213ff04138a35
Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: use getFenceNode in decrementResources

Change-Id: I8351405299f2f109489fdddfc111412164aded77
Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Pass layer_data to verifyWaitFenceState

Change-Id: Id3e0f09254197119b821710f0e9ce27d3458afc9
Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: use getFenceNode in verifyWaitFenceState

Change-Id: I46f4ef4c225e199c0415507bd19dc4ffc98a77bc
Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Remove some MT leftovers

Change-Id: I2b62475951315c6b231342be6f7c74865e3ea2bb
Signed-off-by: Chris Forbes <chrisforbes@google.com>

misc: Update VK_EXT_DEBUG_REPORT_SPEC_VERSION to 3

Change-Id: I2137c0d8c034f66da678bfccbc044114c2f39f2e

layers: Add DSAspectBitsErrors test to documentation

layers: GH644 Validate DS image aspect bit restriction

If a DS image is used in a descriptor, regardless of the underlying
image layout, we need to validate that BOTH DEPTH and STENCIL aspect
bits are NOT set. Only one of the two bits is allowed.

tests: Update documentation with RenderPassIncompatible test

layers: Add bug fix to handle NULL attachment case

Correctly handle null resolve attachment case. When verifying attachment
compatibility, matching null resolves is ok.

layers: Remove output paren from android update script

loader: Increase error message length

120 chars in error message is very conservative as error prefix and LVL
pathname alone are near 100 chars. This can easily cause layer path to get
cut off which makes for a confusing error message. Bumping this to 164
chars to account for most reasonable layer names.

layers: Add PSO render pass compability check

At draw time, the active render pass must be compatible with the render
pass that the gfx pipeline was created with.

tests: Update doc to note BindImageInvalidMemoryType is disabled

layers: GH656, Fix CreateRenderpass validation

In the core_validation layer, much of the validation was occurring
AFTER the API call had been made. Separated validation and tracking.

Change-Id: I6f8502d52dd2861a18254de48141a3736ced257f

layers: Fix dynamic descriptor count

When creating descriptor set layout, we were only incrementing the dynamic descriptor
count by 1 regardless of the number of descriptors in a binding. This fix increments
the count by the number of descriptors in the binding.

demos: Add semaphore between queue and present

LeX #548

Change-Id: Ic34266d759923f2856cbd6efc9e5bdcfd193c366

demos: Rename presentComplete semaphore

to imageAquired - LeX #548
Change-Id: I6c8ee40ac809571aac82a2d6981ee8caa8cdbbc2

layers: Update manifest files for vers 1.0.17

Change-Id: Ifdce033092a72980e1ff685792d486d2a101f35d

misc: Update to Vulkan header version 17

Only changes:

VK_HEADER_VERSION 17
VK_EXT_DEBUG_REPORT_SPEC_VERSION  3

Change-Id: I122bdcabec1f831a6ed1a06b5caf6202a2a94529

docs: Fix weird characters in Loader.md file

layers: GH401, Break out stencil load/storeOp check

Stencil attachment load/storeOp settings were being ignored. Added
format appropriate checks for these flags.

Change-Id: I6a917fc8a28cbb0d0441152e8d6630defb1b81e3

get latest spirv-tools revision

update_external_sources.sh: now get spirv-tools headers from Khronos/SPIRV-Headers

Merge branch 'master' of github.com:KhronosGroup/Vulkan-LoaderAndValidationLayers

update_external_sources.bat: now get headers from Khronos/SPIRV-Headers

demos: fix ppm loader swapped width & height

Change-Id: I277fe91a928223a09f672d07c22a1547e710b847

Only set scripts that starts with a hashbang as executable

Remove executable bit from files like README.md

Change-Id: I6a2a86e8312616c175845f0a3b777515a58c52b9

layers: GH576, Update layer details doc

Added entries for new attachment index tests and enums.

Change-Id: Ida1de0bffc544718e63ccf3641a09dbe577f3c4d

layers: GH576, Validate attachment image usage

Framebuffer attachment images are created with IMAGE_USAGE flags. These
flags are now validated against their attachment location in subpass
description structures. Also added a check for a preserve attachment
being set to UNUSED and fixed a few typos in the attachment index
checking stuff.

Change-Id: Id62d3539c490ce8f381d7ae606402980f11946fa

build: Fix typos in CMakeLists.txt (SPRIV->SPIRV)

Change-Id: I6a2b685cea8cda1fde5ae40a0beaa04f1319c6a3

layers: LX539, update layer details doc

Added new error enum for memory index type mismatches.

Change-Id: I6f857facc4209f103da5cef9f6561a0e4ec3c334

layers: LX539, Validate mem types

Bind[Image|Buffer]Memory did not validate that the memoryPropertyType
matched the memory_requirements memory type flags for the buffer
or image.

Change-Id: I7007588d78408689c34df66a17cc9201a37569b1

tests: GH421 vkUpdateDescriptorSets tests

A) Adding the WriteDescriptorSetIntegrityCheck test, which verifies
some requirements of chapter 13.2.3 of the Vulkan Spec
 1) A uniform buffer update must have a valid buffer index.
 2) When using an array of descriptors in a single WriteDescriptor,
      the descriptor types and stageflags must all be the same.
 3) Immutable Sampler state must match across descriptors
The above tests also exercise cases where descriptors are updated with
consecutive bindings both correctly and incorrectly

B) adding a MissingClearAttachment test that points to a wrong
colorAttachment index in a VkClearAttachment structure passed to
vkCmdClearAttachments

C) update vk_validation_layer_details.md to reflect above changes.

Change-Id: I3f4f2233371b68caf23b5a4111d63a94765121c3

vulkaninfo: Show TexelOffset in decimal, instead of hex.

loader: GH362 - Fix json output of escape chars

The strings from the JSON file already add escape characters.
So, \ becomes \\ in the JSON file strings.  However, the cJSON
library was adding \\ for ever encountered \ when converting
to a string.  This became messy as C:\\vulkanSDK\\layerfile.json
became C:\\\\vulkanSDK\\\\layerfile.json.

Change-Id: I006252e33d6e91e2bef704dd5dee0777105388a7

loader: Fix GH607 by adding proper array support

JSON spec does not allow objects of the same name at the same
level.  But, that's what we used to allow with "layer" definitions.
Update new file version to 1.0.1, add "layers" array object, and
add multiple "layer" definition warning.

Change-Id: I040f07897c689800364d243daf3c247f1b3cace7

loader: return ext not present when applicable

layers: Fix CMakeLists.txt for non-MSVC Win builds

Fix a recent change broke windows builds that don't use the VC IDE or
decorate build trees like VC does

Change-Id: I93c8a559c4255408a6fa890b2a3e4d81822d1cea

vulkaninfo: Clean up XCB window creation code.

Change-Id: I5ef0862326d9a7ce4e59fe7c8947b532f252d604

layers: Update docs for shared utils DLL

Change-Id: I129b4d074f0604bbdf6d81c052333229ece99ac3

layers: Add debug report header

Enable output of a short message header containing definitions of
debug message types.

Change-Id: Ia84fb0e79d599b8d022b7d78267b7b6a0307823e

layers: Make vklayer_utils a shared library

Was previously statically linked.  Converting to a DLL will allow
data sharing and use of utils in samples repo.

Change-Id: I63ccc1f0f57dfad40f7068c60ac623bce5bd714d

layers: Fix Windows Dota2 crash

One of the event tracking lists was not getting cleared upon a reset,
resulting in an invalid reference on reuse.

Change-Id: I45d932822f27197266b363634d6cdccc785209d3

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>

layers: Fix mistracking of semaphores associated with submissions

When a command buffer is simultaneously inflight multiple times, each
can have different semaphores associated. Storing the set of semaphores
on the GLOBAL_CB_NODE caused us to get confused and emit bogus errors.

Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Note broken behavior around stealing tracking from other queues

Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Add members to FENCE_NODE and QUEUE_NODE to hold the handles

We'd like to be able to get back to them from the tracking struct.

Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Add helpers for getting fence and queue nodes

Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Add CB_SUBMISSION struct

Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Unify fence checks for queue functions

Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Remove dead fence members from GLOBAL_CB_NODE

Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: remove tracking of last queue and fence for CB

Unused.

Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Update manifest files for vers 1.0.16

Change-Id: I7468c088d99f7bebabc05d1a84529f6e9e00d424

misc: Update to Vulkan header version 16

Only change is the addition of AMD extensions.

Change-Id: Ia4270600247b2e3f406cdac0cfff3492ec1ce109

Revert "loader: Fix build warning on windows"

This reverts commit 27a58419aab1bd62201e00ea5a6d822b65f99fa3.

loader: Fix build warning on windows

Don't use default library for libcmt.

Change-Id: I68c0f629044713c0c875758af9a0a1dd345dad75

Added {} to if statements.

Change-Id: I5ef0862326d9a7ce4e59fe7c8947b532f252d604

Removed some window event handling code.

Change-Id: I5ef0862326d9a7ce4e59fe7c8947b532f252d604

Applied the "clang-format" obfuscator

Change-Id: I5ef0862326d9a7ce4e59fe7c8947b532f252d604

Show "Presentable Formats" per GPU for Win32/XCB/XLib

Change-Id: I5ef0862326d9a7ce4e59fe7c8947b532f252d604

layers: GH536, Updated layer details doc

Added new cases, removed old enums, moved info from devlimits
to parameter validation.

Change-Id: Iddf6a679dc8596dcf411a6b880a63a26999607ec

layers: GH536, Update CmdFillBuffer validation

Was ignoring VK_WHOLE_SIZE. Moved validation cases for Update/FillBuffer
to Parameter Validation layer (as they were stateless) and added the
remaining additional stateless valid usage checks.

Change-Id: I0f41c6ba55f218a6db9fb34736a2abc52f856edf

winrt: fix error on uninstall

Remove old log files before writing them. When 2 uninstalls
are done in a row, this caused the uninstaller to display
an error

winrtinstaller: display error num in ps warning window

Display an error number in warning window if powershell
script fails.

winrt: configure loader even if ps script fails

winrtinstaller: fixed problems introduced by piping script to powershell.exe

PS script return value is now correctly retrieved by NSIS script.
Logging is less verbose, removed extranous errors.

layers: PR573, Handle imageArraySize rename

Swapchain layer had stale references -- rename things still using
imageArraySize instead of imageArrayLayers.

Change-Id: Iaa4500ab8cf55e50b96636a50136605dd3e6d83e

layers: PR632, Updated layer details doc

Added device_limits MISSING_QUERY_COUNT

Change-Id: I5b129ec12930129f5d5fe46bbe2622977dc2d992

layers: PR632, Change devlimits errors to warnings

Replace call sequence errors with warnings for
vkEnumeratePhysicalDevices and GetPhysicalDeviceQueueFamilyProperties.

Change-Id: I1505f4a2b339ac7ef435da8b3ec2f85612026a42

layers: PR629, Remove subpass warnings

The validation layer checks for dependency between subpasses using the
same attachments, but it also issues warnings if there are "only"
implicit dependencies between two subpasses. This is explicitly
allowed by the spec and should not cause validation warnings.

Change-Id: I1039955d6d445e1ee234116c6da2e4dbaf9c7dd9

layers: Initialize write_in_use in eventMap entries

Change-Id: Ic296788beb078f19a35b93b99df5dbf677c95122

winrtinstaller: avoid executing ps scripts, instead just pipe cmds to ps

demos: Add -lrt to link options for smoke LVL #500

Change-Id: I0480e60e3c6c2f06f24022332640756bcdc3f1a0

tests: GH421 reflect vkAllocateDescriptorSets test

InvalidDescriptorSetLayout test was added with 772fcd1c

tests: GH421 reflect vkCmdBindDescriptorSets test

Test was added with 772fcd1

layers: Add check for image usage in vkCmdClearColorImage

Change-Id: I7ba803a11de29e65ae65fbf0a0168561a2e6739f

winrtinstaller: retry ps script failure with full path to powershell.exe

Also changed diagnostics to log the contents of dir containing
powershell.exe, and log the current value of $PATH.

loader: doc update for merge android and desktop layer interface

Change-Id: Ibbffc0bf05bcbd1aa3b9272caa0b2c1c3a54037d

demos: Added --suppress_popups flag

Added a command line option to cube and tri to suppress
windows message boxes to facilitate Windows testing.

Change-Id: I069f9b199c9740e1e21c42100ceec42d5fb8f0e2

doc: Expand LVL acronym in BUILD.md

layers: Update docs for default dbg msg callbacks

Change-Id: Ic1f192ea7c95762d7e53f091cc9f0ea55be96c04

layers: Move debug action enums from vk_layer.h

Moved debug-report related items from vk_layer.h. Enums are now in
vk_layer_logging.h and DbgFunctionNode has been moved into a new
loader header file vk_loader_layer.h.

Change-Id: I6031146ba474ff01ca039da44ad5d42d054383a3

layers: Add default layer error message config

Allows layers to output error messages even if no vk_layer_settings.txt
config file is present. Sets defaults to LOG_MSG, error, stdout.
A layer settings file will override any default values. If no settings
file is present and an app creates a debug callback, the default
callbacks will be removed and unregistered.

Change-Id: I49f37189665816df58c258b9e9629f2bf76751c8