layers:Verify image barrier queueFamilyIndices

This is VALIDATION_ERROR_1b80093c.

For image barrier in renderPass both src & dst queueFamilyIndex members
must be VK_QUEUE_FAMILY_IGNORED.

scripts: Exclude disable extension enums

Including these enums hosed up the private builds.

Change-Id: I74fcb628171dfd70e7bce03e7ede35d4d22a75b5

layers: Add VUID for LogicOp validation check

Change-Id: I19f10ee8e45af336290b36107db8bfe1c41c3a88

layers: Remove PV dead code, fixup comments

Change-Id: I94804f0b994618982afcb344dc88b92a9bee3336

scripts: Generate PV calls to ranged_enum_array fcn

Change-Id: I106fe73c2199d1b4d8aa090bd22bb79884637d58

layers: Add new ranged_enum_array val routine

Change-Id: I2b78d90bd248a6800f719ef89c534140cb217ad8

scripts: Generate calls to new enum PV function

Change-Id: I9692f1b2101d6cc1eef3bf55582dbf7980c4e8d2

layers: Add new PV ranged enum validation function

Uses new enum lists.

Change-Id: I24629770e1018f5938fa82d2a3da8eb1cd716de7

scripts: Generate param_validation valid enum lists

Change-Id: I0dd6bf95bf54baa9a71f41a482bc52527944aea8

layers: GH1924, Improve unavailable feature msg

Modified the unavailable requested feature error message to print the
feature name instead of just an index.

Change-Id: Idcb66921fc592014330dce6946631984d85cd866

scripts: Add PDevFeature index print helper

In the Enum string helper header, tack on a routine to convert an
index into a VkPhysicalDeviceFeatures structure into its
corresponding feature string.

Change-Id: I8fbfb6fcbfa63361a851ffc93ffd832696050d51

scripts: Switch codegen to use structextends

Replace all usages of the deprecated validextensionstructs member
attribute with the registry.validextensionstructs dict.  Internally the
registry builds this extension dictionary using the new structextends
type attribute.

Change-Id: I072ff707bfdfa1f576266291e4196837da526287

header: Update to version 1.0.56 of the Vulkan hdr

- updated vulkan.h
- updated vk.xml
- updated Win/Lin .json files
- updated vk_validation_error_database.h
- updated vk_validation_error_messages.h
- updated vulkan.hpp

Change-Id: Ibf64c8a1760743b3118794b0ef5f6d34b108bbbc

layers:Refactor image barrier validation

Move image barrier validation within a renderPass into its own function
in order to break up existing code a bit. Add a check to make sure
framebuffer is valid before attempting to use FB state.

layers:Minor refactor for clarity

Add comment and move VkSubpassDescription local variable assignment to
clarify what's going on.

tests:Add image mis-matched layout test

Test for VALIDATION_ERROR_1b800938.
Verify check fires when subpass attachment layout does not match image
barrier layout.

layers:Refactor barrier check order

Do the renderPass barrier check first and early return to avoid
potential redundant errors from remaining layout and barrier checks.

layers:Check image barrier layout vs attachment

This is VALIDATION_ERROR_1b800938.
Make sure that the attachment references by current subpass description
has a layout that matches image barrier layout.

layers:Check barrier image vs subpass attachments

This is the second half of VALIDATION_ERROR_1b800936, for which the
first half was already implemented. Make sure that VkSubpassDescription
for current subpass references the attachment that has an image
matching the image barrier.

layers:Only deref active subpass once

Grab active subpass once initially to avoid repeated defers of cb_state

layers:Migrate image barrier check

This check is only relevant within a renderPass so move it to the
function specifically dedicated for in-renderPass barrier checks.

tests:Test invalid image in VkImageMemoryBarrier

Add case to RenderPassBarrierConflicts where a VkImageMemoryBarrier in
a renderPass refers to an image that's not an attachment of the current
framebuffer.

layers:Add image barrier image check

This is VALIDATION_ERROR_1b800930.
Make sure that image in VkImageBarrier matches one of the images in the
current framebuffer.

tests: Add test for #1910

layers: Resolve TODO about actual layouts

This VU exists now in recent specs.

layers: Implement check for binding of maint1 slice of 3d image to DS

Fixes #1910

layers: Fix mismatched new[] / delete

Likely benign on anything reasonable, but still UB.

layers: Remove now-redundant OT hdr prototypes

Codegen creates its own prototypes, no longer need them in the header
file.

Change-Id: Ibaab124a26391dc4925e49f095ddac83523aa725

build: Switch to generated object_tracker.cpp

- updated desktop/android build
- removed layers/object_tracker.cpp
- updated doc validator script for new location
- updated error database for 124 new checks
- added gen'd VU to GetSwapchainImages

Change-Id: If1b218dfcf62b203a8df96a0f12cbeb8bdcaf826

scripts: Create object_tracker generator

Change-Id: Ia97d2141a54263a9f034958f416661146dd953e2

scripts: Add OT generator to main script

Change-Id: I9419cdfdc39d33f5cc9f769c7498fde0230b144b

scripts: Update vuid_mapping script

Change-Id: I704df0840801734ca35d5137a9d79d5a202b4fb9

layers: Remove flags checks from OT

These two checks were opportunistic and won't automatically be
covered by codegen. Removing now for clarity.

Change-Id: I3ada65c80094e8fa732a4ffc0da8c9ddeb0e11e0

layers: Remove erroneous checks from OT/DB

Change-Id: I57ae325f27b365cd3b4e0f24d033af76ceeaeec7

layers: Update API for vulkan parameter def change

One of the DebugMarkerSetObjectName parameters changes to const.

Change-Id: I23455957b83a84983bc7217a2a0d937ae03055e2

layers: Move GetSwaphchainImages func in OT

Special case function must avoid code generation.

Change-Id: I7c03cb2c0b4053f3d2b0579d532f1da82be80bd4

layers: Move GetDeviceQueue func in OT

Special case function must avoid code generation.

Change-Id: I786d4ef4a8855f10e9cfe1234a5d885056f50a93

layers: Move CreateComputePipelines in OT

Moved to manually-generated, it has an odd corner case that's a one-
off and not worth code-generating.

Change-Id: I2631dd2107fffcd7e641b5a98bee126502d61ba8

loader: Remove executable stack from libvulkan.so

Change-Id: Id77d5560479cbbc6f5453a6362c5a20dd721fdd0

demos: Fix pointer cast warning in vulkaninfo

Change-Id: I6acebbcbb9a10d86f3b7ec56148d9a680d085921

layers: adjust naming style

layers: Don't complain about missing FS outputs if attachment mask is 0.

Fixes #1931.

layers: Pass entire pipeline state down to FS output check

LVL: Add validation check for CreateImageView

Added validation checks, tests, and updated error database.

Change-Id: Id4dfe66bc325ae0060b7668572026c8a40f83d64

demos: Add extension support to vulkaninfo

Change-Id: I7b9c729a90571d2888d7a22a9899546c547e6153

layers:Update descriptor state earlier

This change arises from an issue reported by Alon at Samsung. He
reports that a driver bug is modifying the descriptor set state on
UpdateDescriptorSets() call, so the data is getting corrupted in
validation. Since UpdateDescriptorSets() is a void function, it
doesn't make a difference in validation if we update state before or
after the call, since the call itself won't change the state being
updated. It actually saves us a lock call to update pre-call.

tests:Test invalid renderPass bufferMemoryBarrier

Test case where VkBufferMemoryBarrier is incorrectly issued within a
renderPass.

Kill old code that was doing the same check/test w/o the VUID.

tests:Test barrier dependencyFlags mis-match

This is VUID VALIDATION_ERROR_1b800932. Case where barrier
dependencyFlags mis-matches subpass dependency dependencyFlags.
Also update VU DB.

layers:Add image mem barrier access mask checks

Complete remaining work for VALIDATION_ERROR_1b80092e and
VALIDATION_ERROR_1b800930. Make sure the image memory barriers src/dst
access masks are a subset of corresponding subpass self-dependency
access masks.

layers:Validate bufferMemoryBarrierCount

This is VU VALIDATION_ERROR_1b800934.
For a barrier in renderPass, bufferMemoryBarrierCount must be zero.

layers:Validate barrier dependencyFlags

This is VUID VALIDATION_ERROR_1b800932. Verify that an barrier in a
renderPass has a dependencyFlags value that matches the corresponding
subpass self-depenency dependencyFlags value.

winsdk: Hide runtime in programs and features

Change-Id: Ibc5028f660c91f66d986af19f6c7959eb9b55359

loader: Fix vkEnumerateDeviceLayerProperties

The device layers should list the names of the original layers
enabled during vkCreateInstance.  We were expanding the
meta-layers which caused there to be a mismatch in names.

NOTE: that this list still include Implicit and Environment
variable enabled layers since those are techincally enabled at
vkCreateInstance as well.

Change-Id: I3ef88623e96a12a8f18f8f04e95a6f5444f4be56

layers: Update object_tracker var/func names

Change-Id: Iacc0cb2676cd2346879d7d51e91b9b75232d5ad4

layers: Reorganize ObjectTracker for future codegen

Moved to-be-generated source to object_tracker.cpp, manually-
written and helper functions to object_tracker_utils.cpp (new
file), and reworked header. Updated doc validator to search
new file.

Change-Id: I0fd3bcc4f6d123db07c7f8e2633e6fe16833e231

layers:Handle ALL_*_BIT pipe stage barrier validation

Update barrier validation to handle VK_PIPELINE_STAGE_ALL_GRAPHICS_BIT
and VK_PIPELINE_STAGE_ALL_COMMANDS_BIT cases in the subpass dependency.
Expand GFX bit as it's an alias for a fixed set of bits. If
ALL_COMMANDS is specified then skip validation because we know that
barrier mask must be a subset of subpass mask.

tests:Update self-dependency tests

Move renderPass barrier w/o self-dependency test from InvalidBarriers
to RenderPassBarrierConflicts test case. This prevents having to
re-created the default renderPass in InvalidBarriers test.
Also update framework subpass with self-dependency to set a group of
gfx pipe/access masks instead of EVERY mask, which is not allowed.

tests:Update RenderPassBarrierConflicts

Add accessMask tests to RenderPassBarrierConflicts. Hit the two cases
where src & dstAccessMasks from pMemoryBarriers are mis-matched from
VkSubpassDependency.

layers:Add barrer access mask checks

Verify that src/dstAccessMask from pMemoryBarriers is a subset of the
corresponding mask from the relevant VkSubpassDependency instance.

layers:Fix copy-paste dstStageMask print bug

Fix this copy-and-pasted error message by changing src->dst.

layers:Refactor barrier validation

Migrate validation for renderpass barrier in CmdPipelineBarrier to its
own function. Refactor some device and command buffer params to avoid
duplicate look-ups. Add some const decls to command buffer state that
shouldn't change down the pre-call path.

header: Update to version 1.0.55 of the Vulkan hdr

- updated vulkan.h
- updated generator.py
- updated reg.py
- updated vk.xml
- updated Win/Lin json files
- fixed and updated vulkan.hpp
- updated validation error messages and database
- applied validextensionstruct fixes from 1.0.54 LVL vk.xml file
- fixed parameter_validation DebugMarker prototypes
- fixed unique_objects DebugMarker prototypes
- fixed NV_FILL_RECTANGLE in error DB
- updated error database with new covered VUs

Change-Id: I4eec816a6dc9756105a357beab6af4abb303994d

loader: Do not enumerate extensions from disabled implicit layers

This patch adds a check to vkEnumerateInstanceExtensionProperties to
omit extensions provided by disabled implicit layers. Per section 31.2
(1.0.51), "When pLayerName parameter is NULL, only extensions provided
by the Vulkan implementation or by implicitly enabled layers are
returned." Arguably, implicit layers that have been disabled are not
"implicitly enabled".

layers:Add checks for barrier stage masks

Verify that the src & dst stage masks for a subpass barrier are a
subset of the subpass dependency's src & dst masks respectively.

Add RenderPassBarrierConflicts test to hit these cases. This test hits
the two cases of an image barrier src/dst masks not being a subset of
their corresponding subpass dependency masks.

Update database file to reflect that these checks are implemented and
that this test covers them.

scripts:Update VUID Database

VALIDATION_ERROR_1b800928 is implemented and tested in InvalidBarriers.

layers:Add subpass barrier self-dependency VUID

This is VALIDATION_ERROR_1b800928. Add the ID and slightly touch up the
error language. Also, this check should be general for any barrier
that's inserted for a renderPass so remove extra memBarrierCount
condition.

layers:Handle deleted image view in descriptor

When an image descriptor with a destroyed image view was used we hit an
assert. This is not caught upstream and can occur if a descriptor is
re-used following image view deletion. Add code to catch this case
and report the destroyed image view.

layers: GH1933, Check for mutually exclusive exts

Add check for Maintenance1 and negative_viewport_height extensions being
enabled simultaneously, along with test and a database update.

Change-Id: Id44f2a322fd32883d8b6d7ff1dab56996dec76c7

demos: Add Wayland input handling to smoketest

Change-Id: I82cd3f004410dea1d173ef8022091d6f4463d0f5

demos: Clang-format Wayland and cube.cpp refactor

Change-Id: I73dac6489e41a94fc53a1c7942888a3a6479a805

demos: Add Wayland input handling to cube

Change-Id: I314ad9bc3e297da93f4d5c1d44289c9cca3a8fde

demos: Refactored Demo class in cubepp
Moved 'Demo' declaration to near the top of the file to avoid referencing
problems.

Change-Id: I084231ada5683971baa073f1b761ac214fa02cfd

external: Update commit IDs: spirv-tools

This fixes an MSVC compiler warning that's treated as an error
(and breaks the external sources build on Windows).

build: Fix loader/layer fallthrough warning

Change-Id: I04d563caa9000ccbfd5e6e509bf65ceaed9e388a

layers: Update PV codegen for Windows warning

Code gen declarations of allowed_structs in nested scopes could cause
warnings in Windows.  Modified codegen to append the type name to
allowed_structs (eg allowed_structs_VkGraphicsPipelineCreateInfo) to
avoid name collisions.

Change-Id: I40bd57ff34dcd1df01d3606c79dacbe06b00f464

header: Update to 1.0.54 Vulkan release

Update the headers and xml file with the latest spec changes for
Vulkan 1.0.54.  Also, make corresponding loader and layer changes
to support the new extensions.

Change-Id: I11273c5e3e828743f904e4f0b4b2f2c3a7804df0

layers:Refactor CmdUpdateBuffer to Pre/Post

Refactoring CmdUpdateBuffer function to use the Pre/Post call pattern.
Updated the pre-path so that command buffer state pointer is const
throughout. Pulled the state update code into the post function.

Note that this slightly deviates from the previous pattern by doing
object state look-up in top-level function instead of passing a ptr to
a ptr to Pre* function and having it do the object state look-up.
I prefer this method as it allows const* throughout the Pre* call chain
and avoids the ptr derefs in Pre* function. I'm open to discussion if
others prefer the alternate method however.

layers: Remove spammy debug output in vkCmdBindDescriptorSets

layers:Identify layout from pipelineLayout

When checking IsCompatible() for a DescriptorSetLayout, the passed-in
layout is always from a pipelineLayout, however this was not specified
in the code. Add comment to specify this usage and update the error
message to always distinguish which of the layouts using in the compare
is from the pipelineLayout.

external: Update commit IDs: glslang

Change-Id: I2e635bed2d132f636ab19739693d252e495ea648

layers: Clarify MaxAnisotropy not enabled message

Folks continue to confuse 'enabled' with 'supported', expecially on
this particular feature.

Change-Id: I73df704436a95c9c47194169a53602b87b422174

external: Update commit IDs: glslang,tools,header

Update of external reference commit ids.

Change-Id: Iaa004adc60d8b41c5d894476103453491ed54535

Revert "external: Update commit IDs: glslang,tools,header"

While trying to determine cause of TravisCI failures, back out the
update of LVL externals.

This reverts commit 6f607ea84a2226496628842fab1ed130be355536.

loader: Rename files to better reflect their use

Change-Id: Ie3df06c1c4f78cd515cb4579e93bd80331913669

loader: Fix unknown device trampolines on Windows

Change-Id: I33129c1d80d7c1d1615b7b86a7abfae1c33d0204

loader: Fix unknown device trampolines on Linux

Change-Id: I69d12159534a4569dd2335b4fc52e9ba0bc004b0

loader: Fix unknown phys dev terms on Linux

Change-Id: I36f683c7deb4aac27b27ca253c3324439eb2f2f2

loader: Fix unknown phys dev terms on Windows

Change-Id: I04636aef6ebfcc092d58971574ba5fa0359e08d0

external: Update commit IDs: glslang,tools,header

Update of external reference commit ids.

Change-Id: Iaa004adc60d8b41c5d894476103453491ed54535

layers: GH1922, Fix out-of-tree builds

Change-Id: I82bd4d1285a7ec69c7b8209937235f538375773a

layers: Add VUIDS to allocator checks

Change-Id: If4c37b631de1564dc56bcb1aa69e356538618acb

layers: Add VUs to more PV struct-type_array checks

Change-Id: I9aa1ea41550d24113117d40617689688b96227db

layers: Output VUIDS for PV required_pointer checks

Change-Id: I5b1a9901fdd75b35455fadd749b6ccb31236d23a

layers: Add missing parms to vuid_mapping script

Change-Id: I81e0366e7e2cb2054d8b0f706863e932ef4b8318

layers: Wire up zerobitmask PV VUIDs

Change-Id: Ib9ca0c6b57c4c8faceb9e502c6008624efe8ad23

layers: Wire up ranged-enum PV VUIDs

Change-Id: I334357b239a6f2ad28fad226306897606814095a

layers: Wire in VUIDS for count/array implicit checks

Change-Id: I7957ffe9696f86d1a4cc402b612884e6bd2aa590

scripts: Update vuid_mapping for missing variables

Change-Id: I762509eb848351db7cd5fffbddbac94f8f8aaf0b

header: Update to version 1.0.53 of the vulkan hdr

- updated vulkan.h
- updated reg.py
- updated vk.xml
- updated vk_validation_error_database.txt
- updated vk_validation_error_messages.h
- updated vuid_mapping.py
- updated vulkan.hpp
- fixed error database vuid parser issue
- updated error DB with new VUIDS

Change-Id: I4744f4278981cf808b1b3c9fcf9cf6f5793f1338

build: Fix build on non-x86 Linux systems

Change-Id: I5a35e1bb5f1f9bf9b773588e1a20c24da78ddc35