From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 66D5A43C7D3 for ; Wed, 12 Aug 2026 12:18:08 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786537098; cv=none; b=Ep7tBmTelYNuKQEYgW3fbkKv+IDQ8f1hhKT8IwjmUhUr6sZRVxxdi9Ch5sXVle7HL7Lw5268HvalfzkYEUIkpyDwnyernF0rzEGZvXx6xnMfA9SIyouIYBuzeGFTQF9nMXH8LumC9D3d/uGFi65d+oX8IWDjQIhGo0R0mbmGCpw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786537098; c=relaxed/simple; bh=3zZOiXijHvymRhTyLFfoNXKxs5IL+xmm5ocYX6k0c1w=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=cgLtRsmKViBeQtRoMqX7TlGYmeXQwyqXLA0GTE3eRrbyy1gNuJKZUR3mT6djqBLRrZJiHp+rwbKmQcVJRL1WGi46WHMczZn5JVf6z92HJoWEXyuudtsUGDwXH5xIxzauKZyUbv+PxjOxITW8c9Sr6ZZ5TevKuCLfoxjs697QAQY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=mOf3kSGC; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="mOf3kSGC" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5D3941F00A3F; Wed, 12 Aug 2026 12:18:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1786537088; bh=JfAFNjFFahuQ/02Rkg04vR1FEvVXtWN3blDMb1aBDxU=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=mOf3kSGCfSxxfdiJl7wZodylrQcmlpLnTU/EWOJ2bdq3O0M1rLSC+lwTxkhQFdree FOgtXEMi1Ux59cCoHRWmf+Xv17LYSspapX612WkMbcTBYkcPhsdoiAr4YGEl98OPtI hyH6RPUjS8oPpEJE3a0UGkNCjXSVNLn6/v0RgolnW8QozyqbEe3roMnuI4WcPMSXX8 dORxyvZXF3y0WlD5JEH59V6xfchTrVsDkOZ1mr4igXkHACBOAmSB9oqb77iS8qDM8j sILeNtf+alBzo4UvQ+fpV6kfuqk/kJdb9laFKzmTPLyG2Eb+W6pP5dUSAOSg/6d4JB IPuP33JgEA01w== From: "Barry Song (Xiaomi)" To: akpm@linux-foundation.org, linux-mm@kvack.org Cc: axelrasmussen@google.com, chenridong@xiaomi.com, david@kernel.org, hannes@cmpxchg.org, kasong@tencent.com, lianux.mm@gmail.com, linux-kernel@vger.kernel.org, ljs@kernel.org, lyugaofei@xiaomi.com, mhocko@kernel.org, qi.zheng@linux.dev, shakeel.butt@linux.dev, stevensd@chromium.org, wangzicheng@honor.com, weixugc@google.com, yuanchu@google.com, zhangbo56@xiaomi.com, baolin.wang@linux.alibaba.com, baoquan.he@linux.dev, Barry Song Subject: [RFC PATCH v4 01/16] mm/mglru: improve readability of isolate_folios() Date: Wed, 12 Aug 2026 20:16:43 +0800 Message-Id: <20260812121658.69965-2-baohua@kernel.org> X-Mailer: git-send-email 2.39.3 (Apple Git-146) In-Reply-To: <20260812121658.69965-1-baohua@kernel.org> References: <20260812121658.69965-1-baohua@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: Ridong Chen The for_each_evictable_type() loop in isolate_folios() is misleading: it does not actually iterate over each evictable type. Instead, get_type_to_scan() selects the type to scan, while the iterator `i` merely bounds the number of attempts. Signed-off-by: Ridong Chen Co-developed-by: Barry Song (Xiaomi) Signed-off-by: Barry Song (Xiaomi) --- mm/vmscan.c | 46 ++++++++++++++++++++++++++-------------------- 1 file changed, 26 insertions(+), 20 deletions(-) diff --git a/mm/vmscan.c b/mm/vmscan.c index 17d2b793cbfc..ea058692b9a5 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -4839,35 +4839,41 @@ static int get_type_to_scan(struct lruvec *lruvec, int swappiness) return positive_ctrl_err(&sp, &pv); } +static inline bool is_single_type_reclaim(int swappiness) +{ + return swappiness == MIN_SWAPPINESS || + swappiness == SWAPPINESS_ANON_ONLY; +} + static int isolate_folios(unsigned long nr_to_scan, struct lruvec *lruvec, struct scan_control *sc, int swappiness, struct list_head *list, int *isolated, int *isolate_type, int *isolate_scanned) { - int i; - int total_scanned = 0; + bool type_fallback_allowed = !is_single_type_reclaim(swappiness); int type = get_type_to_scan(lruvec, swappiness); + int total_scanned = 0, scanned, tier; - for_each_evictable_type(i, swappiness) { - int scanned; - int tier = get_tier_idx(lruvec, type); +retry: + tier = get_tier_idx(lruvec, type); + scanned = scan_folios(nr_to_scan, lruvec, sc, + type, tier, list, isolated); - scanned = scan_folios(nr_to_scan, lruvec, sc, - type, tier, list, isolated); + total_scanned += scanned; + if (*isolated) { + *isolate_type = type; + *isolate_scanned = scanned; + return total_scanned; + } - total_scanned += scanned; - if (*isolated) { - *isolate_type = type; - *isolate_scanned = scanned; - break; - } - /* - * If scanned > 0 and isolated == 0, avoid falling back to the - * other type, as this type remains sufficient. Falling back - * too readily can disrupt the positive_ctrl_err() bias. - */ - if (!scanned) - type = !type; + /* + * We are running out of the current reclaim type. Fall back to + * the other type if allowed. + */ + if (!scanned && type_fallback_allowed) { + type = !type; + type_fallback_allowed = false; + goto retry; } return total_scanned; -- 2.34.1