RE: [PATCH 1/1] drm/amdkfd: Fix variable set but not used warning

2022-01-28 Thread Kasiviswanathan, Harish
[AMD Official Use Only]

Reviewed-By: Harish Kasiviswanathan 

-Original Message-
From: amd-gfx  On Behalf Of Philip Yang
Sent: Friday, January 28, 2022 9:39 AM
To: amd-gfx@lists.freedesktop.org
Cc: Yang, Philip 
Subject: [PATCH 1/1] drm/amdkfd: Fix variable set but not used warning

All warnings (new ones prefixed by >>):

   drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_svm.c: In function
'svm_range_deferred_list_work':
>> drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_svm.c:2067:22: warning:
variable 'p' set but not used [-Wunused-but-set-variable]
2067 |  struct kfd_process *p;
 |

Fixes: 8b633bdc86671("drm/amdkfd: Ensure mm remain valid in svm deferred_list 
work")

Reported-by: kernel test robot 
Signed-off-by: Philip Yang 
---
 drivers/gpu/drm/amd/amdkfd/kfd_svm.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_svm.c 
b/drivers/gpu/drm/amd/amdkfd/kfd_svm.c
index 649c1d2b9607..9a509ec8c327 100644
--- a/drivers/gpu/drm/amd/amdkfd/kfd_svm.c
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_svm.c
@@ -2079,13 +2079,10 @@ static void svm_range_deferred_list_work(struct 
work_struct *work)
struct svm_range_list *svms;
struct svm_range *prange;
struct mm_struct *mm;
-   struct kfd_process *p;
 
svms = container_of(work, struct svm_range_list, deferred_list_work);
pr_debug("enter svms 0x%p\n", svms);
 
-   p = container_of(svms, struct kfd_process, svms);
-
spin_lock(>deferred_list_lock);
while (!list_empty(>deferred_range_list)) {
prange = list_first_entry(>deferred_range_list,
--
2.17.1


[PATCH 1/1] drm/amdkfd: Fix variable set but not used warning

2022-01-28 Thread Philip Yang
All warnings (new ones prefixed by >>):

   drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_svm.c: In function
'svm_range_deferred_list_work':
>> drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_svm.c:2067:22: warning:
variable 'p' set but not used [-Wunused-but-set-variable]
2067 |  struct kfd_process *p;
 |

Fixes: 8b633bdc86671("drm/amdkfd: Ensure mm remain valid in svm
deferred_list work")

Reported-by: kernel test robot 
Signed-off-by: Philip Yang 
---
 drivers/gpu/drm/amd/amdkfd/kfd_svm.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_svm.c 
b/drivers/gpu/drm/amd/amdkfd/kfd_svm.c
index 649c1d2b9607..9a509ec8c327 100644
--- a/drivers/gpu/drm/amd/amdkfd/kfd_svm.c
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_svm.c
@@ -2079,13 +2079,10 @@ static void svm_range_deferred_list_work(struct 
work_struct *work)
struct svm_range_list *svms;
struct svm_range *prange;
struct mm_struct *mm;
-   struct kfd_process *p;
 
svms = container_of(work, struct svm_range_list, deferred_list_work);
pr_debug("enter svms 0x%p\n", svms);
 
-   p = container_of(svms, struct kfd_process, svms);
-
spin_lock(>deferred_list_lock);
while (!list_empty(>deferred_range_list)) {
prange = list_first_entry(>deferred_range_list,
-- 
2.17.1