Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 0/5] tests/kms_writeback: Add clone mode subtest
@ 2024-08-29  3:05 Jessica Zhang
  2024-08-29  3:05 ` [PATCH v2 1/5] tests/kms_writeback: clear writeback properties in teardown path Jessica Zhang
                   ` (8 more replies)
  0 siblings, 9 replies; 13+ messages in thread
From: Jessica Zhang @ 2024-08-29  3:05 UTC (permalink / raw)
  To: Petri Latvala, Arkadiusz Hiler, Kamil Konieczny,
	Juha-Pekka Heikkila, Bhanuprakash Modem, Ashutosh Dixit
  Cc: Alex Hung, quic_abhinavk, quic_ebharadw, igt-dev, Jessica Zhang

Add a subtest that will iterate through all pairs of
non-writeback and writeback cloned outputs and dump the writeback
framebuffer for each pair.

This series includes:

- Fixes for properly cleaning up the kms_writeback state and forcing a
  modeset after igt_display_require() to prevent commit failures when
  the writeback mode is the same as the primary display mode
- Adjust the duplicate pipe check in igt_display_refresh() to allow for
  combinations of valid clones
- Moving the writeback dump to its own subtest
- Adding a subtest to dump all writeback/non-writeback cloned pairs

To dump the clone mode buffer, run the following command:

IGT_FRAME_DUMP_PATH=<dump path> FRAME_PNG_FILE_NAME=<file name> \
	./build/tests/kms_writeback [--run-subtest dump-valid-clones] \
	-dc <primary display mode>

This will output a file named <file name>-encoder<writeback encoder
id>-encoder<primary display encoder id>

---
Changes in v2:
- Corrected author email
- Link to v1: https://lore.kernel.org/r/20240828-igt-cwb-v1-0-48aee421fc97@quicinc.com

---
Jessica Zhang (5):
      tests/kms_writeback: clear writeback properties in teardown path
      lib/igt_kms: Add helper to get encoder index
      lib/igt_kms: loosen duplicate check in igt_display_refresh
      tests/kms_writeback: Disable all outputs after igt_display_require()
      tests/kms_writeback: Add dump for valid clone mode

 lib/igt_kms.c         | 57 +++++++++++++++++++++++++++++------
 lib/igt_kms.h         |  2 ++
 tests/kms_setmode.c   | 14 ++-------
 tests/kms_writeback.c | 83 +++++++++++++++++++++++++++++++++++++++++++--------
 4 files changed, 122 insertions(+), 34 deletions(-)
---
base-commit: 3b6b2d238e864ff1af9e33159d3bbf4b7f01d86d
change-id: 20240827-igt-cwb-23fd81d3a7a4

Best regards,
-- 
Jessica Zhang <quic_jesszhan@quicinc.com>


^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2024-09-06 22:06 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-08-29  3:05 [PATCH v2 0/5] tests/kms_writeback: Add clone mode subtest Jessica Zhang
2024-08-29  3:05 ` [PATCH v2 1/5] tests/kms_writeback: clear writeback properties in teardown path Jessica Zhang
2024-08-29  3:05 ` [PATCH v2 2/5] lib/igt_kms: Add helper to get encoder index Jessica Zhang
2024-08-29  3:05 ` [PATCH v2 3/5] lib/igt_kms: loosen duplicate check in igt_display_refresh Jessica Zhang
2024-08-29  3:05 ` [PATCH v2 4/5] tests/kms_writeback: Disable all outputs after igt_display_require() Jessica Zhang
2024-08-29  3:05 ` [PATCH v2 5/5] tests/kms_writeback: Add dump for valid clone mode Jessica Zhang
2024-08-29  3:56 ` ✓ CI.xeBAT: success for tests/kms_writeback: Add clone mode subtest (rev2) Patchwork
2024-08-29  3:57 ` ✗ Fi.CI.BAT: failure " Patchwork
2024-08-29 16:30 ` ✗ CI.xeFULL: " Patchwork
2024-08-29 20:28 ` [PATCH v2 0/5] tests/kms_writeback: Add clone mode subtest Alex Hung
2024-08-30 18:32   ` Jessica Zhang
2024-09-03 23:33     ` Alex Hung
2024-09-06 22:06       ` Jessica Zhang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox