[PATCH v4 00/14] drm: Add a driver for CSF-based Mali GPUs
Boris Brezillon
boris.brezillon at collabora.com
Mon Jan 22 16:30:31 UTC 2024
Hello,
This is the 4th version of the kernel driver for Mali CSF-based GPUs.
A branch based on drm-misc-next and containing all the dependencies
that are not yet available in drm-misc-next here[1], and another [2]
containing extra patches to have things working on rk3588. The CSF
firmware binary can be found here[3], and should be placed under
/lib/firmware/arm/mali/arch10.8/mali_csffw.bin.
The mesa MR adding v10 support on top of panthor is available here [4].
Steve, I intentionally dropped your R-b on "drm/panthor: Add the heap
logical block" and "drm/panthor: Add the scheduler logical block"
because the tiler-OOM handling changed enough to require a new review
IMHO.
Regarding the GPL2+MIT relicensing, I collected Clément's R-b for the
devfreq code, but am still lacking Alexey Sheplyakov for some bits in
panthor_gpu.c. The rest of the code is either new, or covered by the
Linaro, Arm and Collabora acks.
And here is a non-exhaustive changelog, check each commit for a detailed
changelog.
v4:
- Fix various bugs in the VM logic
- Address comments from Steven, Liviu, Ketil and Chris
- Move tiler OOM handling out of the scheduler interrupt handling path
so we can properly recover when the system runs out of memory, and
panthor is blocked trying to allocate heap chunks
- Rework the heap locking to support concurrent chunk allocation. Not
sure if this is supposed to happen, but we need to be robust against
userspace passing the same heap context to two scheduling groups.
Wasn't needed before the tiler_oom rework, because heap allocation
base serialized by the scheduler lock.
- Make kernel BO destruction robust to NULL/ERR pointers
v3;
- Quite a few changes at the MMU/sched level to make the fix some
race conditions and deadlocks
- Addition of the a sync-only VM_BIND operation (to support
vkQueueSparseBind with zero commands).
- Addition of a VM_GET_STATE ioctl
- Various cosmetic changes (see the commit changelogs for more details)
- Various fixes (see the commit changelogs for more details)
v2:
- Rename the driver (pancsf -> panthor)
- Split the commit adding the driver to ease review
- Use drm_sched for dependency tracking/job submission
- Add a VM_BIND ioctl
- Add the concept of exclusive VM for BOs that are only ever mapped to a
single VM
- Document the code and uAPI
- Add a DT binding doc
Regards,
Boris
[1]https://gitlab.freedesktop.org/panfrost/linux/-/tree/panthor-v4
[2]https://gitlab.freedesktop.org/panfrost/linux/-/tree/panthor-v4+rk3588
[3]https://gitlab.com/firefly-linux/external/libmali/-/raw/firefly/firmware/g610/mali_csffw.bin
[4]https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/26358
Boris Brezillon (13):
drm/panthor: Add uAPI
drm/panthor: Add GPU register definitions
drm/panthor: Add the device logical block
drm/panthor: Add the GPU logical block
drm/panthor: Add GEM logical block
drm/panthor: Add the devfreq logical block
drm/panthor: Add the MMU/VM logical block
drm/panthor: Add the FW logical block
drm/panthor: Add the heap logical block
drm/panthor: Add the scheduler logical block
drm/panthor: Add the driver frontend block
drm/panthor: Allow driver compilation
drm/panthor: Add an entry to MAINTAINERS
Liviu Dudau (1):
dt-bindings: gpu: mali-valhall-csf: Add support for Arm Mali CSF GPUs
.../bindings/gpu/arm,mali-valhall-csf.yaml | 147 +
Documentation/gpu/driver-uapi.rst | 5 +
MAINTAINERS | 11 +
drivers/gpu/drm/Kconfig | 2 +
drivers/gpu/drm/Makefile | 1 +
drivers/gpu/drm/panthor/Kconfig | 23 +
drivers/gpu/drm/panthor/Makefile | 15 +
drivers/gpu/drm/panthor/panthor_devfreq.c | 283 ++
drivers/gpu/drm/panthor/panthor_devfreq.h | 25 +
drivers/gpu/drm/panthor/panthor_device.c | 544 +++
drivers/gpu/drm/panthor/panthor_device.h | 393 ++
drivers/gpu/drm/panthor/panthor_drv.c | 1470 +++++++
drivers/gpu/drm/panthor/panthor_fw.c | 1334 +++++++
drivers/gpu/drm/panthor/panthor_fw.h | 504 +++
drivers/gpu/drm/panthor/panthor_gem.c | 228 ++
drivers/gpu/drm/panthor/panthor_gem.h | 144 +
drivers/gpu/drm/panthor/panthor_gpu.c | 482 +++
drivers/gpu/drm/panthor/panthor_gpu.h | 52 +
drivers/gpu/drm/panthor/panthor_heap.c | 596 +++
drivers/gpu/drm/panthor/panthor_heap.h | 39 +
drivers/gpu/drm/panthor/panthor_mmu.c | 2760 +++++++++++++
drivers/gpu/drm/panthor/panthor_mmu.h | 102 +
drivers/gpu/drm/panthor/panthor_regs.h | 239 ++
drivers/gpu/drm/panthor/panthor_sched.c | 3500 +++++++++++++++++
drivers/gpu/drm/panthor/panthor_sched.h | 48 +
include/uapi/drm/panthor_drm.h | 945 +++++
26 files changed, 13892 insertions(+)
create mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-valhall-csf.yaml
create mode 100644 drivers/gpu/drm/panthor/Kconfig
create mode 100644 drivers/gpu/drm/panthor/Makefile
create mode 100644 drivers/gpu/drm/panthor/panthor_devfreq.c
create mode 100644 drivers/gpu/drm/panthor/panthor_devfreq.h
create mode 100644 drivers/gpu/drm/panthor/panthor_device.c
create mode 100644 drivers/gpu/drm/panthor/panthor_device.h
create mode 100644 drivers/gpu/drm/panthor/panthor_drv.c
create mode 100644 drivers/gpu/drm/panthor/panthor_fw.c
create mode 100644 drivers/gpu/drm/panthor/panthor_fw.h
create mode 100644 drivers/gpu/drm/panthor/panthor_gem.c
create mode 100644 drivers/gpu/drm/panthor/panthor_gem.h
create mode 100644 drivers/gpu/drm/panthor/panthor_gpu.c
create mode 100644 drivers/gpu/drm/panthor/panthor_gpu.h
create mode 100644 drivers/gpu/drm/panthor/panthor_heap.c
create mode 100644 drivers/gpu/drm/panthor/panthor_heap.h
create mode 100644 drivers/gpu/drm/panthor/panthor_mmu.c
create mode 100644 drivers/gpu/drm/panthor/panthor_mmu.h
create mode 100644 drivers/gpu/drm/panthor/panthor_regs.h
create mode 100644 drivers/gpu/drm/panthor/panthor_sched.c
create mode 100644 drivers/gpu/drm/panthor/panthor_sched.h
create mode 100644 include/uapi/drm/panthor_drm.h
--
2.43.0
More information about the dri-devel
mailing list