Merge "dEQP-VK.sparse_resources.shader_intrinsics*gather: fix possible of-by-one errors" into vulkan-cts-1.0.1

dEQP-VK.sparse_resources.shader_intrinsics*gather: fix possible of-by-one errors

The test seems to perform GATHER sampling from center of pixels

On un-even image sizes this may cause failures, as texcoord (being float32)
may not exactly represent pixel center, what can cause different 2x2 rectangle to
be gathered.

The fix is to bias texcoord by half a texel, so sampling is always performed
from centre of 2x2 rectangle.

Fixes following tests (for me):
dEQP-VK.sparse_resources.shader_intrinsics.2d_sparse_gather.r32i.503_137_1
dEQP-VK.sparse_resources.shader_intrinsics.2d_sparse_gather.r32i.11_37_1
dEQP-VK.sparse_resources.shader_intrinsics.2d_sparse_gather.r16i.503_137_1
dEQP-VK.sparse_resources.shader_intrinsics.2d_sparse_gather.r16i.11_37_1
dEQP-VK.sparse_resources.shader_intrinsics.2d_sparse_gather.r8i.503_137_1
dEQP-VK.sparse_resources.shader_intrinsics.2d_sparse_gather.r8i.11_37_1
dEQP-VK.sparse_resources.shader_intrinsics.2d_sparse_gather.rgba32ui.503_137_1
dEQP-VK.sparse_resources.shader_intrinsics.2d_sparse_gather.rgba32ui.11_37_1
dEQP-VK.sparse_resources.shader_intrinsics.2d_sparse_gather.rgba16ui.503_137_1
dEQP-VK.sparse_resources.shader_intrinsics.2d_sparse_gather.rgba16ui.11_37_1
dEQP-VK.sparse_resources.shader_intrinsics.2d_sparse_gather.rgba8ui.503_137_1
dEQP-VK.sparse_resources.shader_intrinsics.2d_sparse_gather.rgba8ui.11_37_1
dEQP-VK.sparse_resources.shader_intrinsics.2d_array_sparse_gather.r32i.503_137_3
dEQP-VK.sparse_resources.shader_intrinsics.2d_array_sparse_gather.r32i.11_37_3
dEQP-VK.sparse_resources.shader_intrinsics.2d_array_sparse_gather.r16i.503_137_3
dEQP-VK.sparse_resources.shader_intrinsics.2d_array_sparse_gather.r16i.11_37_3
dEQP-VK.sparse_resources.shader_intrinsics.2d_array_sparse_gather.r8i.503_137_3
dEQP-VK.sparse_resources.shader_intrinsics.2d_array_sparse_gather.r8i.11_37_3
dEQP-VK.sparse_resources.shader_intrinsics.2d_array_sparse_gather.rgba32ui.503_137_3
dEQP-VK.sparse_resources.shader_intrinsics.2d_array_sparse_gather.rgba32ui.11_37_3
dEQP-VK.sparse_resources.shader_intrinsics.2d_array_sparse_gather.rgba16ui.503_137_3
dEQP-VK.sparse_resources.shader_intrinsics.2d_array_sparse_gather.rgba16ui.11_37_3
dEQP-VK.sparse_resources.shader_intrinsics.2d_array_sparse_gather.rgba8ui.503_137_3
dEQP-VK.sparse_resources.shader_intrinsics.2d_array_sparse_gather.rgba8ui.11_37_3

Affects: dEQP-VK.sparse_resources.shader_intrinsics.*gather*

Change-Id: I218d68033e3c83d5270c54731eac3a72c6899173

Merge gerrit/vulkan-cts-1.0.0 into gerrit/vulkan-cts-1.0.1

Change-Id: I623cc43f95bd5d7c967ebbf397bc9fa214585d73

Fix alignment issues in uploading image data in ShaderExecutor

Fixes #485

Change-Id: If04aebab19fb25ef26b51b4af86e718004438d82
(cherry picked from commit 9141d25d08d53c203f8ac5a9c5937a41e1d83d02)

dEQP-VK.compute: Fix layout of storage image descriptor

Affects: dEQP-VK.compute.basic.copy_image_to_ssbo*

IMAGE_STORAGE type descriptor was updated with *_OPTIMAL layout.
This is invalid according to spec:

"Load and store operations on storage images can only
be done on images in VK_IMAGE_LAYOUT_GENERAL layout."

Submitted Vulkan-LoaderAndValidationLayers issue !1034 to
ensure this is caught in val layers.

Additionally one validation warn on 0 src stage in barrier is fixed.

Change-Id: Ie6d6875465f382c62dd9802f286de77c4c808fc2

Merge "Fix required sample counts per spec issue 478" into vulkan-cts-1.0.1

Do not require TRANSIENT usage support without other RT usages

Affects dEQP-VK.api.info.image_format_properties.*

See CTS issue #507 and spec issue #540

Change-Id: Ia3a741c1943eea034a27b9479a0a8b4cc0caa09c

Fix required sample counts per spec issue 478

Affects dEQP-VK.api.info.image_format_properties.*

See CTS issue #507 and spec issue #478

Change-Id: Id08e798f1c598fe5861234778318acdc3098a3df

Bugfix: use normalized coords in glsl sampler tests

Modified tests:
- dEQP-VK.glsl.opaque_type_indexing.sampler.*

Fixes #505

Change-Id: Ie7c7f48276b52bcce3cb8ac41914398b669fe5ae

Merge gerrit/vulkan-cts-1.0.0 into gerrit/vulkan-cts-1.0.1

Change-Id: I23cda772fc12dc44429c6a8b69cb2896b442c285

Remove use of alloc recorder from descriptor_pool.repeated_reset

Recording all allocations can take considerable amount of memory and it
appears that the tests just log the total memory usage and don't
otherwise need or check it.

Fixes #472

Change-Id: Ic8804b6781329a03e42fc481c0d430151b60ca88

Fix alignment issues in uploading image data in ShaderExecutor

Fixes #485

Change-Id: If04aebab19fb25ef26b51b4af86e718004438d82

blit_image: compute validation threshold based on I/O formats

Whereas dEQP-GLES3.functional.fbo.blit.conversion.rgb565_to_rgba32f is
passing on my Haswell machine,
dEQP-VK.api.copy_and_blit.blit_image.all_formats.r5g6b5_unorm_pack16_r32g32b32a32_sfloat_linear
is failing. It seems the criteria is different between the 2 tests. This
change puts the vulkan tests a bit closer to the GLES3 ones.

Change-Id: I4e4879270baaf707e71de030b916ea49ee0f65ba

Fix VK_IMAGE_USAGE_TRANSIENT_ATTACHMENT_BIT handling in query tests

83f90be9 added a check that was intended to make tests treat combination
of TRANSIENT_ATTACHMENT_BIT with non-RT usages invalid. However, that
change was buggy and lead the tests to treat _all_ usages of
TRANSIENT_ATTACHMENT_BIT invalid and thus skip testing them.

Change-Id: I512fbea6046a5c1a87e2ea957eebe38ac713ac87

Fix incorrect limits in device properties test

- alignment/offset type limits must be greater than 0
- nonCoherentAtomSize must be in range [1, 256] (the upper limit was
  relaxed with spec update 1.0.27
- point sizes and line widths must be positive

Modified test:
- dEQP-VK.api.info.device.properties

Fixes #502

Change-Id: I98a956d1cea741dd6d787415a0ad91178ffa0abf

Fix license header on a few Vulkan CTS files

Change-Id: If640754e1581f15e933c118338d835c18c0d7b77

Merge "Build fix for the Glx platform" into vulkan-cts-1.0.1

Build fix for the Glx platform

Change-Id: I9f3e0254c289c015f61570f71215900e0ab9bfd9

Fix clip_volume test drawing points

According to spec, "[...] if no value was written to PointSize, results
are undefined".

Modified tests:
- dEQP-VK.clipping.clip_volume.{inside,outside,depth_clamp}.point_list
- dEQP-VK.clipping.clip_volume.clipped.large_points

Change-Id: I06ed228a9e4563df15801ff402ed5eac946ac9a6

Validate KHR extensions reported by the driver

Change-Id: I30f1c86cd0b4db63b7adf0860cfbc32a0d8b75ee

Added xcb and xlib platform support for Vulkan

Fixes #390

Change-Id: I650838665f630c794f4667f518713a7ae87c8955

Merge gerrit/vulkan-cts-1.0.0 into gerrit/vulkan-cts-1.0.1

Change-Id: Idcc42afbec63aa6463be09e87dc57247275c6818

Merge "Fix barrier stage masks in synchronization tests" into vulkan-cts-1.0.1

dEQP-VK.dynamic_state tests: fix sizeof(size) bug

Change-Id: I7e90b30a88bb59bbe73fec00ecb8ce0fd92ac1f0

Merge "Remove invalid precision qualifier usage" into vulkan-cts-1.0.1

Remove invalid precision qualifier usage

dEQP-VK.pipeline.vertex_input tests were incorrectly using highp
qualifier with double types. That is not allowed, and default precision
in GLSL 4.4 shaders is highp anyway.

Fixes #458

Change-Id: I4b17b2273a8bf258bb8aec5737bd1c2a38aa49ce

Do not use pointer value after deleting it

Pointer value is undefined after 'delete'.

Change-Id: I30dab519db61073932e3e35cd87809bf89b18dfc

Fix barrier stage masks in synchronization tests

Fixes #497

Change-Id: I225344335a13b0347cf6ad34e38c3f0a181d41cd

Merge "Fixes sparse_resources.shader_intrinsics.* tests to check for device feature support." into vulkan-cts-1.0.1

Merge "Run the memory tests ahead of all other tests" into vulkan-cts-1.0.1

Fixes sparse_resources.shader_intrinsics.* tests to check for device feature support.

Change-Id: Ie3461ae4ec32b1f431b21ba7c6f453bde70834c7

Update Android build dependencies in README.md

Change-Id: If159d4b4312183063a0d55483c1383197807eac1

Run the memory tests ahead of all other tests

Other tests don't attempt to allocate that much memory and probably
don't care about fragmentation as much. This would reduce the
possibility of running into out-of-memory 32-bit issues at a later
point in the development of the CTS.

This change also updates the mustpass lists.

Change-Id: I4484e5a775fcf7efa39149f1e53eb049024a77da

Add /LARGEADDDRESSAWARE linker flag for MSVC

This allows 32-bit CTS to run to the end on Windows 32-bit platforms.

Change-Id: Iaec4efcddeb5e1248651fc2d1a1401dbb94bbaf7

Merge gerrit/vulkan-cts-1.0.0 into gerrit/vulkan-cts-1.0.1

Change-Id: I9b5283a65a5b1cb1f9d34e98a916c73c9d5000ca

Merge "Revert "Revert CUBE_COMPATIBLE flag check from "Fix required sample count tests""" into vulkan-cts-1.0.0

Revert "Revert CUBE_COMPATIBLE flag check from "Fix required sample count tests""

This reverts commit 1106b0194f7571aebfdf76766760d91be772d8e7.

Fixes #440

Change-Id: Icdb5f71cd87b970edd1bf1770b4a0c15418f600b

Fix for #474. Add Vertex ordering and Spacing to tessellation shaders.

Change-Id: I33760e0f23e6b8489e38cb93ad00e3483dc73d16

Issue 479: Remove VK_IMAGE_USAGE_DEPTH_STENCIL_ATTACHMENT_BIT

dEQP-VK.glsl.texture_gather.basic.2d.depth32f.size_pot.compare_less.clamp_to_edge_repeat unconditionally creates a VkImage with VK_FORMAT_D32_SFLOAT and VK_IMAGE_USAGE_DEPTH_STENCIL_ATTACHMENT_BIT which is optional.
However that the test does not really need VK_IMAGE_USAGE_DEPTH_STENCIL_ATTACHMENT_BIT because the image is not used as D/S attachment, but as SAMPLED and TRANSFER.

Change-Id: Ia8a9fe62b3f6e0cf4635e25b33f1f79e2538ba3f

Fix parameter order for OpVectorTimesScalar.

The spec requires that the first parameter to OpVectorTimesScalar
must be of vector type, and the second of scalar type.

Change-Id: Ib4bb41bfc237efd000e48e21c2521ec616be35a5

Merge gerrit/vulkan-cts-1.0.0 into gerrit/vulkan-cts-1.0.1

Change-Id: I70aadacf71364b934cb0340752176490b65d637d

Merge "Limit maximum allocation size in memory mapping tests" into vulkan-cts-1.0.1

Limit maximum allocation size in memory mapping tests

Fixes also ReferenceMemory::m_flushed memory usage counting when
estimating reference memory usage (size should be divided by atomSize).

Fixes #468

Change-Id: Iad754e211e8070b09254d2451e03418fc2be541b

Check for imageCubeArray feature in image and image_view tests

Fixes #466

Change-Id: I8fcf807f99eaabdff293efac001b288b67b55cc9

Update Vulkan CTS readme for 1.0.1

Change-Id: I191ebf2b7937c16d796a47534c7c6f2c27c00973

Fix image layout in pipeline barrier tests

(cherry picked from commit a80919a57ef8137712259273c18de0663cc724a0)

Change-Id: I59a6a1facd9348c949f78f808075a0377c1bfe9a

dEQP-VK.sparse_resources: Fixed VkBufferImageCopy buffer offset alignment issue

Fixes #433

Change-Id: I34a416670789979e967281d0ede406dd871862bc

Do not check include guards in iOS porting files

Change-Id: I0c67290322eb36bbc9287145cbbaea22f405d29c

Fix whitespace issues in other text files

Change-Id: Idab51b62589b9702d307022b0952a887bae263af

Fix merged <space><tab> in C/C++ files

Change-Id: Ice7c1ae46a88c941c5834f9d34fcd4822c30debc

Fix trailing whitespace and include guards

Change-Id: If9cf0f5b43ad668243033f7e7c04319c4fcd804d

Merge "Fix typos in pipeline.cache test case names" into vulkan-cts-1.0.1

Fix typos in pipeline.cache test case names

Change-Id: I7715bbd7bd21db4e6b1509d927fc087926253136

Fix race condition in qpTestLog debug asserts

Code should be holding log mutex when checking state of internal data
structure.

Change-Id: I01f8aa851407c58813880a73c13f1966fdb1a2d1

Do not log SPIR-V assembly when logging of shaders is disabled

Fixes #461

Change-Id: I64aa8eb405db85d15aea663f6daba69279d5c6c8

Merge gerrit/vulkan-cts-1.0.0 into gerrit/vulkan-cts-1.0.1

Change-Id: I64e15e4126afc4dde5dc95e6d766f95b6f04e30e

Revert CUBE_COMPATIBLE flag check from "Fix required sample count tests"

Spec does not say CUBE_COMPATIBLE flag forces
vkGetPhysicalDeviceImageFormatProperties to return sample count equal
to 1, so the flag should be irrelevant for sample count check.

Fixes #440
This partially reverts 8753a9e7528ca67b25bfabb4f03915a25721a690.

Change-Id: Ic4005cb0a651f433ff421eab671fb013561c78c4

Merge gerrit/vulkan-cts-1.0.0 into gerrit/vulkan-cts-1.0.1

Change-Id: Ice2ae0c69dc296fcf2b2e75f1647c18798387a35

Merge "Remove OpFUnord* tests from mustpass" into vulkan-cts-1.0.1

Add .editorconfig

Github uses tab width from .editorconfig when displaying code.

Change-Id: I9dcbe192ca21d7606a6ae56c747acd271562358c

Fix image layout in pipeline barrier tests

(cherry picked from commit a80919a57ef8137712259273c18de0663cc724a0)

Change-Id: Id12a18d68376056f8914eaca7774e976b493aa10

Remove OpFUnord* tests from mustpass

There's open spec issue about NaN handling in these instructions.

See #426

Change-Id: I97613857c72f80171b1c6b0cfecd6d564f1c2149

Update Android CTS mustpass

This brings Android CTS mustpass files up-to-date with latest test case
renames.

Change-Id: I436bf70fb685d538b79fb0cb86e9fc1883ee929c

Harmonize test case names

 * Avoid camel case and uppercase
 * Use more common abbreviations

Change-Id: Ica9a3936be96a72ec410d3673d5c6a2dd9e32c44

Remove mustpass XML files not used by Vulkan CTS

Change-Id: I48e43f604c296baebdad5a25c3d7a723bfddb0f0

Merge "Check and enable VK_KHR_sampler_mirror_clamp_to_edge" into vulkan-cts-1.0.1

Fix VkDeviceSize -> size_t implicit conversion warning

Change-Id: Id1742b7830e12799948bf635412974df17b9bce4

Check and enable VK_KHR_sampler_mirror_clamp_to_edge

Several tests in dEQP-VK.pipeline group assume that
VK_SAMPLER_ADDRESS_MODE_MIRROR_CLAMP_TO_EDGE mode is supported. Spec
actually requires that VK_KHR_sampler_mirror_clamp_to_edge is enabled if
that mode is used.

This change enables VK_KHR_sampler_mirror_clamp_to_edge in default
device, if supported, and adds relevant checks in the test code.

In the future we'll want to clean this up by giving test cases more
control about the device configuration they need.

Fixes #339

Change-Id: Ib264c4f2fb147271c9c09d6286f4a26dd175f493

Fix api.buffer.* tests to use sparse capable queue instead the default.

Fix accidental disabling of ASTC decoding internal test

Do not allocate large objects unnecessarily in static memory

Fix memory leaks in pipeline cache tests

Fix tcu::Resource leak in draw and dynamic_state tests

Fix failing assert in copies and blitting tests

Add note why command buffers are not freed individually

Skip testing int lookups for fp32/fp64 formats in internal tests

Overflow in float->int conversion is undefined and results vary between
tested CPUs.

Disallow decoding ASTC sRGB formats in HDR mode

sRGB is not supported in HDR mode

Fix out-of-bounds memory accesses in ASTC decoder

interpolateWeights() was accessing weights outside weight grid bounds.
This resulted in valgrind errors even though values did not contribute
to result colors due to having zero-valued weights.

Improve Vulkan null driver

 * Implement VkCommandPool/VkCommandBuffer lifetime management
correctly.

 * Return sensible values for device feature and limit queries to allow
more tests to run.

Update framework to match vulkan.h at 15aa048

Fix continue target in graphics block order tests

Fixes validation of shaders in:
    dEQP-VK.spirv_assembly.instruction.graphics.selection_block_order.out_of_order_vert
    dEQP-VK.spirv_assembly.instruction.graphics.selection_block_order.out_of_order_tessc
    dEQP-VK.spirv_assembly.instruction.graphics.selection_block_order.out_of_order_tesse
    dEQP-VK.spirv_assembly.instruction.graphics.selection_block_order.out_of_order_geom
    dEQP-VK.spirv_assembly.instruction.graphics.selection_block_order.out_of_order_frag
    dEQP-VK.spirv_assembly.instruction.graphics.switch_block_order.out_of_order_vert
    dEQP-VK.spirv_assembly.instruction.graphics.switch_block_order.out_of_order_tessc
    dEQP-VK.spirv_assembly.instruction.graphics.switch_block_order.out_of_order_tesse
    dEQP-VK.spirv_assembly.instruction.graphics.switch_block_order.out_of_order_geom
    dEQP-VK.spirv_assembly.instruction.graphics.switch_block_order.out_of_order_frag
    dEQP-VK.spirv_assembly.instruction.graphics.opphi.out_of_order_vert
    dEQP-VK.spirv_assembly.instruction.graphics.opphi.out_of_order_tessc
    dEQP-VK.spirv_assembly.instruction.graphics.opphi.out_of_order_tesse
    dEQP-VK.spirv_assembly.instruction.graphics.opphi.out_of_order_geom
    dEQP-VK.spirv_assembly.instruction.graphics.opphi.out_of_order_frag

Fixes https://gitlab.khronos.org/vulkan/vulkancts/issues/443

Fix block order vs. dominance

The OpSwitch must dominate its merge block.  It also must dominate
%case1.

Fixes https://gitlab.khronos.org/vulkan/vulkancts/issues/442

Fix SPIR-V continue construct dominance

Fix structured control flow validation problems in
dEQP-VK.spirv_assembly.instruction.compute.loop_control.*

Fixes https://gitlab.khronos.org/vulkan/vulkancts/issues/441

Fix required sample count tests

According to,

   Table 11.1. Image and image view parameter compatibility requirements

Only non-cube images with type VK_IMAGE_TYPE_2D can have sample counts
greater than one.

In addition, the following properties should hold for multisampled
images: the formats must support being used in a color or depth/stencil
attachment, or in a storage image when shaderStorageImageMultisample is
supported. Images not fitting the above descriptions may have a sample
count of one or zero (in error conditions). Align tests for sample
counts to this behavior.

(cherry picked from commit dc4227d5ec87080f5dad810b5e33ae3b3fa014a4)

ShaderRender tests: do not use copyBufferToImage on MS Images

This is forbidden by spec:

```
dstImage must have a sample count equal to VK_SAMPLE_COUNT_1_BIT
```

Instead image is cleared, and provided `textureData` ignored.

This fixes:
```
dEQP-VK.glsl.texture_functions.query.texturesamples.isampler2dms_fragment
dEQP-VK.glsl.texture_functions.query.texturesamples.isampler2dms_vertex
dEQP-VK.glsl.texture_functions.query.texturesamples.isampler2dmsarray_fragment
dEQP-VK.glsl.texture_functions.query.texturesamples.isampler2dmsarray_vertex
dEQP-VK.glsl.texture_functions.query.texturesamples.sampler2dms_fixed_fragment
dEQP-VK.glsl.texture_functions.query.texturesamples.sampler2dms_fixed_vertex
dEQP-VK.glsl.texture_functions.query.texturesamples.sampler2dms_float_fragment
dEQP-VK.glsl.texture_functions.query.texturesamples.sampler2dms_float_vertex
dEQP-VK.glsl.texture_functions.query.texturesamples.sampler2dmsarray_fixed_fragment
dEQP-VK.glsl.texture_functions.query.texturesamples.sampler2dmsarray_fixed_vertex
dEQP-VK.glsl.texture_functions.query.texturesamples.sampler2dmsarray_float_fragment
dEQP-VK.glsl.texture_functions.query.texturesamples.sampler2dmsarray_float_vertex
dEQP-VK.glsl.texture_functions.query.texturesamples.usampler2dms_fragment
dEQP-VK.glsl.texture_functions.query.texturesamples.usampler2dms_vertex
dEQP-VK.glsl.texture_functions.query.texturesamples.usampler2dmsarray_fragment
dEQP-VK.glsl.texture_functions.query.texturesamples.usampler2dmsarray_vertex
```

Fixes issue #430

dEQP-VK.image: group some test cases together

Strip debug info from SPIR-V binaries

This requires updating glslang to fix ID remapping problem in SPIR-V
remapper.

Fixes #391

Validation fix in dEQP-VK.api.command_buffers.*

- use correct stage masks with vkCmdWaitEvents
- set reserved flags to 0 in VkQueryPoolCreateInfo
- use non-zero srcStageMask with vkCmdPipelineBarrier

Use pipeline cache to speed up synchronization.op tests

Sparse resources fixes

1. Do not allow to request more queues than available
2. Allow to reuse queue if it supports more than one type of operations

dEQP-VK.draw: fix minor issues

- colorWriteMask validation error
- minDepthBounds

Merge "Don't rely on framebuffer contents after eglSwapBuffers"

Update Android CTS mustpass

Change-Id: Icb869468a701a7ef664087ff401f80344719134f

Don't rely on framebuffer contents after eglSwapBuffers

Bug 27129258

Change-Id: Ic1c58d37c665dc9c35daa2b05ca40ecb969ad5dc

Merge upstream vulkan-cts-1.0-dev into AOSP master

DO NOT MERGE revert CTS integration build changes

Reverts CTS integration makefiles to match marshmallow-cts-dev

Change-Id: I2b99024cee587bc823003df5aef0b555c265e381

Add new files to Android.mk

Fix device mismatch bug by creating allocator for the sparse context too

Only use single-mipped MS images when testing textureSamples()

This MR changes two aspects of the
dEQP-VK.glsl.texture_functions.query.texturesamples tests:

 * Only MS images are now bound to sampled image bindings.
 * As per recent spec clarification, MS images should always be
single-mipped.

Fixes #432

pipeline.spec_constant: Remove the data_size tests

These tests are invalid.  The size of the provided data *must* match the
size of the constant being specialized in the SPIR-V.

See also: https://gitlab.khronos.org/vulkan/vulkan/issues/345