dm vdo chapter_index: fix a few small nits

Add missing braces and raise one function arg up a line to eliminate
line wrap.

Signed-off-by: Mike Snitzer <snitzer@kernel.org>
Signed-off-by: Susan LeGendre-McGhee <slegendr@redhat.com>
Signed-off-by: Matthew Sakai <msakai@redhat.com>

diff --git a/drivers/md/dm-vdo/chapter-index.c b/drivers/md/dm-vdo/chapter-index.c
index 7285a842e8888227491df77bb008c912f2378a6b..363991d56218325487d8db6a84344d92ca40e237 100644 (file)
--- a/drivers/md/dm-vdo/chapter-index.c
+++ b/drivers/md/dm-vdo/chapter-index.c
@@ -19,8 +19,7 @@ int uds_make_open_chapter_index(struct open_chapter_index **chapter_index,
        size_t memory_size;
        struct open_chapter_index *index;
 
-       result = uds_allocate(1, struct open_chapter_index, "open chapter index",
-                             &index);
+       result = uds_allocate(1, struct open_chapter_index, "open chapter index", &index);
        if (result != UDS_SUCCESS)
                return result;
 
@@ -197,9 +196,10 @@ int uds_pack_open_chapter_index_page(struct open_chapter_index *chapter_index,
                } while (!entry.at_end);
        }
 
-       if (removals > 0)
+       if (removals > 0) {
                uds_log_warning("To avoid chapter index page overflow in chapter %llu, %u entries were removed from the chapter index",
                                (unsigned long long) chapter_number, removals);
+       }
 
        return UDS_SUCCESS;
 }
@@ -249,13 +249,14 @@ int uds_validate_chapter_index_page(const struct delta_index_page *index_page,
 
                        /* Also make sure that the record page field contains a plausible value. */
                        if (uds_get_delta_entry_value(&entry) >=
-                           geometry->record_pages_per_chapter)
+                           geometry->record_pages_per_chapter) {
                                /*
                                 * Do not log this as an error. It happens in normal operation when
                                 * we are doing a rebuild but haven't written the entire volume
                                 * once.
                                 */
                                return UDS_CORRUPT_DATA;
+                       }
                }
        }
        return UDS_SUCCESS;