diff options
| author | Mike Snitzer <snitzer@kernel.org> | 2023-11-30 19:54:56 -0500 |
|---|---|---|
| committer | Mike Snitzer <snitzer@kernel.org> | 2024-02-20 13:43:17 -0500 |
| commit | 4c79d55678b8bf993cb94c6181180997112bb353 (patch) | |
| tree | 9c81e56bd40f26c09e362d3cfcde7e458d26d31d /drivers/md | |
| parent | 9165dac82273579b150bb56f76c6cf3222eb1a58 (diff) | |
| download | linux-4c79d55678b8bf993cb94c6181180997112bb353.tar.gz | |
dm vdo: fix how dm_kcopyd_client_create() failure is checked
dm_kcopyd_client_create() returns an ERR_PTR so its return must be
checked with IS_ERR().
Signed-off-by: Mike Snitzer <snitzer@kernel.org>
Signed-off-by: Chung Chung <cchung@redhat.com>
Signed-off-by: Matthew Sakai <msakai@redhat.com>
Diffstat (limited to 'drivers/md')
| -rw-r--r-- | drivers/md/dm-vdo-target.c | 8 | ||||
| -rw-r--r-- | drivers/md/dm-vdo/slab-depot.c | 7 |
2 files changed, 11 insertions, 4 deletions
diff --git a/drivers/md/dm-vdo-target.c b/drivers/md/dm-vdo-target.c index 72d2291dd93f26..4fbc148681e517 100644 --- a/drivers/md/dm-vdo-target.c +++ b/drivers/md/dm-vdo-target.c @@ -8,6 +8,7 @@ #include <linux/completion.h> #include <linux/delay.h> #include <linux/device-mapper.h> +#include <linux/err.h> #include <linux/module.h> #include <linux/mutex.h> #include <linux/spinlock.h> @@ -1683,8 +1684,11 @@ static int grow_layout(struct vdo *vdo, block_count_t old_size, block_count_t ne /* Make a copy completion if there isn't one */ if (vdo->partition_copier == NULL) { vdo->partition_copier = dm_kcopyd_client_create(NULL); - if (vdo->partition_copier == NULL) - return -ENOMEM; + if (IS_ERR(vdo->partition_copier)) { + result = PTR_ERR(vdo->partition_copier); + vdo->partition_copier = NULL; + return result; + } } /* Free any unused preparation. */ diff --git a/drivers/md/dm-vdo/slab-depot.c b/drivers/md/dm-vdo/slab-depot.c index 2125e256aa86ce..56d975c987526b 100644 --- a/drivers/md/dm-vdo/slab-depot.c +++ b/drivers/md/dm-vdo/slab-depot.c @@ -7,6 +7,7 @@ #include <linux/atomic.h> #include <linux/bio.h> +#include <linux/err.h> #include <linux/log2.h> #include <linux/min_heap.h> #include <linux/minmax.h> @@ -3445,8 +3446,10 @@ static void initiate_load(struct admin_state *state) handle_operation_error, allocator->thread_id, NULL); allocator->eraser = dm_kcopyd_client_create(NULL); - if (allocator->eraser == NULL) { - vdo_fail_completion(&allocator->completion, -ENOMEM); + if (IS_ERR(allocator->eraser)) { + vdo_fail_completion(&allocator->completion, + PTR_ERR(allocator->eraser)); + allocator->eraser = NULL; return; } allocator->slabs_to_erase = get_slab_iterator(allocator); |