nv-tegra-mirror/linux-nvgpu
1
0
Fork 0
mirror of git://nv-tegra.nvidia.com/linux-nvgpu.git synced 2025-12-22 09:12:24 +03:00
Code Issues Packages Projects Releases Wiki Activity

gpu: nvgpu: fix Cert-C errors in vm.c

Browse Source 
INT30-C requires that unsigned integer operations do not wrap. This
patch adds safe operation APIs to resolve Cert-C errors.

Jira NVGPU-4677

Change-Id: I7dad28e8de9fe8ea1bdc0ca33b8cebe103cac5a7
Signed-off-by: Vedashree Vidwans <vvidwans@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/2264218
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: svc-mobile-coverity <svc-mobile-coverity@nvidia.com>
Reviewed-by: svc-mobile-misra <svc-mobile-misra@nvidia.com>
Reviewed-by: svc-mobile-cert <svc-mobile-cert@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Sagar Kamble <skamble@nvidia.com>
Reviewed-by: Alex Waterman <alexw@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
This commit is contained in:
Vedashree Vidwans
2019-12-17 09:26:14 -08:00
committed by Alex Waterman
parent a50802510f
commit 1ec4a4f8ec
1 changed files with 13 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
drivers/gpu/nvgpu/common/mm/vm.c
View File

@@ -504,11 +504,13 @@ static void nvgpu_vm_init_check_big_pages(struct vm_gk20a *vm,
if (unified_va) {
vm->big_pages = nvgpu_big_pages_possible(vm,
user_vma_start,
user_vma_limit - user_vma_start);
nvgpu_safe_sub_u64(user_vma_limit,
user_vma_start));
} else {
vm->big_pages = nvgpu_big_pages_possible(vm,
user_lp_vma_start,
user_lp_vma_limit - user_lp_vma_start);
nvgpu_safe_sub_u64(user_lp_vma_limit,
user_lp_vma_start));
}
}
}
@@ -555,7 +557,7 @@ static int nvgpu_vm_init_vma(struct gk20a *g, struct vm_gk20a *vm,
u64 kernel_vma_flags;
/* Setup vma limits. */
if (kernel_reserved + low_hole < aperture_size) {
if (nvgpu_safe_add_u64(kernel_reserved, low_hole) < aperture_size) {
/*
* If big_pages are disabled for this VM then it only makes
* sense to make one VM, same as if the unified address flag
@@ -563,14 +565,16 @@ static int nvgpu_vm_init_vma(struct gk20a *g, struct vm_gk20a *vm,
*/
if (!big_pages || unified_va) {
user_vma_start = low_hole;
user_vma_limit = vm->va_limit - kernel_reserved;
user_vma_limit = nvgpu_safe_sub_u64(vm->va_limit,
kernel_reserved);
user_lp_vma_start = user_vma_limit;
user_lp_vma_limit = user_vma_limit;
} else {
user_vma_start = low_hole;
user_vma_limit = nvgpu_gmmu_va_small_page_limit();
user_lp_vma_start = nvgpu_gmmu_va_small_page_limit();
user_lp_vma_limit = vm->va_limit - kernel_reserved;
user_lp_vma_limit = nvgpu_safe_sub_u64(vm->va_limit,
kernel_reserved);
}
} else {
user_vma_start = 0;
@@ -578,7 +582,7 @@ static int nvgpu_vm_init_vma(struct gk20a *g, struct vm_gk20a *vm,
user_lp_vma_start = 0;
user_lp_vma_limit = 0;
}
kernel_vma_start = vm->va_limit - kernel_reserved;
kernel_vma_start = nvgpu_safe_sub_u64(vm->va_limit, kernel_reserved);
kernel_vma_limit = vm->va_limit;
nvgpu_log_info(g, "user_vma [0x%llx,0x%llx)",
@@ -598,7 +602,8 @@ static int nvgpu_vm_init_vma(struct gk20a *g, struct vm_gk20a *vm,
goto clean_up_page_tables;
}
kernel_vma_flags = (kernel_reserved + low_hole) == aperture_size ?
kernel_vma_flags = nvgpu_safe_add_u64(kernel_reserved, low_hole) ==
aperture_size ?
0ULL : GPU_ALLOC_GVA_SPACE;
nvgpu_vm_init_check_big_pages(vm, user_vma_start, user_vma_limit,