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

tests: Skip test for 14a004dc if all offsets are valid

(but first try to artificially lower the attribute offset limit
using LUNARG_device_profile_api)

Also fixes some unexpected VU errors in this test that were
exposed when the main test accidentally passed. Fixes #1781.

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 >=

tests: Handle device extensions provided by layers

Change-Id: I59891fe1d1d7d2a95868fa0bfd4fb2aae5c2adda

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.

tests:Remove unused result var

scripts:Skip doc validation if src dir not found

Fixes #1878

If vkvalidatelayerdoc script is not run from a directly two levels
below the layer source, then just skip running it. Print out a message
on how to run doc validation manually for those interested.

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

scripts:Temporarily disable doc validation

Doc validation makes some directory assumptions that are fragile.
Temporarily disabling doc validation from run_all_tests scripts while
I figure out a better solution for how to organize the script and make
sure it can access all the files it needs.

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.

tests: Remove commented out source code

Change-Id: I44a68dfb8b7015b348553d2f538e84e4b5dadc43

tests: GH1827, Enabled/supported extensions in tests

Also split up imagecopy test for correct operation with extensions.

Change-Id: I08b97c770e0840d83be904c9f18b48b396c25fa3

tests: Add missing parms to InitFramework call

If using InitFramework() instead of Init(), the debug_function and
error_monitor arguments are required for full functionality.

Change-Id: I50a5fd81ee9412ad857d1334bf19664d27d1d309

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.

tests: Fix device_profile_api return types

Change-Id: Iaf039fb68cd20f66fe7f9150e9f695938ce2e70f

tests: Copy device_profile_api json from source

Change-Id: I0277023275f4265ed260d350daaa25a272bd806c

layers: Clean up missing checks list for vkCreateBuffer

tests: Doc validator test changes

Add doc validator back into Windows run-all-tests script.
Add terse_mode option to win/lin run_all_tests scripts

Change-Id: I4fa02ba5caad374a599c2fed42b8661bdb083a77

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

layers: Fix SC interface matching over complex types

tests: Add test for complex types problem

tests: Add positive check for deleted DSLayouts

Deleting DescriptorSetLayouts before associated DescriptorSets
previously caused crashes.

Change-Id: I21fda3826e5836d4885e6baa8807937223117fd3

layers: Use copies of layout data for descriptorsets

DescriptrSets used pointers to descriptorsetlayout data, which caused
invalid references if the layouts were deleted before the descriptor
sets. Changed the layer to copy the data.

Change-Id: I671f1efed2aa0986f3a370b51f2f96c07b555af7

external: update external revisions

Change-Id: I076eb49526ea8b54bde2d584c5779c8251e56a3e

layers: Use correct tracking for display planes query

layers: Untangle validate & update in CmdCopyQueryPoolResults

layers: Split validate & update of CmdWriteTimestamp

layers: Split validate & update of CmdEndQuery

layers: Untangle validate & update of CmdBeginQuery

layers: Convert all the std::bind to lambdas