From: Ketil Johnsen <ketil.johnsen@xxxxxxx> Similar to how the kernel driver notifies userspace about syncobj modifications, the reverse notification is also necessary to let panthor re-evaluate any queues (GPU or XGS) that were blocked on wait operations. Signed-off-by: Mihail Atanassov <mihail.atanassov@xxxxxxx> Signed-off-by: Ketil Johnsen <ketil.johnsen@xxxxxxx> --- drivers/gpu/drm/panthor/panthor_drv.c | 23 +++++++++++++++++++++++ 1 file changed, 23 insertions(+) diff --git a/drivers/gpu/drm/panthor/panthor_drv.c b/drivers/gpu/drm/panthor/panthor_drv.c index 681ac09b6343..1734a52dc3b5 100644 --- a/drivers/gpu/drm/panthor/panthor_drv.c +++ b/drivers/gpu/drm/panthor/panthor_drv.c @@ -1426,6 +1426,28 @@ static int panthor_ioctl_xgs_queue_submit(struct drm_device *ddev, void *data, return ret; } +static int panthor_ioctl_sync_update(struct drm_device *ddev, void *data, + struct drm_file *file) +{ + struct drm_panthor_sync_update *args = data; + struct panthor_device *ptdev = container_of(ddev, struct panthor_device, base); + struct panthor_file *pfile = file->driver_priv; + + if (args->pad) + return -EINVAL; + + if (pm_runtime_get_if_in_use(ddev->dev)) { + /* All sync wait ops will be re-evaluated when we ding the global doorbell */ + gpu_write(ptdev, CSF_DOORBELL(0), 1); + pm_runtime_put_autosuspend(ddev->dev); + } + + panthor_sched_sync_update(ptdev); + panthor_xgs_queue_pool_recheck(pfile); + + return 0; +} + static int panthor_open(struct drm_device *ddev, struct drm_file *file) { @@ -1508,6 +1530,7 @@ static const struct drm_ioctl_desc panthor_drm_driver_ioctls[] = { PANTHOR_IOCTL(XGS_QUEUE_CREATE, xgs_queue_create, DRM_RENDER_ALLOW), PANTHOR_IOCTL(XGS_QUEUE_DESTROY, xgs_queue_destroy, DRM_RENDER_ALLOW), PANTHOR_IOCTL(XGS_QUEUE_SUBMIT, xgs_queue_submit, DRM_RENDER_ALLOW), + PANTHOR_IOCTL(SYNC_UPDATE, sync_update, DRM_RENDER_ALLOW), }; static int panthor_mmap(struct file *filp, struct vm_area_struct *vma) -- 2.45.0