diff options
| author | Olivier Fourdan <ofourdan@redhat.com> | 2021-11-17 09:56:52 +0100 |
|---|---|---|
| committer | Olivier Fourdan <fourdan@gmail.com> | 2021-12-02 08:13:49 +0000 |
| commit | 6dd9709bd85cf5de4067887818c864220b951355 (patch) | |
| tree | d3faa904b2957d2fdbd5e70e037fff9444d3eacd /hw | |
| parent | c5d1fed9fa32244739677ec5c58ea87b261e023b (diff) | |
xwayland/eglstream: Prefer EGLstream if available
Currently, when given the choice, Xwayland will pick the GBM backend
over the EGLstream backend if both are available, unless the command
line option “-eglstream” is specified.
The NVIDIA proprietary driver had no support for GBM until driver series
495, but starting with the driver series 495, both can be used.
But there are other requirements with the rest of the stack, typically
Mesa, egl-wayland, libglvnd as documented in the NVIDIA driver.
So if the NVIDIA driver series 495 gets installed, Xwayland will pick
the GBM backend even if EGLstream is available and may fail to render
properly.
To avoid that issue, prefer EGLstream if EGLstream and all the Wayland
interfaces are available, and fallback to GBM automatically unless
“-eglstream” was specified.
With this, the compositor, given the choice, can decide which actual
backend Xwayland would use by advertising (or not) the Wayland
"wl_eglstream_controller" interface.
This change has no impact on compositors which do not have support for
EGLstream in the first place.
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>
Acked-by: Michel Dänzer <mdaenzer@redhat.com>
Diffstat (limited to 'hw')
| -rw-r--r-- | hw/xwayland/xwayland-glamor.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/hw/xwayland/xwayland-glamor.c b/hw/xwayland/xwayland-glamor.c index 551334cb6..24620605d 100644 --- a/hw/xwayland/xwayland-glamor.c +++ b/hw/xwayland/xwayland-glamor.c @@ -444,14 +444,10 @@ xwl_glamor_select_eglstream_backend(struct xwl_screen *xwl_screen) void xwl_glamor_select_backend(struct xwl_screen *xwl_screen, Bool use_eglstream) { - if (use_eglstream) { - if (!xwl_glamor_select_eglstream_backend(xwl_screen)) + if (!xwl_glamor_select_eglstream_backend(xwl_screen)) { + if (!use_eglstream) xwl_glamor_select_gbm_backend(xwl_screen); } - else { - if (!xwl_glamor_select_gbm_backend(xwl_screen)) - xwl_glamor_select_eglstream_backend(xwl_screen); - } } Bool |