Commit 13058705 authored by Kemeng Shi's avatar Kemeng Shi Committed by Andrew Morton
Browse files

mm/page_alloc: remove unnecessary parameter batch of nr_pcp_free 

We get batch from pcp and just pass it to nr_pcp_free immediately.  Get
batch from pcp inside nr_pcp_free to remove unnecessary parameter batch.

Link: https://lkml.kernel.org/r/20230809100754.3094517-3-shikemeng@huaweicloud.com


Signed-off-by: default avatarKemeng Shi <shikemeng@huaweicloud.com>
Cc: Baolin Wang <baolin.wang@linux.alibaba.com>
Cc: David Hildenbrand <david@redhat.com>
Cc: Matthew Wilcox <willy@infradead.org>
Cc: Mel Gorman <mgorman@techsingularity.net>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent f142b2c2

mm/page_alloc.c

+3 −5
Original line number Diff line number Diff line
@@ -2340,10 +2340,10 @@ static bool free_unref_page_prepare(struct page *page, unsigned long pfn, 
	return true;

}



static int nr_pcp_free(struct per_cpu_pages *pcp, int high, int batch,

		       bool free_high)

static int nr_pcp_free(struct per_cpu_pages *pcp, int high, bool free_high)

{

	int min_nr_free, max_nr_free;

	int batch = READ_ONCE(pcp->batch);



	/* Free everything if batch freeing high-order pages. */

	if (unlikely(free_high))
@@ -2410,9 +2410,7 @@ static void free_unref_page_commit(struct zone *zone, struct per_cpu_pages *pcp, 


	high = nr_pcp_high(pcp, zone, free_high);

	if (pcp->count >= high) {

		int batch = READ_ONCE(pcp->batch);



		free_pcppages_bulk(zone, nr_pcp_free(pcp, high, batch, free_high), pcp, pindex);

		free_pcppages_bulk(zone, nr_pcp_free(pcp, high, free_high), pcp, pindex);

	}

}