[Piglit] [PATCH v2] framework/wflinfo: use x11_egl for gles with mixed_glx_egl

[Dylan Baker]

Quoting Emil Velikov (2018-10-11 03:48:20)
> On Fri, 5 Oct 2018 at 17:06, Dylan Baker <dylan at pnwbakers.com> wrote:
> >
> > Because wflinfo returns GL legacy (non-profile) for gles on glx.
> >
> > v2: - If the platform is pure GLX then warn that things aren't going to
> >       work.
> > ---
> >  framework/wflinfo.py | 21 +++++++++++++++++++--
> >  1 file changed, 19 insertions(+), 2 deletions(-)
> >
> > diff --git a/framework/wflinfo.py b/framework/wflinfo.py
> > index a568bc344..1655aa0ee 100644
> > --- a/framework/wflinfo.py
> > +++ b/framework/wflinfo.py
> > @@ -1,4 +1,4 @@
> > -# Copyright (c) 2015-2016 Intel Corporation
> > +# Copyright (c) 2015-2016,2018 Intel Corporation
> >
> >  # Permission is hereby granted, free of charge, to any person obtaining a copy
> >  # of this software and associated documentation files (the "Software"), to deal
> > @@ -26,6 +26,7 @@ import os
> >  import subprocess
> >  import sys
> >  import threading
> > +import warnings
> >
> >  import six
> >
> > @@ -103,7 +104,23 @@ class WflInfo(object):
> >                  # to something that wflinfo understands.
> >                  platform = OPTIONS.env['PIGLIT_PLATFORM']
> >                  if platform == "mixed_glx_egl":
> > -                    platform = "glx"
> > +                    if 'gles2' in opts or 'gles3'in opts:
> > +                        platform = "x11_egl"
> > +                    else:
> > +                        platform = "glx"
> > +                if platform == 'glx' and 'gles2' in opts or 'gles3' in opts:
> > +                    # There is a bug in waffle Which causes it to return
> > +                    # a legacy (non-profile) OpenGL instead of OpenGL ES when
> > +                    # using the GLX platform (not for mixed_glx_egl). If the
> > +                    # user explicitly asks for GLX then the tests are going to
> > +                    # skip whether we fast skip them or if we let them run.
> > +                    # Since they're going to skip (even if it's wrong), let's
> > +                    # skip them quickly.
> > +                    #
> > +                    # https://github.com/waffle-gl/waffle/issues/50
> 
> Things seems to be working fine on my system. I've added some nits to
> the issue report, can you please confirm we're thinking about the same
> thing.
> 
> Thanks
> Emil

I've commented on the waffle bug. I'm willing to hold off on this patch until we
can confirm the waffle issue as this doesn't affect us (we use gbm in CI).

Dylan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: signature
URL: <https://lists.freedesktop.org/archives/piglit/attachments/20181011/6929e37d/attachment.sig>