From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id F2CA9C83F22 for ; Fri, 18 Jul 2025 02:42:16 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 7557B6B00C3; Thu, 17 Jul 2025 22:42:16 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 705F96B00C4; Thu, 17 Jul 2025 22:42:16 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 5F55D6B00C5; Thu, 17 Jul 2025 22:42:16 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id 492F06B00C3 for ; Thu, 17 Jul 2025 22:42:16 -0400 (EDT) Received: from smtpin11.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id 08008581A7 for ; Fri, 18 Jul 2025 02:42:16 +0000 (UTC) X-FDA: 83675836272.11.E9E451D Received: from out-187.mta1.migadu.com (out-187.mta1.migadu.com [95.215.58.187]) by imf26.hostedemail.com (Postfix) with ESMTP id 3138F140004 for ; Fri, 18 Jul 2025 02:42:13 +0000 (UTC) Authentication-Results: imf26.hostedemail.com; dkim=pass header.d=linux.dev header.s=key1 header.b="C+z0v/Bt"; spf=pass (imf26.hostedemail.com: domain of ye.liu@linux.dev designates 95.215.58.187 as permitted sender) smtp.mailfrom=ye.liu@linux.dev; dmarc=pass (policy=none) header.from=linux.dev ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1752806534; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-transfer-encoding:content-transfer-encoding: in-reply-to:references:dkim-signature; bh=165mTwuoOHXsCCkCN512tFxf4UxYV8qKAC9hxkL+/Lk=; b=4QtmqMUPrLDIc2hBAOpOOqyn1JAyv77BNjtpNKoAofJKHbpwqNZ28+MP9YRWeX6trNp7fO 3SAL8UjXTmZjru5bvTTAkD8h1Towo7m2rAsfnwqhqffObQEc03PQ874gBIaKlbDc1GkEr3 PK0Dcz8oi2Ieyok9csO4Z+aZbPqXYSg= ARC-Authentication-Results: i=1; imf26.hostedemail.com; dkim=pass header.d=linux.dev header.s=key1 header.b="C+z0v/Bt"; spf=pass (imf26.hostedemail.com: domain of ye.liu@linux.dev designates 95.215.58.187 as permitted sender) smtp.mailfrom=ye.liu@linux.dev; dmarc=pass (policy=none) header.from=linux.dev ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1752806534; a=rsa-sha256; cv=none; b=fJUJ/S6zvMLQ+sIVcd/rOYZDt3qj8FqDhkd4utu+oQneHc04QQ762q0t5grETwnCxDFS2H wqwuEo+WaEO7JtpZU0O9aQ9piB7pinszna5MqjEvkIotjV3IZ4FZjCCWjJIHxRveBRDvcE mNaH9FUiACdK57bLTvlb9SM374DjFdo= X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1752806531; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=165mTwuoOHXsCCkCN512tFxf4UxYV8qKAC9hxkL+/Lk=; b=C+z0v/BtEiiH9bIvE3RyS1HXloAmCIbAStbntjBf2mzpaCnw8iYPm5/ned/zGKD+pvLvtb 0Q4d6uYcc7MHKk3uuNZ5T1hA/mtaOkIqc1xQhfbN62oCCmiUeuMA2fvAkJjqtY6Jx0EbXK 2itEeFIJViPwTpLZiavmpOzp9/s1548= From: Ye Liu To: Andrew Morton , David Hildenbrand , Davidlohr Bueso , "Paul E. McKenney" , Josh Triplett , Frederic Weisbecker , Neeraj Upadhyay , Joel Fernandes , Boqun Feng , Uladzislau Rezki , Ingo Molnar , Peter Zijlstra , Juri Lelli , Vincent Guittot , Lorenzo Stoakes Cc: Ye Liu , "Liam R. Howlett" , Vlastimil Babka , Mike Rapoport , Suren Baghdasaryan , Michal Hocko , Steven Rostedt , Mathieu Desnoyers , Lai Jiangshan , Zqiang , Dietmar Eggemann , Ben Segall , Mel Gorman , Valentin Schneider , Zi Yan , Baolin Wang , Nico Pache , Ryan Roberts , Dev Jain , Barry Song , Kemeng Shi , Kairui Song , Nhat Pham , Baoquan He , Chris Li , linux-mm@kvack.org, linux-kernel@vger.kernel.org, rcu@vger.kernel.org Subject: [PATCH] mm: Replace (20 - PAGE_SHIFT) with common macros for pages<->MB conversion Date: Fri, 18 Jul 2025 10:41:32 +0800 Message-ID: <20250718024134.1304745-1-ye.liu@linux.dev> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Migadu-Flow: FLOW_OUT X-Rspam-User: X-Rspamd-Queue-Id: 3138F140004 X-Rspamd-Server: rspam06 X-Stat-Signature: p6kaszsz3h1emmtmrm6babgd5gkmr86t X-HE-Tag: 1752806533-198120 X-HE-Meta: U2FsdGVkX1+7F54gIjOjRuZOCUR/fTQNqfMl8wKtBH576jNKd300TkuCbboeGhaPsngbJEzspYyNLe2uyDs+LN5vN2mC2N+XEEW/rjNt6gA1AOzfdXQrejdpG+h3apJh217meDqgzriHgiJH+OlR6UyO5pQVmsYibDd13+kud0NPYMGiAN1wnGHghKH8hlITW37LoOtdDE84poJzfkUrO9qvn+GdlYHAqnOxahKwr4GT1ZaJFauo9cL95BUWGe6JJQWWl87T7UK9pkQZ559ZVJHtYwNWwwfL9LCH1Vk85rouPlapRYTitczHzXLWpYt/Q7K928MQwQSFEt9uy0M4R1S7sB8DUd8nykrMgvXM3ajfUWX0+335McucISwFrJQ62zcXR3YdxHLCaQcAlrOCWHLfoSnT1eCgcuqZAb8hDqUgNEBiYAymFHP0x3AEKWlpEQtn/2sDgK1vIjyLxqpoNusO1lkm9Q+13j2kNbZ+IXZrhHD+Iheq5qE5UdPSbd+TDgRHjdNvX2keszwFPwvVN2MASLT52b/BKKH8Pp8XHB3J3wphejkDbQ1iPb32Kf9lrv1xGm9J+TM4pcmJC916mYbR1C2lKMtFwJqiEp75p7E40dELuVKrkrS6RoYa4zVq6+t/KaCsuw2w0P30WijOAYhKqDtay/kQ75V5EKeHr/L3yIE4lglTNQPawFm1NxiBu5WxOE+AKWPMGkxZvV+omj/dVSMd3BeFxiO/ua5wxL/3tNkAbLu1sGwURJ05rpgAxxJynJxg8abfnZaIpb9udHbMtnTADFidwm63FuhzS2O+lMY3s9NWtIFt9ZnsWCVDPkkz2nIbZELkmvPUCZd60ACQ50kCEdCeLEkwUeL79GpksGCXSagrhBe2t9GhU3+Hd9dbQWfLYDmSCf48ZnodrBAPSeV71LvlB+JYsQqKyYvyUIhyejQT3awXlZGMsZWpO0GcVPX83rhXDM860II tv37dFEd CmUNDpXzKty/eoCHGpY8DXnIwL8d/FELs9Fp105Eke8f9DsC84pHSsTOcTyhiVGC/+7j+fZebzrWc3/Ve2rWlTii1b6vxMiypJiceWoyC2XXL27SznnAuQLfxiDCaqqiSjnStt5r/Q4MPzcJQqdnEvI+0m3F/m9Yv0mSQsgZ3ZxwatAKDAd2Jy7y2xxwHVfvVf09xGIcM07GKZbIZXc1ECQxrCuisw/mQiuDp4FEoSGvy56DCvvH8F8c9VY0oSzXMbOt5l7T56La0cU2UAhFbRNBnCY8mzl2jO+iccAGHfMT+MxsY8kcKh2M7DLTVC9hQVBBWnqv/hBAFZC7RF+cq0u5kT/aJQvkxTQqVBQhp830hAog= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: From: Ye Liu Replace repeated (20 - PAGE_SHIFT) calculations with standard macros: - MB_TO_PAGES(mb) converts MB to page count - PAGES_TO_MB(pages) converts pages to MB No functional change. Signed-off-by: Ye Liu --- include/linux/mm.h | 9 +++++++++ kernel/rcu/rcuscale.c | 2 +- kernel/sched/fair.c | 5 ++--- mm/backing-dev.c | 2 +- mm/huge_memory.c | 2 +- mm/swap.c | 2 +- 6 files changed, 15 insertions(+), 7 deletions(-) diff --git a/include/linux/mm.h b/include/linux/mm.h index 957acde6ae62..0c1b2c074142 100644 --- a/include/linux/mm.h +++ b/include/linux/mm.h @@ -69,6 +69,15 @@ static inline void totalram_pages_add(long count) extern void * high_memory; +/* + * Convert between pages and MB + * 20 is the shift for 1MB (2^20 = 1MB) + * PAGE_SHIFT is the shift for page size (e.g., 12 for 4KB pages) + * So (20 - PAGE_SHIFT) converts between pages and MB + */ +#define PAGES_TO_MB(pages) ((pages) >> (20 - PAGE_SHIFT)) +#define MB_TO_PAGES(mb) ((mb) << (20 - PAGE_SHIFT)) + #ifdef CONFIG_SYSCTL extern int sysctl_legacy_va_layout; #else diff --git a/kernel/rcu/rcuscale.c b/kernel/rcu/rcuscale.c index b521d0455992..7484d8ad5767 100644 --- a/kernel/rcu/rcuscale.c +++ b/kernel/rcu/rcuscale.c @@ -796,7 +796,7 @@ kfree_scale_thread(void *arg) pr_alert("Total time taken by all kfree'ers: %llu ns, loops: %d, batches: %ld, memory footprint: %lldMB\n", (unsigned long long)(end_time - start_time), kfree_loops, rcuscale_seq_diff(b_rcu_gp_test_finished, b_rcu_gp_test_started), - (mem_begin - mem_during) >> (20 - PAGE_SHIFT)); + PAGES_TO_MB(mem_begin - mem_during)); if (shutdown) { smp_mb(); /* Assign before wake. */ diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index b9b4bbbf0af6..ae1d9a7ef202 100644 --- a/kernel/sched/fair.c +++ b/kernel/sched/fair.c @@ -1489,7 +1489,7 @@ static unsigned int task_nr_scan_windows(struct task_struct *p) * by the PTE scanner and NUMA hinting faults should be trapped based * on resident pages */ - nr_scan_pages = sysctl_numa_balancing_scan_size << (20 - PAGE_SHIFT); + nr_scan_pages = MB_TO_PAGES(sysctl_numa_balancing_scan_size); rss = get_mm_rss(p->mm); if (!rss) rss = nr_scan_pages; @@ -1926,8 +1926,7 @@ bool should_numa_migrate_memory(struct task_struct *p, struct folio *folio, } def_th = sysctl_numa_balancing_hot_threshold; - rate_limit = sysctl_numa_balancing_promote_rate_limit << \ - (20 - PAGE_SHIFT); + rate_limit = MB_TO_PAGES(sysctl_numa_balancing_promote_rate_limit); numa_promotion_adjust_threshold(pgdat, rate_limit, def_th); th = pgdat->nbp_threshold ? : def_th; diff --git a/mm/backing-dev.c b/mm/backing-dev.c index 783904d8c5ef..e4d578e6121c 100644 --- a/mm/backing-dev.c +++ b/mm/backing-dev.c @@ -510,7 +510,7 @@ static void wb_update_bandwidth_workfn(struct work_struct *work) /* * Initial write bandwidth: 100 MB/s */ -#define INIT_BW (100 << (20 - PAGE_SHIFT)) +#define INIT_BW MB_TO_PAGES(100) static int wb_init(struct bdi_writeback *wb, struct backing_dev_info *bdi, gfp_t gfp) diff --git a/mm/huge_memory.c b/mm/huge_memory.c index 389620c65a5f..dcc33d9c300f 100644 --- a/mm/huge_memory.c +++ b/mm/huge_memory.c @@ -911,7 +911,7 @@ static int __init hugepage_init(void) * where the extra memory used could hurt more than TLB overhead * is likely to save. The admin can still enable it through /sys. */ - if (totalram_pages() < (512 << (20 - PAGE_SHIFT))) { + if (totalram_pages() < MB_TO_PAGES(512)) { transparent_hugepage_flags = 0; return 0; } diff --git a/mm/swap.c b/mm/swap.c index 3632dd061beb..cb164f9ef9e3 100644 --- a/mm/swap.c +++ b/mm/swap.c @@ -1096,7 +1096,7 @@ static const struct ctl_table swap_sysctl_table[] = { */ void __init swap_setup(void) { - unsigned long megs = totalram_pages() >> (20 - PAGE_SHIFT); + unsigned long megs = PAGES_TO_MB(totalram_pages()); /* Use a smaller cluster for small-memory machines */ if (megs < 16) -- 2.43.0