vulkaninfo: Show TexelOffset in decimal, instead of hex.

loader: GH362 - Fix json output of escape chars

The strings from the JSON file already add escape characters.
So, \ becomes \\ in the JSON file strings.  However, the cJSON
library was adding \\ for ever encountered \ when converting
to a string.  This became messy as C:\\vulkanSDK\\layerfile.json
became C:\\\\vulkanSDK\\\\layerfile.json.

Change-Id: I006252e33d6e91e2bef704dd5dee0777105388a7

loader: Fix GH607 by adding proper array support

JSON spec does not allow objects of the same name at the same
level.  But, that's what we used to allow with "layer" definitions.
Update new file version to 1.0.1, add "layers" array object, and
add multiple "layer" definition warning.

Change-Id: I040f07897c689800364d243daf3c247f1b3cace7

loader: return ext not present when applicable

tests: Fix type mismatch in wrap_objects

Change-Id: I4544293ae03565f345650a2dd41c8e36b5ab8205

vkjson: Include instance layers and extensions

vkjson: Add ability to dump an entire instance

vkjson: rename vkjson_device.cc -> vkjson_instance.cc

vkjson: Add VkJsonGetInstance()

To prepare for being able to dump an entire instance, vkjson_info now
always gathers info about all devices using VkJsonGetInstance, and
then dumps whichever devices were requested.

vkjson: Rename VkJsonAllProperties to VkJsonDevice

In preparation for introducing VkJsonInstance. Typedefs and inline
functions are included in vkjson.h to preserve source compatibility.

vkjson: Add .clang-format

Future changes to vkjson should use consistent code formatting. The
vkjson-specific .clang-format will define the style to be used; I
picked Chromium style because it's the closest to the style of the
current code.

To avoid polluting git history/blame, I'm not reformatting all of the
existing code at this point. New changes should be run through
git-clang-format to reformat any modified lines. Recommended
configuration is:

  git config clangFormat.style file
  git config clangFormat.extension h,cc

layers: Fix CMakeLists.txt for non-MSVC Win builds

Fix a recent change broke windows builds that don't use the VC IDE or
decorate build trees like VC does

Change-Id: I93c8a559c4255408a6fa890b2a3e4d81822d1cea

vulkaninfo: Clean up XCB window creation code.

Change-Id: I5ef0862326d9a7ce4e59fe7c8947b532f252d604

layers: Update docs for shared utils DLL

Change-Id: I129b4d074f0604bbdf6d81c052333229ece99ac3

layers: Add debug report header

Enable output of a short message header containing definitions of
debug message types.

Change-Id: Ia84fb0e79d599b8d022b7d78267b7b6a0307823e

layers: Make vklayer_utils a shared library

Was previously statically linked.  Converting to a DLL will allow
data sharing and use of utils in samples repo.

Change-Id: I63ccc1f0f57dfad40f7068c60ac623bce5bd714d

tests: remove unused flags

Change-Id: I41ffcd1b89d7e18eb4bab3a1cdfb5e92dff017cb

layers: Fix Windows Dota2 crash

One of the event tracking lists was not getting cleared upon a reset,
resulting in an invalid reference on reuse.

Change-Id: I45d932822f27197266b363634d6cdccc785209d3

layers: Fix test fallout in CommandBufferTwoSubmits

This test was accidentally triggering the fence reuse case as well, and
blindly continuing past that error. It doesn't need to use the fence at
all, so don't supply one.

Also tweak the layer to not run into the assert if the caller elected to
skip -- QueueSubmit still has validation and updates tangled together,
but this improves it slightly.

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

layers: Fix mistracking of semaphores associated with submissions

When a command buffer is simultaneously inflight multiple times, each
can have different semaphores associated. Storing the set of semaphores
on the GLOBAL_CB_NODE caused us to get confused and emit bogus errors.

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

layers: Note broken behavior around stealing tracking from other queues

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

layers: Add members to FENCE_NODE and QUEUE_NODE to hold the handles

We'd like to be able to get back to them from the tracking struct.

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

layers: Add helpers for getting fence and queue nodes

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

layers: Add CB_SUBMISSION struct

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

layers: Unify fence checks for queue functions

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

layers: Remove dead fence members from GLOBAL_CB_NODE

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

tests: Add simultaneous submission test with signal semaphores

This exposes some bad behavior in core_validation, as shown in #626.

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

layers: remove tracking of last queue and fence for CB

Unused.

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

layers: Update manifest files for vers 1.0.16

Change-Id: I7468c088d99f7bebabc05d1a84529f6e9e00d424

misc: Update to Vulkan header version 16

Only change is the addition of AMD extensions.

Change-Id: Ia4270600247b2e3f406cdac0cfff3492ec1ce109

tests: add dispatchable object wrapping layer

Change-Id: I181fd9b4363d98d4da63017b8bdce68e86e8aefc

Revert "loader: Fix build warning on windows"

This reverts commit 27a58419aab1bd62201e00ea5a6d822b65f99fa3.

loader: Fix build warning on windows

Don't use default library for libcmt.

Change-Id: I68c0f629044713c0c875758af9a0a1dd345dad75

Added {} to if statements.

Change-Id: I5ef0862326d9a7ce4e59fe7c8947b532f252d604

Removed some window event handling code.

Change-Id: I5ef0862326d9a7ce4e59fe7c8947b532f252d604

Applied the "clang-format" obfuscator

Change-Id: I5ef0862326d9a7ce4e59fe7c8947b532f252d604

Show "Presentable Formats" per GPU for Win32/XCB/XLib

Change-Id: I5ef0862326d9a7ce4e59fe7c8947b532f252d604

tests: GH536, Updated Fill/UpdateBuffer test cases

Change-Id: I529ea57f0c4d68bc65baec1d953c1ff60fd331b2

layers: GH536, Updated layer details doc

Added new cases, removed old enums, moved info from devlimits
to parameter validation.

Change-Id: Iddf6a679dc8596dcf411a6b880a63a26999607ec

layers: GH536, Update CmdFillBuffer validation

Was ignoring VK_WHOLE_SIZE. Moved validation cases for Update/FillBuffer
to Parameter Validation layer (as they were stateless) and added the
remaining additional stateless valid usage checks.

Change-Id: I0f41c6ba55f218a6db9fb34736a2abc52f856edf

winrt: fix error on uninstall

Remove old log files before writing them. When 2 uninstalls
are done in a row, this caused the uninstaller to display
an error

winrtinstaller: display error num in ps warning window

Display an error number in warning window if powershell
script fails.

winrt: configure loader even if ps script fails

tests: Add missing DestroyRenderPass call in LVT

Change-Id: I0f483883365abe34ce304ab6923e2507d8983367

tests: Fix line-endings in powershell scripts

Change-Id: Ibf90d79a93cc0689e647942271d86faade0bb1ca

tests: Fix instructions in doc validator

Change-Id: Ie0d5d87da5e36e4206c6af73451ff28e3ad4b100

winrtinstaller: fixed problems introduced by piping script to powershell.exe

PS script return value is now correctly retrieved by NSIS script.
Logging is less verbose, removed extranous errors.

layers: PR573, Handle imageArraySize rename

Swapchain layer had stale references -- rename things still using
imageArraySize instead of imageArrayLayers.

Change-Id: Iaa4500ab8cf55e50b96636a50136605dd3e6d83e

layers: PR632, Updated layer details doc

Added device_limits MISSING_QUERY_COUNT

Change-Id: I5b129ec12930129f5d5fe46bbe2622977dc2d992

layers: PR632, Change devlimits errors to warnings

Replace call sequence errors with warnings for
vkEnumeratePhysicalDevices and GetPhysicalDeviceQueueFamilyProperties.

Change-Id: I1505f4a2b339ac7ef435da8b3ec2f85612026a42

layers: PR629, Remove subpass warnings

The validation layer checks for dependency between subpasses using the
same attachments, but it also issues warnings if there are "only"
implicit dependencies between two subpasses. This is explicitly
allowed by the spec and should not cause validation warnings.

Change-Id: I1039955d6d445e1ee234116c6da2e4dbaf9c7dd9

layers: Initialize write_in_use in eventMap entries

Change-Id: Ic296788beb078f19a35b93b99df5dbf677c95122

winrtinstaller: avoid executing ps scripts, instead just pipe cmds to ps

demos: Add -lrt to link options for smoke LVL #500

Change-Id: I0480e60e3c6c2f06f24022332640756bcdc3f1a0

tests: Remove 2 queue requirements for tests that only use 1

Change-Id: I95abf1b5414ca8ffc28397f07a8d20dcb5f405bc

tests: GH421 reflect vkAllocateDescriptorSets test

InvalidDescriptorSetLayout test was added with 772fcd1c

tests: GH421 reflect vkCmdBindDescriptorSets test

Test was added with 772fcd1

layers: Add check for image usage in vkCmdClearColorImage

Change-Id: I7ba803a11de29e65ae65fbf0a0168561a2e6739f

winrtinstaller: retry ps script failure with full path to powershell.exe

Also changed diagnostics to log the contents of dir containing
powershell.exe, and log the current value of $PATH.

loader: doc update for merge android and desktop layer interface

Change-Id: Ibbffc0bf05bcbd1aa3b9272caa0b2c1c3a54037d

demos: Added --suppress_popups flag

Added a command line option to cube and tri to suppress
windows message boxes to facilitate Windows testing.

Change-Id: I069f9b199c9740e1e21c42100ceec42d5fb8f0e2

doc: Expand LVL acronym in BUILD.md

layers: Update docs for default dbg msg callbacks

Change-Id: Ic1f192ea7c95762d7e53f091cc9f0ea55be96c04

layers: Move debug action enums from vk_layer.h

Moved debug-report related items from vk_layer.h. Enums are now in
vk_layer_logging.h and DbgFunctionNode has been moved into a new
loader header file vk_loader_layer.h.

Change-Id: I6031146ba474ff01ca039da44ad5d42d054383a3

layers: Add default layer error message config

Allows layers to output error messages even if no vk_layer_settings.txt
config file is present. Sets defaults to LOG_MSG, error, stdout.
A layer settings file will override any default values. If no settings
file is present and an app creates a debug callback, the default
callbacks will be removed and unregistered.

Change-Id: I49f37189665816df58c258b9e9629f2bf76751c8

winsdkinstaller: add cmd copy as additional diagnosic for ps failure

winrtinstaller: add more logging debug code to powershell script invocation

winrtinstaller: fixes and diagnostics for powershell config script failure

windows: Allow run_all_tests.ps1 to skip tests

Use -TestExceptions to filter out tests with known problems.
Test list should follow a space, separated by a colon, i.e.

powershell -command .\run_all_tests.ps1 -TestExceptions ^
VkLayerTest.RequiredParameter:^
VkLayerTest.UnrecognizedValue

windows: Enable vk_loader_validation_tests

Tested and passed on Windows Skylake

windows: Return exit status from test script

layers: Comment update

layers: Clean up poolMap access

Remove some unneeded passing of poolMap around and use getPoolNode()
function in DescriptorSet class.

layers: Remove unneccessary consts

layers: Update swapchainMap to use unique_ptr

layers: Added getSwapchainNode() helper

Added getSwapchainNode() helper to core_validation and update
DescriptorSet class to use it.

layers: Add getSwapchainFromImage() helper

Add getSwapchainFromImage() to core_validation and use it from
DescriptorSet class.

layers: Remove unneeded layout_map

layers: Add getImageNode() helper

Add getImageNode() helper to core_validation and use it.
Also update DescriptorSet class to use the helper and kill
its image_map member.

layers: Add getImageViewData() helper

Switch imageViewMap to store unique_ptrs. Add getImageViewDate() helper
to core_validation and update DescriptorSet to use it.

layers: Add getSamplerNode() helper

Kill sampler_map_ in DescriptorSet and add getSamplerNode() helper
and use it instead.

layers: Add getBufferViewInfo() helper

Add helper function to core_validation for bufferViewMap look-ups
and use the helper in DescriptorSet class.

layers: Update getMemObjInfo helper

Standardize core_validation use of getMemObjInfo() and update
DescriptorSet class to use it as well.

layers: Add getBufferNode() helper

Move core_validation bufferMap to use unique_ptr to BUFFER_NODE.
Perform bufferMap look-ups using getBufferNode() helper function.
Update DescriptorSet class to use getBufferNode() helper and store
layer_data instead of bufferMap.

layers: Make layer_data visible to DescriptorSet

This is the start of migrating core_validation::layer_data to be
visible in DescriptorSet class.
DescriptorSet class can't see the contents of layer_data, so it also
needs the get<Object>() helper functions to look up items from the
maps. This first change only brings along getSetNode() utility
function so only set_map references can be removed.

Will add follow-on CLs to remove more maps from DescriptorSet class
and bring in more utility functions.

Update BUILD.md

Remove statement indicating that the Linux SDK installs the loader and layers to system libraries;  it no longer does this.

cmake: Fix MSVCRT override

Fix the CMake code that overrides the /MD flag with the /MT flag for
the Windows demo and loader projects.  The CMake debug and release
variables were set to /MTd and /MT, overriding all existing debug
and release settings.

Now replacing /MD with /MT in the debug and release variables,
without losing other debug/release settings.  Also performing
the MD/MT substitution for all release and debug configurations.

Change-Id: Id4a57c38bde0d909a95e7a49d9867029829e2c61

layers: Comment update

The behavior of the PreCallValidateAllocateDescriptorSets slightly
changed so updating comment accordingly.

layers: Pass common data between pre/post AllocDescriptorSets

With the break between PreValidate* and PostRecord* calls in the layers
we can suffer having to do some repeat work in the Post step. In order
to prevent this, this CL slightly modifies the interface to pass common
data between the pre/post calls in a custom AllocateDescriptorSetsData
struct.

I initially attempted to fill this data in a separate function that
would preceed the PreValidate* call, but such a function would need to
include some validation as it includes map checks which may fail.
The simplest solution, then, seems to be passing a ptr to the common
data to the PreValidate* function who then fills the data. If the
validation and call down the chain succeed, the PostRecord* function
then takes a ptr to the common data to prevent having to redo the
work that was done at validation time.

layers: Add DescriptorSet interface for allocation

Create a top-level "Validate" and "Perform" interface for the
vkAllocateDescriptorSets() function in DescriptorSet class.

Refactor AllocateDescriptorSets() in core_validation to use the
new interface. All checks are now performed in a "Pre" function
and all state updates are recorded in a "Post" function.

layers: Migrate DESCRIPTOR_POOL_NODE to type header

layers: GH611 Fix dynamic offset indexing

layers: simplify v0 function implementations

Since each of the layer library contains only a layer, we can simplify v0
functions by making them wrappers to the contained layer's corresponding
commands.

While we have to make changes to each layer individually, all the changes
should be the same in spirit.  The exception is vk-layer-generate.py where
the missing vkEnumerateDeviceExtensionProperties is also added.

tests: Fix extraneous errors in layer validation tests

Change-Id: Idfa248eec6fd57494a7ee6010cf86680daa981a8

layers: More noautovalidity processing for structs

Apply 'noautovalidity' code generation logic to struct members.
Previously, struct pointers tagged as noautovalidity were ignored by
the code generator, but the members of non-NULL struct pointers
were still processed.  Now both the struct pointer and the struct
members are ignored when the struct pointer has the 'noautovalidity'
attribute.

Change-Id: I14bc231210f2440fb8bb3df8c73ef165100f0daa

layers: LX506 fix vkFlushMappedMemoryRanges validation error

Fix LX506: vkFlushMappedMemoryRanges validation error
 with VkMappedMemoryRange::size = VK_WHOLE_SIZE.
Resolution: Change test of size variable to correctly test the
 size passed by application against the relevant mapped memory

Change-Id: I6d855482985512b5096696f5c37346dde8a391ac

winrtinstaller: update signature on powershell config script

winrt: Disable temp log in RT install.

winrt: Add more Powershell logging to RT install

RT install Powershell script was using echo.  Now use transcript
recording (which will also include OS, Powershell version, and other
info).  Also, use Write-Host, Write-Warning, and Write-Error to
make errors and warnings more obvious.

NOTE: This includes intentional debug code enablement so I can
retrieve an RT install that does not remove a copy of the
Powershell log file.  This will be fixed quickly after the RT install
is generated on CruiseControl.

tests: Add test for new renderpass clearValue check

Add test to hit the case where number of CLEAR loadOps in used
by renderPass attachments is greater than clearValueCount given
at renderPass begin time.

Also update documentation.