[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <6216c60b-99c0-47fd-b7e8-b6c104139a45@arm.com>
Date: Tue, 3 Sep 2024 14:27:51 +0200
From: Ketil Johnsen <ketil.johnsen@....com>
To: Mihail Atanassov <mihail.atanassov@....com>,
linux-kernel@...r.kernel.org, Boris Brezillon
<boris.brezillon@...labora.com>, Liviu Dudau <liviu.dudau@....com>,
Steven Price <steven.price@....com>
Cc: dri-devel@...ts.freedesktop.org, Daniel Vetter <daniel@...ll.ch>,
David Airlie <airlied@...il.com>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>,
Alex Deucher <alexander.deucher@....com>,
Christian König <christian.koenig@....com>,
Xinhui Pan <Xinhui.Pan@....com>, Shashank Sharma <shashank.sharma@....com>,
Akash Goel <akash.goel@....com>
Subject: Re: [RFC PATCH 00/10] drm/panthor: Add user submission
On 28.08.2024 19:25, Mihail Atanassov wrote:
> Hello all,
>
> This series implements a mechanism to expose Mali CSF GPUs' queue
> ringbuffers directly to userspace, along with paraphernalia to allow
> userspace to control job synchronisation between the CPU and GPU.
>
> The goal of these changes is to allow userspace to control work
> submission to the FW/HW directly without kernel intervention in the
> common case, thereby reducing context switching overhead. It also allows
> for greater flexibility in the way work is enqueued in the ringbufs.
> For example, the current kernel submit path only supports indirect
> calls, which is inefficient for small command buffers. Userspace can
> also skip unnecessary sync operations.
>
> This is still a work-in-progress, there's an outstanding issue with
> multiple processes using different submission flows triggering
> scheduling bugs (e.g. the same group getting scheduled twice), but we'd
> love to gather some feedback on the suitability of the approach in
> general and see if there's a clear path to merging something like this
> eventually.
>
> I've also CCd AMD maintainers because they have in the past done
> something similar[1], in case they want to chime in.
>
> There are two uses of this new uAPI in Mesa, one in gallium/panfrost
> (link TBD), and one in panvk [2].
Gallium/Panfrost changes to make use of this new user submission API can
now be found here:
https://gitlab.freedesktop.org/ketil.johnsen/mesa/-/commits/panthor_usersubmit/?ref_type=heads
It is worth repeating, this is just a dumb switch from kernel submission
to user submission for the Panfrost Gallium driver, no optimizations
attempted. We have no concrete plans to pursue upstreaming of this for
the time being. Panvk is our focus in that regard.
--
Regards,
Ketil Johnsen
> The Gallium implementation is a naïve change just to switch the
> submission model and exercise the new kernel code, and we don't plan
> on pursuing this at this time.
>
> The panvk driver changes are, however, a better representation of the
> intent behind this new uAPI, so please consider that as the reference
> userspace. It is still very much also a work in progress.
>
> * patch 1 adds all the uAPI changes;
> * patch 2 implements the GROUP_CREATE ioctl changes necessary to expose
> the required objects to userspace;
> * patch 3 maps the doorbell pages, similarly to how the user I/O page is
> mapped;
> * patch 4 implements GROUP_KICK, which lets userspace request an
> inactive group to be scheduled on the GPU;
> * patches 5 & 6 implement XGS queues, a way for userspace to
> synchronise GPU queue progress with DRM syncobjs;
> * patches 7 & 8 add notification mechanisms for user & kernel to signal
> changes to native GPU syncobjs.
>
> [1] https://lore.kernel.org/amd-gfx/CADnq5_N61q_o+5WYUZsZ=qu7VmeXTFHQSxLwTco05gLzHaiswA@mail.gmail.com/t/#m116a36a598d8fad1329e053974ad37a4dc0f28ed
> [2] https://gitlab.freedesktop.org/larsivsi/mesa/-/commits/panvk-v10-usersubmit?ref_type=heads
>
> Ketil Johnsen (7):
> drm/panthor: Add uAPI to submit from user space
> drm/panthor: Extend GROUP_CREATE for user submission
> drm/panthor: Map doorbell pages
> drm/panthor: Add GROUP_KICK ioctl
> drm/panthor: Factor out syncobj handling
> drm/panthor: Implement XGS queues
> drm/panthor: Add SYNC_UPDATE ioctl
>
> Mihail Atanassov (1):
> drm/panthor: Add sync_update eventfd handling
>
> drivers/gpu/drm/panthor/Makefile | 4 +-
> drivers/gpu/drm/panthor/panthor_device.c | 66 ++-
> drivers/gpu/drm/panthor/panthor_device.h | 35 +-
> drivers/gpu/drm/panthor/panthor_drv.c | 233 +++++++-
> drivers/gpu/drm/panthor/panthor_fw.c | 2 +-
> drivers/gpu/drm/panthor/panthor_sched.c | 408 +++++++++-----
> drivers/gpu/drm/panthor/panthor_sched.h | 8 +-
> drivers/gpu/drm/panthor/panthor_syncobj.c | 167 ++++++
> drivers/gpu/drm/panthor/panthor_syncobj.h | 27 +
> drivers/gpu/drm/panthor/panthor_xgs.c | 638 ++++++++++++++++++++++
> drivers/gpu/drm/panthor/panthor_xgs.h | 42 ++
> include/uapi/drm/panthor_drm.h | 243 +++++++-
> 12 files changed, 1696 insertions(+), 177 deletions(-)
> create mode 100644 drivers/gpu/drm/panthor/panthor_syncobj.c
> create mode 100644 drivers/gpu/drm/panthor/panthor_syncobj.h
> create mode 100644 drivers/gpu/drm/panthor/panthor_xgs.c
> create mode 100644 drivers/gpu/drm/panthor/panthor_xgs.h
>
Powered by blists - more mailing lists