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 4FBFDC3DA6E for ; Wed, 20 Dec 2023 15:16:55 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id A6C1E6B007E; Wed, 20 Dec 2023 10:16:54 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id A1B0B6B0083; Wed, 20 Dec 2023 10:16:54 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 8E3036B0087; Wed, 20 Dec 2023 10:16:54 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by kanga.kvack.org (Postfix) with ESMTP id 7B1B46B007E for ; Wed, 20 Dec 2023 10:16:54 -0500 (EST) Received: from smtpin12.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id 49C27803A9 for ; Wed, 20 Dec 2023 15:16:54 +0000 (UTC) X-FDA: 81587549148.12.D1EA8AF Received: from mail-wm1-f50.google.com (mail-wm1-f50.google.com [209.85.128.50]) by imf29.hostedemail.com (Postfix) with ESMTP id 749E1120004 for ; Wed, 20 Dec 2023 15:16:52 +0000 (UTC) Authentication-Results: imf29.hostedemail.com; dkim=pass header.d=google.com header.s=20230601 header.b=OvjF0D8U; dmarc=pass (policy=reject) header.from=google.com; spf=pass (imf29.hostedemail.com: domain of yuzhao@google.com designates 209.85.128.50 as permitted sender) smtp.mailfrom=yuzhao@google.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1703085412; 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-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=jv8bozQkFa6PPje43LsZtl2v4AnPzE/ZYxuMwvQWpw8=; b=HuLYhy6//SCzl9CsXGQWyMHQsL/b6e4aig9QoStHYIbhPKmLlWgEpdRoxsaFAah1wEJR/k AtdoG2wCXKYdmzLFuUddXqnVM9tyMb9F8aOU4YbIjORr7WeL/mA1IdbHukH5caXsdhJ8vq z2s7u0jNQGLIGf1Jzwn0SNHTP6c6SCw= ARC-Authentication-Results: i=1; imf29.hostedemail.com; dkim=pass header.d=google.com header.s=20230601 header.b=OvjF0D8U; dmarc=pass (policy=reject) header.from=google.com; spf=pass (imf29.hostedemail.com: domain of yuzhao@google.com designates 209.85.128.50 as permitted sender) smtp.mailfrom=yuzhao@google.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1703085412; a=rsa-sha256; cv=none; b=pQEW52REFxmfb0NuDiMITehg9RSrFBf0kv/2dBSG5CO2MWtkq3r8RQYO0fd/YvyhyjUA2s Q4rA02/2L6haGpGL/eHgmuSAuLPPyPEqXNmWO09RWH2b1iQRuqVDYb9btf12ygEuoYx90z U7Ytgf/DptQfm2pZsAytnUzPNRc5qm0= Received: by mail-wm1-f50.google.com with SMTP id 5b1f17b1804b1-40c32bea30dso61895e9.0 for ; Wed, 20 Dec 2023 07:16:52 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1703085411; x=1703690211; darn=kvack.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=jv8bozQkFa6PPje43LsZtl2v4AnPzE/ZYxuMwvQWpw8=; b=OvjF0D8UNYdWh8Z3M9VNAjWP2k3r5j4RVom1oQP6IgMIsujcPF+M3K2MGO8gFsKfng R8+X9IlWCgyWnQk4jzk75wkAxdJ+9IKKIp69OUtRExTie+LfcYOjqhZNqM7Xch6jLhpf l2Vl3FfAsWdyLK1X7uA8j2KalUKJoMBC62SdF4ZmAd/eh4qeOX80ihs0/sMh1kpJ0K02 /z8Y8WDMJw/3dGN5hp9bLIHm7Jb7dUykmPKb9JWvBTBkmseNt7yUn4ekRTs/RL03jSlz nRbOOEeZyhGexO8GvdNulLyTYDxj4X51aqqPWO7KDjVQ/uckfkRfWGbvlFY472XMuZN6 YZfA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1703085411; x=1703690211; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=jv8bozQkFa6PPje43LsZtl2v4AnPzE/ZYxuMwvQWpw8=; b=IDqLIs6NgqcMZ9uC7Li88y7+Cuk39VxjJLv1m7XdYOSJtv6iXKCNIBmdX3WXWoAG1Z LY48VEbPhH0jERQE4ufm9ie/Q8ePr/3+Ume6uEW5CgC1OIxeVdCU7iphRU0hQuTXOjV2 yWRuis2dRM88OKcl4MUcZgyuiYBx6lO6EYZgNFIf/JlbUy5K1MRVqzvXLIZ58QRvMqf+ Mx3X3aiLodp82KbNzX11YlvBFI3qm3QsAdTWJqRKkPYpAwhdCmNHkna/HpFxRAKPtLcK w6Etf+M0UrrZurARqZz9RqRudCOvU7gGXoNkF5Bh15OIf5rUugC/sMVA6Y0Y/dfyj+w8 wxcw== X-Gm-Message-State: AOJu0Yw7ipYJRTBcmcCqrLNFi/XAukt/waG8McrFtr7u6FE2hWGNcQPL 8nuWUoXMX+44acVOZhsw0yIQLWnx82jrhpOlBz7HSi6YpsKW92+KdS6UrOPEkIWl X-Google-Smtp-Source: AGHT+IGSDIkLfab95qwSdrd1KWtLRBQD93MvXgb37SthLf+GHm1wG0U815RbZ2UX9X8j8ckCuqY2HwzAFfY0u88kcjk= X-Received: by 2002:a05:600c:1c9a:b0:40d:2bc7:e9b with SMTP id k26-20020a05600c1c9a00b0040d2bc70e9bmr171420wms.4.1703085410638; Wed, 20 Dec 2023 07:16:50 -0800 (PST) MIME-Version: 1.0 References: <20231220040037.883811-1-kinseyho@google.com> <7baea98a-90ff-4c2d-9d00-d3ddc715c235@linux.vnet.ibm.com> In-Reply-To: <7baea98a-90ff-4c2d-9d00-d3ddc715c235@linux.vnet.ibm.com> From: Yu Zhao Date: Wed, 20 Dec 2023 08:16:11 -0700 Message-ID: Subject: Re: [PATCH mm-unstable v3 0/4] mm/mglru: Kconfig cleanup To: Donet Tom Cc: Kinsey Ho , Andrew Morton , linux-mm@kvack.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Rspamd-Server: rspam09 X-Rspamd-Queue-Id: 749E1120004 X-Stat-Signature: k5sokotdmq1fi7cjod4e9jxcc56eaapy X-Rspam-User: X-HE-Tag: 1703085412-880512 X-HE-Meta: U2FsdGVkX18u29h+s27Eb78WYWBu7CNB4zErJ+dKQ67FhBP0lgJ1bHgNGB439c2B5FWRv2oIrL/+Gh7wbHmoz9PuwsFFUT98uhdcimPOe3RoqH3EAGomFRrM6l/+ku66ZBbgedDl0ur4HOHPhvC4sBQYXgDJGUanrfaRi9ZFvL0mgQz67StgBz37UcGudpoerHpSVNbXNhlWX+dPoeIu38e/dNfIFp+iTxKqxNMqQ9mfdGl0TllgTnTce3kUEK+O69X0gZfzJy9PdxVSNzYUF0s7c9KI+D+BVW0wpCeJX7+YFKIvMttD8s5FQJOFNKWhcuDZ9gJcFpQrgK2IOmFnppAVy5Jot9qfBqBMvVc0tLh/3iXw19m0c1roRuCG+FuEssZEMokJDhJMy83vX95NwQfvpOtwBNFbBR98TYfu6BkWwHIBKR69a+DFbcM2witoGLoNW6/rbqZ/LUxVv141jFikZunc1x402PXuGNS5CWbUe++FkBQFrq9Pz+bLA933h/iptuKfHhgDAS8FR5maoVInY5/7MPrFZUF6IYYDpVm47DA8UO40qoOnxxKA15lyX/HW5nfZM0bN5njMA7xuXrSSrzSQLsvtWos34JTFCHS5rXSVDLZra1fKd67DgN5m6RZL3+zRj3qPCcCt+0Fqjw2k7t22YufTmzogl99HON6Bt2x2tGYlqcxSPl2TLxMVwLYKY6wqHhd62Zna7OjFI3r//NO7qbeSvovDVtYTd3/8re1o7msSZb+GnIHkHd7a/bmzqFMS79eFlW5g/vdWSO7hXil70hre0CO5/y27Y01IlZSdastUjhYg5602B3igmtWPJhXT8WwqhgN1B3rM+qEh+oPBgjy+EWYWblzOoY4g/SAtNtJ4l+rJPLvtCdyNVp3p0uK5skDLslAZTpTS6WuZRTLYcBhO7i0gGwjXHr4RIEE6nFAWdRLm8rHRvOuN4vKQLmEvDKJFy3TdrSS ukbr+6j9 t7RIASsII2tgs1osKwsLzFQ7Y4cZSPhN9D/tIUq0pwcJaLzT3PSgCz7lSQlR/LZRhtrCQYlAXQfRaMpnJ1r/o8gcHHgM6oV7CpMpmTZCYGRdgcZ4AlqzB/SDSQ5VyJEuVlY2Hy2AiYxOT1+vOs2Ljn/RDUAPMMzbOFr34f2WpzvUaGFJeKhk4gy6hD2P7byuXKyYeE9Q1Iu1y6V3dBjutbkKcY31SZldX9FUf78HigDK+3mdWkXNECB0CMwfFn3vSdY30bEQXW1lbsHhrDGpR1KR7H84qyn4gQoQQKsCmR4uUtQOtIa0eT0IIj6rs2cUleE91aul7rnssY8E= 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: On Wed, Dec 20, 2023 at 6:45=E2=80=AFAM Donet Tom wrote: > > > On 12/20/23 09:46, Yu Zhao wrote: > > On Tue, Dec 19, 2023 at 9:01=E2=80=AFPM Kinsey Ho = wrote: > >> This series is the result of the following discussion: > >> https://lore.kernel.org/47066176-bd93-55dd-c2fa-002299d9e034@linux.ibm= .com/ > >> > >> It mainly avoids building the code that walks page tables on CPUs that > >> use it, i.e., those don't support hardware accessed bit. Specifically, > >> it introduces a new Kconfig to guard some of functions added by > >> commit bd74fdaea146 ("mm: multi-gen LRU: support page table walks") > >> on CPUs like POWER9, on which the series was tested. > >> > >> > >> Kinsey Ho (4): > >> mm/mglru: add CONFIG_ARCH_HAS_HW_PTE_YOUNG > >> mm/mglru: add CONFIG_LRU_GEN_WALKS_MMU > >> mm/mglru: remove CONFIG_MEMCG > >> mm/mglru: remove CONFIG_TRANSPARENT_HUGEPAGE > >> > >> arch/Kconfig | 8 + > >> arch/arm64/Kconfig | 1 + > >> arch/x86/Kconfig | 1 + > >> arch/x86/include/asm/pgtable.h | 6 - > >> include/linux/memcontrol.h | 2 +- > >> include/linux/mm_types.h | 16 +- > >> include/linux/mmzone.h | 28 +--- > >> include/linux/pgtable.h | 2 +- > >> kernel/fork.c | 2 +- > >> mm/Kconfig | 4 + > >> mm/vmscan.c | 271 ++++++++++++++++++-------------= -- > >> 11 files changed, 174 insertions(+), 167 deletions(-) > > +Donet Tom > > who is also working on this. > > > > Donet, could try this latest version instead? If it works well as the > > old one you've been using, can you please provide your Tested-by tag? > > Thanks. > > Hi Yu Zhao, > > This patch set looks promising. > > I have conducted tests on PowerPC and x86. > > In old patch set there is a cleanup patch which removes > struct scan_control *sc argument from try_to_inc_max_seq() and > run_aging(), Do we need to include that patch? Sorry not for including that patch in this series. It's the first patch in the next cleanup series, which we haven't fully tested yet. It'll be the first order of business after the holiday season (mid Jan), does the schedule work for you? > =3D>Here are some test results from PowerPC. > > # ls -l vmscan.o > -rw-r--r--. 1 root root 3600080 Dec 19 22:35 vmscan.o > > # size vmscan.o > text data bss dec hex filename > 95086 27412 0 122498 1de82 vmscan.o > > # ./scripts/bloat-o-meter vmscan.o.old vmscan.o > add/remove: 4/8 grow/shrink: 7/9 up/down: 860/-2524 (-1664) > Function old new delta > should_abort_scan - 472 +472 > inc_max_seq.isra 1472 1612 +140 > shrink_one 680 760 +80 > lru_gen_release_memcg 508 556 +48 > lru_gen_init_pgdat 92 132 +40 > shrink_node 4040 4064 +24 > lru_gen_online_memcg 680 696 +16 > lru_gen_change_state 3968 3984 +16 > ------ > shrink_lruvec 2168 2152 -16 > lru_gen_seq_write 1980 1964 -16 > isolate_folios 6904 6888 -16 > lru_gen_init_memcg 32 12 -20 > mm_list 24 - -24 > lru_gen_exit_memcg 388 344 -44 > try_to_shrink_lruvec 904 816 -88 > lru_gen_rotate_memcg 832 700 -132 > lru_gen_migrate_mm 132 - -132 > lru_gen_seq_show 1484 1308 -176 > iterate_mm_list_nowalk 288 - -288 > lru_gen_look_around 2284 1984 -300 > lru_gen_add_mm 528 - -528 > lru_gen_del_mm 720 - -720 > Total: Before=3D116213, After=3D114549, chg -1.43% > > =3D>Here are some test results from x86. > > $ ls -l vmscan.o > -rw-r--r--. 1 donettom donettom 2545792 Dec 20 15:16 vmscan.o > > $ size vmscan.o > text data bss dec hex filename > 109751 32189 0 141940 22a74 vmscan.o > $ > > $ ./scripts/bloat-o-meter vmscan.o.old vmscan.o > add/remove: 7/3 grow/shrink: 14/4 up/down: 2307/-1534 (773) > Function old new delta > inc_max_seq - 1470 +1470 > should_abort_scan - 229 +229 > isolate_folios 4469 4562 +93 > lru_gen_rotate_memcg 641 731 +90 > lru_gen_init_memcg 41 99 +58 > lru_gen_release_memcg 282 336 +54 > lru_gen_exit_memcg 306 350 +44 > walk_pud_range 2502 2543 +41 > shrink_node 2912 2951 +39 > lru_gen_online_memcg 402 434 +32 > lru_gen_seq_show 1112 1140 +28 > lru_gen_add_folio 740 757 +17 > lru_gen_look_around 1217 1233 +16 > __pfx_should_abort_scan - 16 +16 > __pfx_inc_max_seq - 16 +16 > iterate_mm_list_nowalk 277 292 +15 > shrink_one 413 426 +13 > lru_gen_init_lruvec 190 202 +12 > ----- > try_to_shrink_lruvec 717 643 -74 > lru_gen_init_pgdat 196 82 -114 > try_to_inc_max_seq.isra 2897 1578 -1319 > Total: Before=3D101095, After=3D101868, chg +0.76% > $ > > > Tested-by: Donet Tom Thanks! Acked-by: Yu Zhao