layers: Stop copying INSTANCE_STATE into device

V2: Squash in second patch; resolve conflicts with @tobine's renames.

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

layers:Pipeline state rename fixup

Overzealous bulk replace changed getPipelineLayout to getPipelineState
layout. This should not have been changed so reverting here.

layers:Rename IMAGE_NODE to IMAGE_STATE

Another node->state rename pass.

layers:Refactor DestroyImage

Add validation flag for DestroyImage and update it to follow the
Pre/Post pattern.

layers:Refactor DestroyRenderPass

Add validation flag for DestroyRenderPass and update it to follow the
Pre/Post pattern.

layers:Rename RENDER_PASS_NODE to *_STATE

Another node->state rename pass.

layers:Bind descriptor pool to cmd buffer

Add pool state to DescriptorSet instead of just the pool.
Then, at bind time, create a dual binding between the cmd buffer
and the pool. This correcly puts pool in-use when cmd buffer is
submitted, as well as flagging cmd buffer as INVALID if/when the
pool is destroyed.

layers:Update some pool_node naming

Change some pool_node names to pool_state in DescriptorSet class.
Also update getPool function prototype to the new name of
getDescriptorSetState().

layers:Add pool member to DescriptorSet class

When creating a DescriptorSet instance, record the pool that the set is
allocated from in a private pool_ variable.
Then, when binding a set to a cmd buffer, also create a binding between
the command buffer and the pool.
This allows us to catch the in-use case when sets are implicitly freed
by calling vkDestroyDescriptorPool().

layers:Rename DESCRIPTOR_POOL_NODE to *_STATE

Continuing the NODE->STATE renaming effort

layers:Kill outdated comment

tests:Add FramebufferInUseDestroyedSignaled test

This test creates a binding between framebuffer and cmd buffer, submits
the cmd buffer and then destroys the framebuffer while still in-use to
verify that the correct error is triggered.
Update database file with new test for check 00422.

layers:Fix framebuffer cmd buffer binding

Wasn't correctly establishing binding between framebuffer object and
cmd buffer object that it's used in. Updated code to use standard
addCommandBufferBinding() function.

layers:Rename FRAMEBUFFER_NODE to *_STATE

Part of ongoing effort to kill nebulous "NODE" moniker and switch to
"STATE" instead.

layers:Refactor DestroyFramebuffer

Add validation flag for DestroyFramebuffer and update it to follow the
Pre/Post pattern. This function didn't previously perform an in-use
check so added that check.
Updated database file.

layers:Add in-use unique error enums

Pass error enum into ValidateObjectNotInUse function and update
all of the existing calls to pass correct enum.
Also update database file to record checks as implemented and note
test status.

layers:Add validation to DestroyDescriptorPool

Add validation flag for DestroyDescriptorPool and update it with
validation code using the Pre/Post pattern.
Flag error if descriptor pool is in use and invalidate any cmd buffers
that it was bound to. Remove pool state from map when it's destroyed.
When the pool is destroyed make sure all of its descriptor sets are
also freed.

layers:Rename PIPELINE_NODE to PIPELINE_STATE

Would like to migrate "NODE" structs to "STATE" as it's more
descriptive of their purpose. This commit changes PIPELINE_NODE
to PIPELINE_STATE.

layers:Reduce locking window

Don't need to lock around these decls in DestroyImage/BufferView

layers:Refactor DestroyPipeline

Add validation flag for DestroyPipeline and update it to follow the
Pre/Post pattern.

layers:Refactor DestroyImageView

Add validation flag for DestroyImageView and update it to follow the
Pre/Post pattern.

Also made a couple minor tweaks to DestroyBufferView based on review.

loader: gh1035- vkDestroyDebugReportCallbackEXT

The changes I made to enable layer wrapping of the debug report
callback broke the cleanup case so that under certain scenarios
vkDestroyDebugReportCallbackEXT would not delete the callback
resulting in messages after the destroy call.

Change-Id: Ie5b26d8c973a08bb3161412468bb9dfdb70b315c

layers: Removed genvk.py

Replaced with an updated and modified version, lvl_genvk.py.

Change-Id: I60f344d8361d1408303185a5d7958575a404870e

layers: Update build for v1.0.30 XML code-gen

- Updated cmakefiles for new codegen files: renamed genvk.py to
  lvl_genvk.py as the file is modified in this repo.
- Modified dependencies and parameters of cmake build macros.
- Updated reg.py to version 1.0.30
- Updated generator.py to version 1.0.30

Change-Id: Ia87fc7fc68e406a724d5ad9822f1f87b964f9977

layers: Add and update custom version of genvk.py

Created a LoaderAndValidationLayers-specific version of the
1.0.30 version of genvk.py, locally named lvl_genvk.py.

    - removed unused generator classes
    - added LVL class to build threading
    - added LVL class to build paramchecker
    - added LVL class to build uniqueobjects

Change-Id: If3ac7f6b554ed5b850aba83b5b20998b84990a41

layers: Set up UniqueObjectsOutputGenerator class

Implements the Vulkan XML API registry-based generation
of the unique_objects layer's unique_objects_wrappers.h
header file.

Change-Id: Ie2456216bb10726dab792d39b0b9d85bb1ba528c

layers: Set up ThreadOutputGenerator class

Implements the Vulkan XML API registry-based generation
of the threading layer's thread_check.h header file.

Change-Id: I50e828aaf9ce9d494520b476c2c73f44dd549a3c

layers: Set up ParamCheckerOutputGenerator class

Implements the Vulkan XML API registry-based generation
of the parameter_validation layer's parameter_validation.h
header file.

Change-Id: Ib7be36121f4f21dd667d09f9bd5e82749a417a33

build:  Add include files to make install target

Change-Id: Ieaf84adc4a9345c1f5ef7646106dd00ff3e5e716

layers: Add error enums to object tracker

- Add error enum to many calls to ValidateObject().
- Specify null_allowed=true when VU allows it.
  (mostly Destroy* cases)

Change-Id: I61d94ea71cf885caad475c2012111c92bc7e9904

loader: gh1026- Fix compiler warnings

Fixed the compiler fixes encountered in the loader.

Change-Id: I4b3bb34b2d7e2231ccae61c684029af337d7c25e

demos: Add .py extension to smoke build script

Change-Id: I7fbcdc26fe774baefbdb3796470f41c3393ddfda

scripts: Add python extension to script

Change-Id: If5c338ba6eb991cc489ef0a37296a111917ddd52

build: Remove dead code from build script

Change-Id: Id6dbde9743573659b57c2ee60634245ca10a827a

loader: Remove dead code from loader gen script

Change-Id: Ie6ad81c1f0097b93c3006a737d57a69d81ee70e8

layers: Validate correct layout for STORAGE_IMAGE descriptors

According to the spec "Load and store operations on storage images can only
be done on images in VK_IMAGE_LAYOUT_GENERAL layout."

This change flags an error if a STORAGE_IMAGE descriptor is updated with an
image layout other than GENERAL.

cube.c: Remove excess tests from presentation fences

Instead of using a bool array and both checking and writing to it every frame for no reason, just create the presentation fences with the VK_FENCE_CREATE_SIGNALED_BIT flag.

layers: Update database file for spec on 10/11/16

Update the error enum database file for spec as of 10/11/16.
There was only one change which was the removal of spurious </usage> tag
from error 500. Since this changed the error message, I had to force the
script to remap new 500 to the original 500.

Command line run to generate update:
python spec.py -update -remap 500-500

scripts: Add database error check to spec.py

Fix broken database line where delimiter was incorrect.
Add check to spec.py script to detect if database line doesn't contain
expected number of elements and report error in that case.

layers: Updating some CreateBuffer checks to use unique enums

Small batch of updates to use unique validation error enums and
messages for a few checks.  Updated the database file and
flagged some other checks that are missing with the error
enums and TODO notes.

Change-Id: I2020a88bbdcf7ea6d725f667523e89b160fe7ab8

layers: Add validation flag for descriptor updates

Add update_descriptor_sets flag to bypass validation for
vkUpdateDescriptorSets() call.

layers: Pass unique error codes through descriptor validation

Descriptor validation builds up error strings at multiple levels such as
UpdateError->WriteUpdate->BufferUpdate->BufferOffset. The final error is
then reported via a log_msg() call at the top level.

In order for this design to allow for the new unique error codes, we need
to pass error codes down the chain in the same way we pass error strings
down the chain.

This change adds error msg passing for descriptor updates. Many error
codes are added in as well as comments for codes that are missing or need
to be updated.

The database file is also updated to account for all of the checks that
were updated and/or reviewed.

layers: Fix Windows build warnings

NOEXCEPT is defined multiple times

Change-Id: I9aef970e9588c17071cd71a31873db4090d50b07

demos: Fix Windows build warnrings

Change-Id: I733704e8904a77aa9071565cfc1d18b23b1c0c1c

docs: GH429 change doc for shader_checker tests

This branch adds tests that challenge the validation layer on various
shader checker cases

Change-Id: I38072fb9afaa52148a24532e919c2ac81111480c

demos: fix cubepp cross-platform build

Change-Id: I823154dc0c61e7358d88fe6afc4f5808b640804d

demos: fix uninitialized variable

Change-Id: If980d3a8d765185148cff0b037cb64e637ae8a75

demos: fix cube cmake

Change-Id: Ia7a79bee7353ac484e691916591c585643a8753d

demos: fix uninitialized variable

Change-Id: I1aa51806a82398fb03cfb81ae23782882d1e07ae

demos: clang-format cube.cpp

Change-Id: I5f55446eb6f3c312b6cc012733b7f4e454ac73ba

demos: add cube cpp sample

Demonstrate how to use vulkan.hpp.

Change-Id: Ia5ef4f55d45219aec607ae04efdb64c6ebbc9578

header: Move vulkan.hpp to version 1.0.30

Change-Id: Ifb8a3577cd41ea4a36fea78c12d2a92ea5437e11

loader: Move SOVERSION to version 1.0.30

Change-Id: I129086e47b0cdb12d31ffdd334d27c63e18dd097

layers: Move Linux json files to version 1.0.30

Change-Id: Ia54d78a9440f40b61601836fedcf68aba3c90dc1

layers: Move Windows json files to version 1.0.30

Change-Id: Iac3446e618e1b84be6d42eb7d3b55d3046b124d6

header: Move vk.xml to version 1.0.30

Change-Id: Id238f998c88d19f1549273359d559a602963c002

header: Move vulkan.h to version 1.0.30

Change-Id: I99dd0d24feca887ea5c69e97856e96728491b0b5

layers: Fix access violation in LVT

Was sending an out-of-scope memory reference to the display driver
in unique_objects::AllocateMemory.

Change-Id: I6b700445125acab040310682c96e7507d0fd91aa

layers: Add flags for some descriptor validation

Add flags that can gate descriptor checks for in-use descriptor,
push constant ranges, and allocate & free descriptors.

layers: Updating descriptor checks to use unique enums

Initial batch of updates to use unique validation error enums and
messages for a few checks. Updated the database file and flagged
some other checks that are missing along with "TODO" notes in the
code.

Fix Android Windows build script

demos: Fix typo in vulkaninfo

Change-Id: Id3ac6abb9288d2a36d68b67a605b4681ce20f539

Layers: Implement unique_objects layer using vk.xml

UniqueObjects was the final LVL layer not using vk.xml for codegen.
The new implementation follows the threading layer model which
(outside of safe_struct dependencies) is more tolerant of API
changes. Also updated for structure and style.

build-android/android-generate.bat - Updated for build
build-android/android-generate.sh  - Updated for build
generator.py                       - Added Unique Objects Generator
genvk.py                           - Added call to U_O generator
layers/CMakeLists.txt              - Updated for build
layers/unique_objects.cpp          - New: manually generated code
layers/unique_objects.h            - Now a true header file
vk-layer-generate.py               - killed!

Change-Id: I0d57871dfd2600829502f0bffd7cf6f926b7d5e7

layers: Fix wayland build break

Cut-and-paste error

layers: Initial codegen of unique error enums

This is the initial commit of spec.py script that's used to generate a header
and database file that contain unique error enums per valid usage line in the
spec.

The basic usage is just "python spec.py" which will write out a header file to
"vk_validation_error_messages.h" that contains unique error enum values for
each valid usage line in the current spec that's online at
https://www.khronos.org/registry/vulkan/specs/1.0/xhtml/vkspec.html

The header also contains validation_error_map with unique enum values as keys
that map to customized output messages for each validation error.

Updated one check to use the new model in core_validation.

The database file contains an enum value and error message on each line, as well
as fields indicating if the enum has been implemented in code and the testname
for a layer validation test that verifies that particular check. Initially the
implemented fields are set to 'U' and testname set to 'Unknown' to indicate that
they haven't yet been checked. Will be updated manually as the code is scrubbed
and new error enums are added.

When updating the header as the spec changes, the "-update" option to spec.py
will pull the database file and use it as a baseline reference to compare against
the updated spec. The script will attempt to keep errors with same message to have
the same error enum. When it's unsure if messages are updated or new, the script
will flag errors for manual review.

layers: Fix buffer_view destroy ordering

Don't destroy buffer_view from map until after invalidating bound cmd buffers.

layers: Migrate surface/swapchain interaction from swapchain to CV

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

layers: Track current and prev swapchains for each surface in CV

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

layers: Add handle to SWAPCHAIN_NODE

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

layers: Change 'VS' to 'vertex shader'

Change-Id: Ib9b20635225528d2fd6d012e6487d95f27a0f95d

layers: Add environment var for settings file

Change-Id: I8a614f915ab9b61eca5b3b94f4cf43a33cacf250

layers: Fix state tracking for vkFlushMappedMemoryRanges()

Checks for underflow and overflow when flushing mapped memory where using the
flush offset instead of the original mapped region offset to determine size of
total mapped region.

layer: Add tracking of surface extensions (and surface objects) to CV

V2: Don't leave commented-out junk lying around; init
SURFACE_STATE::surface in default-constructed case.

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

layers: Unify two generic object structs

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

layers: Only validate access masks if layout changes

Change-Id: I21af32fb017de15760d72d4b32d99d84c383755f

demos - fix cube's present barrier's dstAccessMask

Change-Id: I8f94d27f16864730514beb3e326b604f24bbb65a

layers: Check for srcAccess in present barriers

Change-Id: I7722a31a6ef576cf8bc79c6893e70bc4b1cf1c09

loader: gh1014- Fix physical device dispatch table

Instead of initializing dispatch tabled for up to *pPhysicalDeviceCount devices, initialize
always all dispatch tables.

This fixes corruption _after_ VulkanCTS dEQP-VK.api.info.instance.physical_devices test.

Change-Id: I29ff42456bed9acb6d2c4016fe7f1940b7f4bd3b

loader: Small cleanup of unnecessary func call

We had a function that was called only from one other place and
it wasn't necessary.  So removed the extra function call.

Change-Id: I3579215dc1d2f46b8c30b4ec3cb416c390fcf597

layers: Clean up Windows build warnings

Change-Id: Iec47464d56c862cb1a6f132241d200d6f68efa21

demos: Clean up Windows build warning

Change-Id: I39daaa144e82eeabbe532cb32d334d4e60e6a7e2

cube: demonstrate how to select other VkPresentModeKHR's

This comes from the Khronos-internal discussion of
https://github.com/LunarG/VulkanSamples/issues/98.  Cube will, by default,
choose VK_PRESENT_FIFO_KHR mode.  However, when that change was made, the old
(example) code was lost that showed how to search for the other present modes.
It was also discussed that there isn't good guidance on why an application
might want to use a mode other than FIFO.  This change adds both of those, but
only selects FIFO by default.  #ifdef's can be used to select other modes.
With that code is a reason why an application might wish to use that present
mode.

layers: Tweak object_tracker for 32bit some more

The previous approach doesn't actually work.

build: hide layer symbols in android build

Hide symbols from layers that don't need to be exported.
This makes libraries smaller.
It reduces symbol lookup time and call overhead.
It prevents possible symbol collisions.

build: make android layers favor their own symbols

Use -Bsymbolic linking android layers so they don't get symbols
like vkCreateDebugReportCallbackEXT from outside the layer.

build: hide symbols from SPIRV in layers.

Core_validation was exporting SPRIV symbols that might
collide with application use.

Revert "build: Restore default symbol visibility to layers/tests"

This reverts commit dd96dd21b438d48aa78e9de60ad0c550c856cf29.

layers: mark exported functions in layer utils

This allows layers to hide many functions that don't need export.

layers: Be careful about casts. ptr -> uint64_t& isnt right on 32bit.

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

Move vulkan.hpp to version 1.0.29

Change-Id: I52f48c40290ff56df98d9eae3dfc3fe8748b5008

layers: Update changed extension names for 1.0.29

Change-Id: Ifc3484471e8f1e7ca585055b243f0688e7dfd33c

layers: Move SOVERSION to version 1.0.29

Change-Id: Ieb1befd5370d96404af8abf85bfed30fe43b5995

layers: Move Linux json files to version 1.0.29

Change-Id: Ibea474bc21a54251372f757d21a5cfb401876a35

layers: Move Windows json files to version 1.0.29

Change-Id: If3dbe050a7d8c9954ba1d7bc6aa29123a123a359

header: Move vk.xml to version 1.0.29

Change-Id: Iae35f784b40eeafe07e1cf1af96924bd151ecd59

header: Move vulkan.h to version 1.0.29

Change-Id: I500e04c6545daa53d21b8cf552a922a0462d066c

layers: Add VK_AMD_draw_indirect_count ext to OT

Added CmdDrawIndirectCountAMD and CmdDrawIndexedIndirectCountAMD APIs
for this extension to object_tracker.

Change-Id: I99fcadc9660316864666f86040e256a392565569

layers: Add VK_NV_external_memory_win32 ext to OT

Added GetMemoryWin32HandleNV API for this extension to object_tracker.

Change-Id: I94cc6a3928f90c6ff964f7a2e5cba9aa2548d653

layers: Add VK_NV_external_memory_caps ext to OT

Added GetPhysicalDeviceExternalImageFormatPropertiesNV API for the
VK_NV_external_memory_capabilities extension to object_tracker.

Change-Id: I36ab9a1656dbecbb2384ee173359968ea8e454eb

layers: Add debug marker extension support to OT

Added object_tracker checking for Debug Marker functions SetObjectTag,
SetObjectName, Begin, End, and Insert.

Change-Id: Ibc13192bed2003d176df97719c9162b90051b703