Skip to contentSkip to conversation

amdkfd: Remove unessary void pointer cast

PATCH In review Firo Yang 2 participants 2 messages Apr 23, 2015 – Apr 26, 2015

Subsystems (4)
DRM drivers Kernel janitors Radeon and amdgpu DRM drivers AMD KFD
lore
Share & tools
lore

Firo Yang

author
#0 lore raw

PATCH 1/1 amdkfd: Remove unessary void pointer cast

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 <[email protected]>
---
 drivers/gpu/drm/amd/amdkfd/kfd_process.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
drivers/gpu/drm/amd/amdkfd/kfd_process.c +1 -2
index 945d622..4d7bc95 100644
@@ -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);
Signature · 1 line
2.1.0
#1 lore raw
Quoted 14 lines · Firo Yang

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 <[email protected]>
---
  drivers/gpu/drm/amd/amdkfd/kfd_process.c | 3 +--
  1 file changed, 1 insertion(+), 2 deletions(-)
drivers/gpu/drm/amd/amdkfd/kfd_process.c +0 -0
index 945d622..4d7bc95 100644

drivers/gpu/drm/amd/amdkfd/kfd_process.c static void kfd_process_destroy_delayed(struct rcu_head *rcu) · line 209

Quoted 9 lines
drivers/gpu/drm/amd/amdkfd/kfd_process.c +1 -2
@@ -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