<div dir="ltr">Hi<br><div class="gmail_extra"><br><div class="gmail_quote">On 4 June 2018 at 16:24, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 24 May 2018 at 15:10, Olivier Fourdan <<a href="mailto:ofourdan@redhat.com">ofourdan@redhat.com</a>> wrote:<br>
> The command line option "-eglstream" used to enable EGLi stream support<br>
> for NVidia GPU was made available only when Xwayland was built with EGL<br>
> stream support enabled.<br>
><br>
> Wayland compositors who spawn Xwayland have no easy way to tell whether<br>
> or not Xwayland was built with EGL stream support enabled, and adding<br>
> "-eglstream" command line option to Xwayland when it wasn't built with<br>
> EGL support would prevent Xwayland from starting (“Unrecognized option”<br>
> error).<br>
><br>
> Make sure we support the command line option "-eglstream" regardless of<br>
> EGL stream support in Xwayland, obviously without EGL stream support<br>
> this has no effect.<br>
><br>
> Signed-off-by: Olivier Fourdan <<a href="mailto:ofourdan@redhat.com">ofourdan@redhat.com</a>><br>
> ---<br>
>  hw/xwayland/xwayland.c | 10 ++++------<br>
>  1 file changed, 4 insertions(+), 6 deletions(-)<br>
><br>
> diff --git a/hw/xwayland/xwayland.c b/hw/xwayland/xwayland.c<br>
> index 1d6b49979..b4049d2cc 100644<br>
> --- a/hw/xwayland/xwayland.c<br>
> +++ b/hw/xwayland/xwayland.c<br>
> @@ -96,9 +96,7 @@ ddxUseMsg(void)<br>
>      ErrorF("-rootless              run rootless, requires wm support\n");<br>
>      ErrorF("-wm fd                 create X client for wm on given fd\n");<br>
>      ErrorF("-listen fd             add give fd as a listen socket\n");<br>
> -#ifdef XWL_HAS_EGLSTREAM<br>
>      ErrorF("-eglstream             use eglstream backend for nvidia GPUs\n");<br>
> -#endif<br>
>  }<br>
><br>
>  int<br>
> @@ -117,11 +115,9 @@ ddxProcessArgument(int argc, char *argv[], int i)<br>
>      else if (strcmp(argv[i], "-shm") == 0) {<br>
>          return 1;<br>
>      }<br>
> -#ifdef XWL_HAS_EGLSTREAM<br>
>      else if (strcmp(argv[i], "-eglstream") == 0) {<br>
>          return 1;<br>
>      }<br>
> -#endif<br>
><br>
>      return 0;<br>
>  }<br>
> @@ -988,11 +984,13 @@ xwl_screen_init(ScreenPtr pScreen, int argc, char **argv)<br>
>          else if (strcmp(argv[i], "-shm") == 0) {<br>
>              xwl_screen->glamor = 0;<br>
>          }<br>
> -#ifdef XWL_HAS_EGLSTREAM<br>
>          else if (strcmp(argv[i], "-eglstream") == 0) {<br>
> +#ifdef XWL_HAS_EGLSTREAM<br>
>              use_eglstreams = TRUE;<br>
> -        }<br>
> +#else<br>
> +            ErrorF("xwayland glamor: eglstream backend support not enabled\n");<br>
</div></div>Something is really weird here:<br>
<br>
On one hand '-eglstream' is recognised and used (by potential user) on<br>
the other "... support is not _enabled_" is printed.<br>
Surely you meant "not built", right? After all explicitly passing the<br>
enable (runtime) flag should be enough to enable it ;-)<br></blockquote><div><br></div><div>Yes, I literally mean "enabled at build time".</div><div><br></div><div>Cheers,</div><div>Olivier</div></div></div></div>