trampoline: extract device cration routine

loader: VK_EXT_headless_surface additions

As VK_EXT_headless_surface is a WSI extension, supporting it
requires changes to the loader. This commit makes the necessary
changes, which are mostly straightforward plumbing.

However, this commit also includes a minor tweak to avoid a
segmentation fault when an implicitly enabled layer provides
support for an instance extension.

repo: Bump known good to 1.1.107 header

Change-Id: I7b98293da8a326191cd7787936f373109d60e640

Revert "build: Use Vulkan::Headers target for include paths"

This reverts commit f22cdac76d5cebe4fe354114d6f48861bab477dd.

loader: Fix mishandling of loader logging

This fixes several instances where loader_log() was being called
incorrectly. It also removes a stray va_start within loader_log()

Change-Id: I6e8a66f999e78367e23241fac489db5efc927719

Update Dispatch Table update to actual location

DispatchTable struct now has it's own file.  Updated references to the correct file.

build: Use Vulkan::Headers target for include paths

Change-Id: If693b2234d67f90c2e62a0706bac7c4b0e424f4c

loader: Support VK_EXT_full_screen_exclusive ext

This change adds manual handing for new APIs which need to unwrap an
ICD surface handle before calling down to the ICD.

Change-Id: Id64c8021a3d36800af9a59cafe52577a4d5d69ff

scripts: Update known-good for 1.1.106 header

Changes:
- Integrate upstream script changes: We have to plumb-through the new
conventions object to continue using the makeCParamDecl utility function
- Add GGP to available platforms
- Add handling for extension dependencies: Previously, the codegen for
loader trampolines could not handle an extension command that depends on
more than one extension being present. This removes that limitation
- Add checks for device extensions: This adds a check for two functions
at device creation time:
  * VK_KHR_device_group
  * VK_EXT_full_screen_exclusive
The loader needs to know about these extensions for proper handling
of the vkGetDeviceGroupSurfacePresentModes2EXT terminator
- Update known-good file

Updated:
- `loader/loader.c`
- `loader/loader.h`
- `scripts/common_codegen.py`
- `scripts/dispatch_table_helper_generator.py`
- `scripts/helper_file_generator.py`
- `scripts/known_good.json`
- `scripts/loader_extension_generator.py`
- `scripts/loader_genvk.py`

Change-Id: I9f0828a8eee0e8e95b479e1b8feb31acaa10040d

loader: Tweak code to better match conventions

Change-Id: Iaf91c8ea7e57663a8264efe07cf9f24ee17bc496

loader: Fix implicit layer override behavior

Change-Id: Icb0ce9070b6de1102c319f267fb3eb4a82b2760e

loader: Change override mechanics

Previously, the override's layer list was treated as an authoritative
list of layers. Now it uses a whilelist/blacklist so the override can
enabled, disable, or leave a layer alone.

Change-Id: I58db219deb2b6355c56aeb2d00187ac79ad531c2

loader: Destroy debug callbacks/messengers properly

When creating debug callbacks or messengers, each of them is added to a
linked list from the terminator, with the handle provided by the ICD.

When enabling some validation layers, this handle is renamed as part of
the chain and the application gets a different one.

In the removal process, the application usually receives one of these
function pointers using vkGetInstanceProcAddr:

  - debug_utils_DestroyDebugReportCallbackEXT
  - debug_utils_DestroyDebugUtilsMessengerEXT

Those trigger the chain calls but then try to remove the callback or
messenger from the linked list using the application-level handle, which
is not found because it doesn't match the one from the ICD.

The linked list should be accessed from the destruction terminators,
when the handle has already been unwrapped by the layers.

deps: make update_deps.py the same across all repos

Matching update_deps.py from Khronos/Vulkan-Tools, Khronos/Vulkan-Loader,
Khronos/Vulkan-ValidationLayers, LunarG/VulkanSamples, and
LunarG/VulanTools.

docs: Update CONTRIBUTING.md for CI process

Change-Id: I9ea07d41a917bcd91051514d2ad98a9f0d1908fe

docs: Update BUILD.md for CMake versions

deps: allow a --generator switch to update_deps.py

This allows you to use any CMake generator available
on your system to configure the dependent repositories,
allowing you to match the same generator being used by
the project build.

update_deps.py:
- Take a parameter to specify a CMake generator, and pass it to CMake
  if present
- Allow the user to reduce the count of parallel make jobs that can
  happen at one time
- remove an unnecessary semicolon

build: Correct Metal platform name

In `common_codegen.py`, the Metal platform was defined without the
"_EXT" suffix

Change-Id: Ied604c46b55c669567517f5b2b7613d8c7f8ae64

build: Use Vulkan::Headers target to determine include paths

Change-Id: I73b9b2bfd5e33a9da861937510029cb71f3b2648

build: Update known-good for 1.1.102 header

Additionally, updated copyright dates and added Metal to the platform
list

Updated:
- `scripts/common_codegen.py`
- `scripts/known_good.json`

Change-Id: I54718d18bff98f24b79d6d7bdf075698f7689885

loader: Update `loader.rc` copyright dates

Change-Id: If3c51f4d3177ad6745bfcbcf12e37860e306ece5

docs: Add copyright internal to file as comment

Change-Id: Icbe3be658721c6a32bc3867b2dcc065e7ac94e00

build: Update known-good for 1.1.101 header

Change-Id: Ia8d21df3024394a98cd7be378517b1900d77333c

docs: Add Vulkan Logo

Adding Vulkan Logo for consistency across Vulkan SDK documentation.
Adding creative commons license (because there was no license on the documentation)

build: Update known-good for 1.1.100 header

Change-Id: I15103bacaf978d77b0657ed3c361b72ea50452e0

build: Revert known-good to 1.1.97 header

Reverting known-good file to 1.1.97 due to a spec bug introduced in the
1.1.99 header

Change-Id: Idaebd56d69ecd8736fc603a3bc4e4ce3e9b7a7d8

repo: Bump known good to 1.1.99

Change-Id: I8530c6346abece739536c323cfb1c2f6e2461dc6

scripts: Merge upstream changes to genvk.py

loader: Fix memory not being zeroed after allocate

Change-Id: Iee2abd14ca90c2a3d3725f9c737d70b596dfa122

repo: Remove homebrew python installation in CI

The new Travis CI image seems to come with python 3 already installed.
As a result, trying it install it causes a homebrew error.

Change-Id: If627249e3d7ddc8b35df6d31c906cec65b931fec

Update LICENSE and remove COPYRIGHT

After some discussion, we realized that the COPYRIGHT file was
falling out of date.  Considering that each component that isn't
Apache 2.0 already has it's own LICENSE file, we decided to simplify
our tree.

Change-Id: Ia14dc8f6232bb1caf2d43efc5101be255d5e24eb

repo: Bump known good to 1.1.97

Change-Id: I77de536aab5d98fafefa87aea552824688bab4c9

layers: make win-build happy

fmt: use formatting as suggested by CI

tests: enhance layer tests to be able to handle VkCreateDevice

tests: enhance wrapped objects test

loader: Add comment to duplicate logic

Change-Id: Ib48dd217d18fb05963ae9194d9bc5057eeabee58

loader: Remove duplicate scanned json files

Change-Id: Ia0f222e5f7d8ac5448f341916aaa80380a2af6af

tests: Flock the implicit layer test

Fix #121

Change-Id: I2e060a3ddda3bf919d6be8a3e0af7169ac0fee63

repo: Bump known good to 1.1.96

Change-Id: I2350da2fe487dc3c2031deec9cf3df57cc988136

docs: Add explanation of version tagging scheme

Change-Id: Ie33ffe2d58f737a98b28a51cb90026bfdbcbd88a

docs: Fix grammar and consistency in loader doc

LoaderAndLayerInterface.md:
- entry-point => entry point
- MacOS => macOS (except in API function names)
- on the applications surfaces => on the application's surfaces
- Because layers are optionally, => Because layers are optional,
- a child of a one of the above => a child of one of the above
- action / operation => action or operation
- fixed indentation of the dispatch table description to what I think was intended
- the application and/or system can specify => the application and the system can each specify
- linking to the .dll and .so versions of the loader => linking to the dynamic versions of the loader
  (because .dylib is also a dynamic link)
- platform specific => platform-specific
- dlsym() => `dlsym` (to match usage elsewhere)
- setup => set up (when used as a verb)
- don't break link references across lines
  e.g. [XX XX XX XX](YY) should never be broken in the YY clause
- 3 enabled layers => three enabled layers (spell numbers up to 10)
- backwards compatibility => backward compatibility
- Windows paths use "\" instead of "/"
- libXXX.YYY => `libXXX.YYY` (to match usage elsewhere)
- Layers => layers (to match usage elsewhere, where "Layers" isn't a capitalized term)
- bottommost => bottom-most (for consistency elsewhere)
- the VK_LAYER_XXX ... => the VK_LAYER_XXX layer (for consistency elsewhere)
- either end all list or table items with periods, or none of them
- Khronos approved => Khronos-approved
- hand-shake => handshake
- logical device (This depends => logical device (this depends
- other-hand => other hand
- filtering out of => filtering of
- JSON formatted => JSON-formatted
- /path/to/something => `/path/to/something` (to match usage elsewhere)
- system wide registries => system registries
- fixed some "*" entries (which generally must be escaed, "\*", in markdown)
- environment variable enabled layer => environment-variable-enabled layer
- The implementation of each intercept functions is => The implementation fo each intercept function is
- corrected some "it's" to "its", and vice versa
- the extension suffix of the file ends with ".json" => the extension suffix of the file is ".json"
- added commas to fix illegal JSON file content
- semi-colon separated => semi-colon-separated
- other minor grammar and consistency fixes

CMake: Unset linker flags before testing ASM compilation

Otherwise compilation would fail (at least on Linux with GCC) if the
'LDFLAGS' environment variable, or CMAKE_EXE_LINKER_FLAGS, are set to
any value that isn't supported by 'ld' for ASM, which means most C/C++
linker flags.

build: Update Travis-CI to Ubuntu 16.04

repo: Bump known good to 1.1.95

Change-Id: Ie498d9e2811bc2b9f4af15be96bd0df1f6cc9a06

tests: Modify `vk_layer_logging.h` for hdr 1.1.95

The `VkDebugUtilsObjectNameInfoEXT` member of the
`VkDebugUtilsMessengerCallbackDataEXT` struct was turned const in Vulkan
header 1.1.95 and modifications were required for compilation

Change-Id: I0e52d821239a2b74c8880864a9522359f2285d72

docs: Use "ICD" instead of "layer" in ICD info

Although an ICD is a layer too, in this context, using "ICD" is better
than using "layer".

Change-Id: Ic9619c0a99412c34aaba40cf4c1cc2187a00731b
Signed-off-by: Wang YanQing <udknight@gmail.com>

repo: Bump known good to 1.1.94

Change-Id: Ib3b17bb464e8aea4907ccff1f96e3e890884133b

loader: Fix unwrapped loader instance being used

Change-Id: I64cbfa42e2e2ac83a6977acd59e6fff6bc6f73ef

loader: Downgrade layer json warning to info

The loader used to warn when using the vkGet*ProcAddr fields
in a layer json file from 1.1.0 on. However, it was desirable for
applications to still use this field to allow support from older
loaders. As such, this is no longer a warning.

Change-Id: I401febffe0a15081772dd4a791f5fffbd98d6ac9

repo: Bump known good to 1.1.92

Change-Id: I850fe7799cd13405c3505c164758292a07e2c6c0

loader: Fix override expiration functionality

Change-Id: Ib2415cb1a9218930dc68827abdd1587bd0672342

scripts: Fix windows paths in helper.cmake

Normalize (all slashes going the same direction) and escape ("\" to "\\")
paths when writing helper.cmake.  CMake was previously interpreting the
single '\' characters and whatever followed as invalid escape sequences.
Linux and macOS are unaffected.

repo: Fix MacOS travis build

Travis changed its images to no longer have oclint installed by
default. This means we need to stop manually removing it.

Change-Id: I75790e7fb25815f509cbeb5ee6d5e1902cbc8a98

loader: Fix double free when enabling override

Change-Id: I92f0012528bf1fb4cd9acf3053a4850665b225e8

loader: Fix mac bundle logic

Change-Id: Id8eade275293d4812ecf3fb6ee8e6e1f4ed51885

scripts: Remove Mir exceptions from codegen

Change-Id: Ief43a6294c4cd1a9397d6c6630bb0319c62f387c

repo: Bump known good to 1.1.91

Change-Id: Icae25bc3fbd2408c19efa0a647154380ffc4b17b

loader: Fix VK_EXT_debug_utils termination

Some of the commands aren't needed in the loader, but this would
cause command chains to crash when they hit the NULL.
Also, the ICDs expect the commands to come through vkGetInstantProcAddr
not vkGetDeviceProcAddr.  So, we need to redirect them when generating
the ICD dispatch table because the device commands in the instance
extension exist in the device dispatch table but need to call into
the ICD using the instance queries.

Change-Id: I24cd0ea4f3e5058b1137881452df75bf1ff7487a

loader: Fix python3 warning

Python 3.8 will remove time.clock() and the warnings suggest that
time.process_time() or time.perf_counter() should be used instead.
So, move the script to using time.process_time().

Change-Id: I982da1560adec4aab49f42936334e520456647c1

scripts: Compare to `None` using `is` operator

This is a trivial change that replaces `==` operator with `is` operator, following PEP 8 guideline:

> Comparisons to singletons like None should always be done with is or is not, never the equality operators.

https://legacy.python.org/dev/peps/pep-0008/#programming-recommendations

Change-Id: I4f9f6c921e4158365d4e41965bfcd43b7a3c07e0

repo: Bump known good to 1.1.90

Change-Id: If494bd4abe7817d358eaa3966dd82e1c582cbd85

loader: Fix unknown type

Set var to VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT instead of
VK_OBJECT_TYPE_UNKNOWN.

build: fix ninja on windows

Use target_compile_options to avoid setting invalid compile options on
asm targets.

Change-Id: I5d95cfd77a2d515f41c7b8c2c9193f3aff906be7

Revert "build: Fix ninja on windows build"

This reverts commit 41e6a818ff70370cf4d190d9141ee3caecf01806.

Change-Id: I48f98533126354a8c310aca7dfb321c560873955

scripts: Extend scope of LaTeX handling

The 1.1.90 header introduced the use of "\textrm" in `vk.xml`, requiring
changes to `scripts/helper_file_generator.py`

Change-Id: I0ee5012611d95cfdd1715a4e870a3b2b4e7556ee

loader: Fix multiple SubmitDebugUtils callbacks

If an ICD also supported the VK_EXT_debug_utils extension, then
any call to vkSubmitDebutUtilsMessageEXT would get duplicated.

Change-Id: Ica3224ee598a99a925ec9343b6618d4d8ba190d0

loader: Fix vkSubmitDebugUtilsMessageEXT

If the object count was 0, then no message was reported by the loader.

Change-Id: Ia809d193fb117e85742af0db9856f896be0b2ea3

repo: Remove MIR specific code

Note that references in scripts/common_codegen.py and
scripts/loader_extension_generator.py will need to be
removed later

Change-Id: I7b17c80f7a06a339d7df0c199ff556212a7c6534

repo: Fix Travis CI job for MacOS

A change in the MacOS images caused a conflict when upgrading python3.
This change removes oclint first, since that package is the source of
the conflict

Change-Id: I990a7b0eb1f91f51fcc2ad2a0db120af49c7a754

loader: Handle vkGetDeviceQueue2 correctly

Change-Id: I15f7677d5c6d8b4f0ff62bb7d371d173d58ae5f4

repo: Update known good to 1.1.89

Change-Id: I31a45bd5e7ce076afab6e035a3fc2803be00b90f

repo: Disable gcc 8 stringop warnings

GCC 8 introduced some new "stringop" wanrings. The loader deliberately
does things that trigger these warnings. As such, they should be
disabled

Change-Id: I08f640fae2930a06cbdd48095f11febfc9e866ed

loader: Resolve incorrect error in create instance

Change-Id: Ib3f99bb150ddf9aaa36cbe08919d737f2636baf6

loader: Override layer support and settings

Add support for an override layer in the loader. This allows layers
to be set by an outside application.

Revamp the logic to detect manifest files in the various paths that
we allow them to be placed. This code significantly rewrites
loader_get_manifest_files.

Change-Id: I8abf558864b66eb71ee026ca559b0126cf2fa4e9

build: Fix ninja on windows build

Change-Id: Ie15d8a47636d228732950fb5b3462a2819d5b2af

repo: Bump known good to 1.1.87

Change-Id: If238e0fe6462cf90952d4c0132b49b46bbe96a4e

Create CODE_OF_CONDUCT.md

tests: Use specific version of gtest for CI

scripts: Add fuchsia to platform dictionary

Change-Id: I229825efa86109e3e24de722fac52179963d3e70

Revert LICENSE to Apache 2.0

Update LICENSE.txt

Change Apache 2.0 to Khronos Apache 2.0 CLA

build: Sync update_deps.py

Syncing changes made to update_deps.py
in Vulkan-ValidationLayers.

Change-Id: If617596654b982023ee883fe131390c760c53532

repo: Update known good to 1.1.85 headers

Change-Id: If1383facdfcd39b9ac07164574bb6264dcc82100

build: CMakeLists.txt cleanup, part 1

This is a first pass reorganization of CMake files in this repo.
It consists primarily of dead code/variable removal, simplification,
and reformatting by latest cmake-format.

bump to cmake_minimum_required(3.4) for ccache

simplify excessive use of generator expressions

clarify why CMAKE_OSX_DEPLOYMENT_TARGET is pre-project().

clarify setting Windows install prefix if needed

remove unused DisplayServer variable

remove obsolete argument to endif()

remove unneeded ccache property RULE_LAUNCH_LINK

ccache doesn't affect linking, so the property is meaningless.

remove unneeded CMAKE_VERBOSE_MAKEFILE hardcode.

remove extra CMAKE_MODULE_PATH

remove extra cmake_minimum_required

change PYTHON_CMD to PYTHON_EXECUTABLE

change CMAKE_SYSTEM_NAME to UNIX/APPLE/WIN32
    Use consistent platform identification.
    Note that UNIX evaluates true for OSX, so whenever the code intends
    "Linux only", we use (UNIX AND NOT APPLE).

remove unneeded TARGET_NAMES variable

remove unneeded VK_LAYER_RPATH variable

change 'ln -sf' to 'cmake -E create_symlink'

reorder dependencies for clarity

rearrange order of macro arguments

rename run_vk_xml_generate to GenerateFromVkXml
rename add_vk_layer to AddVkLayer
    Borrowing from the Google style guide, use camel-case
    to more easily identify locally-defined macros/functions.

remove unneeded WRAP_SRCS variable

remove unneeded TEST_SRCS variable

move check of VulkanRegistry_FOUND

move SCRIPTS_DIR closer to point of use

move include(FindPkgConfig) earlier

move CMAKE_MACOSX_RPATH earlier

move option() eariler

move GenerateFromVkXml invocations earlier

Move invocations to be closer to the definition of the macro.
Minor edits to comments.

move API_NAME earlier

remove unused BUILDTGT_DIR variable

move add_executable() earlier

combine if(WIN32) trees

.cmake-format.py 0.4.1

reformat using cmake-format 0.4.1

Change-Id: Ia251e932dbc9a8f2655201bb12248dea8c940c30

repo: Update known-good to 1.1.84

Change-Id: I73b1337dad35d7ecf28c6acfc61c37832cc4497b

loader: Fix unused variable compiler warning

Change-Id: I331c532e55034f004b55e2738214c0301a08375e

loader: Fix overzealous duplicate manifest warning

This downgrades the warning when a manifest file is given multiple
times to a debug message (previously it was a warning). This is
because any Windows system that has multiple GPUs that use the same
driver will encounter duplicates as correct behavior.

Change-Id: Id64f4c34d58c4b806164b3e7cd7a07dab36ab88f

repo: Bump known good to 1.1.83

Change-Id: Ie85da5d3a20b8bad4f4c4a5f7429c0fb7463a7b2

loader: Add missing entry point to gpa helper

Resolves an issue where vkGetBufferMemoryRequirements2 was not present
in gpa helper, which caused problems when calling GetProcAddr

Change-Id: I40933ec5adba54b09b88200d46b307150bb52d9d

build: Correct Windows architecture options

build: Unify `update_deps.py` between repos

docs: Fix path typo in interface doc

loader: Remove layer negotiation from device chain

Previously, when the loader created a device chain it would attempt
to negotiate the layer interface version. Now, it just saves the
version when creating the instance chain.

Change-Id: I8a48914474e5600e023d837b01198c23da9c8464

loader: Save gipa and gpda ptrs in instance chain

Change-Id: Iaf7d0084cbeb41e7542787a5b013a918a2431b82

loader: Fix VK_EXT_debug_utils crash

Fix a crash where the debug utils extension would crash if no layers
were present that implemented a given entry point.

Change-Id: Ied1da23dfdc952893a5aaad4ae01f69bd6010523

repo: Bump known good to 1.1.82 headers

Change-Id: I7ac0d6f31bd1c09d37a6436c022769bd134fea58

Fix build with CFLAGS="-std=c11 -D_GNU_SOURCE"

repo: Bump known good to 1.1.81 headers

Change-Id: I1da4208c4a57914e3795eef5907128d350029b1a

docs: Remove vk_layer_dispatch_table.h from BUILD.md

Change-Id: I34fded0bcc970fd09640993ed4dfd67fbc29dcbe