loader: Remove uneeded ICD and layers scanned boolean

Artifact of when layers and ICDs weren't scanned initially.

loader: Support gettting JSON ICD manifest files from registry

Windows now supports the JSON manifest file for ICDs

loader: Fix compilation warnings for debug_report on Windows

loader: Add math library to the build for linux  cJSON files

loader: Add parsing of JSON file to find location of ICD libraries

Works on Linux not yet on Windows.

cube: fix validation and clean up errors

Layers: Fix screenshot seg fault, changed accesses to dispatch tbl

demos: Ensure VertexInputState is NULL if unused

Was passing in pointer to a null VertexInputState structure, but
driver needs null pointer if structure unused.

Revert "loader: Remove CreateDevice from loader's instance"

This reverts commit b81161d0fa45c3b1333d8954220c992c5eb8aac4.

Change did not match commit message.

loader: Need to use ICD's GetDeviceProcAddr

The loader_GetDeviceProcAddr is the terminator
for the CreateDevice chain. It needs to call the ICD's
GetDeviceProcAddr to get the correct function pointer
not the object's current dispatch table that points
at the beginning of the chain.
Saw issue when running cube with only DRAW_STATE layer
enabled.

loader: Remove CreateDevice from loader's instance

Need to remove the CreateDevice function pointer from the
loader's instance table.

loader: Move CreateDevice to device table

Discovered an issue where a layer was doing cleanup
in it's DestroyDevice function but the CreateDevice
was never called.
This happened because the extension was only enabled
on the device chain and the device chain doesn't actually
call CreateDevice. That happens on the Instance chain.
Making it so that we can call down the device chain -
which is terminated by the ICD.
We need to know the real device object to construct the
device chain heiarchy and when calling down the device
chain it should end with the ICD doing the actual device
object creation.

This patch fixes the issue by using the
same process as CreateInstance. The loader will call
the ICD's CreateDevice and pass that in the *pDevice
argument. The layers then ignore the PhysicalDevice parameter
and use the *pDevice to access the device chain.
To prevent the ICD from being called twice needed to
stub in a special loader_GetDeviceChainProcAddr to provide
a stub for only CreateDevice as the end of the chain.

integrate review feedback.

layers: Do not advertise support for WSI

Layers that do not implement WSI features yet can 'trace'
WSI calls do not need to advertise WSI as an extension.
Any WSI calls made will land in the layer if it hooks
those calls. Otherwise should not care about WSI.

layers: Abort logging if null debug_report ptr

If a layer were to only be enabled on the device chain
it would not have had a chance to allocate a debug_report
data structure. Just treat such cases as having debug_report
disabled.

layers: Remove unneeded asserts

We'll soon break the rule being asserted, so remove
them.

generator: Add more debug statements

The location comments in the generated code help
a lot in determining what part of the generator generates
what code.

generator: Clear dispatch table

Finding entries that are not filled in properly is easier
if the table is initialized to zeros.

loader: Need to search device extension cache

Code was searching global extension cache for
extensions enabled at CreateDevice time. List are similar
but not identical and cause vulkaninfo to hit a segfault.

loader: Add debug info

vulkaninfo: Use app's extension count, not system

Was telling the driver to enable more extensions than
intended.

vulkaninfo: handy debug output

vulkaninfo: Need WSI extension for some queries

vulkan.h: V110 #13864 -- Border Color Values Underspecified

layers: ObjectTracker cleanup

Removed some unused error codes.
Updated INVALID_OBJECT error to correctly be flagged.

vulkan.h: Bug 14171 - remove unused structure type enums

vulkan.h: Bug 14153 - remove maxInlineMemoryUpdateSize

vulkan.h: Bug 14136 - reorder logicop enums

vulkan.h: Bug 13900 - Remove initialCount from semaphores

layers: Use new param checker layer.

vulkan.h: Bug 13616 Clarification of types of sample counts

vulkan.h: Bug 13939 Remove references to depth/stencil resolve

layers: Flag DrawState INVALID_RENDERPASS error if NULL RenderPass used to Begin/End/Continue a RenderPass

layers: Warn in DrawState if DescriptorSet bound that was never updated.

layers: In DrawState, validate correct use of RenderPass

This change validates certain API calls are made inside of or outside of a RenderPass as appropriate. Exact restrictions included in this changelist are included below.

Also fixed cube and tri demos to Bind Pipeline & DescriptorSets within RenderPass and updated some layer validation tests.

The following calls must be within RenderPass:
vkCmdBindPipeline() w/ GFX pipelineBindPoint
vkCmdBindDescriptorSets() w/ GFX pipelineBindPoint
vkCmdBindDynamicStateObject()
vkCmdBindIndexBuffer()
vkCmdBindVertexBuffers()
vkCmdEndRenderPass()

The following calls must be outside of RenderPass:
vkCmdBindPipeline() w/ COMPUTE pipelineBindPoint
vkCmdBindDescriptorSets() w/ COMPUTE pipelineBindPoint
vkCmdBlitImage()
vkCmdResolveImage()
vkCmdBeginRenderPass()

layers: Add DrawState check to verify Draw* cmds only issued within RenderPass

layers: Add DrawState check to verify that Pipeline layout from vkCmdBindDescriptorSets() matches layout from PSO at Draw time.

Also fix a bug where PSO status flags were not getting set into Cmd Buffer.

layers: Flag DrawState INVALID_RENDERPASS error if NULL RenderPass used to Begin/End/Continue a RenderPass

vulkan.h: V109 -- Remove minLod from VkImageViewCreateInfo

vulkan.h: V108 -- Static Types, Bug# 13919

layers: Correctly set Object Type for Dynamic State when flagging error

layers: Report error in DrawState if vkCmdBlitImage() called during RenderPass.

layers: Added DrawState check to flag error if compute pipeline bound during RenderPass

Bug 14014: Allow multiple entrypoints in modules

https://cvs.khronos.org/bugzilla/show_bug.cgi?id=14014
Header and minimal code changes to include support for
multiple entrypoints in a shader module.
Driver implementation TBD.

bug 14016: Make vkResetFences take const pFences

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

screenshot: remove unneeded initialization code

The device dispatch is setup in GetDeviceProcAddr, don't
need to do it in layer's CreateDevice.

screenshot: Need to wrap more functions

Need to include CreateInstance, DestroyInstance
and CreateDevice in screenshot's GetInstanceProcAddr
otherwise cannot process setup correctly.

screenshot: Need a DestroyInstance

Need way to clean up resources allocated at CreateInstance.

loader: Use loader_GetInstanceProcAddr

By using the vkGetInstanceProcAddr function in the loader's
instance dispatch table we will trace that function
during initialization of layers. Don't want to do that so
call loader internal function.

image: Need to get layer_data from correct place

vulkan: remove unused error code

vulkan.h: Remove vkGetFormatInfo, add features/limits (#12827, v125)

NOTES:

1/ Some layers impact from vkGetFormatInfo ->
vkGetPhysicalDeviceFormatInfo; some checks are currently disabled in
ParamChecker pending discussion on the best way to do this. Similar
checks in Image layer implemented via additional layer_data member
to link back from VkDevice -> VkPhysicalDevice.

2/ VkPhysicalDeviceFeatures, VkPhysicalDeviceLimits members all zero
for now; also some further churn to be done to the contents of these
structures.

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>

bug 13038: Remove support for multiple GPUs

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

loader: Add {}'s so that loader compilers on Windows

layers: Add DrawState check to verify that Pipeline layout from vkCmdBindDescriptorSets() matches layout from PSO at Draw time.

Also fix a bug where PSO status flags were not getting set into Cmd Buffer.

vulkan.h: V104 -- Unify compute/graphics pipeline shader passing semantics, Bug# 13800

layers: ObjectTracker per-instance storage, updated logging

o  Moved non-generated code into object_track.h header file
o  Added support for per-instance storage
o  Updated for new debug logging mechanism/callbacks

layers: Fix DrawState compile fail

Missed adding in layout when resolving merge conflict.

layers: Add DrawState check to verify Idx buffer bound before an Indexed Draw

Also some minor formatting clean-up to reduce lines of code

layers: Add DrawState check to verify BeginCommandBuffer was called before updating or ending a Cmd Buffer

Also migrated code to set Dynamic State back into CmdBindDynamicState function since that's the only call to set Dynamic State.

tri/cube: Convert GPA usage/error-checking/reporting to a macro.

tri: Change to vkGetDeviceProcAddr() for WSI functions.

vulkan.h: V103 -- Add VkPipelineLayout to vkCmdBindDescriptorSets, Bug# 14047

layers: Addition of source_line_info.py script to simplify codegen debugging

vk-layer-generate.py and vk_helper.py now use added source_line_info.py script to insert comments throughout generated code indicating where the code was generated from.

Layers: Correctly handle WSI_LUNARG extension.

Other code cleanup.

MemTracker: Advertise WSI_LUNARG extension in GetPhysDevExtInfo

layers: Erase proper key in draw state.

layers: Fix iterator comparison.

layers: Move image validation to separate layer.

layers: Comment for newbies.

MemTracker: make sure get_mem_binding_from_object uses dispatchable object

layers: Add DrawState check for Invalid PSO CreateInfo state

Report any violations in PSO restrictions as DRAWSTATE_INVALID_PIPELINE_CREATE_STATE error and don't call into driver.
Also fix a couple of broken error calls that had NULL dispatchable object parameter.

layers: Add active_shaders flag to Pipeline node struct in DrawState

Layers: ported to screenshot layer to master

Windows: Fix build issues on Windows

Generate vulkan.def from header, fix use of alloca

layers: Remove dead code.

demos: Fix output location in tri.vert.

layers: Make WSI_LUNARG a device extension  and check if enabled before use

Return NULL GetProcAddr if not enabled. Also fix DEBUG_MARKER bugs
in DrawState and ParamChecker

Also advertise WSI_LUNARG as a device extension in layers where it is used.

loader: Remove support for WSI as an instance extension

WSI is now a device extension so remove loader support

demos: Add missing spv dependency for cube

layers: Migrate draw_state.cpp to use log_msg utility for dbg reporting

Unified all reporting of INVALID_CMD_BUFFER messages under the getCBNode() function which eliminated lots of duplicate code.

layers: Update DrawState intercept of Create/Destroy MsgCallback

misc: Remove VK_PHYSICAL_DEVICE_INFO_TYPE_DISPLAY_PROPERTIES_WSI

This is only used for getting display object which is no longer needed
since the GetDisplayInfoWSI() functions is removed

misc: Remove GetDisplayInfoWSI()

This entry point was a challenge for loader to handle correctly. Since it is
going away in the new WSI proposal remove it in all components.

loader: Fix memory free bug in destroying icd from DestroyInstance

layers: Add linefeed after log message

mem_tracker: Cannot get dispatch key after destroy

get_dispatch_key looks for the dispatch table pointer at
the beginning of the object. Once it's destroy that value
isn't available so grab the key before it's destroyed.

vulkan.h: V102 -- Rename min/maxDepth for DepthStencil, Bug# 13917

Renamed depth-stencil minDepth and maxDepth to minDepthBounds and
maxDepthBounds.

vulkan.h: V101 -- Remove programPointSize, Bug# 13928

vulkan.h: V100 -- Remove vkClearDescriptorSets, Bug# 13783

Conflicts:
layers/draw_state.cpp

vulkan.h: V99 -- Remove fixed-function point parameters, Bug# 13928

Remove pointSize and pointFadeThreshold from VkDynamicRsStateCreateInfo.

vulkan.h: V99 -- Remove vkPinSystemMemory, Bug# 13926

vulkan.h: V99 -- Remove vkSetMemoryPriority, Bug# 13925

Conflicts:
layers/mem_tracker.cpp

layers: Report DEBUG_MARKER as supported in ParamChecker and DrawState

layers:  Add GetPhysicalDeviceExtInfo to DrawState

Allows DrawState to be enabled in layers_validation_test

DrawState: Fix destroying of maps so the map entrys get removed

They were hanging around and causing  double delete  assertions.

tests: Remove the enablement of the "Validation" extension

This accidentally got added resulting in validation layers always running.

WIP: Debugging - REMOVE

loader: Grab next before destroying the data

layers: Fix memtracker segfault for ill-behaved apps

Freeing object memory before destroying object resulted in a segfault.
This is illegal but shouldn't crash the layer.

shaderChecker:  Add GetPhysDevExtInfo  and fix CreateInstance