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

layers: Fix ResetDescriptorPool in OT

Previously this didn't bother to clean up the descriptor sets associated
with the pool.

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

layers: Merge Validate{Non,}DispatchableObject in OT

Same deal.

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

layers: Merge Create{Non,}DispatchableObject in OT

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

layers: Combine Destroy{Non,}DispatchableObject

These did exactly the same thing, except for the slight wrinkle of
needing to cast to uint64_t & in the nondispatchable case to account for
MSVC crankiness. We can just do that always.

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

layers: Remove allocator consistency check from swapchain

This is now covered by object_tracker

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

layers: Wire up INCONSISTENT_ALLOCATOR object tracker error

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

layers: Track whether any object was created with a custom allocator

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

layers: Hold device dispatch table by value too

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

layers: Hold instane dispatch table by value

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

layers: Rename my_data->instance_data

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

layers: Split CV device and instance data

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

layers: Move queue_family_properties into PHYSICAL_DEVICE_STATE

This isn't instance-wide.

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

layers: Dont pretend this is phy_dev_data in
GetPhysicalDeviceQueueFamilyProperties

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

layers: Get rid of remaining use of single physical_device_state in
instance

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

layers: Store ptr from device data to instance data

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

layers: Pull enabled_features out into layer_data, rename for clarity

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

layers: Stop pretending phys devices have their own layer_data

They don't -- the dispatch key is the same as the instance.
EnumeratePhysicalDevices just made a mess if there were multiple
physical devices in the system.

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

layers: Rename my_data -> instance_data in EnumeratePhysicalDevices

Makes it clearer what this is.

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

layers: Use member initialization for CV state

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

layers: Delete some dead structs in CV

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

layers: Get rid of duplication of logic op test between PV and CV

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

layers: Use safe_VkSwapchainCreateInfoKHR in CV

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

layers: Clean up layer settings file comments.

Normalize and clean up internal documentation for the
vk_layer_settings.txt file.
No logic or behavior changes.

Change-Id: Iceb45efb50bcb670b099027980244cc1e2358643

Fix CMake modules not being added correctly

This allows this project to be more easily included as a
sub-project of another project.

Change-Id: I098f26707f17d20353c91a4adb7bfb83d90a99fa

build: gh52-CMake changes to enable make install on Linux

See BUILD.md for details.

Change-Id: Ide1f635a57b57af384d9d1baac20c2256629f812

loader: gh971 Fix double free on physical devices

Change-Id: I97428dd579217340586809577eb64277ed4bb2cd

layers: Add VK_NV_external_memory_win32 ext to PV

Added GetMemoryWin32HandleNV function for this extension to the
parameter validation layer.

Change-Id: Ib1b870ab2b0433a19fcdca6bb6e0bffbeda74a13

layers: Add VK_NV_external_memory_caps ext to PV

Added GetPhysicalDeviceExternalImageFormatPropertiesNV API for the
VK_NV_external_memory_capabilities extension to parameter_validation.

Change-Id: I5011fcf33efffc8c432cde03e2ae4957766488b4

layers: Add debug marker extension support to PV

Added parameter validation for Debug Marker functions SetObjectTag,
SetObjectName, Begin, and Insert.

Change-Id: I4e95b350a822fe296cbd750ea228a7d98e29241a

layers: gh965 Reading undefined memory is now a warning

Change-Id: I03d021784649f4dbdced4dc34bcf39e7c7a0cfac

header: Move vulkan.hpp to version 1.0.28

layers: Updated vulkan.py for changed extension def

Change-Id: I9d52e7a3104d21fdd355292d6f17076cd719d2e4

layers: Fix thread/PV codegen for new notation

Embedded parameter length variables are now specified in vk.xml as
being separated by '::' instead of '->'.

Change-Id: I53befe20912d7d7ee628f6819dbd013bcd2a9414

layers: Whitelisted new extensions for 1.0.28

Change-Id: Ife1621716ec9593de3ca528d071b707515acb8c2

loader: Move SOVERSION to version 1.0.28

Change-Id: I4e011b3b37f805fde1100cf519042ea242c680d0

layers: Move Windows json files to version 1.0.28

s Please enter the commit message for your changes. Lines starting

Change-Id: I292aba8b4b4df46d60e3c8d4752e5fb9ccee1217

layers: Move Linux json files to version 1.0.28

Change-Id: I51e3624312bd83a0b6f440638088ef362d7f7ee5

header: Move vk.xml to version 1.0.28

Change-Id: If860d6d0df348fc758f3b59429523cd5ee2db95f

header: Move vk_platform.h to version 1.0.28

Change-Id: I3bc47016e3be0c9870f05254b02718099d990eaf

header: Move vulkan.h to version 1.0.28

Change-Id: I346877d18a2600e114ebee844fb5b5055c0ea5c5

layers: Complain about subpass mismatch between CB and pipeline

Fixes #933.

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

layers: Use cmd buffer as src object in callback

One less TODO.

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

demos: Remove tri from Android build

demos: Set srcAccessMask correctly for present barriers

Change-Id: I58cd90304ad4bd4e3c5c1622002bbc46fcf66cd9

cmake: Fix whitespace

Change-Id: Ia2d35a87c14c0c73bc26588694b398e04a191203

build: gh27 Allow builds in paths with spaces

Add quotes around shell var expansions in the update_external_sources scripts.

For Windows, remove the cmake directives to add a linker option for the DEF
files.  These were redundant with the DEF files getting added via the
cmake add_library() directive.  They were also causing difficult-to-fix
problems with paths that have spaces in them.

Change-Id: I741bac31bbf27deae59031b6aa916c6ab48383a6

layers: Added flags for in_use checks

Added object_in_use flag to disable all in_use checks.
Added destroy_buffer_view flag to disable validation checks at
DestroyBufferView() time.

layers: Update DestroyBufferView to use Pre/Post paradigm

Add PreCallValidate* and PostCallRecord* functions to DestroyBufferView
according to long-term core_validation architecture plans.

layers: Add binding between cmd buffer and bufferView

For a texel descriptor update, add binding between bufferView and the
command buffer as well as between underlying buffer and command buffer.

layers: Add in-use detection for VkBufferView

When VkBufferView is destroyed, flag error if in-use and mark any
bound cmd buffers as invalid.

demos: Remove tri demo

Removing tri demo for maintenance reasons.  Cube should be the demo
to go to for examples of good Vulkan

Change-Id: I4442a1882b5c168cfe88501eccc41a554ae4d026

layers: Add bool to guard command buffer state check

This is the first check-in demonstrating the boolean model to allow validation
checks to be disabled. Going forward we'd like to guard all checks in such
bools which can then be set based on VkValidationCheckEXT enum values passed
in at vkCreateInstance() time.

In the course of adding this noticed a bug where instance_state was not getting
initialized in device layer_data at CreateDevice time. Fixed this bug.

layers: Indicate correct source for validateCommandBufferState() error

Fixes #982

Instead of hard-coding vkQueueSubmit, take a parameter to clarify if the error
originates from vkQueueSubmit() or vkCmdExecuteCommands()

docs: Add CubeWithLayers to Android section