diff options
| author | Mike Snitzer <snitzer@kernel.org> | 2024-02-13 23:57:11 -0500 |
|---|---|---|
| committer | Mike Snitzer <snitzer@kernel.org> | 2024-02-20 13:43:19 -0500 |
| commit | dea93aab18341d6d96a49e840e5605cebca7e8ea (patch) | |
| tree | ee84b65bb2e276e3fa5feb58d1088908ed412837 /drivers/md | |
| parent | 571eff3969a58cce63b02acda4462aa0ca12e116 (diff) | |
| download | linux-dea93aab18341d6d96a49e840e5605cebca7e8ea.tar.gz | |
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>
Diffstat (limited to 'drivers/md')
| -rw-r--r-- | drivers/md/dm-vdo/chapter-index.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/drivers/md/dm-vdo/chapter-index.c b/drivers/md/dm-vdo/chapter-index.c index 7285a842e8888..363991d562183 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; |