zink: move the kopper present fence to the displaytarget object
this is otherwise broken when front/back buffers share the same dt
Fixes: 8ade5588e3
("zink: add kopper api")
Reviewed-by: Adam Jackson <ajax@redhat.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/16070>
This commit is contained in:
parent
c5cc9ed29b
commit
64d5f71bbc
|
@ -148,6 +148,7 @@ zink_kopper_deinit_displaytarget(struct zink_screen *screen, struct kopper_displ
|
|||
VKSCR(DestroySurfaceKHR)(screen->instance, cdt->surface, NULL);
|
||||
cdt->swapchain = cdt->old_swapchain = NULL;
|
||||
cdt->surface = VK_NULL_HANDLE;
|
||||
util_queue_fence_destroy(&cdt->present_fence);
|
||||
}
|
||||
|
||||
static struct kopper_swapchain *
|
||||
|
@ -322,6 +323,7 @@ zink_kopper_displaytarget_create(struct zink_screen *screen, unsigned tex_usage,
|
|||
cdt->refcount = 1;
|
||||
cdt->loader_private = (void*)loader_private;
|
||||
cdt->info = *info;
|
||||
util_queue_fence_init(&cdt->present_fence);
|
||||
|
||||
enum pipe_format srgb = PIPE_FORMAT_NONE;
|
||||
if (screen->info.have_KHR_swapchain_mutable_format) {
|
||||
|
@ -403,7 +405,7 @@ kopper_acquire(struct zink_screen *screen, struct zink_resource *res, uint64_t t
|
|||
}
|
||||
if (timeout == UINT64_MAX && util_queue_is_initialized(&screen->flush_queue) &&
|
||||
p_atomic_read_relaxed(&cdt->swapchain->num_acquires) > cdt->swapchain->max_acquires) {
|
||||
util_queue_fence_wait(&res->obj->present_fence);
|
||||
util_queue_fence_wait(&cdt->present_fence);
|
||||
}
|
||||
VkSemaphoreCreateInfo sci = {
|
||||
VK_STRUCTURE_TYPE_SEMAPHORE_CREATE_INFO,
|
||||
|
@ -600,7 +602,7 @@ zink_kopper_present_queue(struct zink_screen *screen, struct zink_resource *res)
|
|||
cpi->info.pResults = NULL;
|
||||
res->obj->present = VK_NULL_HANDLE;
|
||||
if (util_queue_is_initialized(&screen->flush_queue)) {
|
||||
util_queue_add_job(&screen->flush_queue, cpi, &res->obj->present_fence,
|
||||
util_queue_add_job(&screen->flush_queue, cpi, &cdt->present_fence,
|
||||
kopper_present, NULL, 0);
|
||||
} else {
|
||||
kopper_present(cpi, screen, 0);
|
||||
|
|
|
@ -62,6 +62,7 @@ struct kopper_displaytarget
|
|||
struct kopper_swapchain *old_swapchain;
|
||||
|
||||
struct kopper_loader_info info;
|
||||
struct util_queue_fence present_fence;
|
||||
|
||||
VkSurfaceCapabilitiesKHR caps;
|
||||
VkImageFormatListCreateInfoKHR format_list;
|
||||
|
|
|
@ -104,7 +104,6 @@ zink_destroy_resource_object(struct zink_screen *screen, struct zink_resource_ob
|
|||
|
||||
zink_descriptor_set_refs_clear(&obj->desc_set_refs, obj);
|
||||
if (obj->dt) {
|
||||
util_queue_fence_destroy(&obj->present_fence);
|
||||
FREE(obj->bo); //this is a dummy struct
|
||||
} else
|
||||
zink_bo_unref(screen, obj->bo);
|
||||
|
@ -521,7 +520,6 @@ resource_object_create(struct zink_screen *screen, const struct pipe_resource *t
|
|||
if (loader_private) {
|
||||
obj->bo = CALLOC_STRUCT(zink_bo);
|
||||
obj->transfer_dst = true;
|
||||
util_queue_fence_init(&obj->present_fence);
|
||||
return obj;
|
||||
} else if (templ->target == PIPE_BUFFER) {
|
||||
VkBufferCreateInfo bci = create_bci(screen, templ, templ->bind);
|
||||
|
|
|
@ -91,7 +91,6 @@ struct zink_resource_object {
|
|||
bool new_dt;
|
||||
bool dt_has_data;
|
||||
bool indefinite_acquire;
|
||||
struct util_queue_fence present_fence;
|
||||
// }
|
||||
|
||||
|
||||
|
|
Loading…
Reference in New Issue