diff options
| author | Christoph Hellwig <hch@lst.de> | 2025-10-17 05:45:47 +0200 |
|---|---|---|
| committer | Christian Brauner <brauner@kernel.org> | 2025-10-29 15:54:31 +0100 |
| commit | 151d0922bf638a4e4235758d04b31f48bfcbb798 (patch) | |
| tree | 741470dc87c9cc2512cce8c6aaeba2e58730ec82 | |
| parent | 211c43d0938dd57c2937366339f8b651e7990d1a (diff) | |
writeback: cleanup writeback_chunk_size
Return the pages directly when calculated instead of first assigning
them back to a variable, and directly return for the data integrity /
tagged case instead of going through an else clause.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Link: https://patch.msgid.link/20251017034611.651385-2-hch@lst.de
Reviewed-by: Damien Le Moal <dlemoal@kernel.org>
Reviewed-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Nirjhar Roy (IBM) <nirjhar.roy.lists@gmail.com>
Reviewed-by: Jan Kara <jack@suse.cz>
Signed-off-by: Christian Brauner <brauner@kernel.org>
| -rw-r--r-- | fs/fs-writeback.c | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/fs/fs-writeback.c b/fs/fs-writeback.c index 4448de35ec8b..30de37865fa1 100644 --- a/fs/fs-writeback.c +++ b/fs/fs-writeback.c @@ -1908,16 +1908,12 @@ static long writeback_chunk_size(struct bdi_writeback *wb, * (maybe slowly) sync all tagged pages */ if (work->sync_mode == WB_SYNC_ALL || work->tagged_writepages) - pages = LONG_MAX; - else { - pages = min(wb->avg_write_bandwidth / 2, - global_wb_domain.dirty_limit / DIRTY_SCOPE); - pages = min(pages, work->nr_pages); - pages = round_down(pages + MIN_WRITEBACK_PAGES, - MIN_WRITEBACK_PAGES); - } + return LONG_MAX; - return pages; + pages = min(wb->avg_write_bandwidth / 2, + global_wb_domain.dirty_limit / DIRTY_SCOPE); + pages = min(pages, work->nr_pages); + return round_down(pages + MIN_WRITEBACK_PAGES, MIN_WRITEBACK_PAGES); } /* |