diff options
author | Stephen Warren <swarren@nvidia.com> | 2017-05-22 14:27:40 -0400 |
---|---|---|
committer | mobile promotions <svcmobile_promotions@nvidia.com> | 2017-05-26 06:34:30 -0400 |
commit | 2e338c77eac4edffb94c8c9480dbd72712c7696f (patch) | |
tree | 9d5da6355bc653ef2ba9014ea696ad0391c523a1 /drivers/gpu/nvgpu/vgpu/mm_vgpu.c | |
parent | 726900b8433294fd89a6d730d2fec9de8e33afda (diff) |
gpu: nvgpu: remove duplicate \n from log messages
nvgpu_log/info/warn/err() internally add a \n to the end of the message.
Hence, callers should not include a \n at the end of the message. Doing
so results in duplicate \n being printed, which ends up creating empty
log messages. Remove the duplicate \n from all err/warn messages.
Bug 1928311
Change-Id: I99362c5327f36146f28ba63d4e68181589735c39
Signed-off-by: Stephen Warren <swarren@nvidia.com>
Reviewed-on: http://git-master/r/1487232
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
Diffstat (limited to 'drivers/gpu/nvgpu/vgpu/mm_vgpu.c')
-rw-r--r-- | drivers/gpu/nvgpu/vgpu/mm_vgpu.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/gpu/nvgpu/vgpu/mm_vgpu.c b/drivers/gpu/nvgpu/vgpu/mm_vgpu.c index db120d76..287567d6 100644 --- a/drivers/gpu/nvgpu/vgpu/mm_vgpu.c +++ b/drivers/gpu/nvgpu/vgpu/mm_vgpu.c | |||
@@ -110,7 +110,7 @@ static u64 vgpu_locked_gmmu_map(struct vm_gk20a *vm, | |||
110 | map_offset = __nvgpu_vm_alloc_va(vm, size, | 110 | map_offset = __nvgpu_vm_alloc_va(vm, size, |
111 | pgsz_idx); | 111 | pgsz_idx); |
112 | if (!map_offset) { | 112 | if (!map_offset) { |
113 | nvgpu_err(g, "failed to allocate va space\n"); | 113 | nvgpu_err(g, "failed to allocate va space"); |
114 | err = -ENOMEM; | 114 | err = -ENOMEM; |
115 | goto fail; | 115 | goto fail; |
116 | } | 116 | } |
@@ -138,7 +138,7 @@ static u64 vgpu_locked_gmmu_map(struct vm_gk20a *vm, | |||
138 | vm->gmmu_page_sizes[gmmu_page_size_big]) { | 138 | vm->gmmu_page_sizes[gmmu_page_size_big]) { |
139 | pgsz_idx = gmmu_page_size_big; | 139 | pgsz_idx = gmmu_page_size_big; |
140 | } else { | 140 | } else { |
141 | nvgpu_err(g, "invalid kernel page size %d\n", | 141 | nvgpu_err(g, "invalid kernel page size %d", |
142 | page_size); | 142 | page_size); |
143 | goto fail; | 143 | goto fail; |
144 | } | 144 | } |
@@ -160,7 +160,7 @@ static u64 vgpu_locked_gmmu_map(struct vm_gk20a *vm, | |||
160 | 160 | ||
161 | return map_offset; | 161 | return map_offset; |
162 | fail: | 162 | fail: |
163 | nvgpu_err(g, "%s: failed with err=%d\n", __func__, err); | 163 | nvgpu_err(g, "%s: failed with err=%d", __func__, err); |
164 | return 0; | 164 | return 0; |
165 | } | 165 | } |
166 | 166 | ||