| Message ID | 20221228113413.10329-1-wangkefeng.wang@huawei.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | mm: convert page_idle/damon to use folios | expand |
Hi Kefeng, On Wed, 28 Dec 2022 19:34:06 +0800 Kefeng Wang <wangkefeng.wang@huawei.com> wrote: > v3: > - more thoroughly converting in page_idle_get_folio() suggested by Matthew > also do it in damon_get_folio(). > - address some commets from SeongJae > > v2: > - drop unsafe pfn_to_online_folio(), convert page_idle_get_page() and > damon_get_page() to return a folio after grab a reference > - convert damon hugetlb related functions > - rebased on next-20221226. > > Kefeng Wang (7): > mm: page_idle: Convert page idle to use folios > mm/damon: introduce damon_get_folio() > mm/damon: convert damon_ptep/pmdp_mkold() to use folios > mm/damon: paddr: convert damon_pa_*() to use folios > mm/damon: vaddr: convert damon_young_pmd_entry() to use folio > mm/damon: remove unneed damon_get_page() > mm/damon: vaddr: convert hugetlb related function to use folios Thank you so much for your efforts on this work. I found I forgot mentioning some trivial typos in the first, the sixth and the seventh patches, so commented. I should have commented those earlier, sorry. Other than those, Reviewed-by: SeongJae Park <sj@kernel.org> Thanks, SJ > > mm/damon/ops-common.c | 38 +++++++++++++++------------- > mm/damon/ops-common.h | 2 +- > mm/damon/paddr.c | 58 +++++++++++++++++++------------------------ > mm/damon/vaddr.c | 38 ++++++++++++++-------------- > mm/page_idle.c | 47 ++++++++++++++++++----------------- > 5 files changed, 91 insertions(+), 92 deletions(-) > > -- > 2.35.3 >
On 2022/12/29 1:55, SeongJae Park wrote: > Hi Kefeng, > > On Wed, 28 Dec 2022 19:34:06 +0800 Kefeng Wang <wangkefeng.wang@huawei.com> wrote: > >> v3: >> - more thoroughly converting in page_idle_get_folio() suggested by Matthew >> also do it in damon_get_folio(). >> - address some commets from SeongJae >> >> v2: >> - drop unsafe pfn_to_online_folio(), convert page_idle_get_page() and >> damon_get_page() to return a folio after grab a reference >> - convert damon hugetlb related functions >> - rebased on next-20221226. >> >> Kefeng Wang (7): >> mm: page_idle: Convert page idle to use folios >> mm/damon: introduce damon_get_folio() >> mm/damon: convert damon_ptep/pmdp_mkold() to use folios >> mm/damon: paddr: convert damon_pa_*() to use folios >> mm/damon: vaddr: convert damon_young_pmd_entry() to use folio >> mm/damon: remove unneed damon_get_page() >> mm/damon: vaddr: convert hugetlb related function to use folios > > Thank you so much for your efforts on this work. I found I forgot mentioning > some trivial typos in the first, the sixth and the seventh patches, so > commented. I should have commented those earlier, sorry. > Oh，please forgive my bad English spelling, and thanks for Andrew with corrected word when patches are queued. > Other than those, > > Reviewed-by: SeongJae Park <sj@kernel.org> Thanks you very much. > > > Thanks, > SJ > >> >> mm/damon/ops-common.c | 38 +++++++++++++++------------- >> mm/damon/ops-common.h | 2 +- >> mm/damon/paddr.c | 58 +++++++++++++++++++------------------------ >> mm/damon/vaddr.c | 38 ++++++++++++++-------------- >> mm/page_idle.c | 47 ++++++++++++++++++----------------- >> 5 files changed, 91 insertions(+), 92 deletions(-) >> >> -- >> 2.35.3 >>