Received-SPF: pass (google.com: domain of linux-kernel+bounces-46470-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
From: Zhiguo Jiang <justinjiang@vivo.com>
To: Andrew Morton <akpm@linux-foundation.org>,
	linux-mm@kvack.org,
	linux-kernel@vger.kernel.org
Cc: opensource.kernel@vivo.com,
	Zhiguo Jiang <justinjiang@vivo.com>
Subject: [PATCH v2] mm:vmscan: shrink skip folio mapped by an exiting task
Date: Wed, 31 Jan 2024 21:12:44 +0800
Message-ID: <20240131131244.1144-1-justinjiang@vivo.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?tppp+YJyDlcGyLm97UXcXcfPvLRDjepHKe03EOIaMIveIaQKufQSPQ80PbnQ?=
 =?us-ascii?Q?O2vMfrU73pK8dPb/6Wh/rb2bdmelWFxyhpY11IrMV36ut4Qxquj5c8FjVX2I?=
 =?us-ascii?Q?RkWCcGYw3YbL/XIEDGb/yYEZyEAcZnNOJi2ruOaDBET4cmTNnW6br+i60W7s?=
 =?us-ascii?Q?maWoPJpew6T1wQsoXa9Qlrd5xGawf3PoqGAvYEJp4u1v6bjteII7pTUOMqnk?=
 =?us-ascii?Q?jsthtUOwd7aklsUvAE5HXQP3PYBgpuJU2HEjjqXl58yNRMNvurd807n8/vdE?=
 =?us-ascii?Q?iBhDnuvAnDImJxHq88ubWot7eXq+LYwNC/vbeP+Q4ZdRnpMC1sPuZOyCVjxj?=
 =?us-ascii?Q?tTY4RMy83JTfx00srTKFTVwAu51BrRe9rHU+J+QMRA8TrPsi1RReOYuTXWaK?=
 =?us-ascii?Q?JbWJSpxn0jBS6IAKP4OJ4TZMsfwc0akE0Xq316lIK5Cu7EYKqIiBg1zveiiW?=
 =?us-ascii?Q?/XWq5z+0oMTLYc0VNddX2MUbiVtsrTqYhCGFuXECteY+YFMKhYKEKKhuVW5Z?=
 =?us-ascii?Q?DFaWaoHvPbPcEceVvpa+0yh49kPHrnONwbRujbjiRXvB1MTnje8jU8yySFWY?=
 =?us-ascii?Q?T3pJUIS3akgFs+UStyp3vwZlslHIVlO3tCtF5B6s1C1qF9BZwnsfabZywWKS?=
 =?us-ascii?Q?ZDlohuSvvrC5E1Qy+vAQJFkFfVcBJfFeCfz8cueHRjoi1noCw1VtKvzPJi+U?=
 =?us-ascii?Q?eG6/Y04nBT6pwTAmMU4GbeH2E0VpaQUtl2xJrjehyAN92sPSqg5JY7AV6jIN?=
 =?us-ascii?Q?fCYjDrxWqSWDlANLfJ0YuXmO5x6zTl6EzBxOGm5HIFHdu6ALo6U5mJUkJ5I2?=
 =?us-ascii?Q?ddy858ZIh+sO7HExyfNiEU9tFqAphv/Tp35S8oaBgYza7SYyfZLnOgwIxXyL?=
 =?us-ascii?Q?JKNt2btoHI5LKf0kz8TxCXOya497ju87uFVY3YUznGw1WgxEuEn7CWM2GMV3?=
 =?us-ascii?Q?OIOiExn1Dr35uqPxbjZnINsASgH4eGFKQVuLiq5MRyunryLePgY6T5mh8kxx?=
 =?us-ascii?Q?rX8UK5kgF1AS3BNhELQxpnXKnJr5gvsdM33mRGsYFwPul4o5ctTgUJzsWncE?=
 =?us-ascii?Q?+jqhmP6A03ECcpy/r2lf6mVUuXz0u9rlEauRM959My4qqzWFg/Q3qhQJgX/2?=
 =?us-ascii?Q?FL+gzsOsHWykGl707kaIe4t4IgxFEgMKZ10nNl4dqV/yOhvZ4+FNmaH/r7Hr?=
 =?us-ascii?Q?ih/Y4GcAUXFbDaaSuT5MoA9kOML+lhMyMNlbqFx2J0EmiBKO2iFNcNicTSOL?=
 =?us-ascii?Q?+Zue2WN3n84NWnBZ4ElndtMDXEBk/544f53wW4JJKUs1o2z2KJhYXQ4TAvGk?=
 =?us-ascii?Q?Y885Z8C6JndxdAbK5BmbljESciG5vE/v+WSlwik+tcIQiTfmZoz2Nii0L0Vj?=
 =?us-ascii?Q?4949HGG/UtoiEH8AHdqkPvMgSAMjkOFaktLHQMgcvdaBF5lMZj1vTKZnnera?=
 =?us-ascii?Q?wgAxsTXQu1fdfmaZdJT36k/EtZH1BG8K7LjnyquWXDka41kMjg2mH9Yg0GvM?=
 =?us-ascii?Q?UeJXr9GUbMpEjPLDllptEMpcQQFWfJUCtf6fnHE+hQhL1SGaQfvidhcurvW2?=
 =?us-ascii?Q?6iMPiu4ate0r/OJVOH8mhCWSJiH39uoIpiaoTHww?=
X-OriginatorOrg: vivo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: ebf97237-3913-4c5f-4c85-08dc225e54df
X-MS-Exchange-CrossTenant-AuthSource: JH0PR06MB6849.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 31 Jan 2024 13:12:52.6685
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 923e42dc-48d5-4cbe-b582-1a797a6412ed
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: jpiFNTHZ2XMFJFX04bgu0M9EGtpK0sh7NxMvuQPC048JB6+S+i6paegBtAwDGjAJMIzQkWwaqr2Wqu24yO0X3A==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: JH0PR06MB7354

If the folio shrinked by shrink_inactive_list is mapped by an exiting
task, this folio should be freed in the task exit flow rather than be
reclaimed in the shrink flow, because the former takes less time.

When the exiting tasks and shrink_inactive_list occur at the same time,
the lruvecs's folios which shrink_inactive_list reclaims may be mapped
by the exiting tasks. And when system is low memory, it more likely to
occur, because more backend applidatuions will be killed.

The shrink_inactive_list reclaims the exiting tasks's folios in lruvecs
and transforms the exiting tasks's anon folios into swap memory, which
will lead to the increasing load of the current exiting tasks.

This patch can alleviate the load of the tasks exiting process. Because
it can make that the exiting tasks release its anon folios faster
instead of releasing its swap memory from its anon folios swap-in in
shrink_inactive_list.

Signed-off-by: Zhiguo Jiang <justinjiang@vivo.com>
---

Change log:
v1->v2:
1.The VM_EXITING added in v1 patch is removed, because it will fail
to compile in 32-bit system.

 mm/rmap.c | 7 +++++++
 1 file changed, 7 insertions(+)
 mode change 100644 => 100755 mm/rmap.c

diff --git a/mm/rmap.c b/mm/rmap.c
index 1cf2bffa48ed..e6702bfafdde
--- a/mm/rmap.c
+++ b/mm/rmap.c
@@ -840,6 +840,13 @@ static bool folio_referenced_one(struct folio *folio,
 	int referenced = 0;
 	unsigned long start = address, ptes = 0;
 
+	/* Skip this folio if it's mapped by an exiting task */
+	if (unlikely(!atomic_read(&vma->vm_mm->mm_users)) ||
+		unlikely(test_bit(MMF_OOM_SKIP, &vma->vm_mm->flags))) {
+		pra->referenced = -1;
+		return false;
+	}
+
 	while (page_vma_mapped_walk(&pvmw)) {
 		address = pvmw.address;
 
-- 
2.39.0