List improvements for the test suite with some notes. Signed-off-by: Karolina Stolarek <karolina.stolarek@xxxxxxxxx> --- drivers/gpu/drm/ttm/tests/TODO | 24 ++++++++++++++++++++++++ 1 file changed, 24 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..3095c3225cec --- /dev/null +++ b/drivers/gpu/drm/ttm/tests/TODO @@ -0,0 +1,24 @@ +TODO +===== + +- Add a test case where the only evictable BO is busy +- Add tests for ttm_bo_vm_ops (that includes ttm_bo_kmap() testing) +- 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 +================= + +- We want to stick to running UML to provide hardware-independent testing +- Triggering ttm_bo_vm_ops callbacks from KUnit (i.e. kernel) might be + a challenge, but it seems doable. Use selftests like + i915/gem/selftests/i915_gem_mman.c for inspiration +- The test suite uses UML, so ioremap() returns NULL. Simply switching + to x86 won't solve the issue, because we also get NULL when trying to + remap RAM. This means that we can only test a part of ttm_bo_ioremap() + (helper called in ttm_bo_kmap() function) -- 2.34.1
