On Mon, Feb 14, 2022 at 08:00:09PM +0000, Matthew Wilcox (Oracle) wrote: > invalidate_inode_page() is the only caller of invalidate_complete_page() > and inlining it reveals that the first check is unnecessary (because we > hold the page locked, and we just retrieved the mapping from the page). > Actually, it does make a difference, in that tail pages no longer fail > at this check, so it's now possible to remove a tail page from a mapping. There is a comment that referneces invalidate_complete_page in kernel/futex/core.c which needs updating. Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>
