vulkaninfo: Check VkResult of vkGetPhysicalDeviceSurfaceSupportKHR

patch #125

issue 126: fix vulkaninfo segfault on 1.0 devices

We didn't quite get the check on whether a format can be legally
queried on a particular GPU quite right.  We were checking against
the instance version only.  We need to check agains the driver's
supported API version as well.

vulkaninfo: shorten format dump in console

vulkaninfo: Make queue families report present support

vulkaninfo: Properly check vk command error codes

vulkaninfo: Remove non-existent device extension

winrt: Update runtime license dates

winrt: Update runtime license

build: Update version in manifest JSON files

Change-Id: I9616bc29dda4680007da5b4e23fa3c6252401da8

build: Update known good glslang revision

Change-Id: I09db85f27310a012c177555eed87722cc3e15adc

Remove COPYRIGHT.txt

This file was not kept up to date.  Any items added that aren't
Apache 2.0 will have their own LICENSE in their source tree.

Change-Id: I8492cabd3f5cc0f52f3d83f004fe52ab49fd5f25

build: Update known-good for 1.1.97 header

Updated:
- `build-android/vulkan-headers_revision_android`
- `scripts/known_good.json`
- `scripts/kvt_genvk.py`

Change-Id: Ifa3f61bfd50fb15bff122f51edb63108511922d9

vulkaninfo: Update copyright date to 2019

Change-Id: Ie4f1aefae4603e6b27ec75b7d7b6a36f8cd94bc3

vulkaninfo: Add VK_EXT_fragment_density_map

Added output for the following structures from
extension VK_EXT_fragment_density_map:

    VkPhysicalDeviceFragmentDensityMapFeaturesEXT
    VkPhysicalDeviceFragmentDensityMapPropertiesEXT

Change-Id: I6d9a09a3a1bf889c5e44ac0eff66cdcf572f7b19

vulkaninfo: Add VK_EXT_scalar_block_layout

Added output for the following structure from
extension VK_EXT_scalar_block_layout:

    VkPhysicalDeviceScalarBlockLayoutFeaturesEXT

Change-Id: I63cf81d7f690060a3d1b81c57ac0dc4e3cda457e

vulkaninfo: Add VK_EXT_transform_feedback

Added output for the following structures from
extension VK_EXT_transform_feedback:

    VkPhysicalDeviceTransformFeedbackFeaturesEXT
    VkPhysicalDeviceTransformFeedbackPropertiesEXT

Change-Id: I60c907a26071ddc315d588b9cdc395f4d3bc446d

vulkaninfo: Add VK_KHR_shader_atomic_int64

Added output for the following structure from
extension VK_KHR_shader_atomic_int64:

    VkPhysicalDeviceShaderAtomicInt64FeaturesKHR

Change-Id: I2979e656db9268c5a11290bb5c77e1ebd3059ea1

icd: Return better size in GetBufferMemoryReq

Track the buffer create info from CreateBuffer so that
GetBufferMemoryRequirements can return an actual size
instead of a hard-coded guess.

Needed to survive VLT setup during GPU validation testing.

vulkaninfo: minor fix after review

Changed
    for (unsigned int i = 0,...)
to
    for (uint32_t i i = 0,...)

for consistency with the rest of the file.

vulkaninfo: only query supported formats

vulkaninfo must not query formats that are not supported on a
given instance, as this provokes undefined behavior (it could
appear to work, or it could segfault in the driver).

These changes ensure that a format is only queried if it
is supported in either the base Vulkan instance or, if it is an
extension format, if that extension is supported by the Vulkan
instance.

- Moved instance version information to the AppInstance struct
  (initialized by AppCreateInstance()), which makes sense and
  allows lower-level code to determine whether a format is
  supported on the current instance version
- Formats are now encoded with their owning Vulkan version
  and extension name, so they can be checked for validity
  before making a query.

Note that this change can cause behavior changes; instances
that didn't crash on unsupported formats (and thus appeared to
support more formats than they actually did) will no longer
report those formats.  Instances that would segfault when
an unsupported formatn was queried should no longer crash.

vulkaninfo: Fix merge conflicts in PR #98

Fixed some merge conflicts in PR #98 so that it
can be added to vulkaninfo

Change-Id: I24c7110ec77c8b9dabd371dd8435c3d25db24928

vulkaninfo: Update enums to 1.1.89 headers

vulkaninfo: Update enums to 1.1.74 headers

Fixes #72

docs: Add explanation of version tagging scheme

Change-Id: I8a83d5e2303d09fe22425cb350473262694f4642

build: Update known-good for 1.1.96 header

Change-Id: Id4ab3023bfc526ea469b7be6f7d98ce5829235d1

vulkaninfo: Add VK_EXT_pci_bus_info

Added VK_EXT_pci_bus_info extension to Vulkan Info.
Print data from struct
VkPhysicalDevicePCIBusInfoPropertiesEXT.

Change-Id: I65161fbb5ac9b28f817268ea59eac8f8b5e14741

vulkaninfo: Set missing sType and pNext in surf_formats2 array.

This fixes #109

vulkaninfo: Cherry-pick GitLab extension code.

Cherry-picking code for the following extensions
and structures from GitLab.

Extensions:

    KHR_driver_properties
    KHR_shader_float_control
    KHR_shader_float16_int8

Structures:

    VkPhysicalDeviceDriverPropertiesKHR
    VkPhysicalDeviceFloatControlsPropertiesKHR
    VkPhysicalDeviceFloat16Int8FeaturesKHR

Change-Id: I2a7e970e681602e644f58d7cd10e66943f5650b5

vulkaninfo: Fix issue 72

Fixed issue 72, which prevented vulkaninfo from
listing image formats past
VK_FORMAT_ASTC_12x12_SRGB_BLOCK for Vulkan 1.1

Change-Id: If448c5c8263a8af500ec1defb38aef69a0df7c08

build: Update known-good for 1.1.95 header

Change-Id: I23159914c83c87f8201bb8ea6730b803e564ce66

vulkaninfo:  Fix memory init/leak

- Clear the structures allocated for the pNext chains.  While this
  is not strictly necessary, any pNext structure that is not
  filled in by a driver (because the extension isn't there) gets
  left in an untouched state.  This can be a false positive flagged
  by valgrind.
- Use XFree to free storage allocated by Xlib's XGetVisualInfo().
  This fixes an actual leak.

build: Update Travis-CI to Ubuntu 16.04

vulkaninfo: Fix extra lines in HTML file

Removed the extra line breaks appearing after
certain data structures listed in the HTML output
file of Vulkan Info.

build: Re-add wayland-client path for mock_icd

Fix issues #1444 and #1989 of Vulkan-LoaderAndValidationLayers **again**.

Change-Id: I2c0e95df321ec48ea8fcef94115355ab00e2f088

Cube: Use c++_static instead of unsupported gnustl_static for STL

Change-Id: If351431424a2b1d30cfa30d43a6515bca034ac40

cube: Update cube/vkcube naming

Change-Id: I633c97e9b82a6b5069cc2e7a83106f5d17600c6b

vulkaninfo: More spacing cleanup

Additional spacing cleanup in the surface capabilities section.

Remove last MIR reference

Change-Id: Ib7fd104466b189d2d8bb0c70cd6e7aa2623ea0e7

Update Vulkan wrappers from latest header

Change-Id: I61d6c28aa36f86f2b16c534fd2311fe0df06b288

build: Update known-good for 1.1.92 header

Updated:
- `build-android/vulkan-headers_revision_android`
- `scripts/known_good.json`

Change-Id: Id7ddc93bc242f35270a16e8728865cfc55cd905f

vulkaninfo: Remove unnecessary spacing

Removed some unnecessary spacing after @whydoubt's Wayland
changes.

Change-Id: Ia076dc1f8b45a75efc8f06eea4092996d230c474

vulkaninfo: Add wayland surface extension support

Added code for setting up a Wayland surface and dumping information
about it when the VK_KHR_wayland_surface extension is present.

vulkaninfo: Describe multiple surface extensions

Describe all available surface extensions, and adjust formatting of
human-readable output to improve appearance when describing multiple
surface extensions.

vulkaninfo: Create AppDumpSurfaceExtensions

Refactored the code for setting up and dumping information about each
available surface extension.

build: Update MoltenVK commit and CMake files

Update MoltenVK commit ID in known_good.json
and modified CMake files for cube, cubepp, and
vulkaninfo to accomidate the new file structure
of the MoltenVK repo.

Change-Id: I229aa1403b5d1d47060560876b177ae93190391e

scripts: Fix windows paths in helper.cmake

Normalize (all slashes going the same direction) and escape ("\" to "\\")
paths when writing helper.cmake.  CMake was previously interpreting the
single '\' characters and whatever followed as invalid escape sequences.
Linux and macOS are unaffected.

cube: Fix fixup_bundle not set to proper cube

Changed fixup_bundle pointing at cube.app and
cubepp.app to point to vkcube.app and vkcubepp.app

Rename cube and cubepp exes to vkcube and vkcubepp

Change-Id: Iaecb0e7e21a086c8523b8cd4bc9eccd05e1cbca0

build: Update known-good for 1.1.91 header

Updated:
- `scripts/known_good.json`

Change-Id: Ia07d6af7a77e835f413d7eecfdc8ef47267744b7

cmake: Fix build crashing on make install

Fixed make install crashing when running
fixup_bundle on MacOS

Change-Id: I2aee07c8a5235fcf2fc559d6d9774c48e1792e38

layers: Bump glslang known-good

Updated:
- `scripts/known_good.json`

Change-Id: I5fe86c76b7df569b1c87566e2c32c9a2b76f8c13

scripts: Update known-good for 1.1.90 header

Change-Id: I655aaa453c5cffc928e769e3267503dd5afeb55d

scripts: Extend scope of LaTeX handling

The 1.1.90 header introduced the use of "\textrm" in `vk.xml`, requiring
changes to `scripts/vulkan_tools_helper_file_generator.py`

Change-Id: Ia43b516ac56b4e3cd82272761ef1b524eea72944

demos: Fix cubepp with latest vulkan.hpp

Add explicit casts for nullptr values passed to vulkan.hpp so the
correct function overload is selected.

repo: Remove MIR specific code.

Still need to remove MIR from scripts/common_codegen.py
and from the Vulkan wrapper.

Change-Id: Ib00cbe10b3e80878fb5e15c1e628685f33d31331

mock_icd: fix vkGetEventStatus output

VK_SUCCESS is not a valid return value for this function.  VK_EVENT_SET
is used instead.

build: Update MoltenVK known_good.json

Updated MoltenVK commit ID in known_good.json.

Change-Id: I90365c153bec14a54422fb6400f97a5e05355286

build: Update known-good for 1.1.87 header

Change-Id: Idd2f285653deb0eb70eacd5fd649e74d7723f964

scripts: Add fuchsia to platform dictionary

Change-Id: I9b7027a1dd11f5ea7d3b502c62f4edd05416fd52

Create CODE_OF_CONDUCT.md

Ignore window erases

build: Add Shannon to CI email notifications

build: Update MoltenVK known_good.json

Updated MoltenVK commit ID in known_good.json

Change-Id: Ibf3501e451b700dd44a7f595a98648f30f05e4ee

build: Sync update_deps.py

Syncing update_deps.py with changes
made in Vulkan-ValidationLayer

Change-Id: If2369787e27e37e4d595a41d3d17a8bc991d9a30

Revert LICENSE to Apache 2.0

repo: Update glslang known good

Change-Id: Ie35e275c84bc3e955a2cdc2eaafe9073e5fca044

Update LICENSE.txt

Change Apache 2.0 to Khronos Apache 2.0 CLA

cube: Use staging buffer instead of staging image

There is no guarantee that linear images are supported

Change-Id: Ie59f3bf7bfc1d77e17f1d06ffc524e886dca46a6

repo: Update known good commits for 1.1.85

Change-Id: I4b4be8815a45e8cd0c2356fc03c4dabf99b664f2

cube: Use working url to fetch glslang binaries

Change-Id: I6157cf2210c1249ce9a6bcf0ed6279597b16a53f

cube: Look for GLSLANG_INSTALL_DIR env variable

Change-Id: I2df195a1b100a51a7a5d7be827743d3dd9c5da46

build: CMakeLists.txt cleanup, part 1

This is a first pass reorganization of CMake files in this repo.
It consists primarily of dead code/variable removal, simplification,
and reformatting by latest cmake-format.

bump to cmake_minimum_required(3.4), for ccache support.

add USE_CCACHE

change 'ln -sf' to '-E create_symlink'

clarify why CMAKE_OSX_DEPLOYMENT_TARGET is pre-project().

sync FindVulkan.cmake from upstream CMake v3.8.0
  Ensure our repos use identical copy of upstream FindVulkan.cmake
  Copied from Modules/FindVulkan.cmake
  https://gitlab.kitware.com/cmake/cmake.git
  tag: v3.8.0 (commit da7833c5bb1f331162d46a2c664a443c1c641089)

change $<CONFIGURATION> to $<CONFIG>
  The $<CONFIGURATION> genexp is officially deprecated.

change CMAKE_SYSTEM_NAME to UNIX/APPLE/WIN32
  Note that UNIX evaluates true for OSX, so whenever the code intends
  "Linux only", we use (UNIX AND NOT APPLE).

change TOOLS_TARGET_FOLDER to TOOLS_HELPER_FOLDER
  for consistency with VVL and VL repos.

set Windows install prefix if needed

remove unused DisplayServer variable

remove deprecated GLSLANG_REPO_ROOT
  The *_REPO_ROOT mechanism for finding packages is deprecated.

remove extra cmake_minimum_required

change LIBVK to Vulkan::Vulkan

change PYTHON_CMD to PYTHON_EXECUTABLE

section rulers, other cosmetics

.cmake-format.py 0.4.1

reformat using cmake-format 0.4.1

Change-Id: Id8e5b26fdcf5dc0b383de94cbec75a567704a55e

cube: Fix new validation error

As of Vulkan-ValidationLayers commit 2872f4a, the layers now check that
image layouts specified in descriptors match the current image layout
state.  LAYOUT_GENERAL is not a wildcard for this matching.

Change the image layouts for the combined image sampler descriptors
from LAYOUT_GENERAL to LAYOUT_SHADER_READ_ONLY_OPTIMAL.

vulkaninfo: Add VK_KHR_8bit_storage support

Added output for VkPhysicalDevice8bitStorageFeaturesKHR
struct info

Change-Id: I906a49e6e541e95cbd3b48db7b836921d1f09f33

repo: Update known-good for Vulkan-Headers

Change-Id: I1fa27ee2564cf676418be543d6e2e2cb7f1cd906

build: Unify `update_deps.py` between repos

vulkaninfo: Remove warning msgs from json output

Messages emitted from the DbgCallback function were previously sent to
stdout. These messages are now sent to stderr and no longer affect the
validity of json output.

winrt: Delete old rtf license file if present

build: Add known-good support

Change-Id: I05fe4162054fc90edd1642ab08644f6ee7950548

repo: Update layer json versions

Change-Id: I5d36af8c799c19a18e592522b3cfb5f9b0d063ea

vulkaninfo: Fix output whitespace irregularities

vulkaninfo: Change titles to unify naming scheme

vulkaninfo: Fix html output for multiple gpus

This change addresses erroneous indentation and nesting of html output
when multiple gpus are present

script: Fix fetch_glslangvalidator.py SSL error

Changed urllib.request.urlretrieve to
urllib.request.urlopen and set to not check SSL

Change-Id: Ia4fa57ef698cc2110c9b19f0715d0eacdf9a00e7

vulkaninfo: Add device+driver UUID/LUID/NodeMask

When the VK_KHR_external_memory extension is
present, the device UUID, LUID, NodeMask, and
driverUUID are now printed in addition to the
other per-GPU VkPhysicalDevice info.

cmake: Clean up macOS install target

- Stop hardcoding CMAKE_INSTALL_PREFIX to CMAKE_BINARY_DIR so install
  products are in a predictable location
- Use install/fixup_bundle to create stand-alone app bundles for cube,
  cubepp, vulkaninfo-bundle at CMAKE_INSTALL_PREFIX
- Set INSTALL_RPATH_USE_LINK_PATH for bundle targets so fixup_bundle can
  find libraries when run from the install tree
- Remove rpath from install copy of vulkaninfo
- Remove extraneous copy of libMoltenVK.dylib from CMAKE_INSTALL_PREFIX/demos
- Update BUILD.md

icd: Update json files for 1.1.79 header

vulkaninfo: Fix warnings in maintenance3 printing

Fixed incorrect number of arguments in print statements
for VkPhysicalDeviceMaintenance3PropertiesKHR.

Change-Id: If69e6873964364f324853ff996a4e405877f771a

build: Beautify cmake files

Add .cmake-format.py
Add cmake-format docs to CONTRIBUTING.md
Minor edits to existing files

Due to a bug in cmake-format, use this script to reformat:

for i in CMakeLists.txt cube/CMakeLists.txt cube/macOS/cube/cube.cmake cube/macOS/cubepp/cubepp.cmake icd/CMakeLists.txt mac_common.cmake vulkaninfo/CMakeLists.txt vulkaninfo/macOS/vulkaninfo.cmake
do
    sed --in-place='' 's/^  *#/#/' $i
    cmake-format --in-place $i
done

Change-Id: Ie451aacd7402828ad4d62adc04048449d0ea2abe

vulkaninfo: Add VK_KHR_maintenance3 support

Added output for VkPhysicalDeviceMaintenance3Properties
struct info.

Change-Id: I8c7a21e27d0bdaf77af49881d777764f854d874c

cmake: Update CMake to check env for MoltenVK

Updated CMake files to use MOLTENVK_REPO_ROOT as an
environment variable if no CMake argument is provided.
CMake argument takes priority over environment variable.

Change-Id: I661a56f7bc6c8c43fe4d4f7013e26dde87fb41e4

build: Insert copyright in cmake files

cmake: Add Kitware copyright

Get latest Kitware copyright notice from
https://gitlab.kitware.com/cmake/cmake/blob/master/Copyright.txt @ d45d5e2

repo: Update json files from 1.1.77 to 1.1.78

docs:Fix Linux cmake instructions

Missing ".."

header: Update MacOS json files to 1.1.77 header

Change-Id: I41477ea32e73987983baee80e1b8fbd3e0f3d671

winrt: Update line wrapping in license file

doc: Update BUILD.md [skip ci]

build: Fix dest of vcxproj file copy

These vcxproj.user files are not useful anyway since the
layers dir isn't here.  But this fix at least prevents the
creation of a demos directory in the build dir.

android: Restore toolchain update in build_all