piglit-run.py: record whether the test run was concurrent

cc: ben@bwidawsk.net
Reviewed-by: Damien Lespiau <damien.lespiau@intel.com>
Reviewed by: Kenney Phillis <kphillisjr@gmail.com>
Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
piglit-run.py: resume honors saved concurrency flag

This breaks incomplete runs started before applying the previous patch
in this series, since there will be no key for concurrency

v2: - Add this patch (Damien Lespiau)

Reviewed-by: Damien Lespiau <damien.lespiau@intel.com>
Reviewed by: Kenney Phillis <kphillisjr@gmail.com>`
Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
piglit-run.py: Cleanups

This patch tidies up a few things, and lays groundwork for later patches
in this series.

Reviewed by: Kenney Phillis <Kkphillisjr@gmail.com>
Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
piglit-run.py: Record all attributes of Environment and restore them

This causes all attributes in the Environment instance env to be dumped
into the json and reloaded on a resume. It adds an __iter__ magic method
to core.Environment to allow this.

Reviewed by: Kenney Phillis <Kkphillisjr@gmail.com>
Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
Split resume functionality out of piglit-run.py

The resume functionality actually shares only a little boilerplate with
the run functionality, and there is a lot of extra boilerplate that
exists (or should exist) to make resumes work correctly with all of the
command line options allowed by piglit-run. Therefore, it makes more
sense to split it out into it's own file.

Reviewed by: Kenney Phillis <Kkphillisjr@gmail.com>
Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
draw-range-elements-base-vertex: Require GLSL 1.30.

Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Brian Paul <brianp@vmware.com>
gl-3.1/minmax: require at least 0 in GL_NUM_COMPRESSED_TEXTURE_FORMATS
summary: fix regression and fixes summaries

Somebody broke this.
core: don't report warn if stderr is not empty

The usefulness of this is questionable to say the least.

Acked-by: Eric Anholt <eric@anholt.net>
Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
piglit: don't report fail for WAFFLE_ERROR_BUILT_WITHOUT_SUPPORT

If we build waffle w/out ES support and we run all.tests, quick.tests,
etc. we get lots of 'fail' results for the ES tests.  This change causes
those tests to be reported as 'skip' instead.

Reviewed-by: José Fonseca <jfonseca@vmware.com>
genmipmap-errors: fix/add extension checks

GL_DEPTH24_STENCIL8 is defined by GL_EXT_packed_depth_stencil.
GL_DEPTH32F_STENCIL8 is from GL_ARB_depth_buffer_float, not
GL_ARB_texture_float.

Reviewed-by: Eric Anholt <eric@anholt.net>
arb_viewport_array: add test to check for a MacOS AMD bug

glScissorIndexed() doesn't seem to behave properly.

Reviewed-by: Jose Fonseca <jfonseca@vmware.com>
s3tc-errors: fix bad call to glPixelStorei()

Passing a negative value for GL_UNPACK_SKIP_PIXELS raises a GL error
that we don't want.  Just pass 0 for both SKIP_PIXELS and SKIP_ROWS.

Reviewed-by: Jose Fonseca <jfonseca@vmware.com>
Reviewed-by: Eric Anholt <eric@anholt.net>
rendermode-feedback: add static, const qualifiers

Reviewed-by: Jose Fonseca <jfonseca@vmware.com>
Reviewed-by: Eric Anholt <eric@anholt.net>
glean/blendFunc: use larger test stride for --quick option

Reviewed-by: Jose Fonseca <jfonseca@vmware.com>
glean/pixelformats: use test stride of 13 for --quick option

Reviewed-by: Jose Fonseca <jfonseca@vmware.com>
glean/pointatten: use test stride of 5 for --quick option

Reviewed-by: Jose Fonseca <jfonseca@vmware.com>
shader_runner: add missing 'void' in function declarations

Reviewed-by: Jose Fonseca <jfonseca@vmware.com>
shader_runner: check for no GLSL program in setup_ubos()

If we're running an ARB_vertex/fragment_program test, there will be
no GLSL shader program and 'prog' will be zero.  Return early to avoid
generating an error from the GL_ACTIVE_UNIFORM_BLOCKS query.

Reviewed-by: Jose Fonseca <jfonseca@vmware.com>
Reviewed-by: Eric Anholt <eric@anholt.net>
bugzilla 71822: Fix regression introduced to piglit-run.py

The commit 0618aa38d4a8a7c82994fb28a41576da9a2cc414 introduces a
regression when using the -t/--include-filter or -x/--exclude-filter
options. This is caused by the __iter__ magic method in the
core.Environment class returning self.__dict__.iteritems(), which
returns compiled regex objects. This will cause the json encoder to
choke.

This patch corrects the __iter__ method in core.Environment to yield
regex items as regex.pattern, and attribute storing the original string
used to create the pattern.

Tested-by: Brian Paul <brianp@vmware.com>
Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>