On 04/23/2015 12:58 PM, Firo Yang wrote:
> kmalloc() returns a void pointer - no need to cast it in
> drivers/gpu/drm/amd/amdkfd/kfd_process.c::kfd_process_destroy_delayed()
>
> Signed-off-by: Firo Yang <firogm at gmail.com>
> ---
>   drivers/gpu/drm/amd/amdkfd/kfd_process.c | 3 +--
>   1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_process.c 
> b/drivers/gpu/drm/amd/amdkfd/kfd_process.c
> index 945d622..4d7bc95 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_process.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_process.c
> @@ -203,8 +203,7 @@ static void kfd_process_destroy_delayed(struct rcu_head 
> *rcu)
>
>       mmdrop(p->mm);
>
> -     work = (struct kfd_process_release_work *)
> -             kmalloc(sizeof(struct kfd_process_release_work), GFP_ATOMIC);
> +     work = kmalloc(sizeof(struct kfd_process_release_work), GFP_ATOMIC);
>
>       if (work) {
>               INIT_WORK((struct work_struct *) work, kfd_process_wq_release);
>
Thanks!
Applied to my -next tree

        Oded

Reply via email to