layers:Check renderPass/framebuffer compatibility

Add check for VUID VALIDATION_ERROR_12000710. This is a check at
CmdBeginRenderPass time to make sure that the render pass being begun
is compatible with the render pass that the referenced framebuffer was
created with.

layers:Generalize validateRenderPassCompatibility

We have 2 separate function for checking render pass compatibility that
are nearly identical. This change is refactoring one of the functions
to make it more general-purpose so that it can be used for all of the
render pass compatibility checks. This includes passing through the
caller, a VUID, and some strings for the types of objects that are
having their render passes compared.

Added some temp comments on VUIDs that will be added for various render
pass compatibility checks.
Update framebuffer state struct to store render pass state ptr instead
of just the render pass create info.

external: Update Android commit IDs

With this, LVL's Android externals match those of shaderc's known-good
as of cd52beb9

Change-Id: Ic408469d007b632f16229b3e2be0abe82abe5869

layers: Add script to create VU stats in CSV

A simple utility bash script to generate Valid Usage completion
statistics in CSV format, for loading and analyzing in a spreadsheet.

Change-Id: Id45a20ea83d555029cb11418ac5d19c38fd28a15

external: Update spirv commit IDs

Update externals commit IDs for spriv headers and tools.

With this, LVL's externals match those of shaderc's known-good as of cd52beb9

Change-Id: I610c68d1c7910e70acfa7065c64c23862a7d25c5

docs: Update loader doc for pnp registry changes

Change-Id: I3d41b7b01140243457ac35cfa2c22235f35e4e83

loader: Allow layer loading from pnp registry

Change-Id: I70682cf5904b95b52881457e14bbc2f9a772259d

loader: Fix PnP 32-bit registry handling

Change-Id: I25eae83b4783e702ca5fbe74a99d7bdeae6218f0

layers: Fix leaking backing sets for push descriptors

We'd previously leak any temporary descriptor set left bound to the
pipeline at the end of the command buffer. Rearrange things so we can
use unique_ptr and assure it's always cleaned up correctly.

headers: Fix a typo in vk_icd.h

Change-Id: I9ede42e379605329ffd25a9466e0971b3ab7dd77

build: Touch appveyor config to invalidate cache

layers: Remove remnants of old perf warning for disturbing sets

Half of this was removed, but the surrounding plumbing was left in
place. If we want to quietly add some 'was disturbed' bits so we can
produce a better error at a later draw, that should go on the
record-side of this validation, not here.

Removes the other case that the previous change missed, along with the
vestigial bits of the previous check.

Remove subtest and surrounding scaffolding that tested this

CMake: GH1989, add wayland include dir to cmake

On some Linux distributions (e.g  openSUSE) the wayland headers are in
a subdirectory.
Adding WAYLAND_CLIENT_INCLUDE_DIR to include_directories in CMakeLists.txt
will fix this issue.

external: Update commit ID: glslang

Update to most recent glslang-specified commit.

Change-Id: I35d8438833319e2d662dcf20353199d52f0b2ac0

layers: Account for push descriptor sets in CV

Change-Id: I1095edce0f26e2a4ced6fa301143622dc5cd0afd

layers: Match state record pattern for VOID APIs

Also straightened out the locking/unlocking for PushDescriptorSets.

Change-Id: Ic72bc693d16e7fa85df117d29abc1adc5a29ada6

layers: Update desc binding state for PushDescSets

Using PushDescriptorSets in a pipeline with normal Allocated descriptor
sets resulted in incorrect descriptor binding validation errors.
PushDescriptors are now run through the descriptor binding
state-update functions.

Change-Id: Ib702949af6de0b8638401c7eeb841c2590998e82

layers: Intercept PushDescriptorSets in CV

Change-Id: Ifabf7058b4ded33ab191a0043d9ef390b4c8cb06

layers: Refactor CmdBindDescSets for pre/post

Validation and state updates are now somewhat less tangled.

Change-Id: Ia666215a73ce02d2fbc326fb5b37b3361d83bbe3

android: Update shader toolchain revisions

Pulled from:
https://github.com/google/shaderc/blob/90a90d6f745704ac05afe70b7e8f759bec2ff493/known_good.json

loader: Fix GetProcAddress loading illegal funcs

The loader would return an unknown device extension handler when
specifying a function from a layer that was available, but not
enabled.

Change-Id: I0d77a7489d66e4d77aa8446018ec545c930f5bcf

external: Update commit IDs: glslang, tools, header

Change-Id: Ic0777dda7b6958d2454dfec288630044e6f70db3

build: Improve component enables usage text

build: Add no-sync to update_ext_srcs batch file

Complete support for cygwin environment by adding --no-sync support
to update_external_sources.bat. Also change command line parsing for
the the .bat script to be similar to the .sh script.

Change-Id: Ie813938654e7f4bb9575eef2587f36ca2933f30c

build: Improve cygwin build support

Partially support cygwin environment by adding --no-build to
update_external_sources.sh

Change-Id: I9fa35d6c720dd05a11c307d3da4d09415934899e

build: Add missing quotes

Change-Id: Icff4a71bbedc3ee95d2dc2b85224a3c2eee74cb6

scripts: PV codegen handle multiple requires

Modify parameter validation codegen to handle multiple require blocks in
an extension, each with additional extension dependencies.

Change-Id: I336569ca19808f3dca342eb65fd076599c4fb329

layers: use unique_ptr for pipeline map

Loader: Fix out-of-memory handling in loaderGetDeviceRegistryFiles

Wrongly handled OOM error causes
dEQP-VK.api.object_management.alloc_callback_fail test to fail, because wrong
error code is returned.

layers: Fix crash in CreateGraphicsPipelines reading from moved-from ptr

Revert "Revert "layers: unique_ptr for temp compute pipelines""

This reverts commit 615dc4e3ea0495bc28130377b9e637e9979a560e.

Revert "Revert "layers: Introduce some unique_ptr into pipeline creation""

This reverts commit 5d6aaa553b634747fb2761c6e5b665bdae54d206.

Revert "layers: Introduce some unique_ptr into pipeline creation"

This reverts commit 7d35703bdc03d94cad272b5ea3ede8d586f57e88.

Revert "layers: unique_ptr for temp compute pipelines"

This reverts commit cfc1ed15584a12ef62e113c8daf673c6a68ce29c.

layers: unique_ptr for temp compute pipelines

layers: Introduce some unique_ptr into pipeline creation

scripts: Enable instance extension checks in PV

Parameter_validation's hammer for excluding some pdev-ish checks was
too large.  This adds extension dependency checks for about 50 more
APIs.

Change-Id: Icb44cdb36c8c0384781e04e207ceb18d9f67ff0c

header: Update to version 1.0.59 of the vulkan hdr

- updated vulkan.h
- updated vk.xml
- updated Win/Lin json files
- rebuilt and updated vulkan.hpp
- updated vk_validation_error_database.txt
- updated vk_validation_error_messages.h
- added new code-generated VUIDs (16) to error database
- modified object tracker for VUID corner-case
- updated vulkaninfo.c for STYPE name change

Change-Id: I0870e8bc4e84867e05bd466c13c7428b9b7365ac

layers: Use correct data layer in vkCreateInstance in param validation

layers:Remove disturbed descriptor perf warning

Fixes #2020

Support from devs to remove this perf warning so just killing it.
Added a comment for future work where we can track which descriptors
are disturbed and reference that if a user attempts to use a
disturbed descriptor that isn't bound.

Also commented out associated test and filed #2022 to track enhancement
making note of disturbed descriptors if used unbound.

layers: Fix memory leaks in functions

Change-Id: I8e25adabefde3a04be5536ce4c5c2d5920a76140

vkjson: Add support for Android

This commit adds vkjson and vkjson_info to the Android build.
It is only generated as an executable for devices with root
access, but it could be expanded to an APK with some more
boilerplate work.

See updates to BUILD.md for how to build and run on the device.

layers: Wire up static_status check for CmdSetScissor

layers: Wire up static_status check for CmdSetViewport

layers: Drop error for binding compute pipeline inside renderpass

It's perfectly valid to bind the compute pipeline inside a renderpass.
An application only may not /dispatch/ compute work inside a
renderpass.

layers: Drop out some redundant checks already covered by OT

layers: Calculate viewport/scissor state bits correctly

layers: (re)-add viewport and scissor cbstatus flags

We won't actually check these at draw time since we have more precise
per-viewport and per-scissor support there, but we need these in order
to easily reject CmdSetViewport/CmdSetScissor when the pipeline doesn't
have the appropriate dynamic state enabled.

android: Updates for NDK r15

Removes app_dummy from apps that use native activity.

For more details refer to: https://github.com/android-ndk/ndk/issues/381

layers: Fix img layout state in CmdExecuteCommands

Previously the layout state in the primary command buffer was just
overwritten with the state in the secondary command buffer.

This can lead to spurious errors upon submission of the primary CB, if
an image used in the secondary CB is used prior to CmdExecuteCommands
in the primary CB with a different initial layout. This is because the
initialLayout field for the image on the primary CB is overwritten
with that of the secondary CB.

An example case that was incorrectly erroring (these commands refer to
the same image, newly created):

  (primary buffer)
  vkCmdPipelineBarrier, transition UNDEFINED -> TRANSFER_DST_OPTIMAL
  vkCmdExecuteCommands
    (secondary buffer)
    ...
    vkCmdCopyBufferToImage, layout TRANSFER_DST_OPTIMAL

Just before the CmdExecuteCommands, the image had initialLayout =
UNDEFINED and layout = TRANSFER_DST_OPTIMAL on the primary CB, and
initialLayout = layout = TRANSFER_DST_OPTIMAL on the secondary.

After CmdExecuteCommands, the primary CB state is overwritten, so
it has initialLayout = layout = TRANSFER_DST_OPTIMAL. At submission,
this then errors because the global image layout is UNDEFINED.

Fix by only setting initialLayout on the primary CB from the secondary
if the image hasn't already been used on the primary.

Change-Id: Iae248ed09b1f87598fd192ba3d6f0bcf7ad38ada

layers: Copy queue submit fcns in CmdExecuteCommands

When a secondary command buffer is executed, copy all functions in its
queue_submit_functions list to the primary command buffer.

This is needed e.g. to ensure that functions registered to validate and
update memory validity state for a command in the secondary CB get run
upon submission of the primary CB.

Fixes spurious memory invalid warnings on later commands after an
image's memory has been initially made valid inside a secondary CB.

Change-Id: I39d5a34f074942d5dedbf4c5a0295af957a92361

layers: Update Database to fix CI

Couple of VUIDS were not added to the database.

Change-Id: I8ad8868182c8d129622d616241f5e0a14cb5054e

layers: Check cb::static_status for most dynamic state setters

layers: Track static state bits separately

This gives us two advantages:
- We no longer need to recalculate the state bits to remove for the old
pipeline when binding a new pipeline.
- We now have the set of bits to test against when trying to bind
dynamic state for #614

layers: Invalidate old pipeline's static state when binding

Previously we'd only ever or in new bits, causing us to miss cases where
some state changes from static to dynamic within a CB.

Also only fiddle with the state masks when binding a graphics pipeline
-- previously we'd miss some other cases because binding a compute
pipeline would set all the state bits.

layers: inline set_cb_pso_status into sole caller

Split out dynamicstate->flags conversion

layers: Move PV custom funcptrs into map

Simplifies steps necessary for new custom functions.

Change-Id: Ib2d80346ebc42011dce74fdd8cee62adb4b3432c

layers: Re-architect parameter validation layer

Changed the codegen to autogenerate validation functions for any new
APIs without user intervention. Manual updates are only necessary to
extend PV functionality.

- parameter_validation.h is now checked into the layers directory
- parameter_validation.cpp is now generated
- parameter_validation_utils.cpp is new and contains any custom
      (non-generated) or housekeeping functions.
- parameter_validation_utils.h is deprecated
- updated database for new VUID coverage

Change-Id: Ib9261894386145573ba9e8906ba4ad4e9fecbdfd

layers: Add checks for Image/ImageView usage

Change-Id: Ibb2ac5b5f4b4176f5745007598c2fe64665237c2

demos: Fix disappearing objects in smoketest

When an object reaches the end of its "curve" path
it receives a new curve.  To ease the transition from
the old curve to the new curve, the origin of the new
curve is the ending point of the old curve.  However,
this can result in the starting point of the new curve
being out of the view.  This results in objects slowing
leaving the view.

Add code to move the origin of the new curve back inside
the original view bounds only if the end of the old curve
is out of the view bounds.  This does not add any additional
jarring because any object getting a new origin is already
outside the view anyway.

Change-Id: I50e88be453f31b9c37878f63c97d4c7c8154e718

header: Update to version 1.0.58 of the vulkan hdr

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

Change-Id: I8c525b6a2bd69914440c6d7984a67446b27e6c23

demos: Add compositeAlphaFlags fix to cube.c

Change-Id: Id19d11e909ffc21c4aa6a419b071ba53594ae94f

demos: cube.cpp - iterating over compositeAlphaFlags

Error in for loop condition

Change-Id: I8c775f961a6c05dd36e7acfdda6e94399365f90d

layers: Fix layer settings file debug action name

Change-Id: Ic5a6828d57515efb50c12e21ef45e690c058c96a

layers: GH1991, Implement BREAK debug action

Change-Id: I19ee76db3a380116c7c25819a112751d988e23b4

loader: Replace overzealous warnings with info

The new ICD search functionality was logging warnings in some places
where information was more appropriate, such as when opening a device
or for some errors that simply indicated that the new format is not
supported yet.

Change-Id: Ifc8ce33d778783f3f55102783bf6ede0dd7c2a00

loader: Don't load manifest files more than once

Change-Id: I2802ad0c1fd16c0287ed24b96ec6d81c4b3c711a

loader: Fix size handling when scanning registries

loaderGetRegistryFiles() should not start from 4096 size, if
loaderGetDeviceRegistryFiles function has already appended something
into the buffer.

loader: Add new ICD search paths to loader

This change extends the functionality of searching for ICD JSONs by
adding registry locations specific to given display adapter and
software components associated with this display adapter.

The exact locations in registry are queried using Windows public PnP
Configuration Manager API[1].

This change is required, as previous ICD locations (constant path
in "HKLM/Software") may be unreachable for drivers and their
installers on Windows RS3[2].

Similar change is being made for OpenCL[2]

[1]https://msdn.microsoft.com/en-us/library/windows/hardware/ff549713.aspx
[2]https://github.com/KhronosGroup/OpenCL-ICD-Loader/pull/21

layers: PR1950, Update subresource range VUIDs

With updated test names in database.

Change-Id: Ie6cb2dfa9fa84de2dd0dea645249fe03c6780eba

layers:Fix consecutive binding updates

Fixes #1992

When we're wrapping a descriptor update for consecutive bindings, fix
code so that index into update array consistently increases.

build: PR1972, Install vk_layer_dispatch_table.h hdr

vk_layer.h is installed and includes it. Also, vk_layer.h is needed
to write third party layers, so I don't think not installing vk_layer.h
is an option.

Change-Id: I3e34d8484fdc5092edff3bab72991303a4aeb2c7

layers: Untangle side effects in CmdWaitEvents

layers: Fix CmdPushDescriptorSet object tracker

Ignore the dstSet field of VkWriteDescriptorSet as indicated
by the spec

Change-Id: I28ca6ea3059f5528bdf369954e1621ef670e0d54

Update commit IDs: glslang,tools

Change-Id: Ie24b136b2df3718ba1c9ee493d69716304795a11

demos: Revert AppDevInitFormat changes

Calling vkGetPhysicalDeviceFormatProperties with extension formats is
not valid and causes crashes on some drivers.  Revert the list of
VkFormats to the original range.

Change-Id: I2b7f0bd5bb49a66cd073216e93e3bba35c67ab98

layers: Fix swapchain extent check

This fixes a regression introduced -- by me :( -- in commit
583b0c41dfd0876d0481920f2e6d8d87433bc1d7. The swapchain's
imageExtent must always be within the surface's min/max extent,
regardless of the surface's current extent.

layers: Fix incorrect VUID for negative viewport ht

Layers used the wrong VUID for the checks for negative viewport
height if the extensions were not enabled.

Fixed existing test, added a new test, and updated the database,
and removed special case from validation_stats script.

Change-Id: Ia165e6245990a4fabb3745102b345a7b4f1b0202

layers: Don't do bogus transitions in invalidateCommandBuffers

Total nonsense like:

  RECORDING -> INVALID_INCOMPLETE -> INVALID_COMPLETE

was possible when there were two invalidations.

header: Update to version 1.0.57 of the vulkan hdr

- updated vulkan.h
- updated vk.xml
- updated cgenerator.py
- updated Win/Lin json files
- updated and fixed vk_validation_error_database.h
- updated vk_validation_error_messages.h
- updated vulkan.hpp

Change-Id: Idebc490660833daac43d234db44131e9bf2b910b

external: Update commit IDs: glslang,tools,header

Update external reference commit IDs for Desktop and Android

Change-Id: I0b79c9e030e06479d1f07d4d3372dcc96bfa2f0e

layers:Rename lambda function container

Change name to queue_submit_functions to reflect the fact that validate
functions in this container are executed at QueueSubmit() time.

layers:Pass framebuffer into barrier lambda

Pass framebuffer into image barrier validation lambda so that secondary
CB state doesn't have to be updated in CmdExecuteCommands. Just pass in
framebuffer from primary CB instead.

Update secondary CB lambda container name to reflect the fact that those
functions are executed at CmdExecuteCommands() time.

layers:Delay barrier validation for secondary CB

When a secondary command buffer doesn't have a framebuffer, queue up
any renderPass barrier validation until submit time when we know we'll
have a framebuffer bound.

Added a separate vector of validation functions that are added to
secondary command buffer for this case and then executed at
CmdExecuteCommands() time.

Migrated the image validation that needs to be delayed to its own
function. This makes it easy to add to the delayed function array and
capture a copy of the image barrier that needs to be validated.
In CmdExecuteCommands() when validation runs inherit the activeFB
from the primaryCB into the secondary CB.

demos: Fix cube to use correct pipeline stage for barrier

Change-Id: Ia85e0acc67694d859077044ad24b13b0ad834811

tests: Update VALIDATION_ERROR_1b800942

Change-Id: I14651b387848d3d1cc8bd3f5e0051ba8dce12e66

layers:Verify dstAccessMask supported by stageMask

This is VALIDATION_ERROR_1b800942.
Verify that barrier drcAccessMask is supported by drcStageMask.

tests:Test srcAccessMask/srcStageMask conflict

New test case for VALIDATION_ERROR_1b800940.
Create srcAccessMask conflict with srcStageMask in memory barrier.

Fix a bunch of existing test cases that were triggering this new
check b/c srcStageMask conflicted w/ barrier srcAccessMask.

layers:Verify accessMask supported by stageMask

This is VALIDATION_ERROR_1b800940.
Verify that barrier srcAccessMask is supported by srcStageMask.

tests:Test queueFamilyIndex for image barrier

Add test for VALIDATION_ERROR_1b80093c to RenderPassBarrierConflicts.
For image barrier in renderPass, src&dstQueueFamilyIndex must be
VK_QUEUE_FAMILY_IGNORED.

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