Re: [PATCH 4/4] drm/todos: add entry for drm_syncobj error handling

[Daniel Vetter <daniel.vetter@ffwll.ch>]

On Mon, Aug 26, 2024 at 02:25:41PM +0200, Christian König wrote:
> That would be rather nice to have and the kernel side is really trivial,
> only the userspace side might be a bit more complex.
> 
> Signed-off-by: Christian König <christian.koenig@amd.com>

I think patch 1 should ideally have acks from a few drivers using
drm/sched, but patches 2-4 make me happy, thanks a lot for creating them,
so on those.

Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>

Cheers, Sima

> ---
>  Documentation/gpu/todo.rst | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)
> 
> diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst
> index 96c453980ab6..c771f0c9610f 100644
> --- a/Documentation/gpu/todo.rst
> +++ b/Documentation/gpu/todo.rst
> @@ -834,6 +834,22 @@ Contact: Javier Martinez Canillas <javierm@redhat.com>
>  
>  Level: Advanced
>  
> +Querying errors from drm_syncobj
> +================================
> +
> +The drm_syncobj container can be used by driver independent code to signal
> +complection of submission.
> +
> +One minor feature still missing is a generic DRM IOCTL to query the error
> +status of binary and timeline drm_syncobj.
> +
> +This should probably be improved by implementing the necessary kernel interface
> +and adding support for that in the userspace stack.
> +
> +Contact: Christian König
> +
> +Level: Starter
> +
>  Outside DRM
>  ===========
>  
> -- 
> 2.34.1
> 

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch