Added request for VkShaderCreateInfo.stage check

build: fix update_external_sources.bat to pick up SPIRV rev32 for glslang

loader: Remove validation checks from loader.c

These checks are covered by ObjectTracker.

bug-14365: add dynamic state to PSO

This commit covers phase 2 of the removal of dynamic
state objects. Now, an application can include an array
of VK_DYNAMIC_STATE_* values that tell the driver
which PSO dynamic state elements to use.
I.e. if VK_DYNAMIC_STATE_LINE_WIDTH was specified in the
pDynamicState array then the ICD should use the lineWidth
defined at PSO create time and ignore any set using
vkCmdSetLineWidth.
To accomplish that the driver will make a copy of the
dynamic state specified in the PSO as well as a bitmask
of the affected state. When vkCmdSet* is called, the
driver will check if a PSO override is current and ignore
the call if so.
At PSO bind time the command buffer's dynamic state
will be updated and the PSO override bitmask set so that
any future vkCmdSet*'s will be appropriately ignored.
TODO: Validation layer should probably indicate a warning
if app tries to do vkCmdSet on state defined by the PSO.

bug 14365: make separate viewport and scissor cmds

bug 14365: replace dynamic state objects

This is phase 1 of the replacement of dynamic state objects.
In this commit we remove the create and destroy functions
and rename the CmdBind... with CmdSet...

layers: Add function to check if msg will be logged

A layer can use will_log_msg to determine if any output
at the designated notification level(s) will be sent
somewhere. This will allow a validation layer to avoid
high compute validation tests if the results are going
to be looked at.

intel: Removed ICD validation checks from dev.c

build: move to SPIR-V rev 32

intel: Removed ICD validation checks from img.c

Moved validation into device_limits layer, created layer utils
files for format utilities, added validation tests, updated validation
doc, updated test framework to call GetPhysicalDeviceProperties.

Remove libpng and zlib dependencies, add copyrights

loader: Add lock around cJSON so big loader lock is not needed as much

cJSON not thread safe.  Add a lock around cJSON calls so instance
extension and layer queries don't need the big lock.  This allows layers
to do queries from CreateInstance.

layers: DeviceLimits warns if queue properties not queried prior to vkCreateDevice()

demos: Update Cube to select graphics queue based on query of queue properties

Also enabling DeviceLimits and Image layers when --validate option selected.

build: fixes for latest SPIR-V rev 31

Primarily, this deals with new binding field in metadata rather than having
location field do double duty.

WSI: Small code changes to deal with latest WSI header.

A structure's sType was missed in the grand renaming that took place a while
ago, and was fixed with the latest WSI header changes.

WSI: Update vk_ext_khr_device_swapchain.h to revision 53 (SDK 0.9)

layers: ShaderChecker can attempt calls on errors

Some errors are non-lethal.  Make calls after finding errors.
Allow attempts to read an unwritten output
or to use a fragment shader that will not write all outputs.
Just make the error callback and continue with that API call unless
the boolean callback result is true, indicating the call should be prevented.

layers: fix ShaderChecker hang on stage mismatch

validate_interface_between_stages was sometimes iterating past end of maps.

layers: make DrawState globals static

Prevent symbol name collisions, especially with other layers.

build: bring update_external_sources.sh to latest rev 31 SPIR-V

Finished migration of validation checks out of cmd_pipeline.c

Added check to DrawState layer for submission of Primary cmd buffer in vkCmdExecuteCommands() call. Added a test to verify the check.

Replaced some other checks in cmd_pipeline.c with asserts and the checks themselves are already handled in ParamChecker.

layers: Add Idx buffer alignment check to DrawState

At vkCmdBindIndexBuffer() time, verify that offset param falls on an aligment boundary as set by indexType param. New check enum is DRAWSTATE_VTX_INDEX_ALIGNMENT_ERROR.
Removed checks for this from the driver and added a validation test and documentation for the new check.
ParamChecker has a check to verify indexType param is an acceptable value, but it's after the call so documented a TODO for ParamChecker to push this check prior to call.

Remove old checks from pipeline.c and verify that they're handled in validation

Most checks were already handled in DrawState.
Added additional check for invalid patchControlPoints along with a new test, but test can't run on sample driver b/c TESS shaders not supported by the shader compiler.

For check that VBO vtx & attrib binding counts don't exceed VB array limit, added comment noting that this is a device-specific limit that should be captured in API-defined "maxVertexInputBindings" and added to DeviceLimits.

Also added pending task for ParamChecker to fix how it handles arrays in vkCreateGraphicsPipelines() call. It has a bug in how it handles check that each shader "stage" is within acceptable enum limits.

layers: Fix bug where logging_callback was not being correctly cleaned up

layers: Make passing a non-SPIRV shader to CreateShaderModule an error

Various real drivers are not tolerant of the wrapped-GLSL-blob hack that
was used early in Vulkan development. Rather than simply bypassing most
of the validation when one of these blobs is seen, make it an error.

Structurally, move the validation out of the shader_module ctor in
preparation for the callback being able to signal whether to bail out.

V2: Adjustment for descriptor validation landing

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
Reviewed-by: Tobin Ehlis <tobin@lunarg.com>

layers: Fix APIDump to handle "stdout" special case

Also setting example vk_layer_settings.txt file to have APIDump write to stdout.

loader: Fix a few possibly unitialized variables

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.