From patchwork Mon Feb 13 12:34:39 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Huang, Ying" X-Patchwork-Id: 13138385 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 21547C636CC for ; Mon, 13 Feb 2023 12:35:35 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 96F1B6B007D; Mon, 13 Feb 2023 07:35:34 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 8FB4A280001; Mon, 13 Feb 2023 07:35:34 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 7718B6B0080; Mon, 13 Feb 2023 07:35:34 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id 644406B007D for ; Mon, 13 Feb 2023 07:35:34 -0500 (EST) Received: from smtpin14.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay04.hostedemail.com (Postfix) with ESMTP id 3E0E71A05F0 for ; Mon, 13 Feb 2023 12:35:34 +0000 (UTC) X-FDA: 80462214588.14.1C02532 Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by imf22.hostedemail.com (Postfix) with ESMTP id D4CF3C0007 for ; Mon, 13 Feb 2023 12:35:31 +0000 (UTC) Authentication-Results: imf22.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=Pm74s98j; spf=pass (imf22.hostedemail.com: domain of ying.huang@intel.com designates 192.55.52.136 as permitted sender) smtp.mailfrom=ying.huang@intel.com; dmarc=pass (policy=none) header.from=intel.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1676291732; 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:in-reply-to:references:references:dkim-signature; bh=ZFIi0s5drFbPlmDXPFosTCFwtCRMcMEqzhBaJWzq/ss=; b=mKcehGYwNBxPv79QQlQ+L27PbigyxLz+WNn+FqKjkiO1Xt68GVM9FB0vnZApezQOoj7FU+ zit97xPF8ufYzfsdpWZEKsZUkYdmMyC1byplxwAon73cJYtr/Iy7NIw25LV1fwbGRH+kDe 6G4iDY4APoTi2+PlmKXTXxk7QnQSjgQ= ARC-Authentication-Results: i=1; imf22.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=Pm74s98j; spf=pass (imf22.hostedemail.com: domain of ying.huang@intel.com designates 192.55.52.136 as permitted sender) smtp.mailfrom=ying.huang@intel.com; dmarc=pass (policy=none) header.from=intel.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1676291732; a=rsa-sha256; cv=none; b=EX77N7G/KC7L3bhUiNw0ToQ1TL/6hTPQ9QynYx13we54Y81zR8sZbG0RCzL2AqtPCT8cH6 n3y498q+IL1NJU1UgG7zAj2ZyYeSTsOOlasyiPysMAULOvdr0IWr9IpdjuYEFHDLOYgKA4 UUr1iDr5EZLl/8e9oc712WW6jmOlfO8= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1676291731; x=1707827731; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=3XEbG6JD2LdI1CjVNNq0obsv14/SFsm/rzfKm+kf4eA=; b=Pm74s98jQmkqJKLjbHgQZV/Tj3/8F4Z05naZJNZGCZuhngUPPjAheQn2 BJNlg5jtpm8/wADUt1f1kCido3hXdz0yovnqrB2liI4ILCFoQLqn7Ujzo UFi4pZoC83OCcxf0CAKyX6n0uxw6YHJCVhSWCDwEBtizShzMm6brgKgnM TqQRYdX+rA1F4HfyAxRP9sX4u3CcFec48ZcOnc90X+qMCCpwVdXlYRFgd JDLYKBuxIGi6R8NOptMM1xfZXLayWdvBbtxhEdPJFMK5Q3afc3D25yu/U yQGk6ZctyidsqOrvs7cG5JeHyTSWkGt/vGfyxrw3Cdke4+rc6VLKN31rG g==; X-IronPort-AV: E=McAfee;i="6500,9779,10619"; a="310513224" X-IronPort-AV: E=Sophos;i="5.97,294,1669104000"; d="scan'208";a="310513224" Received: from orsmga006.jf.intel.com ([10.7.209.51]) by fmsmga106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 Feb 2023 04:35:31 -0800 X-IronPort-AV: E=McAfee;i="6500,9779,10619"; a="646366631" X-IronPort-AV: E=Sophos;i="5.97,294,1669104000"; d="scan'208";a="646366631" Received: from changxin-mobl2.ccr.corp.intel.com (HELO yhuang6-mobl2.ccr.corp.intel.com) ([10.255.28.171]) by orsmga006-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 Feb 2023 04:35:27 -0800 From: Huang Ying To: Andrew Morton Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org, Huang Ying , Baolin Wang , Xin Hao , Zi Yan , Yang Shi , Oscar Salvador , Matthew Wilcox , Bharata B Rao , Alistair Popple , Minchan Kim , Mike Kravetz , Hyeonggon Yoo <42.hyeyoo@gmail.com> Subject: [PATCH -v5 4/9] migrate_pages: split unmap_and_move() to _unmap() and _move() Date: Mon, 13 Feb 2023 20:34:39 +0800 Message-Id: <20230213123444.155149-5-ying.huang@intel.com> X-Mailer: git-send-email 2.35.1 In-Reply-To: <20230213123444.155149-1-ying.huang@intel.com> References: <20230213123444.155149-1-ying.huang@intel.com> MIME-Version: 1.0 X-Rspam-User: X-Rspamd-Server: rspam04 X-Rspamd-Queue-Id: D4CF3C0007 X-Stat-Signature: 8rb33pzwcensm9oj94yb49zb1oypm3q5 X-HE-Tag: 1676291731-154814 X-HE-Meta: U2FsdGVkX1/bDRF8T/OiAi5emGzEtP2WXpGJcJKvjMphrXvXwGlLuFv7tt48ynvGlvB75OTCkyB6YIvxaeCOeGnBQHUjLzrgv60RA4jxq0+e9n5U438nlSd7rtCrrAppVdG1GnhVUj0K3jD42MI46U9POH6572G6s4WSoxLNE5YwrvenR92tk4j7X8k0CJEttvXuCEa8ZWjvqvAzhvDvfj0xrNAcVI7B3fbeIlvS7crk5E2Bq/RuwemGKe3BIZLLqIM3MOt2hsrFLc41nrUcOEyLpHJDfq8pShc/TlXpKI9bG551o14HGjDdOK8vW6+bQJiNoH3+yIAiqA1HCm7U1D5Ls1rmPZKLtKxAO9EUiQxYVSFPxOgx7M6awKTeKwLFbD66LNJoZMMkBTH081DRSjBc1pC1Xk/YQilNjoQqU6+rrezCtokrOb5BXfkt/4OzUs2hjKfswut8eQsMidhBZLt8e2+BnGrs2mAJAlu+OrbkDZGYLd/7qm4y7IV1+l05ab/Xkknbzzs0GKhmAKaKpvOzpX8bpXeFeyBYofHnfQF9Fdu6iMRKhkBpiCO/uRui4fcF++bNtd+m+LlnPXm+8aGzyGYSix0oh4XR7wMWb6ntiDtD1PlsHT/lNxAaWvFaBePNCy7y6nCa7BUZAZNM+RrV7bSiZdeWFhV+A03T8iBQ4AEB1jjKbHJ1GfalZS5lJGu0uUGsOkKEuGZRfK/Q3xFZ4zLI35JheT9ovyCwijXAjeIxhAqkTYC17Z7Dpa3S+viDQFRPPWOHGaT8+2vgoeSBo4EU8UrxoD0C8PUfTVziVM919fkuC62OW4aZAwsjzydGpJTV8rgvGHfpppylrhyhF+ksXWipTTbam2b/X2mjzNgq7zSGI2zV5G0YvsB5kLGUZKVpGWdDq80S1l1SpbsrpugbP/CBO6cTRYE/NqWoegdTJPRVChZmOZvAXFT3g9dPQh0NCTdb+Ss63W4 GQ+HecLI VsAD8t2N8ZBMAvRA9nMGb+hCm7qTuXOe9hOdsIUoiufBzT8ZjKqGoUsM59sAXa1hEwkBHNlS84OvU0KGa49ReQFMwXDG6HLFmkRPVuQUCiNY2iqrKwSK4dSsqQAObafjA5SHJQQutXsXmO+RRuHA0BfWNdx24gm8QsTwRhuShcqpu/53anR/Iwzwd2a43wzua1ffXCfAemBxI2pL+oqq276/S2dDoSVMbYBBuMDlpinATtqkzZZKgOOAz+/HtKg17KoHVBzIjO622MzoJ+ooejkN1RY/rC4lCqlunj+yILski+i8ZEO6P/H+rPhqo4rSDNOeN9eAVovLqrzSVwiRiZuSxhIVJoJbqCYRoVvk9zMsb4QhmF5TXDM4UAJ3Z7rXBhFKf+0eCY/HNDukx9xxTBlaPLAf2DKzgf93YMLbKCtnBr6no49LHKJw7RYCVeN57ycnh7a6vf2c+cbw7llLO25VxswJBKvlXxUlsqqBW+ynXjK2WWdrHQGS+yPQYIs+mNnvnrO088baEY5VhKwhevCOKsQ== 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: This is a preparation patch to batch the folio unmapping and moving. In this patch, unmap_and_move() is split to migrate_folio_unmap() and migrate_folio_move(). So, we can batch _unmap() and _move() in different loops later. To pass some information between unmap and move, the original unused dst->mapping and dst->private are used. Signed-off-by: "Huang, Ying" Reviewed-by: Baolin Wang Reviewed-by: Xin Hao Cc: Zi Yan Cc: Yang Shi Cc: Oscar Salvador Cc: Matthew Wilcox Cc: Bharata B Rao Cc: Alistair Popple Cc: Minchan Kim Cc: Mike Kravetz Cc: Hyeonggon Yoo <42.hyeyoo@gmail.com> --- include/linux/migrate.h | 1 + mm/migrate.c | 169 ++++++++++++++++++++++++++++++---------- 2 files changed, 129 insertions(+), 41 deletions(-) diff --git a/include/linux/migrate.h b/include/linux/migrate.h index bdff950a8bb4..c88b96b48be7 100644 --- a/include/linux/migrate.h +++ b/include/linux/migrate.h @@ -18,6 +18,7 @@ struct migration_target_control; * - zero on page migration success; */ #define MIGRATEPAGE_SUCCESS 0 +#define MIGRATEPAGE_UNMAP 1 /** * struct movable_operations - Driver page migration diff --git a/mm/migrate.c b/mm/migrate.c index d436f35fa145..5fd18a7cce62 100644 --- a/mm/migrate.c +++ b/mm/migrate.c @@ -1027,11 +1027,53 @@ static int move_to_new_folio(struct folio *dst, struct folio *src, return rc; } -static int __unmap_and_move(struct folio *src, struct folio *dst, +/* + * To record some information during migration, we use some unused + * fields (mapping and private) of struct folio of the newly allocated + * destination folio. This is safe because nobody is using them + * except us. + */ +static void __migrate_folio_record(struct folio *dst, + unsigned long page_was_mapped, + struct anon_vma *anon_vma) +{ + dst->mapping = (void *)anon_vma; + dst->private = (void *)page_was_mapped; +} + +static void __migrate_folio_extract(struct folio *dst, + int *page_was_mappedp, + struct anon_vma **anon_vmap) +{ + *anon_vmap = (void *)dst->mapping; + *page_was_mappedp = (unsigned long)dst->private; + dst->mapping = NULL; + dst->private = NULL; +} + +/* Cleanup src folio upon migration success */ +static void migrate_folio_done(struct folio *src, + enum migrate_reason reason) +{ + /* + * Compaction can migrate also non-LRU pages which are + * not accounted to NR_ISOLATED_*. They can be recognized + * as __PageMovable + */ + if (likely(!__folio_test_movable(src))) + mod_node_page_state(folio_pgdat(src), NR_ISOLATED_ANON + + folio_is_file_lru(src), -folio_nr_pages(src)); + + if (reason != MR_MEMORY_FAILURE) + /* We release the page in page_handle_poison. */ + folio_put(src); +} + +static int __migrate_folio_unmap(struct folio *src, struct folio *dst, int force, enum migrate_mode mode) { int rc = -EAGAIN; - bool page_was_mapped = false; + int page_was_mapped = 0; struct anon_vma *anon_vma = NULL; bool is_lru = !__PageMovable(&src->page); @@ -1107,8 +1149,8 @@ static int __unmap_and_move(struct folio *src, struct folio *dst, goto out_unlock; if (unlikely(!is_lru)) { - rc = move_to_new_folio(dst, src, mode); - goto out_unlock_both; + __migrate_folio_record(dst, page_was_mapped, anon_vma); + return MIGRATEPAGE_UNMAP; } /* @@ -1133,11 +1175,42 @@ static int __unmap_and_move(struct folio *src, struct folio *dst, VM_BUG_ON_FOLIO(folio_test_anon(src) && !folio_test_ksm(src) && !anon_vma, src); try_to_migrate(src, 0); - page_was_mapped = true; + page_was_mapped = 1; } - if (!folio_mapped(src)) - rc = move_to_new_folio(dst, src, mode); + if (!folio_mapped(src)) { + __migrate_folio_record(dst, page_was_mapped, anon_vma); + return MIGRATEPAGE_UNMAP; + } + + if (page_was_mapped) + remove_migration_ptes(src, src, false); + +out_unlock_both: + folio_unlock(dst); +out_unlock: + /* Drop an anon_vma reference if we took one */ + if (anon_vma) + put_anon_vma(anon_vma); + folio_unlock(src); +out: + + return rc; +} + +static int __migrate_folio_move(struct folio *src, struct folio *dst, + enum migrate_mode mode) +{ + int rc; + int page_was_mapped = 0; + struct anon_vma *anon_vma = NULL; + bool is_lru = !__PageMovable(&src->page); + + __migrate_folio_extract(dst, &page_was_mapped, &anon_vma); + + rc = move_to_new_folio(dst, src, mode); + if (unlikely(!is_lru)) + goto out_unlock_both; /* * When successful, push dst to LRU immediately: so that if it @@ -1160,12 +1233,10 @@ static int __unmap_and_move(struct folio *src, struct folio *dst, out_unlock_both: folio_unlock(dst); -out_unlock: /* Drop an anon_vma reference if we took one */ if (anon_vma) put_anon_vma(anon_vma); folio_unlock(src); -out: /* * If migration is successful, decrease refcount of dst, * which will not free the page because new page owner increased @@ -1177,19 +1248,15 @@ static int __unmap_and_move(struct folio *src, struct folio *dst, return rc; } -/* - * Obtain the lock on folio, remove all ptes and migrate the folio - * to the newly allocated folio in dst. - */ -static int unmap_and_move(new_page_t get_new_page, - free_page_t put_new_page, - unsigned long private, struct folio *src, - int force, enum migrate_mode mode, - enum migrate_reason reason, - struct list_head *ret) +/* Obtain the lock on page, remove all ptes. */ +static int migrate_folio_unmap(new_page_t get_new_page, free_page_t put_new_page, + unsigned long private, struct folio *src, + struct folio **dstp, int force, + enum migrate_mode mode, enum migrate_reason reason, + struct list_head *ret) { struct folio *dst; - int rc = MIGRATEPAGE_SUCCESS; + int rc = MIGRATEPAGE_UNMAP; struct page *newpage = NULL; if (!thp_migration_supported() && folio_test_transhuge(src)) @@ -1200,20 +1267,49 @@ static int unmap_and_move(new_page_t get_new_page, folio_clear_active(src); folio_clear_unevictable(src); /* free_pages_prepare() will clear PG_isolated. */ - goto out; + list_del(&src->lru); + migrate_folio_done(src, reason); + return MIGRATEPAGE_SUCCESS; } newpage = get_new_page(&src->page, private); if (!newpage) return -ENOMEM; dst = page_folio(newpage); + *dstp = dst; dst->private = NULL; - rc = __unmap_and_move(src, dst, force, mode); + rc = __migrate_folio_unmap(src, dst, force, mode); + if (rc == MIGRATEPAGE_UNMAP) + return rc; + + /* + * A folio that has not been unmapped will be restored to + * right list unless we want to retry. + */ + if (rc != -EAGAIN) + list_move_tail(&src->lru, ret); + + if (put_new_page) + put_new_page(&dst->page, private); + else + folio_put(dst); + + return rc; +} + +/* Migrate the folio to the newly allocated folio in dst. */ +static int migrate_folio_move(free_page_t put_new_page, unsigned long private, + struct folio *src, struct folio *dst, + enum migrate_mode mode, enum migrate_reason reason, + struct list_head *ret) +{ + int rc; + + rc = __migrate_folio_move(src, dst, mode); if (rc == MIGRATEPAGE_SUCCESS) set_page_owner_migrate_reason(&dst->page, reason); -out: if (rc != -EAGAIN) { /* * A folio that has been migrated has all references @@ -1229,20 +1325,7 @@ static int unmap_and_move(new_page_t get_new_page, * we want to retry. */ if (rc == MIGRATEPAGE_SUCCESS) { - /* - * Compaction can migrate also non-LRU folios which are - * not accounted to NR_ISOLATED_*. They can be recognized - * as __folio_test_movable - */ - if (likely(!__folio_test_movable(src))) - mod_node_page_state(folio_pgdat(src), NR_ISOLATED_ANON + - folio_is_file_lru(src), -folio_nr_pages(src)); - - if (reason != MR_MEMORY_FAILURE) - /* - * We release the folio in page_handle_poison. - */ - folio_put(src); + migrate_folio_done(src, reason); } else { if (rc != -EAGAIN) list_add_tail(&src->lru, ret); @@ -1534,7 +1617,7 @@ static int migrate_pages_batch(struct list_head *from, new_page_t get_new_page, int pass = 0; bool is_large = false; bool is_thp = false; - struct folio *folio, *folio2; + struct folio *folio, *folio2, *dst = NULL; int rc, nr_pages; LIST_HEAD(split_folios); bool nosplit = (reason == MR_NUMA_MISPLACED); @@ -1561,9 +1644,13 @@ static int migrate_pages_batch(struct list_head *from, new_page_t get_new_page, cond_resched(); - rc = unmap_and_move(get_new_page, put_new_page, - private, folio, pass > 2, mode, - reason, ret_folios); + rc = migrate_folio_unmap(get_new_page, put_new_page, private, + folio, &dst, pass > 2, mode, + reason, ret_folios); + if (rc == MIGRATEPAGE_UNMAP) + rc = migrate_folio_move(put_new_page, private, + folio, dst, mode, + reason, ret_folios); /* * The rules are: * Success: folio will be freed