Avoid division by zero

Fix warnings in copies and blit tests

Merge 70-copies-and-blitting

Merge branch 'dkoch_mv_draw_shaders' into 'master'

Move draw shaders to correct location

The draw/ tests are expecting the path vulkan/draw/ instead of just draw/.

See merge request !212

Merge branch 'imageLoadStore-too-small-descriptorPool' into 'master'

vktImageLoadStoreTests: fix *__layer test cases - fix too small descriptor pool

In dEQP-VK.image.store.1d_array.r32g32b32a32_sfloat_single_layer test case, there are 8 descriptor sets allocated from descriptor pool that has space only for one.

See merge request !210

Merge branch 'renderpass-proper-aspect-for-cmdClearAttachments' into 'master'

Renderpass test: use proper aspect for cmdClearAttachments

dEQP-VK.renderpass.attachment.3.206 tests uses this chunk of code to clear D32 image, so VK_IMAGE_ASPECT_STENCIL_BIT is invalid.

See merge request !209

Merge branch '80-ApiObjectManagement-desriptorPool-size' into 'master'

Fix descriptor pool size in api.object_management.multiple_shared_resources.descriptor_set

Fixes issue #80

- there are multiple sets allocated, so desciptor VkDescriptorPoolSize.descriptorCount must be multipled by number
of sets

See merge request !208

Merge branch 'api-bufferView-image-layout' into 'master'

vktApiBufferViewAccessTests: image cannot be created with COLOR_ATTACHMENT_OPTIMAL layout.

- According to spec, the only valid layouts in vkCreateImage are UNDEFINED and PREINITIALIZED
- This changes the test so image is cratead with valid layout, than transitioned to COLOR_ATTACHMENT_OPTIMAL using additional barrier

See merge request !207

Merge branch '117-push-constant-wrong-glsl' into 'master'

Fix bugs of the shaders in push constant tests

This fix issue #117 and #118

-fix the offset of the uniform block member to align with it's size

-modify the shader to fix wrong expected value for dEQP-VK.pipeline.push_constant.graphics_pipeline.data_update_partial_1

See merge request !199

Merge branch 'jekstrand-fix-timestamp-c++11' into 'master'

PipelineTimestampTests: Stop relying on C++11 features

The cbegin and cend functions weren't added to std::vector until c++11.
They also aren't really serving a purpose here given that there's no
automatic type inference going on, just a regular for loop.  This fixes the
build on Fedora 23 when using clang and default dEQP build flags.

See merge request !198

Merge branch '76-fix-commandbuffer-params' into 'master'

fix VkCommandBufferBeginInfo struct values, fixes #76

just a short fix to change those two mixed struct values

See merge request !190

Merge branch 'jbolz_fix_ubo' into 'master'

Fix ubo.per_block_buffer tests

See merge request !187

Merge branch 'jekstrand-binding-model-hang-fix' into 'master'

api.binding_model: Wait for idle before throwing NotSupportedError

If we don't, then the GPU could be busy and some of the objects destroyed
by the exception propagation could be in-use.  This is something of a
stop-gap measure; the problem of exceptions and the GPU being busy is
something that needs to be solved in general.  However, it does fix GPU
hangs on some drivers.

This is related to #20.

See merge request !185

api.binding_model: Wait for idle before throwing NotSupportedError

If we don't, then the GPU could be busy and some of the objects destroyed
by the exception propagation could be in-use.  This is something of a
stop-gap measure; the problem of exceptions and the GPU being busy is
something that needs to be solved in general.  However, it does fix GPU
hangs on some drivers.

This is related to #20.

Move draw shaders to correct location

The draw/ tests are expecting the path vulkan/draw/ instead of just draw/.

Style fixes for the Copy and blitting

Add copy BufferToBuffer tests.

Add Image to image copies

Fix the wrong bufferSize in CopiesAndBlitting

Set queueFamilyIndices in CopiesAndBlitting

Rename ImageToImageCopies to CopyImageToImage

Fix warnings in the CopiesAndBlitting

Remove leftover comment

Add CopyBufferToImage test

Add image to image test

Finish the buffer to buffer copies base class

Rename testgroup of the copies and blitting

Implement copyRegions for the buffer to buffer copies

Remove m_regions from the CopiesAndBlitting class

Fix the CopyImageToBuffer test

Add WIP version of BufferToBufferTest.

Improve TestParams in CopiesAndBlittingTests

Several fixes around the readImage and copyImage

Improve the calculateSize utility function

Refactor uploadBuffer and uploadImage.

Fix the upload image and the ImageToBuffer test

Remove BufferType enum, add uploadBuffer.

Add readImage

Implement uploadImage utility function in CopiesAndBlittingTests

Refactor checkResult function

Add generateExpectedResult function

Implement generateBuffer function

Add base of the CopyImageToBuffer test

Add base of the copy image to image

Add base of Copies And Blitting tests

Copies and blitting: update the test specification

vktImageLoadStoreTests: fix *__layer test cases - fix too small descriptor pool

In dEQP-VK.image.store.1d_array.r32g32b32a32_sfloat_single_layer test case, there are 8 descriptor sets allocated from descriptor pool that has space only for one.

Renderpass test: use proper aspect for cmdClearAttachments

dEQP-VK.renderpass.attachment.3.206 tests uses this chunk of code to clear D32 image, so VK_IMAGE_ASPECT_STENCIL_BIT is invalid.

Fix descriptor pool size in api.object_management.multiple_shared_resources.descriptor_set

- there are multiple sets allocated, so desciptor VkDescriptorPoolSize.descriptorCount must be multipled by number of sets

vktApiBufferViewAccessTests: image cannot be created with COLOR_ATTACHMENT_OPTIMAL layout.

- According to spec, the only valid layouts in vkCreateImage are UNDEFINED and PREINITIALIZED
- This changes the test so image is cratead with valid layout, than transitioned to COLOR_ATTACHMENT_OPTIMAL using additional barrier

Fix couple of invalid VkBufferCreateInfos and ws violations in command buffer tests

Merge 90-command-buffers

Merge 41-image-load-store-tests

Fix whitespace violations

Fix VkCommandPoolCreateInfo in compute test utils.

Merge 24-compute-shader-tests

Merge branch '119-fix-buffer-size' into 'master'

SSBO: fix buffer size when creating a buffer descriptor

Fixes #119

See merge request !203

Merge branch 'jbolz_glsl_matrix1' into 'master'

fix matrix download in glsl.matrix tests

See merge request !195

Merge branch 'shaderRenderLoopTest-fix-getFloatFractionUniformType' into 'master'

Shader render loop test: fix getFloatFractionUniformType  function

It seems getFloatFractionUniformType should return fraction "uniform types", not integral values.

it is used to generate uniform values used in test - which, according to shader sources, should be fractions.

See merge request !202

Merge branch 'draw-tests-indexed-instanced' into 'master'

draw tests: make draw_instanced_indexed_* test use instancing (test case list bug)

See merge request !201

Merge branch 'clean-up-mr-189' into 'master'

Remove unused code in opaque type indexing tests

See !189

See merge request !196

Merge branch 'pipeline-input_assembly-pointSize' into 'master'

InputAssemblyTests: fix point list test

The test compares image generated by Vulkan to image rendered by reference renderer.

As Vulkan path sets gl_PointSize to 3, same must be done in ref renderer to make test pass.

See merge request !200

Merge branch 'ssbo-initialize-acBuffer' into 'master'

Fix dEQP-VK.ssbo tests: initialize the contents of the result atomic counter buffer

The atomic counter buffer used for storing test results is not initialized - as shaders are only incrementing atomit counters, the results may be undefined, thus making the test randomly fail.

See merge request !194

Merge branch 'buffer_view_alloc_first' into 'master'

We need to allocate memory before we can create a buffer view - issue wth the dEQP-VK.api.buffer_view

There were issues with the dEQP-VK.api.buffer_view test group that Jeannot has addressed with this patch. Please merge.

See merge request !197

Review comments and clang error fixes

Image tests

Issue #41

- store, load/store
- format reinterpret
- memory qualifiers
- imageSize

SSBO: fix buffer size when creating a buffer descriptor

Fixes #119

vktShaderRenderLoopTests: fix getFloatFractionUniformType function

it seems getFloatFractionUniformType should return fraction "uniform types", not integral values.

draw tests: make draw_instanced_indexed_* test use instancing (test case list bug)

Fix clang compilation errors

Updated compute tests to API version 210

Add compute shader tests

Basic compute shader tests

    Reference implementation:
    modules/gles31/functional/es31fBasicComputeShaderTests.cpp

    - empty shader
    - UBO to SSBO invert
    - SSBO to SSBO invert
    - invert SSBO in-place
    - write to multiple SSBO's
    - SSBO local barrier
    - shared var
    - shared var atomic operation
    - copy image to SSBO
    - copy SSBO to image
    - SSBO barrier
    - image barrier
    - image atomic operation

Indirect Compute Dispatch Tests

    Ported indirect compute dispatch tests from OpenGLES 3.1 to Vulkan.

    Reference implementation:
    modules/gles31/functional/es31fIndirectComputeDispatchTests.cpp

Compute Shader Built-in variable tests

    Tests that check if compute shader build in variables store correct
    values. Tests ported from OpenGLES 3.1

    Reference file:
    modules\gles31\functional\es31fComputeShaderBuiltinVarTests.cpp

    NOTE:
    The tests were changed from the original to use std430 ssbo data
    alignment. In the original tests the alignment of ssbo data structures was
    queried via a call to

    const Glenum queryParam = GL_ARRAY_STRIDE;
    getProgramResourceiv(program, GL_BUFFER_VARIABLE, index, 1, &queryParam, 1, DE_NULL, (int*)&value);

    We do not see an equivalent vkGet function in Vulkan. Does it mean that
    packed and shared layouts are not supported by Vulkan?

Fix dEQP-VK.ssbo tests: initialize the contents of the result atomic counter buffer

InputAssemblyTests: fixed point list test

1. Updated the test spec according to the new Vulkan spec.
2. Added all test cases from test spec to the list. The ones that are not yet implemented or don't work properly are commented out.

TODO: Dummy tests for all missing cases.

moved order of execution test to new api

1. Created simple tests for Command Pool management.
2. Removed free command buffers tests as there's not way to check the result

Removed incorrect tests related to improper recording of buffers.

Implemented a simple memory allocator and the pool_create_non_null_allocator test.

fixed merge request 149 problems

1. Implemented recording with active queries tests.
2. Removed two test cases for end recording as this command is covered extensively by all other tests.

1. Removed invalid buffer reset tests.
2. Fixed formatting.

Removed stubs of tests what won't be implemented.

Removed vktSimpleAllocator.hpp and replaced it with the framework allocation utility.

changed vktApiComputeInstanceResultBuffer to ComputeInstanceResultBuffer

Fix bugs of the shaders in push constant tests

Change-Id: Icbb47b2a184e5b28365826658c260bff3e80139d

PipelineTimestampTests: Stop relying on C++11 features

The cbegin and cend functions weren't added to std::vector until c++11.
They also aren't really serving a purpose here given that there's no
automatic type inference going on, just a regular for loop.  This fixes the
build on Fedora 23 when using clang and default dEQP build flags.

We need to allocate memory before we can create a buffer view

Convert indentation to tabs in timestamp tests

Fix a few warnings in timestamp tests

Merge 29-timestamp-tests

Remove unused code in opaque type indexing tests

See !189

Fix invalid include guards and ws violations in pipeline barrier tests

Enable draw tests and fix a few warnings

Merge 77-draw-tests

Merge 110-OpFRem-spirv-assembly

Merge branch 'push-constants-bufferCreateInfo' into 'master'

pipeline.push_constant test: fix wrong ordering of BufferCreateInfo structure parameters

Parameters for BufferCreateInfo structure are given in wrong order

See merge request !193

Merge branch 'jbolz_shader_executor_misc2' into 'master'

ShaderExecutor: fix zero-sized buffer and copy command buffer submission

See merge request !189

Merge branch 'api-createBuffer-usage0' into 'master'

api.buffer.createBuffer: buffer usage cannot be 0.

Spec: (Valid Usage for VkBufferCreateInfo): "usage must not be 0"

See merge request !191

fix matrix download in glsl.matrix tests

pipeline.push_constant test: fix wrong ordering of BufferCreateInfo structure parameters

draw tests: fix review comments

fix commented type for bufferCount

api.buffer.createBuffer: buffer usage cannot be 0.

Spec: (Valid Usage for VkBufferCreateInfo): usage must not be 0

fix VkCommandBufferBeginInfo struct values, fixes #76

# This is a combination of 3 commits.
# The first commit's message is:
update to test spec

Command buffer submission tests.

Updated the part of test spec relevant to command buffer submission.

Updated the part of the test spec relevant to Command Pools

A minor fix in spec formatting

Removed three irrelevant testcases from the part of the test spec related to secondary command buffers.

Removed three irrelevant testcases from the part of the spec related to FrameBuffers.

1. Removed unused #include statements
2. Fixed formatting

fixed the formatting problem in four files

more fixes to formatting

even more whitespace fixes

removed unnecessary includes, added google to copyright

removed resetting buffers that should not take place

added test for secondary commnad buffers execution - currently crashing

order of execution test - first

Updated the test spec according to API v170.2

Updated the part of test spec relevant to command buffer submission.

A minor fix in spec formatting

Removed three irrelevant testcases from the part of the test spec related to secondary command buffers.

Removed three irrelevant testcases from the part of the spec related to FrameBuffers.

removed unnecessary includes, added google to copyright

removed resetting buffers that should not take place

command buffers recording tests

added first command buffer test, fix to nexus compilation

added test for secondary commnad buffers execution - currently crashing

Updated the test spec according to API v170.2

order of execution test - first

Updated the part of test spec relevant to command buffer submission.

A minor fix in spec formatting

Removed three irrelevant testcases from the part of the test spec related to secondary command buffers.

Removed three irrelevant testcases from the part of the spec related to FrameBuffers.

removed unnecessary includes, added google to copyright

1. Removed all compilation errors:
  - fixed all testcases barring order_of_execution
  - commented out order_of_execution and related code in ComputeInstanceResultBuffer.cpp

# This is the 2nd commit message:

Commented out the one test that caused a segfault.

# This is the 3rd commit message:

Reverted several unintended changes.

Draft of test spec for Command Buffers.

Fixed two typos pointed out in the merge request review.

Minor updates and fixes for Command Buffers test spec:
1. Split that one big table into several smaller ones, corresponding to chapters in Vulkan's spec.
2. Added placeholders for missing testspecs - allowed commands and render pass

Fixed colum width in Command Buffer testspec

First part of Render Pass test spec.

1. Replaced repeating parts of lengthy descriptions fir ellipses for the sake of clarity.
2. Added three more test case descriptions.

Added the remaining validity tests for structures used while creating render passes.

Added testcases for Render Pass destruction. This concludes the test spec for Render Pass.

Framebuffer testcases covered.

Fixed an error with numbering testcases for Framebuffers

Fixed command buffer test spec according to Alon's pointers.
1. Removed A LOT of testcases that would result in undefined behaviour.
2. Added a number of testcases to better cover possible uses of the command buffer API
3. Cleaned up the section of the document related to command buffers

added first command buffer test, fix to nexus compilation

added test for secondary commnad buffers execution - currently crashing

1. Command Buffer lifetime tests
- explicit reset
- implicit reset
- bulk reset

2. Pulled the timeout variable out and made it a global (did not modify code in first two test cases)

Updated the test spec according to API v170.2

Fixed style errors in test cases related to resetting command buffers.

order of execution test - first

1. Added CK_CHECK whenever possible
2. Minor formatting fixes
3. Removed several redundance vk:: uses
4. Switched from using wrapper methods to direct calls to DeviceInterface's methods when more than one buffer and event are needed.

Fix binding issues with opaque_type_indexing tests. Descriptor layout must go through builder, fix binding number for UBO array.

Fix a bug and some coding style issues after 2nd code review.

Add Context::getDeviceProperties. Fix a couple tessellation issues