[Piglit] [PATCH 0/27] Various cleanups for profiles

Dylan Baker dylan at pnwbakers.com
Mon Oct 24 19:54:46 UTC 2016


This series is a large collections of cleanups for th profile module and
particularly for the TestProfile and TestDict classes. These began their
lives as the start of a series to serialize the TestDict at compile time
and to move startup overhead.

This series is just a bunch of cleanups and removes a *lot* of code, and
simplifies a number of things, as well as fixing a few long standing
bugs in piglit. The biggest of these fixes is that it is now possible to
pass two derivatives of all.py (say glslparser.py and shader.py) at the
same time.

There is one change in this series that is RFC, though the rest of it is ready
to run. That one patch adds a new requirement for python < 3.4, and it's
possible some might prefer not to have that requirement.

Dylan Baker (27):
  framework/profile: drop pre and post run hooks.
  framework/profile: Drop unneeded except
  piglit: Only catch parsed.func AttributeError
  framework: Split the run method out of profile.
  framework: remove concurrent from OPTIONS, pass directly to profile
  [RFC] framework: Use an enum for concurrency.
  framework/programs/run: Remove use of TestrunResult
  framework/profile: add a copy method to profile
  tests: Copy profiles to allow them to be run in parallel
  framework/programs/run: Only allow --test-list if one profile
  framework/profile: Don't merge profiles
  framework/profiles/run: remove redundant list() around comprehension
  framework/profile: Drop TestProfile.filter_tests
  framework/programs/run: import framework.profile as profile
  framework: Remove exclude_tests from options.OPTIONS
  framework: Pull {include,exclude}_filter out of Options
  framework/profile: Factor out check_all closure
  framework/profile: Don't alter the TestProfile before running
  framework/profile: Update __all__
  framework/profile: Update docstrings
  framework/profile: Split try/except block
  framework/profile: replace Testprofile.{dmesg,monitoring} with dict
  framework/profile: remove unused argument from TestProfile.group_manager
  framework/profile: Move group_manager from TestProfile to TestDict
  framework/test/base: Remove timeout parameter
  framework/test/glsl_parser_test.py: split the parser out of the class
  tests/all.py: Make add_fbo_depthstencil_tests take an adder

 framework/options.py                         | 144 +-----
 framework/profile.py                         | 528 ++++++-----------
 framework/programs/run.py                    | 134 ++--
 framework/test/base.py                       |  36 +-
 framework/test/glsl_parser_test.py           |  85 +--
 piglit                                       |   4 +-
 tests/all.py                                 | 605 +++++++++-----------
 tests/cl.py                                  |   8 +-
 tests/cpu.py                                 |   6 +-
 tests/glslparser.py                          |   6 +-
 tests/gpu.py                                 |   8 +-
 tests/igt.py                                 |   7 +-
 tests/llvmpipe.py                            |   4 +-
 tests/quick.py                               |  14 +-
 tests/quick_cl.py                            |   4 +-
 tests/sanity.py                              |   2 +-
 tests/shader.py                              |   6 +-
 tests/xts-render.py                          |   9 +-
 tox.ini                                      |   1 +-
 unittests/framework/test/test_base.py        |  14 +-
 unittests/framework/test/test_shader_test.py |   2 +-
 unittests/framework/test_options.py          | 178 +------
 unittests/framework/test_profile.py          | 323 +++--------
 23 files changed, 810 insertions(+), 1318 deletions(-)

-- 
git-series 0.8.10


More information about the Piglit mailing list