Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH i-g-t v3 0/3] Add initial Panthor tests
@ 2025-11-04 20:28 Daniel Almeida
  2025-11-04 20:28 ` [PATCH i-g-t v3 1/3] lib: add support for opening Panthor devices Daniel Almeida
                   ` (6 more replies)
  0 siblings, 7 replies; 15+ messages in thread
From: Daniel Almeida @ 2025-11-04 20:28 UTC (permalink / raw)
  To: adrinael, arek, kamil.konieczny, juhapekka.heikkila,
	bhanuprakash.modem, ashutosh.dixit, karthik.b.s, boris.brezillon,
	liviu.dudau, steven.price, aliceryhl, jeffv
  Cc: Daniel Almeida, intel-gfx, igt-dev, dri-devel

This series adds basic Panthor tests. In particular, these are being
used to test both Panthor[0] and Tyr[1], i.e.: the new Rust GPU driver
that implements Panthor's uAPI (i.e.: panthor_drm.h). Most of the
initial tests were chosen in order to have something to test Tyr with,
but this series lays the groundwork so that more interesting tests can
be added to test more of Panthor itself.

This work is being tested on a RockPi 5, featuring an rk3588 SoC and
Mali-G610 Valhall.

Note that there's a few (less than five?) remaining checkpatch.pl
comments about long lines. IMHO there's no way to format them better so
I hope we can live with this.

[0]: https://patchwork.freedesktop.org/patch/msgid/20240229162230.2634044-12-boris.brezillon@collabora.com
[1]: https://lore.kernel.org/dri-devel/aMLB0Vs0dJ_AkU4z@google.com/

Changes from v2:

Thanks, Boris {
  - New helpers for command stream manipulation (see "struct cs_instr")
  - Reworked bad indentation and formatting
  - Removed BE support (UMD doesn't care about this either)
}

Thanks, Daniel Stone {
  - Introduced igt_panthor_get_first_core(), which correctly computes
    the first available core using ffs()
  - Stopped hardcoding the page size and switched to getpagesize()
    instead,
  - Switched to more specific versions of igt_assert as appropriate (like
    igt_assert_neq(), etc)
  - Simplified panthor_group.c considerably by relying on the helpers in
    igt_panthor.[ch]
  - Switched to the syncobj_create() and syncobj_destroy() helpers.
}

- Added group creation helpers with sensible defaults (see
  igt_panthor_group_create_simple(), for example)
- Picked up tags

- Link to v2: https://lore.kernel.org/dri-devel/20250912181931.3738444-1-daniel.almeida@collabora.com/

Changes from v1:
- Rebased on top of the latest master
- Squashed patch 3 from v1 into patch 2.
- Switched to /* */ comments in headers
- Initialized padding fields to 0 as applicable in group_destroy and
  vm_destroy
- Removed wrong assert(gpu_rev != 0)
- Changed indentation to use tabs
- Rework igt_panthor_mmap_bo to take an offset (so we don't call the
  mmap_offset ioctl twice)
- Added igt_describe and docs to the functions igt_panthor.c
- Linked to the driver in the cover letter and patch 1.
- Improved the commit message for patch 1.
Link to v1: https://lore.kernel.org/dri-devel/20250828130402.2549948-1-daniel.almeida@collabora.com/



Daniel Almeida (3):
  lib: add support for opening Panthor devices
  panthor: add initial infrastructure
  tests/panthor: add panthor tests

 lib/drmtest.c                 |   1 +
 lib/drmtest.h                 |   1 +
 lib/igt_panthor.c             | 372 ++++++++++++++++++++++++++++++++++
 lib/igt_panthor.h             | 221 ++++++++++++++++++++
 lib/meson.build               |   1 +
 meson.build                   |   8 +
 tests/meson.build             |   2 +
 tests/panthor/meson.build     |  15 ++
 tests/panthor/panthor_gem.c   |  72 +++++++
 tests/panthor/panthor_group.c | 123 +++++++++++
 tests/panthor/panthor_query.c |  29 +++
 tests/panthor/panthor_vm.c    |  84 ++++++++
 12 files changed, 929 insertions(+)
 create mode 100644 lib/igt_panthor.c
 create mode 100644 lib/igt_panthor.h
 create mode 100644 tests/panthor/meson.build
 create mode 100644 tests/panthor/panthor_gem.c
 create mode 100644 tests/panthor/panthor_group.c
 create mode 100644 tests/panthor/panthor_query.c
 create mode 100644 tests/panthor/panthor_vm.c

-- 
2.51.0


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

end of thread, other threads:[~2025-11-06 17:13 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-11-04 20:28 [PATCH i-g-t v3 0/3] Add initial Panthor tests Daniel Almeida
2025-11-04 20:28 ` [PATCH i-g-t v3 1/3] lib: add support for opening Panthor devices Daniel Almeida
2025-11-06  9:24   ` Boris Brezillon
2025-11-04 20:28 ` [PATCH i-g-t v3 2/3] panthor: add initial infrastructure Daniel Almeida
2025-11-06  9:25   ` Boris Brezillon
2025-11-04 20:28 ` [PATCH i-g-t v3 3/3] tests/panthor: add panthor tests Daniel Almeida
2025-11-04 21:04   ` Boris Brezillon
2025-11-06 11:13     ` Daniel Almeida
2025-11-06 14:03       ` Pawel Sikora
2025-11-06  9:26   ` Boris Brezillon
2025-11-06 17:10     ` Kamil Konieczny
2025-11-05 22:08 ` ✗ i915.CI.BAT: failure for Add initial Panthor tests (rev3) Patchwork
2025-11-05 22:51 ` ✓ Xe.CI.BAT: success " Patchwork
2025-11-06  9:29 ` [PATCH i-g-t v3 0/3] Add initial Panthor tests Boris Brezillon
2025-11-06 10:40 ` ✗ Xe.CI.Full: failure for Add initial Panthor tests (rev3) Patchwork

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