Skip to content

Commit

Permalink
LU-12477 lustre: use delete_from_page_cache() for directory pages.
Browse files Browse the repository at this point in the history
lustre sometimes uses the internal function truncate_complete_page()
to remove a page of a directory.
Much of what this function does, does not apply to directory pages
as there is no invalidatepage function, and at these times, the
page is not dirty.
The only useful part of the function is delete_from_page_cache(),
so just call that directly.

Linux-commit: d17fa2f3a0b9b40be48e0c3cc88eb3b3cea1b701

Change-Id: I54795e71e107c50f662bd2015c6f621bfe436e0a
Acked-by: Oleg Drokin <[email protected]>
Signed-off-by: NeilBrown <[email protected]>
Reviewed-on: https://review.whamcloud.com/35462
Tested-by: jenkins <[email protected]>
Reviewed-by: Lai Siyao <[email protected]>
Tested-by: Maloo <[email protected]>
Reviewed-by: Oleg Drokin <[email protected]>
  • Loading branch information
NeilBrown authored and Oleg Drokin committed Jul 17, 2019
1 parent f114d4d commit 43d5806
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion lustre/llite/dir.c
Original file line number Diff line number Diff line change
Expand Up @@ -171,7 +171,7 @@ void ll_release_page(struct inode *inode, struct page *page,
if (remove) {
lock_page(page);
if (likely(page->mapping != NULL))
truncate_complete_page(page->mapping, page);
delete_from_page_cache(page);
unlock_page(page);
}
put_page(page);
Expand Down
2 changes: 1 addition & 1 deletion lustre/mdc/mdc_request.c
Original file line number Diff line number Diff line change
Expand Up @@ -1116,7 +1116,7 @@ static void mdc_release_page(struct page *page, int remove)
if (remove) {
lock_page(page);
if (likely(page->mapping != NULL))
truncate_complete_page(page->mapping, page);
delete_from_page_cache(page);
unlock_page(page);
}
put_page(page);
Expand Down

0 comments on commit 43d5806

Please sign in to comment.