[PATCH 1/2] drm/amdgpu/vm: free pte update job on error

[Benjamin Cheng]

In the happy path, the job gets freed when the job is submitted.
However, if amdgpu_vm_ptes_update failed, we don't submit the job, so we
would leak the job.

Fixes: 009f401fb98b ("drm/amdgpu: move amdgpu_vm_bo_update() call into correct 
location")
Signed-off-by: Benjamin Cheng <benjamin.ch...@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
index 2ba33212d507..f6cb2b5854dd 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
@@ -1274,8 +1274,11 @@ int amdgpu_vm_update_range(struct amdgpu_device *adev, 
struct amdgpu_vm *vm,
 
                tmp = start + num_entries;
                r = amdgpu_vm_ptes_update(&params, start, tmp, addr, flags);
-               if (r)
+               if (r) {
+                       if (params.job)
+                               amdgpu_job_free(params.job);
                        goto error_free;
+               }
 
                amdgpu_res_next(&cursor, num_entries * AMDGPU_GPU_PAGE_SIZE);
                start = tmp;
-- 
2.56.0