Skip to content

Commit

Permalink
mm/truncate: Convert __invalidate_mapping_pages() to use a folio
Browse files Browse the repository at this point in the history
Now we can call mapping_evict_folio() instead of invalidate_inode_page()
and save a few calls to compound_head().

Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Miaohe Lin <linmiaohe@huawei.com>
  • Loading branch information
Matthew Wilcox (Oracle) committed Mar 21, 2022
1 parent d6c75dc commit b4545f4
Showing 1 changed file with 9 additions and 9 deletions.
18 changes: 9 additions & 9 deletions mm/truncate.c
Original file line number Diff line number Diff line change
Expand Up @@ -507,27 +507,27 @@ static unsigned long __invalidate_mapping_pages(struct address_space *mapping,
folio_batch_init(&fbatch);
while (find_lock_entries(mapping, index, end, &fbatch, indices)) {
for (i = 0; i < folio_batch_count(&fbatch); i++) {
struct page *page = &fbatch.folios[i]->page;
struct folio *folio = fbatch.folios[i];

/* We rely upon deletion not changing page->index */
/* We rely upon deletion not changing folio->index */
index = indices[i];

if (xa_is_value(page)) {
if (xa_is_value(folio)) {
count += invalidate_exceptional_entry(mapping,
index,
page);
folio);
continue;
}
index += thp_nr_pages(page) - 1;
index += folio_nr_pages(folio) - 1;

ret = invalidate_inode_page(page);
unlock_page(page);
ret = mapping_evict_folio(mapping, folio);
folio_unlock(folio);
/*
* Invalidation is a hint that the page is no longer
* Invalidation is a hint that the folio is no longer
* of interest and try to speed up its reclaim.
*/
if (!ret) {
deactivate_file_page(page);
deactivate_file_page(&folio->page);
/* It is likely on the pagevec of a remote CPU */
if (nr_pagevec)
(*nr_pagevec)++;
Expand Down

0 comments on commit b4545f4

Please sign in to comment.