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

android: Fix typo in Mac steps

android: minor indentation correction in Android.mk

android: Minor clean up for windows build template

android: fixed issues in windows build

tested: windows/mac

android: set hard float for arm7 correctly and minor cleanup

tested: Mac X

android: Changed android-generated.bat for windows (not tested )

android: Adding template for gradle wrapper build

layers: GH241 vkSetEvent should affect queue stageMasks

vkSetEvent may occur asynchronously from cmd buffer execution
and the results of vkSetEvent should be immediately visible
across all queues of a device.

This fix will update any event that a queue has seen so that
its VK_PIPELINE_STAGE_HOST_BIT is set at vkSetEvent() time.

loader: Fix the std_validation version to really be loader's built version

Change-Id: I2ff3b6dab4209b152f487649db011cdfa7c8975b

winrtinstaller: re-sign powershell script

layers:  INVALID_ATTACH_COUNT -> INVALID_ATTACHMENT_COUNT

Fix error reported by vkvalidatelayerdoc test.

Change-Id: I7e1bb4e5cb8c113202d479ef95cc0d37334040b1

loader: Preserve order during layer list expansion

As requested by Jon -- instead of appending the expanded layers, insert
them in place of the meta layer.

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

loader: Don't scribble on caller memory in CreateInstance,CreateDevice

expand_... / unexpand_... scribbled on both the CreateInfo struct and
the list of layer strings, and then unscribbled them on the way back
out. This is a lousy thing to do, and just blows up if the memory isn't
writable (which it needn't be, given the API takes ptrs to const).

Instead, copy the *CreateInfo into a shadow struct on the stack, and be
careful in expand_layer_names never to scribble on the caller's layer
names array.

V2: slight tweak (missed initializer)

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

demos: Add missing spaces in error messages.

Change-Id: Ic9b2b2a2ea14bb087d74f6c0b697449b141899a4

MR 289: layers: Changed other text/variables from "owned" to "acquired".

MR 289: layers: Change the test about "too many images".

Given new spec language coming out in early April (clarifying various aspects
of acquiring presentable images), a change was needed to the WSI-swapchain
validation layer.  The new validate-layer test will work for "BlitModel"
drivers that advertise a minImageCount of 1.  Failing the test now results in
an error instead of a performance warning.

loader: temp workaround for advertising linux wsi surface extensions

Change-Id: I9a03c6ea16abe9e362ef81ee87ac217b8b3c6c92

layers: Don't exceed max allowed color attachments.

layers: Move call to validateCommandBuffersNotInUse inside lock

Fixes segv in CTS multithreaded command pool test

Change-Id: I80da12fd2bb6856e3c6824f17dd92f1597121a54

loader: Update specification doc for dispatchable object initialization

Change-Id: I80770b7ab14795316400279dc4e7c954fa5ad374

layers: Fix 32-bit Windows build

Change reinterpret_cast<uint64_t> to reinterpret_cast<uint64_t&>.

Change-Id: I87b8479df6ba7f93b922bab11edbbf6c4fbfe7f6

layers: Ensure bind point for subass is graphics.

layers: Fix MSVS value truncation warning

Changed the type of the string validation constants to from char to
uint8_t to fix warning C4309: 'initializing' : truncation of constant
value, due to an implicit unsgined char to signed char conversion.

Change-Id: I86cca4b1a8c3f818a2e7a9a7e38f05ee8cd01150

demos/tests: Fix MSVS type warnings

Fixes the following MSVS warnings:
- C4800 - forcing value to bool 'true' or 'false'
- C4244 - conversion from type1 to type2, possible loss of data
- C4267 - conversion from size_t to type, possible loss of data

Change-Id: I78576c48c5d42e4c9116a83c5f4a4f4a5035e0fc

layers: Remove plumbing through of unused dispatchable object through MT

Much of this was eventually going to go away anyway, but getting rid of
this thread of it was trivial.

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

layers: Merge binding maps into image and buffer maps

mem_tracker layer had specific maps to track memory binding for images
and buffers. This change merges the data from those maps into IMAGE_NODE
and BUFFER_NODE respectively and eliminates the binding maps.

Update vk_layer.h

Correct comment for VkLayer*CreateInfo's sType

loader: Fix regression with dEQP-VK.api.object_management.alloc_callback_fail.device

When the loader vkCreateDevice trampoline code calls down to
vkCreateDevice don't update the dev->device field with a bogus device
handle.

Change-Id: I703acb5283650ab0013483bda0f478da9b46e3ea

layers: Insist on each shader's stage being unique in a pipeline

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

Validate attachment index is in range.

layers: Implement relaxed vec-vec and vec-scalar matching rules

- Track whether we've seen anything disqualifying -- non-stripped
  arrays, matrices, structures, etc.
- Add relaxed vec/scalar and vec/vec matches if not disqualified.

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

layers: Explicit VK_TRUE/VK_FALSE checks

Add explicit VK_FALSE/VK_TRUE conditional checks.

Change-Id: Idb5c82ce53d54c9f2c378dc557152efc695ae2b1

layers: Fix unsafe bool mix warnings (part 2)

Fixes the MSVS warnings regarding unsafe mix of VkBool32 and bool for the
device_limits, image, parameter_validation, and threading layers.
Converts all Boolean usage internal to the layers to bool/true/false.  The
VK_TRUE and VK_FALSE values continue to be used with the Vulkan API
components.

Change-Id: I612d3f26050fadcd77ffca4d8723a7c734613816

layers: Fix unsafe bool mix warnings

Fixes the MSVS warnings regarding unsafe mix of VkBool32 and bool for the
core_validation layer.  Converts all Boolean values internal to the layer
to bool/true/false.  The VK_TRUE and VK_FALSE values continue to be used
with the Vulkan API components.

Change-Id: I9a298e2a9f5f92a388a2b7c158451d237fdb4fc0

loader: remove unused instance_dispatch.png

Change-Id: I67ebce2e4e46a5a1439bb35fb966ed522c80ee52

winrtinstaller: sort in config script: <prerelease> is least significant

Also added execute permission to CreateInstallerRT.sh