layers: Add string validation to param_checker

layers: Add UTF-8 String validation utility

layers: lock use of threading layer cmd pool map

Access to command_pool_map data structure in threading validation layer
needs a mutex.

winrtinstaller: update license text

layers: MR215 Update spec version to match json files

loader: MR215 device extensions not enumerated

If a layer's device extension didn't define any entrypoints
it was ignored by the loader. Now add it to the
list of layer extensions.

layers: In GetInstanceProcAddr, intercept instance procs even if NULL instance

This problem showed up when running cube --validate and the order
of the layers was arranged so that the threading layer is NOT first.
It can also show up when running vktrace and the threading layer
is in any position, because the vktrace layer becomes first.

The symptom is generally a failure to create an instance when the
threading layer is enabled.

The fix is to check for the non-instance-based procs first by
calling layer_intercept_instance_proc before bailing if the instance
is NULL.

layers: Fix 32-bit Windows build

A reinterpret_cast from a non-dispatch handle to uint64_t was failing on
windows 32-bit where non-dispatch handles are defined as 'typdef uint64_t
object'.  Changed reinterpret_cast to a C-style cast, as is consistent
with the rest of the non-dispatch handle to uint64_t conversions in
draw_state.

layers: Fix CreateSemaphore/CreateEvent conflict

Reorder includes to avoid conflicts between the VkLayerDispatchTable_
CreateSemaphore/CreateEvent fields and the WIN32 API.

layers: MR217 Turn on more layers for Android build

layers: MR217 Fix Android build of swapchain

layers: MR217 Fix unique_objects android build

Need to guard vkCreateXcbSurfaceKHR in appropriate ifdef

winrtinstaller: update licenses

Fix so that store images and buffers are assumed written.

layer: GL125 only check for fence in_use on valid fence handles

layers: GL127 GL128 fix duplicate in_use updates

Only increment in_use at vkQueueSubmit() time. Also only need to
check ONE_TIME_SUBMIT case at that time. We were duplicating some
checks incorrectly at vkCmdExecuteCommands() time.

layers: thread checks of VkCommandBuffer pool read

Object reads of VkCommandBuffer should also be reads of their pool.
A single thread may have multiple uses of a pool in one call.
That will be allowed because the uses are all in a single thread.

layers: thread checks for vkDestroyInstance and vkDestroyDevice

Add instance write to vkDestroyInstance.
Add device write to vkDestroyDevice.

layers: allow one thread multiple uses of object

A single call in a thread may use an object more than once.
This first appeared with vkCmdExecuteCommands and buffer pools.
That is not distinguishable from recursive vulkan calls.
The error that this was looking for only happens if calling vulkan
while within another vulkan call.  Such calls from signal handlers
or callbacks should be very rare errors.  So just allow one thread
to have multiple uses of an object and keep accurate counters.

layers: use less of namespace std

Change to std::unordered_map instead of "using namespace std".

layers: change android build to xml threading

Use threading validation layer generated from vk.xml for android.

layers: Handle 32-bit threading validation

Non-dispatchable handles are all of the same type using 32-bit pointers.
Use a single counter for uses of such handles in that situation.

layers: Add threading layer generation

Generate threading layer wrappers in thread_check.h.
Change path for generated vulkan.h.

layers: bring xml and scripts from documentation

These are repo git@gitlab.khronos.org:vulkan/vulkan.git
from commit 51ffe091889e08ca5196c5635363a5358c85ff82

winrtinstaller: Improve search for layer json files in SDK Bin dir

windowsrtinstaller: Change lincense info.

Also removed extra white space.

windowsrtinstaller: updated license

windowrtinstaller: Add more details to README.txt

layers: LX324, Fix descriptor indices in draw_state

Typo in BindDescriptorSets was using incorrect array index.

layers: MR213 GL126 Fix render pass compatibility check

Use correct array size and add early-out if primary and secondary passes exactly match.

layers: Fix subscript-out-of-range crash in mem_tracker

Render pass map was not being updated for destroyed renderPasses.  When a
renderPass handle was reused by the driver its data became invalid causing
crashes in Windows.

layers: Fix vkEnumeratePhysicalDevices()--was always returning an error.

layers: Move logging callback destroy after logging calls

Swapchain was using deleted memory pointer causing crashes in
windows.

Conflicts:
layers/swapchain.cpp

loader: filter device extensions in the terminator instead of trampoline

layers: MR212 Add locking to the swapchain layer.

layers: Fix for GL119 to correctly index into pBindings

Added bindingToIndexMap to LAYOUT_NODE to map from the binding # to
the index into the pBinding array of layout create info.
Populate the map at vkCreateDescriptorSetLayout() time and use the
map when performing descriptor updates and when verifying that a
descriptor binding exists for given layout.

layers: Verify that command buffer is recorded when submitted to a primary.

layers: Validate secondary command buffers are most recently bound.

Secondary cmd buffer bound to new primary cmd buffer cannot be submitted
under the first primary cmd buffer.

This is based on spec valid usage restriction in vkQueueSubmit:

Any given element of pCommandBuffers must not contain commands that execute a
secondary command buffer, if that secondary command buffer has been recorded in
another primary command buffer after it was recorded into this VkCommandBuffer

layers: Validate command buffer is not being used simultaneously unless specified.

layers: MR208 Improve locking in the mem_tracker layer.

layers: Improve locking in the draw_state layer.

This includes generating return values within locks, and doing the actual
return outside of the lock.

layers: Fix 32-bit Windows build

A reinterpret_cast from a non-dispatch handle to uint64_t was failing on
windows 32-bit where non-dispatch handles are defined as 'typdef uint64_t
object'.  Changed reinterpret_cast to a C-style cast, as is consistent
with the rest of the non-dispatch handle to uint64_t conversions in
draw_state.

Remove WTFPL from LICENSE.txt

demos: Fix window resize on Linux

layers: Validate that fence is not in use when submitted.

layers: Validate subpass index in begin info is valid.

layers: Make sure fb render pass and secondary render pass are compatible.

layers: Validate that the framebuffer specified in the secondary command buffer is the same as the current one.

layers: Validate render pass compatiblity for executed secondary command buffers.

layers: Fix issues with some command buffer tracking structures being left uninitialized

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

layers: Fix 32-bit Windows build

A reinterpret_cast from a non-dispatch handle to uint64_t was failing on
windows 32-bit where non-dispatch handles are defined as 'typdef uint64_t
object'.  Changed reinterpret_cast to a C-style cast, as is consistent
with the rest of the non-dispatch handle to uint64_t conversions in
draw_state.

loader: GL120, move critical section to cover loader_get_icd_and_device

demos: Copyright update

layers: MR209 Fix swapchain's vkDestroyInstance to error about devices ...

not being previously destroyed, instead of physical devices not being
previously destroyed.

layers: Valdiate query is not active when cmd buffer is ending.

layers: GL122 Init queueFamilyIndex in command pool.

layers: Fix Windows build error in device_limits

layers: GL96 validateCommandBufferState should return skipCall

When validation checks were rolled into separate validateCommandBufferState
function, forgot to return the skipCall value from that function.

layers: GL64 Don't warn on descriptor set that's not updated

Previously was not handling the case of 0 sets in a layout.
Now catching this case by verifying that a descriptor set's
layout bindingCount is not zero before incrementing internal
descriptorCount variable.

layers: Check that queryFlags is valid if enabled.

layers: Add check to make sure inherited queries in enabled before use.

layers: Add occlusion query precise check.

layers: Improve in flight cmd buffer validation

Validation in flight cmd buffers are not reset.
Validate cmd buffer is not in use when freed.

layers: Add queue bit checks.

For each command check to make sure if the queue bits are set
correctly for that command.

demos: Minor cube/tri updates

Minor updates/fixes for cube and tri:
 - Break from demo_check_layers search loop when finding a match.
 - Skip memory allocation and layer/extension processing when 0
   layers/extensions are enumerated.
 - Fix potential NULL pointer dereference and exit with message when
   no accessible devices are available.
 - Fix device_extensions memory leak.
 - Remove trailing whitespace.

demos/tests: Enable unique_objects layer

Using VK_LAYER_GOOGLE_unique_objects with 'cube --validate' and when
running all tests with layers.

windowsrtinstaller: update estimated size for Programs and Features

loader: Fix bug on pass by value for created_instance

Revert "layers: Clang-format changes for files in layers dir of repo"

This reverts commit e260acfb96ea20945149ede6a250ef622ca7fd03.
The clang-format changes need to be postponed until just prior
to public release to facilitate MRs.

license: Update the main MIT license to reflect white space and copyrights

license: remove licenses for componenets that were moved to different repo.

loader:Update copyright and apply clang-format (no logic changes)

Update copyright info in python scripts

layers: Clang-format changes for files in layers dir of repo

windowrtinstaller: use new image files names in installer

windowsrtinstaller: changed images

layers: Validation Layer Copyright Updates

demos: Update copyright and apply clang-format

Update BUILD.md and README.md for the reduced scope of this repo

scripts: Remove icd generation specific code

androidBuild: remove unused layers  api_dump, generic, basic, multi

layers: Remove non-validation layers

Removes api_dump, generic, basic, multi.

misc: Remove vktrace and icd from this repository

include: Update copyright and apply clang-format (no logic changes)

windowssdk: Handle silent installer/uninstaller message boxes appropriately.

If Silent Install or Uninstall is enabled, we want all the message boxes we
have control over to automatically make a selection.  In addition, return
an error if we are supposed to reboot, and the silent install doesn't show
the message for the user to choose.

misc: Modify build_windows_targets.bat to build quietly.

Making these changes allows errors/warnings to be more obvious.

windowsrtinstaller: Don't display dialog when silent uninstall requires reboot

scripts: Fix python helper to find ext headers under vulkan dir

windowsrtinstaller: change windows icon

Merge branch 'marky_working'

misc: Modify update_external_sources.bat to build quietly.

This speeds up the build as well as makes warnings and errors more obvious.

Revert "misc: Change repo name references to LoaderAndValidationLayers"

This reverts commit 015e74cda0b9ad46bd6127d816f14d699697ec4a.

misc: Change repo name references to LoaderAndValidationLayers

windowsrtinstaller: Handle error conditions during install by uninstalling.

If an error occurs during install, uninstall the portions that have been
installed, and exit with an error code.

layers: MR182/GL113 Fix dynamic offset validation

Dynamic offsets are bound to CB, so move the tracking struct into CB struct.
At draw time, iterate over all of the active sets and verify that each
dynamic set does not overstep its buffer based on the corresponding dynamic offset.

windowsrtinstaller: change icon used by installer

docs: remove old obsolete documents

windowsrtinstaller: change licensing and change logo displayed in UI

layers: MR181/GL102 Fix unique_objects to correctly assign local vars under ptrs

If a variable that needs to be restored is below a ptr, then need to declare
it at the top level, but only assign it if the ptr(s) that it's below are
non-null.

layers: MR178, Merge-related fix