update_external_sources: get glslang from gitlab

This is the new way to get SPIR-V rev32 source.

Only check blend state if blend is enabled.

Add basic validation for the shader module.

layers: Add DrawState checks for dynamically set Viewport and Scissors

Make sure that counts from dynamically set viewport and/or scissors match the PSO counts.
Added tests to verify these cases.

layers: Drawstate verify viewport and scissor state at PSO creation time

Make sure that viewportCount and scissorCount match
If viewport is not dynamic, make sure viewport state is supplied and if viewportCount is non-zero, that viewport data is non-NULL.
If scissor is not dynamic, make sure scissor state is supplied and if scissorCount is non-zero, that scissor data is non-NULL.

Added tests to hit these new cases.

loader: Fixes in trampoline code to support layers which wrap objects

Loader trampoline code may see wrapped objects.  Don't do value comparisons
of dispatchable objects but instead compare dispatch tables to find objects.
PhysicalDevice objects where may have multiple gpus with same instance dispatch
will be fixed in a later patch.

vktrace: Update README

tests: Add DeviceLimts and Image layers to run_all_tests_with_layers.sh

layers: Validation -- updated layer validation doc file

Corrected validation layer error code.

loader: fix use of freed memory

vkEnumerateInstanceExtensionProperties was using freed memory.
Avoid free of NULL pointers.

layers: Updated README.md and details doc

README.md was woefully out of date so this is a big improvement, but still more work to be done. May want to merge this doc with details doc in the future as they cover similar ground.

Add "Swapchain" layer to the "run_all_tests_with_layers" test.

Build doc: Correction to the cmake command to be used for windows

vulkaninfo demo: Reduce the size of the console window so it fits on smaller screens

sdk: Update release notes for 0.9 SDK

loader: Fix vkGetInstanceProcAddr to handle debug_report extension

Need loader entrypoints for debug_report extension including the utility
functions. Don't call down the instance chain GPA for this extension.

Remove instance extensions decoding when GPA instance == NULL as don't want
to return extension entrypoints unless they are enabled.
This meant the WSI swapchain instance GPA was no longer used so remove it.

layers: Validate draw commands appropriately inside/outside render pass

Updated DrawState layer and validation tests to check that all draw commands
are correctly called inside or outside (or either) of an active render pass.

tests: Update ResetUnsignaledFence test to expect WARN rather than ERROR

layers: Fix DrawState to correctly handle dynamic state from PSO and CmdSet* calls

Track which dynamic states are set via PSO and merge that with dynamic state set by CmdSet* cmds.

Updated viewport not set test to pass when scissor is flagged as missing. Currently scissor and viewport set together so scissor error masks viewport error.

layers: DrawState allows callback to set fail on DescriptorType mismatch

If the update DescriptorType does not match the layout DescriptorType, DrawState would previously bail automatically. This update flags the error and will only bail if the callback return code is VK_TRUE, like with all other validation errors.

Merge branch 'mlentine_layers4' into 'master'

Make reseting an unsignaled fence a warning.

See merge request !38

bug-14365: fix misunderstanding in dynamic state

I had interpreted the pDynamicState array to indicate which
states defined in the PSO should be used. It should be
which states are dynamic. Had to invert the meaning inside
the driver and fix up the render framework to think of
all PSO state as static unless told to be dynamic.

vktools: Need to capture & replay new pDynamicState

Set dynamic pipeline state as dynamic

Make reseting an unsignaled fence a warning.

vktrace: Some TODO additions and deletions

loader: Change GetXXProcAddr to support NULL dispatchable object

Also change GetInstanceProcAddr to return the first entry down the chain rather
than global (trampoline entrys) all the time when a non-null instance is passed.

layers: DrawState handles dynamic state set via PSO

For dynamic states flagged in PSO, consider them set at draw time.
Clarify various dynamic state error messages to make them more consistent with updated dynamic state API.
Add test for scissor not bound.

layers: Disable vktrace layer if loaded incorrectly

The vktrace layer is not meant to be run as a stand-alone layer --
it must be started by the VkTrace application.  This patch will
disable the layer preventing a flood of failure messages if the
layer is loaded like other layers.

tests: Remove std140 to avoid warnings

layers: Intercept vkEnumerateInstance[Layer|Extension]Properties() funcs in DeviceLimits

Fix windows build issues in cube.c and device_limits.cpp

layers: DeviceLimits device feature checks and dead code removal

Refactored DeviceLimits to store maps in the layer_data struct in order to reduce map look-ups.

Added new checks and documentation related to Features checking. If specific features are requested at vkCreateDevice() time by the app, then they are checked against the supported features of the physical device. Requesting unsupported features cause validation errors. If the user did not query features and is asking for any unsupported features, then a warning message recommending that they query features is also printed along with the error(s) for unsupported features.

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.

tests: cleanup valgrind errors

Integrate changes from merge request #5

tri: cleanup valgrind errors

Integrate changes from merge request #5.

cube: Fix valgrind warnings

tests: cleanup issues found by valgrind

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.

Remove validation checks from query.c

These checks are already handled by the ParamChecker and ObjectTracker validation layers.

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.

bug-14365: fix viewport & scissor count

loader: Remove instance validation checks from debug_report.c

Cannot reach these points with an invalid instance value.

vktrace: remove quoting to stop cmake warnings

Stop cmake complaints about quoting CMAKE_C_COMPILER_ID in build_options.cmake.
Stop cmake complaints about quoting SRC_DIR in CMakeLists.txt include path.

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.

tests: Remove duplicate calls to InitState

Remove duplicate calls to InitState before VKTriangleTest.
VKTriangleTest calls InitState.

tests: Destroy objects after each validation test.

Add calls to destroy objects after validation tests.

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: Verified that driver validation checks are covered for fb.c

intel: Removed ICD validation checks from dev.c

intel: Verified that driver validation checks in queue.c are covered

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

vktrace: tracer set env vars for enabling Screenshot layer if needed

No Longer need to enable ScreenShot layer manually if the tracer has
screenshot option it will enable the layer prior to forking child.

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

vkreplay: Fix CreateInstance so the ScreenShot layer is inserted if needed

Also replay replay settings debugLevel as part of general cleanup of replay
CreateInstance.

vktrace: Remove the tracer library options and support for multiple tracers

Only support one tracer now Vulkan and it is loaded into the process as a
Vulkan layer library.

vktrace: Remove getting tracer ID from the tracer library

vktrace binary required to load tracer library because of this. Change it to
use Vulkna tracer always.

First step in removing the -l0 requirement.

Fix error return code in pipeline.c

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.