diff options
author | Matthew Wilcox (Oracle) <willy@infradead.org> | 2020-04-11 14:44:43 -0400 |
---|---|---|
committer | Matthew Wilcox (Oracle) <willy@infradead.org> | 2020-04-11 16:17:34 -0400 |
commit | ac640390ff8147c26821ea243d9be7388b18032a (patch) | |
tree | f1072f52b7a081ef56226f588223ed51f073a78d /fs/9p/acl.c | |
parent | b032227c62939b5481bcd45442b36dfa263f4a7c (diff) |
fs: Rename page_offset() to page_pos()
page_offset() is at best an ambiguous name for this function (returning
the byte offset in the file of this page). At least one regular
contributor thought it returned the pgoff_t of the page. We tend to use
a variable named page_offset to mean the byte offset from the start of
the page, which is also not what page_offset() does.
Fortunately, we have an unambiguous name for the offset in bytes from
the beginning -- pos. The most common name for an loff_t is 'pos'
(by 773 to 575 for 'offset'), so page_pos() seems like the best name
for this function.
Conversion performed automatically by Coccinelle with this script:
@@ expression a; @@
- page_offset(a)
+ page_pos(a)
Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Acked-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: William Kucharski <william.kucharski@oracle.com>
Diffstat (limited to 'fs/9p/acl.c')
0 files changed, 0 insertions, 0 deletions