All of lore.kernel.org
 help / color / mirror / Atom feed
From: Karolina Stolarek <karolina.stolarek@intel.com>
To: dri-devel@lists.freedesktop.org
Cc: "Christian König" <christian.koenig@amd.com>,
	"Matthew Auld" <matthew.auld@intel.com>,
	"Amaranath Somalapuram" <asomalap@amd.com>,
	"Karolina Stolarek" <karolina.stolarek@intel.com>
Subject: [PATCH v11 10/10] drm/ttm/tests: Add TODO file
Date: Wed, 17 Apr 2024 15:03:23 +0200	[thread overview]
Message-ID: <af96920897379fa0cb10619bb35c22917b884ce4.1713357042.git.karolina.stolarek@intel.com> (raw)
In-Reply-To: <cover.1713357042.git.karolina.stolarek@intel.com>

List improvements for the test suite with some notes.

Signed-off-by: Karolina Stolarek <karolina.stolarek@intel.com>
---
 drivers/gpu/drm/ttm/tests/TODO | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)
 create mode 100644 drivers/gpu/drm/ttm/tests/TODO

diff --git a/drivers/gpu/drm/ttm/tests/TODO b/drivers/gpu/drm/ttm/tests/TODO
new file mode 100644
index 000000000000..b29d55167d56
--- /dev/null
+++ b/drivers/gpu/drm/ttm/tests/TODO
@@ -0,0 +1,25 @@
+TODO
+=====
+
+- Add a test case where the only evictable BO is busy
+- Update eviction tests so they use parametrized "from" memory type
+- Improve mock manager's implementation, e.g. allocate a block of
+  dummy memory that can be used when testing page mapping functions
+- Suggestion: Add test cases with external BOs
+- Suggestion: randomize the number and size of tested buffers in
+  ttm_bo_validate()
+- Agree on the naming convention
+
+Notes and gotchas
+=================
+
+- These tests are built and run with a UML kernel, because
+  1) We are interested in hardware-independent testing
+  2) We don't want to have actual DRM devices interacting with TTM
+     at the same time as the test one. Getting these to work in
+     parallel would require some time (...and that's a "todo" in itself!)
+- Triggering ttm_bo_vm_ops callbacks from KUnit (i.e. kernel) might be
+  a challenge, but is worth trying. Look at selftests like
+  i915/gem/selftests/i915_gem_mman.c for inspiration
+- The test suite uses UML where ioremap() call returns NULL, meaning that
+  ttm_bo_ioremap() can't be tested, unless we find a way to stub it
-- 
2.34.1


      parent reply	other threads:[~2024-04-17 13:04 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-04-17 13:03 [PATCH v11 00/10] Improve test coverage of TTM Karolina Stolarek
2024-04-17 13:03 ` [PATCH v11 01/10] drm/ttm/tests: Fix a warning in ttm_bo_unreserve_bulk Karolina Stolarek
2024-04-18  8:15   ` Christian König
2024-04-18  8:21     ` Karolina Stolarek
2024-04-17 13:03 ` [PATCH v11 02/10] drm/ttm/tests: Delete unnecessary config option Karolina Stolarek
2024-04-17 14:07   ` Nirmoy Das
2024-04-17 13:03 ` [PATCH v11 03/10] drm/ttm/tests: Set DMA mask in KUnit device Karolina Stolarek
2024-04-17 13:54   ` Nirmoy Das
2024-04-17 13:03 ` [PATCH v11 04/10] drm/ttm/tests: Use an init function from the helpers lib Karolina Stolarek
2024-04-17 13:03 ` [PATCH v11 05/10] drm/ttm/tests: Test simple BO creation and validation Karolina Stolarek
2024-04-17 13:03 ` [PATCH v11 06/10] drm/ttm/tests: Add tests with mock resource managers Karolina Stolarek
2024-04-17 13:03 ` [PATCH v11 07/10] drm/ttm/tests: Add test cases dependent on fence signaling Karolina Stolarek
2024-04-17 13:03 ` [PATCH v11 08/10] drm/ttm/tests: Add eviction testing Karolina Stolarek
2024-04-17 13:03 ` [PATCH v11 09/10] drm/ttm/tests: Add tests for ttm_tt_populate Karolina Stolarek
2024-04-17 13:03 ` Karolina Stolarek [this message]

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=af96920897379fa0cb10619bb35c22917b884ce4.1713357042.git.karolina.stolarek@intel.com \
    --to=karolina.stolarek@intel.com \
    --cc=asomalap@amd.com \
    --cc=christian.koenig@amd.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=matthew.auld@intel.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.