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

Emil Velikov emil.l.velikov at gmail.com
Thu Oct 11 10:48:20 UTC 2018


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


More information about the Piglit mailing list