Signed-off-by: Sagar Ghuge <sagar.ghuge@intel.com>

Most of our dependencies no longer support python 3.3, it isn't widely
available in distros (for reference debian old stable has 3.4), it's not
longer supported by upstream python, and even for very old or LTS
distros we still have python 2.7 support.

Also, the requirements section in README.md has been updated with an
earlier patch.

Suggested-by: Rhys Kidd <rhyskidd@gmail.com>
Reviewed-by: Emil Velikov <emil.velikov@collabora.com>

Test includes:
   - texture uploads
   - mipmap generation
   - framebuffer creation
   - rendering to
   - reading from

This test includes only GL_BYTE based formats. R16_SNORM, RG16_SNORM
and RGBA16_SNORM are tested in GL_EXT_texture_norm16 tests.

v2: cleanups and fixes (Nanley Chery)

Signed-off-by: Tapani Pälli <tapani.palli@intel.com>
Reviewed-by: Nanley Chery <nanley.g.chery@intel.com>

Signed-off-by: Tapani Pälli <tapani.palli@intel.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107550


Reviewed-by: Alejandro Piñeiro <apinheiro@igalia.com>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>

Matt Arsenault authored Mar 29, 2018 and Jan Vesely committed Aug 14, 2018

Make sure if a CSR VGPR is needed for SGPR spilling, it is
properly saved and restored.

Acked-by: Jan Vesely <jan.vesely@rutgers.edu>

Matt Arsenault authored Aug 13, 2018 and Jan Vesely committed Aug 13, 2018

These are just bigger versions of the existing struct
calls tests so that they stress using byval/sret. The
existing call with struct tests are now passed directly
in registers.

v2: Rename struct member

Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>

Matt Arsenault authored Oct 27, 2017 and Jan Vesely committed Aug 13, 2018

Doesn't really matter.

Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>

Signed-off-by: Tapani Pälli <tapani.palli@intel.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98699


Reviewed-by: Dylan Baker <dylan@pnwbakers.com>
Reviewed-by: Timothy Arceri <tarceri@itsqueeze.com>

Reviewed-by: Rhys Kidd <rhyskidd@gmail.com>

Fixes: 7356bdd0 "dispatch: fallback to eglGetProcAddress for
                             core functions"
Signed-off-by: Eric Engestrom <eric.engestrom@intel.com>
Reviewed-by: Emil Velikov <emil.velikov@collabora.com>

As python 3.3 is deprecated and no longer supported by piglit, remove redundant workaround
for lack of readlines() support with mock in python 3.3.

Signed-off-by: Rhys Kidd <rhyskidd@gmail.com>
Reviewed-by: Dylan Baker <dylan@pnwbakers.com>

Signed-off-by: Rhys Kidd <rhyskidd@gmail.com>
Reviewed-by: Dylan Baker <dylan@pnwbakers.com>

Dylan Baker authored Aug 06, 2018 and Rhys Kidd committed Aug 07, 2018

Reviewed-by: Rhys Kidd <rhyskidd@gmail.com>
Reviewed-by: Dylan Baker <dylan@pnwbakers.com>

Emil Velikov authored Jun 27, 2016 and Emil Velikov committed Aug 07, 2018

The definition and/or interpretation what is core function may vary
between vendors. Thus let's opt for the more robust option and use
eglGetProcAddress in case dlsym() fails.

Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Adam Jackson <ajax@redhat.com>

Emil Velikov authored Jun 27, 2016 and Emil Velikov committed Aug 07, 2018

In order to do that move the function further up, and while where doing
that use tabs to indent, as the rest of the file.

Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Adam Jackson <ajax@redhat.com>

Emil Velikov authored Jun 27, 2016 and Emil Velikov committed Aug 07, 2018

Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Adam Jackson <ajax@redhat.com>

Emil Velikov authored Jun 27, 2016 and Emil Velikov committed Aug 07, 2018

The symbol is already the default, thus there's no need to specify it.

Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Adam Jackson <ajax@redhat.com>

Emil Velikov authored Jun 27, 2016 and Emil Velikov committed Aug 07, 2018

Analogous to previous commit.

Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Adam Jackson <ajax@redhat.com>

Emil Velikov authored Jun 27, 2016 and Emil Velikov committed Aug 07, 2018

piglit_use_vertex_program/piglit_use_fragment_program are used _after_
piglit_init() at which point the winsys layer is already up and the
dispatch is already setup.

Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Adam Jackson <ajax@redhat.com>

Emil Velikov authored Dec 11, 2017 and Emil Velikov committed Aug 07, 2018

Current Mesa implementation was overly cautious, flagging an error
where it shouldn't - a fix for Mesa can be found at patchwork
https://patchwork.freedesktop.org/patch/187983/



See the test for details.

v2:
 - added test to lists
 - uncomment required setenv

Cc: Ian Romanick <ian.d.romanick@intel.com>
Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Tapani Pälli <tapani.palli@intel.com>

Emil Velikov authored Dec 11, 2017 and Emil Velikov committed Aug 07, 2018

Allows us to consolidate native platform specifics in once place.
Will be reused in later commit(s).

Cc: Ian Romanick <ian.d.romanick@intel.com>
Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Tapani Pälli <tapani.palli@intel.com>

Emil Velikov authored Dec 11, 2017 and Emil Velikov committed Aug 07, 2018

The latter is a legacy and somewhat of a hack.

Since we explicitly know that this requires the X11, simply use the
helper which will use the correct EGL_PLATFORM API.

Cc: Ian Romanick <ian.d.romanick@intel.com>
Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Tapani Pälli <tapani.palli@intel.com>

Emil Velikov authored Dec 11, 2017 and Emil Velikov committed Aug 07, 2018

Some places (like the next patch) want to pass non-default dpy.

Cc: Ian Romanick <ian.d.romanick@intel.com>
Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Tapani Pälli <tapani.palli@intel.com>

Emil Velikov authored Dec 11, 2017 and Emil Velikov committed Aug 07, 2018

Calling eglBindAPI before eglInitialize is allowed by the spec since
the former is per thread, while the latter per display.

At the same time it's fairly inconsistent with rest of piglit and
other EGL users.
Both approaches should work and having a consistently looking code is
always a plus.

v2: Reword the commit message (Tapani)

Cc: Ian Romanick <ian.d.romanick@intel.com>
Cc: Michel Dänzer <michel.daenzer@amd.com>
Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Tapani Pälli <tapani.palli@intel.com>

Reviewed-by: Brian Paul <brianp@vmware.com>

This was missed from commit a440e439.

Signed-off-by: Tapani Pälli <tapani.palli@intel.com>
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>

There is a hardware bug in i965/gen9+ with provoking vertices.
Test if workaround works correctly.

Ref: https://bugs.freedesktop.org/show_bug.cgi?id=103047



v2: - Vertices now have distinctive color
    - Update  message to reflect a nature of the bug which was
      clarified by Kenneth. He will also provide workaround later.

Signed-off-by: Danylo Piliaiev <danylo.piliaiev@globallogic.com>

Add tests for glRenderbufferStorage and glCopyTexImage2D.

Signed-off-by: Tapani Pälli <tapani.palli@intel.com>
Reviewed-by: Alejandro Piñeiro <apinheiro@igalia.com>

Reviewed-by: Eric Anholt <eric@anholt.net>

Reviewed-by: Eric Anholt <eric@anholt.net>

So that random keys show up somewhat nicely.

And into a generic sub element in the output. This makes information
like glxinfo and clinfo not a hard requirement and will allow any sort
of system information to be encoded.

Test polygon culling with glPolygonMode for better coverage.

Reviewed-by: Charmaine Lee <charmainel@vmware.com>

The GLSL compiler can easily unroll this loop and simplify the function
down to a single assignment.  That defeats the purpose of the test.
Replace literals with uniforms to prevent unrolling.

Reviewed-by: Charmaine Lee <charmainel@vmware.com>

This avoids modifications in the original filters object.

Detected through Travis CI.

Fixed also a couple of typos.

Fixes: 1f0f2ad2 ("framework: Add filters class")
Cc: Dylan Baker <dylanx.c.baker@intel.com>
Cc: Michel Dänzer <michel.daenzer@amd.com>
Signed-off-by: Andres Gomez <agomez@igalia.com>
Reviewed-by: Dylan Baker <dylan@pnwbakers.com>
Reviewed-by: Rhys Kidd <rhyskidd@gmail.com>

pytest-warnings is not needed since it was integrated into pytest's
core since 3.1

pytest-timeout >= 1.3.0 requires pytest >= 3.6.0 and we want to keep
using 3.2.5

Cc: Juan A. Suarez Romero <jasuarez@igalia.com>
Cc: Dylan Baker <dylan.c.baker@intel.com>
Signed-off-by: Andres Gomez <agomez@igalia.com>
Reviewed-by: Dylan Baker <dylan@pnwbakers.com>

Python 3.3 reached end-of-life in September 2017 [0].

Continuous integration testing has identified a number of dependencies are
now failing on python 3.3, such as:

  Collecting wheel
  wheel requires Python '>=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*' but the running Python is 3.3.6
  ...
  OSError: Command /home/travis/build/E...generator/bin/python - setuptools pip wheel failed with error code 1

[0] https://www.python.org/dev/peps/pep-0398/#x-end-of-life



Signed-off-by: Rhys Kidd <rhyskidd@gmail.com>
Reviewed-by: Eric Engestrom <eric.engestrom@intel.com>

This is required to make the shader_runner multiple tests in one process
mode work, as it cannot mix gles and gl shaders together.

v2: - drop -gl32 suffix as well as -gles* suffix

Reviewed-by: Francisco Jerez <currojerez@riseup.net>

This just adds a helpful message about what directory the conflicting
shaders are in.