From: Andrew Donnellan <hidden> Date: 2020-02-26 04:42:51
In ocxl_context_free() we note that the AFU reference we're releasing was
taken in "ocxl_context_init", a function that doesn't actually exist.
Fix it to say ocxl_context_alloc() instead, which I expect was what was
intended.
Fixes: 5ef3166e8a32 ("ocxl: Driver code for 'generic' opencapi devices")
Cc: Frederic Barrat <redacted>
Signed-off-by: Andrew Donnellan <redacted>
---
drivers/misc/ocxl/context.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
@@ -287,7 +287,7 @@ void ocxl_context_free(struct ocxl_context *ctx)ocxl_afu_irq_free_all(ctx);idr_destroy(&ctx->irq_idr);-/* reference to the AFU taken in ocxl_context_init */+/* reference to the AFU taken in ocxl_context_alloc() */ocxl_afu_put(ctx->afu);kfree(ctx);}
In ocxl_context_free() we note that the AFU reference we're releasing was
taken in "ocxl_context_init", a function that doesn't actually exist.
Fix it to say ocxl_context_alloc() instead, which I expect was what was
intended.
Fixes: 5ef3166e8a32 ("ocxl: Driver code for 'generic' opencapi devices")
Cc: Frederic Barrat <redacted>
Signed-off-by: Andrew Donnellan <redacted>
---
ocxl_context_init() used to exist. It was renamed to
ocxl_context_alloc() for good reasons as part of later work. So it
should really be:
Fixes: b9721d275cc2 ("ocxl: Allow external drivers to use OpenCAPI
contexts")
Acked-by: Frederic Barrat <redacted>
@@ -287,7 +287,7 @@ void ocxl_context_free(struct ocxl_context *ctx)ocxl_afu_irq_free_all(ctx);idr_destroy(&ctx->irq_idr);-/* reference to the AFU taken in ocxl_context_init */+/* reference to the AFU taken in ocxl_context_alloc() */ocxl_afu_put(ctx->afu);kfree(ctx);}
From: Michael Ellerman <hidden> Date: 2020-06-09 05:33:39
On Wed, 26 Feb 2020 15:39:23 +1100, Andrew Donnellan wrote:
In ocxl_context_free() we note that the AFU reference we're releasing was
taken in "ocxl_context_init", a function that doesn't actually exist.
Fix it to say ocxl_context_alloc() instead, which I expect was what was
intended.