On Fri, 1 Nov 2019 at 14:32, Chris Wilson <ch...@chris-wilson.co.uk> wrote:
>
> Quoting Eric Engestrom (2019-10-31 14:06:40)
> > On Thursday, 2019-10-31 07:35:04 +0000, 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");
> >
> > Good idea!
> > Reviewed-by: Eric Engestrom <eric.engest...@intel.com>
> >
> > I might even suggest going one step further and make that an _EGL_WARNING,
> > so that users are always informed of this by default, without having to
> > set EGL_LOG_LEVEL.
> >
> > I think most users don't want to disable their hardware, so the annoyance
> > if this warning showing up for users who want it should be completely
> > offset by the usefulness of this information for those who don't.
>
> Is there a consensus to go with an always visible _EGL_WARNING instead?
There's been users in the past that explicitly set LIBGL_ALWAYS_SOFTWARE.
In general if you're debugging and EGL_LOG_LEVEL is not set to debug
you're already lost.
That said, personally I'm fine with either DEBUG or WARNING.
-Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
