layers: Fix object_trackers dereferencing of invalid pointers

vkUpdateDescriptorSets->VkWriteDescriptorSet: object_tracker should only validate
array specified by VkDescriptorType.

layers: Fix problems with screenshot to get it working again

- Set the layerCount member of VkImageSubresourceLayers
- Add image transition memory barriers (needed for some drivers)
- Add dispatch table to command buffer object created by this layer.
  This lets it get dispatched by a downstream layer, if present.

layers: MR140, Add get_type_def helper in DrawState

layers: MR140, Add SPIRV instruction stream iterator helper

layers: MR140, Pass whole shader_module to build_type_def_index

layers: MR140, Remove some commented merge remnants

layers: LX257, Validate Uniform/Storage buffer offset alignments

loader:  Return the loader created instance rather than from the instance chain

Fix's issue with layers that wrap instance.

installer: Fix LunarXchange issue #287.

Apply David's fix to force the registry to be created.

layers: Fix json file for multi layer

laoder: Fix phys_dev to use unwrapped version in loader_CreateDevice

layers: Add minimal actual analysis of interface blocks to DrawState

There's definitely still room for improvement here (SPIRV-Tools might have some stuff that will make some of these more complex module walks a little clearer); however, this adds enough support for interface blocks for us to generate all the same kinds of mismatch errors we were previously generating for loose inputs and outputs.

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

layers: Fix mishandling of VI vs VS validation (Gitlab#69)

This wasn't quite correct. We'd end up trying to run it_a off beyond attribs.end(),
which is invalid and upsets the MS debug stdlib. This is most likely the root cause
of the weirdness that caused people to add _at_end, _first, etc to this function long ago,
so that can all disappear -- but for now, let's just deal with the actual bug.

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

nulldrv: Initialize image format properties

Previously this was just left uninitialized. Image layer would then randomly fail image creation based on comparisons against junk.

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

nulldrv: Add more proper support for ShaderModule objects

Again not OK to claim success but return junk rather than a proper handle. Fixes various layers exploding when running against nulldrv.

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

nulldrv: Add minimal support for pipeline cache objects.

It's not really OK to claim success but leave junk in the caller's memory rather than a real handle. This prevents object tracker randomly exploding when running against the null driver.

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

nulldrv: Claim that our supported image formats can be used as color attachments

Previously if an app would be unable to find any color renderable format, so only
an app that didn't bother (and wasn't running under validation) would get anywhere.

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

nulldrv: Expose some present modes

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

nulldrv: Expose some basic formats as presentable

Previously we just left whatever junk the caller had in this memory, which generally crashed a caller.

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

nulldrv: Claim to support any WSI surface.

Previously this just left whatever junk the caller had, with unpredictable results.

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

nulldrv: Initialize memory types mask

Previously apps would randomly succeed or fail to find a matching memory type based on whatever junk happened to be here. Instead, claim to support all memory types.

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

layers: gitlab #62, object_tracker allows NULL semaphore in AcquireNextImage

spirv-tools: update version due to build problems seen by norbert

loader: Fix bad memory ref and CreateInstance fail don't write ouptut

Gitlab #58
CreateInstance shouldn't update output parameter on failure.

Change api_version to 1.0.1 in json files

From 0.210.0

layers: Gitlab #71, Fix copy/paste error in mem_tracker

nulldrv: MR131, Update api_version in nulldrv json file

layers: MR130, Handle VK_WHOLE_SIZE properly in mem_tracker:initializeAndTrackMemory

VK_WHOLE_SIZE is ~0ull. If this were passed as `size`, we'd try to allocate ~1ull
bytes, and then memset them all to 0xb. This tends to explode.

layers: MR129, Remove incorrect asserts from param_checker and object_tracker

For apps that run with multiple instances, it's perfectly valid to have instances
remaining at DestroyInstace() time so removing that assert from param_checker.
Also, the assert appears to have been cut-and-pasted into DestroyDevice of both
param_checker and object_tracker. The same logic applies for apps that use
multiple devices. Running vulkancts with layers enabled found these issues.

loader: Add support for implicit layer device extensions enumeration

Partial fix for Bugzilla #15367.  Need to do similar things for instance
extensions.

Conflicts:
loader/loader.c

loader: Fix so unwrapped instance gets used for layer instance extension dispatch

Conflicts:
loader/trampoline.c

Windows RT Installer: Fix layer reg entry setup, no layers we being installed

demos: Remove the message box dialog for errors in vulkaninfo

demos: Fix vulkaninfo version reporting error

Windows RT Installer: add buildno to version

layers: MR128, Don't complain about declarations without Builtin or Location

This is a stopgap until we have proper interface block analysis landed.
For now, this just suppresses the false positives. A correct SPIRV
module will not hit this path for anything other than interface blocks.

Conflicts:
layers/draw_state.cpp

layers: MR128, Fix mismatched new[]/delete in validate_pipeline_shaders

In draw_state layer.

layers: MR128, Revert "layers: Work around Windows release build crash in cube"

This reverts commit 470d0226d222ac1532a88e5c5f0b78b7a391f7ce.

Conflicts:
layers/draw_state.cpp

layers: LX273, Fix accessMask validation for depth/stencil input attachments

When using an imageView of a depth/stencil image to populate a descriptor set,
the aspectMask must only include one bit and selects whether the imageView is
used for depth reads or stencil reads.

spirv-tools: update version for SDK 1.0 IHV

header: move to version 1.0.1

loader: Fix so unwrapped instance gets used for layer instance extension dispatch

layers: Remove extra subpass increment in draw state

loader: update linux version to 1.

layers: Gitlab 65, Add image layout extension enum to paramchecker validation

layers: Fix image layer format mutability validation

layers: MR127, Update DrawState handling of non-updated descriptors

At CmdBindDescriptorSets() time, only warn on non-updated descriptor if that descriptor is not empty.
At draw time, if a descriptor is being used and it has not been updated, trigger an error.

loader: Update README.md for ICD<->loader interface

loader: Fix old ICD interface to not use vkGetInstanceProcAddr(NULL,)

glslang: upgrade glslang version for SDK 1.0 IHV

header: Move to version 1.0.0 from 0.222

Whitespace only change.

misc: Move to Vulkan header version 0.222

layers: Gitlab 126, Fix for swapchain layer which was checking wrong pointer

layers: LX264, Validate immutable format bit in image layer

Fixes: Fixed compiler warning about &

Someone checked in a bitwise & instead of a boolean comparison &&.

Fixes: More compilation warning fixes

This fixes the main issues with VkBool32 versus bool.

layers: Gitlab 41 & 61, Fix invalid descriptor dereferences in helper codegen

Descriptor type arrays should only be dereferenced depending on the specified
descriptorType parameter. Caused crashes in api_dump.

layers: MR124, Fix semaphore ordering validation in mem tracker layer

layers: LX263, Validate minimum imageArrayLayers value in swapchain layer

layers: Fix unique_object  gpa function; no intercept of func without an
NDO
Also fix bug in list of KHR functions for table.

layers: LX256, Suppress warnings on pre-signaled fences in MemTracker

misc:  make sure host memory alignment is a power of two

Use sizeof(int) as a default generally.  Is that reasonable?

layers: MR123, Improved ObjectTracker codegen

Updated ObjectTracker to correctly generate the required permutations of validate_<object>()
functions. This removed some hand-coded functions from the header. Updated the codegen for
ObjectTracker to use improved dict from UniqueObjects that eliminates repeated "if" and
"for" blocks. Also updated valid null object checking to limit valid null object cases
to appropriate API calls.

swapchain: MR122, queueFamilyIndexCount must be > 1 for CONCURRENT

This was clarified via "<validity>" language in the "vk.xml" file.  It is the
case for VkBufferCreateInfo::queueFamilyIndexCount, and MR 995 will make it
consistent for VkSwapchainCreateInfoKHR::queueFamilyIndexCount.

layers: MR121, Use _BitScanForward for u_ffs on windows

The linux ffs function scans from least-significant-bit.
To match that, u_ffs on windows should use _BitScanForward instead of
_BitScanReverse.

layers: Output function names for deferred memory validation checks

layers: MR119, Defer validation until command buffer has been submitted

Conflicts:
layers/mem_tracker.cpp

misc: Remove the unique_objects layer from being enabled in tests and demos

tests: Fix a couple of test bugs revealed by unique_objects layer

layers: UniqueObject updates based on review

Remove all of the DebugReport stuff, this layer doesn't use it.
Update some corner cases to make sure objects are correctly restored.

layers: Updated UniqueObjects with improved codegen

Updated UniqueObjects object-use data struct so that generated code is much cleaner and handles a few of the more complex functions that were previously hand-coded. New data struct eliminates repeat "if" and "for" blocks.

layers: Add UniqueObjects layer to wrap objects in unique ptr

This layer should sit at the BOTTOM of the layer stack (closest to the driver, furthest from the app). It interecpts all created non-dispatchable-objects and wraps them in a struct, returning that struct ptr up the chain.

For all API calls that use NDOs, this layer will unwrap them, and overwrite the wrapped handle with the actual handle before passing it down. Then, after calling down to the driver, the layer will re-overwrite the NDO handles with the original, wrapped handle.

When an NDO object is destroyed, the actual handle is passed down, then the wrap struct object is destroyed.

update_external_sources: backed out last commit, broke win64 build

layers: Minor fixes to draw_state for new semaphore checks

Conflicts:
layers/vk_validation_layer_details.md

layers: MR118, Add validation for semaphore ordering

Conflicts:
layers/draw_state.cpp
layers/draw_state.h

layers: MR109, Fix second command buffer validation

Conflicts:
layers/draw_state.cpp

layers: MR108, Merge related updates and fixes

Also removed obsolete query test case from vkbase test

layers: MR108, Validate get queries

Conflicts:
layers/draw_state.cpp

icd: Use new loader interface for finding ICD entry points Remove library exports on all Vulkan entry points but vk_icdGetInstanceProcAddr.

loader: Find ICD entrypoints via vk_icdGetInstanceProcAddr first in ICD library
Also support prior method (now deprecated) of looking up vkGetInstanceProcAddr.
But now other globals are gotten from GetInstanceProcAddr calls.

Fixes: Fixed more Visual Studio warnings which appeared with the merge

Some new warnings popped up with the latest merge.

Fixes: Fixed a missing file change with Visual Studio warnings.

Fixed more Visual Studio warnings so we have a clean compilation.

Fixes: Fixed Visual Studio warnings during compilation.

Changes to properly detect proper Visual Studio and fix build warnings.

Fixes: Automatically detect Visual Studio version on Windows

Automatically detect and use the correct version of Visual Studio/MSBuild for Cmake.

Added X server DRI3 config info for Ubuntu 14.04.3 LTS.

draw_state layer: add validateAndIncrementResources return value

Bump Windows ABI to 1

layers: Update layer details doc for new validation errors

layers: MR107, Verify that clear is not used when first use is shader read

Conflicts:
layers/mem_tracker.h

layers: MR82, merge-related fixes

layers: MR82, Add lifetime validation for buffers to DrawState

Conflicts:
layers/draw_state.cpp
layers/draw_state.h

layers: LX259, Clarify draw_state accessMask warnings

Swapchain: Reconcile MD file with Swapchain layer.

Conflicts:
layers/vk_validation_layer_details.md

Bug 15392 (Swapchain): Don't check VkPresentInfoKHR's waitSemaphoreCount & pWaitSemaphores

The VkPresentInfoKHR's waitSemaphoreCount may be zero and pWaitSemaphores may
be NULL.  These are optional values.

Swapchain: Use an INFO instead of an ERROR for when pNext is non-NULL.

This is a compromise.  Currently, this is an error situation.  However, another
extension could be added that uses pNext, and then if the Swapchain layer isn't
updated, a false-error situation could occur.  By using an INFO message, the
user is still alerted of the possibility of a problem, but won't be annoyed in
the case of a new extension.

Swapchain: Change way p*Count is checked--make sure not bigger than should be.

Swapchain: Only check that queueFamilyIndexCount is greater than 0.

From review feedback.  We're not sure, and so we won't check that it is greater
than 1.

Swapchain: Update my authorship to include Google.

Swapchain: No longer validate VkSwapchainKHR (object_track will do) ...

Actually, leave one VkSwapchainKHR check in.  This is checking the
pCreateInfo->oldSwapchain value, which I'm not sure object_track will look
for).  Can remove this later, if object_track does check this.

Swapchain: No longer validate VkDevice (object_track will do).