Skip to content
  • Jonas Ådahl's avatar
    egl/wayland: Avoid race conditions when on non-main thread · 36b9976e
    Jonas Ådahl authored and Emil Velikov's avatar Emil Velikov committed
    
    
    When EGL is used on some other thread than the thread that drives the
    main wl_display queue, the Wayland EGL dri2 implementation is
    vulnerable to a race condition related to display round trips and global
    object advertisements.
    
    The race that may happen is that after after a proxy is created, but
    before the queue is set, events meant to be emitted via the yet to be
    set queue may already have been queued on the wrong queue.
    
    In order to make it possible to avoid this race, wayland 1.11
    introduced new API that allows creating a proxy wrapper that may be used
    as the factory proxy when creating new proxies via Wayland requests. The
    queue of a proxy wrapper can be changed without effecting what queue
    events emitted by the actual proxy will be queued on, while still
    effecting what default queue proxies created from it will have.
    
    By introducing a wl_display proxy wrapper and using this when performing
    round trips (via wl_display_sync()) and retrieving the global objects (via
    wl_display_get_registry()), the mentioned race condition is avoided.
    
    Signed-off-by: default avatarJonas Ådahl <jadahl@gmail.com>
    Cc: mesa-stable@lists.freedesktop.org
    Reviewed-by: default avatarDaniel Stone <daniels@collabora.com>
    36b9976e