Messages in this thread | | | Date | Fri, 8 Dec 2023 09:42:53 +0800 | Subject | Re: [PATCH RFC 12/39] mm/rmap: remove page_add_file_rmap() | From | "Yin, Fengwei" <> |
| |
On 12/4/2023 10:21 PM, David Hildenbrand wrote: > All users are gone, let's remove it. > > Signed-off-by: David Hildenbrand <david@redhat.com> Reviewed-by: Yin Fengwei <fengwei.yin@intel.com>
> --- > include/linux/rmap.h | 2 -- > mm/rmap.c | 21 --------------------- > 2 files changed, 23 deletions(-) > > diff --git a/include/linux/rmap.h b/include/linux/rmap.h > index a4a30c361ac50..95f7b94a70295 100644 > --- a/include/linux/rmap.h > +++ b/include/linux/rmap.h > @@ -235,8 +235,6 @@ void page_add_new_anon_rmap(struct page *, struct vm_area_struct *, > unsigned long address); > void folio_add_new_anon_rmap(struct folio *, struct vm_area_struct *, > unsigned long address); > -void page_add_file_rmap(struct page *, struct vm_area_struct *, > - bool compound); > void folio_add_file_rmap_ptes(struct folio *, struct page *, unsigned int nr, > struct vm_area_struct *); > #define folio_add_file_rmap_pte(folio, page, vma) \ > diff --git a/mm/rmap.c b/mm/rmap.c > index 1614d98062948..53e2c653be99a 100644 > --- a/mm/rmap.c > +++ b/mm/rmap.c > @@ -1422,27 +1422,6 @@ void folio_add_file_rmap_pmd(struct folio *folio, struct page *page, > #endif > } > > -/** > - * page_add_file_rmap - add pte mapping to a file page > - * @page: the page to add the mapping to > - * @vma: the vm area in which the mapping is added > - * @compound: charge the page as compound or small page > - * > - * The caller needs to hold the pte lock. > - */ > -void page_add_file_rmap(struct page *page, struct vm_area_struct *vma, > - bool compound) > -{ > - struct folio *folio = page_folio(page); > - > - VM_WARN_ON_ONCE_PAGE(compound && !PageTransHuge(page), page); > - > - if (likely(!compound)) > - folio_add_file_rmap_pte(folio, page, vma); > - else > - folio_add_file_rmap_pmd(folio, page, vma); > -} > - > /** > * page_remove_rmap - take down pte mapping from a page > * @page: page to remove mapping from
| |