habanalabs: context cleanup cosmetics

No need to check the return value if the following action is the same for
both cases. In addition, now that hl_ctx_free() doesn't print if the
context is not released, its name can be misleading as the context might
stay alive after it is executed with no indication for that.
Hence we can discard it and simply put the refcount.

Signed-off-by: Omer Shpigelman <oshpigelman@habana.ai>
Reviewed-by: Oded Gabbay <ogabbay@kernel.org>
Signed-off-by: Oded Gabbay <ogabbay@kernel.org>

diff --git a/drivers/misc/habanalabs/common/context.c b/drivers/misc/habanalabs/common/context.c
index 22978303ad63f192e6b75810b42d68deb2e3ec60..d0aaccd4df2cdd133f32e1e2a0818c9b596134f2 100644 (file)
--- a/drivers/misc/habanalabs/common/context.c
+++ b/drivers/misc/habanalabs/common/context.c
@@ -181,12 +181,6 @@ out_err:
        return rc;
 }
 
-void hl_ctx_free(struct hl_device *hdev, struct hl_ctx *ctx)
-{
-       if (kref_put(&ctx->refcount, hl_ctx_do_release) == 1)
-               return;
-}
-
 int hl_ctx_init(struct hl_device *hdev, struct hl_ctx *ctx, bool is_kernel_ctx)
 {
        int rc = 0;
@@ -392,7 +386,7 @@ void hl_ctx_mgr_fini(struct hl_device *hdev, struct hl_ctx_mgr *mgr)
        idp = &mgr->ctx_handles;
 
        idr_for_each_entry(idp, ctx, id)
-               hl_ctx_free(hdev, ctx);
+               kref_put(&ctx->refcount, hl_ctx_do_release);
 
        idr_destroy(&mgr->ctx_handles);
        mutex_destroy(&mgr->ctx_lock);