Re: [PATCH v9 4/6] drm/doc: Add a section about "Task information" for the wedge API

["Christian König" <christian.koenig@amd.com>]

On 6/17/25 14:49, André Almeida wrote:
> Add a section about "Task information" for the wedge API.
> 
> Reviewed-by: Krzysztof Karas <krzysztof.karas@intel.com>
> Reviewed-by: Raag Jadav <raag.jadav@intel.com>
> Signed-off-by: André Almeida <andrealmeid@igalia.com>

Reviewed-by: Christian König <christian.koenig@amd.com>

> ---
> v5:
>  - Change app to task in the text as well
> v4:
>  - Change APP to TASK
> v3:
>  - Change "app that caused ..." to "app involved ..."
>  - Clarify that devcoredump have more information about what happened
>  - Update that PID and APP will be empty if there's no app info
> ---
>  Documentation/gpu/drm-uapi.rst | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/Documentation/gpu/drm-uapi.rst b/Documentation/gpu/drm-uapi.rst
> index 4863a4deb0ee..263e5a97c080 100644
> --- a/Documentation/gpu/drm-uapi.rst
> +++ b/Documentation/gpu/drm-uapi.rst
> @@ -446,6 +446,23 @@ telemetry information (devcoredump, syslog). This is useful because the first
>  hang is usually the most critical one which can result in consequential hangs or
>  complete wedging.
>  
> +Task information
> +---------------
> +
> +The information about which application (if any) was involved in the device
> +wedging is useful for userspace if they want to notify the user about what
> +happened (e.g. the compositor display a message to the user "The <task name>
> +caused a graphical error and the system recovered") or to implement policies
> +(e.g. the daemon may "ban" an task that keeps resetting the device). If the task
> +information is available, the uevent will display as ``PID=<pid>`` and
> +``TASK=<task name>``. Otherwise, ``PID`` and ``TASK`` will not appear in the
> +event string.
> +
> +The reliability of this information is driver and hardware specific, and should
> +be taken with a caution regarding it's precision. To have a big picture of what
> +really happened, the devcoredump file provides should have much more detailed
> +information about the device state and about the event.
> +
>  Consumer prerequisites
>  ----------------------
>