Commit 59d336bd authored by Wang ShaoBo's avatar Wang ShaoBo Committed by Linus Torvalds
Browse files

mm/page_alloc: use clamp() to simplify code 

This patch uses clamp() to simplify code in init_per_zone_wmark_min().

Link: https://lkml.kernel.org/r/20211021034830.1049150-1-bobo.shaobowang@huawei.com


Signed-off-by: default avatarWang ShaoBo <bobo.shaobowang@huawei.com>
Reviewed-by: default avatarDavid Hildenbrand <david@redhat.com>
Cc: Wei Yongjun <weiyongjun1@huawei.com>
Cc: Li Bin <huawei.libin@huawei.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 9c25cbfc

mm/page_alloc.c

+4 −8
Original line number Diff line number Diff line
@@ -8477,16 +8477,12 @@ int __meminit init_per_zone_wmark_min(void) 
	lowmem_kbytes = nr_free_buffer_pages() * (PAGE_SIZE >> 10);

	new_min_free_kbytes = int_sqrt(lowmem_kbytes * 16);



	if (new_min_free_kbytes > user_min_free_kbytes) {

		min_free_kbytes = new_min_free_kbytes;

		if (min_free_kbytes < 128)

			min_free_kbytes = 128;

		if (min_free_kbytes > 262144)

			min_free_kbytes = 262144;

	} else {

	if (new_min_free_kbytes > user_min_free_kbytes)

		min_free_kbytes = clamp(new_min_free_kbytes, 128, 262144);

	else

		pr_warn("min_free_kbytes is not updated to %d because user defined value %d is preferred\n",

				new_min_free_kbytes, user_min_free_kbytes);

	}



	setup_per_zone_wmarks();

	refresh_zone_stat_thresholds();

	setup_per_zone_lowmem_reserve();