Commit 70e314c9 authored by Kefeng Wang's avatar Kefeng Wang Committed by Andrew Morton
Browse files

mm/damon: convert damon_ptep/pmdp_mkold() to use a folio 

With damon_get_folio(), let's convert damon_ptep_mkold() and
damon_pmdp_mkold() to use a folio.

Link: https://lkml.kernel.org/r/20221230070849.63358-5-wangkefeng.wang@huawei.com


Signed-off-by: default avatarKefeng Wang <wangkefeng.wang@huawei.com>
Reviewed-by: default avatarSeongJae Park <sj@kernel.org>
Cc: David Hildenbrand <david@redhat.com>
Cc: Matthew Wilcox (Oracle) <willy@infradead.org>
Cc: Vishal Moola (Oracle) <vishal.moola@gmail.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent 5e012bba

mm/damon/ops-common.c

+10 −10
Original line number Diff line number Diff line
@@ -40,9 +40,9 @@ struct folio *damon_get_folio(unsigned long pfn) 
void damon_ptep_mkold(pte_t *pte, struct mm_struct *mm, unsigned long addr)

{

	bool referenced = false;

	struct page *page = damon_get_page(pte_pfn(*pte));

	struct folio *folio = damon_get_folio(pte_pfn(*pte));



	if (!page)

	if (!folio)

		return;



	if (pte_young(*pte)) {
@@ -56,19 +56,19 @@ void damon_ptep_mkold(pte_t *pte, struct mm_struct *mm, unsigned long addr) 
#endif /* CONFIG_MMU_NOTIFIER */



	if (referenced)

		set_page_young(page);

		folio_set_young(folio);



	set_page_idle(page);

	put_page(page);

	folio_set_idle(folio);

	folio_put(folio);

}



void damon_pmdp_mkold(pmd_t *pmd, struct mm_struct *mm, unsigned long addr)

{

#ifdef CONFIG_TRANSPARENT_HUGEPAGE

	bool referenced = false;

	struct page *page = damon_get_page(pmd_pfn(*pmd));

	struct folio *folio = damon_get_folio(pmd_pfn(*pmd));



	if (!page)

	if (!folio)

		return;



	if (pmd_young(*pmd)) {
@@ -82,10 +82,10 @@ void damon_pmdp_mkold(pmd_t *pmd, struct mm_struct *mm, unsigned long addr) 
#endif /* CONFIG_MMU_NOTIFIER */



	if (referenced)

		set_page_young(page);

		folio_set_young(folio);



	set_page_idle(page);

	put_page(page);

	folio_set_idle(folio);

	folio_put(folio);

#endif /* CONFIG_TRANSPARENT_HUGEPAGE */

}