From: Matthew Wilcox (Oracle) Date: Fri, 4 Mar 2022 19:05:23 +0000 (-0500) Subject: iomap: Simplify is_partially_uptodate a little X-Git-Tag: v6.1-rc5~1637^2~9 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=2756c818e5cbcc4c9d13d7399273e86068e95156;p=platform%2Fkernel%2Flinux-starfive.git iomap: Simplify is_partially_uptodate a little Remove the unnecessary variable 'len' and fix a comment to refer to the folio instead of the page. Signed-off-by: Matthew Wilcox (Oracle) Reviewed-by: Christoph Hellwig Reviewed-by: Al Viro Acked-by: Al Viro --- diff --git a/fs/iomap/buffered-io.c b/fs/iomap/buffered-io.c index 49dccd9..8ce8720 100644 --- a/fs/iomap/buffered-io.c +++ b/fs/iomap/buffered-io.c @@ -435,18 +435,17 @@ bool iomap_is_partially_uptodate(struct folio *folio, size_t from, size_t count) { struct iomap_page *iop = to_iomap_page(folio); struct inode *inode = folio->mapping->host; - size_t len; unsigned first, last, i; if (!iop) return false; - /* Limit range to this folio */ - len = min(folio_size(folio) - from, count); + /* Caller's range may extend past the end of this folio */ + count = min(folio_size(folio) - from, count); - /* First and last blocks in range within page */ + /* First and last blocks in range within folio */ first = from >> inode->i_blkbits; - last = (from + len - 1) >> inode->i_blkbits; + last = (from + count - 1) >> inode->i_blkbits; for (i = first; i <= last; i++) if (!test_bit(i, iop->uptodate))