layers: Add push constant validation to draw_state

Validate that push constants don't exceed maxPushConstantSize at create or update time.
Also validate that size at creation time is >0 and a multiple of 4.

winrtinstaller: Don't create a menu item for each version of RT installed

winrtinsaller: change comment

misc: Remove lunarg_debug_marker extension

This device extensions was not completely supported in any layer. It had partial
support (intercept) in draw_state  and device_limits. This extension is being
changed and has a formal proposal in Khronos as debug_marker_ext. Removing
the existing extension since it is deprecated; once the debug_marker_ext gets
registered by Khronos, layers might want to start using the ObjectTag/ObjectName
commands to supplement error report messages in debug_report_ext. Since
debug_report_ext might get changed as part of Khronos review, don't add it yet.

layers: Allow use of QUEUE_FAMILY_IGNORED for BufferMemoryBarriers

This fixed a bug in drawstate that produced invalid error messages
running apps that used FAMILY_IGNORED.

layers: add missing error check

Change-Id: I18e36f8007d6555a152ac4e0632e2d38a03352be

loader: Update documentation to remove editing comments.
Also better describe layer requirements for layers which creat new dispatchable
objects.

layers: fix imageLayout bug

layers: Fix false positive on initialLayout in image layer

loader: ghlvl 33, ghlvl 34 Fix aliasing of VkPhysicalDevice

This fixes problems with not selecting the proper PhysicalDevice within
the loader if multiple PhysicalDevices are enumerated by the loader.
Fix involves wrapping the VkPhysicalDevice object in both trampoline code
and terminator code.

Also get rid of the ICD device_extension_cache. It is now always queried from
ICD whenever needed.

LX #410 - Use ${PYTHON_EXECUTABLE} for PYTHON_CMD

smoke: use $<TARGET_FILE> instead of LOCATION prop

layers: gllavl#78 validate base pipeline

Change-Id: I08313295ac545652aca2060744b55274ed3cd4ca

layers: gllavl#77 validate layout during clear

Change-Id: I427cd1fcc6ecf115dbaf521387a909f117c13d07

layers: gllavl#76 validate initialLayout

Change-Id: Iede6dcf3bc83a5663b80b62c3239ef54bc74278a

layers: Update QueueFamilyIndex validation

Add handling of VK_QUEUE_FAMILY_IGNORED and allow VkImageMemoryBarrier types
inside render passes. Also plumbed functionName into ValidateBarriers.

loader: refactor EnumerateDevice*Properties terminate/trampoline code

Should be no functional change.
Restructure so the terminator functions don't contain trampoline code.

layers: GH LVL#49, Updated layer docs/README for new layers

layers: GL107, Validate queueFamilyIndices for CreateSwapchain

layers: GL207, Validate queue_family_index

Validates index is less than number of queue families for these APIs:
CmdWaitEvents, CmdPipelineBarrier, CreateBuffer, and CreateImage.
Also:  Updated layer details doc with additional error code.
       Added layer validation test for CreateBuffer

Conflicts:
layers/draw_state.h
layers/vk_validation_layer_details.md

layers: LX322, Track semaphore state acroos QueueBindSparse calls

build: GH PR17, Remove unused displayserver ENV settings

build: GH PR17, Add Android WSI info to Windows version of codegen script

layers: GH PR17, Add support for building for Android on Windows

A better fix would be to stop using sys.platform when making
decisions, but this is quick.

cmake: GH PR17, Improve error detection for displayserver selection

loader: GH PR17, Add windows support for new displayserver types

cmake: GH PR17, Fix vulkan.py code generation script for Windows builds

cmake: GH PR17, Add DisplayServer variables for the WSI interfaces

Build: GH PR17, Add Android WSI info to android codegen script

Conflicts:
buildAndroid/android-generate.sh

cmake: GH PR17, Add WSI options to loader codegen script

layers: GH PR17, Add WSI argument passing to vk-generate.py

cmake: GH PR17, Pass WSI arg to layer generating scripts

layers: GH PR17, Add Wayland, Xlib, Android & Mir extensions to codegen scripts

Conflicts:
vulkan.py

loader: GH PR17, Fix typo in comment

docs: Add CONTRIBUTING.md

demos: Add Hologram snapshot as Smoke test/demo

loader: Better delineate trampoline and terminator functions

No functional changes!

Trampoline functions  have the Vulkan API function name and are in trampoline.c.
Terminator functions have terminator_<Vulkan API name> (omitting the "vk").
Instance extension trampoline and terminator functions are in together in the
same file for the extension.

layers: Fix compliation error in draw_state

android: Drop layer validation tests for now

windows: Add android-generate.bat

layers: remove unused file

layers: Fix file permissions

layers: Fix clang warning

layers: Fix typo in vk_validation_layer_details.md

layers: Validate image barrier in draw_state

layers: Fix descriptor resource tracking in mem_tracker

Add tracking for the buffers associated with bufferViews of texel buffer desriptor types.
Update tracking for all relevant descriptor types to track resources for each descriptor in descriptorCount instead of just the first resource.

layers: draw_state validates VkPipelineColorBlendStateCreateInfo

This is a squash of 3 commits/validationtests:

- If independentBlend not enabled, validate that all elements of pAttachments
  must be identical (for the vkCreateGraphicsPipelines() function).

- If logicOp not enabled, validate that logicOpEnable is VK_FALSE.

- If logicOpEnable is VK_TRUE, logicOp must be a valid VkLogicOp value.

Docs: GH39, Update BUILD.md for directions on installing python lxml module

docs: Update BUILD.md

Better minimum list of required Linux packaged to build this repo, plus other misc corrections and improvements.

Use API Version 1.0.0 instead of using VK_API_VERSION for tests/demos

layers: Add component awareness to SC interface matching

- Adds more tests for mismatch by component and by location
- Removes dead handling of builtins (we only care about collecting builtin /decorations/ to aid in the interface block handling;
    we don't really care about producing the set of builtins used overall.)

Some SC error messages change slightly, as interstage locations now include the component index.

Fixes internal bug 26609754

V2: Remove dead code

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

loader: Fix MSVS warnings

Apply branch 'fix-warnings' of https://github.com/null77/Vulkan-LoaderAndValidationLayers into null77-fix-warnings

layers: Fixed markStoreImagesAndBuffersAsWritten() in mem_tracker

It now correctly treats images as images instead of buffers.

loader: Update doc to add the Linux search path

$HOME/.local/share/vulkan/icd.d or explicit_layer.d or implicit_layer.d

loader: Add example code

loader: doc cleanup

loader: doc cleanup

Fix some awkward wording.

Conflicts:
loader/LoaderAndLayerInterface.md

layers: Make layer option names more consistent and update docs

Check boolean values in vkCreateSampler

demos: Fix vulkaninfo on win32 to use existing console

The vulkaninfo utility is compiled as /SUBSYSTEM:APPLICATION but
attempts to allocate its own console and print to it. Not only
does this appear to be unreliable (sometimes no output is printed),
it is unnecessary if the utility is compiled as /SUBSYSTEM:CONSOLE.

This removes unnecessary console manipulation code. By using
/SUBSYSTEM:CONSOLE, the ordinary main() entrypoint function can be
used.

The console is still enlarged if it was opened exclusively for the
process by detecting whether it was so opened. Likewise, the process
still sleeps forever if the console is exclusive to the process.
The process behaves like a normal command line process when executed
from the command line.

Fixes lunarg issue 350.

docs: fix bad link in layers/README.md

layers: Added TODO in drawstate for Khronos-github issue #30

layers: draw_state VK_WHOLE_SIZE in barriers

Allow use of VK_WHOLE_SIZE in barriers validation.

layers: Allow draw_state layout mismatch with UNDEFINED

Also allow CB initialLayout of VK_IMAGE_LAYOUT_UNDEFINED to match
any layout.

layers: Fix memory leak in mem_tracker

Certain codepaths could result in the validation_functions vector growing
unbounded, causing performance hits in QueueSubmit.

layers: Remove unnecessary null pointer checks

Fixes GitHub Issue #6

layers: LX232, Removed invalid object_tracker QueueBindSparse error message

layers: MR247, Snap start and end to page to compare for mem_tracker

loader: Fix typos, spelling, grammar in loader interface doc

build: Remove LunarGLASS from Win update ext sources.

demos: Properly destroy DebugReportCallback

Error resulted in warnings when shutting down tri --validate.
Also added DebugReportMessage setup.

Update build info to reflect ICD moving to VulkanTools

demos: Fix drawstate transition errors in tri.

Build: Remove LunarGLASS from update_external_sources.sh

LunarGLASS is no longer built from here since it not needed for this repo.
Rather, an update_external_sources.sh has been added to VulkanTools which
can be use to build LunarGLASS.

glslang: Update revision to pick up recent fixes

loader: Fix Wayland support

All that was needed was to fix some misplaced braces, fix a couple of
copy+paste problems from Mir, and add a lookup function.  Getting Mir
support working should be pretty easy.

Conflicts:
layers/swapchain.cpp

cmake: Add options for each of the linux wsi interfaces

We default everything except XCB to off since that is the only one that has
been well-tested up to now.

Don't use width=0,height=0 on window minimize

Vulkan doesn't support width=0 and height=0 so we just ignore the
minimize request on Windows.

demos: Add --validate option to tri

First tries to use standard_validation, if not found then a list of
validation layers.

loader: Fix standard_validation enumeration on instance chain

Fixed regression and still handle no layers found case.

winrtinstaller: update README.txt to better explain rt installer exit codes

layers: Allow draw_state layout mismatch with UNDEFINED

loader: Fix threading layer name in loader standard_threading

loader: Environment variable override got broken with recent change

This fixes that

Conflicts:
update_external_sources.bat

layers: Fix android build and update hash.

Update android build code to fix generated paramchecker.

Fix draw_state custom hash algorithm for ImageSubresourcePair.

loader: Fix windows build from last loader checkin

loader: Add a ICD and layer search path for loader to discover JSON files

path is $HOME/.local/share/vulkan/*

layers: Fix typo in mem_tracker

layers: param_check- enable android build

Update android build to use new param_check header files

layers: Fix possible crash in draw_state layer

Uninitialized descriptor slots are not filled this could cause
null pointer dereferences.

layers: Fix UpdateDescriptorSets crash in mem_tracker

Validation was handling TEXEL_BUFFER descriptorTypes incorrectly.
Removing for now.

demos: Adjust vulkaninfo failure cases

For Windows, after vulkaninfo allocates a console, it tries to resize the
console and change the title text.  Failure of either operation was
treated as console icreation failure, which led to vulkaninfo printing
to the console and then closing the console immediately.  Now treating
AllocConsole() failure as console creation failure and ignoring resize and
title text change failures.  With this change, resize failure will not
stop the app from pausing after printing info to the console.

winrtinstaller: change error message

winrtinstaller: Unique error codes

layers: MR243: NULL pointer and struct type checks

Verify that required parameters are not NULL and that structures with
sType fields have the correct VkStructureType values.  Messages are
logged for the following conditions:
 - A pointer is NULL and is not marked as optional in the XML
 - An array is NULL and is not marked as optional in the XML, unless
   its count is 0
 - An array count is 0 and is not marked as optional in the XML
 - A structre's sType value does not match the value specified
   in the XML

layers: MR243: Add param checker code generation

Generate param checker code to verify that required pointer/array
parameters are not NULL and that structures with sType fields
have the correct VkStructureType value.  The checks are generated
from the Vulkan XML API registry based on the optional parameter
tag and sType comments.

spirv-tools: update versions for 1.0.3

layers: Fix android build of safe_struct wrappers

Handle default case in switch statement for descriptor types.

layers: MR221: Correct name for unique_objects layer

layers: MR221: rename threading test

The threading validation layer has been basically
re-written, so updating the vendor name accordingly