[igt-dev] [PATCH i-g-t 0/8] Promote kms_flip_tiling to general

Ville Syrjala ville.syrjala at linux.intel.com
Wed Oct 13 22:17:19 UTC 2021


From: Ville Syrjälä <ville.syrjala at linux.intel.com>

Got fed up with the piles of copy-pasted code in kms_flip_tiling
and so proceeded to fully generalize it. After this it tests flipping
between all ordered pairs of modifiers. I did have to optimize it a bit
to keep the runtime within reasonable bounds since we now test a lot
more than before.

The only reason why I still left it ina tests/i915/ is that it relies
on the new igt_fb_modifier_name() to generate the dynamic subtest
name, and I didn't populate any non-i915 modifiers in there. So if/when
that gets filled out a bit more we can move this back out from the i915
dungeon.

Ville Syrjälä (8):
  lib/fb: Introduce igt_fb_modifier_name()
  tests/kms_plane: Use IGT_MODIFIER_{FMT,ARGS}
  tests/i915/kms_flip_tiling: Drop ancient stride change restrictin
  tests/i915/kms_flip_tiling: Replace i915 interlace check with
    try_commit
  tests/i915/kms_flip_tiling: Generalize away copy-pasta
  tests/i915/kms_flip_tiling: Drop useless i915 include
  tests/i915/kms_flip_tiling: Stick pipe_crc into data_t
  tests/i915/kms_flip_tiling: Keep CRC running all the time

 lib/igt_fb.c                 |  26 +++
 lib/igt_fb.h                 |   4 +
 tests/i915/kms_flip_tiling.c | 301 ++++++++---------------------------
 tests/kms_plane.c            |  16 +-
 4 files changed, 108 insertions(+), 239 deletions(-)

-- 
2.32.0



More information about the igt-dev mailing list