gpu: nvgpu: Keep lock while unreffing mappings

In the vm_area free code, when unreffing the mappings owned by
the vm_area, we need to continue holding the VM lock.

Also add a comment specifying this requirement in the VM code.

Bug 2156667

Change-Id: If0b430f045e4c585fcba2d3176163e5b19be8326
Signed-off-by: Alex Waterman <alexw@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1763235
Reviewed-by: svc-mobile-coverity <svc-mobile-coverity@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Terje Bergstrom <tbergstrom@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>

1 files changed, 2 insertions, 1 deletions

diff --git a/drivers/gpu/nvgpu/common/mm/vm_area.c b/drivers/gpu/nvgpu/common/mm/vm_area.c
index 663d8999..5d3b0526 100644
--- a/drivers/gpu/nvgpu/common/mm/vm_area.c
+++ b/drivers/gpu/nvgpu/common/mm/vm_area.c
@@ -199,7 +199,6 @@ int nvgpu_vm_area_free(struct vm_gk20a *vm, u64 addr)
                 return 0;
         }
         nvgpu_list_del(&vm_area->vm_area_list);
-        nvgpu_mutex_release(&vm->update_gmmu_lock);
 
         nvgpu_log(g, gpu_dbg_map,
                   "DEL vm_area: pgsz=%#-8x pages=%-9llu "
@@ -219,6 +218,8 @@ int nvgpu_vm_area_free(struct vm_gk20a *vm, u64 addr)
                 nvgpu_ref_put(&buffer->ref, __nvgpu_vm_unmap_ref);
         }
 
+        nvgpu_mutex_release(&vm->update_gmmu_lock);
+
         /* if this was a sparse mapping, free the va */
         if (vm_area->sparse)
                 g->ops.mm.gmmu_unmap(vm,