Re-arrange/update loader docs

Update the loader documentation in the following ways:
 1. Move documentation to new "docs" folder"
 2. Split LoaderAndLayerInterface.md into multiple docs to focus
    on target audience:
      - Top-Level     LoaderInterfaceArchitecture.md
      - Applications  LoaderApplicationInterface.md
      - Layers        LoaderLayerInterface.md
      - ICDs          LoaderImplementationInterface.md
 3. Upload newer images and their corresponding original Inkscape files.
 4. Cleanup and update sections on Linux directory search
 5. Add new sections to detail items missing
      - VkConfig
      - Handling undef Fuchsia
 6. Language cleanup

Thanks to @charles-lunarg and @smcv for feedback and various section language
corrections.

Only return pre-instance functions when instance is NULL

vkGetInstanceProcAddr should only return the pre-instance, aka global
functions, when the VkInstance handle is NULL. This brings the desktop
vulkan loader into alignment on the spec and with the android loader.
https://github.com/KhronosGroup/Vulkan-Docs/issues/1605

The repo previously did follow this rule, but was changes in the
1.0.42 header version timeframe for unknown reasons.

build: Update to header 1.2.196

- Update known-good
- Generate source

Update README.md to remove mention of Vk-Headers

test: Update WrapObjects test to new framework

Moved over the WrapObjects test logic into the new framework, removing all
the old tests infrastructure that is no longer needed.

Zero-init values passed into EnumPhysDevs

Only parse git information if .git/HEAD exists

If the source comes from somewhere else (eg. the release tarballs) then
.git/HEAD does not exist, so the cmake step was failing. Only try to
colelct the git information if the file exists.

Fixes #714.

build: Update to header 1.2.195

- Update known-good
- Generate source

test: remove stub functions

Test code should not use stub functions because its undefined behavior. This
resulted in SEH exceptions on Win32.

test: Use correct name in def files

test: Use correct path in json files

Previously a Windows path in a json file did not include the correct
backslahses. This commit rectifies that.

test: Update ExtraLoaderTests

Updated the enable/disable env-var tests to use the test framework.
Removed the run_extra_loader_tests.sh.

test: Move more old tests to new framework

More tests can now be written with the new framework. Several of the
shell scripts used tests in vk_loader_validation_tests which were phased
out so no longer necessary.

ci: Enable tests on MacOS GithubActions

Tests only get built if you enable them now, instead of when
googletest is in the external folder.

loader: Remove old debug path printing

This commit only prints paths using the new, more structured, output.
It wasn't removed until now due to the paths being used in tests which
have since been modernized, thus no longer needing to keep the old output
style.

test: Add XDG & VK_LAYER_PATH tests

Tests check to see if the paths are searched by looking at the log.
Makde FrameworkEnvironment contain a debug_log, this is to prevent any
lifetime issues. This prevents the possibility of creating a DebugUtilsLogger
after the InstWrapper.
Had to redo the test framework a bit to make them work.

test: Fixup XDG path handling, add XDG tests

build: Revert generated loader.rc

Revert the commit that uses the vulkan-headers version and instead uses the
1.0.1111.2222 version number while still allowing for the CMake build arg
BUILD_DLL_VERSIONINFO to dictate the actual version.

loader: Fix setting `libdir` in generated vulkan.pc on Windows

Fixes #668

loader: Only include loader_windows.h on windows

This header should only be included on windows platforms, since non-windows platforms
may not know that the header should be present (which is the case for GN builds).

build: Move test dependencies to known_good.json

This makes the requisite changes to make the dependencies needed for testing automatically
downloaded by cmake when the UPDATE_DEPS option is given.

build: Update update_deps.py to match VVL

Was out of date with the feature set VVL contained.

Fixed an issue where backslashes were used on windows, which caused issues when
cmake decided to escape a string one more time than necessary.
"C:\\Users\\..." -> "C:\Users" which has a `\U` escape symbol mucking things up.

build: Update to header 1.2.194

- Update known-good
- Generate source

build: Add RELEASE_BUILD option for Windows SDK builds

This CMake option controls the information put in the loader.rc file,
specifically the "Dev Build" string in the version info.

Use CMake to define SECURE_GETENV variables

Use `target_compile_definitions` to provide the HAVE_SECURE_GETENV and
HAVE___SECURE_GETENV macro defines. This removes the need to configure
a file and makes the definition global to the loader, rather than just
from whoever included the header file.

build: Set DLL version automatically

The loader.rc file was used to set the version used in the loader. However,
this was always done manually (either by hand or a script to edit the file).
This commit makes the version number needed generated automatically based on
the current generated header version or uses a provided version, such as in
the case of SDK builds with a 'build revision', eg "1.2.191.2".

This commit also renames the atrosciously named
loader_generated_header_version_generator to a nicer
loader_versioning_generator.

build: Add UPDATE_DEPS cmake arg

Add an UPDATE_DEPS cmake arg that can be used to automatically run
update_deps.py as needed.

test: Isolate Env-Vars from test framework

Previously if any of the env-vars the loader looks for were set on the
system, the loader would find and use them, leading to highly confusing
logs. Removing them preemptively at tests start make this a non-issue.

build: Fix using wrong Windows SDK for VS 2015

CMake 3.19 disallowed VS 2015 builds from selecting an SDK higher than
10.0.14393. Unfortunately, the loader requires SDK version 10.0.17763 or
higher. This commit uses the 3.19 capability to disable that limit set in
place by CMake so that we can use an SDK that suits our needs.

build: Update to header 1.2.193

- Update known-good
- Generate source

loader: Add cJSON prefix to update function

build: Update to header 1.2.192

- Update known-good
- Generate source

loader: Add loader.rc to Windows GN build

This omission was causing the build dll to be missing version info.

test: Fix 32 bit test suite build error

loader: Move struct decls to loader_common.h

Not all structs declarations in loader.h were moved to loader_common.h. This
caused compilation errors due to not having the struct declaration available
in the headers it was needed.

test: Add test which calls GetPDProps inside CreateInstance

This test makes sure that any layer calling GetPDProps inside of CreateInstance
succeeds and doesn't crash.

test: Add ICD Interface V6 TestICD binary

test: Add callback to layers create instance and device

This allows some code to be written inside the test which then gets run
during the call to vkCreateInstance and vkCreateDevice.

Update more logging info for the Loader

Output more information to the "info" debug target when the loader loads
so we can determine more info about:
 - Where loader was loaded from (Windows)
 - What Git Branch was used to build it
 - What Git tags were most recent when it was built.
Also, add more detailed logging of implicit layers:
 - Log manifest file location
 - Layer library location
 - Type of layer (implicit/explicit)
 - Disable enum if layer is implicit

loader: Reverst asserting on size 0 in malloc

loader: Move disable_environment into function body

The function `loader_read_layer_json` does not modify the
disable_environment parameter that is passed in. It can safely be made
a variable local to the function.

Change-Id: I25ef3f3e5f008082dc9514be8f607d2acbf212a1

build: Use generated source everywhere

The previous commit that added a generated header version didn't apply
it to mac or linux specific versioning. This commit fixes that.

test: Add test for memory leak in setup_tramp_phys_devs

Tries to trick the loader by increasing the number of physical devices
after the call to vkEnumeratePhysicalDevices to get the count.

loader: Fix freeing of uninitialized list

The memset of icd_tramp_list in terminator_EnumerateInstanceExtensionPropeties
was inside an else-branch. If an error occured in the preceding if branch, this
would cause the cleanup code to run on a non-zerod out icd_tramp_list.

Change-Id: I7af1da31580f01c54c4ffababbb9c7ecfa8966d8

loader: Remove deadstore when breaking from a loop

Change-Id: Ic72cf396cad918ce255b331fa9a9c3a652b8764d

loader: Prevent use-after-free in error path.

When `setup_loader_term_phys_devs` and `setup_loader_term_phys_dev_groups` hit an
error, they attempt to delete all the alocated elements in `new_phys_devs`. However,
the code first may copy elements from another array into it first before hitting
the error. This causes the deletion code to free those 'good' elements which were
copied, leading to use-after-free's.

Additionally, an error message used total_gpu_count instead of total_icd_count, this
is fixed with this commit.

Change-Id: I8f714f699ff09cc99d094cf2ed8190cedcad684b

loader: Don't pass null pointer into strcmp

strcmp wasn't correctly guarded against a null first parameter when
doing the shallow search in loader_name_in_dev_ext_table and
loader_name_in_phys_dev_ext_table.

Change-Id: Ia751e9860334895930210defa2dcb2becbfd6daa

loader: Remove dead store of total_count

`loaderScanForLayers` total_count is not used after determining
if it is not zero.

Change-Id: I2ff3a1a5d7dde42ce5ad7d57aab63831ccf58156

loader: Prevent stack_alloc of 0

`loader_create_device_chain` would previously attempt to use stack allocations
when `expanded_activated_layer_list.count` was zero, which is undefined behavior

Change-Id: I56fab835a1728914191f1a7adc210dae58afa155

loader: Get DeviceGroups early exits if 0 are found

This commit prevents undefined behavior by exiting
`setupLoaderTermPhysDevGroups` if total_count is 0.

Change-Id: Ieef2747724f79ec26a08a74884904bc0e0485c17

loader: Remove redundant stores

`layer_node = layer_node->next;` is redundant in error paths that goto out.

Change-Id: I889b03c825f896f458a1ed3171b951d64d8827d7

loader: assert if trying to malloc 0 bytes

The behavior of malloc is undefined if size is 0. This commit checks for that
and prevents a possible malloc of zero in device extension property enumeration
by only malloc'ing if size is greater than 0.

Change-Id: Id506ab93d404bd4895503c660ef74bcb9c4550a3

loader: Format WSI error messages

Change-Id: I3040be426a8e70ca41e7beede1fae6b52e1381af

loader: Use abort to force a crash where needed

Many places in the loader need to intentionally crash. This commit makes the
loader use `abort();` in those places, including replacing numerous
`assert(false)` statements. Using abort clearly signals intent instead of
having it do a null pointer dereference to cause the crash.

Change-Id: I6874eab5702c94af59b7984fab00f6eaf03f2f45

loader: Fix potential memory leak

Also remove a redundant setting of an int to 0

Change-Id: Ia099ad75e9da14aacf9874d75267716c12794305

loader: Handle OOM in loader_phys_dev_ext_gpa

Removing some dead code revealed a case where a potencial OOM situation
was not being handled.

Change-Id: Ibc5ad607f6adc0f590f663ab928c1abca6732d96

loader: Make trampolineGetProcAddr snake_case

loader: Move simple terminators to terminator.c

Keeps the complex terminators in loader.c for the time being.

loader: Rename SetupLoaderTermPhysDevGroups

Forgot this name when renaming all the functions.

gn: Fix gn build for refactor changes

loader: Move Windows logic to dedicated file

Relocated most windows specific functionality into loader_windows.h/.c
This removes the need for loader.c to include windows specific header files.

loader: Move struct declarations into header

loader: Move init functions closer together

loader: Header include cleanup

Separated headers into common groups, with system headers being first, then
Vulkan API headers, followed by loader headers.

Then alphabetized headers in each group. Due to moving _GNU_SOURCE to being set
by cmake, there isn't any implicit dependency between the include order used.

Made sure headers should be able to be included independently, such as
including the vulkan headers where appropriate.

loader: Remove headers from cmake source list

Headers do not do any functional thing here. Better to remove them so they dont
clutter up or make people thing they need to be there.

loader: Make logging and getenv their own headers

This commit pulls the logging capabilities into a single log.h/.c file and does
the same for the getenv functionality.

loader: Make _GNU_SOURCE a cmake definition

Moves the many instances of #define _GNU_SOURCE into a single cmake call which
sets the macro in all translation units for the vulkan library. This way it
cannot be forgotten and makes behavior across the library consistent.

loader: Move alloca into allocation.h

loader: Remove duplicate macro defines

loader: Make vk_loader_layer.h a proper header

Makes it possible to include this file without worry of not having the required
headers already included.

loader: Remove unneeded parts of loader_common.h

loader: Shift around vk_loader_platform headers

Moved the includes into their own block, instead of living in the block for
each operating system. This way is easier to see what includes are in the file
in its entirety.

Also, _GNU_SOURCE was uncommented since it appears in each source file
individually and is needed for alloca and secure_getenv

loader: Dont include C++ libraries in C++ mode

loader: Move allocation functions to a header

Functions pertainting to allocation (malloc, realloc, free) have been moved to
their own translation unit for better organizational purposes.

loader: Remove TLS

The only purpose TLS was serving was as a channel for allocation callbacks to
be useable inside cJSON and dirent_on_windows. TLS didn't solve any specific
threading problem. Instead it served as an information channel. On top of that,
the only time when the TLS was used was inside vkCreateInstance as the
pre-instance calls would have the TLS pointer to loader_instance be set to
NULL.

loader: cJSON to use regular heap instead of TLS

Refactored the cJSON library to take an loader_instance pointer everywhere it
allocates memory. This way a TLS loader_instance pointer doesn't need to be
set at initialization and then at vkCreateInstance.

loader: Dont use TLS in dirent_on_windows

Loader will now pass in the instance handle directly rather than rely on TLS to
pass it. This makes the loader code simpler to reason about and better since
if a second instance is created on the same thread the TLS wont be overwritten.

loader: Add gpa_helper.c

Make gpa_helper.h not contain function definitions, moving them into the
appropriate header file.

loader: Move loader struct defs into common file

This commit separates the structure definitions from the function declarations.
The reason to do this is to allow splitting of loader.h/.c functionality into
individual headers, which may require access to these structs but don't need to
know about the other functions.

loader: Update Copyright for 2021

loader: Use #pragma once everywhere

The #pragma once include guard is sufficiently widespread enough to allow
usage of it everywhere.

loader: Update Copyright

build: Alphabetize loader source file list

test: Add test for Override Meta Layer

test: Fix extra } in layer manifest printer

loader: Move layer cleanup to a single function

Previously the logic to free all the components of loader_layer_properties was spread
across a half dozen different places, resulting in leaks. This commit moves the logic
into a single location, centralizing the process and making it easier to add new
fields to the loader_layer_properties struct.

build: Update to header 1.2.191

- Update known-good
- Generate source

test: Improved Test Wrapper interface

Made InstanceWrapper and DeviceWrapper simpler to use by moving them to test_environment
then making the Create() member functions and doing the gtest assertions in the function
itself.

Additionally:
Removed the `detail` namespace in the test_environment.
Renamed get_new_test_icd/layer to just `reset_icd/layer`. This is clearer about what it
is doing and makes the interface more expressive with code such as
`env->reset_icd().SetMinInterfaceVersion(5);`

loader: Make all loader functions use snake_case

Before it was a mix of camelCase, PascalCase, and snake_case.

"Vulkan" functions, like terminator_GetPhysicalDeviceProperties, remain in camelCase.
This way it looks visually similar with the actual vulkan functions they correspond to.

Fix loader not knowing about extensions enabled in layers

This change allows the loader to know extensions enabled in the layers
as well as extensions enabled but handled by layers.
This should fix several issues where the enabled extensions is not
known soon enough for layers to use.

Update XDG paths

Issue #245 was created because we were not properly using the XDG
paths.
This change updates the path search order to be more correct.
While an argument could be made that the loader really should only
be looking in the DATA folders, we already are looking in the
CONFIG folders.  Therefore, just correct the order and also make
sure the proper _HOME variable for CONFIG and DATA is searched
first (but only in non-superuser mode).

test: Add handle_assert helper functions

Added functions which check handle values using GTEST. Includes:
checking for null, not null, equality of two handles, and variants that
operate on vectors and arrays.

test: Move DeviceGroup tests to regression suite

test: Add TestICD DeviceGroup support

Add support for tests which use device groups by adding a new structure that
holds pointes to PhysicalDevice structs and implementing the necessary vulkan
API calls.

This commit also:
 - Formats test_icd.cpp
 - Makes vk_icdGetPhysicalDeviceProcAddr return stuff by refactoring the
   instance procaddr functions

scripts: Remove loader_icd_init_entries duplicate

Seems to be a copy-paste error that makes the function declaration appear twice.

build: Update to header 1.2.190

- Update known-good
- Generate source

build: Silence MSVC warning of replacing /GR

test: Update 32/64 tests to check the log

Use the DebugUtilsLogger to verify that the loader is downgrading the error
level of dll/so's from ERROR to INFO

test: Add DebugUtilsWrapper

This wrapper allows easy reading of the log output through the debug utils
messenger infrastructure. Since getting the stdout/stderr is not well
supported by googletest, this allows tests to be written which check for
specific log messages to be emitted, allowing another way to verify the
loader is behaving as expected.

loader: Downgrade wrong arch errors on windows

On linux, when the loader fails to load a .so if it was due to being on the
wrong architecture, the loader downgraded the error to INFO level. This
commit does the same thing for windows. Error code 193 is winapi's way to
signify that the dll failed to load cause of the architecture.