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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 5B931C433F5 for ; Tue, 24 May 2022 19:15:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=sqJTFX+Kge0PkA2GhLiOYcuFazpWdWdtP7iY4Iguxso=; b=PlpJaOdSFO1s4w 5nBBCbOtltILzxePTj26PbKiPTEyJCGgP2gAa/u07Ywgk/w8e0FRf6fcczWMnPb7QCT7q48zsA7ql mNF4BIi0PyPL6TCaU1VUuMaVYV5FzYChUH3d9r+k/JLXT0Ov2YryhvbvKaYC+jujxYc9cbfFi32wO pX91XJnfWCZpUwp1q15mLo4CFnFteo+QSpHxRfCO4aIgdrdOAftFPjBmgLHWZHn61qJOqLWLiOO1H HqziMxdnF9iSaBovfKWIwUOSAz9v4RBdVdZzuhRuXqT6l53KJAvLYxSOll5Vur72vIUnLV+Sd80sO Uoaz78HMQCLp0vfeYWyw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1ntZzC-008zqb-DF; Tue, 24 May 2022 19:14:30 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1ntZyv-008zjb-KS for linux-arm-kernel@lists.infradead.org; Tue, 24 May 2022 19:14:17 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 25A55616AB; Tue, 24 May 2022 19:14:12 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id AAEC7C34118; Tue, 24 May 2022 19:14:08 +0000 (UTC) Date: Tue, 24 May 2022 20:14:04 +0100 From: Catalin Marinas To: Barry Song <21cnbao@gmail.com> Cc: Andrew Morton , Will Deacon , Linux-MM , LAK , LKML , hanchuanhua , =?utf-8?B?5byg6K+X5piOKFNpbW9uIFpoYW5nKQ==?= , =?utf-8?B?6YOt5YGl?= , Barry Song , "Huang, Ying" , Minchan Kim , Johannes Weiner , Hugh Dickins , Shaohua Li , Rik van Riel , Andrea Arcangeli , Steven Price Subject: Re: [PATCH] arm64: enable THP_SWAP for arm64 Message-ID: References: <20220524071403.128644-1-21cnbao@gmail.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220524_121413_887186_EC0F81E6 X-CRM114-Status: GOOD ( 32.68 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, May 24, 2022 at 10:05:35PM +1200, Barry Song wrote: > On Tue, May 24, 2022 at 8:12 PM Catalin Marinas wrote: > > On Tue, May 24, 2022 at 07:14:03PM +1200, Barry Song wrote: > > > diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig > > > index d550f5acfaf3..8e3771c56fbf 100644 > > > --- a/arch/arm64/Kconfig > > > +++ b/arch/arm64/Kconfig > > > @@ -98,6 +98,7 @@ config ARM64 > > > select ARCH_WANT_HUGE_PMD_SHARE if ARM64_4K_PAGES || (ARM64_16K_PAGES && !ARM64_VA_BITS_36) > > > select ARCH_WANT_LD_ORPHAN_WARN > > > select ARCH_WANTS_NO_INSTR > > > + select ARCH_WANTS_THP_SWAP if ARM64_4K_PAGES > > > > I'm not opposed to this but I think it would break pages mapped with > > PROT_MTE. We have an assumption in mte_sync_tags() that compound pages > > are not swapped out (or in). With MTE, we store the tags in a slab > > I assume you mean mte_sync_tags() require that THP is not swapped as a whole, > as without THP_SWP, THP is still swapping after being splitted. MTE doesn't stop > THP from swapping through a couple of splitted pages, does it? That's correct, split THP page are swapped out/in just fine. > > object (128-bytes per swapped page) and restore them when pages are > > swapped in. At some point we may teach the core swap code about such > > metadata but in the meantime that was the easiest way. > > If my previous assumption is true, the easiest way to enable THP_SWP > for this moment might be always letting mm fallback to the splitting > way for MTE hardware. For this moment, I care about THP_SWP more as > none of my hardware has MTE. > > diff --git a/arch/arm64/include/asm/pgtable.h b/arch/arm64/include/asm/pgtable.h > index 45c358538f13..d55a2a3e41a9 100644 > --- a/arch/arm64/include/asm/pgtable.h > +++ b/arch/arm64/include/asm/pgtable.h > @@ -44,6 +44,8 @@ > __flush_tlb_range(vma, addr, end, PUD_SIZE, false, 1) > #endif /* CONFIG_TRANSPARENT_HUGEPAGE */ > > +#define arch_thp_swp_supported !system_supports_mte > + > /* > * Outside of a few very special situations (e.g. hibernation), we always > * use broadcast TLB invalidation instructions, therefore a spurious page > diff --git a/include/linux/huge_mm.h b/include/linux/huge_mm.h > index 2999190adc22..064b6b03df9e 100644 > --- a/include/linux/huge_mm.h > +++ b/include/linux/huge_mm.h > @@ -447,4 +447,16 @@ static inline int split_folio_to_list(struct folio *folio, > return split_huge_page_to_list(&folio->page, list); > } > > +/* > + * archs that select ARCH_WANTS_THP_SWAP but don't support THP_SWP due to > + * limitations in the implementation like arm64 MTE can override this to > + * false > + */ > +#ifndef arch_thp_swp_supported > +static inline bool arch_thp_swp_supported(void) > +{ > + return true; > +} > +#endif > + > #endif /* _LINUX_HUGE_MM_H */ > diff --git a/mm/swap_slots.c b/mm/swap_slots.c > index 2b5531840583..dde685836328 100644 > --- a/mm/swap_slots.c > +++ b/mm/swap_slots.c > @@ -309,7 +309,7 @@ swp_entry_t get_swap_page(struct page *page) > entry.val = 0; > > if (PageTransHuge(page)) { > - if (IS_ENABLED(CONFIG_THP_SWAP)) > + if (IS_ENABLED(CONFIG_THP_SWAP) && arch_thp_swp_supported()) > get_swap_pages(1, &entry, HPAGE_PMD_NR); > goto out; I think this should work and with your other proposal it would be limited to MTE pages: #define arch_thp_swp_supported(page) (!test_bit(PG_mte_tagged, &page->flags)) Are THP pages loaded from swap as a whole or are they split? IIRC the splitting still happens but after the swapping out finishes. Even if they are loaded as 4K pages, we still have the mte_save_tags() that only understands small pages currently, so rejecting THP pages is probably best. -- Catalin _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel