[igt-dev] [PATCH i-g-t 0/4] Refactoring of *_fill libraries

Katarzyna Dec katarzyna.dec at intel.com
Thu Apr 5 13:53:02 UTC 2018


This series is removing duplications in gpgpu_fill and media_fill
libraries. As a first step I moved gpgpu and media helper functions
to gpu_fill library. In second patch I adjusted code to our coding
style. In the third not obvious duplications were removed (like
adding in gen7 functions conditions for future gens). Last patch
adds missing parameters that make GPU hang on gen9 and gen9+.

In first version of this series there was a comment about moving
batch_alloc/copy etc. functions to intel_batchbuffer library.
Because there is a lot of code to review already this change will
be introduced in another series (rendercopy, media_fill, gpgpu_fill
and media_spin code is affected by this).

It is possible that more changes around gen*_media.h and media_spin
is needed, but this will be done as a next step.

v2: Removed not obvious duplications. Adjusted code to review comments.

Katarzyna Dec (4):
  lib: Move common gpgpu/media functions to gpu_fill library
  lib: Adjust refactored gpu_fill library to our coding style
  lib: Removing duplications in gpu_fill library
  lib/gpgpu_fill: Add missing configuration parameters for gpgpu_fill

 lib/Makefile.sources    |   3 +-
 lib/gpgpu_fill.c        | 609 ++------------------------------------------
 lib/gpgpu_fill.h        |  18 +-
 lib/gpu_fill.c          | 659 ++++++++++++++++++++++++++++++++++++++++++++++++
 lib/gpu_fill.h          | 123 +++++++++
 lib/intel_batchbuffer.c |   4 +-
 lib/intel_batchbuffer.h |   3 +-
 lib/media_fill.h        |  33 +--
 lib/media_fill_gen7.c   | 288 +--------------------
 lib/media_fill_gen8.c   | 313 ++---------------------
 lib/media_fill_gen8lp.c | 367 ---------------------------
 lib/media_fill_gen9.c   | 339 ++-----------------------
 lib/meson.build         |   2 +-
 tests/gem_gpgpu_fill.c  |   2 +-
 tests/gem_media_fill.c  |   2 +-
 15 files changed, 886 insertions(+), 1879 deletions(-)
 create mode 100644 lib/gpu_fill.c
 create mode 100644 lib/gpu_fill.h
 delete mode 100644 lib/media_fill_gen8lp.c

-- 
2.14.3



More information about the igt-dev mailing list