diff mbox series

[4/5] dm vdo chapter_index: fix a few small nits

Message ID 70cbd54d6bdd58a734b9765a2f2ed8bd11e1ada3.1707886374.git.msakai@redhat.com (mailing list archive)
State Accepted, archived
Delegated to: Mike Snitzer
Headers show
Series dm vdo: fix various small nits | expand

Commit Message

Matthew Sakai Feb. 14, 2024, 4:57 a.m. UTC 
From: Mike Snitzer <snitzer@kernel.org>

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>
---
 drivers/md/dm-vdo/chapter-index.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)
diff mbox series

Patch

diff --git a/drivers/md/dm-vdo/chapter-index.c b/drivers/md/dm-vdo/chapter-index.c
index 7285a842e888..363991d56218 100644
--- 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;