layers: Fix object lifetime bugs in DescriptorSetLayout

Incomplete copy construction of DescriptorSetLayout and safe_* types
causes embedded pointers (such as pImmutableSamplers) to become invalid
before they should. Fixing this for now by moving DescriptorSetLayout
and its embedded safe_* struct to ptrs within their containers and
explicitly deleting.

winrtinstaller: update license

layers: Break out VkDescriptorSetLayout code into its own class

This is the start of a refactor to pull code out of core_validation.cpp
into its own separate classes. I'm starting with descriptor set code as
it isolates reasonably well and it's old and could use some updating anyway.

For this first piece I've broken VkDescriptorSetLayout into its own
class currently called DescriptorSetLayout. I don't know if that's a great
name as it's close to VkDescriptorSetLayout, so I'm open to changing it.
Provided a brief class description in comment in new file descriptor_sets.h.

I made the class interfaces based on what other code is currently using.
I'm planning to pull more descriptor set code into its own classes and I
anticipate that will cause some flux in the class interfaces until most
of the work is done.

loader: ghlvl#180 Support a ICD interface version negotiation

Change-Id: I2e5c3717dd7fa86992f617bcc0f93de8ba60759e

loader: update spec doc for ICD interface versioning

Change-Id: I7c149a4542623e537b44720766006652abe33948

json: Update to version 11.

Change-Id: Ifcdd5423ab03110326897dd9cda272eda05acc01

layers: GH384 Fix to correctly recognize immutable samplers

Actual fix this time. If a binding is immutable samplers
skip over it when verifying descriptor bindings.

Merge pull request #342 from KhronosGroup/update_toolchain_20160414

toolchain: Bump everything to pull in required SPIR-V tools revision

toolchain: Bump everything to pull in required SPIR-V tools revision

Encompasses pull request #261 from Chris.

Also gets things building again reflecting upstream changesi, mainly
HLSL libs for glslang.

Desktop build/test results w/ update_external_sources.sh/bat
 Windows on Skylake: _run_all_tests.ps1, demos with --validate
 Ubuntu on Skylake: vk_layer_validation_tests, cube --validate

Android build/test results w/ update_external_sources_android.sh/bat:
 Windows: VkLayerValidationTests pass
 OSX: VkLayerValidationTests pass
 Ubuntu: VkLayerValidationTests pass

header: Update to version 1.0.11

No functional change.

Change-Id: I190a2af5de66686f070b093bf7bd1459afb9da64

doc: remove rules on vkEnumerate*Properties for layers

doc: formalize the current loader-layer interface

Expand the section "Discovery of layer entry points" into "Layer Library
Interface".  In general, the new section defines an interface that allows
a loader to discover and insert layers while staying compliant.  Other
sections have additional guidelines for layers to follow.

demos/smoke:  Fix validation errors in push-constants mode.

Change-Id: I48e736ef1c777af831d7e888c8ac6e1647d1f6b2

layers: Reorder to prevent Null deref; add test

Found by running Hologram with validation and push constants
using args "-p -v".
Hologram is issuing a memory barrier in the push constants
mode.  This will be fixed in Hologram, but validation should
not be crashing.

Change-Id: I1e76c3fca0152fc3bf8c1de3db3b044f650c202f

loader: fix LX475

Silently ignore VK_NULL_HANDLE in DestroyInstance/Device.

Change-Id: I96305088db75932b6f3f824852db9c72b8352120

layers: Add fix for lx172 to catch invalid line widths.

Validate line wdith in both vkCreateGraphicsPipelines and in
vkCmdSetLineWidth.  Also, add a warning in vkCmdSetLineWidth
if the user calls it but doesn't enable dynamic line width.

Also, updated demos to fix missing lineWidth setting as well.

Change-Id: I62118da9cb5282fcc22b1506e9be2db82b5f4a40

layers: Fix cmd buffer tracking.

layers: Cleanup core_validation fence tracking

There were two separate mechanisms for tracking fence state in core_validation.
This change removes the last[Retired|Submitted]Id per queue method.
We also no longer over-write the fence createInfo to track fence state, but
just use the needsSignaled bool for all fence state tracking.
Finally, there was a validation test/check flagging an error if an
unsignaled fence was reset, but this is valid based on the spec so removed
the check and changed test to be a positive test expected to pass.

layers: Consider #locations consumed by large VkFormats

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

layers: Consider element type bit width for #locations consumed.

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

demos: Add xlib wsi path to cube

Change-Id: I8df42b71a40318b6d46a6b65cc9d8060a8f912a8

loader: ghlvl 361, Simplify library resource handling

Layer and ICD libraries are opened and closed as needed. No ref counting,
no knowledge of if a library contains multiple layers or ICDs.

Change-Id: Ie88b671cd1671187a42d3d838d20e3af1afc67cc

android: Unify source used for build.gradle

In previous commit we modified build.gradle to be cross platform,
so we can stop using the windows specific version.

gradle: Workaround for clang on Windows problems

Use gcc instead if building on Windows.

python: Update script to include all platforms when generating ifdef code

This commit cleans up the platform detection code that was emitting some
(but not all) ifdef wrappers around platform specific code.  It changes
the logic to wrap all the functions that are platform specific.  This
allows us to use these generated files on multiple platforms, not just
the platform is was generated on.

layers: Fix for clearing layouts and cmdbuffers.

layers: Add handle NULL checks to codegen

Add support for validating that required handle parameters are not
specified as VK_NULL_HANDLE to the parameter validation layer's code
generator.
 - Add new parameter validation utility functions to validate required
   handles and arrays of handles.
 - Add new parameter validation layer entrypoints for functions that
   previously had no parameters to validate.
 - Add handle validation logic to the parameter validation code generator.

Change-Id: I7a5680954245db4c1b12587f78e30e17c3903d6c

layers: Add missing 64bpc formats to get_format_type

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

layers: Fix get_format_type for VK_FORMAT_A8R8G8B8_{S,U}INT_PACK32

Fixes GH #330.

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

layers: block members do not express extra array level in member type

Fixes GH #377.

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

layers: GH384 Fix descriptor update check for immutable samplers

If a descriptor is an immutable sampler of type VK_DESCRIPTOR_TYPE_SAMPLER
then it doesn't need to be explicitly updated so don't flag errors in this case.

winrtinstaller: change formatting in RT rtf license

winrtinstaller: formatting of license files, make them more readable

winrtinstaller: fix COPYRIGHT.txt filename

winrtinstaller: get LICENSE.txt file from LVL

demos: Fix paths in gradle file.

misc: Update licenses to Apache 2.0

Change-Id: Id447f4c8c53d33f9192940bdd3ae727b7b738a97

layers: LX468, Prevent bad core_validation error message

Properly track commandBuffer/FrameBuffer references.

Change-Id: Ibbc078b582b204bee26cdc510575c57023cf9d6b

Update typo in doc

layers: Fix object_tracker bugs

A couple of bugs in object_tracker hand-coded sections that MarkY tracked down.
In DestroyInstance case we were attempting to delete wrong map key.
Then, for vkDestroyCommandPool we were using wrong function to clean-up the
command buffers in that pool which would result in occassional crashes.

doc: fix typos in doc

layers: Make logicOp param validation conditional

- Only validate that VkPipelineColorBlendStateCreateInfo::logicOp is
  a valid VkLogicOp enumeration value when
  VkPipelineColorBlendStateCreateInfo::logicOpEnable is set to VK_TRUE.
- Remove redundant VkPipelineColorBlendStateCreateInfo validation (hand
  written code replaced by generated code).

Change-Id: I075c58342678219c5ed1dd5c401e0da2cea53232

layers: Improved generated struct validation code

Improvements to the parameter validation code generator, primarily to
eliminate string concatenations performed when processing structs.  A
validation function was being produced for each struct, which received a
string specifying the name of the struct parameter.  This string was then
concatenated with the names of the struct members.  The struct validation
code is no longer in a separate function, and a single string containing
the full names of each struct member (eg. pCreateInfo->pApplicationInfo)
is created at code generation time.

Change-Id: I2800bfbc26247e4e4c6f8c885c0e7c4b8b3776a2

layers: Simplify generated param validation code

- Remove unnecessary conditional checks for input vs. output parameters
  from the generated C++ code.
- Split some generator code into smaller functions.

Change-Id: I32e47d417ab884e5cc0fc7af40cb5657b39c176d

layers: core_validation - use std mutex

Change-Id: I88af1e86c0ef93dbeab303977b8190ad1968c00d

layers: swapchain - use std mutex

Change-Id: I2c5584a5d36edc097b061db2e49fba2505bd1d65

layers: object_tracker - use std mutex

Change-Id: I5f9ed9dc92daccaada895808e3fea3189ef81212

layers: threading - use std mutex

Change-Id: I773ea148807e5b634ad021ea58b001cd3d1bad87

layers: image - use std mutex

Change-Id: Ie132289c0e601397f54b06c1ef758c65b712f906

layers: Correctly remove secondary cmd buffers from in-flight

Secondary command buffers are considered executing, or "in-flight"
at the time they're bound into a primary command buffer. This leads
to a couple of corner case bugs if the primary command buffer is
never submitted on a queue. This change fixes those bugs:

1. If a secondary cmd buffer is being freed and is in the global
in-flight set, remove it if associated primary is not in-flight
2. When a primary cmd buffer is reset, remove its secondary
cmd buffers from in-flight

layers: Fix vkResetDescriptorPool

When we reset descriptor pool, need to remove the allocated
descriptors from setMap and free them.

Revert "loader: Temp workaround to add WSI surface extensions enumerated."

This reverts commit 884065fcd89540dca9a710389641bf1c39cedce0.

Revert "loader: temp workaround for advertising linux wsi surface extensions"

This reverts commit 23b248d32c943c136589e679e1fcc3444488d906.

layers: Update JSON files to 1.0.10

Change-Id: I2020999bac6090fe40688f977f15c9818b828cf4

header: Update to 1.0.10 header
No functional changes.

Change-Id: Ia57a172a2ab0c7e082ad30aef64354cd1427b528

loader: Remove the default directory paths for ICDs and layers

These are not needed since rely on OSes library loader to figure out default path.

Change-Id: Ide5eec4ed4301c51de7dc9171f7e040deb1878e7

docs: Add libx11-dev to linux build dependencies

loader: Run clang-format

Last several commits to loader weren't run through it.

Change-Id: I512b2df75c7831ee6433e4c60c6664db85184584

layers: Fix 32-bit VS2013 build

Change reinterpret_cast<uint64_t> to reinterpret_cast<uint64_t&> for
unique_objects generated and static code.

Change-Id: I8be5680063ce8a26b58c444e59cb325c4cf6e98b

loader: `Remove the ICD and Layers search path for Windows

This makes it more like Linux which leaves it up to the system
LoadLibrary() call to find the .dll as specified in the .json files.
Hard-coding "C:" or any other search path creates the possibility of
picking up the wrong .dll. The LoadLibrary() implementation does the
right thing.

Change-Id: I58cbdf230a1e86481067efb4aec03447ee55f1ff

loader: dont free and reallocate physical devices

Change-Id: I68ac2681b7c4923213c65dc720f54d81a36c99ef

layers: Fix lx474 by adding image extent = 0 validation check.

Check only the valid dimensions for the imageType provided in
the create info.

Change-Id: I11d484caf1085b7b0ae4a402453b5fdcadd7b67d

layers: Overhaul unique_objects

Unique objects now uses a 64-bit counter as the unique handle returned
to the app. It stores an internal mapping of handles to actual object
ptrs that are used for all non-dispatchable objects. This virtually
eliminates the possibility of having a repeat handle as objects
are created and destroyed.

All counter increments and map accesses are protected by a mutex.

demos: Init var in tri.c

layers: Remove error when only implicit dependencies are used.

layers: Validate ONE_TIME_SUBMIT for secondary cmd buffers

Previously were only validating *_ONE_TIME_SUBMIT_BIT for primary cmd buffers.
After clarification on spec, moving the check so it will also hit secondary
command buffers.

layers: Update core_validation to fix lx265.

Tobin discovered an issue with my previous change.  This resolves
the issue with the blend state targeting only a specific subpass
and verifying the subpass attachment count matches the blend
attachment count.

Change-Id: Iacdd880dde32267da095a3e9baa9e8eb66fdc760

layers: lx464 Add drawtime validation of bound buffer memory.

Add a drawtime check to validate that storage or uniform buffers
have valid memory bound to them at draw time before using.

Change-Id: Ia0a07f221dfcb2ea4315e4f52ee04d7cdda22cf7

layers: lx465 update core_validation image layout warning.

If sub-resource data is present, print out that info as part of
the error message when the image layout is incorrect.

Change-Id: Ibd4d95e22d846b823b5c76986e8646624c109bc6

loader: Added some missing function entry points to loader_icd_init_entrys.

KHR_surface, KHR_display extension functions were not exposed properly.
Specifically, added LOOKUP_GIPA entries for:

* GetPhysicalDeviceDisplayPropertiesKHR
* GetDisplayModePropertiesKHR
* CreateDisplayPlaneSurfaceKHR
* GetPhysicalDeviceDisplayPlanePropertiesKHR
* GetDisplayPlaneSupportedDisplaysKHR
* CreateDisplayModeKHR
* GetDisplayPlaneCapabilitiesKHR
* DestroySurfaceKHR

Added missing function declaration DestroySurfaceKHR in loader_icd
struct.

Checking if KHR_surface extension is enabled before proceeding in the
terminator_CreateDisplayPlaneSurfaceKHR definition.

Change-Id: I0599efddedc5064859ae1f4e52bdbbf8788d5c12

layers: LX466. Resolution of "Typo in validation layer", human spell checker invoked

layers: Fix windows build

VkObjects should be cast to uint64_t by using 'reinterpret_cast<uint64_t &>'

Change-Id: I403efdf389dad61c373df40c5fa1fbed7cf96abd

layers: GH337 Track fence from vkQueueBindSparse

Correctly update fence tracking for vkQueueBindSparse() call.

layers: replace remaining set with unordered_set

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

layers: only construct stringstreams in error paths

These are actually stupidly expensive to construct -- with most of the
cost going to the contained std::locale.

Added up to about 6% of profile on my machine in `smoke -s` with
VK_LAYER_LUNARG_core_validation enabled.

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

layers: Validate correct vkSetEvent() use

Make sure vkSetEvent() is not called on an event that's in use
by and in-flight cmd buffer. Such use violates queue forward progress.

layers: Change access bit checks to warnings.

layers: Include all 4 Linux WSI interfaces

Change-Id: I20b0d64c90e6da940f45b31abf0330b2a415655d

layers: Update typo in comment

layers: Fix comment indentation & remove tabs

layers: cv: get rid of device-scoped current renderpass tracking

This was completely redundant to the (correct) CB-scoped renderpass
tracking, and only served to break things when different threads
simultaneously recorded command buffers.

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

layers: Fix vkCreateDescriptorPool max allocation sizes for each type

VkDescriptorPoolSize array can contain more than one of same VkDescriptorType

Revert "layers: LX459,GH#99,#100, Fix semaphore reference count"

This reverts commit 541bb5afc50581e7c765fc5c05400540369f0044.

Reverting this as the original issue is still present and being worked on
so restoring original state until correct fix is complete.

layers: Clean up semaphore state in core_validation

No need to have 3 semaphore states. Killed SemaphoreState struct and
updated all the code to just use the "signaled" bool.

Also minor clean-up of queueMap destroy code.

layers: Cleanup of various cmd buffer functions in core_validation

Killed some mem_tracker functions that were doing redundant work.
Removed some calls to checkCBCompleted that were superceded by
validateCommandBuffersNotInUse() calls.
Clarified some "my_data" vars as "dev_data."

layers: Cleanup core_validation physical device properties structs

Pull physical device memory properties into layer_data and make sure it's
correctly populated at vkCreateDevice() time.
Clarify a few ambiguous "my_data" variables as "dev" or "instance".
Rename physical device properties to be lower case. This is the start of
a slow move to unify code on common standard with lowercase variable names.

Windows build: Add arg processing to select targets to build.

layers: renderpass compatibility: there is only one d/s attachment

Previously this function assumed there were as many d/s attachments as
color attachments, which is invalid. Besides not making sense, we'd end
up dereferencing memory beyond that pointed to by
VkSubpassDescription::pDepthStencilAttachment if we had multiple color
attachments.

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

winrtinstaller: correction of File Description Properties of uninstaller.exe

loader: ghlvl #66, Ensure std_validationadvertises underlying layer extensions

Change-Id: If6b6b24fdf02b960c83148bd002114941eb1adc6

layers: Use real enum value for alignment checks in device_limits

Previously this used '1'.

Also clean up the enum, and description of device_limits checks.

Found by inspection while reviewing #290.

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

build: rename layers utils lib and fix Windows CMake code

Rename utils lib to VkLayer_utils to use same namespace as
the layers.
The previous Windows CMake code attempted to build both
dynamic and static utils libs, succeeding with only the static.
Change the CMake code to only attempt to build the static lib.

Change from using lxml to xml.etree

Use xml.etree for xml parsing for greater portability.

Change-Id: Ic9b3b487e9b363a89d134d048db9fa79fbd66f6f

layers: sort dictionaries affecting code order

Sort python dictionaries that are used in vk-layer-generate.py
which otherwise produce source code with unpredictable order.

Change-Id: I9b26fa61325098998ff510063a649414807d1378

layers: remove unicode header from vk-layer-generate.py

A unicode header snuck into the front of vk-layer-generate.py.
It does not belong there.

Change-Id: Iad04156d0ed9a0e0c880f88a259b7f5f76406b1d

winrtinstaller: don't delete non-SDK layer registry values during install

build: Fix whitespace

android: Update to get gradlew working again

android: Update documentation

android: Clean up Android.mk