| From 36b6144df6b987ac86c61468e63ac88bea66d7e9 Mon Sep 17 00:00:00 2001 |
| From: Sasha Levin <sashal@kernel.org> |
| Date: Wed, 9 Jun 2021 21:43:52 +0300 |
| Subject: habanalabs: remove node from list before freeing the node |
| |
| From: Koby Elbaz <kelbaz@habana.ai> |
| |
| [ Upstream commit f5eb7bf0c487a212ebda3c1b048fc3ccabacc147 ] |
| |
| fix the following smatch warnings: |
| |
| goya_pin_memory_before_cs() |
| warn: '&userptr->job_node' not removed from list |
| |
| gaudi_pin_memory_before_cs() |
| warn: '&userptr->job_node' not removed from list |
| |
| Signed-off-by: Koby Elbaz <kelbaz@habana.ai> |
| Reviewed-by: Oded Gabbay <ogabbay@kernel.org> |
| Signed-off-by: Oded Gabbay <ogabbay@kernel.org> |
| Signed-off-by: Sasha Levin <sashal@kernel.org> |
| --- |
| drivers/misc/habanalabs/gaudi/gaudi.c | 1 + |
| drivers/misc/habanalabs/goya/goya.c | 1 + |
| 2 files changed, 2 insertions(+) |
| |
| diff --git a/drivers/misc/habanalabs/gaudi/gaudi.c b/drivers/misc/habanalabs/gaudi/gaudi.c |
| index a03f13aa47f8..0c6092ebbc04 100644 |
| --- a/drivers/misc/habanalabs/gaudi/gaudi.c |
| +++ b/drivers/misc/habanalabs/gaudi/gaudi.c |
| @@ -4901,6 +4901,7 @@ already_pinned: |
| return 0; |
| |
| unpin_memory: |
| + list_del(&userptr->job_node); |
| hl_unpin_host_memory(hdev, userptr); |
| free_userptr: |
| kfree(userptr); |
| diff --git a/drivers/misc/habanalabs/goya/goya.c b/drivers/misc/habanalabs/goya/goya.c |
| index ed566c52ccaa..45c9065c4b92 100644 |
| --- a/drivers/misc/habanalabs/goya/goya.c |
| +++ b/drivers/misc/habanalabs/goya/goya.c |
| @@ -3249,6 +3249,7 @@ already_pinned: |
| return 0; |
| |
| unpin_memory: |
| + list_del(&userptr->job_node); |
| hl_unpin_host_memory(hdev, userptr); |
| free_userptr: |
| kfree(userptr); |
| -- |
| 2.30.2 |
| |