drm/amdgpu: use failed label to handle context init failure
authorHuang Rui <ray.huang@amd.com>
Wed, 26 Oct 2016 09:07:03 +0000 (17:07 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 27 Oct 2016 19:18:09 +0000 (15:18 -0400)
Signed-off-by: Huang Rui <ray.huang@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c

index a5e2fcbef0f0f24f54bcf54164eb610c463edd49..6d86eaef934c4705ffad0b80172a7db47a64d94d 100644 (file)
@@ -55,18 +55,18 @@ static int amdgpu_ctx_init(struct amdgpu_device *adev, struct amdgpu_ctx *ctx)
                r = amd_sched_entity_init(&ring->sched, &ctx->rings[i].entity,
                                          rq, amdgpu_sched_jobs);
                if (r)
-                       break;
+                       goto failed;
        }
 
-       if (i < adev->num_rings) {
-               for (j = 0; j < i; j++)
-                       amd_sched_entity_fini(&adev->rings[j]->sched,
-                                             &ctx->rings[j].entity);
-               kfree(ctx->fences);
-               ctx->fences = NULL;
-               return r;
-       }
        return 0;
+
+failed:
+       for (j = 0; j < i; j++)
+               amd_sched_entity_fini(&adev->rings[j]->sched,
+                                     &ctx->rings[j].entity);
+       kfree(ctx->fences);
+       ctx->fences = NULL;
+       return r;
 }
 
 static void amdgpu_ctx_fini(struct amdgpu_ctx *ctx)