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

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.

Fix flushing of index and indirect command bufs in draw tests

Fixes #249

Drop a few checks from VkImageFormatProperties validation

The spec around vkGetPhysicalDeviceImageFormatProperties() has to be clarified
before these checks can be made in the test.

Fixes #252

Fix format support check in ShaderRenderCase

Fixes #254

Add first draft of conformance submission package description

Change-Id: If6581ee5ae7b497f5f7cec7bce0e42598860a5bc

Check in Vulkan CTS 1.0.0 RC mustpass

Merge in changes from Khronos repository

Fix accidental library name change

Merge branch '238-fix-compressed-sliced-3d' into 'master'

Check for sliced 3D compressed texture support in pipeline.{image,image_view,sampler} tests

This MR also enables sliced 3D ASTC cases as some implementations may support that. There are no real 3D ASTC formats in core Vulkan currently.

See merge request !374

Merge "Add dEQP-VK.info tests" into vulkan

Merge "Add vk::Platform::describePlatform()" into vulkan

Add dEQP-VK.info tests

Purpose of dEQP-VK.info tests is to automatically collect details
required for conformance submissions.

Change-Id: I86302dc24c95e30510faf3ff7480dcbf2899f037

Add vk::Platform::describePlatform()

describePlatform() should log platform-specific architecture and version
information.

Implementation provided for Android, Win32, and X11/unix.

Change-Id: I0babdf2765f22d820f19c2f4c964b885e3395afc

Merge "Add basic tests for vkGetPhysicalDeviceImageFormatProperties()" into vulkan

Merge "Add tests for vkGetPhysicalDeviceFormatProperties()" into vulkan

Check compressed 3D format support in PipelineImageSamplingInstance

Fixes #238

Change-Id: I7496a27b445e98b341bdc10c9da76aff5ad996cd

Enable sliced 3D ASTC cases in pipeline.{image,image_view,sampler}

Support for sliced 3D ASTC textures is optional.

Merge in changes from Khronos repository

Remove invalid check from tcu::CompressedTexture

No real 3D ASTC block formats are supported, but sliced 3D textures work
as expected.

Clear message log at the beginning of async debug tests.
am: 13fa7290b1

* commit '13fa7290b19863e6a300428ed26e65d74bbb03fe':
  Clear message log at the beginning of async debug tests.

Add basic tests for vkGetPhysicalDeviceImageFormatProperties()

Change-Id: If1008832e6c28ed815cc93ac890cc80f3a567b1f

Update glslang

Fixes #244

Add tests for vkGetPhysicalDeviceFormatProperties()

Change-Id: I608af954332cc9029d636bc49de5b1a1900f304e

Check vkGetPhysicalDeviceImageFormatProperties() result value