vulkan/wsi/wayland: remove unnecessary wl_proxy_set_queue call
authorSimon Ser <contact@emersion.fr>
Thu, 24 Jun 2021 18:27:42 +0000 (20:27 +0200)
committerSimon Ser <contact@emersion.fr>
Thu, 8 Jul 2021 10:05:20 +0000 (12:05 +0200)
A wl_proxy inherits its queue from its parent.
display->dmabuf.wl_dmabuf already has its queue correctly set up,
so it's unnecessary to set it again on the child
zwp_linux_buffer_params_v1 proxy.

Signed-off-by: Simon Ser <contact@emersion.fr>
Acked-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/11589>

src/vulkan/wsi/wsi_common_wayland.c

index 4ebc059..d5069ac 100644 (file)
@@ -954,7 +954,6 @@ wsi_wl_image_init(struct wsi_wl_swapchain *chain,
    if (display->dmabuf.wl_dmabuf) {
       struct zwp_linux_buffer_params_v1 *params =
          zwp_linux_dmabuf_v1_create_params(display->dmabuf.wl_dmabuf);
-      wl_proxy_set_queue((struct wl_proxy *) params, chain->display->queue);
 
       for (int i = 0; i < image->base.num_planes; i++) {
          zwp_linux_buffer_params_v1_add(params,