mm/compaction: remove unused parameter pgdata of fragmentation_score_wmark
authorKemeng Shi <shikemeng@huaweicloud.com>
Wed, 9 Aug 2023 09:49:10 +0000 (17:49 +0800)
committerAndrew Morton <akpm@linux-foundation.org>
Mon, 21 Aug 2023 20:37:50 +0000 (13:37 -0700)
Parameter pgdat is not used in fragmentation_score_wmark. Just remove it.

Link: https://lkml.kernel.org/r/20230809094910.3092446-1-shikemeng@huaweicloud.com
Signed-off-by: Kemeng Shi <shikemeng@huaweicloud.com>
Reviewed-by: David Hildenbrand <david@redhat.com>
Acked-by: Mel Gorman <mgorman@techsingularity.net>
Reviewed-by: Baolin Wang <baolin.wang@linux.alibaba.com>
Cc: Matthew Wilcox <willy@infradead.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
mm/compaction.c

index fe7b4e7c5d2401aecbefd5290492c4fba0d1e2f3..216081ab325aaa6f48bd718fbbdb4c7a7b3688b5 100644 (file)
@@ -2140,7 +2140,7 @@ static unsigned int fragmentation_score_node(pg_data_t *pgdat)
        return score;
 }
 
-static unsigned int fragmentation_score_wmark(pg_data_t *pgdat, bool low)
+static unsigned int fragmentation_score_wmark(bool low)
 {
        unsigned int wmark_low;
 
@@ -2160,7 +2160,7 @@ static bool should_proactive_compact_node(pg_data_t *pgdat)
        if (!sysctl_compaction_proactiveness || kswapd_is_running(pgdat))
                return false;
 
-       wmark_high = fragmentation_score_wmark(pgdat, false);
+       wmark_high = fragmentation_score_wmark(false);
        return fragmentation_score_node(pgdat) > wmark_high;
 }
 
@@ -2199,7 +2199,7 @@ static enum compact_result __compact_finished(struct compact_control *cc)
                        return COMPACT_PARTIAL_SKIPPED;
 
                score = fragmentation_score_zone(cc->zone);
-               wmark_low = fragmentation_score_wmark(pgdat, true);
+               wmark_low = fragmentation_score_wmark(true);
 
                if (score > wmark_low)
                        ret = COMPACT_CONTINUE;