Re: [PATCH 1/2] drm/amdkfd: remove unused code

[Philip Yang]

On 2025-05-28 13:19, James Zhu wrote:

upages is assigned under cpages = 0, so it isn't really used in this function.

Signed-off-by: James Zhu <james....@amd.com>

Reviewed-by: Philip.Yang<philip.y...@amd.com>

---
  drivers/gpu/drm/amd/amdkfd/kfd_migrate.c | 1 -
  1 file changed, 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c 
b/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
index 79251f22b702..f0b690d4bb46 100644
--- a/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
@@ -736,7 +736,6 @@ svm_migrate_vma_to_ram(struct kfd_node *node, struct 
svm_range *prange,
        if (!cpages) {
                pr_debug("failed collect migrate device pages [0x%lx 0x%lx]\n",
                         prange->start, prange->last);
-               upages = svm_migrate_unsuccessful_pages(&migrate);
                goto out_free;
        }
        if (cpages != npages)