scripts:Mark 639 unimplemented in the database

I believe VALIDATION_ERROR_00639 was marked as implemented because the
case is handled in parameter_checker, but the code itself has not been
updated so I'm switching it back to 'N' in the database for now until
the error code is actually updated.

scripts:Add exception for error 1790

VALIDATION_ERROR_01790 is a special case that provides an exception
for valid usage when VK_AMD_negative_viewport_height extension is
enabled. There's no actual error to flag here, but the code does
handle the exception case correctly so adding it as an exception
in the stat script as well to reflect that it's implemented.

tests:Update AllocDescriptorFromEmptyPool

Update the AllocDescriptorFromEmptyPool test with a new case where
we attempt to allocate more sets than available.
Update the error checking for both cases to use the unique error enum
instead of string matching.

Also update database file to reflect new test case.

layers:Unique enums for vkAllocateDescriptorSets

Add appropriate unique error enums to the code for allocating
descriptor sets from a pool without enough sets left and without
enough descriptors left.
These were already flagged as implemented but the enums hadn't been
added to the code.

layers:Separate push constant max size errors

There's separate valid usage language regarding the relationship
between a device's maxPushConstantsSize and the size and offset of
VkPushConstantRange. Splitting up the validation callbacks to flag
these two errors separately along with their respective unique error
enums.

layers:Separate two error checks

There's separate valid usage language for VkPushConstantRange size
being non-zero and being multiple of 4. Breaking the validation
callbacks to flag these two errors separately along with their
respective unique error enums.

layers: Handle partial failure in vkCreate*Pipelines

This applies to vkCreate*Pipelines creating multiple objects with
a single call.

Creation of *some* pipelines may fail and an error code will be
returned. In this case invalid handles will be set to NULL, and
the remaining handles must be managed properly.

externals: Update glslang and spriv-tools revision

Update glslang to commit 1c573fb and spirv-tools to commit
6fa6a37.

Change-Id: I22692c213fdc1bada0ee14f110373b1d8c76b043

layers: Fix windows build warnings

Change-Id: Ifead20bae1e3c4f237ffc6f49422f20621b6349e

tests: Update misc image VE enums for desired msgs

Change-Id: I08413ccb984f71a91ea89233be742a23331059d7

tests: GH1182, Add buffer/image copy test cases

Added cases to MiscImageLayerTest and updated the VE database.

Change-Id: Ic76d9c5aafc1532cc761c92feaffbf5745e86f93

layers: GH1182, Validate VkBufferImageCopy depth

Validate that for buffer<-->image copies that (for 1D and 2D image
types) the offset is 0 and the depth is 1.

Change-Id: I8114ce78be926411a2938148bf4c60d86006f8a4

layers: Removed obsolete CV validation checks

Removed obsolete CopyImageToBuffer/CopyBufferToImage checks
that output errors if the layerCount was zero and removed
corresponding LVT.

Change-Id: I0a2bada5f896c30fb13cad8a4b3fe157d6d788d3

layers: Moved image/buffer aspect check

Moved this validation check into a routine shared by both
copyImageToBuffer and copyBufferToImage.

Change-Id: If53581cb7dd847492f52b9c2c95b0be25654074f

layers: Update ValidateBufferImageCopyData err enums

Updated enums and database.

Change-Id: If04fa10a233ffab9a1622a6bcdaad803de9c84a6

layers: Update BufferImageCopy var names for style

Change-Id: If39d85522b27e571ab3e7c0e9fb8cc4e63ac0cf9

layers: Update call structure for BufferImageCopies

Converted over to PreCallValidate structure for CmdCopyImageToBuffer
and CmdCopyBufferToImage.

Change-Id: I9e68376bbcaa4c7058eb820b55125c524983b1be

layers: Clarify BufferImageCopy validation name

This routine was used in both ImageToBuffer and BufferToImage but
the name was misleading.

Change-Id: I8ff99cff1757dbaa888cf543451e6a3a4872b003

layers: Fix BindImageMemory typo

Change-Id: I77359a73635d4cdae066255e774a096a9b643a3e

layers:Avoid copy of cmd buffer set

No need to copy the entire unordered_set<GLOBAL_CB_NODE *> in
invalidateCommandBuffers(). Change it to take a "const &" instead.

layers:Use ptr to cmd buffer object in log_msg

This new error was incorrectly referencing the ptr to the cb_node,
which is internal validation state, instead of the cmd buffer object
that the node wraps.

scripts: Improve paths in update_ext_resources.sh

Script was relying on $PWD for determining the current path which
is unreliable in some situations. Updated to a more robust method
allowing spaces in paths.

Change-Id: Ibdb780c479ad7fa831d113e445cc598830806806

build: Add check for spirv-headers revision file

Change-Id: I9bc225275c4d890f44c7ab4dd2076b981778ac20

build: Move revision files into external_revisions

Moved glslang, spirv-tools and spirv-headers revision files into
the external_revisions subdirectory.  Modified the Win/Lin update
external sources files to use the new locations.

Change-Id: I56138d5ae21430bcc564e6a42723accae1bcb9de

docs: Fix loader doc mention of linux dirs

The Linux "Properly-Installed ICDs" section wasn't coming across
into HTML properly.

Change-Id: I2538b3c027c88ad5923d23f554bba1734c60ec06

loader: Fix debug report memory leaks

Found a couple of memory leaks in the debug report code in the loader
while testing out allocators in cube.

Change-Id: I56b401394ca43bd8eb9b4f85baa52dfa597a6f49

scripts:Some database clean-ups

Fix a bad test name in database file.
Update two checks which are implemented but didn't have "Y" in their
implemented column of the database file.
Fix a bug in stats script that failed to turn off red txt coloring.

tests: Update layer doc to reflect recent test changes

Change-Id: I52ed15e0455b88ad7cf7474baa1d1790a13a86f0

layers:Database update

Update database to record that check 1006 is implemented and has a test

tests:Add InvalidQueryPoolCreate test

This test intentionally creates a device with the
pipelineStatisticsQuery feature disabled and then attempts to create a
QueryPool that for queryType VK_QUERY_TYPE_PIPELINE_STATISTICS.

layers:Verify pipe stats are enabled for device

From Riku Salminen

At vkCreateQueryPool() time, make sure that if PIPELINE_STATISTICS
are requested, then the device must have pipelineStatisticsQuery
enabled.

tests:Add DSUpdateEmptyBinding test

Try to update an empty descriptor binding and verify that correct error
is flagged.

Also updated database file to record testname and the fact that the
check is implemented.

tests:Use unique error enum

Update InvalidDSUpdateIndex test to use unique error enum.

Update database to record test for this case.

layers:Remove redundant periods

These error messages are all amended to error messages that already
include a period at the end so deleting the extra periods.

layers:Check for update of empty binding

Validation error if descriptor update performed on an empty binding.

layers: Tweak comments for swapchain checks to be less brittle

Referring to specific lines within an expression breaks every time this
code changes.

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

layers: Remove (mostly poor) descriptions from CV error enums

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

test: Disable the existing WSI tests

The existing support in Swapchain for this is going to be generalized to
every known extension entrypoint in PV, but that hasn't happened yet.

Disable this test for now so we don't get random breakage as other bits
of swapchain get disassembled.

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

layers: Remove husk of capabilities tracking from swapchain

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

layers: Remove remaining bits of image tracking from swapchain

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

layers: Remove husks of Acquire/Present from swapchain

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

layers: Move too many images case to CV

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

layers: Move most of remaining swapchain creation checks to CV

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

layers: Move complaints about presenting on bad queues to CV

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

layers: Have the instance WSI flags unconditionally exist

Code that changes its behavior based on these flags shouldn't also need
to be a mess of ifdef.

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

layers: Track surface support for gpu+queue

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

layers: Move caps query / swapchain creation interaction to CV

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

layers: Capture surface capabilities

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

build: Fix 32-bit Linux notes

demos: Set attachment description flag bits in cube

Fixes a cubepp crash running on AMD

Change-Id: Ie8ef8625a2e8a8a416bcbfe4a62871fef5e07f71

build: Update docs for Linux 32-bit

Change-Id: I38c19a166b25af8dba6bdac4d4c09eb1cfdc698d

build: loader: Fix warnings for 32-bit linux

Change-Id: Ied7668aa7edeef01675dd4f358916d3e8bc07b27

build: Update .gitignore w/new __pycache__ location

Change-Id: I096d61d7e1ff20ab34672efe24a3cfaa888f0018

build: Move vk.xml file into scripts dir

- Update android-generate batch/script files
- Update layers CMakeLists.txt file

Change-Id: I6423f441d61daa20aef2cc5e8bc719113fd7978d

build: Move XML codegen scripts into scripts dir

- Update android-generate.bat and android-generate.sh
- Updated layer CMakeLists.txt files for new path

Change-Id: I1b1c9dbc9e944c90e95542fcfdda1d2e804517f3

build: Move codegen scripts into scripts dir

- Set SCRIPTS_DIR in root CMakeLists.txt
- Update android-generate.bat and android-generate.sh
- Updated layer/loader/tests CMakeLists.txt files for new path
- Updated test layer doc scripts for new location

Change-Id: I2d519ec40883ebeec4e9daab8641ad65515bb08c

scripts: Move determine_vs_version into scripts dir

- Moved determine_vs_version.py into scripts dir
- Updated build_windows_targets.bat
- Updated update_external_sources.bat

Change-Id: I951a05763a7004f2ba641dad72ae3c109ce0aa5c

layers:Warn on cmd buffer invalidate

From Mikko Strandborg

When a resource gets deleted, any command buffer that references it
gets invalidated, and an error gets thrown whenever that command buffer
is submitted. However, for a developer, it's quite hard to deduce what
exactly caused that to happen. This patch emits a warning whenever a
command buffer that's in CB_RECORDING state (meaning that it has had
its vkBeginCommandBuffer called but not End). This way the developer
can set a breakpoint on that to see exactly what causes the buffer to
be invalidated.

Destroying a resource that's being used in an open command buffer is
almost certainly a bug, but there might still be cases where it might
be valid behavior (for example, when the application decides to abort
an operation), therefore it's not an error but a warning instead.

scripts:Update unique error enums as of 11/21/16

4 Updated checks and 1 new check.

Cmd line was:
python spec.py -update -remap 918-768,2:1096-936:2170-1964 > output.txt

scripts:Check image.cpp for unique error enums

Stats script wasn't checking image.cpp when looking for implemented
unique error enums.

Also updated script to include number of enums that it finds which are
claiming to be implemented but not located in source.

Update out of range attachment check to use unique enum

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

layers: Only call ValidateLayouts if no prior error

The long-term solution here is to merge some of this code together, but
validating the layouts later and only if nothing else was bogus is a
good start.

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

tests: Add a test for an out of range attachment reference.

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

layers: Improve wording of attachment-index-out-of-range error

This wasn't correct for the == case.

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

vulkaninfo: Fix variable, struct and function names

Cleanup to match Google c++ style guide.

Change-Id: Ifecbf5c8680159684e24a184421e907d1d815627

vulkaninfo: Add surface present modes

Change-Id: Icb6c30a81743cf5854fd7ecc974d87e814be856d

demos: fix validation error

Change-Id: Idd89d98bf8932c07df2c1452f27d2f3002fb2d8a

demos: call DeviceWaitIdle before DestroyDevice

Change-Id: Ic5819027b706e8c5d66ae78d0d0da1133c16085b

demos: make present mode a command line option

Change-Id: I0dedd685225f5aecaba968ba81da487e5d05da26

demos: move staging texture to demo struct

Change-Id: I8b0319d0f0bab8aba6c41e8fa324f0a9f7128baf

demos: move cmd bfr creation out of layout util

Change-Id: I01c4f59497f4ac22f2a96bda535c7965d34156d8

header: Update header files to version 1.0.34

 - Updated vulkan.h
 - Updated vk.xml
 - updated windows/linux *.json files
 - updated Linux SOVERSION
 - updated vulkan.hpp

Change-Id: I288e2646445aec88aa4a3a9243dd65293fe1028a

layers:Check reqs before call to BindImageMemory

There's an implicit spec requirement that GetImageMemoryRequirements()
should be called prior to calling BindImageMemory() b/c various return
values from GetImageMemoryRequirements() are expected to be complied
with.

This change adds a warning if GetImageMemoryRequirements() has not
been called prior to BindImageMemory(). In this case it will make the
call itself in order to populate internal state tracking.

layers:Check reqs before call to BindBufferMemory

There's an implicit spec requirement that GetBufferMemoryRequirements()
should be called prior to calling BindBufferMemory() b/c various return
values from GetBufferMemoryRequirements() are expected to be complied
with.

This change adds a warning if GetBufferMemoryRequirements() has not
been called prior to BindBufferMemory(). In this case it will make the
call itself in order to populate internal state tracking.

layers:Rename BUFFER_NODE->BUFFER_STATE

layers:Check all memory bindings

There are a few places in the code where we assume that an image or
buffer doesn't have a sparse binding. These cases break with sparse
bindings.

To fix I added a function to BINDING class to retrieve all memory
bindings and then updated a few spots with bad assumption to make use
of this new GetBoundMemory() function.

layers:Check buffer offset against required size

Fixes #1030

Use the required memory size instead of bound memory size when
validating the offset into a memory buffer. This will correctly handle
both sparse and non-sparse cases.

layers:Save memory requirements

Add memory requirements member to the BINDABLE class and save memory
requirements for images and buffers in their respective
Get*MemoryRequirements() functions.

codegen: change script to handle NV ext structs

Modified vk_helper.py to handle NV-specific structures

layers: GH949, Fix ObjTrkr to destroy NULL handles

ObjectTracker emitted errors when a destroy/free call was made with
VK_NULL_HANDLE, which *is* allowed by the spec.

Change-Id: I2b4e19999e539d68b5856139566d4a7a8a530ca6

tests:Add test DynamicOffsetWithInactiveBinding

This is a positive test that features an inactive binding with a
dynamic descriptor and an associated dynamic offset passed in at bind
time.

This previously caused an invalid validation error b/c we weren't
skipping entries in the dynamic offset array for inactive bindings

layers:Fix descriptor dynamic offset handling

Fixes #1162

The dynamic offsets passed in at descriptor bind time are ordered based
on the binding order and any consecutive array indicies within those
bindings. When we validate the dynamic offsets at draw time, we were
using the entire dynamic offset array, but only the active bindings.

If we had an inactive dynamic binding before the end of the bindings
we would use the wrong offsets.

This change fixes the issue by creating a mapping between bindings and
dynamic offset array indicies at descriptor layout creation time. At
draw time the mapping is then used to lookup the correct dynamic offset
array index for a given binding.

Revert "layers: Update vulkan.py with new extensions"

This reverts commit f504ff6e45bb15a0517a8de2e2d75e90df7e4627.
This caused huge issues in the VulkanTools repo, reverting for
now.

layers: Update vulkan.py with new extensions

Change-Id: I1b0623587d7d02849ece97de00ddb1c6bad58819

header: Update header files to version 1.0.33

- Updated vulkan.h
- Updated vk.xml
- updated windows/linux *.json files
- updated Linux SOVERSION
- updated vulkan.hpp

Change-Id: I8128a610d1cd24a1ac49712f28d566f712f4cc94

docs: add libxkbcommon dependency (Keyboard handling)

layers: Update error enum database

Updated status for new MappedMemoryRanges device limits checks.

Change-Id: Id5e1973fa2b7d798abfa3e54f751b69c1b29f2d1

tests: Add LVTs for MapMemRange device limits

Tacked them onto InvalidMemoryMapping test.

Change-Id: I62a094461730bcac14649a2ad6adf5cfdd0f2c58

tests: Fix unexpected errors in InvalidMemoryMapping

Old tests for offset/size range violations did not respect device
limits.

Change-Id: I83a1360197db1cdf6a2d9fcc521627643d3ef74d

layers: GH329, Add MappedMemoryRanges limits checks

When VkMappedMemoryRange structures are used, the offset and size
members must be a multiple of the device limit nonCoherentAtomSize.
Added these checks in core_validation.

Change-Id: If7eded4c52d31c1a25740ac08ec5434c93c5cecc

layers: Fixup MappedMemRanges call ordering

Moved validation out of API calls.

Change-Id: I95f3b24d69d469ea98b7c955bbab56c9b47ad487

layers: Fixup MappedMemory variable names for style

Change-Id: I4f8aae213753dc69ae28a945d7f34cff2af7ccd9

layers: Fixup validateMemoryIsMapped for style

Var names.

Change-Id: I69b441f986c017f2d34a62669c8b3e7511a2dd5f

layers: Update ValidateMemIsMapped error enums

Plumbed in VU error enums, updated tests and database.

Change-Id: I114d269c686cddd6b1d3acb098d0b2f5fe4fdc17

layers: Updated error database for new check

Vertex attribute formats, 1413.

Change-Id: Ieddd05c0f40a643db074ade79659f9c09699c436

tests: Add test for Vtx Attr Format check

Change-Id: Iee24796050cc87d6f912033ccdf0b18785bb2f60

layers: GH1143, Validate vertex attribute formats

Formats specified as vertex attributes (createGraphicsPipelines->
pCreateInfos->pVertexInputState->pVertexAttributeDescriptions.format)
must have the VK_FORMAT_FEATURE_VERTEX_BUFFER_BIT set.

Change-Id: I907c37e2edbcfb7bcb405f912cea8d215acccb11

layers: Save PDev in device's layer data

Needed for physical-device calls or to access data belonging to
the physical device.

Change-Id: I2a1f6eadcaf82c2247a68c91c39d97ef3284b22a

layers: Update CGP var names for style

Changed CreateGraphicsPipeline local vars to match style guidelines.

Change-Id: I3204ce878b37ef3cb0ce58ecff00d65a4fed4f28

layers: Move val to PreCallCreateGraphicsPipelines

In core validation, created a pre-call function and moved the
validation calls into it.

Change-Id: Ib7831c3d2f36628e24b90083be902ac724f25483