demos: Query device features in cube prior to creating device

layers: Add constructors for layer_data

layers: Remove object map before vkDestroy*

Don't use object as dispatchable after destroy.

WSI Validation: Removed 2 checks that are driver-specific.

The purpose of this layer is to validate the application's usage of WSI, not to
validate the driver's conformance with WSI.

misc: Change all JSON files to abi version 0.9.0

demos: vulkaninfo: increase console buffer size so that output can be read

layers: remove static declaration of get_my_data_ptr

Windows compiler won't allow static on specialization.

layers: Image layer does not need to reserve space

Image layer had been calling imageMap.reserve to recover from memset of imageMap.
That is no longer needed.

layers: don't memset layer data in get_my_data_ptr

get_my_data_ptr was zeroing data after using new.
It should leave the layer data as its constructor set it.

Remove WSI from "General Pending Work" section of layer documentation.

Move documentation for Swapchain -> it's a validation layer

WSI Validation: Add error enum and document Swapchain layer.

misc: Move snprintf define to vk_sdk_platform.h

Windows doesn't define snprintf so we need to #define
it to what Windows does use in our SDK platform header.

Make layers build on android.

tri: cleanup valgrind errors

Integrate changes from merge request #5.

cube: Fix valgrind warnings

Change ERROR -> PERF_WARN in vkAcquireNextImageKHR().

WSI Validation: Initial WSI "swapchain" validation layer.

cube/tri: Changed invalid <size> argument value for vkMapMemory()

layers: LunarXChange #123 fix - Validate Update/FillBuffer outside RP

Validated that these are not called in an active RP, added validation tests
for these and some other renderpass-dependent APIs.

layers: Note pending work for ObjectTracker

Instance and device specific objects.
Refcounts of duplicate non-dispatchable objects.

layers: Added Image layer to validation test suite with new test for bad ImageView

layers: Update Image layer to support validation callback error return status

layers: Update Image layer to offload view.c validation checks from driver

Moved checks related to ImageView creation from the driver into the Image layer. Added image.h header and ENUMs for various Image error types, along with documenation for each type.

bug-14715: DrawIndirect fix

This patch contains fixes to structure layout for draws.
These structures are accessed by HW and must be in a specific order.
This change also includes reordering of the parameters for
vkCmdDraw and vkCmdDrawIndexed.

layers: Remove unneeded layer_common.h

The header only had one include in it.

layers: layer generate confused by Count parameter

The generator was getting confused by the argument list of
vkCmdDrawIndexed(cmdBuffer, indexCount, instanceCount, firstIndex, vertexOffset, firstInstance)
It saw instanceCount and then wanted to expand firstInstance.
I had to terminate it match search after one miss to stop it recognizing firstInstance as an array of Instance items.

loader: Remove instance validation checks from debug_report.c

Cannot reach these points with an invalid instance value.

layers: release mutex in DrawState

Release mutex when last instance is destroyed.

layers: VkDestroyInstance+VkDestroyDevice clear ObjectTracker maps

Don't complain about lingering objects more than once.
Once their device or instance is gone don't keep their information around.
This should change dramatically to have independent maps for each instance/device.
Don't complain about other devices remaining in vkDestroyDevice.

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)