Skip to content

Commit 2408900

Browse files
Matthew Wilcox (Oracle)brauner
authored andcommitted
netfs: Use folio_next_pos()
This is one instruction more efficient than open-coding folio_pos() + folio_size(). It's the equivalent of (x + y) << z rather than x << z + y << z. Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org> Link: https://patch.msgid.link/20251024170822.1427218-9-willy@infradead.org Acked-by: David Howells <dhowells@redhat.com> Reviewed-by: Paulo Alcantara (Red Hat) <pc@manguebit.org> Cc: David Howells <dhowells@redhat.com> Cc: Paulo Alcantara <pc@manguebit.org> Cc: netfs@lists.linux.dev Signed-off-by: Christian Brauner <brauner@kernel.org>
1 parent ac97520 commit 2408900

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

fs/netfs/buffered_write.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -535,7 +535,7 @@ vm_fault_t netfs_page_mkwrite(struct vm_fault *vmf, struct netfs_group *netfs_gr
535535
folio_unlock(folio);
536536
err = filemap_fdatawrite_range(mapping,
537537
folio_pos(folio),
538-
folio_pos(folio) + folio_size(folio));
538+
folio_next_pos(folio));
539539
switch (err) {
540540
case 0:
541541
ret = VM_FAULT_RETRY;

fs/netfs/misc.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -298,7 +298,7 @@ bool netfs_release_folio(struct folio *folio, gfp_t gfp)
298298
if (folio_test_dirty(folio))
299299
return false;
300300

301-
end = umin(folio_pos(folio) + folio_size(folio), i_size_read(&ctx->inode));
301+
end = umin(folio_next_pos(folio), i_size_read(&ctx->inode));
302302
if (end > ctx->zero_point)
303303
ctx->zero_point = end;
304304

0 commit comments

Comments
 (0)