layers: Add documentation for SHADER_CHECKER_MISSING_DESCRIPTOR

layers: Require pipeline layout to contain all referenced descriptors

If the SPIRV image uses a descriptor (for constant buffer, image, etc)
we must declare it in the pipeline layout.

We won't complain about unused junk in the pipeline layout, but not
declaring something that *is* used is an error.

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
Reviewed-by: Courtney Goeltzenleuchter <courtney@lunarg.com>

layers: Collect descriptor usage from SPIRV images

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
Reviewed-by: Courtney Goeltzenleuchter <courtney@lunarg.com>

layers: Shadow descriptor set layout in ShaderChecker

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
Reviewed-by: Courtney Goeltzenleuchter <courtney@lunarg.com>

layers: Update DrawState array of PIPELINE_NODEs to be vector

layers: Add return value for Generic vkCreateInstance() function

misc: Enable nulldriver on Linux

Build the null driver on linux so it's easier to keep it up-to-date
with header changes.

bug 14537: nulldrv: add CreateFlags to ImageFormatProperties

nulldrv: Remove unused function

nulldrv: Remove return code for void functions

misc: bump header version to 170

This is version of header included in the provisional spec release
branch: release_provisional_I_20150910

Bug 14318: Clarify image layout transitions in a RenderPass

merge request 337.

bug 14640: add pNext/sType to VkDeviceQueueCreateInfo

Bug 14644 - add poolUsage/maxSets to VkDescriptorPoolCreateInfo

Bug 14644 - vkCreateDescriptorPool should have poolUsage/maxSets as part of VkDescriptorPoolCreateInfo
merge request #342

bug 14437: Remove VK_PIPELINE_STAGE_TRANSITION_BIT

misc: Bump header version

shader_checker: Do not hold lock across API call.

bug 14564: and initial layout for image creation

misc: Add maxVertexInputBindings

Merge request 298.

misc: code movement due to header build changes

Bug 14643: use VkClearDepthStencilValue

bug-14538: Remove validation error codes

https://cvs.khronos.org/bugzilla/show_bug.cgi?id=14538

bug 14509: Rename Layer and Extension queries

https://cvs.khronos.org/bugzilla/show_bug.cgi?id=14509
https://gitlab.khronos.org/vulkan/vulkan/merge_requests/334

layers: Move some fixed code into header files for APIDump and Generic

layers: Fix intercept of WSI instance entrypoints

ApiDump, Generic, ObjectTracker now intrcept WSI instance entrypoint and
only intercept if the extension is enabled

layers: Fix several bugs in WSI extension handling in layers

Layers weren't intercepting the WSI device extensions proerly, or weren't
NOT intercepting them if the extension wasn't enabled.  Fixes to APIDump,
Generic, ScreenShot, MemTracker, ObjectTracker

layers: No assert on init instance table map.size != 1

Instance table map can have more than one entry if multiple instances.

layers: Remove the DEBUG_REPORT entrypoints from device dispatch structure.

these entrypoints are instance level.

loader: Remove windows disable of global optimization.

Now that vktrace now longer uses mhook, can reenable global optimize on Windows.

layers: Update DrawState to handle creation of multiple gfx pipelines

layers: Various memtracker TODOs, fixed some error messages

layers: Update vk_layer_settings.txt to clarify CALLBACK option and change DebugAction for all layers to "LOG_MSG"

layers: Improve invalid object checks of ObjectTracker

ObjectTracker now correctly stores queried PhysicalDevice and Queue information and validates that these objects are correct whenever passed in as params. Now handling arrays of Objects passed into API functions as parameters. Removed special exceptions for VkFence, VkDeviceMemory, and VkImage types that were causing them to skip some checks.

Moved a few validation tests that had been under MemTracker into ObjectTracker as it now catches the invalid VkImage and VkDeviceMemory cases. Updated master document to reflect this.

demos: trivial fixes for cube problems

loader: Need to init instance dispatch without layers

Application that enabled DEBUG_REPORT but did not register
any layers would get into a deadlock due to improperly
initialized instance dispatch table.

layers: Add return value to non-void function

layers: Migrate Fence checks from ObjectTracker to MemTracker

layers: Example vk_layer_settings.txt file

layers: Fix layer STDOUT logging

layers: Migrate MemTracker to allow log_msg callback to signal early return

demos/tests: Fix aspectMask updates in a few spots

layers: Another CreateDevice fix (was using WSI instance extension name)

layers: Check if CreateDevice enabling WSI device extension, not instance

Some of the layers were checking whether VK_EXT_KHR_SWAPCHAIN_EXTENSION_NAME
(the WSI instance extension) is in the list of device extensions being enabled.
This is wrong.  They should be checking whether
VK_EXT_KHR_DEVICE_SWAPCHAIN_EXTENSION_NAME (the WSI device extension) is in the
list of device extensions being enabled.

vktrace:  Really put vktrace_layer.json and vktrace_layer.dll in the correct output dirs

This time for sure!

vktrace: Put vktrace_layer.json and vktrace_layer.dll in the correct output dirs

bug-14583: Remove the unneeded vkBufferViewType

No longer have viewType to distinguish the type, so pull
it internal and use for descriptor sets.

Bug 14583: Updates based on initial feedback

Bug 14583: Remove VkBufferView for UBO/SSBO descriptor types

For descriptor types:
VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER
VK_DESCRIPTOR_TYPE_STORAGE_BUFFER
VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER_DYNAMIC
VK_DESCRIPTOR_TYPE_STORAGE_BUFFER_DYNAMIC
No longer need to create a VkBufferView, instead can
put the buffer, offset and range directly into a descriptor.
Many places in the driver assumed it could point to a persistent
buffer view and that is no longer the case. Now cache the
view info rather than a pointer to deal with that.

vulkan: Add maxDescriptorSet*BuffersDynamic to limits

vulkan: Add missing _BIT to some bitfield enums.

Merge request #290 (https://gitlab.khronos.org/vulkan/vulkan/merge_requests/290)

vulkan: Bump version to 166

Bug 12998: change ImageSubresourceRange's aspect to aspectMask

Bug 12998: change imageAspectMast to aspectMask

change vkCmdClearDepthStencilAttachment imageAspectMast to aspectMask

bug 14313: Allow copies to do multi-layer copies

layers: Migrate ObjectTracker to updated Validation error reporting

layers: Fix spurious WSI memory binding error messages in MemTracker

Bug caused invalid errors in some games.

layers: Migrate DrawState to new error reporting allowing for callback to signal early return

Bug 12998: Document required depth formats and layouts in buffer memory

bug 14530: Remove IMAGE_CREATE_INVARIANT_DATA_BIT

Bug 14587: Rename flags: XXX_CREATE_SPARSE_BIT

XXX_CREATE_SPARSE_BIT --> XXX_CREATE_SPARSE_BINDING_BIT

bug 14313: Rename ArraySlice to ArrayLayer

Bug 14313: Remove VK_FORMAT_FEATURE_CONVERSION_BIT

bug 14516: rename VK_IMAGE_USAGE_DEPTH_STENCIL_BIT

into VK_IMAGE_USAGE_DEPTH_STENCIL_ATTACHMENT_BIT

bug 14541: rename VkTexAddress to VkTexAddressMode

bug 14537: Improve VkImageFormatProperties

Bug 14537: Improve VkImageFormatProperties

Commit: e45df2e

bug-13139: refactor sparse properties

layers: Reenable MemTracker reference tracking

Which was disabled during the type-safety dark times.

nulldrv: Fix compiler issues

vulkan.h: Change return types of some functions to void

Functions that should be thought of as "impossible to fail in the face
of valid parameters" have had their return types changed to void.
This includes all of the vkDestroy functions, vkFreeMemory, and vkUnmapMemory.
vkUpdateDescriptorSets is also included, because of the frequency the function
is expected to be called.

layers: Add note about vkCreateDevice behavior

vulkan: Add validation error result

When running with validation layers a validation layer
may abort an API call (due to app callback indicating so).
Since there are no validation error codes in vulkan.h
we need to define one here to return for such cases.
Details of the validation failure are available via the
DEBUG_REPORT callback.

demos: Do not bail on validation error

draw_state: Implement bailout case for one test

Proof out the bailout case. Fill in others in following
commit(s)

layers: Add bailout flag to debug report callback

Layer validation tests will deliberately induce
errors that the validation layer should catch.
In these cases we don't really want the layer to call
down the chain once it's detected a failure. We can't
know that in the layer so changing the return value
on the callback from void to VkBool32 so that the
callback can indicate if the call should continue
or not. true = bail.
That allows the call chain to execute normally,
not segfault in the driver and allow the test
to clean things up.

mem_tracker: validation comment

bug-14538: Remove driver validation checks

Marked validation checks done in the driver with
TODOVV. Once we verify the check is covered in a
validation layer we can remove the driver code.

layers: Remove unused functions

layers: Add DeviceLimits json files

layers: In DrawState only validate bound VBO index at the time of Draw

Previously validating this at the time of VBO bound and when PSO bound. This could result in checking against stale state, though, and thereby causing false positives.

layers: DrawState fix to prevent false positives when matching PipelineLayouts

Updated existing check where PipelineLayout found with vkBindDescriptorSets was always checked against PipelineLayout from PSO. Since there are cases when a Draw requires no Descriptors the check was modified for now to only occur when vkCmdBindDescriptorSets has been called. This is still not perfect but will prevent false positives.
Really need to identify when it's valid to not call vkCmdBindDescriptorSets and, if so, don't perform the PipelineLayout match check.

demos: Fix demos to correctly query physicalDevice count

layers: Fix ScreenShot layer to handle EnumeratePhysicalDevices query count case

layers: Initial framework for DeviceLimits layer

This includes many initial entrypoints and setting up function intercepts with only a few actual checks to begin. DeviceLimits layer is intended to capture two broad categories of errors:
1. Incorrect use of APIs to query device capabilities
2. Attempt to use API functionality beyond the capability of the underlying device

DeviceLimits stores its own internal record of underlying device capabilities and flag errors if requests are made beyond those limits.

Initial checks and documentation include verification of EnumeratePhysicalDevices call and some basic queue create/get validation.

loader: Use GetProcAddr function names from layer manifest file

No longer assume vkGetInstanceProcAddr or <layerName>GetInstanceProcAddr to find
layer's instance GetProcAddrs. And similiar for device ProcAddr. Instead use
any value specified in manifest file. If none specified in manifest file then
assume vkGetInstanceProcAddr or vkGetDeviceProcAddr.

demos: Prevent tri rendering during shutdown

vulkan.h: Refactoring physical device sparse properties and limits, bug# 13139

Part of changes for header revision V161.

vulkan.h: Add bufferFeatures to VkFormatProperties, bug #14546

Windows build: rename vulkan.0.dll to vulkan-0.dll

documentation: Detail DRI3 use on ubuntu 15.04

Updated DRI3 discussion in README.md and BUILD.md.

loader: Fix Windows build for alloc stuff

loader: make thirdparty code use allocation callbacks

Use TLS to keep instance pointer and use this for cJSON alloc/free
callback.

demos: Add allocation callback to tri test CreateInstance

loader: Make use of app provided allocation callbacks

Convert all heap allocs/frees to use loader_heap_alloc() or loader_heap_free().
Before CreateInstance this will use malloc/free. At (and after) CreateInstance
alloc callbacks are stored and used for any allocations.

Exceptions are thirdparty code (cJSON  and dirent_on_windows) still always use
malloc/free. Plan to address these in later patch if ti makes sense.

loader: Add support for realloc using the app allocation callbacks.

loader: convert to using loader_stack_alloc for consistency

WSI: Convert from VERSION to REVISION in nulldrv

All extensions single revision number instead of major.minor.patch

This was decided during the August face-to-face.  Originally, the WSI
extensions did it the normal OpenGL way (single revision number), and the debug
extensions did it the new major.minor.patch way.  When I tried to convert WSI,
an objection was raised.  We decided to go with the OpenGL way.

This commit also changes vulkaninfo to report the single revision number.

This commit also changes the extension "number" assigned to the 2 debug
extensions, basd on a "registration" email that Ian Elliott sent to Jon Leech
on 21 August, 2015.  The initial set of "registered" extensions are (with their
numbers):

1.VK_EXT_KHR_swapchain
2.VK_EXT_KHR_device_swapchain
3.VK_EXT_KHR_display
4.VK_EXT_KHR_display_swapchain
5.VK_EXT_LUNARG_debug_report
6.VK_EXT_LUNARG_debug_marker

WSI: Convert from VERSION to REVISION

WSI: Update headers for WSI swapchain extensions (versions 51 & 17).