Remove extra semicolons

Fix warnings for Clang builds with -Wextra-semi, -Wextra-semi-stmt and
-Wc++98-compat-extra-semi.

Components: Framework, OpenGL, OpenGL ES, Vulkan

VK-GL-CTS issue: 3080

Change-Id: I1caa81b65415a1e94797c6df48fc8a4539e9635a

Fix warning in gcc-8

The code for setting up the output buffer was duplicated into each of
the cases for different input types, and the compiler doesn't understand
that DE_ASSERT marks an unreachable path. Changing to using standard
assert fixes the warning:

vktShaderFConvertTests.cpp:541:19: error: ‘outputBufferSizeInfo::memorySizeBytes’
may be used uninitialized in this function [-Werror=maybe-uninitialized]
   541 |  BufferSizeInfo   outputBufferSizeInfo;
       |                   ^~~~~~~~~~~~~~~~~~~~

Component: Vulkan
Affects: dEQP-VK.glsl.builtin.precision_fconvert.*

Change-Id: Icf6302166d8bce7c3ae1447973c895c34465a726

Enable Vulkan matrix determinant and inverse precision tests

A set of new matrix3x3/matrix4x4 determinant and inverse precision
tests for Amber.

Components: Vulkan

VK-GL-CTS Issue: 2878

New tests:
dEQP-VK.glsl.builtin.precision.determinant.highp.mat3
dEQP-VK.glsl.builtin.precision.inverse.highp.mat3
dEQP-VK.glsl.builtin.precision_fp16_storage16b.determinant.compute.mat3
dEQP-VK.glsl.builtin.precision_fp16_storage16b.inverse.compute.mat3
dEQP-VK.glsl.builtin.precision_fp16_storage32b.determinant.compute.mat3
dEQP-VK.glsl.builtin.precision_fp16_storage32b.inverse.compute.mat3
dEQP-VK.glsl.builtin.precision_double.determinant.compute.mat3
dEQP-VK.glsl.builtin.precision_double.inverse.compute.mat3
dEQP-VK.glsl.builtin.precision.determinant.highp.mat4
dEQP-VK.glsl.builtin.precision.inverse.highp.mat4
dEQP-VK.glsl.builtin.precision_fp16_storage16b.determinant.compute.mat4
dEQP-VK.glsl.builtin.precision_fp16_storage16b.inverse.compute.mat4
dEQP-VK.glsl.builtin.precision_fp16_storage32b.determinant.compute.mat4
dEQP-VK.glsl.builtin.precision_fp16_storage32b.inverse.compute.mat4
dEQP-VK.glsl.builtin.precision_double.determinant.compute.mat4
dEQP-VK.glsl.builtin.precision_double.inverse.compute.mat4

Change-Id: I5d748c1ecbaf4e8444a2d09eac8da60bfa54271c

Do not overflow when copy data

As `vertexDataSize` is aligned with `nonCoherentAtomSize`, it could be
bigger than the actual size of m_data.

This is actually causing a heap-buffer overflow when executing in our
Rpi4 Vulkan driver.

Affected tests:
dEQP-VK.spirv_assembly.instruction.graphics.cross_stage.basic_type.flat

Components: Vulkan

VK-GL-CTS issue: 3062

Change-Id: I6b1f449e9f0a42740adb051ed037ff911dac763a

Issue in serial storage address computation

When serial storage and src/dst buildTypes are different, serial storage address computation is not correct. In the beginning of the test case, serial storage buildType is either set to host or device according to test case being run (cpu-built or gpu-built).
In the 2nd phase of this test case (deserialize all from the previous step to a new top-level AS) VkCopyMemoryToAccelerationStructureInfoKHR is filled with srcAddress according to testCase buildType, and dst acceleration structure is device.
In this stage, if testCase builtType is device, the serialStorage address computation is correct. However, if testCase buildType is host then the issue arises. The address is computed according to host, although device address is needed.

That is, for serial storage address computation: buildType of the acceleration structure (src or dst) should be used rather than serial storage buildType which is initially set according to test case being run.

Affects:
dEQP-VK.ray_tracing_pipeline.acceleration_structures.header_bottom_address.*

Components: Vulkan
VK-GL-CTS issue: 3028

Change-Id: If8c0387862ce6f1db00fc61016dd0e602ff72ba8

Add coverage for sampling a cubemap that has been rendered to

This test checks that a cubemap will be sampled correctly if its
contents have been modified by a draw call.

VK-GL-CTS issue: 2506

New Tests:
dEQP-VK.image.sample_cubemap.write_face_0

Components: Vulkan

Change-Id: I1ee62842d6c952b5628822fe49ee6a91c26ead04

Merge vk-gl-cts/vulkan-cts-1.2.7 into vk-gl-cts/master

Change-Id: Ieb90dde42cb20ab43cebd0f2b5d3ad4091c3b57a

Merge vk-gl-cts/vulkan-cts-1.2.6 into vk-gl-cts/vulkan-cts-1.2.7

Change-Id: Ieaff21c7c6228a4507325afe13ec2a3807b345cb

Check support in synchronization operation tests

Synchronization tests were creating pipelines before checking for support,
causing unnecessary use of CPU time when encountering an operation
later on that was not supported.

Affects:

dEQP-VK.synchronization.op.*

Components: Vulkan

VK-GL-CTS issue: 2731

Change-Id: Ia83afde4767ced1549a1b0573cdc9e8af8526870

Read stencil aspect from image in DEPTH_ATTACHMENT_STENCIL_READ_ONLY_OPTIMAL  layout

The test was using sampler to read depth aspect from image in
combined depth+stencil format and in
VK_IMAGE_LAYOUT_DEPTH_ATTACHMENT_STENCIL_READ_ONLY_OPTIMAL
layout - what is invalid.

Instead, the test now reads (and tests) stencil aspect of this
image. As the test code already supported testing stencil formats,
just a few conditions are changed/added to follow the stencil path
if VK_IMAGE_LAYOUT_DEPTH_ATTACHMENT_STENCIL_READ_ONLY_OPTIMAL
is used.

VK-GL-CTS Issue: 3048

Components: Vulkan

Affects: dEQP-VK.synchronization2.none_stage.*

Change-Id: I918a5cdad83bcd25b2f8b6cee642b56218c2b7fe

Optimize image clearing tests

This CL optimizes image clearing tests.
The tests did a lot redundant calculation for each pixel in the verify stage.
This CL moves those calculations to outside of the pixel verify function.

Affects:

dEQP-VK.api.image_clearing.*

Components: Vulkan

VK-GL-CTS issue: 2933

Change-Id: Ie3403c591c9f084115a13cc78e0a66c54c48817c

Ensure that destroying old swapchain has no effect

There should be no crash when recreating Vulkan swapchain with
old swapchain and later calling vkDestroySwapchainKHR on old swapchain.

New tests:

dEQP-VK.wsi.*.swapchain.destroy.old_swapchain

Components: Vulkan

VK-GL-CTS issue: 2935

Change-Id: I7f03e55767ececec8a6a70ef553f0676d3a3c616

Conservative rasterization with SampleMask

Additional test coverage for conservative rasterization Conservative
rasterization used in conjunction with various multisampling setup
including FullyCoveredEXT built-in, minSampleShading, postDepthCoverage
and SampleMask.

Components: Vulkan

VK-GL-CTS issue: 2868

Affects:
dEQP-VK.pipeline.multisample.conservative_with_full_coverage.*

Change-Id: I8c2858b81cfdf27e622acf74e273db9014499c9c

Update Vulkan headers

Components: Vulkan

Change-Id: Iefdf1bc052c5f9c63bdf9062c01c96a4c11a92c6

Implement getProcAddress for surfaceless platform

Currently this virtual function is not overridden, causing a crash
whenever a test calls getProcAddress in a surfaceless context.

Affects:
KHR-GL3{0,1,2}.transform_feedback.draw_xfb_instanced_test
KHR-GL3{0,1,2}.transform_feedback.draw_xfb_stream_instanced_test

Components: Framework

VK-GL-CTS issue: 3021

Change-Id: Idb29b5ca9d70c5f3be6a32fdf2c827d5d7591050

Fix for the various multisample tests with GL 4.5

Affects:
dEQP-GLES31.functional.shaders.sample_variables.*
dEQP-GLES31.functional.sample_shading.*
dEQP-GLES31.functional.shaders.multisample_interpolation.*
dEQP-GL45.functional.shaders.sample_variables.*
dEQP-GL45.functional.sample_shading.*
dEQP-GL45.functional.shaders.multisample_interpolation.*

Components: AOSP
VK-GL-CTS issue: 3014

Signed-off-by: Tapani Pälli <tapani.palli@intel.com>
Change-Id: I6e4297a0558a31a1dc2258da7434d3fcc1900b73

Fix to multisample texture test shaders with GL 4.5

Affects:
dEQP-GL45.functional.texture.multisample.*

Components: AOSP
Issue: 3016

Signed-off-by: Tapani Pälli <tapani.palli@intel.com>
Change-Id: Icf29df0c1b8cb5306ebf4a9444c63d8e1afa2d7c

Fix for the common function tests with GL 4.5

Affects:
dEQP-GL45.functional.shaders.builtin_functions.common.*

Components: AOSP
VK-GL-CTS issue: 3015

Signed-off-by: Tapani Pälli <tapani.palli@intel.com>
Change-Id: Icf1343225a2dab84f70e2bc986f556c247454b4c

Fix partial_image_npot_diff_format_clear fail

The tests added in fffc9484834599de9eb5559205cec7cf0105379b /
VK-GL-CTS Issue: 2678 are missing a pipeline barrier between
the new clearing operation the following transfer operation
targeting the same resource.

Components: Vulkan

Affects:
dEQP-VK.api.copy_and_blit.*.image_to_image.simple_tests.
partial_image_npot_diff_format_clear

VK-GL-CTS issue: 3056

Change-Id: I3edf4d98e697e97c4b7934238cdbfc86f2d48632

Remove shaders.arrays from the Khronos mustpass

Components: OpenGL

VK-GL-CTS issue: 2540

Affects: KHR-GLES3.shaders.arrays.*

Change-Id: I291c26f1105bd72ca16e4057f23bb2188aed6dc4

Fix Android system memory requirements

This CL changes how the available system memory is defined in Android.
The actual available system memory size is now used instead of
determining the available system memory by the CDD.

Affects:

dEQP-VK.api.buffer.suballocation.*
dEQP-VK.api.buffer.dedicated_alloc.*
dEQP-VK.api.object_management.*
dEQP-VK.memory.allocation.*
dEQP-VK.memory.device_group_allocation.*
dEQP-VK.pipeline.render_to_image.*.small.*
dEQP-VK.pipeline.render_to_image.*.huge.*

Components: Framework, Vulkan

VK-GL-CTS issue: 2621

Change-Id: I92fd157ec3928313ac92c29e6cbb3413c64a5096

Test multiViewport with different depth ranges

This change extends existing tests to also cover creation of
viewports with different depth ranges, and make sure that
depth clipping/clamping works correctly across the different
viewports.

Components: Vulkan

VK-GL-CTS issue: 2500

Affects:
dEQP-VK.draw.depth_clamp.*

Change-Id: Ifa76daa1b53f04e48871a5efe07f5d3863066ca1

Test OpImageQueryLod when fwidth(UV) == 0

Tests verify that lambda returned by OpImageQueryLod
is -22 or lower when fwidth(UV) == 0.

Components: Vulkan

VK-GL-CTS issue: 3027

Affets:
dEQP-VK.glsl.texture_functions.query.texturequerylod.*

Change-Id: I879024646c7a62d6cef7cb4b46a154b5e4750f36

Merge vk-gl-cts/github-master into vk-gl-cts/master

Change-Id: I9bf5cb55d51036216ffb8aafde67062b6c860c1f

Merge pull request #276 from Ella-0/master

Test behaviour of color write enable with colorWriteMask

Allow Vulkan CTS 1.2.7.1

Affects: dEQP-VK.api.driver_properties.properties

Components: Vulkan

Change-Id: Ic313a2b15d7c2be5c5b1f95aa189dc6bf1adbf70

Add tests for VK_EXT_shader_atomic_float2

New tests:
dEQP-VK.glsl.atomic_operations.*float16*
dEQP-VK.glsl.atomic_operations.min_float*
dEQP-VK.glsl.atomic_operations.max_float*
dEQP-VK.image.atomic_operations.min*r32f*
dEQP-VK.image.atomic_operations.max*r32f*

Affects:

dEQP-VK.glsl.atomic_operations.*float*
dEQP-VK.image.atomic_operations.add*r32f*
dEQP-VK.image.atomic_operations.exchange*r32f*

Components: Vulkan

VK-GL-CTS issue: 2566

Change-Id: I7462cc74e799c2245407220fa9db7e5b659f86f4
(cherry picked from commit ca4e5aff66bccbbe88935c10e2c55a3147c1fc03)
(cherry picked from commit 218eb73c4f30e720bd19a61704f5ad6ebcd73d48)

Tests for VK_KHR_present_id and VK_KHR_present_wait

Add tests for VK_KHR_present_id and VK_KHR_present_wait. This commit
also refactors some code so it can be reused in several places, hence
marking all WSI tests as affected.

Affected tests:
dEQP-VK.info.device_mandatory_features
dEQP-VK.wsi.*

New tests:
dEQP-VK.wsi.*.present_id_wait.*

Components: Vulkan
VK-GL-CTS issue: 2098
VK-GL-CTS issue: 2099

Change-Id: I1c95fbf031d651773a9fff1c10a23038d73f6e53
(cherry picked from commit 8bf210fd592883b3adebb57c3d6cc49188d1758e)
(cherry picked from commit b7a8721e2ffb9153283b434658463daac7ed1313)

Update glslang, SPIR-V Tools, headers

Components: Framework

Change-Id: Ie259277466b22ba816843c362da272a4284fcb93
(cherry picked from commit c1e67a6d8689bdaad205ee0287ea4b490141c4d3)

Update Vulkan headers

Components: Vulkan

Change-Id: Id59fb9125cac9068d68a8e5f219c3dc53a093ad0
(cherry picked from commit ab44aa0a0a3a2c1ec5937a71c4cb8936f22fb247)

Update Vulkan headers

Components: Vulkan

Change-Id: I6000ca21cc306130dd546887e68d1cecd4dc0e1f
(cherry picked from commit 9aa91388484350028ff7c7c4c063ee87146070b6)

Merge vk-gl-cts/vulkan-cts-1.2.6 into vk-gl-cts/vulkan-cts-1.2.7

Change-Id: I34e25155c4d2fc3a37f8699507a1330f86ef78a9

Merge vk-gl-cts/vulkan-cts-1.2.5 into vk-gl-cts/vulkan-cts-1.2.6

Change-Id: I8d78e920b4007ea7f47e66d64852693dfed22003

Merge vk-gl-cts/vulkan-cts-1.2.4 into vk-gl-cts/vulkan-cts-1.2.5

Change-Id: Id1af44b78cbe4ba82cc76c656e0a22421ac20073

Merge vk-gl-cts/vulkan-cts-1.2.3 into vk-gl-cts/vulkan-cts-1.2.4

Change-Id: I13ff1ac57c064dc46bbc7c555ba1fed07e04080c

Merge vk-gl-cts/vulkan-cts-1.2.2 into vk-gl-cts/vulkan-cts-1.2.3

Change-Id: Iab13500b9aaa9b6da0e99ea9d93fe9cabc66da33

Do proper checking for the presence of Xorg/XCB

In XcbDisplay::hasDisplay() API, hasthe incorrect
assumption that the xcb_connection_t is always valid,
if its value isn't NULL. XcbDisplay::hasDisplay() must
ensure that there are no connection errors by
using the xcb_connection_has_error() API.
Failure to do so would result in incorrectly trying
to exercise the XCB related test cases against
other than XCB WSIs, such as Wayland, for example.

Components: Vulkan
Affected tests: dEQP-VK.wsi.*
                dEQP-VK.image.swapchain_mutable.*
VK-GL-CTS issue: 3037

Change-Id: Id722b3a3b3d82161ff00096518a026701b291751

The existence of a DRM device is not mandatory

The VK_EXT_physical_device_drm spec states:
DRM defines multiple device node types. Each physical device may
have one primary node and one render node associated.
Physical devices may have no primary node (e.g. if the device does
not have a display subsystem), may have no render node (e.g. if it is
a software rendering engine), or may have neither (e.g. if it is a
software rendering engine without a display subsystem).

According to the above statement, a DRM device does not have to
always be present for a device to be compliant with the
VK_EXT_physical_device_drm spec.

Components: Vulkan

VK-GL-CTS Issue: 3029

Affects: dEQP-VK.api.device_drm_properties.drm_files_exist

Change-Id: I6d48b15d142650ee0db2dafd5e1b7f3c30e7a837

Add missing items to android cts mustpass list

Some tests were missing from Android vk-master-2021-03-01 test list.
This CL adds those tests to the list.

Components: Vulkan

VK-GL-CTS issue: 3050

Change-Id: I973508a169e70d203dd465e9f22c41405c1c24a7

Fix bug in compressed PixelStorageModes tests

Compressed pixel storage modes are tested by using the extension
ARB_compressed_texture_pixel_storage.
ARB_compressed_texture_pixel_storage is a core extension since
OpenGL 4.2. Therefore, the cases testing this behaviour need to
check for its support in GL context versions before 4.2.
Without this check drivers that do not provide this extension,
will fail this test instead of being marked as not supported.
This CL adds the mentioned extension checking.

Affects:
KHR-GL33.pixelstoragemodes.compressedteximage2d.*
KHR-GL33.pixelstoragemodes.compressedteximage3d.*

Components: OpenGL

VK-GL-CTS issue: 3020

Change-Id: Ib9a4edeb57995582c07aa11c7db11c6e7bbe1984

Work around bogus GCC 9 warnings

Bogus warning can be seen with this example:

   #include <stddef.h>

   long broken()
   {
      struct
      {
         long a, b;
      } value;
      for (size_t i = 0; i < sizeof(long); ++i)
      {
         ((char *)&value.a)[i] = (char)0xde;
         ((char *)&value.b)[i] = (char)0xde;
      }
      return value.b;
   }

Compiled with -O3 -Warray-bounds GCC 9 warns:

   In function 'long int broken()':
   cc1plus: warning: 'void* __builtin_memset(void*, int, long unsigned int)' offset [9, 16] from the object at 'value' is out of the bounds of referenced subobject 'broken()::<unnamed struct>::a' with type 'long int' at offset 0 [-Warray-bounds]

It looks like GCC is recognising the loop as two memsets, merging them,
then forgetting that it has done this.

It seems this can be worked around by just using memset rather than
implementing it manually, which is also simpler and produces better
code.

The example compiles without warning on GCC 10+ (you can try on
https://godbolt.org/), so presumably this bug has been fixed in later
versions of gcc.

Change-Id: I09799d6dd833b1fba8608cddaa36a32e555cf861

Fix a GraphicsFuzz coverage test

This commit simplifies the result verification at the end of cov-apfloat-module-small-number.

Affects: dEQP-VK.graphicsfuzz.cov-apfloat-module-small-number

Components: Vulkan

VK-GL-CTS issue: 3035

Change-Id: Ia0e0347880cbddc2433cd3867c96a09645c5a35d

Increase error threshold for some geometry shader tests

Components: Vulkan
VK-GL-CTS issue: 3034

Affected tests:
dEQP-VK.geometry.*

Change-Id: I5ed440019ab45f6f31787d199695630d06bb2a51

Fix variable pointer tests for geometry stages on tilers

On tilers we may execute any geometry stage more than once, not just
the vertex shader.

Components: Vulkan
VK-GL-CTS issue: 3033

Affected tests:
dEQP-VK.spirv_assembly.instruction.graphics.variable_pointers.graphics.writes_single_buffer_*

Change-Id: I3ee5db4aa8176d09b10720d7683338ef8e8a8098

Fix DRM modifier list tests

* dEQP-VK.drm_format_modifiers.list_modifiers.* tests now check every
  supported modifier instead of returning prematurely if any of the
  modifiers was not supported for the image usage flags the tests uses.

* dEQP-VK.drm_format_modifiers.create_list_modifiers.* tests check
  modifier compatibility first before attempting to use compatible
  modifier lists of different lengths. Test results no longer depend on
  the order of modifiers returned by the implementation and do not
  return NotSupported prematurely.

* dEQP-VK.drm_format_modifiers.export_import.* tests suffered from
  similar issues. They tried to detect which of the supported modifiers
  for a format met a list of requirements and returned early with
  NotSupported if any of them failed to match those requirements,
  instead of running the tests in all possible modifiers.

Affected tests:
dEQP-VK.drm_format_modifiers.*

Components: Vulkan
VK-GL-CTS issue: 3026

Change-Id: I4c6600b87f3c2b4b9b165fdc17988efe2e0cf670

Relax protected heap allocation tests

Some implementations have limitations on protected heap
allocations, and these limitations cannot be queried
using the Vulkan API. This change allows an out-of-memory
to happen after a certain number of allocations to
compensate that.

VK-GL-CTS Issue: 3038

Affects:

dEQP-VK.memory.*allocation.*

Components: Vulkan
Change-Id: I3a34637c8c15eb8670613dce9b6e394a622a4c71

Added layered AHB EGL tests

Added new EGL image format tests for using Android Native
Hardware buffers with layers. They are mapped as 2D texture
arrays using EGL image API.

EGL internal test filter was also removed as the EGL AHB
tests are now using public Android API.

VK-GL-CTS Issue: 2671

New tests:

dEQP-EGL.functional.image.*android_native*

Components: EGL, Framework
Change-Id: I34a7560703bb06b75be276dc8ba5247c3763877a

Improve buffer to buffer copy tests

Buffer to buffer copy tests were filling the source and destination
buffers with the same contents, using functions that were both used for
images and buffers.

When filling source buffers, a linear memory allocation of R32_UINT
elements was filled with color red, which resulted in all bytes
containing value 255.

When filling destination buffers, the same procedure was followed using
color white. However, due to R32_UINT only having one component, this
also resulted in every buffer byte containing value 255.

The copy operation that followed could fail (e.g. with the
implementation not doing anything) and the test could still pass. This
commit makes the destination buffer be filled with zeros so as to verify
regions are being interpreted correctly.

Affected tests:
dEQP-VK.api.copy_and_blit.*buffer_to_buffer*

Components: Vulkan
VK-GL-CTS issue: 3008

Change-Id: I49e5ea5a4843540e57be22c67107db82b7903da2

Merge vk-gl-cts/vulkan-cts-1.2.7 into vk-gl-cts/master

Change-Id: Ie8036c780cda38f67032afa000852326c57b7860

Merge vk-gl-cts/vulkan-cts-1.2.6 into vk-gl-cts/vulkan-cts-1.2.7

Change-Id: I2ac7dec5e1cd15b13dec902ceeff581992d2678c

Test multiview with VK_EXT_multi_draw

Add new VK_EXT_multi_draw test variants that check multi_draws together
with multiview.

New, affected and removed tests:
dEQP-VK.draw.multi_draw.*

Components: Vulkan
VK-GL-CTS issue: 2985

Change-Id: I0e2b36efa912629b3f6457913f3c3b6d9eeebfe8

Add sample mask test for FSR

This change adds single test that uses pipelineFragmentShadingRate
with fragmentSize {2,2}, 4xMSAA and a sample mask of 0x9.
After drawing random triangles there is a check that samples are
written only to samples present in the sample mask.

VK-GL-CTS issue: 2397

Components: Vulkan

Affects:
dEQP-VK.fragment_shading_rate.misc_tests.*

Change-Id: I38bbddeb2f8d7e8aac1c3803b171ec5be2f0c21a

Fix transform_feedback.simple.multiquery* tests

Tests were using host query reset which may not be present pre-VK 1.2.
Change tests to use device query reset (there are other dedicated tests which test HOST_QUERY_RESET).

Components: Vulkan
VK-GL-CTS Issue: 2866

Affects:
dEQP-VK.transform_feedback.simple.multiquery*

Change-Id: Ib8c943a53542f4e21167f6afb0005200043477c6

Fix depth/stencil subpass dependencies in renderpass allocation tests

This seems like a copy/paste error, but the last subpass should have
VK_ACCESS_DEPTH_STENCIL_ATTACHMENT_WRITE_BIT and the current subpass
should have VK_ACCESS_DEPTH_STENCIL_ATTACHMENT_READ_BIT, analagous to
the color attachment case.

Affects:

dEQP-VK.renderpass.dedicated_allocation.attachment_allocation.input_output.*

Components: Vulkan

VK-GL-CTS Issue: 3019

Change-Id: I0f46316f9c5147d2391c6f0f2a37fb1e56420078

Remove fallback and unify testing for image clearing tests

Image clearing tests were previously using render pass
to pre-clear an image. This commit removes the use of
render pass as a clearing method and instead uses a buffer
to clear the image.

Affects:

dEQP-VK.api.image_clearing.core.*

Components: Vulkan

VK-GL-CTS issue: 2604

Change-Id: I82b6ac6cb6f8a2d79e4f7ffa12abc636bc5a961a

Portability: fix osx target

Currently cts can be be build on MacOS using default target
this CL adds missing libraries also to osx target.

Components: Framework

VK-GL-CTS public issue: 277

Affects:
*

Change-Id: Ic890bea33bd90240e84dd71b985cf146a1a4df0c

Test behaviour of color write enable with colorWriteMask

This commit adds new tests to test the interaction of color write enable and
color write mask.

New and affected tests:
dEQP-VK.pipeline.color_write_enable.*

Components: Vulkan
Reviewed-by: Ricardo Garcia <rgarcia@igalia.com>

Add more tests for YCbCr plane compatibility

The only existing tests are using other UNORM formats, and a validation
layer change to allow compatible formats here has shown that this isn't
working properly on some drivers. This extends the tests that were added
as part of #1630 (CL 7356).

Components: Vulkan
VK-GL-CTS issue: 1630
New tests: dEQP-VK.ycbcr.plane_view.*_compatible_format_*int
Affects: dEQP-VK.ycbcr.plane_view.*_compatible_format_*

Change-Id: Ia14357fc8873566333ea888ef18a2155ffa6ef51

Remove invalid reserved keywords tests

These identifiers are not listed as reserved in any spec that I can
find.

Components: OpenGLES
Removed Tests: dEQP-GLES3.functional.shaders.keywords.reserved_keywords.image*Shadow_*

Change-Id: I444d9a8a49d3dda7402faa50b79859aaef0ee690

Test line stipple factor is ignored if not used

This commit adds a few extra line test variants that check
lineStippleFactor in VkPipelineRasterizationLineStateCreateInfoEXT is
ignored when stippledLineEnable is false. The new tests use both zero
and a large number to check everything continues to work.

New tests:
dEQP-VK.rasterization.primitives.no_stipple.*factor_0
dEQP-VK.rasterization.primitives.no_stipple.*factor_large

Affected tests:
dEQP-VK.rasterization.primitives*

Components: Vulkan
VK-GL-CTS issue: 3009

Change-Id: I6e329e8572cb0bae043119553bd4ea8d5682f660

Test zero-size unused specialization constants

According to the spec for VkSpecializationMapEntry

> If a constantID value is not a specialization constant ID used in the
> shader, that map entry does not affect the behavior of the pipeline.

So it should be legal to use VkSpecializationMapEntry structures
indicating size zero for a given constant as long as the constant is not
used in the shaders.

This commits adds new test variants that verify implementations handle
such cases.

New tests:
dEQP-VK.pipeline.spec_constant.*.basic.*unused_*

Affected tests:
dEQP-VK.pipeline.spec_constant.*

Components: Vulkan
VK-GL-CTS issue: 3011

Change-Id: I2a8439368722e6caaa828fd91e061cfd5e2be61d

Fix build on CLang failing due to -Wunused-private-field

Dereference a member uses only in Debug builds.

VK-GL-CTS Issue: 3013
Components: Vulkan

Change-Id: I7a607328bf033b24c1bba877f78c3fa4fd24e9e1

SwiftShader script improvements

This commit applies varying fixes proposed in the issue.

VK-GL-CTS issue: 2956

Change-Id: I6f73bea9f5ab875b8d40990649fb1f9d9a2d603d

Merge vk-gl-cts/vulkan-cts-1.2.7 into vk-gl-cts/master

Change-Id: I4bf831e86123de179132f435a7c7bf838d0c89d3

Update glslang, SPIR-V Tools, headers

Components: Framework

Change-Id: Ie259277466b22ba816843c362da272a4284fcb93

Add tests for VK_EXT_shader_atomic_float2

New tests:
dEQP-VK.glsl.atomic_operations.*float16*
dEQP-VK.glsl.atomic_operations.min_float*
dEQP-VK.glsl.atomic_operations.max_float*
dEQP-VK.image.atomic_operations.min*r32f*
dEQP-VK.image.atomic_operations.max*r32f*

Affects:

dEQP-VK.glsl.atomic_operations.*float*
dEQP-VK.image.atomic_operations.add*r32f*
dEQP-VK.image.atomic_operations.exchange*r32f*

Components: Vulkan

VK-GL-CTS issue: 2566

Change-Id: I7462cc74e799c2245407220fa9db7e5b659f86f4
(cherry picked from commit ca4e5aff66bccbbe88935c10e2c55a3147c1fc03)

Tests for VK_KHR_present_id and VK_KHR_present_wait

Add tests for VK_KHR_present_id and VK_KHR_present_wait. This commit
also refactors some code so it can be reused in several places, hence
marking all WSI tests as affected.

Affected tests:
dEQP-VK.info.device_mandatory_features
dEQP-VK.wsi.*

New tests:
dEQP-VK.wsi.*.present_id_wait.*

Components: Vulkan
VK-GL-CTS issue: 2098
VK-GL-CTS issue: 2099

Change-Id: I1c95fbf031d651773a9fff1c10a23038d73f6e53
(cherry picked from commit 8bf210fd592883b3adebb57c3d6cc49188d1758e)

Update Vulkan headers

Components: Vulkan

Change-Id: Id59fb9125cac9068d68a8e5f219c3dc53a093ad0

Merge vk-gl-cts/vulkan-cts-1.2.6 into vk-gl-cts/vulkan-cts-1.2.7

Change-Id: I8f8c8aba438300565a896d21d27c2892eef477df

Merge vk-gl-cts/vulkan-cts-1.2.5 into vk-gl-cts/vulkan-cts-1.2.6

Change-Id: I1e865999cbff46e84b9045400eb2dde830296094

Merge vk-gl-cts/vulkan-cts-1.2.4 into vk-gl-cts/vulkan-cts-1.2.5

Change-Id: I22fcd718aadfed74408323615cc4bf58618d979a

Merge vk-gl-cts/vulkan-cts-1.2.3 into vk-gl-cts/vulkan-cts-1.2.4

Change-Id: If23bb885846d42651d7ac77b375b6c44ea02fa93

Merge vk-gl-cts/vulkan-cts-1.2.2 into vk-gl-cts/vulkan-cts-1.2.3

Change-Id: If1bb9f63d56815c3ced2ac72d17d4e55ec090f39

Fix early destruction of VkPipelineLayout

Components: Vulkan
VK-GL-CTS issue: 3004

Affected:

    dEQP-VK.sparse_resources.shader_intrinsics.*

Change-Id: Id5508fac89961c5bb934005257d2caa26d32f2dd
Signed-off-by: Samuel Iglesias Gonsálvez <siglesias@igalia.com>

Fix validation errors in AHB tests

AHB protected memory tests were also removed as the test
tried to use protected memory without creating a device
and queue that would support protected operations.

VK-GL-CTS Issue: 2508

Affects:

dEQP-VK.api.external.memory.android_hardware_buffer.*

Components: Vulkan
Change-Id: I307e7450d06d20b6497f2a529e5724ae0538e5a9

Clean ExternalMemoryUtil

This change separates choosing of memory type from
allocateExportableMemory utility function. This makes
use of this functions more clear.
Previous implementation needed to be done ASAP and
involved passing of non-const reference to
allocateExportableMemory which was then filled.
This workaround made code unnecessarily hard to follow.

Components: Vulkan

VK-GL-CTS issue: 587

Affects:
dEQP-VK.api.external.memory.*
dEQP-VK.synchronization.signal_order.*
dEQP-VK.synchronization.cross_instance.*

Change-Id: I781388db390f13c7c03785dd0ae34ae0128ec31c

Fix uninitialized local variables in generated shader.

There is uninitialized variable which is used as texture sampler's
coordinate. The behavior is undefined according to spec.

Affects:

dEQP-GLES3.functional.shaders.random.*

Components: Framework, OpenGL
VK-GL-CTS issue: 2972

Change-Id: I3e68a1a2158d248b24c927c481ecba4001496b3f

Fix most dEQP-VK.sparse_resources.shader_intrinsics.* validation errors

The majority of the validation errors were due to creating sparse
images before the derived class had the opportunity to check
the feature support of the format for the respective tests.

Change it to check that before anything else.

Components: Vulkan
VK-GL-CTS issue: 3006

Affected:

   dEQP-VK.sparse_resources.shader_intrinsics.*

Change-Id: I37308d4a2650cf5cc45ecbffb86a934e6fd182cc
Signed-off-by: Samuel Iglesias Gonsálvez <siglesias@igalia.com>

Additional dynamic depth bias tests

Additional tests to make sure the depth bias dynamic state is not
ignored and works properly, also when used in conjunction with the
dynamic depth bias enable state.

New tests:
dEQP-VK.dynamic_state.rs_state.nonzero_depth_bias*
dEQP-VK.pipeline.extended_dynamic_state.*dynamic_bias_params

Affected tests:
dEQP-VK.pipeline.extended_dynamic_state.*.depth_bias*

Components: Vulkan
VK-GL-CTS issue: 2965

Change-Id: Ic2b4428d112547ca054161728b8e1d403f6394b4

Test VK_EXT_multi_draw with geometry and tessellation shaders

This commit adds new VK_EXT_multi_draw test variants that use geometry
and tessellation shaders in the pipeline to make sure they do not break
with VK_EXT_multi_draw.

New and affected tests:
dEQP-VK.draw.multi_draw.*

Components: Vulkan
VK-GL-CTS issue: 3000

Change-Id: Idfaa6e49895c9d94e484c89d7097af9599b4ed51

Remove unused test cases from gen-keywords script

The script generated test cases for double underscore in identifiers,
but they were not included in the checked-in copy of the generated
output. The tests appear incorrect, so remove them from the script.

Components: OpenGLES
Affects: None

Change-Id: If301f22afac06a147f1ff58156bcf0ea4cb3b037

Test dynamic vertex stride of 0.

Goes beyond existing specification and tests if we can use stride 0 as
well.

Components: Vulkan
Vulkan issue: 2742

Added tests:

   dEQP-VK.pipeline.extended_dynamic_state.*.zero_stride_with_offset

Change-Id: I4838b60321f27dd4b4546b1e42ed2f6043aec5c1

Add Image Store tests to BGRA images

These are said to be an important usecase for storage without format but
they're not currently tested. Add tests for these formtas. Only store
tests are currently supported because of limitations in the underlying
test structure.

Components: Vulkan
New tests: dEQP-VK.image.store.without_format.*.b8g8r8a8_*
           dEQP-VK.image.format_reinterpret.*.b8g8r8a8_*

Change-Id: I243a14681b262941d95e08e0b5a87ca9ccee9d43

Test fragment shading rate limits and rates

This change verifies all limits for VK_KHR_fragment_shading_rate
and tests vkGetPhysicalDeviceFragmentShadingRatesKHR.

VK-GL-CTS issue: 2397

Components: Vulkan

Affects:
dEQP-VK.fragment_shading_rate.misc.*

Change-Id: I81332ec819009817b1ca0580058617bbc07a53e2

Add more test related to early fragment tests

- Added tests for EarlyFragmentTest followed by a shader which
  discards. Then check that depth buffer is updated but the color
  buffer and other outputs are not.

- Added tests that write to gl_SampleMask and checked that do not affect
  the depth values written to the depth buffer.

Components: Vulkan
VK-GL-CTS issue: 2968

Added tests:

   dEQP-VK.fragment_operations.early_fragment.discard*
   dEQP-VK.fragment_operations.early_fragment.samplemask*

Signed-off-by: Samuel Iglesias Gonsálvez <siglesias@igalia.com>
Change-Id: I4a0bd49ab9319747e8c92aaa26f64a4e077fa8a5

Remove replacement of bool32

Components: Vulkan

VK-GL-CTS issue: 2963

Affects: none

Change-Id: I4ccb580ba8b142e6425373d1395443ac747172f1

Replace DWORD with uint32_t

This change replaces DWORD with uint32_t instead of deUint32

Components: Vulkan

Affects: dEQP-VK.api.*create_win32*

VK-GL-CTS issue: 2963

Change-Id: I32e958863508e4641f769feebd450c25bfc95492

Fix early destruction of VkPipelineLayout

Components: Vulkan
VK-GL-CTS issue: 3004

Affected:

    dEQP-VK.sparse_resources.shader_intrinsics.*

Change-Id: Id5508fac89961c5bb934005257d2caa26d32f2dd
Signed-off-by: Samuel Iglesias Gonsálvez <siglesias@igalia.com>

Reorder vulkan enum elements

Reorders lines in linear vulkan enums so that *_LAST is placed
after all unique values, followed by equivalence enums and
*_MAX_VALUE (last).

VK-GL-CTS issue: 2684
Components: Vulkan

Change-Id: I129a0b0afe3574a91ccc860ed6ea8de27671f9ff

Merge vk-gl-cts/vulkan-cts-1.2.7 into vk-gl-cts/master

Change-Id: I97026a16399e415525246d8f9a0be895f2d101be

Use uintptr_t in the framework

This change replaces deUintptr with uintptr_t
in the framework files

Components: Vulkan

Affects: dEQP-VK.*

VK-GL-CTS issue: 2963

Change-Id: Ic9bc75782e4b8825030044459b5ff90f78711eb3

Change initializer for array of structures to avoid GCC bug

Works around the following gcc 4.8.4 internal compiler error:

    vktReconvergenceTests.cpp:1169:35: internal compiler error: in gimplify_init_constructor, at gimplify.c:4271
       SubgroupState stateStack[10] = {};

Components: Vulkan
Affected tests: dEQP-VK.reconvergence.*
VK-GL-CTS issue: 3001

Change-Id: I42ec5aa978351f84ad41aae4af9f4977d3263461

Fix FreeBSD build of VK_EXT_physical_device_drm tests

Linux defines the major() and minor() macros in
the sys/sysmacros.h header, but FreeBSD does not
have this header and instead defines them in
sys/types.h.

Components: Vulkan
Affects: dEQP-VK.api.device_drm_properties.*

Change-Id: I0bed149d24241e152437f530f5eb137738c46122

Merge vk-gl-cts/vulkan-cts-1.2.6 into vk-gl-cts/vulkan-cts-1.2.7

Change-Id: I3e12f2ef494b16c16e6aa957c99c543a057a7003

Use [u]int64_t in the framework

This change replaces deUint64/deInt64 with uint64_t/int64_t
in the framework files

Components: Vulkan

Affects: dEQP-VK.*

VK-GL-CTS issue: 2963

Change-Id: I76b24fbfadbd3fb31c9a20b172c021e0305290c2

Extend image tests to support formats with no formats

Unless the test actually needs a SPIR-V format token, allow testing of
Vulkan formats for which there is no corresponding SPIR-V token. At the
moment this functionality is not used, but tests will soon be added that
exercise it.

Components: Framework, Vulkan
Affects (trivially): dEQP-VK.image.*

Change-Id: I11e16b6951ca40444131fa1e8d5e7bce6b075f85

Define a framework function for getting SPIR-V format tokens

These are used in a couple of different tests, so it doesn't make sense
to define the format tokens in two places.

Components: Framework, Vulkan
Affects (trivially): dEQP-VK.image.*

Change-Id: Icffd8ac4b0d78e5dab076780e0221af3f358d74c

Merge vk-gl-cts/vulkan-cts-1.2.5 into vk-gl-cts/vulkan-cts-1.2.6

Change-Id: I73e4fbe8f154f15a77498121181c6a48028203fe

Remove tests with too many overlapping instances in the accelleration structure

These tests generate a grid of random triangles and assign them to the large
number of instances in the acceleration structure. Because of the randomness,
all instances overlap with each other and the accelleration structure is extremely
slow to trace - as each ray must traverse through all instances.

This change removes some tests cases with 65536 instances and tests cases with
1048576 instances to avoid device timeouts.

The tests apply only to implementations with host accelleration structure builds
supported.

Components: Vulkan

VK-GL-CTS Issue: 3005

Change-Id: I94bcde596cd097672cf8e00c6fe018328998cb37
Affects: dEQP-VK.ray_tracing_pipeline.build.*

Test blitting from compressed formats

This change extends existing tests to also test blitting of BC*,
ETC and EAC compressed formats onto all compatible uncompressed
formats.

Functionality from PipelineImageUtil is reused in blit tests
to generate random compressed data that is also stored in
uncompressed version. Compressed data is used to do the blit
on GPU while uncompressed version is used for verification on CPU.

VK-GL-CTS issue: 2366

Components: Vulkan

Affects:
dEQP-VK.api.copy_and_blit.*

Change-Id: Ifab885dc09b2d663d95b49e25458873c2eb0b0c4