diff options
| author | Samuel Pitoiset <[email protected]> | 2017-11-22 16:13:28 +0100 |
|---|---|---|
| committer | Samuel Pitoiset <[email protected]> | 2017-11-24 12:26:35 +0100 |
| commit | 1cc00b8e0eb839096c1d6856df9213d6bad474ef (patch) | |
| tree | 35dca15301cde45f214c3292a6f9307b5225ee74 /src | |
| parent | dc391a406ab03fb5f2ba6f7042e6216b52e2ceab (diff) | |
Revert "radv: remove unnecessary memset() in radv_AllocateCommandBuffers()"
This fixes two CTS regressions:
- dEQP-VK.api.object_management.alloc_callback_fail_multiple.command_buffer_primary
- dEQP-VK.api.object_management.alloc_callback_fail_multiple.command_buffer_secondary
These two tests are part the mustpass lists, so presumably they
are correct and my change was wrong.
This reverts commit 0f68208f1d1d3b7b2963dab40e84c60212518692.
Signed-off-by: Samuel Pitoiset <[email protected]>
Reviewed-by: Bas Nieuwenhuizen <[email protected]>
Diffstat (limited to 'src')
| -rw-r--r-- | src/amd/vulkan/radv_cmd_buffer.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index 7d86eee9791..bd72ba2a878 100644 --- a/src/amd/vulkan/radv_cmd_buffer.c +++ b/src/amd/vulkan/radv_cmd_buffer.c @@ -2139,6 +2139,9 @@ VkResult radv_AllocateCommandBuffers( VkResult result = VK_SUCCESS; uint32_t i; + memset(pCommandBuffers, 0, + sizeof(*pCommandBuffers)*pAllocateInfo->commandBufferCount); + for (i = 0; i < pAllocateInfo->commandBufferCount; i++) { if (!list_empty(&pool->free_cmd_buffers)) { |