[Mesa-dev] [PATCH] egl: Mention if swrast is being forced

[Martin Peres]

On 31/10/2019 09:35, Chris Wilson wrote:
> The system can be disabling HW acceleration unbeknowst to the user,
> leading to a long debug session trying to work out which component is
> failing. A quick mention that it is the environment override would be
> very useful.
> ---
>  src/egl/main/egldriver.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/src/egl/main/egldriver.c b/src/egl/main/egldriver.c
> index 0d8919aa0e1..132b12ab4cb 100644
> --- a/src/egl/main/egldriver.c
> +++ b/src/egl/main/egldriver.c
> @@ -92,6 +92,8 @@ _eglMatchDriver(_EGLDisplay *disp)
>     /* set options */
>     disp->Options.ForceSoftware =
>        env_var_as_boolean("LIBGL_ALWAYS_SOFTWARE", false);
> +   if (disp->Options.ForceSoftware)
> +      _eglLog(_EGL_DEBUG, "Found 'LIBGL_ALWAYS_SOFTWARE' set, forcing swrast");

Maybe we want to remain generic rather than just saying swrast, but not
sure if LIBGL_ALWAYS_SOFTWARE would enable llvmpipe if available rather
than swrast. Maybe we can say "will use a CPU renderer".

In any case, the change is:
Acked-by: Martin Peres <martin.peres at linux.intel.com>

Martin

>  
>     best_drv = _eglMatchAndInitialize(disp);
>     if (!best_drv && !disp->Options.ForceSoftware) {
>