| From c4b003f50515741f7b7042c0c5ede28fcee56937 Mon Sep 17 00:00:00 2001 |
| From: Jack Zhang <Jack.Zhang1@amd.com> |
| Date: Wed, 1 Apr 2020 20:06:58 +0800 |
| Subject: [PATCH] drm/amdkfd: kfree the wrong pointer |
| |
| commit 3148a6a0ef3cf93570f30a477292768f7eb5d3c3 upstream. |
| |
| Originally, it kfrees the wrong pointer for mem_obj. |
| It would cause memory leak under stress test. |
| |
| Signed-off-by: Jack Zhang <Jack.Zhang1@amd.com> |
| Acked-by: Nirmoy Das <nirmoy.das@amd.com> |
| Signed-off-by: Alex Deucher <alexander.deucher@amd.com> |
| Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> |
| |
| diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device.c b/drivers/gpu/drm/amd/amdkfd/kfd_device.c |
| index 765b58a17dc7..af67c13ae386 100644 |
| --- a/drivers/gpu/drm/amd/amdkfd/kfd_device.c |
| +++ b/drivers/gpu/drm/amd/amdkfd/kfd_device.c |
| @@ -1001,9 +1001,9 @@ int kfd_gtt_sa_allocate(struct kfd_dev *kfd, unsigned int size, |
| return 0; |
| |
| kfd_gtt_no_free_chunk: |
| - pr_debug("Allocation failed with mem_obj = %p\n", mem_obj); |
| + pr_debug("Allocation failed with mem_obj = %p\n", *mem_obj); |
| mutex_unlock(&kfd->gtt_sa_lock); |
| - kfree(mem_obj); |
| + kfree(*mem_obj); |
| return -ENOMEM; |
| } |
| |
| -- |
| 2.7.4 |
| |