loader: Make sure layer list from manifest file is all cleaned up properly

loader: Move long DEBUG_REPORT comments into debug report source code

loader: Add accessor functions to add/delete from layer property lists.

Use these to handle building and destroying instance and device layer property
lists.

loader: Convert layer versions from manifest file to Vulkan int version

loader: Allow null string for CMAKE_INSTALL_PREFIX to prevent extra paths

NULL string was making loader search extra local paths such as /share/...
Also expand log buffer .

Initialize VkDesciptorSet to VK_NULL_HANDLE

Compare the handle object to VK_NULL_HANDLE, not its internal integer value

layers: Validate renderPass.

cmake: Set CMAKE_INSTALL_PREFIX to "" so loader will be built with sys paths

CMake seems to default to non-null but for system SDK builds want this to
be empty string.

layers: Rework APIDump changes to fix windows compilation errors

MSVC compiler hated unions of structs with non-default constructors.

demos: use -o on glslangValidator to allow parallel make

loader: Fix Windows layers json files for backslash as separator

Screenshot layer: Works on SDK 0.2 branch

ObjectTracker Layer: Fix crash on windows in CreateInstance

function explicit_CreateInstance was crashing when it tried to
use a mutex before it was initialized. Determined mutex wasn't needed
at the start of this function, so remove the lock.

loader: Fix alloca ro loader_stack_alloc() for Windows

loader: get_registry_files needs to parse a list

layers: Add separate windows and linux json

xchange-94: Use VK_ prefix for environment vars

Use the Vulkan namespace (VK_ prefix) on debug environment
variables used in the loader and the Intel sample driver.
This resolves LunarXchange bug #94.
https://vulkan.lunarg.com/app/issues/55b76baa7ef24d0001000072

xchange-93: Use VK_ prefix on DEBUG_REPORT_..._NAME

Use the Vulkan namespace (VK_ prefix) on DEBUG_REPORT_EXTENSION_NAME
This resolves LunarXchange bug #93.
https://vulkan.lunarg.com/app/issues/55b76ae37ef24d0001000070

layers: Fix vkGetPipelineCacheSize validation.

layers: Enabled detailed output of void* memBarrier parameters

APIDump now interprets these void* pointers and prints the appropriate
information.

layers: Fix misc bugs in param checker.

Fix sType validation.
Improve nullptr validation.
Remove more false positives.
Fix VkResult validation.

loader: Fix bug in Windows for mod a static string DEFAULT_VK_LAYERS_INFO

loader: fix typo in log message

cube: We reuse cmd bufs, so dont do one_time flag

bug-14352: Update vulkan.h version

bug-14352: Missing size parameter in VkSparseMemoryBindInfo

https://cvs.khronos.org/bugzilla/show_bug.cgi?id=14352

demos: Add dependency for SPIRV shader components

Occasional saw issues with parallel make because dependency
was not specified for the shader SPIRV components used by tri
and cube.

bug-14330: Initialize handle on Vulkan objects

https://cvs.khronos.org/bugzilla/show_bug.cgi?id=14330
Accepted in gl_common 7/27

layers: Add validation case for mismatched renderpass attachements and subpasses

Added to the Image validation layer.

WSI: Initialize WSI entry points

Had missed initializing some WSI function pointers for
the device dispatch table.

vulkan: Add vkCmdPushConstants entry point

This patch fixes LunarXchange issue #84.
Also adds corresponding stub entry point to the Intel
sample driver.

intel: Add support for GetRenderAreaGranularity

tri/cube: Correct WSI swapChainPresentMode fallback is FIFO.

A copy and paste error from the spec made it seem that all ICDs will support
VK_PRESENT_MODE_IMMEDIATE_WSI (a.k.a. "immediate").  That's not true.  They all
need to support VK_PRESENT_MODE_FIFO_WSI.  Changed the comment and code so that
the preference of these demos is:

1) VK_PRESENT_MODE_MAILBOX_WSI
2) VK_PRESENT_MODE_IMMEDIATE_WSI
3) VK_PRESENT_MODE_FIFO_WSI

layers: Adjust SC to validate bulk pipeline creation

Previously we passed `count` down to the validator and then dropped it
on the floor, ignoring anything other than pipeline 0.

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
Reviewed-by: Courtney Goeltzenleuchter <courtney@lunarg.com>

Further layer search path fix: s/implicit/explict

A cut+paste error resulted in using the same value
for the explicit and implicit layer config paths.
This change remedies that.

Signed-off-by: James Jones <jajones@nvidia.com>

Added ICD loading path debug messages

Reviewed-by: James Jones <jajones@nvidia.com>

Fix loader paths for "etc" directories.

The previous libdir/confdir patches mistakenly
used /usr/etc rather than /etc as the default
location for configuration files.  This patch
corrects that mistake.

Signed-off-by: James Jones <jajones@nvidia.com>
Tested-by: Liam Middlebrook <lmiddlebrook@nvidia.com>

linux: Fix standard GNU path usage

The previous attempt at using GNU standard paths
limited searching to the prefix where the loader
was installed.  What we really want though is to
always search in /usr, also search in whatever
local prefix the user installed at if it wasn't
/usr, and use the standard GNU libdir, sysconfdir,
and datadir under those prefixes.  This change
implements that behavior.

Signed-off-by: James Jones <jajones@nvidia.com>
Tested-by: Liam Middlebrook <lmiddlebrook@nvidia.com>

linux: Use standard GNU install paths

These should get inherited from the build environment, which should
handle the variation between /usr/lib/$triple on debian vs /usr/lib{,64}
on redhat.

Signed-off-by: Adam Jackson <ajax@redhat.com>

Revert "vulkan: Updated LICENSE.txt with all known license information"

This reverts commit 3d2dc08ed28b159f1d86476add4719b3ec975b1b.

Just adding attributiones here does not match our SDK
and loader redistributable plans. Will add attributions to
specific areas rather than a global attribution file.

Windows nulldrv: Add stub for vkGetPhysicalDeviceImageFormatProperties

loader: Only check 32-bits for magic value

The loaderMagic grew to be 64-bits to look better
but need to mask it to 32bits for the validation check.

loader: Update the Linux and Windows manifest file documentation to match khr

loader: Change VK_LAYERS_FOLDERS to VK_LAYER_FOLDERS to match documentation

This effects Windows layers.

loader: Add missing entrypoints CmdPushConstants, GetRenderArea, GetPDImgFormat

Also fix several other missing entrypoints from dispatch table related functions.
Also reorder some of these functions to better match header list order so
can more easily spot missing entry points.

misc: Add vkGetPhysicalDeviceImageFormatProperties as a stub

layers: eliminate Windows compiler warning

Fix win32 build

compiler: upgrade to latest glslang and LunarGLASS for bug fixes

layers: update README.md

loader: have loader_magic be uintptr_t

Twice we've have feedback on this structure not being
correct - folks missed the union statment - so changing
this to be the same size as a pointer to make it more
clear that loaderMagic is the same size as *loaderData.

cube: Add support for debug break

When cube sees --break on the command line it will
install the DebugBreak helper callback for layer validation
errors and warnings.

loader: Add support for debug break helper

Implement BreakCallback that an application can use
to set a debugger breakpoint when running on Visual Studio
or gdb for requested debug message flags.

loader: Do not include dbg helpers in dispatch

The DEBUG_REPORT callback helpers do not need to be
included in the dispatch table as they are not API entrypoints.
They are functions specifically and only for DEBUG_REPORT
DbgCreateMsgCallback.

loader_log: add OutputDebugString("\n");

loader: Fix ICD and layer directory scan order on Linux

The convention is to scan /etc for host-specific settings before looking
in /usr/share, which may be a shared network filesystem.

Signed-off-by: Adam Jackson <ajax@redhat.com>

demos: Add more validation layers and fix errors/warnings

layers: objtrack has to call destroy in FreeDescriptorSets

headers: Updated debug object-to-string fcn with missing object types

In vk_debug_report_lunarg.h

layers: Add destroy_obj calls for ObjectTracker

This eliminates a ton of bad destroy messages, also removes check for
leftover instances when device is destroyed.

layers: Only verify data ptr not report_data

report_data is only defined if DEBUG_REPORT is enabled,
so asserting that it must be non-null prevents things from
running for no reason.

nulldrv: Link with -Bsymbolic so vkGetDeviceProcAddress behaves

cmake: Specify -std=c++11

gcc 5.1 will complain (and refuse to compile) if you use 'auto' in the
C++11 sense but haven't asked for C++11.

cmake: Don't check for installed glm, it's in-tree

Minor SDK-0.2 fixes for "tri"

Changes to "tri" are similar to the fixes for "cube":

1) WSI function were being called with an uninitialized pPlatformWindow.
Fixed by splitting demo_init_vk into two functions.

2) VkAttachmentViewCreateInfo.format wasn't being set.

3) VkShaderCreateInfo.pName wasn't being set.

4) demo->cmdPool was being leaked.

cube: Fill in pPlatformWindow after you have a window

demo_init_vk(), which sets the surface_description.pPlatformWindow was
getting called before the window was created so surface_description.pPlatformWindow
was always NULL. I added a new demo_init_vk_wsi() which splits demo_init_vk()
in two so that the window is created before surface_description.pPlatformWindow is assigned.

cube: set VkAttachmentViewCreateInfo.format

cube: Keep track of queue_count in demo structure

Need to know the number of queues in multiple places so just
save it as part of the demo info data structure.

loader: Switch to using layer manifest file info for layer extension info

Remove the direct queries for the layer extensions and properties.

layers: In DrawState correctly return rasterSamples count for PSO

vulkan: Updated LICENSE.txt with all known license information

loader:Remove dead code involving layer library queries now in manifest file

loader: Update loader readme and add ICD/ layer discovery documentation

loader: Use GetInstanceProcAddr rather than dlsym for most ICD entrypoints

Conflicts:
loader/loader.c

icd: Generate vkInstanceProcAddr with real entrypoints

loader: use VkDevice when initializing device table

Had ICD that actually checked the device parameter at GetProcAddr
and discovered we weren't using the device object.

demos: Set size of uniform buffer in cube

demos: Remove reference to undefined variable

misc: Create new glslang directory if needed

glslang has now moved to git. The update script will
recreate the glslang folder if the existing one is
the svn version.

Updated BUILD.md to ref the specs on how the loader finds ICDs/layers.

Since specifications now exist, those specs are pointed to instead of
duplicated in this document.

bug-14282: remove const

Clean up warnings from release build

mem_tracker: Use device object for dispatch tables

Need to use the device object to lookup the right dispatch
table now that WSI objects are non-dispatchable.

mem_tracker: Don't try to copy data if none available

Check that call received data before trying to copy.

Windows: Clean up more compiler errors and warnings

Windows: more changes for windows compiler

Windows: more fixes for windows compile

Windows: More windows compilier fixes

windows:  clean up windows compile issues

Conflicts:
layers/object_track.h
vk-layer-generate.py

layers: Remove dead code.

layers: In MemTracker remove non-intercepted WSI funcs from GetProcAddr

vulkan: Fix definition of LayerProperties

layers: MemTracker fix to skip COLOR usage bit check for now

Now that Attachments are generalized and not color or DS specific, need to make sure we only check usage bits appropriate for a given image.

layers: Fix some failing layer validation tests

Correctly set stageCount to 1 for simple VS-only pipelines. Moved some DrawState code so state is only updated in event of no validation ERROR.

demos: Hack to get cube working--only 2 swap chain images.

Other people have hard-coded the number of framebuffers, command buffers,
etc. as being equal to DEMO_BUFFER_COUNT (i.e. 2).  Without reworking all of
that code, this demo won't work.  Go with the flow for the time being, and
hard-code the number of swap chain images as 2.

loader: Changes to use new WSI swapchain extensions.

Use device extension list, not global.  When validating the PhysicalDevice
extensions really need to use the device extension list not the loader's global
list.

Fix include to find aligned_alloc

icd-common: Changes to use new WSI swapchain extensions.

layers: INITIAL Changes to use new WSI swapchain extensions.

There are still some FIXME's that must be addressed, as well as fully
supporting the new entrypoints.  This patch serves as a basis for review and
further work with other Vulkan engineers.

nulldrv: Changes to use new WSI swapchain extensions.