diff options
author | Alex Waterman <alexw@nvidia.com> | 2018-03-23 16:35:34 -0400 |
---|---|---|
committer | mobile promotions <svcmobile_promotions@nvidia.com> | 2018-03-26 17:06:50 -0400 |
commit | a3a00f1d941f0bd180e0191a708a23579b1bede9 (patch) | |
tree | a470af9677e7a884bba4d9f815b4020dfadff4f3 /drivers | |
parent | c392a7270fd13fa0eb1e26ef0ef9f8f4479630a1 (diff) |
gpu: nvgpu: Check return code in gp10b_get_pde0_pgsz()
nvgpu_mem_begin() needs to have it's return code checked to
make sure that it did not fail.
Coverity defect id: 2883904
Bug 200291879
Change-Id: Ic845a73439a905997a1f871708b1f847f5b93293
Signed-off-by: Alex Waterman <alexw@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1681249
Reviewed-by: Terje Bergstrom <tbergstrom@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/gpu/nvgpu/gp10b/mm_gp10b.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/gpu/nvgpu/gp10b/mm_gp10b.c b/drivers/gpu/nvgpu/gp10b/mm_gp10b.c index 43d9fd28..978b6f50 100644 --- a/drivers/gpu/nvgpu/gp10b/mm_gp10b.c +++ b/drivers/gpu/nvgpu/gp10b/mm_gp10b.c | |||
@@ -345,7 +345,9 @@ static enum gmmu_pgsz_gk20a gp10b_get_pde0_pgsz(struct gk20a *g, | |||
345 | if (!pd->mem) | 345 | if (!pd->mem) |
346 | return pgsz; | 346 | return pgsz; |
347 | 347 | ||
348 | nvgpu_mem_begin(g, pd->mem); | 348 | if (nvgpu_mem_begin(g, pd->mem)) |
349 | return pgsz; | ||
350 | |||
349 | for (i = 0; i < GP10B_PDE0_ENTRY_SIZE >> 2; i++) | 351 | for (i = 0; i < GP10B_PDE0_ENTRY_SIZE >> 2; i++) |
350 | pde_v[i] = nvgpu_mem_rd32(g, pd->mem, pde_offset + i); | 352 | pde_v[i] = nvgpu_mem_rd32(g, pd->mem, pde_offset + i); |
351 | nvgpu_mem_end(g, pd->mem); | 353 | nvgpu_mem_end(g, pd->mem); |