bug 14313: Rename ArraySlice to ArrayLayer

Bug 14313: Remove VK_FORMAT_FEATURE_CONVERSION_BIT

bug 14516: rename VK_IMAGE_USAGE_DEPTH_STENCIL_BIT

into VK_IMAGE_USAGE_DEPTH_STENCIL_ATTACHMENT_BIT

bug 14541: rename VkTexAddress to VkTexAddressMode

bug 14537: Improve VkImageFormatProperties

Bug 14537: Improve VkImageFormatProperties

Commit: e45df2e

bug-13139: refactor sparse properties

layers: Reenable MemTracker reference tracking

Which was disabled during the type-safety dark times.

nulldrv: Fix compiler issues

vulkan.h: Change return types of some functions to void

Functions that should be thought of as "impossible to fail in the face
of valid parameters" have had their return types changed to void.
This includes all of the vkDestroy functions, vkFreeMemory, and vkUnmapMemory.
vkUpdateDescriptorSets is also included, because of the frequency the function
is expected to be called.

layers: Add note about vkCreateDevice behavior

vulkan: Add validation error result

When running with validation layers a validation layer
may abort an API call (due to app callback indicating so).
Since there are no validation error codes in vulkan.h
we need to define one here to return for such cases.
Details of the validation failure are available via the
DEBUG_REPORT callback.

demos: Do not bail on validation error

draw_state: Implement bailout case for one test

Proof out the bailout case. Fill in others in following
commit(s)

layers: Add bailout flag to debug report callback

Layer validation tests will deliberately induce
errors that the validation layer should catch.
In these cases we don't really want the layer to call
down the chain once it's detected a failure. We can't
know that in the layer so changing the return value
on the callback from void to VkBool32 so that the
callback can indicate if the call should continue
or not. true = bail.
That allows the call chain to execute normally,
not segfault in the driver and allow the test
to clean things up.

mem_tracker: validation comment

bug-14538: Remove driver validation checks

Marked validation checks done in the driver with
TODOVV. Once we verify the check is covered in a
validation layer we can remove the driver code.

layers: Remove unused functions

layers: Add DeviceLimits json files

layers: In DrawState only validate bound VBO index at the time of Draw

Previously validating this at the time of VBO bound and when PSO bound. This could result in checking against stale state, though, and thereby causing false positives.

layers: DrawState fix to prevent false positives when matching PipelineLayouts

Updated existing check where PipelineLayout found with vkBindDescriptorSets was always checked against PipelineLayout from PSO. Since there are cases when a Draw requires no Descriptors the check was modified for now to only occur when vkCmdBindDescriptorSets has been called. This is still not perfect but will prevent false positives.
Really need to identify when it's valid to not call vkCmdBindDescriptorSets and, if so, don't perform the PipelineLayout match check.

demos: Fix demos to correctly query physicalDevice count

layers: Fix ScreenShot layer to handle EnumeratePhysicalDevices query count case

layers: Initial framework for DeviceLimits layer

This includes many initial entrypoints and setting up function intercepts with only a few actual checks to begin. DeviceLimits layer is intended to capture two broad categories of errors:
1. Incorrect use of APIs to query device capabilities
2. Attempt to use API functionality beyond the capability of the underlying device

DeviceLimits stores its own internal record of underlying device capabilities and flag errors if requests are made beyond those limits.

Initial checks and documentation include verification of EnumeratePhysicalDevices call and some basic queue create/get validation.

loader: Use GetProcAddr function names from layer manifest file

No longer assume vkGetInstanceProcAddr or <layerName>GetInstanceProcAddr to find
layer's instance GetProcAddrs. And similiar for device ProcAddr. Instead use
any value specified in manifest file. If none specified in manifest file then
assume vkGetInstanceProcAddr or vkGetDeviceProcAddr.

demos: Prevent tri rendering during shutdown

vulkan.h: Refactoring physical device sparse properties and limits, bug# 13139

Part of changes for header revision V161.

vulkan.h: Add bufferFeatures to VkFormatProperties, bug #14546

Windows build: rename vulkan.0.dll to vulkan-0.dll

documentation: Detail DRI3 use on ubuntu 15.04

Updated DRI3 discussion in README.md and BUILD.md.

loader: Fix Windows build for alloc stuff

loader: make thirdparty code use allocation callbacks

Use TLS to keep instance pointer and use this for cJSON alloc/free
callback.

demos: Add allocation callback to tri test CreateInstance

loader: Make use of app provided allocation callbacks

Convert all heap allocs/frees to use loader_heap_alloc() or loader_heap_free().
Before CreateInstance this will use malloc/free. At (and after) CreateInstance
alloc callbacks are stored and used for any allocations.

Exceptions are thirdparty code (cJSON  and dirent_on_windows) still always use
malloc/free. Plan to address these in later patch if ti makes sense.

loader: Add support for realloc using the app allocation callbacks.

loader: convert to using loader_stack_alloc for consistency

WSI: Convert from VERSION to REVISION in nulldrv

All extensions single revision number instead of major.minor.patch

This was decided during the August face-to-face.  Originally, the WSI
extensions did it the normal OpenGL way (single revision number), and the debug
extensions did it the new major.minor.patch way.  When I tried to convert WSI,
an objection was raised.  We decided to go with the OpenGL way.

This commit also changes vulkaninfo to report the single revision number.

This commit also changes the extension "number" assigned to the 2 debug
extensions, basd on a "registration" email that Ian Elliott sent to Jon Leech
on 21 August, 2015.  The initial set of "registered" extensions are (with their
numbers):

1.VK_EXT_KHR_swapchain
2.VK_EXT_KHR_device_swapchain
3.VK_EXT_KHR_display
4.VK_EXT_KHR_display_swapchain
5.VK_EXT_LUNARG_debug_report
6.VK_EXT_LUNARG_debug_marker

WSI: Convert from VERSION to REVISION

WSI: Update headers for WSI swapchain extensions (versions 51 & 17).

WSI: Convert WSI swapchain extension usage to new KHR style

WSI: Update headers for WSI swapchain extensions (versions 47 & 15).

BIG RENAME: The names change for the extensions, header files, enums,
functions, etc.

WSI: Remove old loader code for the VK_WSI_LunarG extension.

demos: Set depth image aspect correctly

layers: If layers receive bad filename for output log, report ERROR and fallback to stdout

layers: Update some layers to avoid repeated call to get_dispatch_key()

vulkan.h: Bug-14476: Rename texelCoords to unnormalizedCoordinates

bug 14517: Rename cs parameter to stage

bug-14537: Improve VkImageFormatProperties

https://cvs.khronos.org/bugzilla/show_bug.cgi?id=14537
* Add VkSampleCountFlags that describes set of power-of-two sample
  counts (VK_SAMPLE_COUNT_N = (1<<N))

* Use VkSampleCountFlags to describe exact supported sample count set
  in VkImageFormatProperties

layers: MemTracker fixes for image usage flag validation.

Bug 14522: Remove VkAttachmentView

https://cvs.khronos.org/bugzilla/show_bug.cgi?id=14522
Several parts of the driver really want an attachment view
so I've included the attachment information in the
intel_img_view structure and initialize both when the
view is created.
Parts of the meta path only need intel_att_view and
I kept that in place.

bug-14515: Add occlusionQueryNonConservative feature

windows: Fix a couple signed/unsigned warnings

layers: Modify ScreenShot usage and layout flags to work cross vendor

layers: Port screenshot fixes to master

Sourced from sdk-0.2 branch commit c98372f1

headers: Add vk_sdk_platform.h for Win/Linux differences

headers: Moved utility routine out of to-be-upstreamed header file

Moved string_VkDbgObjectType function into object_track.h -- helps
resolve some windows build issues.

layers: Automated checking of validation layer documentation

Addition of vk_layer_documentation_generate.py script to check hand-written vk_validation_layer_details.md doc against the layer source code.
Currently this will verify documented validation checks against checks declared in source headers and verify that API calls in document are actual API calls.
This includes many documentation and layer updates to get everything in synch and allow the script to pass.

layers: Various corrections to vk_validation_layer_details.md

A number of API functions were incorrectly named so I fixed those.
Updated DynamicState binding function names.
Removed old references to VkObject APIs.
Update Threading table to have final 2 columns of other tables for easier automated parsing.
Added TODO for WSI validation layer(s)

vktrace: Add vktrace codegen dirs to .gitignore

These directories contain only generated files.

bug-14532: Rename VK_LAST_* to VK_REMAINING_*

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

misc: tool updates, no functional change

Bug-14177: Remove Float16 feature.

Due to the fact that this isn't currently well-specified and there is expected
implementation variability on current HW, this is removed from
the current API.

bug-13970: Sparse Texture VA Size

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

wsi: Fix image usage flags in tests and demos

vktrace: Glave renaming -- removed all references to *glave* and *glv*

layers: Add todo flagging destroy while on queue

This starts with the case of early VkDestroySemaphore seen in demos.
It is a much larger general case of objects that are active on queues.

demos: vkDestroySemaphore only after it is unused.

Semaphores should not be destroyed when in use.
Call vkQueueWaitIdle before calling vkDestroySemaphore.

layers: Handle bad filename in APIDump settings file

If APIDumpLogFilename requested in vk_layer_settings.txt file has a bad path or cannot be opened for some other reason, then APIDump will write out an ERROR message w/ the bad filename and notify user that STDOUT will be used instead.

vktrace: Add support for debug_marker_lunarg extension

layers: Fix ObjectTracker to handle dyanamic array params

ObjectTracker will now check for valid objects across all entries of arrays that are passed as params. This includes all objects embedded in arrays of structs such as the pCreateInfos array in vkCreateGraphicsPipelines().

vk_platform: Update to upstream vk_platform.h

Gitlab bug #9: https://gitlab.khronos.org/vulkan/LoaderAndTools/issues/9
noted that the SDK's vk_platform.h did not match the upstream vk_platform.h.
This patch makes them identical updates impacted code.

demos: Fix compile warning setting renderPass

nulldrv: Make build work on Windows with destroydynamiclinewidth

loader: Fix SegFault if ICD JSON file doesn't exist to return gracefully

vktrace: GLAVE file/directory renaming

Glave/glv files and directories renamed. Default trace files now named
*.trace.

Remove extraneous temp file

layers: Fix APIDump to correctly set output file name from settings file

Merge branch 'check-cube-arrays-complete' into 'master'

check that cube array views are complete cubes

See merge request !29

check that cube array views are complete cubes

v156: Bump API minor version

v156: Bug 14451 - Rename dynamic state objects

Bug 14474: Remove *_USAGE_GENERAL

v155: Bug 14435 - Remove abbreviations in VkClearColor

v154: Bug 14417 - Added VkShaderStage to VkShaderCreateInfo

v153: Minor version bump for bugs 14323, 14400, 14339, 14406

Bug 14406: Add support for front/back stencil reference dynamic state

Bug 14339: Add the missing alphaToOneEnable to VkPipelineColorBlendStateCreateInfo

Bug 14400: Add FORMAT_FEATURES for blits

Bug 14323: RasterLine and RasterDepthBias changes

loader: Add support for relative pathnames in the ICD manifest file lib_path

Fix formatting for GitLab web interface

layers: Initial commit of Validation Details md file

This is intended to be a comprehensive list of validation checks in the Vulkan validation layers. For each layer, a table is included that describes the checks performed by that layer, the relevant API calls, the associated ERROR/WARNING code, any tests that exercise that check, and further notes or TODOs. Also each layer has a "pending work" section so that checks to be added in the future can be documented here.

This is intended to be a working document where the document stays in synch with the code. Also, if missing validation checks are identified, but cannot be immediately implemented, they should be added to an appropriate "pending" section. Finally, if someone wants to contribute to the validation efforts, they can identify existing "pending" work to implement.

loader: Make so layer manifest files are retained at CreateInstance

layer manifest files were rescanned after CreateInstance as needed for
CreateDevice and GetPhysicalDeviceXXXProperties. Now stored the device
and instance layer list in instance object.

Also fix mem leak of extension list in some cases.
Also fix bug where deep copy of instance  to device layer properties was
not being done. Only a shallow copy was being done.
Also remove the lib_info struct from layer properties structure as only the
lib name field was being used. Other layer library info is stored in the
layer_library_list once the layer libraries are opened.

loader: Make the ICD library scan happen as needed rather than once

Add a one time loader init routine also since this was done by ICD scan.
Add icd library struct to instance object.

loader: Remove the device_extension list from from scanned icds, unused

loader: Remove the global extension list from scanned_icds struct

This list is regenerated as needed and stored in the instance object.

loader: Make the global/instance ext list once for each instance

Also fix some bugs in the heap_alloc/heap_free.
Also fix some memory leaks.

loader: Remove global instance extension list and rebuild it as needed

This will be necessary once ICDs are rescanned at CreateInstance and prior.

tests: Fix tests that re-submit Cmd Buffers w/ ONE_TIME_SUBMIT flag

Added a way to pass VkCmdBufferBeginInfo through test framework.

For tests that do multiple submits, update BeginCommandBuffer() calls to not set ONE_TIME_SUBMIT flag.