summaryrefslogtreecommitdiffstats
path: root/mm/page_alloc.c
diff options
context:
space:
mode:
authorKemeng Shi <shikemeng@huaweicloud.com>2023-09-27 12:35:14 +0200
committerAndrew Morton <akpm@linux-foundation.org>2023-10-18 23:34:15 +0200
commit0dfca313a009c83e2ad44b3719dc1222df6c6db5 (patch)
treeebd3d7b57797ced85e48564e6bd99931fe03d8bb /mm/page_alloc.c
parentmm/page_alloc: remove unnecessary check in break_down_buddy_pages (diff)
downloadlinux-0dfca313a009c83e2ad44b3719dc1222df6c6db5.tar.xz
linux-0dfca313a009c83e2ad44b3719dc1222df6c6db5.zip
mm/page_alloc: remove unnecessary next_page in break_down_buddy_pages
The next_page is only used to forward page in case target is in second half range. Move forward page directly to remove unnecessary next_page. Link: https://lkml.kernel.org/r/20230927103514.98281-3-shikemeng@huaweicloud.com Signed-off-by: Kemeng Shi <shikemeng@huaweicloud.com> Acked-by: Naoya Horiguchi <naoya.horiguchi@nec.com> Cc: Matthew Wilcox (Oracle) <willy@infradead.org> Cc: Oscar Salvador <osalvador@suse.de> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Diffstat (limited to 'mm/page_alloc.c')
-rw-r--r--mm/page_alloc.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index fdb68b7c8240..f46e519618a0 100644
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
@@ -6462,20 +6462,18 @@ static void break_down_buddy_pages(struct zone *zone, struct page *page,
int migratetype)
{
unsigned long size = 1 << high;
- struct page *current_buddy, *next_page;
+ struct page *current_buddy;
while (high > low) {
high--;
size >>= 1;
if (target >= &page[size]) {
- next_page = page + size;
current_buddy = page;
+ page = page + size;
} else {
- next_page = page;
current_buddy = page + size;
}
- page = next_page;
if (set_page_guard(zone, current_buddy, high, migratetype))
continue;