[PATCH] drm/amdkfd: Fix resource leak in riu rsetore queue

Alex Deucher alexdeucher at gmail.com
Mon Sep 9 17:55:30 UTC 2024


On Sun, Sep 8, 2024 at 9:20 PM Jesse.zhang at amd.com <jesse.zhang at amd.com> wrote:
>
> To avoid memory leaks, release q_extra_data when exiting the restore queue.
> v2: Correct the proto (Alex)

should be criu, not riu.

Alex

>
> Signed-off-by: Jesse Zhang <jesse.zhang at amd.com>
> ---
>  drivers/gpu/drm/amd/amdkfd/kfd_process_queue_manager.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_process_queue_manager.c b/drivers/gpu/drm/amd/amdkfd/kfd_process_queue_manager.c
> index 20ea745729ee..b439d4d0bd84 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_process_queue_manager.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_process_queue_manager.c
> @@ -1046,6 +1046,7 @@ int kfd_criu_restore_queue(struct kfd_process *p,
>                 pr_debug("Queue id %d was restored successfully\n", queue_id);
>
>         kfree(q_data);
> +       kfree(q_extra_data);
>
>         return ret;
>  }
> --
> 2.25.1
>


More information about the amd-gfx mailing list