Merge branch 'fix-missing-capabilities' into 'vulkan-cts-1.0'

Add missing capabilities in assembly tests

Add 'OpCapability ClipDistance' and 'OpCapability CullDistance' when
they are required.

See merge request !449

Add missing capabilities in assembly tests

Add 'OpCapability ClipDistance' and 'OpCapability CullDistance' when
they are required.

Fix compiler warning in render pass tests

Merge branch '306_use_transfer_dst' into 'vulkan-cts-1.0'

Images used in dynamic tests need TRANSFER_DST usage flags

Fixes #306

See merge request !442

Merge branch '294_format_properties_fix' into 'vulkan-cts-1.0'

Fixes format_properties test with required formats from Vulkan 1.0 spec

Fixes #294

See merge request !446

Fixes format_properties test with required formats from Vulkan 1.0 spec

Related to issue #294

Merge branch '308-fix-ssbo-array-descriptors' into 'vulkan-cts-1.0'

Fix ssbo array descriptors

This is a partial solution to #308.  It fixes the ssbo tests to properly use array descriptors.

See merge request !443

SSBOLayoutCase: Make proper use of array descriptors

Previously, the SSBO tests were using arrays of blocks in the shader but
declaring them as individual descriptors in the descriptor set layout.
This is in violation of the spec and was causing issues on some
implementations.

As a side-effect of this change, we now update most of the bindings in a
single update.  The Vulkan spec allows descriptor writes that go over the
bounds of the current binding to spill into the next as long as the
bindings have the same type.  This makes updating substantially simpler
especially when we have bindings with different numbers of descriptors.

Merge branch 'jekstrand-linux-lib-name' into 'vulkan-cts-1.0'

Use the correct library name on Linux

Shortly before release, the official library name was changed from libvulkan-1.so to libvulkan.so.1.  We should be loading the right one.

See merge request !444

Merge branch '304-disable-robust-buffer-access' into 'vulkan-cts-1.0'

Disable robustBufferAccess in the default VkDevice to avoid performance issues on some HW

Addresses #304

See merge request !437

Use the correct library name on Linux

Merge branch 'fix-memory-barrier-tests' into 'vulkan-cts-1.0'

Pipeline barrier test fixes

- Set image layout to undefined after memory has been written by host
  or through buffer.
- Fix CacheState transitivity by removing bit not operation.
- Require bound buffer for render operations.

Change-Id: I7c44f5be51ef5e23cc3304fc90f8970ebe135a52

See merge request !431

Pipeline barrier test fixes.

- Set image layout to undefined after memory has been written by host
  or through buffer.
- Fix CacheState transitivity by removing bit not operation.
- Require bound buffer for render operations.
- Skip correct stages in CacheState::barrier.

Change-Id: I7c44f5be51ef5e23cc3304fc90f8970ebe135a52

Merge branch 'memory-barrier-fix' into 'vulkan-cts-1.0'

Fix issues with memory barrier tests.

There are multiple issues currently in the dEQP-VK.memory.pipeline_barrier.* tests.

I found 3 ocasions where the values provided to srcAccessMask and dstAccessMask are the other way around.
In particular, after a transfer operation the barrier responsible for flushing the output of the transfer writes uses srcAccessMask=0 and dstAccessMask=VK_ACCESS_TRANSFER_WRITE_BIT, when actually VK_ACCESS_TRANSFER_WRITE_BIT should have been set in srcAccessMask not in dstAccessMask.

See merge request !421

Issue #306

Images used in dynamic tests need TRANSFER_DST usage flags.

Change-Id: I74581d038a56914582c79084d0ccec9d06c67d87

Fix issues with memory barrier tests.

Fix type conversion warnings

Bump #version to 310 in vktApiSmokeTests.cpp

fixes #272

Update glslang and spirv-tools in fetch_sources.py to public versions

Disable robustBufferAccess in the default VkDevice to avoid performance issues on some HW

Merge branch 'aosp-copyright' into 'vulkan-cts-1.0'

Add AOSP copyright to tests ported from dEQP GLES tests

See merge request !436

Merge branch '305-fix-access-flags' into 'vulkan-cts-1.0'

Fix for dEQP-VK.image.load_store* lacks setting access flags for image memory barrier

Fixes #305

See merge request !438

dEQP-VK.image.load_store* lacks setting access flags for image memory barrier

Add AOSP copyright to tests ported from dEQP GLES tests

Merge branch '259-fix-arm-gcc-issue' into 'vulkan-cts-1.0'

Image load/store: remove DynArray

As per description in #259 this code posed a problem in some situations, so this is an attempt to sidestep an issue altogether.

Fixes #259

See merge request !419

Merge branch 'fix-290-291-ds-advanced' into 'vulkan-cts-1.0'

Fix dEQP-VK.dynamic_state.ds_state.stencil_params_advanced #290, #291

- Add missing barriers to ensure fragment loads wait
- Add LOAD_OP_LOAD for stencil aspect as the data is always needed
- Add TRANSFER_DST usage bit for both color and depth-stencil, as images are target of vmCmdClear* operations

Change-Id: Ia01304fb5287698ffbbd493cc800ef78a820a82c

See merge request !432

Merge branch '289-renderpass-formats' into 'vulkan-cts-1.0'

Round quad coordinates in dEQP-VK.renderpass.formats.* to pixel edges.

Round the (-0.5,-0.5) - (0.5,0.5) quad coordinates to pixel edges, to fix #289.

See merge request !430

Round quad coordinates in dEQP-VK.renderpass.formats.* to pixel edges.

Fixes #289

Merge branch '295-fix-glsl-discard-missing-barriers' into 'vulkan-cts-1.0'

Fix for dEQP-VK.glsl.discard.* missing barriers

ShaderRender is missing the appropriate barriers before and after cmdCopyImageToBuffer

See merge request !434

Fix dEQP-VK.dynamic_state.ds_state.stencil_params_advanced

- Add missing barriers to ensure fragment loads for color and ds wait
- Add LOAD_OP_LOAD for stencil aspect as the data is always needed
- Add TRANSFER_DST usage bit for both color and depth-stencil, as images are target of vmCmdClear* operations

Change-Id: Ia01304fb5287698ffbbd493cc800ef78a820a82c

Remove header snippets from Vulkan API test plan

ShaderRender is missing the appropriate barriers before and after cmdCopyImageToBuffer

Add top-level README

Merge branch 'remove-confidentiality-clause' into 'vulkan-cts-1.0'

Remove confidentiality clause from Vulkan CTS license

See merge request !433

Remove confidentiality clause from Vulkan CTS license

Merge branch '274-remove-round-mustpass' into 'vulkan-cts-1.0'

Remove rounding tests from the mustpass list

See merge request !429

Merge branch '283-reset-query-pool-before-use' into 'vulkan-cts-1.0'

Add missing commands for reseting queryPool prior to use

Fixes #283

See merge request !428

Merge branch 'origin/280-move_vkCmdCopyQueryPoolResults_outside_renderpass' into 'vulkan-cts-1.0'

#280: End the renderpass before trying to copy the query results

End the renderpass before trying to copy the query results,
as vkCmdCopyQueryPoolResults is only allowed outside.

Fixes #280.

See merge request !424

Remove rounding tests from the mustpass list

Compute, image: remove DynArray

Merge branch '287-puniq' into 'vulkan-cts-1.0'

Make buffer-pointer types unique.  Fixes #287

See merge request !425

Add missing commands for reseting queryPool prior to use

Make buffer-pointer types unique.

End the renderpass before trying to copy the query results, as
vkCmdCopyQueryPoolResults is only allowed outside.

Change-Id: I07cdd6c056017418f5080bdabb56746daeea73e7

Merge branch '277-log-original-assembly-of-SPIRV-assembly-tests' into 'master'

Add original assembly of the SPIR-V assembly tests to the log

fixes #277

Old MR: !416 (I had to create a new MR one because of renaming the branch.)

See merge request !420

Add original assembly of the SPIR-V assembly tests to the log

fixes #277

Merge branch '278-proposed_texcoord_offset_is_too_low_fix' into 'vulkan-cts-1.0'

Fixes #278

Merge branch '276-fix-incorrect-buffer-usage-flag' into 'vulkan-cts-1.0'

dEQP-VK.compute.basic.shared_atomic_op* fix incorrect usage flag

Fixes #276

See merge request !418

dEQP-VK.compute.basic.shared_atomic_op* fix incorrect usage flag

Increase texcoord offset used by pipeline.sampler conformance tests from 0.001 to 0.002

Merge branch 'gstrm' into 'vulkan-cts-1.0'

Eliminate Stream decorations.

They require the `GeometryStreams` capability, which I'm not sure we should require of every implementation.

See merge request !406

Merge branch 'jbolz_createBuffer_memoryType' into 'vulkan-cts-1.0'

Change buffer tests to use queried memory type index

Previously this test assumed/required that memory type 0 was capable of sparse.

See merge request !407

Merge branch '266-dEQP-VK.renderpass-Test-Fixes' into 'vulkan-cts-1.0'

dEQP-VK.renderpass-Test-Fixes

* cmdPipelineBarrier should not be called with srcStageMask and dstStageMask set to zero.
* Adding code to initialize image attachment usage.

Fixes issue #266

See merge request !408

Merge branch '271-uninitialized-size' into 'vulkan-cts-1.0'

Don't use uninitialized variables in dEQP-VK.api.copy_and_blit.buffer_to_image

The buffer to image test was using the uninitialized src image extent
for initializing the src buffer and the uninitialized dst buffer size
for initializing the dst image.

Fixes #271

See merge request !414

Merge branch '273-vertex-input-stride-fix' into 'vulkan-cts-1.0'

dEQP-VK.pipeline.vertex_input.*: corrected vertex input binding strides.

Consecutive strides did not result in aligned accesses to the vertex
attributes.

See merge request !415

dEQP-VK.pipeline.vertex_input.*: corrected vertex input binding strides.

Consecutive strides did not result in aligned accesses to the vertex
attributes.

Get sampled and storage usage flags from format properties.

Don't use uninitialized variables in dEQP-VK.api.copy_and_blit.buffer_to_image

The buffer to image test was using the uninitialized src image extent
for initializing the src buffer and the uninitialized dst buffer size
for initializing the dst image.

Merge branch '268-command-buffers-memory-leak-fix' into 'master'

268 dEQP-VK.api.command_buffers test fix.

Fixes the memory leaks in the CommandBuffers tests. (vulkan/vulkancts#268)

See merge request !412

Merge branch '267-spirv_assembly.instruction-Fixes' into 'master'

267 spirv assembly.instruction fixes

1- missing ArrayStride: Array type in uniform/storage block must have their stride explicitly specified, otherwise the stride infomation in block layout is lost. Affects:

    spirv_assembly.instruction.compute.opcopymemory.*

2- OpSelect's condition operand doesn't follow spec:

    OpSelect's condition operand should have same component number as OpSelect's result type.

    SPIR-V spec p110:
    OpSelect
    ...
    Condition must be a scalar or vector of Boolean type. It must have the same number of components as Result Type.

This addresses issue #267

See merge request !410

Removed correct code by mistake, adding that back and back out the originally intended code.

CommandPool creations in some tests are fixed.

Apply formating fixes
Remove ArrayStride code.

OpSelect's condition operand didn't follow spec
OpSelect's condition operand should have same component number as OpSelect's result type.

SPIR-V spec p110:
OpSelect
...
Condition must be a scalar or vector of Boolean type. It must have the
same number of components as Result Type.

spirv_assembly.instruction.compute.opcopymemory.* Add 2 missed ArrayStride, array type in uniform/storage block must have their stride explicitly specified, otherwise the stride infomation in block layout is lost.

Fix Renderpass tests cmdPipelineBarrier should not be called with srcStageMask and dstStageMask set to zero. Adding code to initialize image attachment usage.

Change buffer tests to use queried memory type index

Eliminate Stream decorations.

Fix capability for Stream decorations.

Add CTS submission package verification script

Minor formatting fixes

Remove cmdline prefixes to make copy-pasting commands easier

Clarify submission package contents

Fix some formatting, document how to include multiple products

Fix Vulkan README.md formatting issues

Merge branch 'issue_260' into 'vulkan-cts-1.0'

Improvements to submission package documentation

Fixes #260

- documentation and waiver requirement for non-porting changes
- describe waiver process
- describe the submission package

See merge request !404

Merge branch '242-blend-mustpass-exclusion' into 'vulkan-cts-1.0'

Excluded blend tests where propagated errors may be large

Addresses issue #242

See merge request !403

Fix typo

Improvements to submission package documentation

Fixes !260

- documentation and waiver requirement for non-porting changes
- describe waiver process
- describe the submission package

mustpass: Issue 242: Excluded blend tests where propagated errors may be large

Merge branch 'fix-undefined-trailing-data' into 'vulkan-cts-1.0'

Fix undefined trailing data in pipeline barrier tests.

Set memory properly undefined when memory layout is changed and set image
contents undefined when buffer is written.

See merge request !402

Merge branch '258-required-format-support' into 'vulkan-cts-1.0'

Adjust Vulkan format feature support requirements

Fixes #258

See merge request !401

Merge branch '250-limit-pipeline-count' into 'vulkan-cts-1.0'

Limit number of pipelines in object_management.max_concurrent

See #250

See merge request !400

Merge branch 'jekstrand_fix_blend_test_swizzling' into 'vulkan-cts-1.0'

Blend Test: Do not swizzle the ImageView

Some formats are weird. The A2R10G10B10_UNORM_PACK32 format, for example, is
stored out of order with respect to the name. From 29.3.1 Format definitions of
the spec:

A four-component, 32-bit packed unsigned normalized format that has a
2-bit A component in bits 30..31, a 10-bit R component in bits 20..29, a
10-bit G component in bits 10..19, and a 10-bit B component in bits
0..9.

The existing code will use this fact to determine that the underlying
organization for the format is BGRA, and as such will introduce swizzles in the
view. However, this swizzling is not the desired effect used by the reference
renderer, and this resulted in a miscompare (reverse RED AND BLUE channels) for
potentially any format who underlying storage order is reverse of its logical
order, AND, whose underlying storage order is not in RGBA (swizzle free) order.

If you switch the format to A2G10B10R10, the underlying organization is
RGBA and the resulting swizzle is the identity swizzle, it still works.
Also, if you add B8G8R8A8_UNORM to the list of formats, the newly added
tests all fail.

See merge request !398

Merge branch 'awoloszyn-fix-block-decoration' into 'vulkan-cts-1.0'

Fixes block order and decoration group tests.

Fixes #257

Round the input values to integer numbers so that the computations
never have to round.

The tests in question only perform addition and subtraction, and all
well within the representable range of a float. (-1500 -> 1500)

See merge request !397

Merge branch '255-limit-mem-allocs' into 'vulkan-cts-1.0'

Limit number of memory allocations to 4000.

Limit memory allocation count to 4000 in vulkan memory
allocation tests. Fixes #255

See merge request !399

Merge branch 'tzlatinski_memory_allocaiton_size_fix' into 'master'

CTS Memory tests fix invalid allocation sizes

allocateRandom() routine sometime requests enormous sizes
because of incorrect random size calculation logic.

Within the following formula:
m_heap.size / MAX_MEMORY_USAGE_DIV) - m_usage - 1ull
If m_heap.size / MAX_MEMORY_USAGE_DIV) - m_usage is "0"
the result is negative, but since the number is unsigned
max takes the max of 1 and the (unsigned long long)-1.

The change, makes the comparison signed and also introduces
explicit test checks on the allocation sizes.

See merge request !389

Fix undefined trailing data in pipeline barrier tests.

Set memory properly undefined when memory layout is changed and set image
contents undefined when buffer is written.

Change-Id: I688c014244f5ba5d9c602d9cdcad8a63c66b5357

Improve Vulkan CTS README

 * Add details how to select Android ABI at install time

 * Add note about using git cherry-pick -x when cherry-picking bugfixes.

 * Add initial conformance criteria section

Adjust Vulkan format feature support requirements

Fixes #258

Limit number of pipelines in object_management.max_concurrent

See #250

Limit number of memory allocations to 4000.

Limit memory allocation count to 4000 in vulkan memory
allocation tests. Fixes issue #255.

Change-Id: I9a1bc129556ce44e1d3e3e8c474c6dc668cea814

Fixes block order and decoration group tests.

Fixes #257

Round the input values to integer numbers so that the computations
never have to round.

The tests in question only perform addition and subtraction, and all
well within the representable range of a float.

Merge branch '249-fix-draw-mapped-memory-flushes' into 'vulkan-cts-1.0'

Fix flushing of index and indirect command bufs in draw tests

Fixes #249

See merge request !393

Merge branch '252-fix-imageformatproperties-validation' into 'vulkan-cts-1.0'

Fix VkImageFormatProperties validation

Fixes #252

See merge request !391

Merge branch '254-fix-shaderrendercase-format-check' into 'vulkan-cts-1.0'

Fix format support check in ShaderRenderCase

Fixes #254

See merge request !392

Merge branch '256-fix-renderpass-build-warnings' into 'vulkan-cts-1.0'

fix renderpass test's build warnings, fixes #256

See merge request !394

Blend Test: Do not swizzle the ImageView

Some formats are weird. The A2R10G10B10_UNORM_PACK32 format, for example, is
stored out of order with respect to the name. From 29.3.1 Format definitions of
the spec:

A four-component, 32-bit packed unsigned normalized format that has a
2-bit A component in bits 30..31, a 10-bit R component in bits 20..29, a
10-bit G component in bits 10..19, and a 10-bit B component in bits
0..9.

The existing code will use this fact to determine that the underlying
organization for the format is BGRA, and as such will introduce swizzles in the
view. However, this swizzling is not the desired effect used by the reference
renderer, and this resulted in a miscompare (reverse RED AND BLUE channels) for
potentially any format who underlying storage order is reverse of its logical
order, AND, whose underlying storage order is not in RGBA (swizzle free) order.

If you switch the format to A2G10B10R10, the underlying organization is
RGBA and the resulting swizzle is the identity swizzle, it still works.
Also, if you add B8G8R8A8_UNORM to the list of formats, the newly added
tests all fail.

fix renderpass test's build warnings, fixes #256

CTS Memory tests fix invalid allocation sizes

allocateRandom() routine sometime requests enormous sizes
because of incorrect random size calculation logic.

Within the following formula:
m_heap.size / MAX_MEMORY_USAGE_DIV) - m_usage - 1ull
If m_heap.size / MAX_MEMORY_USAGE_DIV) - m_usage is "0"
the result is negative, but since the number is unsigned
max takes the max of 1 and the (unsigned long long)-1.

The change, makes the comparison signed and also introduces
explicit test checks on the allocation sizes.