Re: [GIT PULL] Rename page_offset() to page_pos()
From: Linus Torvalds
Date: Sat Apr 11 2020 - 18:03:13 EST
On Sat, Apr 11, 2020 at 2:48 PM Matthew Wilcox <willy@xxxxxxxxxxxxx> wrote:
>
> I wasn't entirely forthcoming ... I actually want to introduce a new
>
> #define page_offset(page, x) ((unsigned long)(x) & (page_size(page) - 1))
No, no, no.
THAT would be confusing. Re-using a name (even if you renamed it) for
something new and completely different is just bad taste. It would
also be a horrible problem - again - for any stable backport etc.
Just call that "offset_in_page()" and be done with it.
Linus