Take snapshot of the AOSP master mustpass and use it in GL CTS

Components: OpenGL
VK-GL-CTS issue: 23

Change-Id: I9ebb11fd8c69f1b697bb08ac7351033c9b06ef94

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

Change-Id: Ice81ca8779a7f7cfea5d8fab0e4b48fe6e603ec2

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

Components: AOSP, Framework

Change-Id: I0de6207abfc1c990a85fcf977d78ed1103c52a1b

Use glReadnPixels only if KHR_robustness is supported

glReadnPixels is introduced in KHR_robustness/GLES 3.2.
Use of this function in GLES 3.1 tests will result in segfault
on implementations that don't support KHR_robustness.

Components: AOSP, Framework

Affects: dEQP-GLES31.functional.debug.negative_coverage.callbacks.buffer.read_pixels_format_mismatch

VK-GL-CTS issue: 166

Change-Id: Idbe8734f7d82882ea99c75a8b74d693ac6dd831f

Fix indirect draw line offset calculation

The GL_LINES test was correctly avoiding undefined rasterizer tie-break
conditions but other tests that draw lines were not. Include the fix in
the other tests.

Affects:
KHR-GLES31.core.draw_indirect.basic-mode-draw*-line*_adjacency

Components: OpenGL
VK-GL-CTS issue: 193

Change-Id: I58862469b0bae3b5f968bad8586adc7cad6ad7c7

Merge "Modify EGL robustness tests to use VBOs"

Modify EGL robustness tests to use VBOs

Remove use of client-side vertex arrays and use buffer objects instead.
Delete buffers on teardown and in destructor.
Move test description logs to start log file.

Affected tests:
- dEQP-EGL.functional.robustness.reset_context.*

Change-Id: I9432258f30d6c7e02362c0b2f6c36fc3694bd72d

Fix "null" ID in sRGB decode tests

Use 0 as "null" GL object ID as -1 is potentially valid ID.

Change-Id: I95f9f0c300d5b6e95c0c7c81e29fd28ac9ca3b82

Add CTS_ARB_fragment_shader_interlock specification proposal

This is CTS_ARB_fragment_shader_interlock specification proposal

components: OpenGL

VK-GL-CTS issue: 61

Change-Id: I0a7827e87c807d1044d7434ac1e9833530be3cde

Fix narrowing conversion warning

Change-Id: I07b20837e5dac2f6a88504b582732a7db10cf5c8

Modify single_tessellation_stage to check for GL_NV_gpu_shader5

That extension permits the use of a tessellation control shader
without a tessellation evaluation shader. Affected tests:

dEQP-GLES31.functional.debug.negative_coverage.*.tessellation.single_tessellation_stage

Fix a compilation error on MSVC12

There's a problem with uint32_t, so better just use delib defines.

Components: Vulkan

Change-Id: I589ac7f220a7e651ed31910d16c643866314f681

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

Change-Id: I453e9f9eedc6e07d785d054b69c6822c0a9e80b1

Merge vk-gl-cts/vulkan-cts-1.0.1 into vk-gl-cts/vulkan-cts-1.0.2

Change-Id: I1a17ff8600abeab32614212bdab2e4646e50bc25

Merge vk-gl-cts/vulkan-cts-1.0.0 into vk-gl-cts/vulkan-cts-1.0.1

Change-Id: I9eec4dd1fa88a85374eba8d0851b00a28c50b4ac

Add new robustness extension tests

New tests for KHR_robustness extension. Attempts to provoke context
reset scenarios and test GL state after a reset has occurred.

New test group:
- dEQP-EGL.functional.robustness.*

New tests:
- create_context.query_robust_access
- create_context.no_reset_notification
- create_context.lose_context_on_reset

- reset_context.shaders.infinite_loop.*   (various shaders)
- reset_context.shaders.out_of_bounds.*   (various resource types)
- reset_context.fixed_function_pipeline.* (index and vertex buffers)

- negative_context.invalid_robust_context_creation
- negative_context.invalid_robust_shared_context_creation

Note: Test groups ending in *_non_robust are for internal use only.

Change-Id: I91d7e3e95ec52f908dbaa1f49c56dcd94d60fd8f

Fix tessellation support checks in negative binding tests

Bug: 35618877
Test: Run the tests on couple of devices supporting tessellation
Change-Id: Ieca6a28a57a278a07ff688f760eb7da807e66d49

Merge Vulkan CTS 1.0.2.1 into AOSP

Change-Id: Ib0aaf12ec60902ebc31d00ad02291bd3d1cd7fae

Fix bug in reallocation callback validation

vk::validateAllocationCallbacks() didn't correctly handle a case where
reallocate(NULL, size) returned previously seen pointer value.

VK-GL-CTS issue: 176
Components: Vulkan

Change-Id: I0822f8e4176d20d7bf296cd85f5c5223e189d250

Fix input attachment layout in renderpass tests

Input attachment layout was hardcoded for descriptor set update.
Added input attachment layout getter.

Affects:

dEQP-VK.renderpass.*

Components: Vulkan

VK-GL-CTS issue: 189

Change-Id: I2083c7b0cf4c68072f98d369f6c3089c6eb2b184

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

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

Change-Id: I9d823bd148d6767786e6ad28f19fc381070d149f

Fix extension support checks in negative api tests

Add checks for tessellation and geometry shader support for negative
shader storage and primitive bounding box tests.

Move all uses of cubemap array textures inside the extension checks in
the texture tests.

Bug: 29835218
Change-Id: I6051fff231285ed7d9c71015c2bb11a9fc7b909b
Signed-off-by: Pierre Couillaud <pierre@broadcom.com>

Fetch gl.xml, egl.xml from Github repos

SVN repository containing gl.xml and egl.xml is no longer updated.
The new locations for these files are OpenGL-Registry and
EGL-Registry repositories on Github.
Adjust fetching scripts to use the new locations.

Components: Framework
Affects: headers generation

Change-Id: Ic091a46c636d446b6a4751bd3b797bcd19f90014

Fix [Named]FramebufferTextureLayer tests

The test was failing because it was reusing the texture ID used while
testing FramebufferTexture. But that is a GL_TEXTURE_2D, that is a
wrong target for [Named]FramebufferTextureLayer. So for the other
subtests, that were checking things like invalid attachment, etc,
there was two possible errors, and mesa driver raised first the one
related to a wrong target.

This patch reuses a texture ID with a valid target, being also used to
test [Named]FramebufferTextureLayer

For what it is worth, [Named]FramebufferTextureLayer with a wrong
target is also tested.

Affects:
GL45-CTS.direct_state_access.framebuffers_texture_attachment_errors

Components: OpenGL

VK-GL-CTS issue: 173

Change-Id: I388e1593e67d8708433d0f4763c716a4f3f03067

Make vertex shader boilerplate to set gl_Position

If we don't generate any fragment from the previous stages, the
fragment shader might not be run. Hence invalid results for
GL45-CTS.shader_image_load_store.multiple-uniforms in the fragment
stage.

Components: OpenGL
Affects: GL45-CTS.shader_image_load_store.multiple-uniforms
VK-GL-CTS issue: 30

Change-Id: I6342e27ffd51725879fe1519d81122f41cb81eef

Merge vk-gl-cts/opengl-es-cts-3.2.2 into vk-gl-cts/master

Change-Id: I039aa2432d455394b254e0556a1ab63f02fba35a

Fix post-depth coverage shader test case

The tests uses 4 different versions of GLSL shader code for
the fragment shader that are supposed to be assigned to the
variables m_fragShader{1..4} that are then used in the 4
test cases implemented. However, the last shader code is not
assigned to m_fragShader4, instead it immediately overwrites
m_fragShader3. The test would then fail as it attempts to build
a shader program with an empty string for the fragment shader
when it uses m_fragShader4.

Components: OpenGL
VK-GL-CTS issue: 181

Affects:
GL45-CTS.post_depth_coverage_tests.PostDepthShader

Change-Id: I49474f04668de2037b4afc5570082bd4110f4fab

DirectStateAccessTexturesTests: fix textures_parameters_errors

Right now GL45-CTS.direct_state_access.textures_parameter_errors
expects INVALID_OPERATION when using a wrong target.

From OpenGL 4.5 spec, section 8.11.2, Texture Parameter Queries:

   "An INVALID_ENUM error is generated if the effective target is not
    one of the texture targets described above."

The test even has a comment about expecting a INVALID_ENUM, so this
seems a typo when calling CheckErrorAndLog

Affects:
GL45-CTS.direct_state_access.textures_parameter_errors

Components: OpenGL

VK-GL-CTS issue: 180

Change-Id: Id82559090ccefa5713b0dd01d0ef26fdf4860ed3

Use glInvalidateBuffer[Sub]Data correctly

These function take a buffer name, not a bind point.

Modules: OpenGL
Affects:
GL45-CTS.sparse_buffer_tests.BufferStorageTest

VK-GL-CTS issue: 178

Change-Id: Iffdecfb5dd738ea82073ccb1b8771ca411f8f3b8

Fix shaders using reserved keywords incorrectly

"input" is a reserved keyword in GLSL 1.40 (and other versions,
for that matter); this test should never have passed for any
correct implementation.

"unsigned int" should be "uint".

Affects:
GL45-CTS.sparse_buffer_tests.BufferStorageTest

Modules: OpenGL
VK-GL-CTS Issue: 177

Change-Id: I68bba7548225ada7949182a1239d423eff15fc35

GetTextureSubImageTests: Remove m_texture_rectangle_compressed

GetTextureSubImage::Errors test attempts to setup a compressed texture
using GL_TEXTURE_RECTANGLE as target, during test preparation. This is
explicitly forbidden by the spec.

From OpenGL 4.5 PDF, section "8.7. COMPRESSED TEXTURE IMAGES", page 214:

    "An INVALID_ENUM error is generated if the target parameter to any
     of the CompressedTexImagenD commands is TEXTURE_RECTANGLE or
     PROXY_TEXTURE_RECTANGLE ."

This patch removes the m_texture_rectangle_compressed texture from the
test, and all the associated test sub-cases.

Affects: GL45-CTS.get_texture_sub_image.errors_test
VK-GL-CTS issue: 169
Components: OpenGL

Change-Id: Ia3c2008cb948662f064dbc13fcd4d61c7d2caebf

Unsized array of arrays in uniform block

From the GL_ARB_arrays_of_arrays spec:

   "For unsized arrays, only the outermost dimension can be lacking a
    size. A type that includes an unknown array size cannot be formed
    into an array until it gets an explicit size."

This is later further detailed as per OpenGL® Shading Language 4.40.

From page 3 (page 9 of the PDF) of the GLSL 4.00 v9 spec:

  "* Bug 11987: Allow array-of-array constructors and initialized
     variables to have any dimension unsized (not just the outer
     dimension) and get their size from the content of the constructor
     or initializer."

From page 37 (page 43 of the PDF) of the GLSL 4.00 v9 spec:

  " vec4 a[][] = { vec4[2](vec4(0.0), vec4(1.0)), // okay, size to
                   vec4[2](vec4(0.0), vec4(1.0)), //       a[3][2]
                   vec4[2](vec4(0.0), vec4(1.0)) };"

From page 38 (page 44 of the PDF) of the GLSL 4.00 v9 spec:

  "For implicitly-sized or run-time-sized arrays, only the outermost
   dimension can be lacking a size. A type that includes an unknown
   array size cannot be formed into an array until it gets an explicit
   size, except for shader storage blocks where the only unsized array
   member is the last member of the block."

Hence, in the absence of a constructor/initializer, unsized arrays are
only allowed for the outermost dimension of an AoA.

Affects:

GL43-CTS.arrays_of_arrays_gl.InteractionUniformBuffers3

Components: OpenGL
VK-GL-CTS issue: 146

Change-Id: Icbfe49c16e8ce7ffd55b92adede90405ac8e5f26

Merge "Drop MAX_TESS_CONTROL_TOTAL_OUTPUT_COMPONENTS to 2048"

Unsized array of arrays in shader storage block

From the GL_ARB_arrays_of_arrays spec:

   "For unsized arrays, only the outermost dimension can be lacking a
    size. A type that includes an unknown array size cannot be formed
    into an array until it gets an explicit size."

This is later further detailed as per OpenGL® Shading Language 4.40.

From page 3 (page 9 of the PDF) of the GLSL 4.00 v9 spec:

  "* Bug 11987: Allow array-of-array constructors and initialized
     variables to have any dimension unsized (not just the outer
     dimension) and get their size from the content of the constructor
     or initializer."

From page 37 (page 43 of the PDF) of the GLSL 4.00 v9 spec:

  " vec4 a[][] = { vec4[2](vec4(0.0), vec4(1.0)), // okay, size to
                   vec4[2](vec4(0.0), vec4(1.0)), //       a[3][2]
                   vec4[2](vec4(0.0), vec4(1.0)) };"

From page 38 (page 44 of the PDF) of the GLSL 4.00 v9 spec:

  "For implicitly-sized or run-time-sized arrays, only the outermost
   dimension can be lacking a size. A type that includes an unknown
   array size cannot be formed into an array until it gets an explicit
   size, except for shader storage blocks where the only unsized array
   member is the last member of the block."

Hence, in the absence of a constructor/initializer, unsized arrays are
only allowed for the outermost dimension of an AoA.

Affects:

GL43-CTS.arrays_of_arrays_gl.InteractionStorageBuffers3

Components: OpenGL
VK-GL-CTS issue: 145

Change-Id: Ic751b3ff52a84e3aee604552edf9441211890a11

Unsized array of arrays in uniform variable

From the GL_ARB_arrays_of_arrays spec:

   "For unsized arrays, only the outermost dimension can be lacking a
    size. A type that includes an unknown array size cannot be formed
    into an array until it gets an explicit size."

This is later further detailed as per OpenGL® Shading Language 4.40.

From page 3 (page 9 of the PDF) of the GLSL 4.00 v9 spec:

  "* Bug 11987: Allow array-of-array constructors and initialized
     variables to have any dimension unsized (not just the outer
     dimension) and get their size from the content of the constructor
     or initializer."

From page 37 (page 43 of the PDF) of the GLSL 4.00 v9 spec:

  " vec4 a[][] = { vec4[2](vec4(0.0), vec4(1.0)), // okay, size to
                   vec4[2](vec4(0.0), vec4(1.0)), //       a[3][2]
                   vec4[2](vec4(0.0), vec4(1.0)) };"

From page 38 (page 44 of the PDF) of the GLSL 4.00 v9 spec:

  "For implicitly-sized or run-time-sized arrays, only the outermost
   dimension can be lacking a size. A type that includes an unknown
   array size cannot be formed into an array until it gets an explicit
   size, except for shader storage blocks where the only unsized array
   member is the last member of the block."

Hence, in the absence of a constructor/initializer, unsized arrays are
only allowed for the outermost dimension of an AoA.

Affects:

GL43-CTS.arrays_of_arrays_gl.InteractionUniforms2

Components: OpenGL
VK-GL-CTS issue: 144

Change-Id: Icd3e710f1bb69b35355a5c4172dc8839b9f98c82

BufferStorageTests: do not unmap with MapOwner if not needed

This test uses the following method to call MapRange on a buffer:

   Buffer::MapOwner Buffer::MapRange(glw::GLintptr offset,
                                     glw::GLsizeiptr length,
                                     glw::GLenum access)

It returns a MapOwner, that unmaps the buffer when it is destroyed.

For some tests, it is called like this:

{ /* start test */
   {
     buffer.MapRange(<params>);
   } /* freed here */

   /* Do stuff with the buffer. Assumes to be still mapped */

   /* Checks the buffer */

} /* end test */

The problem with this approach is that as the Mapowner returned by
MapRange is not used, it is destroyed (on the previous code snippet at
"freed here"), and the buffer is unmapped unintentionally.

This patch gets out the call to MapRange, so it is not unmapped until
the end (on the code snippet, at "end test").

Fixes the following tests (only Mesa Intel driver tested):
GL45-CTS.buffer_storage.get_buffer_parameter
GL45-CTS.buffer_storage.map_persistent_buffer_sub_data
GL45-CTS.buffer_storage.map_persistent_read_pixels

Components: OpenGL
VK-GL-CTS issue: 148

Change-Id: I676eeaf0808e9bb19617bde7d658c14859f1357d

Merge pull request #30 from BigBigos/master

Fix GL45-CTS.pipeline_statistics_query_tests_ARB.functional_default_qo_values

Drop MAX_TESS_CONTROL_TOTAL_OUTPUT_COMPONENTS to 2048

GL_MAX_TESS_CONTROL_TOTAL_OUTPUT_COMPONENTS minmax has been changed to
2048 (from 4096) in OpenGL ES 3.2 spec in response to bug 12823.

Bug: 35619900
Change-Id: I59d75d23df02d339bf7094a8e73d3a24d2e785be

Add render to mipmaps tests

This test repeatedly draws into the same image using consecutive
mip levels as color and dept/stencil attachments. All mip levels
and array layers are exercised.

Modified tests:

dEQP-VK.pipeline.render_to_image.*

New tests:

dEQP-VK.pipeline.render_to_image.*.mipmap.*

Components: Vulkan
Vulkan CTS issue: 487, 535

Change-Id: I7e4b3c6978d069c0accc377e9ab6e9d05e8e3cc6

Add max size attachments tests

This test creates very big color, and depth/stencil attachments
in all possible dimensions and layers.

Original tests in the affected group were moved to *.small.*
subgroup.

Modified tests:

dEQP-VK.pipeline.render_to_image.*.small.*

New tests:

dEQP-VK.pipeline.render_to_image.*.huge.*

Components: Vulkan
Vulkan CTS issue: 451, 535

Change-Id: I1e3bd1f7a657ff837b830bbada539a1240b1fd04

Add framebuffer attachment size tests

New tests:
- dEQP-VK.pipeline.framebuffer_attachment.*

VK-GL-CTS issue: 72
Components: Vulkan

Change-Id: Iab4c077d60471633688476a87570e779d56bcda8

Modify tessellation_not_active to check for GL_NV_gpu_shader5

That extension permits use of GL_PATCHES for stages other than
tessellation stages. Affected test:

dEQP-GLES31.functional.debug.negative_coverage.get_error.tessellation.tessellation_not_active

Bug 35670519

Merge "Fix whitespace issues"

Add non-seperable TCS-but-no-TES subcase and fix draw time check subcase

- Add a non-separable TCS-but-no-TES (new subcase).

Section 7.3 of the OpenGL ES 3.2 specification says:

   "Linking can fail for [...] any of the following reasons:

    * program contains an object to form a tessellation evaluation
      shader [...] and [...] the program is not separable and
      contains no object to form a tessellation control shader"

Add a test to ensure the implementation does raise a link error.

- Make program separable in TES-but-no-TCS draw time check test (fix)

Since the test is trying to check for a draw time error, it should use
a separable program - otherwise we may hit a link error.

Changes made to test case:
dEQP-GLES31.functional.debug.negative_coverage.*.tessellation.single_tessellation_stage

Bug: 35354738
Change-Id: I902106864059cfbe7d718da0d0527284de04a915

Fix whitespace issues

Change-Id: I31cd4e4d3e113192fce0bccd20fd8fa14d182620

Add GetFrameTimestamps tests

Add tests for EGL_ANDROID_get_frame_timestamps.

Verifies:
1) Frame IDs properly identify frames.
2) The frame timestamp ordering is correct
3) The compositor timing values are reasonable.
4) All timestamps from the most recently swapped frame can
   be retrieved without additional swaps.

Test: --deqp-case=dEQP-EGL*get_frame_timestamps*
Change-Id: I393a3bc79f019951a2fddb83760b6e1d919663b6

Merge "Fix multiple resource leaks in new srgb_texture_decode tests"

Merge "Reduce read pixel calls in copy image tests"

Merge "Restructure copy image test iterations"

Fix multiple resource leaks in new srgb_texture_decode tests

There were multiple issues here, ranging from never freeing resources,
to overwriting resources, to only freeing them at class destruction
instead of deinit time. This was resulting in hitting OOM on platforms
without tons of memory.

Bug: 35457323
Change-Id: I492177bb9e71bdbc7efc9d3a7c8a34bef35c709e

Build fix (uint32_t used instead of deUint32)

It looks like uint32_t is not used within the CTS.
Replacing it with deUint32 allows build to pass.

Components: Vulkan

VK-GL-CTS Issue #66

Change-Id: Ia23f3fb7fe87ec21349c46df52c7b9f83dd14a32

Merge changes Icb29d2da,Ida906e21

* changes:
  Add a non-separable TCS-but-no-TES subcase.
  Make program separable in TCS-but-no-TES draw time check test.

Workaround SSL cert issues, enable python 3

 * The script command line switch --insecure added to bypass SSL
   certificate validity checks. Though bypassing is not recommended,
   sometimes the issue can be resolved on the remote side only,
   which we have no control over. For example please see
   https://sourceforge.net/p/forge/site-support/14336/
   The parameter is supported for pythons 2.7.9 and 3.4.3 and newer,
   due to older versions of urlopen do not support context parameter

 * Minor changes allow to run fetch_sources.py under both v2 and v3
   pythons. Changes required due to python 3 prohibits statement-style
   usage for print and forces print statements to be converted into
   functions. Also urlopen() is in library urllib.request in python 3

Components: Framework

VK-GL-CTS Issue #64

Change-Id: If4354332ea19878f8f72352e2df5cac9ee254ac6

Fix incorrectly defined image memory barrier

In ResolveImageToImage::iterate() method a memory barrier is defined
to change the layout of source image. By mistake destination image
parameters are used to define subresource range. It causes validation
errors as in some cases the subresource range extends beyond
the dimensions of the source image.

The fix uses source image parameters to define the memory barrier.

Modified tests:
dEQP-VK.api.copy_and_blit.resolve_image.*

Components: Vulkan
VK-GL-CTS issue: 160

Change-Id: I0ef72fbc99b127129441b7c66b43afd7b5a1e0ef

Move DrawElementsBaseVertexTests to GL40 package

These tests make use of tessellation shaders, which were introduced
in 4.0.

The "using deqp::TestCaseGroup" is added to fix compilation:
gl3cTestPackages.cpp refers to glcts::TestCaseGroup. Similar using
directives are spread throughout the code base, one of them is in
esextcDrawElementsBaseVertexTests.hpp, which is no longer included.

Affects:
GL45-CTS.draw_elements_base_vertex_tests.*

Components: OpenGL
VK-GL-CTS issue: 28
Change-Id: I7fd2d2a641532ee4ac44ad2d71544ce08b9b69ec

dEQP-VK: Enable b10g10r11 and e5b9g9r9 blitting tests

This enables 620 new tests for blitting to/from e5b9g9r9 and b10g10r10.
Looking through vulkan.gpuinfo.org, most implementations support the
BLIT_SRC_BIT on these formats.  I have a branch of our driver that
supports BLIT_DST_BIT for these formats and the tests all work fine.

Change-Id: I964abad8c44d3d7cc52a79bbb73ee6b9833845d6

Fix dethread build for OS X and iOS

Change-Id: I6fb2f2af89d7568bd473597a0d413c3987e457ae

Fix atan2() precision tests and add to mustpass

atan2() had a bug where tests were rounding codomain -PI..PI inwards,
not outwards, causing it to reject perfectly valid roundings of constant
PI.

Changed tests:

dEQP-VK.glsl.builtin.precision.atan2.*

Change-Id: Ib75da103944f773dd819c459b9aa1257c58ca921
Components: Vulkan

Test descriptor_update_template, push_descriptor

Extended the existing binding_model tests with 3 new variants,
'with_template', 'with_push_template' and 'with_push'.

The least invasive method to support push descriptor updates without
template was to persist the DescriptorSetUpdateBuilder object in the
test instance classes, so that it can be referenced when building the
command buffer.

Change-Id: I731fa41c5ff99ab1d390d77d7aabb4424e091e35
(cherry picked from commit 7f5ba3914c1d9033dd4f896ef6fcaa22f5c5e038)

Fix coding error in protocol detection

* add 'return' statement to detectProtocol
* call detectProtocol as a class member not as a global function

Components: Framework
Affects: KC CTS checkout

Change-Id: Id226166d0c35f6d10b23307e5cdec0fbd29102e8

Add sRGB skip decode test

New Tests:
- deqp-case=dEQP-GLES31.functional.srgb_texture_decode.*

Change-Id: I0257e63b15ee0334efad985411d8070ddecdd51a

Reduce read pixel calls in copy image tests

Reduce glReadPixels calls in the copy image tests by rendering multiple
texture sub-images for each read back. Significant improvement for
certain platforms, no impact on others.

Bug: 33965234
Test: Run tests on Pixel C and Pixel XL, Check that old N9 results still
have same failures. Spot checks with 240x200 target surface.
Change-Id: I133c56b7855a664f164f1eb0b1307259462e098f

Restructure copy image test iterations

Packing test iterations so that work per buffer swap is maximized
without running the risk of hitting per-iteration timeout. Yields
execution time drops of roughly 50% on at least 2 GPU architectures.

Bug: 33965234
Test: Copy image tests on multiple devices and L MR1 N9 for detection capability.
Change-Id: I7de5231da0d8a1d2bd22ec3741f1481e2ebd5475

Add create functions for common Vulkan types.

These functions simplify creating the Vulkan objects by taking the
arguments directly instead of passing through struct. Multiple test
cases already reimplement these functions.

Change-Id: I09f6b0163b5aa2198617230767cd072185afe45c

Improve precision handling in texture.explicit_lod

This change improves precision requirement handling in
dEQP-VK.texture.explicit_lod test group by adding separate controls for
format conversion and filtering math.

Changed tests:
dEQP-VK.texture.explicit_lod.*

Components: Vulkan

Change-Id: Id64da1d63d87d02c73c757a3f2d5d50f2fbc19c0

Start new mustpass for Vulkan CTS 1.0.3

Vulkan CTS 1.0.2 mustpass is now archived under mustpass/1.0.2 and new
mustpass can be found under mustpass/1.0.3.

Components: Vulkan

Change-Id: I7c073dc7436a434558ba1710e01988377a8d686c

Add a non-separable TCS-but-no-TES subcase.

   "Linking can fail for [...] any of the following reasons:

* program contains an object to form a tessellation control
  shader [...] and [...] the program is not separable and
  contains no object to form a tessellation evaluation shader"

Add a test to ensure the implementation does raise a link error.

The new subcase is part of:
dEQP-GLES31.functional.debug.negative_coverage.*.tessellation.single_tessellation_stage

Change-Id: Icb29d2da50b2c327bfd2196af9be56bb7d8644f9

Make program separable in TCS-but-no-TES draw time check test.

Section 7.3 of the OpenGL ES 3.2 specification says:

   "Linking can fail for [...] any of the following reasons:

    * program contains an object to form a tessellation control
      shader [...] and [...] the program is not separable and
      contains no object to form a tessellation evaluation shader"

Since the test is trying to check for a draw time error, it should use
a separable program - otherwise we may hit a link error.

Fixes a bug in:
dEQP-GLES31.functional.debug.negative_coverage.*.tessellation.single_tessellation_stage

Change-Id: Ida906e21c3e3534fd2a8c7bac0334083881bdb19

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

Change-Id: I0fc80b05b1d3a795fff26f6c971e57064d224478

Update Android CTS Vulkan mustpass

Following commits modify Vulkan CTS 1.0.2 mustpass:

 * fbaba83dbee101b3096ce33f21ae398efee3fa67
 * c09596ed4b8d17f057f77a42475d59c1ac126797

This change simply updates Android version of mustpass to match Khronos
Vulkan CTS 1.0.2 mustpass.

Change-Id: I79e2696ec4c457d11b59e03cf2a73977ab98d8d5
Components: AOSP

Update Android CTS Vulkan mustpass to match 1.0.2

This CL updates Android CTS mustpass for Vulkan to match Khronos Vulkan
CTS 1.0.2.0.

Change-Id: Ia2f82b5f6675c67940679eb98f59510f264d62bf
(cherry picked from commit 15f342274521c1be1032e993dbfd4a46b878f337)

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

Change-Id: Ie75f95c0972b43843398ca6d68fd39a538f23761

Merge vk-gl-cts/vulkan-cts-1.0.1 into vk-gl-cts/vulkan-cts-1.0.2

Change-Id: Ida077bae420c265f4a92d78d68b0f1c9b8d1c85c

Write gl_PointSize value when it is undefined

gl_PointSize is undefined unless expicitly written to. The only
exception to this rule is the case when
shaderTessellationAndGeometryPointSize is disabled, in which case it
defaults to 1.0 for these stages.

Since we don't have access to the physical device features at shader
compile time, implement two variants of the tessellation evaluation
shader when point mode is enabled, one that writes gl_PointSize and
one that doesn't and decide the one we need at run time based on
whether the feature is enabled or not.

Components: Vulkan
Fixes VK-GL-CTS issue: 159

Affects:
dEQP-VK.tessellation.primitive_discard.*point_mode

Change-Id: Idadbc70a704f00ed601ff65afeb2a3edfbeb3f19

Fix a few bugs in a EGL_KHR_no_config_context test.

The dEQP-EGL.functional.create_context.no_config test had the eglBindAPI
check backwards - so it was skipping APIs assuming they failed when they
actually succeeded.

Secondly, it failed to actually set the test result to "pass" if all
cases passed, which led to assertion failures:

    Assertion `testResult != QP_TEST_RESULT_LAST' failed.

Change-Id: I7a20467012fd6c174f79679bbf640917c327574c

Fix crashes in dEQP-VK.glsl.matrix.inverse.*

This is a regression caused by altering the value of the GRID_SIZE
in commit 2ba66fa2, which results in the inverse() tests generating
input matrices which determinant is zero, and thus, cannot be
inverted, which makes the tests hit an assertion.

This patch adds a mechanism for shader tests to specify a grid size
different from the default. This allows shader testst that are
sensitive to this value, such as matrix inverse tests, to define
their own requirements.

Fixes VK-GL-CTS isssue: 66
Components: Vulkan

Affects:
dEQP-VK.glsl.matrix.inverse.*

Change-Id: I31a34c2d156ee5b98d2de1b10ac4cec1a90d480e

geometry_shader/api: Fix linking error

The tests redeclared the built-in 'gl_PerVertex' interface block in the
Vertex shader when run in a GL context. However, they did not redeclare
'in gl_PerVertex gl_in[]' and they used it, leading to an undesirable
linking error which is not what the test wished to check.

The patch adds the 'in gl_PerVertex gl_in[]' redeclaration to the
Geometry shader when the tests are run under GL.

From the GLSL 4.5 specification, "7.1 Built-In Language Variables"
(page 130):

"If multiple shaders using members of a built-in block belonging to
the same interface are linked together in the same program, they
must all redeclare the built-in block in the same way, as described
in section 4.3.9 “Interface Blocks” for interface-block matching, or
a link-time error will result. It will also be a link-time error if
some shaders in a program redeclare a specific built-in interface
block while another shader in that program does not redeclare that
interface block yet still uses a member of that interface block. If
a built-in block interface is formed across shaders in different
programs, the shaders must all redeclare the built-in block in the
same way (as described for a single program), or the values passed
along the interface are undefined."

VK-GL-CTS Issue: 130

Components: OpenGL

Affects:

GL44-CTS.geometry_shader.api.insufficient_emitted_vertices
GL44-CTS.geometry_shader.api.incompatible_draw_call_mode
GL45-CTS.geometry_shader.api.insufficient_emitted_vertices
GL45-CTS.geometry_shader.api.incompatible_draw_call_mode

Change-Id: I7d5f54b287b778e46376ef6ac9effcd90778705c

Take GL_MAX_IMAGE_SAMPLES into account

GL45-CTS.shader_texture_image_samples_tests.image_functional_test
tests imageSamples(), but binds images without checking whether their
number of samples are within the bounds allowed by
GL_MAX_IMAGE_SAMPLES.

This change brings the test in line with the shader_image_size_test,
which also skips unsupported sample counts.

Affects:
GL45-CTS.shader_texture_image_samples_tests.image_functional_test

Components: OpenGL
VK-GL-CTS issue: 29
Change-Id: Ia945a0f6472317f9014d4f820572eb006743450a

Add CTS_ARB_sparse_texture_clamp test specification proposal

This is CTS_ARB_sparse_texture_clamp test specification proposal.

components: OpenGL

VK-GL-CTS issue: 60

Change-Id: Ic11f1752e7f875d2cf55ad3fa4bf954c8a714fe3

Enable GL support for Android build

- Add GL45Activity which creates a GL4.5 core activity
- Enable android build scripts to take 'gl' as argument for
  -glcts-gtf-target
- update documentation for Android building and running

Components: Framework, AOSP, OpenGL

Change-Id: I2231adac800ac641287f157c712a25d9ddd66853

Limit changes by xor to upper 8 bits in mixed atomic tests am: 6bc3c7a634
am: eef2e716e7

Change-Id: Icadf536794c067056e66f6a2041fbb937952f35a

Limit changes by xor to upper 8 bits in mixed atomic tests
am: 6bc3c7a634

Change-Id: I354ee074ad57d53c2b4b0a84d8bc02a02912f19d

Android: Warn about inconsistent OpSMod and OpSRem

Change-Id: Ib0e32c55f9236e4f77ad461341329cce41c22e0a

Add copy image to image and blit image tests with various image layouts. Refactor copy and blit tests generation.

Tests took into account only optimal layout cases. Added general and mixed ones.
Structure of all copy and blit tests was too flat. Lowered granularity of test groups.

New tests:
- dEQP-VK.api.copy_and_blit.image_to_image.all_formats.*optimal_optimal*
- dEQP-VK.api.copy_and_blit.image_to_image.all_formats.*optimal_general*
- dEQP-VK.api.copy_and_blit.image_to_image.all_formats.*general_optimal*
- dEQP-VK.api.copy_and_blit.image_to_image.all_formats.*general_general*

- dEQP-VK.api.copy_and_blit.blit_image.all_formats.*optimal_optimal*
- dEQP-VK.api.copy_and_blit.blit_image.all_formats.*optimal_general*
- dEQP-VK.api.copy_and_blit.blit_image.all_formats.*general_optimal*
- dEQP-VK.api.copy_and_blit.blit_image.all_formats.*general_general*

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

Components: Vulkan

Change-Id: I5724bb7c0a3ccb2e36319a20671761a9b16fe4ed

Merge "Use deFloatIsNaN, not isnan."

Extend gl_FrontFacing tests

Update gl_FrontFacing tests to include more tests as well as to use the
vktDrawUtil utility class

Change-Id: If94fb552f5f9366c79ffa3fcac6f085611a23f65

Add ability to generate images with the reference renderer

*Separate DrawContext into a DrawContext and DrawContextState to reflect
  graphics API's are split between state and action parts
* Add abstract DrawContext class

Test: Test clipping and synchronization primitives

Change-Id: I9f3945c6c3de169325fadea7bf70a9d5d7bfd932

Add Vulkan DrawContext utility class

This is the first in a series of changes to move the DrawContext class
for more general use.

Test: run clipping tests on Vulkan
Affects: dEQP-VK.clipping.*

Change-Id: Ia9b8953d25c10ec2a6c227bf300637dcd62ea898

Use deFloatIsNaN, not isnan.

Discovered because this file includes cmath rather than math.h, so
some header cleanup made isnan go away (it should have been
std::isnan).

Test: mma
Bug: http://b/34724220
Change-Id: Idb6b26a90307f510ad1a118b694a34e32ce7507c

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

Change-Id: I3d822f0aafe2cb7cfae6bd55e6d29303a7dad0cf

dEQP-VK.info.*image_format_properties2* VkImageUsageFlags fix

The usage flags loop iterator starts currently with 0, which is not
a valid image usage value. It is reported as an error by Validation
Layers.

The patch contains also a simple fix for iterating over valid sample
count flag bits.

Modified tests:
dEQP-VK.info.image_format_properties2*
dEQP-VK.info.sparse_image_format_properties2*

Components: Vulkan
VK-GL-CTS issue: 133

Change-Id: I0edbdbc389fa3bd9a71ca4d3b7777ede38a7b353

Android: Test that mandatory extensions are supported

Change-Id: Id0e44299f1d8441bf1e7ecc91d82d3b06400e7e6

Android: Test that no Vulkan layers are enumerated

Change-Id: I43dacf306b468485fc160f175160555b9cf1c86d

Android: Check KHX, GOOGLE, and ANDROID extensions

Change-Id: I6ee24e584666e1f2542adf35943a2973c9392ad9

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

Change-Id: Ia916f3edd0d85b78a7a109c0605908fd8c7793cb

Merge vk-gl-cts/vulkan-cts-1.0.1 into vk-gl-cts/vulkan-cts-1.0.2

Change-Id: I32aa09e131335c13262518708fd5970191aad77b

Fix memory management, image layout issues in uploadImageSparse()

Fixes an issue where ShaderRenderCaseInstance::uploadSparseImage()
would first configure sparse image bindings and then deallocate
the memory objects, prior to actually sampling the image.

Fixes an issue where the sparse image, after having been filled with
contents, would first be transitioned to SHADER_READ_ONLY_OPTIMAL
layout, and then used in a DS which was expecting it to be in
GENERLA layout.

Finally, the patch addresses a problem, where the function would not
correctly recognize the VK_SPARSE_IMAGE_FORMAT_SINGLE_MIPTAIL_BIT,
as reported by the running driver.

The change affects dEQP-VK.glsl.*sparse* tests

VK-GL-CTS issue: 156

Change-Id: Ie5717841fe0ef9c0672b5075380ff96bba9855b9
(cherry picked from commit 70fc0800efe725f4f91f4d03b09aa0e567dbb09c)

Do not require unsupported sparse images in image_format_properties test

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

This change removes the requirement for supporting sparse residency images:
- with compressed format
- with non-color format
- with format of pixel size that is not power ot two (for example: RGB).

 Additionally do not require 1D sparse residency images (not in spec),
 and require 2D/3D residency images basing on the device capabilities.

Spec 28.1. Sparse Resource Features:
"A sparse image created using VK_IMAGE_CREATE_SPARSE_RESIDENCY_BIT
supports all non-compressed color formats with power-of-two element
size that non-sparse usage supports.

(...)

sparseResidencyImage2D: Support for creating 2D single-sampled VkImage
objects with VK_IMAGE_CREATE_SPARSE_RESIDENCY_BIT.

sparseResidencyImage3D: Support for creating 3D VkImage objects with
 VK_IMAGE_CREATE_SPARSE_RESIDENCY_BIT.
"

Component: Vulkan
VK-GL-CTS Issue: 131

(cherry picked from commit 5534b44a15b530ece4b310e1dd935fb9b0cefeb6)

Change-Id: Ia2481fbcd12a32a6470af26df213a2f930c2e6db

Merge "Fix out of bounds accesses in TexImage*DParamsCase."