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

loader: Support multiple layers in one library by using <layerName>GetProcAddr

Get*ExtensionInfo string was being used to see if layer extensions are aliases.
Change this over to Get*ProcAddr  pointer since that will be different in
libraries that have multiple layers/ICDs in them.

Conflicts:
loader/loader.c

loader: Remove the test for device layers activated; they follow create/destroy

loader: Add device struct so can destroy layer info at DestroyDevice

This also allows multiple logical devices per icd/gpu combo.  Old code
conflated a icd/gpu combo with a logical device

mem_tracker: Process config file debug actions

Now that debug report functionality is in place can use
that mechanism to handle logging enabled by vk_layer_settings.txt

layers: Remove callback as runtime action

Doesn't make sense to have a callback action to
a config file setting, so remove that option.

layers: Add config parser to compute debug flags

layers: Move dbg flag print to config header

The print function seems to go better here with the
config file parsing as they use similar values.

layers: Add logging callback function for layers

Now that new debug report mechanism is in place, need
a callback function to output a log message.
Can then have layers register callback with debug_report
to have messages sent to stdout / FILE as well as
whatever the application asked for.

layers: Set helpers as static inline

static inline allows the code to be unused

layers: make log_msg format log strings for perf

Performance analysis showed a lot of time being spent
formating unused log messages (particularly INFO). By
moving the formatting into the logging function we can
only do the formatting if the message will be used.

layers: Ifdef out debug code

mem_tracker: Add some more debug code

mem_tracker: Use dispatch access functions

Hopefully makes code easier to read when using the
dispatch table access functions.

mem_tracker: Need dispatch object for logging

Needed to make sure we had a dispatchable object to
perform logging. Can't trust that mem or object are
correct, so have caller pass in user's dispatchable object.

mem_tracker: Add debug asserts

mem_tracker: Initialize new mem record

Need to intialize the record put on the list.
Removed the unused, newed record.

draw_state: Update to use private dispatch map

layers: Use static dispatch table maps

Changes to use dispatch table access functions.