Merge pull request #16 from jchen10/image_layout

Fix Pipeline barrier test

Update sample conformance statement with latest release tag

Turn on well defined signed integer overflow

clang and gcc assume that signed integers never overflow.
This leads to unexpected bugs when optimizer removes bound checks etc.

Bug: 27703278
Bug: 22715185
Change-Id: Iab4c884fa190d05c0ac9236d01eeeb190e765e69
(cherry picked from commit ae6c306e39080de59a8535cf1797b27951dcaa6c)

Add Android NDK r11 toolchain as new default

New NDK r11 toolchain fixes libc++ on x86 enabling glslang support in
Android builds.

Change-Id: I5141b97f3152cfb1420a116cae40d671477ed254
(cherry picked from commit de7fb14fc30a8dd60e0215c333950b6a996c825a)

Update glslang to (7b3096a) to include bitfield functions precision fix

This fixes issue #385

Fix Pipeline barrier test

Set the old layout of color attachment image to VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL

Update glslang to (af4592) - include proper RelaxedPrecision handling

Without this all build-in functions get RelaxedPrecision decoration in fragment shader, making _many_ tests fail.

Remove instances of pDynamicState->dynamicStateCount = 0

Specification explicitly disallows using
VkPipelineDynamicStateCreateInfo with dynamicStateCount = 0 when
creating pipelines. Instead pDynamicState must be set to NULL.

Change-Id: Iff9e2f10e0080ffae0f6698aff342132f148b41d
(cherry picked from commit 86a8ae9e0d61bbfe1f1e273fdbb026369bdb5f29)

ShaderRender tests: do not use images in UNDEFINED layout

Just transition them at the beginning of renderpass

(cherry picked from commit 9f60392d15906ca26e0ae2ec97fe3e2c40f69a1f)

Merge branch '357-ports-to-release-1.0' into 'vulkan-cts-1.0'

Check existing tests with validation layers - partial port to 1.0 branch

This is a port of subset of MRs listed in #357 - the ones that fix usage of images in UNDEFINED layout.

We need this one because of implementation change on our side.

The contents of this MR equals !526 + !516 + !510, all of these are in MERGED state.

See merge request !534

Revert to original boilerplate in Vulkan CTS license appendix

Validation fixes in dEQP-VK.pipeline

- Corrected out of range minDepthBounds value
- Added missing image layout transitions
- timestamp, cache tests: pass data through geometry shader
- cache test: use correct initialLayout in attachment when starting
  a second render pass

(cherry picked from commit 1d3585c5ec71cf90d6519913db6a01ef2c0f840c)

Conflicts:
external/vulkancts/modules/vulkan/pipeline/vktPipelineCacheTests.cpp

Bug Fix: Corrected barriers in the tests dEQP-VK.compute

(cherry picked from commit 9b3e984f996ac8d7cc18c16e466a149eef2397e3)

Bug Fix: Corrected barrier in the tests dEQP-VK.ubo

(cherry picked from commit ee29316060b9858e85c5aabb2a6c563c6d34826a)

Fix unused parameter warning in vktDrawIndirectTest.cpp

Merge branch '355-fix-tesc-tese-executors' into 'vulkan-cts-1.0'

Fix a few issues in tessellation shader executors

 * Off-by-one buffer overflow in TessEvaluationExecutor

 * Set patchControlPoints to sensible values

Fixes #355

See merge request !519

Merge branch 'scygan-draw-tests-split-firstInstance-1.0' into 'vulkan-cts-1.0'

deqp-vk-draw: split indirect_draw tests into groups depending on `drawIndirectFirstInstance` optional feature

Fixes #337 on 1.0 branch, as an alternative to !507

See merge request !512

deqp-vk-draw: split indirect_draw tests into groups depending on `drawIndirectFirstInstance` optional feature

Remove initialization check for deviceName

Initialization checking code doesn't currently handle null-terminated
strings correctly. Implementation is not required to initialize bytes
following the terminator 0 byte in string fields.

Fix VkPhysicalDeviceProperties init validation

dEQP-VK.api.info.device.properties was using incorrect member offset
table to verify initialization of VkPhysicalDeviceProperties.

Fixes #367

Improve VK_MAKE_VERSION and VK_BIT macros

VK_MAKE_VERSION was causing int->uint conversion warnings.

Put arguments inside parenthesis to make evaluation unambiguous.

Change-Id: Ia47efdf77789905f4c45a551c08eb0ff3496c556
(cherry picked from commit dd99172aee72493b3e508dc8990e7eac95a40f8f)

Merge branch '361-memory-properties' into 'vulkan-cts-1.0'

Fix dEQP-VK.api.info.device.memory_properties

Includes both fix for missing propertyFlags 0 as well as improved logging.

Fixes #361

See merge request !518

Fix a few issues in tessellation shader executors

 * Off-by-one buffer overflow in TessEvaluationExecutor

 * Set patchControlPoints to sensible values

Change-Id: Ie94cf02c2fccd5e018a0e4de02c956f0d2abc6b0

Add 0 to list of valid memory property flags

Fixes #361

Improve logging in dEQP-VK.api.info.device.memory_properties

Change-Id: I70927510e3dce6dbc05cc0f49f24f5d331f49412
(cherry picked from commit cdf80ba37ef186bb2d8fc5a3a97dd633e6a67d85)

Merge branch 'extend-query-validation' into 'vulkan-cts-1.0'

Extend feature and limit query validation

Manual cherry-pick of extended validation code in `vktApiFeatureInfo.cpp` from the development to the release branch.

Includes framework API update to 1.0.5 as the new code requires `VK_FORMAT_FEATURE_SAMPLED_IMAGE_FILTER_LINEAR_BIT`

See merge request !499

Extend feature and limit query validation

This change manually cherry-picks extended feature and limit query
validation code from development branch to the release branch.

Merge branch '340-cube-corner-mustpass' into 'vulkan-cts-1.0'

Remove test affected by cube corner texel selection issue

See #340

See merge request !505

Merge branch '352-missed-pipeline-barrier' into 'vulkan-cts-1.0'

Add missing memory barrier to draw tests

Fixes #352

See merge request !493

Merge branch '350-LOD-precision-for-snorm-cts-1.0' into 'vulkan-cts-1.0'

Use larger threshold for EAC formats in sampler tests

Fixes #350

See merge request !490

Add missing memory barrier to draw tests.

Fixes #352

Use larger threshold for EAC formats in sampler tests

Fixes #350

Remove test affected by cube corner texel selection issue

 Fix int->float conversion warning

Change-Id: I3a8c01a9223b1b261b2cf162ab0ef9babb8bbb45
(cherry picked from commit de443fdd0e0f38f5fd8bd02aa27e6931063a39e3)

Update Vulkan framework to API 1.0.5

Change-Id: I4e469b9384701303f96826c25a51e623316719dc
(cherry picked from commit 931d1df9bc5ac16200e2e63dfce17dcbbbb35d36)

Merge branch 'change-license-rel' into 'vulkan-cts-1.0'

Change license in release branch

See merge request !481

Change Vulkan CTS license to Apache 2

Fix malformed copyright boxes

Merge branch '341-fix-invocations' into 'vulkan-cts-1.0'

Remove invalid Invocations=0 from geometry shaders.

All of these geometry shaders are default, so they do not need any
Invocations decoration.

Fixeds #341

See merge request !477

Remove invalid Invocations=0 from geometry shaders.

All of these geometry shaders are default, so they do not need any
Invocations decoration.

Merge branch 'side-effect-of-322_api-patch-version-test' into 'vulkan-cts-1.0'

Add a test for checking random values of PATCH component of VKApplicationInfo.apiVersion

This is a continuation of #322, where negative test for checking random patch versions is removed.

Here, a positive test is added instead to ensure that vkCreateInstande does NOT fail if PATCH version is unrecognized, weird or just 0.

See merge request !462

Merge branch '329-fix-undef-tests' into 'vulkan-cts-1.0'

Fix createOpUndef tests.

If we get a Nan from the OpFMul %f32 %undef %c_f32_0 then zero that
out as well.

Fixes #329

See merge request !473

vktApiDeviceInitializationTests patch iteration added

Merge branch 'jbreton-buffer-test-heap-limit' into 'vulkan-cts-1.0'

Fix createBuffer tests to clamp the buffer size based on the size of the selected heap

Right now, those tests clamp the allocation size based on the size of heap 0.
This is incorrect if the allocation is not done on heap 0.
In order to get the first compatible heap, we need to create the buffer and then call getBufferMemoryRequirements to query the requirement.

Then if we detect if the size need to be clamped we will recreate another buffer with the clamped size.

Fixes #326

See merge request !464

clamp allocation size based on heap size

Fix createOpUndef tests.

If we get a Nan from the OpFMul %f32 %undef %c_f32_0 then zero that
out as well.

Merge branch '325-spirv-assembly-avoid-rounding' into 'vulkan-cts-1.0'

dEQP-VK.spirv_assembly.instruction.compute.*: avoid platform-dependent rounding

Avoid platform-dependent rounding in tests for OpCopyObjectGroup, OpPhiGroup, SelectionControlGroup and FunctionControlGroup.

See merge request !461

Merge branch 'issue-307' into 'vulkan-cts-1.0'

Issue 307: Add new --deqp-log-flush command line option

Fixes issue #307

It defaults to enabled. When disabled it skips calling fflush() for every XML token and line of log.

See merge request !441

dEQP-VK.spirv_assembly.instruction.compute.*: avoid platform-dependent rounding in tests for OpCopyObjectGroup, OpPhiGroup, SelectionControlGroup and FunctionControlGroup.

Merge branch '323_timestamp_buffer_overflow_fix' into 'vulkan-cts-1.0'

Fixes buffer overflow in pipeline timestamp tests

Fixes #323

See merge request !459

Merge branch '310-update-glslang' into 'vulkan-cts-1.0'

Update glslang to include "SPV: Use heuristic to avoid geometry multi-streams"

Fixes issue #310

See merge request !463

Merge branch '308-fix-opaque-type-indexing-array-descriptors' into 'vulkan-cts-1.0'

ShaderExecutor: use descriptor arrays when uniform/buffer arrays are used in the…

… shader code. Another partial solution to #308.

See merge request !455

Merge branch 'spec-176-api-remove-patch-version-check' into 'vulkan-cts-1.0'

Issue 322: dEQP-VK.api.device_init.create_instance_invalid_api_version - remove patch version check

In the spirit of spec bug #176, the implementation should not emit error on unrecognized PATCH component of VkApplicationInfo.apiVersion.

This makes implementation compatible with all header patch versions that may be used by the application.

This fixes #322

See merge request !454

binding_model: Respect the StoresAndAtomics feature bits

This takes care of #320 for the binding_model tests

Update glslang to include "SPV: Use heuristic to avoid geometry multi-streams"

Fixes buffer overflow in pipeline timestamp tests

Fixes #323

Merge branch '319_memory_blit_scale_fix' into 'vulkan-cts-1.0'

Fixes ImageBlitFromImage tests scaling blit verification code

Fixes #319

See merge request !456

Fixes ImageBlitFromImage tests scaling blit verification code

Fixes #319

Merge branch 'update-glslang' into 'vulkan-cts-1.0'

Update glslang

See merge request !460

Update glslang

Merge branch '316-dEQP-VK.api-system-memory-limitations' into 'vulkan-cts-1.0'

316-dEQP-VK.api-system-memory-limitations

Reducing the number of command buffers allocated due to system memory limitations on 32-bit systems.

This would allow us to pass these tests on 32-bit systems, which offer less system memory to the driver causing out-of-memory issues. The following are the affected tests:

* dEQP-VK.api.object_management.max_concurrent.command_buffer_primary
* dEQP-VK.api.object_management.max_concurrent.command_buffer_secondary
* dEQP-VK.api.command_buffers.allocate_many_primary
* dEQP-VK.api.command_buffers.allocate_many_secondary

Ultimately, we need to modify the test to factor in the system memory available or let a test pass if the driver reports out-of-memory errors gracefully.

See merge request !457

Reducing the number of command buffers allocated due to system memory limitations of 32-bit systems

ShaderExecutor: use descriptor arrays when uniform/buffer arrays are used in the shader code. Another partial solution to #308.

Issue 322: dEQP-VK.api.device_init.create_instance_invalid_api_version - remove patch version check

In the spirit of spec bug #176, the implementation should not emit error on
unrecognized PATCH component of VkApplicationInfo.apiVersion.

This makes implementation compatible with all header patch versions
that may be used by the application.

Merge branch 'update-glslang' into 'vulkan-cts-1.0'

Update glslang to latest revision

See merge request !448

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.

Update glslang to latest revision

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

Add new --deqp-log-flush command line option

It defaults to enabled. When disabled it skips calling fflush() for
every XML token and line of log.

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