On 4 May 2017 at 10:33, Daniel Stone <[email protected]> wrote: > Though most swapchain operations used a queue, they were racy in that > the object was created with the queue only set later, meaning that its > event could potentially be dispatched from the default queue in between > these two steps. > > Use proxy wrappers to avoid this race, also assigning wl_buffers created > for the swapchain to the event queue. > > Signed-off-by: Daniel Stone <[email protected]> > --- > src/vulkan/wsi/wsi_common_wayland.c | 52 > +++++++++++++++++++++++++------------ > 1 file changed, 36 insertions(+), 16 deletions(-) > > diff --git a/src/vulkan/wsi/wsi_common_wayland.c > b/src/vulkan/wsi/wsi_common_wayland.c > index c67ba089f9..cb72183efd 100644 > --- a/src/vulkan/wsi/wsi_common_wayland.c > +++ b/src/vulkan/wsi/wsi_common_wayland.c > @@ -510,6 +510,7 @@ struct wsi_wl_swapchain { > struct wl_event_queue * queue; > struct wl_surface * surface; > uint32_t surface_version; > + struct wl_drm * drm_wrapper; > Based on a quick skim 3/4 aligns with commit 36b9976e1f99e8070c67cb8a255793939db77d02. At the same time we don't seem to have a drm_wrapper in egl/wayland - is that intentional?
The series looks like important bugfixes - please add a Fixes or Cc: mesa-stable tag. Thanks Emil _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
