All of lore.kernel.org
 help / color / mirror / Atom feed
From: "André Almeida" <andrealmeid@igalia.com>
To: igt-dev@lists.freedesktop.org, Jeevan B <jeevan.b@intel.com>,
	Kamil Konieczny <kamil.konieczny@linux.intel.com>
Cc: kernel-dev@igalia.com, "Vitaly Prosyak" <vitaly.prosyak@amd.com>,
	"Alex Hung" <alex.hung@amd.com>, "Melissa Wen" <mwen@igalia.com>,
	"Rodrigo Siqueira" <siqueira@igalia.com>,
	"André Almeida" <andrealmeid@igalia.com>
Subject: [PATCH v6 0/4] tests/kms_async_flips: Create subtest for overlay planes
Date: Mon,  7 Apr 2025 21:02:17 -0300	[thread overview]
Message-ID: <20250408000221.140459-1-andrealmeid@igalia.com> (raw)

This patchset creates a subtest for overlay planes. This is supported by amdgpu
and is merged in the kernel:

https://lore.kernel.org/lkml/173948734065.719858.7405160715916126757.b4-ty@linaro.org/

v6:
 - Added missing `test_init_ops()` for "overlay-atomic" test
 - Reworked `test_init_fbs()`: it destroys all buffers when the modifier
 changes, and recreate then if they are need for the next test. It also destroys
 overlay buffers if they are not needed anymore

v5:
 - Added a new patch "lib/ioctl_wrappers: let the caller handle capability check
   result" by Melissa Wen
 - This patch solved an issue that the test was exiting instead of skipping when
   the driver did not support ATOMIC_ASYNC_PAGE_FLIP
 - test_finish() is now called test_init_ops() and is called in the beginning of
   every test. This was done because some asserts prevented the test to reach
   the final of the run_test() function.
 - Fixed a typo to correctly set the overlay fb to the overlay plane
 - Added code to remove the overlay fb by the end of the test
 - Fixed a bug that test_init_fbs() was skipping the fb creation for overlay
   planes
 - Fixed require_overlay_flip_support() by changing the fb to be flipped with
   the async flip

v4:
 - Created a patch to check for DRM_CAP_ATOMIC_ASYNC_PAGE_FLIP, so drivers
 without support for it, skips instead of failing (Melissa)
 - Encapsulated all test paths options into a function to easy maintenance (Melissa)
 - Brought back test description for GitLab compilation

v3:
 - Fixed a bug that was mixing `alternate_sync_async = true` with the overlay path
 - Refactored how the test parameters are being set/unset (patch 1/2)
 - Fixed a bug where a primary buffer was being used with an overlay plane

v2: Add test description for GitLab compilation

André Almeida (3):
  tests/kms_async_flips: Check for atomic async flip cap
  kms_async_flips: Refactor data options
  tests/kms_async_flips: Create subtest for overlay planes

Melissa Wen (1):
  lib/ioctl_wrappers: let the caller handle capability check result

 lib/ioctl_wrappers.c    |  13 ++--
 lib/ioctl_wrappers.h    |   2 +-
 tests/kms_async_flips.c | 131 ++++++++++++++++++++++++++++++++++------
 3 files changed, 216 insertions(+), 24 deletions(-)
 create mode 100644 tests/xor.c

-- 
2.49.0


             reply	other threads:[~2025-04-08  0:02 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-04-08  0:02 André Almeida [this message]
2025-04-08  0:02 ` [PATCH v6 1/4] lib/ioctl_wrappers: let the caller handle capability check result André Almeida
2025-04-08 12:08   ` Kamil Konieczny
2025-04-08 16:09   ` Melissa Wen
2025-04-08  0:02 ` [PATCH v6 2/4] tests/kms_async_flips: Check for atomic async flip cap André Almeida
2025-04-08 16:01   ` Melissa Wen
2025-04-08 16:07     ` André Almeida
2025-04-08 18:48       ` Melissa Wen
2025-04-08  0:02 ` [PATCH v6 3/4] kms_async_flips: Refactor data options André Almeida
2025-04-08  0:02 ` [PATCH v6 4/4] tests/kms_async_flips: Create subtest for overlay planes André Almeida
2025-04-08 16:16   ` Melissa Wen
2025-04-08 18:53     ` Alex Hung
2025-04-08 19:26       ` André Almeida
2025-04-08 20:45         ` André Almeida
2025-04-08 21:46           ` Alex Hung
2025-04-08  0:05 ` [PATCH v6 0/4] " André Almeida
2025-04-08  1:02 ` ✓ Xe.CI.BAT: success for " Patchwork
2025-04-08  1:12 ` ✓ i915.CI.BAT: " Patchwork
2025-04-08  3:06 ` ✗ i915.CI.Full: failure " Patchwork
2025-04-08  8:01 ` ✗ Xe.CI.Full: " Patchwork

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250408000221.140459-1-andrealmeid@igalia.com \
    --to=andrealmeid@igalia.com \
    --cc=alex.hung@amd.com \
    --cc=igt-dev@lists.freedesktop.org \
    --cc=jeevan.b@intel.com \
    --cc=kamil.konieczny@linux.intel.com \
    --cc=kernel-dev@igalia.com \
    --cc=mwen@igalia.com \
    --cc=siqueira@igalia.com \
    --cc=vitaly.prosyak@amd.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.