Hi, On 07/28/2016 08:13 PM, Chunming Zhou wrote: > Change-Id: Ie3d5440dc0d2d3a61d8e785ab08b8b91eda223db > Signed-off-by: Chunming Zhou <[email protected]> > --- > 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 11c1263..1d58577 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > @@ -1682,7 +1682,7 @@ int amdgpu_vm_init(struct amdgpu_device *adev, struct > amdgpu_vm *vm) > r = amd_sched_entity_init(&ring->sched, &vm->entity, > rq, amdgpu_sched_jobs); > if (r)
Hmm while we are here I think we should be explicit that non-zero, negative return values indicate an error path, so: - if (r) + if (r < 0) This then follows precisely the semantics documented for 'amdgpu_vm_init()' invocations. Kind Regards, Edward. > - return r; > + goto err; > > vm->page_directory_fence = NULL; > > @@ -1725,6 +1725,9 @@ error_free_page_directory: > error_free_sched_entity: > amd_sched_entity_fini(&ring->sched, &vm->entity); > > +err: > + drm_free_large(vm->page_tables); > + > return r; > } > >
signature.asc
Description: OpenPGP digital signature
_______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
