Patches 1, 3, 4, 5, 6, 7, 8, 9 look good to me. I think those can
land. Just a minor comment on patch 2.
Alex
On Wed, Sep 3, 2025 at 4:59 AM Jesse.Zhang <jesse.zh...@amd.com> wrote:
>
> From: Alex Deucher <alexander.deuc...@amd.com>
>
> Add two new function pointers to struct amdgpu_userq_funcs:
> - preempt: To handle preemption of user mode queues
> - restore: To restore preempted user mode queues
>
> These callbacks will allow the driver to properly manage queue
> preemption and restoration when needed, such as during context
> switching or priority changes.
>
> Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_userq.h | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_userq.h
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_userq.h
> index ec040c2fd6c9..5111d7dce86f 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_userq.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_userq.h
> @@ -77,6 +77,10 @@ struct amdgpu_userq_funcs {
> struct amdgpu_usermode_queue *queue);
> int (*map)(struct amdgpu_userq_mgr *uq_mgr,
> struct amdgpu_usermode_queue *queue);
> + int (*preempt)(struct amdgpu_userq_mgr *uq_mgr,
> + struct amdgpu_usermode_queue *queue);
> + int (*restore)(struct amdgpu_userq_mgr *uq_mgr,
> + struct amdgpu_usermode_queue *queue);
> };
>
> /* Usermode queues for gfx */
> --
> 2.49.0
>
