Merge branch 'synchronization-tests-fixes' into 'vulkan-cts-1.0-dev'
authorPyry Haulos <phaulos@google.com>
Sun, 10 Apr 2016 18:03:10 +0000 (18:03 +0000)
committerPyry Haulos <phaulos@google.com>
Sun, 10 Apr 2016 18:03:10 +0000 (18:03 +0000)
commit4a7fb3bef3227e12c0ce2caef33cc2976a5d91e2
tree8b7a8490d10595c9221f54b802fdaadb073d51de
parentf23cc02846e220d6da2442b41807c716d2804f88
parent271157f1dfdc1129586f47e8064874d107f3edae
Merge branch 'synchronization-tests-fixes' into 'vulkan-cts-1.0-dev'

Various fixes for synchronization tests

- Tests were not querying the available queueCount in the queue family properties.
- Tests were using the "universal" queue family index when the device created in createTestDevice() could be using queues belonging to a different family index.
- 3D image view was being used with a 2D image.
- Command buffer Deleter was not holding any reference to the command pool.

- Fences tests:
    * Fence status VK_NOT_READY was expected between queueSubmit() and waitForFences(). Status could be VK_SUCCESS if the work finishes before waitForFences().
    * It seems that the first waitForFences() was intended for multiple fences, but it was waiting for one of the fences only.
    * Added check for default status of fence 1 (unsignaled).

- Code style:
    * DE_NULL was being used as a boolean parameter in createFences()
    * Use VkResult names instead of error codes in the error messages.
    * Fixed some typos and indentation.

See merge request !491
external/vulkancts/modules/vulkan/vktSynchronization.cpp