Revert "Fix location of vk_layer_settings.txt on Android"

This reverts commit 3d22f33e862d4bed46edbd5ee948f4ebe9c84a6f.

Fix location of vk_layer_settings.txt on Android

Commit 969ab0680dd434f0 ('api_dump: Support config file and logging on Android')
added a custom location for vk_layer_settings.txt on Android. However, commit
750837a5dedf524c81c (Merge branch 'trunk') ended up partly overwriting these changes,
breaking the functionality.

As a result, using the config file on Android hasn't worked since October 2016. This commit
restores the changes in the original commit to fix this.

layers: Add missed entries for some nv extensions

Nothing uses these, so I didn't notice they weren't set up.

layers: Don't generate T* = reinterpret_cast(T*) new T()

No need for the cast, these types already line up.

layers: use Unwrap in UO codegen

layers: use WrapNew in UO codegen

layers: Move UO helpers to header

layers: UO: rework GetDisplayPlaneCapabilitiesKHR

Not sure what this was trying to achieve -- we should have seen every
VkDisplayMode as a return from GetDisplayModePropertiesKHR?

layers: Rework UO GetDisplayModePropertiesKHR

Similar overkill to function fixed in earlier commit

layers: Mark bogus looking reverse lookup in UO

layers: Fix UO GetPhysicalDeviceDisplayPlanePropertiesKHR

This was doing odd things. pProperties is an output-only array. Its
contents are undefined before the call, so we shouldn't be trying to
unwrap the handles in there. We don't even need the safe_* here.

layers: Use helpers in most swapchain UO impls

layers: Write helpers for UO Unwrap+WrapNew operations

Allows us to concentrate all the ID generation and cast insanity in one
place.

layers: Store instance data in UO::layer_data rather than gpu handle

layers: Use InstanceExtensions support in UO

layers: hold UO dispatch tables by value in layer_data

Trims out some allocations; fixes leak of dispatch tables

layers: UO: split device and instance data

layers: Use InstanceExtensions in PV

layers: Move CV instance extension tracking to helper

layers: Move PV device extension handling into helper

layers: Move CV::DeviceExtensions into helper

layers: Make device extension enables consistent

I want to get rid of the "_enable" suffix everywhere to allow for a
second instance of the same structure to represent the /supported/ set
for better diagnostics.

layers: Alias devExts in checkDeviceRegisterExtension

layers: Remove redundant check for extension enable in fptr

layers: Move unrelated data out of devExts type

layers: clang-format only, no code changes

clang-format of buffer_validation.cpp, layer_validation_tests.cpp

Change-Id: I98c581f6b343929f7812bf012084ad0abbb5a690

layers: Add VUs 01202..01207

Added checks for 6 CopyImage VUs to buffer validation. Updated copy
size tests to provoke each new VU.

Change-Id: Ice7821b7c1b8aa0c90e68e8386c56bd021a616ed

build: set CMAKE_INSTALL_PREFIX explicitly

Recent changes to upstream CMakeFiles changed the default install
location.  Explicitly specify our desired install location.

Changed for Linux only; Windows was already doing this.

Change-Id: Id71945d426ac8f0a445dca3855bfa037870aaed9

layers: Special case threading and VK_NULL_HANDLE

If a parameter is VK_NULL_HANDLE, then it needs no use count.

Fixes #1712

external: Update external reference ID for glslang

Change-Id: Ied5cecf65eaafcab9274376fea452934bda3f69b

layers: Migrate surface support check to core

- fix memory leak of `pQueueFamilyIndexSupport`
- remove vestige of queue family support check

layers: Convert capability handling to data

Replace the huge switch + duplicated calls with a map of ptr-to-member.

layers: Fix validation of compute pipelines

skip handling was inverted for this case.

layers: Remove check for imageExtent == currentExtent

The condition tested in this check is only valid on platforms where
the swapchain dimensions must match the surface dimensions (e.g. Windows,
Xlib). In such cases, the surface *must* report minExtent == maxExtent ==
currentExtent, and this check would already be handled by the code directly
above the deleted check.

Platforms like Android allow the swapchain's imageExtent to be anywhere
within the surface minExtent/maxExtent range; if this does not match the
surface dimensions, it will be scaled accordingly.

layers: Invert old 'pass' checks for shader validation

Make this consistent with everything else, which uses 'skip'.

cube: Throttle rendering rather than presentation

It is currently impossible to reliably throttle presentation per the
Vulkan spec.

The previous code was relying on fences returned by
vkAcquireNextImageKHR() to throttle.  The only information this fence
holds is whether it is possible to render to that image since the *last
time* it was presented, which could have happened several frames ago.

Instead, we can throttle the rendering by passing a fence to
vkQueueSubmit().

The previous code (only the cube.c version) was using a fence there to
synchronize a vkMapMemory() in demo_update_data_buffer(), which doesn't
seem necessary.

Before this commit, we were effectively throttling to the number of
frames in the swapchain rather than on FRAME_LAG.

In the FIFO present mode, this could schedule too much work in the
presentation channel (since we have to account for VBLANK events)
and thus causing undesired side effects, such as stutters when trying
to move the cube window on a desktop, which is I assume why the
throttle code was added in the first place.

layers: Don't insist on memory dependency for present

When transitioning to/from VK_IMAGE_LAYOUT_PRESENT_SRC_KHR we don't need
to insist on VK_ACCESS_MEMORY_READ_BIT. On the transition TO, it's
automatically handled; in the transition FROM, it's a WAR hazard rather
than RAW/WAW so only the execution dependency is needed.

Fixes #1717.

layers: Rewrite VerifyQueueStateToSeq to be less wasteful

When we have the same queue appearing multiple times in the dependency
tree of the operations between queue->seq and seq, we would scrub that
part of the submission list recursively, potentially to a great depth,
eventually blowing the stack.

[This used to work fine when it was done as part of RetireWorkOnQueue, as
we'd have brought queue->seq up as we went, and the recursive call would
end up doing no work.]

Instead, explicitly track where we're up to on each queue as well as
where we'd like to be, and use an explicit stack of work to do rather
than using the call stack.

Fixes crash in VkLayerTests.LongSemaphoreChain with 32k semaphores.

V2: Explain what's going on
V3: Tweak for windows build

external_revisions: bump spirv-tools

CV needs the new bitpattern interface for specialization.

V2: also bump for android
V3: also bump headers
V4: also bump shaderc for android build

layers: Change size of D32_SFLOAT_S8_UINT

Change the size of the other format with 'optional' bytes, to be
consistent with the X8_D24_UNORM_PACK32 change. We'll now assume
the optional bytes when calculating buffer/image sizes for these
formats.

Change-Id: I7c194b85611795cc680f76de04513c3b45a88c4b

layers: Change size of X8_D24_UNORM_PACK32

Change-Id: If049e2afaea83f0a705a7031888d4278784be49d

tests: Update vkCreateFramebuffer tests

- Add tests for width/height/layers of zero.
- Fix one VU check that wasn't using its corresponding error code.
- Migrate string-based tests to error-code-based variants.
- database update.

layers: Add check for VkFramebuffer dimensions of zero

layers: Guard checks of pQueueFamilyIndices

Only examine pQueueFamilyIndices if sharingMode is VK_SHARING_MODE_CONCURRENT
This fixes a crash in CTS when sharing mode was VK_SHARING_MODE_EXCLUSIVE
and queueFamilyIndexCount was VK_QUEUE_FAMILY_IGNORED

Change-Id: I72b1f140e8949e711723e98bdaead15655ab07a5

tests:Test VALIDATION_ERROR_00302

InvalidBarriers test was already checking case VALIDATION_ERROR_00302
but it was doing so based on setting COLOR aspect bit.

Add another case where depth bit is excluded and verify that error is
correctly triggered.

layers:Add barrier imageAspect check for DS image

Fixes #1703

Add validation check to make sure that both depth & stencil aspects of
DS images are transitioned in image memory barrier.

demo/smoke Android build update: use latest cmake with Android Studio

winsdk: Make copyright signature editable

Change-Id: Ie28b5d1e5ba7fbd727122399ef1f35d1eb54305a

layers: Remove GLOBAL_CB_NODE::imageSubresourceMap

This served no purpose. We can get the same result in the one place that
matters, by consulting GLOBAL_CB_NODE::imageLayoutMap

loader: gh1681 - Restrict error in JSON

Restrict the loader's JSON errors when reading ICD manifest files
to only return an overall error if there are no valid drivers.

Change-Id: I10edb1cdc7e4db9cfdc0b3595416f0614ed22867

ci: Add myself to mail recipients

layers: Fix some function name passing

- This was constructing a temporary std::string for no reason
- Missing () as other similar cases use.

layers: Drop unused param for SetSparseMemBinding

There is only one function that can call this, and we didn't care
anyway.

layers: fix typo in vkCmdExecuteCommands error strings

layers: Drop dead device param to CreatePassDAG

layers: Simplify slightly

layers: Fix format utils SSCALED/USCALED descriptions

These aren't about colorspaces -- comment was previously mangled from
sRGB case.

layers: Inline deleteCommandBuffers into sole caller

layers: Fix more VkBool32 vs bool

layers: Fix 'secondary' typo

layers: Fix more signedness noise

layers: Use format utils rather than open-coded list in SC

layers: Drop layer_data from some leaf funcs that dont use it

layers: Fix some signedness mess

layers: Fix some confused VkBool32 vs bool

These aren't the same type!

layers: Simplify DESCRIPTOR_POOL_STATE

Just use safe_* here rather than copying things manually.

layers: Remove dead update_cmd_buf_and_mem_references

No callers.

layers:Validate display_timing swapchainCount

For the VK_GOOGLE_display_timing extension validate that the
swapchainCount in the VkPresentTimesInfoGOOGLE extension struct matches
the swapchainCount in the VkPresentInfoKHR struct.

Update unique ID database.

layers:Validate objects for display_timing

Update object_tracker validation for swapchain objects in new functions
for VK_GOOGLE_display_timing extension.
Add in missing Error IDs and update database.

layers: Fix enum use in GetStateStructPtrFromObject

Everything around this was updated to use the new enums, but not this
function. Mostly works by accident since many of these coincide.

layers: Inline VkCommandBuffer overload of clear_cmd_buf_...

This was silly. On the very next line, we called GetCBNode again.

layers: Drop dead string conversion for command types

No callers.

layers: Banish 'skip_call' once and forever

Except for swapchain.cpp, which is undergoing a merge into CV.

Change-Id: I8b18518ef662a9a11a808ef3dce1b8f60d858fb3

tests: Update database to make validation happy

Change-Id: I2002d23d09208e4154fe06a864ae9c8d78f0bc49

layers: Force all device extensions disabled initially

Should really fix device initialization so we don't need to do this
explicitly.

layers: Final object-type cleanup and loose ends

Fixed a few DebugReport object type casts and a couple of missed
spots with internal object type usage.

Change-Id: Iaa8bdcf60d5ec89e1da64227f2b7367ce6c64dc1

vulkaninfo: Fix various entries in VkPhysicalDeviceFeatures output

The VkPhysicalDeviceFeatures dump missed the multiViewport and
samplerAnisotropy fields. The alphaToOne field was at the wrong position
(compared to the official definition in the Vulkan specification).

The fields {vertex,fragment}StoresAndAtomics as well as
shaderTessellationAndGeometryPointSize had the misleading and incorrect
names {vertex,tessellation,geometry}SideEffects.

layers: Change obj type conversion routine name

Change-Id: I3b4208ca3777ac46b2c5ab2839c268ed4381a2ac

layers: Remove unused string helper from CV

Change-Id: I36daf405a6c8170d7ee61f5a07bdad045f8d6304

layers: Switch CV over to layer object types

Convert internal uses of VkDebugReportObjectTypeEXT for the core-
validation layer to use the internal layer-povided object-type.

Also modified tests for changed error text.

Change-Id: If080f4bf452ead0c6fecc648309bfd95b27fc0ce

scripts: Make object string conversion array const

Prevents unused var warnings by changing to be a constant.

Change-Id: Ie8f6cc837dd589074ae2278de34d4bd0a3e66236

layers: Use UNKNOWN object type instead zero-cast

Change-Id: I36c175e78d7a33667f1485ed0e53f1dadab62a31

layers: remove PIPELINE_STATE::vertexAttributeDescriptions

Nobody used this.

layers: Simplify ValidateCmd

Nested !=, == was needlessly weird.

layers: Remove GLOBAL_CB_NODE::numCmds

Nobody used this.

layers: slim down draw counting

We used to care how many draws there were. Now we just care whether
there were /no/ draws (in one place)

layers: Fix naming of enable flags for swapchain exts

Should be consistent with the actual extension names

layers: Tidy up tracking of enabled device extensions in CV

- These are all guaranteed initialized.
- Convert code to data.

scripts:Updated Validation IDs for spec 1.0.48

This was much nicer after that last update. Cmd line was:
python spec.py -update -remap 2937-3217:557-375:669-2819:739-536:
751-2106,4:1263-2956,5:2237-3104:2906-3203:2960-3222

Updated parser to BeautifulSoup so we can now use the HTML spec
directly from online again. You may have to install BeautifulSoup
python module separately to be able to run.

Had to manually import errors 911 and 912 which are only present in
the non-extension spec. This is an issue we're planning to fix when
unique ids are integrated into the spec which may acutally finally
happen within a month.

docs: Update Android documentation

android: Add script that installs APKs

android: Add test_APK script for layer validation

This is a script we've been using internally for a while, no reason
not to make it public.  It installs our layer validation tests, runs
them, and gathers output.  Improvements are welcome!

Usage:

  ./test_APK.sh -s <serial number> -p <plaform name> -f <gtest_filter>

android: Add build script that includes APK packaging

header: Update to 1.0.48 Vulkan version

- updated include/vulkan/vulkan.h
- updated scripts/vk.xml
- updated layers json files
- updated tests json files
- updated vulkan.hpp

Change-Id: I90407d181ad6a4652e1d1f210ee9b9d21c7857b0

layers:Remove old INFO message

INFO message in vkCreateDescriptorPool() is not needed and it was
returning a validation error if the callback returned "true" which is
just wrong. Killing the code altogether.

layers: Special-case UO generation of QueuePresentKHR

VkPresentInfo::pResults is an /output/ array. The generated code makes a
copy on the way in (pointlessly, but other clients of safe_* might find
this reasonable), but neglects to copy it back on the way out.

Lift the current code generated for QueuePresentKHR back into the
special cases, and add the correct post-call copy.

Fixes #1670.

V3: Just open-code the loop. I don't want to turn off MSVC's fussy mode,
and it's convinced both std::copy and std::copy_n are unsafe with all
raw pointers.

Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers:Add image layout validation for descriptors

This change adds validation to make sure that an image layout at the
time the image is used in a descriptor matches the layout that was
given when the descriptor was updated.

Because image view covers a range of mip levels, loop over each level
and verify layouts one at a time.

Also Updated a number of validate functions to use cont ptr params for
data that they aren't changing.

scripts: Convert ugly case statment to nice array

Since this was codegen'd, we can just create a static array for the
object type conversion and do away with all of the switch statement
mess.

Change-Id: I5fb20778d64f0675afa9c1563e2d44d22d5b4a02

layers: Switch to codegen'd obj_type string helper

Updated LVTs for error string changes as well.

Change-Id: I5e0da92a9859e8ada7374af42a21a3f8f7629fdd

layers: Use gen'd object types in object_tracker

Change-Id: I2166407bffcbf037d9b7898d91c674ed60fc95ba