On 12/09/2025 14:19, Adrián Larumbe wrote:
> From: Boris Brezillon <boris.brezil...@collabora.com>
>
> Minor revision of the driver must be bumped because this expands the
> uAPI. On top of that, let UM know about the available priorities so that
> they can create contexts with legal priority values.
>
> Signed-off-by: Boris Brezillon <boris.brezil...@collabora.com>
> Signed-off-by: Adrián Larumbe <adrian.laru...@collabora.com>
Reviewed-by: Steven Price <steven.pr...@arm.com>
> ---
> drivers/gpu/drm/panfrost/panfrost_drv.c | 36 ++++++++++++++++++++++---
> 1 file changed, 33 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/panfrost/panfrost_drv.c
> b/drivers/gpu/drm/panfrost/panfrost_drv.c
> index be384b18e8fd..69e72a800cd1 100644
> --- a/drivers/gpu/drm/panfrost/panfrost_drv.c
> +++ b/drivers/gpu/drm/panfrost/panfrost_drv.c
> @@ -109,6 +109,14 @@ static int panfrost_ioctl_get_param(struct drm_device
> *ddev, void *data, struct
> #endif
> break;
>
> + case DRM_PANFROST_PARAM_ALLOWED_JM_CTX_PRIORITIES:
> + param->value = BIT(PANFROST_JM_CTX_PRIORITY_LOW) |
> + BIT(PANFROST_JM_CTX_PRIORITY_MEDIUM);
> +
> + if (panfrost_high_prio_allowed(file))
> + param->value |= BIT(PANFROST_JM_CTX_PRIORITY_HIGH);
> + break;
> +
> default:
> return -EINVAL;
> }
> @@ -286,9 +294,6 @@ static int panfrost_ioctl_submit(struct drm_device *dev,
> void *data,
> if (args->pad)
> return -EINVAL;
>
> - if (args->jm_ctx_handle)
> - return -EINVAL;
> -
> if (!args->jc)
> return -EINVAL;
>
> @@ -552,6 +557,27 @@ static int panfrost_ioctl_set_label_bo(struct drm_device
> *ddev, void *data,
> return ret;
> }
>
> +static int panfrost_ioctl_jm_ctx_create(struct drm_device *dev, void *data,
> + struct drm_file *file)
> +{
> + return panfrost_jm_ctx_create(file, data);
> +}
> +
> +static int panfrost_ioctl_jm_ctx_destroy(struct drm_device *dev, void *data,
> + struct drm_file *file)
> +{
> + const struct drm_panfrost_jm_ctx_destroy *args = data;
> +
> + if (args->pad)
> + return -EINVAL;
> +
> + /* We can't destroy the default context created when the file is
> opened. */
> + if (!args->handle)
> + return -EINVAL;
> +
> + return panfrost_jm_ctx_destroy(file, args->handle);
> +}
> +
> int panfrost_unstable_ioctl_check(void)
> {
> if (!unstable_ioctls)
> @@ -619,6 +645,8 @@ static const struct drm_ioctl_desc
> panfrost_drm_driver_ioctls[] = {
> PANFROST_IOCTL(PERFCNT_DUMP, perfcnt_dump, DRM_RENDER_ALLOW),
> PANFROST_IOCTL(MADVISE, madvise, DRM_RENDER_ALLOW),
> PANFROST_IOCTL(SET_LABEL_BO, set_label_bo, DRM_RENDER_ALLOW),
> + PANFROST_IOCTL(JM_CTX_CREATE, jm_ctx_create, DRM_RENDER_ALLOW),
> + PANFROST_IOCTL(JM_CTX_DESTROY, jm_ctx_destroy, DRM_RENDER_ALLOW),
> };
>
> static void panfrost_gpu_show_fdinfo(struct panfrost_device *pfdev,
> @@ -715,6 +743,8 @@ static void panfrost_debugfs_init(struct drm_minor *minor)
> * - 1.3 - adds JD_REQ_CYCLE_COUNT job requirement for SUBMIT
> * - adds SYSTEM_TIMESTAMP and SYSTEM_TIMESTAMP_FREQUENCY queries
> * - 1.4 - adds SET_LABEL_BO
> + * - 1.5 - adds JM_CTX_{CREATE,DESTROY} ioctls and extend SUBMIT to allow
> + * context creation with configurable priorities/affinity
> */
> static const struct drm_driver panfrost_drm_driver = {
> .driver_features = DRIVER_RENDER | DRIVER_GEM | DRIVER_SYNCOBJ,
