bug 14892 part 2: rename more "count" (WIP)

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

bug 14778: VK_IMAGE_VIEW_CREATE_READ_ONLY_DEPTH_BIT underspecified (WIP)

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

v192: bump API minor version

bug 14918 part1: consistency/cleanup ideas (from jbolz/pdaniell/dkoch) (WIP)

Change alignment types and replace byRegion by VkDependencyFlags.

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

v191: bug 14574: Capability bit for stores from non-fragment/compute shaders (WIP)

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

v190: bump API minor version

bug 14641: VkXXXCreateFlags are not consistently used for the various VkXXXCreateInfo (WIP)

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

bug 14848: Verifying Sparse Behavior (WIP)

Merge vkQueueBindSparse*() to one.

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

v189: bump API minor version

bug 14543: rename VkTex* enums to VkTexture* similarly for VK_TEX_* to VK_TEXTURE_* (WIP)

s/VK_TEX_FILTER_/VK_FILTER_/g
s/VkTexFilter/VkFilter/g
s/VK_TEX_MIPMAP_MODE_/VK_SAMPLER_MIPMAP_MODE_/g
s/VkTexMipmapMode/VkSamplerMipmapMode/g
s/VK_TEX_ADDRESS_MODE_/VK_SAMPLER_ADDRESS_MODE_/g
s/VkTexAddressMode/VkSamplerAddressMode/g
s/mipMode/mipmapMode/g

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

v188: bump API minor version

bug 14914: Should strictLines be a feature or a limit (WIP)

A limit.

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

bug 12921: Memory callback (WIP)

The per-object allocator is ignored for now.

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

bug 14892: 'count' parameters and members named inconsistently in many cases (WIP)

s/waitSemCount/waitSemaphoreCount/g
s/signalSemCount/signalSemaphoreCount/g
s/cmdBufferCount/commandBufferCount/g
s/queueFamilyCount/queueFamilyIndexCount/g
s/layerCount/enabledLayerNameCount/g
s/extensionCount/enabledExtensionNameCount/g

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

bug 14909: Define "length" as it relates to push constants be in units of bytes not words (WIP)

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

bug 14905: Reserve VkStructureType values for loader/layer bootstrapping (WIP)

Fix vk_helper.py to ignore them.

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

v187: bump API minor version

bug 14630: Issues with regard to primitive restart index (WIP)

Remove primitiveRestartForPatches.

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

bug 14857: Adding extensibility to VkSubmitInfo (WIP)

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

Merge branch 'master' of gitlab.khronos.org:vulkan/LoaderAndTools

layers: Fix Swapchain include dependency for Windows build break

demos: Initialize all of subresource fields

layers: Use descriptor type to decide which pointer to free

layers: Migrate Image dispatch tables into layer_data struct

layers: Migrate Swapchain dispatch tables and global data into layer_data struct

layers: Migrate ShaderChecker dispatch tables and global data into layer_data struct

layers: Remove mid/mdd functions from MemTracker

layers: Migrating MemTracker global data into layer_data struct

layers: Add dispatch tables into MemTracker layer_data struct

ubuntu 15.10 directions for intel DRI3

v186: bump API minor version

bug 14567: What is VK_QUEUE_EXTENDED_BIT? (WIP)

Removed.

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

bug 14876: Can timestamps be written inside a renderpass? (WIP)

Timestamps are written to a query pool.

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

bug 14362: VK_NULL_HANDLE does not work in C (WIP)

Drop type safety on 32-bit.

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

v185: bump API minor version

bug 14864: Remove redundant VkPipelineShaderStageCreateInfo.stage field? (WIP)

Removed.

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

MR 461: Documented updated vkQueueSubmit and remove wait + signal semaphore operations on queues. (WIP)

https://gitlab.khronos.org/vulkan/vulkan/merge_requests/461

support and use VkSubmitInfo semaphore signaling and waiting

bug 14868: Replace VkTimestampType with VkPipelineStageFlagBits (WIP)

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

v184: bug 14719: vkCmdCopyBufferToImage alignment requirements (WIP)

Update vulkaninfo to print new limits.

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

bug 14822: timestampFrequency underspecified (WIP)

Replaced by timestampPeriod, number of nanoseconds per tick.

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

bug 14725: Remove or replace VK_SPARSE_MEMORY_BIND_REPLICATE_64KIB_BLOCK_BIT (WIP)

Renamed.

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

bug 14462: endianness of code for shader modules (WIP)

Change the type of pCode to uint32_t*.

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

bug 14855: vkGetPipelineCacheSize is unique (consistency issue) (WIP)

Update vkGetPipelineCacheData() to replace vkGetPipelineCacheSize().

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

bug 14861: rasterization review issues (WIP)

Update vulkaninfo to print features->strictLines.

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

bug 14365: Rename slopeScaledDepthBias to depthBiasSlopeScaled (WIP)

s/slopeScaledDepthBias/depthBiasSlopeFactor/g
s/depthBias\b/depthBiasConstantFactor/g

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

bug 14844: Query Pool issues (WIP)

We did not and still do not honor the stride nor flags.

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

vulkan.py: use generated "core" API

The parsed vkCmdSetBlendConstants prototype makes a difference to vktrace.
But vktrace should handle it just fine.

vulkan.py: fix vulkan.h parsing

Ignore PFN_vkVoidFunction typedef.  Fix layer dispatch table output.

layers: Fix MemTracker uninitialized variables

Caused issues in SDK tests on Windows

demos: Properly destroy command pool in cube demo

layers: Add DestroyImage to image layer's device GPA table

loader: Update Linux and Windows ICD/layer discover documentation

loader: Update linux documentation on where ICD lives.

loader: Move utility function so it is found on both Linux and Windows

loader: set DEFAULT_VK_{DRIVER,LAYER}S_PATH to "" on Linux

When loader_get_fullpath() gets an empty string as the 'dir' parameter,
it immediately writes out the original, unexpanded filename. Setting
the default drivers and layers paths to an empty string on Linux will
allow the unmodified filenames to be passed to dlopen(3) without having
to maintain a separate code path for Windows.

Now that there is no longer a default path for drivers or layers on
Linux, ${CMAKE_INSTALL_LIBDIR} no longer needs to be baked into the
loader binary. Note that any ICDs or layers that are currently getting
installed to the previous default paths (e.g. /usr/lib/vulkan/icd or
/usr/lib/vulkanlayer) will not be discoverable when specified as plain
filenames, unless these directories happen to be configured in the
loader's search path on the system in question.

loader: add helper for combining paths

Add a helper function to combine an arbitrary number of path elements,
separating them with the platform-specific directory separator, and
use it in places where path combining is currently achieved through
direct string manipulation.

As an additional cleanup, update loader_get_fullpath() to duplicate
the search path string internally, rather than relying on the caller
to do so. Since loader_get_fullpath() and loader_expand_path() no
longer modify input strings, and loader_expand_path() no longer
requires that the caller append the DIRECTORY_SYMBOL ahead of time,
acrobatics to duplicate strings to prevent modification and/or to add
directory separators are no longer necessary.

This change will result a behavior change on Windows: the existing
logic for loading an ICD that is specified as a bare filename, and
not as an absolute or relative path, is to search for a driver with
the given ICD name in the DEFAULT_VK_DRIVERS_PATH search path on
Linux, and to use the given ICD name as-is on Windows. This could
lead to DLL injection attack on Windows. Merge the Linux/Windows
behavior to limit the search for ICD filenames to the default search
path on Windows as well, which is also more consistent with the
behavior described in the loader spec.

Conflicts:
loader/loader.c

demos: fclose in cube and tri

Balance every fopen with fclose.

demos: windows fixes for cube and tri resize

Defer resize until after first prepare.
Record new window size from WM_SIZE.
Leave demo->swapchain.handle valid to pass to CreateSwapchainKHR.

Layers: MemTracker check vkCmdCopyQueryPoolResults

MemTracker check of buffer memory and usage flags in vkCmdCopyQueryPoolResults

demos: add occlusion features to vulkaninfo

Add reporting of VkPhysicalDeviceFeatures occlusionQueryNonConservative to vulkaninfo.

use conditional compilation based on _WIN32 instead of WIN32 and __linux__

demos/cube: Provide a non-null VertexInputStateCreateInfo

In the spec for VkGraphicsPipelineCreateInfo, some structs are explicitly
listed as being able to be NULL if they are not used.
VertexInputStateCreateInfo is not one of them.

demos/cube: Explicitly specify std140 layout on UBOs

Without this, glslang won't actually provide the offsets and strides
required by SPIR-V.

layers: Validate buffer view usage flags

layers: Validate number of layers in subresource.

Reviewed by Tobin.

tests: Added CopyDescriptorUpdateErrors test to verify 3 errors during descriptorset copy updates

layers: Add DrawState validation for copy descriptor updates

layers: Simplify descriptor validation in DrawState

Many descriptor validation helper functions were returning skipCall based on the potential for a bad update structure type. However, at the beginning of update validation, we validate that the update type is ok. Therefore, the other helper functions don't need to continually repeat this check, and can be simplified to return relevant values directly.

layers: Improve DrawState write descriptor update

Validate that stageFlags are the same for a single write update.
Validate that all sampler updates from a single write update are either immutable or non-immutable.
Refactor update contents check to have a loop per switch case instead of switching for each loop iteration.
Added two new validation errors:

DRAWSTATE_DESCRIPTOR_STAGEFLAGS_MISMATCH
DRAWSTATE_INCONSISTENT_IMMUTABLE_SAMPLER_UPDATE

layers: Remove DrawState g_lastBoundDescriptorSet variable - it's no longer used

demos: Clean up tri depth

We animated tri to make it clear it was rendering correctly based
on ISV feedback. It looks better, but still questionable.

This change makes the depth cutoff a straight line by giving the top
two vertices the same depth.

draw_state: Fix enum in validate_draw_state_flags

DRAWSTATE_STENCIL_NOT_BOUND -> DRAWSTATE_SCISSOR_NOT_BOUND

windows: Fix vector resizing in draw_state

demos: print new device limits in vulkaninfo

misc: Remove dynamic array definititions

Windows doesn't like dynamicly sized arrays. Replace
with new'd array of objects or object pointers instead.

misc: Add intrin.h to support ffs utility

misc: Windows does not like named structure initialization

Bug 14621: Change vkGet* functions to not return VkResult

Missed one function in prior patch.

tri/cube: Adjust demo_resize() to API changes.

tri/cube: Add WM_SIZE to Win32 event handler to resize a window.

cube: Removed some #if 0's that had bee left in the code.

tri/cube: Handle window resizes (from WSI or XCB).

If either vkAcquireNextImageKHR() or vkQueuePresentKHR() return
VK_ERROR_OUT_OF_DATE, a new demo_resize() function is called, that replaces the
current state, command buffers, swapchain, etc, with new ones that reflect the
new size of the window.  As part of this, the app picks up the window size from
vkGetSurfacePropertiesKHR().

The same occurs if the application receives an XCB_CONFIGURE_NOTIFY event, with
a changed window size.

layers: Improve DrawState Descriptor validation

Adding further validation for the contents of vkUpdateDescriptorSets() call.
Make sure that provided sampler/image/buffer data is consistent with type and layout of the update.

For imageView updates, make sure that aspect bits are set correctly for Color and DS images and make sure that the imageLayout is compatible with the underlying image format.

This results in 4 new validation checks:
DRAWSTATE_SAMPLER_DESCRIPTOR_ERROR
DRAWSTATE_IMAGEVIEW_DESCRIPTOR_ERROR
DRAWSTATE_BUFFERVIEW_DESCRIPTOR_ERROR
DRAWSTATE_BUFFERINFO_DESCRIPTOR_ERROR

Each of the new checks has a basic check to verify that the underlying Vk object (sampler, imageview, bufferview, buffer) for the respective type of check is valid. The ImageView error also includes cases where format of the image is not consistent with the layout (Color vs DS).

Also added some enhancements to existing check:
DRAWSTATE_INVALID_IMAGE_ASPECT - verify D&S bits not both set, but that one is for a DS layout, verify COLOR bit set for Color layout

This change included adding and/or enhancing the tracking in DrawState for image[View] and buffer[View].
There is documentation for each of the new checks.
There are new tests added for each of the new error cases as well as the ASPECT error case.

layers: Modify Image layer to store IMAGE_STATE structs instead of their unique_ptrs

loader: Check for NULL ICD in CreateDevice

Related to lunargxchange #163

misc: Bump to header version 183

draw_state: Remove obsolete check

bug-14827: Remove DescriptorInfo union

The DescriptorInfo union is invalid in C++

bug-14258: Remove maxCommandBuffers

commit #53c2f7e

layers: Update ObjectTracker to handle double-embedded loops of array struct params w/ object arrays in them

ObjectTracker still only goes 1-level deep on object validation so if an array struct param has an embedded struct that then has an object, validating those objects is not yet done by ObjectTracker.

layers: Fix objectTracker to validate DSs in CmdBindDescriptorSets()

bug-14258: Make descriptor pools and command pools more consistent

misc: bump to header version 182

bug-14575: normalized queue priorities

Bug 14789: Clean up limit structure

layers: Fix windows build error

Image layer had a list initialzation inside a member initializer list

layers: Add state-dependent image aspect/format checks

misc: Update header to v181

bug-14827: Make DescriptorInfo a union of bufferInfo, imageInfo, and bufferView