dm: avoid useless 'else' after 'break' or return'
authorHeinz Mauelshagen <heinzm@redhat.com>
Tue, 7 Feb 2023 19:35:25 +0000 (20:35 +0100)
committerMike Snitzer <snitzer@kernel.org>
Tue, 14 Feb 2023 19:23:07 +0000 (14:23 -0500)
Signed-off-by: Heinz Mauelshagen <heinzm@redhat.com>
Signed-off-by: Mike Snitzer <snitzer@kernel.org>
drivers/md/dm-stripe.c
drivers/md/dm-thin-metadata.c
drivers/md/dm-verity-target.c
drivers/md/dm-writecache.c
drivers/md/persistent-data/dm-btree.c
drivers/md/persistent-data/dm-space-map-common.c

index e0e665efb2854d8d1f8c30180e8559a100359dc7..4335828a43e424c86c7c50f356b313a05eb01d8d 100644 (file)
@@ -259,11 +259,11 @@ static int stripe_map_range(struct stripe_c *sc, struct bio *bio,
                        sc->stripe[target_stripe].physical_start;
                bio->bi_iter.bi_size = to_bytes(end - begin);
                return DM_MAPIO_REMAPPED;
-       } else {
-               /* The range doesn't map to the target stripe */
-               bio_endio(bio);
-               return DM_MAPIO_SUBMITTED;
        }
+
+       /* The range doesn't map to the target stripe */
+       bio_endio(bio);
+       return DM_MAPIO_SUBMITTED;
 }
 
 static int stripe_map(struct dm_target *ti, struct bio *bio)
index 2b12e0e2c87915a85b7427d27958f1834a53d379..fd464fb024c3c4ffa0e67c93331b13a9d909bdd8 100644 (file)
@@ -1612,8 +1612,8 @@ static int __find_mapped_range(struct dm_thin_device *td,
                if (r) {
                        if (r == -ENODATA)
                                break;
-                       else
-                               return r;
+
+                       return r;
                }
 
                if ((lookup.block != pool_end) ||
index 0f9814099d1bf8cc4b2d55768efc10e830ebe364..ade83ef3b4392406d26d05da4cb2d0b1e890bc07 100644 (file)
@@ -110,23 +110,24 @@ static int verity_hash_update(struct dm_verity *v, struct ahash_request *req,
                sg_init_one(&sg, data, len);
                ahash_request_set_crypt(req, &sg, NULL, len);
                return crypto_wait_req(crypto_ahash_update(req), wait);
-       } else {
-               do {
-                       int r;
-                       size_t this_step = min_t(size_t, len, PAGE_SIZE - offset_in_page(data));
-
-                       flush_kernel_vmap_range((void *)data, this_step);
-                       sg_init_table(&sg, 1);
-                       sg_set_page(&sg, vmalloc_to_page(data), this_step, offset_in_page(data));
-                       ahash_request_set_crypt(req, &sg, NULL, this_step);
-                       r = crypto_wait_req(crypto_ahash_update(req), wait);
-                       if (unlikely(r))
-                               return r;
-                       data += this_step;
-                       len -= this_step;
-               } while (len);
-               return 0;
        }
+
+       do {
+               int r;
+               size_t this_step = min_t(size_t, len, PAGE_SIZE - offset_in_page(data));
+
+               flush_kernel_vmap_range((void *)data, this_step);
+               sg_init_table(&sg, 1);
+               sg_set_page(&sg, vmalloc_to_page(data), this_step, offset_in_page(data));
+               ahash_request_set_crypt(req, &sg, NULL, this_step);
+               r = crypto_wait_req(crypto_ahash_update(req), wait);
+               if (unlikely(r))
+                       return r;
+               data += this_step;
+               len -= this_step;
+       } while (len);
+
+       return 0;
 }
 
 /*
index 2dfa612fa1efbb5c297f43d016ba208c4eb911df..70a110a41c44b9fb9c94259400c7587320dc2aff 100644 (file)
@@ -623,15 +623,15 @@ static struct wc_entry *writecache_find_entry(struct dm_writecache *wc,
                if (unlikely(!node)) {
                        if (!(flags & WFE_RETURN_FOLLOWING))
                                return NULL;
-                       if (read_original_sector(wc, e) >= block) {
+                       if (read_original_sector(wc, e) >= block)
                                return e;
-                       } else {
-                               node = rb_next(&e->rb_node);
-                               if (unlikely(!node))
-                                       return NULL;
-                               e = container_of(node, struct wc_entry, rb_node);
-                               return e;
-                       }
+
+                       node = rb_next(&e->rb_node);
+                       if (unlikely(!node))
+                               return NULL;
+
+                       e = container_of(node, struct wc_entry, rb_node);
+                       return e;
                }
        }
 
index f4c63060ea27001f8692975ef35d833795db360e..bb8b34ffeafc95df2292ee169fc4333f8e106fda 100644 (file)
@@ -1352,8 +1352,8 @@ static int find_key(struct ro_spine *s, dm_block_t block, bool find_highest,
                i = le32_to_cpu(ro_node(s)->header.nr_entries);
                if (!i)
                        return -ENODATA;
-               else
-                       i--;
+
+               i--;
 
                if (find_highest)
                        *result_key = le64_to_cpu(ro_node(s)->keys[i]);
index e927ed85321560e9d13c09936bf339479a99801e..591d1a43d0357216874d190334ac5eaeb5fffbe1 100644 (file)
@@ -793,13 +793,12 @@ static int __sm_ll_dec_overflow(struct ll_disk *ll, dm_block_t b,
        rc = le32_to_cpu(*v_ptr);
        *old_rc = rc;
 
-       if (rc == 3) {
+       if (rc == 3)
                return __sm_ll_del_overflow(ll, b, ic);
-       } else {
-               rc--;
-               *v_ptr = cpu_to_le32(rc);
-               return 0;
-       }
+
+       rc--;
+       *v_ptr = cpu_to_le32(rc);
+       return 0;
 }
 
 static int sm_ll_dec_overflow(struct ll_disk *ll, dm_block_t b,