build: Update to header 1.3.226

- Update known-good
- Generate source

loader: Add stub function for OpenBSD executable path

Dont pass portability bit to ICDs that dont expect it

There was a small issue with the initial version of the portability
enumeration extension where the portability enumeration flag bit would
be passed down to ICDs which did not expect flags to contain anything
other than zero.

While an argument could be made for those drivers to ignore flags they
do not recognize, just like extensions and other 'unknown' things, it
is best to play nice as this is the first instance creation flag bit
added.

loader: Add OpenBSD support

build: Update to header 1.3.225

- Update known-good
- Generate source

Separate Major.minor version checks from Full version checks

loader_make_Version previously would decode the major, minor, and patch info out of version data. This results in
erroneous version checks that took into account patch version when they shouldn't of. The loader_make_full_version
function is introduced for code that wishes to get the full major.minor.patch version and uses of
loader_make_version that need the full version have been replaced.

build: Update to header 1.3.224

- Update known-good
- Generate source

Use memmove for copying pApplicationInfo

Replace memcpy with memmove in terminator_CreateInstance to prevent possible issues
with the same memory being the source of the copy and the dest, as there is a for
loop which makes it possible for that to happen.

build: Update to header 1.3.223

- Update known-good
- Generate source

build: Update to header 1.3.222

- Update known-good
- Generate source

Fix typo in setting linux environment variables

Replace `;` with `:` in setting linux environment variables.

build: Update to header 1.3.221

- Update known-good
- Generate source

loader: Dynamically load Win8+ functions

Allow loader to run on Windows 7 by dynamically loading function
pointers for Windows 8 and above APIs.

Disable VulkanOn12/Dozen in Windows 7

Chromium still supports building for Win7, as such the newly added support for
VulkanOn12/Dozen needs to be excluded from said build. This is achieved by using
the SDK version macros defined in <sdkddkver>

Fix corrupted pNext chain in vkCreateDevice

When creating a device, the loader looks for the VkDeviceGroupCreateInfo
structure and replaces it with its own. This allows the loader to edit the
struct. However, to do this required editing the pNext chain. Because the
edited chain contained pointers to structures whose lifetimes end when the
vkCreateDevice function returns, the pNext chain is now corrupted.

This commit fixes that by storing a pointer to the user's
VkDeviceGroupCreateInfo and fixing up the pNext chain to use that instead.

Initialize properly when statically linked

Previously, the loader supported static linking. This capability was restricted
to MacOS only, however the necessary functionality was never implemented. This
commit adds the required code to properly initialize the loader when statically
linked with MacOS

Since the test framework was designed to dynamically load everything, it would
require significant rearchitecting to support it. As such, a simple verification
executable was added to the live_verification folder, instead of full support
in the test framework.

Refactor vk_loader_platform.h

Move includes of vulkan headers into loader_common.h, delete dead code
and unnecessary macro defines, and generally clean up the header.

build: Update to header 1.3.220

- Update known-good
- Generate source

Enable the Vulkan loader to load VulkanOn12/Dozen out of the D3DMappingLayers app package

Make tests get files in a consistent order

Use the order defined the FolderManager's to define the order readdir
uses, rather than leaving it undetermined. This makes tests more consistent
by forcing layers and drivers to always be found in the same order on all
systems.

Note: MacOS doesn't currently have consistent ordering due to a crash with
ASAN. But even if ASAN isn't running, the dirent structure is being filled
out incorrectly, causing further test failures. While not ideal to disable
consistent ordering on MacOS, it is needed for linux and thus the issue
isn't high enough priority to resolve.

build: Update to header 1.3.219

- Update known-good
- Generate source

Fix BUILD.gn

Add newly added file stack_allocation.h

Fixes #846

loader: device array dealloc fix

loader: Add unicode support

Add 32 & 64 bit field to json manifests

Allows drivers and layers to specify if they are 32 bit or 64 bit in the
manifest file. This makes the loader able to prune manifests without
loading the library and finding that it failed to load.

Fix use-after-free in loader_add_layer_properties

Occurs after file_vers is freed and when the layer manifest has a layers field
but has a version of 1.0.0.

only descend into layer GPDPA from trampoline

Prevent loader config from overriding project config

The loader options dummy targets were being linked into the public
interface of the loader, meaning that they were also set in projects the
include the loader as a sub-project. These override the containing
projects settings and can break things. There don't seem to be any
interface requirements in these settings, so include them as PRIVATE, so
that they stay local to the loader.

build: Update to header 1.3.218

- Update known-good
- Generate source
- Add missing enums to test framework

tests: Shuffle devices for sort test

Shuffle the bus IDs for discrete devices pd0, pd3, and pd4. Notably this
puts pd0 from icd0 in between pd4 and pd3 from icd2, making default
sorted order extremely unlikely if not impossible.

build: Update to header 1.3.217

- Update known-good
- Generate source

Only check first GPDPA in the layer chain.

When checking for unknown physical device functions, check the first layer that supports
vk_layerGetPhysicalDeviceProcAddr in the chain starting with the layer closest to the
application. This prevents unecessary work being done, and if any layer wraps VkInstance
will properly call through the wrapping layer first without calling into any other layer.

build: Update to header 1.3.216

- Update known-good
- Generate source

Revert "fix #948 Only calls the first GPDPA in layer chain"

This reverts commit a8c17d6a367417509aa7b11cf65083eac2b2f4c9.

Revert "Comment tweak to reflect new behaviour."

This reverts commit d86dc61e76f66b32b6955e8e7cbfa3edc7ed7b32.

Comment tweak to reflect new behaviour.

fix #948 Only calls the first GPDPA in layer chain

Update layer and driver vkGetPhysDevProcAddr docs

Better describe the function's purpose and behavior, moving the
reason for introducing vk_layer|icdGetPhysicalDeviceProcAddr to
a following section for clarity.

fix typo

resulsts -> results

Fix crashes from OOM in vkDestroyInstance

Various situations could cause an OOM to turn into a hard crash due to double freeing
of memory that was improperly cleaned up. Also fixed memory leaks when layers would
report OOM.

Keep allocated debug callbacks until destroy instance

Previously the loader would destroy any debug callbacks created during instance creation
to later create new ones during instance destruction. This required a memory allocation
to occur inside vkDestroyInstance, which can cause leaks if an OOM occurs during
instance destruction.

This commit simplifies the logic by keeping around the allocations made during instance
creation by moving them into their own debug node chain. Then during instance destruction
moves them back.

Also renames several functions to better describe their intended purpose.

Use calloc to allocate memory for the search path

This prevents bugs where uninitialized memory is treated as a real path. It
should fix an issue where sometimes CI would fail due to what appears to be a
spurious path with junk characters.

Fix typo in env var table

Introduced a typo in my last commit in the table regarding the
disabling of instance extensions.  This is not a Linux only env
var.

Clean up environment var docs

Split up into two tables:
  - Active
  - Deprecated

Clean up restrictions into their own column for easier viewing.
Made text slightly smaller to fit more in a smaller space.

Clarify JSON Manifest "api_version"

Clarify what is indicated by the "api_version" field in both
layer and driver manifest files.

Fixes issue #336.

Fix a NULL inst crashing during loader_icd_scan

When checking for the portability driver field of driver manifests,
the loader did not check if inst was NULL first. Since this function
is called in pre-instance functions, this crashes the loader.

Use calloc instead of alloc+memset

Replace naked uses of malloc & free with loader_alloc & loader_free.

Use VkAllocationCallback in cJSON

Make cJSON pass the VkAllocationCallbacks instead of the loader_instance.
This removes and unecessary dependency between cJSON and the loader headers.
It also somewhat simplifies the interface by not requiring the inst parameter
everywhere, just in the creation of cJSON pointers and freeing.

Use VkAllocationCallbacks in windows dirent

Pass the allocation callbacks directly to dirent_on_windows.
This removes a unecessary dependency between the loader headers and
dirent_on_windows.

Refactor loader allocation functionality

Created wrapper functions loader_alloc, loader_calloc, loader_free,
and loader_realloc. Made the existing loader allocation functions use
them. Replaced manual usage of VkAllocatorCallbacks with the new
wrapper functions.

build: Update to header 1.3.215

- Update known-good
- Generate source

docs: Fix typo in MoltenVK url

Update supported versions in the docs

Mention the supported versions that were missing.

Signed-off-by: Ludovico de Nittis <ludovico.denittis@collabora.com>

build: Update to header 1.3.214

- Update known-good
- Generate source

build: Update to header 1.3.213

- Update known-good
- Generate source

Revert "Implement unknown function intercept in layers"

This reverts commit 0fd2ff83845abb924db9426b1741882e5ca064bf.

Output message if override app_keys mismatch on app name

Output a layer message if the override layer is present and the
app name is not in the app_keys list.
Fixes issue #917.

Update layer manifest file

Some fields specific to manifest files were missing.  Also updated
file to clearly indicate where fields are valid and usable.

Make portability drivers not load by default

Unless the portability enumeration extension is enabled and the create instance flags
contain VK_INSTANCE_CREATE_ENUMERATE_PORTABILITY_BIT_KHR, do not enumerate drivers that
contain `is_portability_driver` in their JSON Manifest. This is phase 2 of the release
for the VK_KHR_portability_enumeration extension.

An error message will be printed when no drivers were reported but there was a
portability driver which was skipped over.

Fix CMake code for getting Git Version information

The ${CMAKE_SOURCE_DIR} uses the wrong directory when the loader is included as a
subproject. By using ${CMAKE_CURRENT_LIST_DIR}, the CMake code to find the git
commit and branch name will use the correct directory.

Additionally, the logic was ammended to curtail this issue in the future. Quotes are
included in the string to allow it to be used as a string literal, instead of
needing a const char[] variable and macros. Also, the CMake code was restructured
to always define GIT_BRANCH_NAME and GIT_TAG_INFO.

Implement unknown function intercept in layers

Make it possible for layers to declare that they support unknown functions
and set up the layer dispatch table appropriately. A very niche use case
but helpful when developing validation for functions not yet out in the
public headers.

Correctly check for NULL in loader_get_dispatch

Fix typo

ci: Remove fbactions/setup-winsdk

The default version 18362 started 404'ing and a new enough version is
present in the base image.

build: Update to header 1.3.212

- Update known-good
- Generate source

Add app_key OverrideMetaLayer tests

Update LoaderDriverInterface.md

Fix link-syntax.

Refactor FrameworkEnvironment API a bit

Redo some of API's for adding layers and ICD's to the test framework.

Document VK_KHR_portability_enumeration support

Warn if portability_enumeration extension should be enabled

Log an error if an application creates a VkDevice from a physical device which
was enumerated from a driver that is a portability driver but the application
didn't correctly enable the portability enumeration flag & extension.

Increase discovery timeout for Gtest test to 100 seconds

Revert 32bit linux unknown ext chain change

Fix breaking of builds due to the dropping of GOT addressing mode.

build: Update to header 1.3.211

- Update known-good
- Generate source

Allow meta layers to have newer component layers

Relax the requirement that all component layers in a meta layer must equal the version of the meta layer.
This allows enabling layers that do not have the same API version as the meta layer.

Clean up version checking logic

Add a few helper functions to simplify checking that a X.Y.Z version is sufficient.

Fix 32 bit linux unknown function handling

Building and running the regression tests reveals that the 32 bit unknown function
handling on linux is broken. It required the same fixes as did windows 32 bit, since
the code is almost identical.

wsi.c also would not compile in 32 bit linux due to the warnings-as-errors being enabled,
which has also been fixed.

Fix missed Physical Devices in Windows

The previous logic would cause any non-sorted physical devices to be missed
when aggregating the final list. This caused crashes due to the phys_dev
value not being initialized. The fix is to make sure both sorted and non
sorted physical devices are included in the final output of
setup_loader_term_phys_devs().

Use correct TestICD in tests

Many tests were using the V6 TestICD, which means that it exports the
EnumerateAdapterPhysicalDevices. The loader then only looks for devices
using that functionality and if the test didn't set it up, the driver
won't have its devices found.

Also:
* Cleaned up handle_validation_tests
* Added WSI setup helpers, which put all the setup code for WSI in one place
* Created a to/from_nondispatch_handle function for TestICD, probably should
be in a more general location

Add test for different ICD interface versions

Fix Test Framework not skipping Drivers which have a different LUID.

build: Update to header 1.3.210

- Update known-good
- Generate source

Dont emit warnings on unknown manifest file versions

Since the loader is meant to be forward compatible, it makes little sense to emit
a warning when an ICD or Layer manifest is found with a version that it doesn't
understand. Thus they now are INFO level.

cmake: Add ENABLE_WERROR option

This adds a choice for the user whether to treat warnings as errors.
The remaining compiler options are moved where it was before.

Fixup several small issues found by Visual Studio

The VS Analyzer went through the codebase and pointed out a dozen warnings and
one possible crash.

Enable /W4 errors and support clang-cl

Enable /W4 errors on MSVC and fix all corresponding errors.

Refactor the CMakeLists.txt to allow for compiling with clang-cl. This required
figuring out that -Wall -Wextra mangle the set of warnings used. The solution
is to use /W4 instead when compiling with clang-cl on Windows.

Replace TEST_F with TEST in most tests

Test fixturing (TEST_F) is useful when there is a lot of common code that needs
to be run for a suite of tests. However, the FrameworkEnvironment encapsulates
almost all of the setup work all tests needs, and so doesn't need to be in a
fixture. Making the add_icd call inside the tests makes it clear which binary
is being used.

This commit also uses the corrent signed/unsigned constants in tests, as was
warned by compilers when enhanced warnings were enabled (/W4 in msvc).

Update issue templates

Expand Bug report and add Feature Request template.

Add Bug Reporting Template

Add initial bug reporting template with requested fields.

Apply code review comments from @charles-lunarg

Fix #888 crash in Lutris with Linux sorting

The sorting algorithm needed to take into account both the application
API version as well as the driver API version.

This required additional changes to the sorting algorithm for the fallback
since even if the instance supports the extension or Vulkan 1.1, the individual
drivers may not.

Also, add supporting tests which would catch these cases in the future.
In the process, I realized we assumed that the presence of an extension in the
test_icd indicated "enablement" which was incorrect.  So I separated out
that into a set of "enabled instance extensions"

Remove LDP_DRIVER_6

The cause for this policy was due to dynamic linker behavior which could have
resulted in the wrong symbols being used. However, this didn't occur in
practical usage of the dynamic linker. Also drivers exporting Vulkan
entrypoints was a useful behavior to enable testing and more flexible usage
that this policy actively interfered with.

Fixup windows test infrastructure

Cleaned up several bugs in the testing framework in the windows shim.
Rewrote the tests that exercise DXGI/D3DKMT functionality.

Fix non-sorted surface indexing

The physical device terminator was missing the ICD index in the
non-sorted path.  This caused crashes in Angle before it was realized
that the sorting code was unintentionally disabled in that build
path.

Also, add tests to catch this case in the future in the WSI code, but
this required converting all the TEST_F tests to TEST since Gtest
didn't like mixing the 2 on my system.

Finally, fix a few WSI error messages in the loader which were
missing spaces.

Fixes #863 for non-sorting paths

Update BUILD.gn

Add  LOADER_ENABLE_LINUX_SORT to workaround ANGLE crashes on Linux

build: Update to header 1.3.209

- Update known-good
- Generate source

Log when core GetToolingProperties is NULL.

The core entrypoint for vkGetPhysicalDeviceToolingProperties didn't log if the function in the
terminator was NULL. This makes it easier for implementations to know when they aren't
exporting the symbol.

More clean up to make sure we have enough space

Modify logging text output

Modify text output for logs so that we only flag types of messages
we're interested in and also try to add spacing so items line up
cleaner.

Add device log info on vkCreateDevice

To make things clearer on output, log the device information for
the physical device being used on vkCreateDevice if either the
layer or the driver log bits are set.

build: Update to header 1.3.208

- Update known-good
- Generate source

Tests close layer handles when necessary

The fs::FolderManager can't close objects that are in use, thus if the tests
keep open a binary then the destructor will fail to delete the file. This
commit manually cleans out the FrameworkEnvironment's loaded layers to prevent
this from happening.