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

Katarzyna Dec katarzyna.dec at intel.com
Tue Apr 10 08:12:28 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.
v3: Series needed reorganization because it introduced bug to ALP,
which was hard to find. That is why patch 1 is now almost only moving functions to gpu_fill with removing duplications, such as the same functions. Also applied comments from review.
v4: Added #defines and copyrights to new gpu_fill library. Changed functions
order in gpu_fill library.

Signed-off-by: Katarzyna Dec <katarzyna.dec at intel.com>
Cc: Lukasz Kalamarz <lukasz.kalamarz at intel.com>
Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>


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

 lib/Makefile.sources    |   3 +-
 lib/gpgpu_fill.c        | 600 ++--------------------------------------------
 lib/gpgpu_fill.h        |  12 +-
 lib/gpu_fill.c          | 624 ++++++++++++++++++++++++++++++++++++++++++++++++
 lib/gpu_fill.h          | 135 +++++++++++
 lib/intel_batchbuffer.c |   4 +-
 lib/media_fill.h        |  23 +-
 lib/media_fill_gen7.c   | 278 +--------------------
 lib/media_fill_gen8.c   | 305 +----------------------
 lib/media_fill_gen8lp.c | 367 ----------------------------
 lib/media_fill_gen9.c   | 308 +-----------------------
 lib/meson.build         |   2 +-
 12 files changed, 815 insertions(+), 1846 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