Received-SPF: pass (google.com: domain of linux-kernel+bounces-74662-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
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] mm: shrink skip folio mapped by an exiting task
Date: Wed, 21 Feb 2024 19:49:04 +0800
Message-ID: <20240221114904.1849-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?buXcU6hcuiH2ux83hEi55PKdYlrly23dHJ0tO7Kzg+xx6boPThpp1yh8y/8Q?=
 =?us-ascii?Q?15IK6ZGtUoJ+TAn8TZs8GpKZUDgRT7Rhk4kR7WiqDUQ3fkRtYIzb/W6YyDlW?=
 =?us-ascii?Q?qPa1QdbW7TDXmPohAGGwUgu45mabzdg5Le5ZSXz9F7tRynGXMjpSDRS4LKBY?=
 =?us-ascii?Q?Nc4nOwDzWqlO6FBoNeQp4BWZdWHG7FtKFc8WhcS21s7wFn33qaav+ABn/pvW?=
 =?us-ascii?Q?cVQRdXET3ds3LJvwDfYSfeHxr4fUnnkqEjymshfUB16ya4rWZDDl/45ytYNq?=
 =?us-ascii?Q?CeXLWi8z9W5u8xW+NbPGEbdpXFtHcg2NlbrN7qddWkyyHH4w3g5ITI8FvLaC?=
 =?us-ascii?Q?Fgy3ErIkIgyGr+VeQ+pSE77GGgabJSING5GXklc1nNKMrjRyMzx6RGqLLF3q?=
 =?us-ascii?Q?BdKY+DHVYQGScV2gA2lPa27PkOjGnafkUJ564Z20Qwc0O8bwAWACbXIgqXb7?=
 =?us-ascii?Q?zkatbypZRDMJT0yae2CXHCNZQcf2dYeR58Vi/Qb9QiPQqj2yaUgpsZbKqH2k?=
 =?us-ascii?Q?/MPIWZg0s0Emgvzo1IChhQY5NZyLo2NmVeYNTXACzM4ZIDijPuvL2w1Ey4Ix?=
 =?us-ascii?Q?FN8TCEzioLqsAGOuyPSbZdOzvtd4Dj9l5fhYkbM6qUiVjM6HSfYj7W26acQV?=
 =?us-ascii?Q?xAQqlp2Z9j55xtv2PoeFIclQnSDIkb8yE178koHmrHOdC7Lk07in09CstTFY?=
 =?us-ascii?Q?LknNPTNifMl1DZPG0/6Znz2sT7lEVfwRuDE0se0z2SuVzZ/DLoZa4vqsRQvx?=
 =?us-ascii?Q?x8fN/8FHCE2tvEfEF0M08jd+MfD5/EPh0JJ+kFADqHSetKDwN3bi5NEtRxDg?=
 =?us-ascii?Q?oBrEwWmPAQNBnIVD3ZEh+ZSoIjCkD+Kf+e64WwulV2fxr2KAGhvBOsJlYeuq?=
 =?us-ascii?Q?jVYL7ni7sXleih9UXzae6mqyb3eoPSQf8I65Ejkd4CrDT2L4T/SUCrv29EMy?=
 =?us-ascii?Q?3rjuimC/vnX3fKefrV1sM7nuHhfUJ7oUGJ167+rXHAPw9gBaGrr1ZUNS2jdA?=
 =?us-ascii?Q?Zojgl+axGK9petfrsxapGZ7MA7wwsUJt6EwuIp1iKNV+rDGAbbeftj/Uf4sh?=
 =?us-ascii?Q?i8BgwvS88Jkg9j+acywRyij/GdELUDqGIV52dLY7HEwCrgGSHbjHfaHLdy4A?=
 =?us-ascii?Q?hHxt2BOez/tR10TFc/jLykeFFRlkj8aP1IZhgvwlt8YOZ9zsMQAHfcKbO56v?=
 =?us-ascii?Q?Lm3kuWhIzpJZ5p+SwZRBYfjEZc2kW0NowWvb1g53AUb4392RG3g1AE4DkIUa?=
 =?us-ascii?Q?YKSv7pxe+sTA0OJbg77wbkceggNxb+XaOwluIeOAXGB5oOhfSAmUr0HOpSud?=
 =?us-ascii?Q?kJp9GX+ukbgEmdAFrwXVqiy6lhjOE5QtViIzQibb7TpLYZO3in3BJ/1H6p5l?=
 =?us-ascii?Q?jnOleLjdgLaT9750TvlMXPRCb7shY/0+bZ97A18r7QbMgqPLwJAlc8D8WMxw?=
 =?us-ascii?Q?wEQqSy76EoGTs6iIQ/AtlYMK50BSrJbY7T/ID2zj1cAQwRa88zxZSlNry6id?=
 =?us-ascii?Q?hmPpHgBTyCREoGlWMkSThJxuhtS/57pEQfzZn7R0SPqVZmhjcR13W9mzmoTY?=
 =?us-ascii?Q?CgAJG2TbDiKeeicjCe7GfaAn+0d8a1knKXG/Z3JN?=
X-OriginatorOrg: vivo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c7203eb6-45e9-4325-48ac-08dc32d32167
X-MS-Exchange-CrossTenant-AuthSource: JH0PR06MB6849.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Feb 2024 11:49:16.0993
 (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: HrSEpTS+7aWCWV8UrtTxGD/qd9siPrjr2Ur9AVwpGU+fLr8cSyO722Tdu/p8bJHOY3+OoIVgCxwXCB4be4Ex7Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SEZPR06MB7523

If an anon folio reclaimed by shrink_inactive_list is mapped by an
exiting task, this anon folio will be firstly swaped-out into
swapspace in shrink flow and then this swap folio is freed in task
exit flow. But if this folio mapped by an exiting task can skip
shrink and be freed directly in task exiting flow, which will save
swap-out time and alleviate the load of the tasks exiting process.
The file folio is also similar.

And when system is low memory, it more likely to occur, because more
backend applidatuions will be killed.

This patch can alleviate the load of the tasks exiting process.

Signed-off-by: Zhiguo Jiang <justinjiang@vivo.com>
---
 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 3746a5531018..146e5f4ec069
--- 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