]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
mm/damon/paddr: support only damos->ops_filters
authorSeongJae Park <sj@kernel.org>
Tue, 4 Mar 2025 21:19:09 +0000 (13:19 -0800)
committerAndrew Morton <akpm@linux-foundation.org>
Mon, 17 Mar 2025 07:05:38 +0000 (00:05 -0700)
DAMON physical address space operation set implementation (paddr) started
handling both damos->filters and damos->ops_filters to avoid breakage
during the change for the ->ops_filters setup.  Now the change is done, so
paddr's support of ->filters is only a waste that can safely be dropped.
Remove it.

Link: https://lkml.kernel.org/r/20250304211913.53574-6-sj@kernel.org
Signed-off-by: SeongJae Park <sj@kernel.org>
Cc: Jonathan Corbet <corbet@lwn.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
mm/damon/paddr.c

index 2b1ea568a4311c43d61aba29d3cc7445bf3ca540..dded659bb110989ae5d4e3209f0af664ff7ef0dd 100644 (file)
@@ -256,10 +256,6 @@ static bool damos_pa_filter_out(struct damos *scheme, struct folio *folio)
        if (scheme->core_filters_allowed)
                return false;
 
-       damos_for_each_filter(filter, scheme) {
-               if (damos_pa_filter_match(filter, folio))
-                       return !filter->allow;
-       }
        damos_for_each_ops_filter(filter, scheme) {
                if (damos_pa_filter_match(filter, folio))
                        return !filter->allow;
@@ -288,12 +284,6 @@ static unsigned long damon_pa_pageout(struct damon_region *r, struct damos *s,
        struct folio *folio;
 
        /* check access in page level again by default */
-       damos_for_each_filter(filter, s) {
-               if (filter->type == DAMOS_FILTER_TYPE_YOUNG) {
-                       install_young_filter = false;
-                       break;
-               }
-       }
        damos_for_each_ops_filter(filter, s) {
                if (filter->type == DAMOS_FILTER_TYPE_YOUNG) {
                        install_young_filter = false;
@@ -546,8 +536,6 @@ static bool damon_pa_scheme_has_filter(struct damos *s)
 {
        struct damos_filter *f;
 
-       damos_for_each_filter(f, s)
-               return true;
        damos_for_each_ops_filter(f, s)
                return true;
        return false;