layers: Minor housekeeping in draw_state.

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

demos: Fix a few layout transition bugs in cube

Re-enabled layout validation in draw_state revealed a few bugs in cube. This change fixes those bugs by starting tex image in PREINITIALIZED state with the HOST_WRITE_BIT set.

layers: Fix and re-enable layout checks.

Handle layout transitions for subresources and re-enable layout validation.

layers: Update unique_objects to make use of safe_structs

Stop overwriting const* data in unique_objects.
Instead, declare local safe_struct copies of data, update that
data in place with unwrapped objects, and pass down the local
safe_struct copy, which spoofs the original struct layout.

This includes code to build the layer using vk_safe_struct.h/cpp files.

layers: Update vk_helper to generate safe_struct wrappers

Create vk_safe_struct.h/cpp files that contain wrapped versions of vk structs.
The idea is to wrap structs that contain non-dispatchable objects so that
the non-dispatchable objects are no longer contained within or below "const" decls.
The safe struct wrappers are intended to exactly match the original structures
in how they appear, but they also contain some utility functions to contruct and
desctruct themselves so that ptr chains are automatically replicated.

For unique_objects, the idea is to use the safe_struct wrappers as follows:
foreach struct that contains non-dispatchable objects
   declare local safe_struct version of struct
   initialize struct to exactly match original
   unwrap non-dispatchable objects in place in the safe_struct

layers: Validate specialization entry data is fully contained within the specialization data block.

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

Validate buffer barrier size and offset.

layers: Correctly handle VK_WHOLE_SIZE for dynamic offsets

layers: Validate event is not in use when deleted.

layers: Also allow fences to be submitted via acquire next image.

layers: Validate semaphore is not in use when deleted.

layers: Validate fences not in use when reset.

winrtinstaller: Fix typo in messagebox

winrtinstaller: add section to README on ProductVersion property

bug-135: param_checker incorrectly checking optional strings

https://gitlab.khronos.org/vulkan/LoaderAndTools/issues/135

bug 134: param_checker using layer private data before it's been created

https://gitlab.khronos.org/vulkan/LoaderAndTools/issues/134

licensing: Add RenderDoc to LICENSE.txt

Merge branch 'stroyan_draw_state_beginCB' into 'master'

layers: draw_state allow pInheritanceInfo NULL

The pInheritanceInfo field in VkCommandBufferBeginInfo may be NULL.
Test for that before using it in draw_state layer.

See merge request !229

winrtinstaller: Remove mention of SDK from RT license file

winrtinstaller: Add file properties to rt installer executable

More api version bumps

layers: Update api version to 1.0.3 in json files

include: Use single-quote no paths when including sibling headers

Remove Khronos confidential clause from license text.

winrtinstaller: change to README so it matches sdk_1.0.3 version

winrtinstaller: Add section on error return values.

winrtinstaller: add more error checks during install, uninstall

layers: Validate fence not in use when deleted.

layers: Validate command buffer doesn't have duplicate query type.

layers: Validate pipeline stats query with secondary cmdbufs.

layers: Validate no active queries unless enabled.

layers: Enable Wall and Werror.

layers: draw_state allow pInheritanceInfo NULL

The pInheritanceInfo field in VkCommandBufferBeginInfo may be NULL.
Test for that before using it in draw_state layer.

layers: Don't skip over tessellation stages when validating stage interfaces

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

winrtinstaller: A new dir is created for reinstall of same version

Windows 10 was displaying a warning when an uninstall resulted
in the install dir not being deleted. (As happens when there are
multiple installs of the same version.) So we create a folder
for each re-install, the folder is removed on uninstall, and
Windows 10 doesn't get upset.

winrtinstaller: Allow uninstall to be run from any dir

winrtinstaller: fix install dir not recorded in registry correctly

winrtinstaller: install to to same dir if already installed. PUBLISHER changed.

PUBLISHER is set to YourComany, Inc by default

header: Update to version 1.0.3 which adds some version macros

loader: Change error to warning for missing layer registry key

Continue to report an error for missing ICD registry key

loader: add Architecture and interface document

windowssdk: Fix 32-bit install issue with Powershell script.

Fix the install Powershell script which populates the ExplicitLayers
registry entries.  There were 2 problems: 1) it was using newer
Powershell commands that failed on Windows 7, 2) it was trying to
populate the 32-bit data on a 32-bit system with the 64-bit Bin
folder.

Add .clang-format: LLVM style with IndentWidth of 4.

Adjust .gitignore to remove vktrace files and add cmake user file.

draw_state: Add FIXME/TODO for vkAcquireNextImageKHR(..., fence, ...).

layers: Redo draw_state locking--all in top-level functions.

All locking is now done in top-level, API-entrypoint functions.  Before,
locking was done at all levels of functions, which caused unnecessary locking
(e.g. within inner loops), and was not very maintainable.  This is cleaner,
easier to maintain, and should perform better.

linux: use linux naming conventions

loader: Null-terminated string checks

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