Support of DAMOS ops filter and sz_filter_passed support is up to each operation set. DAMON_VADDR supports both for only damos_stat action. It intentionally does not support those for madvise-based actions because applying filters could make it unnecessarily complicated and expensive. The vaddr implementation of migrate_{hot,cold} action does support DAMOS ops filters but sz_filter_passed. There is no real reason to not support sz_filter_passed. Add the support. Pass the sz_filter_passed pointer to the filter applying logic and accumulate the number. Closes: https://sashiko.dev/#/bug/linux-986e8b64-8e65-4e32-970f-4d3ebdf8206e Signed-off-by: SJ Park --- mm/damon/vaddr.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/mm/damon/vaddr.c b/mm/damon/vaddr.c index b1d7bcf7305bb..6fd0e07b60705 100644 --- a/mm/damon/vaddr.c +++ b/mm/damon/vaddr.c @@ -776,6 +776,7 @@ static bool damos_va_filter_out(struct damos *scheme, struct folio *folio, struct damos_va_migrate_private { struct list_head *migration_lists; struct damos *scheme; + unsigned long *sz_filter_passed; }; /* @@ -846,7 +847,7 @@ static int damos_va_migrate_pmd_entry(pmd_t *pmd, unsigned long addr, struct damos_va_migrate_private *priv = walk->private; struct list_head *migration_lists = priv->migration_lists; struct damos *s = priv->scheme; - struct damos_migrate_dests *dests = &s->migrate_dests; + unsigned long *sz_filter_passed = priv->sz_filter_passed; struct folio *folio; spinlock_t *ptl; pte_t *start_pte, *pte, ptent; @@ -865,6 +866,7 @@ static int damos_va_migrate_pmd_entry(pmd_t *pmd, unsigned long addr, if (damos_va_filter_out(s, folio, walk->vma, addr, NULL, pmd)) goto huge_out; s->last_applied = folio; + *sz_filter_passed += folio_size(folio); damos_va_migrate_dests_add(folio, walk->vma, addr, dests, migration_lists); huge_out: @@ -893,6 +895,7 @@ static int damos_va_migrate_pmd_entry(pmd_t *pmd, unsigned long addr, if (damos_va_filter_out(s, folio, walk->vma, addr, pte, NULL)) continue; s->last_applied = folio; + *sz_filter_passed += folio_size(folio); damos_va_migrate_dests_add(folio, walk->vma, addr, dests, migration_lists); } @@ -971,6 +974,7 @@ static unsigned long damos_va_migrate(struct damon_target *target, priv.migration_lists = kmalloc_objs(*priv.migration_lists, nr_dests); if (!priv.migration_lists) return 0; + priv.sz_filter_passed = sz_filter_passed; for (int i = 0; i < nr_dests; i++) INIT_LIST_HEAD(&priv.migration_lists[i]); -- 2.47.3