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

layers: More CmdBlitImage VU checks

Add 11 more VU checks to CmdBlitImage.  Fix layer_validation_tests
errors exposed by these new checks, add new tests for the new checks.
Update database file and clang-format the modified source files.

Change-Id: I3457dd0a31e04ab001bc8fb5bf638eb3f990f5d9

demos: Use do-while instead of clunky while

When acquiring swapchain images

Change-Id: I8c502e06490f73fc5c9cd8057cd12869a66ea942

layers: Add names to objects and print in validation

Change-Id: Ie6587c26c8528ffac5698b8cb3ca2c4cc86b1dfd

layers: Fill out the rest of debug marker calls

Change-Id: I0c4085e36e5c06e11c0c09d1ca126ae6db35526d

loader: Add validation for apiVersion

Add a check to determine if the API version being requested can be
handled by the loader.  If it can't, then we return incompatible driver.
If we can, we continue along our merry way.

An addendum is that we also bumped the Loader/ICD interface version.
This was requested by Nvidia so that the ICDs know that we pay attention
to the version in the loader.  If they don't see the new Loader/ICD
interface version, they will handle the failing on all non-1.0 API
requests.

Change-Id: Icb7dd45e754c9f6a6c8186198333bacc68077b93

layers: Update error db for missed check

Was causing CI to fail.

Change-Id: I38a86432718ff21dd7cf58055b55102190c76937

layers: Add VUIDs for some -parameter implicit checks

Modified generator script and updated the VUID database.

Change-Id: I78da826981521170f1fa620ca0b91be0d4624cc7

scripts: Add missing funcs/structs to mapping table

Change-Id: I124e31e436ec193a6bf55348654c6a99e6daeadc

layers: Plumb in PV implicit requiredbits vuids

Change-Id: I6a9296d7f7bd73bf90855b32c522ab9d2b8bb04a

scripts: Add missing variable names to vuid_mapping

Change-Id: Ib6fb0e6d9bb7d2a75bb0bd751a2ec75a161569ea

scripts: Output VUID for PV pNext errors

Change-Id: I79be1658f827682330e1a9ea6abd9931b0121211

layers: Add VUID parameter to pNext validation

In parameter_validation, plumbed VUID value through to validation
routine, added VUIDs for manual PV functions, updated generator to
compute correct VUID.

Change-Id: Ida15c874ba63826902fe93869cecac87415e0259

scripts: Update vuid_mapping for missing structs

Change-Id: I7f3850738de4ac7bf5dc0bc58285267c3e6044ae

layers: Clarify message for unknown pNext type

Change-Id: I6f7ed5cdfc20401237833c670a8bbc83ce98579e

build: Enable control flow guard on Windows

This enables the 'Control Flow Guard' security feature, which
prevents memory corruption vulnerabilites by placing a restriction
on where application can execute from.

The change is applied to both VulkanRT and it's NSIS installer.

WARN:  This change requires re-patching and rebuillding NSIS with
attached patch.
WARN#2: This change requires Visual Studion 2015 or vs140 toolset to
take effect.

Control Flow Guard documentation:
https://msdn.microsoft.com/en-us/library/windows/desktop/mt637065(v=vs.85).aspx

Change-Id: I7683c0570087d994961c79fb64419509f83536a8

layers: Don't copy all the image subresources every submit

The set of subresources touched by this submission is likely to be very
small compared to the complete set we're tracking. It doesn't make any
sense to copy this entire hashtable in each submit call.

Instead, maintain an overlay table with the submission's modifications
in it.

Saves approx 12s in an internal workload.

layers: Declare aliases for std::mutex and its RAII wrappers

I want to be able to swap out std::mutex for something the VS
concurrency profiler can see, so let's get all the mentions of it to one
place.

scripts: Updated validation_stats error output

Change-Id: I41f592c8fa7a9180d5bd07446d965fa9eb287558

layers: Add sType VUIDs to Parameter Validation

- Updated validation check to accept VUID
- Modified PV codegen to calculate VUIDs for sType checks
- Modified PV codegen to verify VUIDs are valid enums before using
- Updated error database for 65 new VUIDs covered

Change-Id: Ia964822d5a12db0da9c8b425e0ccb72d1bd550a8

scripts: Add parameter_validation.h to doc validator

As the python script doesn't know where the generated source ends up,
added a search of the build, dbuild, and release layers dirs. This
generated file contains LOTS of VUIDs.

Change-Id: I767ae2c3af11a0686ee629edb9964b294b5f7e08

scripts: Remove noisy print from vuid_mapping.py

This script is now used in all builds and was bogging things down.

Change-Id: Ifd8c7d4c5328ce093e1dd9a111536936e1320ca0

scripts: Remove unnecessary library from vuid script

BeautifulSoup4 caused build farm issues. Seems that this lib is
not necessary.

Change-Id: I2463fbc1bc8f8f7ceec86bfbc823f21b5bb64c37

scripts: Update vuid_mapping.py with new APIs

Change-Id: I67ae851149973455f5a2f7c156f5f2d87f0dd51e

loader: Fix handling of JSON

If a JSON layer manifest file says it contained device extensions
but didn't provide a list, or the list was malformed, it caused
issues.

Change-Id: I94b97b190aca35f9464918eb7195486566becaa0

scripts: Move layer scripts into scripts directory

Simplifies sharing code among python modules, keeps everything together.

- Moved python scripts out of layers dir into scripts dir
- Updated relative paths for input/output files in layers dir
- Updated doc validation tests to use correct paths

Change-Id: I57e6ad6f551d4fb652583315fe3dccb23b187db1

build: Exit with return code in update_external bat

Change-Id: Idf9d85d36c66a63de0c07322a367b45cfe84e3de

build: Fix assembler warning on Win64

Change-Id: I234f65c3422628c7a66c53e01044045126c4a7b2

loader: Modify VK_EXT_debug_marker scripts

There's a discussion of whether or not to make the pointers in
VK_EXT_debug_marker const.  If that happens, the current loader
would break.  Fix the loader so that it would work with
constant pointers.

Change-Id: I105c22d9e6228a8ab68b1bca69514ec227f988ea

layers: Do the bulk of pipeline validation unlocked

layers: Rename verifyPipelineCreateState to match new role

layers: Inline pre-validate into CreateGraphicsPipelines

All this did was loop over the pipelines and call a function.

layers: Move vertex format validation down a level

layers: Remove pointless instance lookup in pipeline creation

layers: pass pipe_state vector by const ref

layers: Unlock CreateGraphicsPipelines a bit earlier

layers: test against maxVertexInputBindingStride should be >, not >=

build: Update Travis-CI config

Container updates for 2017Q2:

https://blog.travis-ci.com/2017-06-21-trusty-updates-2017-Q2-launch

layers: Don't accidentally copy pipeline layout again in SC

layers:Make DSLayout const in shared_ptrs

Declare DescriptorSetLayout's const for shared_ptrs in Pipeline layout
and DSAllocate structs.

layers:IsCompatible uses raw DSLayout ptr

Update IsCompatible calls for DescriptorSetLayout class to use raw ptrs
instead of shared_ptr as the calls just temporarily use the ptr and
don't affect ptr ownership.

layers:Set DescriptorSetLayout in DS to const

The DescriptorSetLayout pointed to by shared_ptr in DescriptorSet is
const so add const across uses as appropriate.

layers:Pass shared_ptr by const ref

DescriptorSetLayout shared_ptr will always survive the lifetime of the
DescriptorSet constructor so pass by const ref.

layers:Use make_shared for share_ptr creation

std::make_shared is more efficient for shared_ptr creation, using a
single heap allocation for both the object and control logic, vs. two
separate allocations of using "new" w/ shared_ptr constructor.

layers:Migrate DSLayout to shared_ptr

Update Descriptor Set Layout copy in Descriptor Set, as well as various
ptr references to use shared_ptr.
This fixes some potential memory leaks and makes Descriptor Set class
lighter weight.

layers:Used shared_ptr for DSLayout

Modify DescriptorSetLayout map to use shared_ptr. Will follow-on with
changes to Descriptor Set class that share the ptrs.

layers: GH1889, check for CLAMP_TO_EDGE ext enable

Using this addressing mode in CreateSampler should generate an error
if the extension is not enabled.
Also added layer validation test for this check and updated the error
database.

Change-Id: If0a434976aa4bb30ab23ff6babb95df3af7a4efd

build: Quote asm arg to allow bld w/spaces in path

Put quotes around an assembler argument in CMake file
to reenable building this repo on a path that contains
space characters.

Change-Id: Ic89737f4611161ec0bf4fe53ea59d9e2cd553d9b

demos: Add missing break

Treat suboptimal as success.

Change-Id: I18f0e8592e212c4d1f99294de24cac945bcdb7c3

don't copy vector of pipelines everywhere

layers: Avoid renderpass lookup in verifyPipelineCreateState

layers: Move pipeline renderpass/subpass validation earlier

demos: Destroy instance after closing Xlib connection

In the cube demos, destroy the instance after closing the
display system connection.  It is possible for the driver to
register callback functions with a library like Xlib.  If the
driver is unloaded when Xlib calls those callback functions,
a segfault results.

Fixes #1894

Change-Id: Ieb25a00f727c4ac05ff24b41c3582b293abf4b95

demos: Update cubepp

Modify correct uniform buffer for next frame.
Fix fence sync after new throttling scheme.
Select alpha based on capabilities in cube.
Add uniform buffer per swapchain image to cube.
Cube updated to use triple buffering.

Change-Id: Ia929bde0e6c3c2129dc9164a65696af1ee0c0329

layers: check attachment finalLayout for invalid values

layers: Further removal of CMD_TYPE plumbing

Remove dead cmd_type param from PreCallRecordCmdClearImage

Remove last cmd machinery

This isn't useful now.

layers: Split CB_INVALID state

There are really two states here:
- Had a complete recording that was valid at some point, but not now.
- Fouled during recording

layers: Move validate_functions clear into resetCB

layers: Remove dead TODO for MT/CV merge

build: Enable linux 32-bit builds with ASFLAGS

Change-Id: Ic88fdb9d6c0f23a0b345c82852826a9c1abb8097

layers: Remove debug code from vk_layer_table.cpp

Change-Id: I146461055a1f925ed17c1433c8451aa8a65c5578

layers: Fix memory leak in instance destroy_dispatch_table

Change-Id: I8152960f30f73a997386aa9bf3561d2e0e8b408d

layers: Fix memory leak in device destroy_dispatch_table

Change-Id: I14aa005db87caab3cbb28a4e06b1d2a76c931341

build: Do not overwrite CMAKE_ASM-ATT_FLAGS

Allows passing flags to assembler in order to build 32-bit targets
on 64-bit platforms.

A 32-bit build can be produced by passing -DCMAKE_ASM-ATT_FLAGS=--32
to cmake.

scripts:Include string.h in ext helper

vk_extension_helper.h uses strcmp so add include of string.h to make
sure this dependency is correctly met.

layers: Clean up missing checks list for vkCreateBuffer

layers: Allow suppression of doctest output

Added checking in the doc validator script to look for a 'terse_mode'
option which suppresses its informational/warning messages.

Change-Id: I986fb6070b0de53d56e45f5da4b05211203a1b8e

loader: Fix alloc issue

We were allocating the wrong struct for the dispatch table object
when creating the instance.  This left us using invalid memory
when using the physical device extension trampolines.

Change-Id: I9939a8c9fe320b0d07592ab4beb5b6faaba40383

build: Fix ninja build

A previous change of mine broke the ninja build in the loader. This
commit resolves that by fixing the include path

Change-Id: Iaabf3488b323bb3d2d0e40a7d555769c528e61e3

demos: Modify correct uniform buffer for next frame

Change-Id: I78b8f5ef63fb2c14a3ce431f3e6f6a809e65972f

layers: Remove dead comment about interface block handling

loader: Fix unknown phys dev trampolines on linux

Change-Id: I6a8de7feb44587601cc72342f996a26f8ae6bcd8

loader: Fix unknown phys dev trampolines on win32

Convert unknown physical device extension trampolines to assembly
on windows. This is necessary because the previous method relied on
compiler optimizations that were not being run by MSVC, even with
full optimizations enabled.

Change-Id: Ibefa6ed24f8c91b1e723fab9c1cfd3039fabd64b