vulkan: Use ALL_COMMANDS_BIT for waits/signals instead of ~0

This is a bit more accurate for what's going on and, while all Mesa
drivers today seem to be ok with extra bits, ensures we're passing a
valid Vulkan thing.

Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/16145>
This commit is contained in:
Jason Ekstrand 2022-04-25 12:25:17 -05:00 committed by Marge Bot
parent 02fea6c179
commit e0f27eca7a

View file

@ -800,7 +800,7 @@ vk_queue_submit(struct vk_queue *queue,
assert(submit->signals[signal_count].sync == NULL);
submit->signals[signal_count++] = (struct vk_sync_signal) {
.sync = mem_sync,
.stage_mask = ~(VkPipelineStageFlags2KHR)0,
.stage_mask = VK_PIPELINE_STAGE_2_ALL_COMMANDS_BIT,
};
}
@ -808,7 +808,7 @@ vk_queue_submit(struct vk_queue *queue,
assert(submit->signals[signal_count].sync == NULL);
submit->signals[signal_count++] = (struct vk_sync_signal) {
.sync = vk_fence_get_active_sync(info->fence),
.stage_mask = ~(VkPipelineStageFlags2KHR)0,
.stage_mask = VK_PIPELINE_STAGE_2_ALL_COMMANDS_BIT,
};
}
@ -1071,7 +1071,7 @@ vk_queue_signal_sync(struct vk_queue *queue,
submit->signals[0] = (struct vk_sync_signal) {
.sync = sync,
.stage_mask = ~(VkPipelineStageFlags2KHR)0,
.stage_mask = VK_PIPELINE_STAGE_2_ALL_COMMANDS_BIT,
.signal_value = signal_value,
};