From patchwork Fri Sep 8 07:42:23 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vlastimil Babka X-Patchwork-Id: 13377205 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 5893AEE57E0 for ; Fri, 8 Sep 2023 07:42:32 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 9294B6B00A0; Fri, 8 Sep 2023 03:42:31 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 8D94A6B00A1; Fri, 8 Sep 2023 03:42:31 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 77A5C6B00A2; Fri, 8 Sep 2023 03:42:31 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id 67DFD6B00A0 for ; Fri, 8 Sep 2023 03:42:31 -0400 (EDT) Received: from smtpin07.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay07.hostedemail.com (Postfix) with ESMTP id 3B67C16019C for ; Fri, 8 Sep 2023 07:42:31 +0000 (UTC) X-FDA: 81212637702.07.1531A1D Received: from smtp-out1.suse.de (smtp-out1.suse.de [195.135.220.28]) by imf23.hostedemail.com (Postfix) with ESMTP id 5F840140003 for ; Fri, 8 Sep 2023 07:42:29 +0000 (UTC) Authentication-Results: imf23.hostedemail.com; dkim=pass header.d=suse.cz header.s=susede2_rsa header.b=0Tr1RiLs; dkim=pass header.d=suse.cz header.s=susede2_ed25519 header.b=r9UDzDCi; spf=pass (imf23.hostedemail.com: domain of vbabka@suse.cz designates 195.135.220.28 as permitted sender) smtp.mailfrom=vbabka@suse.cz; dmarc=none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1694158949; 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:references:dkim-signature; bh=NHJKcpKlZ/+yii6ZB0cLfqW7JVxO+ELAe/deKUrmJ80=; b=1Tkl4ZKZWUS2riDHIHsDJfdyijb2r8vNgid1XPmsaXahk9e1aDIV4N4y3r2XRXfdwQG8UP EfGQ7KbIptkEPW1UVd6FpKbJij68nCgRL3lzso8LmPhQ1DF7A/NmL2PNh2r3li69ppjHtO iQcf7XBYP5x1bTRwOurQ/nkWBY84PaA= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1694158949; a=rsa-sha256; cv=none; b=qa6lBLhL8jBzKfb+nYeRurTxGjDcj0SeIjrTMlOaPPQjDu3rAJlffM3XW+rZq09rVs10iO pW0BOLVs4T6vVECWjlxchAu281ywMqdl+1xGyZhZR+id4sRtT5mKIep5zRjfsexK++26Ny tGAatFPgBTYUAFQFrBbuZxXyZNKsquI= ARC-Authentication-Results: i=1; imf23.hostedemail.com; dkim=pass header.d=suse.cz header.s=susede2_rsa header.b=0Tr1RiLs; dkim=pass header.d=suse.cz header.s=susede2_ed25519 header.b=r9UDzDCi; spf=pass (imf23.hostedemail.com: domain of vbabka@suse.cz designates 195.135.220.28 as permitted sender) smtp.mailfrom=vbabka@suse.cz; dmarc=none Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by smtp-out1.suse.de (Postfix) with ESMTPS id 2588421832; Fri, 8 Sep 2023 07:42:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.cz; s=susede2_rsa; t=1694158947; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=NHJKcpKlZ/+yii6ZB0cLfqW7JVxO+ELAe/deKUrmJ80=; b=0Tr1RiLsGK9TKLdxwSc9RwxLIVKmEae8PGUWUTGhYIhqkvcnxmKXE3HYt5+5SqUjjUpxEZ DCTZmPbwNt5ToqgWZJ00UT0I0Plhw1i14U2HJUY0Px379BJbaU56+RSKEu8z+V/xOMo2xd woPLeoFeQ7kp5dfAXicDw3PDice1ftM= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.cz; s=susede2_ed25519; t=1694158947; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=NHJKcpKlZ/+yii6ZB0cLfqW7JVxO+ELAe/deKUrmJ80=; b=r9UDzDCi5/bP4K3dBarcJVpQtk/FZHh8rlyYCrXu3EqfT7loeAs6cgqxhTivXkBssY2kKX lV8TgWuOzutVQyDw== Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id 96D2A132F2; Fri, 8 Sep 2023 07:42:26 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id WIYoJGLQ+mTfFQAAMHmgww (envelope-from ); Fri, 08 Sep 2023 07:42:26 +0000 From: Vlastimil Babka To: seanjc@google.com Cc: ackerleytng@google.com, akpm@linux-foundation.org, anup@brainfault.org, aou@eecs.berkeley.edu, chao.p.peng@linux.intel.com, chenhuacai@kernel.org, david@redhat.com, isaku.yamahata@gmail.com, jarkko@kernel.org, jmorris@namei.org, kirill.shutemov@linux.intel.com, kvm-riscv@lists.infradead.org, kvm@vger.kernel.org, kvmarm@lists.linux.dev, liam.merwick@oracle.com, linux-arm-kernel@lists.infradead.org, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mips@vger.kernel.org, linux-mm@kvack.org, linux-riscv@lists.infradead.org, linux-security-module@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, mail@maciej.szmigiero.name, maz@kernel.org, michael.roth@amd.com, mpe@ellerman.id.au, oliver.upton@linux.dev, palmer@dabbelt.com, paul.walmsley@sifive.com, paul@paul-moore.com, pbonzini@redhat.com, qperret@google.com, serge@hallyn.com, tabba@google.com, vannapurve@google.com, vbabka@suse.cz, wei.w.wang@intel.com, willy@infradead.org, yu.c.zhang@linux.intel.com Subject: [PATCH gmem FIXUP v2] mm, compaction: make testing mapping_unmovable() safe Date: Fri, 8 Sep 2023 09:42:23 +0200 Message-ID: <20230908074222.28723-2-vbabka@suse.cz> X-Mailer: git-send-email 2.42.0 MIME-Version: 1.0 X-Rspamd-Queue-Id: 5F840140003 X-Rspam-User: X-Stat-Signature: 9xikpjgu5rfx3x9dnfz31cxsjg7p3szx X-Rspamd-Server: rspam03 X-HE-Tag: 1694158949-647543 X-HE-Meta: U2FsdGVkX1/91x6pi7xW73V/4OHASfNxKnJsazkBA2DLeqdfyTRaty9D9etdhOP1F/HFkTZsA9GunouhoqNaCT8X/GBSk7htoN0UKLj62bg9sg34dHNrjgnpQxmWYFbF3FL7p9Nj7+UMpSyPAJPU1IlV7Cjf6Ld4JZdB7c1/D2N2zqeu34vZZTTfHDmFZ/4inkKDyVwg5guACY3T4dtCPp0sC6ZncyQ529ql6O4xc8w2Qzzd5YGDIY4UNieAWx/RL5o2CY63n9DrIGva+1Hee5WSl0kSysM63qe6GYFG0qXGbpl3wvbqdkC3WaWkZHIkhZnoAfWN5XgqvisP6Fd3G9OwXwWCL12WywJbu0QaoC/xENRUe/68iOygb+57CDftR0m9ocy2GkDOa9Cc7nTIE8MBkwgc4Qg96AhmqXAMupebnbd0theyrhYx9qtgCh748FvxNZ1eZxFKzgJ3JpymfbbZ26z5BaNespaMmeDEyfYOcFrx1ua0k/Cp0YYxj57JLbH2LcsGytjK3XM5Ir483eJzG+I09ULgA/M2eltIr8T5r4YUrLT8Npc0hrl6oBnt6xrh++9AWGaIQMJP50sbwNNLqrYvGsZSuSX0J+yYFg8Kd/JyEu67mSBpe2tDmQIRbNwiLmJ7rLjT3v8fgJd01pntURlCnsRoxfrTKLztT1f2m7vpL3QGdKgMSlYzwmhXdSvu4EgN04E0a3+KJwFcnS6apiXmaZNqf9PbkyY9nz6tri5jp7ULZ4b0q38tkRO/zLGQ4bWHLo6SsRJ7sSyn3Kn0boIneXw+4n80cx0AKBgFER3N9fbueGuc6wKgOxtEWjbswlwr/djG4d+YOS751TWQjEtijJ16LtMmm+ncwW92/IEpICRxKREhiGxJFLFYDymi2K+bX1tqc8eaOYdKOoioeRBcSW7mhYynZnhAOcsHleNyWwcxk8PsxbnaHa/++SG6d656wvCX7wlZSpg PKMTb3bT gNbuciR8zU1/vcsE/UxjDIDfzA7b3kYfAOVnSsNcD6mriHpJCYJ+rqIpHHzzXp3Mt3vtVpZv+B1Y+spllAyCA256AMFn73y40KTvcyIoyUC6cTBnAZXKTzBAEgLJrwxZjaZ25O3Kz4JpcaNHY09jbVEn8Iee3cceSAVMJ8plxCdwNb9r4G3EWDzjBuRry3PNvIDFaXrP/Y6AooYQcbHKwObW0cRE/e2Vq+s4SbUdWbe5o/LdufSrecT+mHDfQ07n4M5zWDENVTRoxKdrUekdAhTT9V8oDnm2AY6AFLFKu7stvw7XqQHzPbeg8TA== 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: As Kirill pointed out, mapping can be removed under us due to truncation. Test it under folio lock as already done for the async compaction / dirty folio case. To prevent locking every folio with mapping to do the test, do it only for unevictable folios, as we can expect the unmovable mapping folios are also unevictable. To enforce that expecation, make mapping_set_unmovable() also set AS_UNEVICTABLE. Also incorporate comment update suggested by Matthew. Fixes: 3424873596ce ("mm: Add AS_UNMOVABLE to mark mapping as completely unmovable") Signed-off-by: Vlastimil Babka --- v2: mapping_set_unmovable() sets also AS_UNEVICTABLE, as Sean suggested. include/linux/pagemap.h | 6 +++++ mm/compaction.c | 49 +++++++++++++++++++++++++++-------------- virt/kvm/guest_mem.c | 2 +- 3 files changed, 39 insertions(+), 18 deletions(-) diff --git a/include/linux/pagemap.h b/include/linux/pagemap.h index 931d2f1da7d5..4070c59e6f25 100644 --- a/include/linux/pagemap.h +++ b/include/linux/pagemap.h @@ -276,6 +276,12 @@ static inline int mapping_use_writeback_tags(struct address_space *mapping) static inline void mapping_set_unmovable(struct address_space *mapping) { + /* + * It's expected unmovable mappings are also unevictable. Compaction + * migrate scanner (isolate_migratepages_block()) relies on this to + * reduce page locking. + */ + set_bit(AS_UNEVICTABLE, &mapping->flags); set_bit(AS_UNMOVABLE, &mapping->flags); } diff --git a/mm/compaction.c b/mm/compaction.c index a3d2b132df52..e0e439b105b5 100644 --- a/mm/compaction.c +++ b/mm/compaction.c @@ -862,6 +862,7 @@ isolate_migratepages_block(struct compact_control *cc, unsigned long low_pfn, /* Time to isolate some pages for migration */ for (; low_pfn < end_pfn; low_pfn++) { + bool is_dirty, is_unevictable; if (skip_on_failure && low_pfn >= next_skip_pfn) { /* @@ -1047,10 +1048,6 @@ isolate_migratepages_block(struct compact_control *cc, unsigned long low_pfn, if (!mapping && (folio_ref_count(folio) - 1) > folio_mapcount(folio)) goto isolate_fail_put; - /* The mapping truly isn't movable. */ - if (mapping && mapping_unmovable(mapping)) - goto isolate_fail_put; - /* * Only allow to migrate anonymous pages in GFP_NOFS context * because those do not depend on fs locks. @@ -1062,8 +1059,10 @@ isolate_migratepages_block(struct compact_control *cc, unsigned long low_pfn, if (!folio_test_lru(folio)) goto isolate_fail_put; + is_unevictable = folio_test_unevictable(folio); + /* Compaction might skip unevictable pages but CMA takes them */ - if (!(mode & ISOLATE_UNEVICTABLE) && folio_test_unevictable(folio)) + if (!(mode & ISOLATE_UNEVICTABLE) && is_unevictable) goto isolate_fail_put; /* @@ -1075,26 +1074,42 @@ isolate_migratepages_block(struct compact_control *cc, unsigned long low_pfn, if ((mode & ISOLATE_ASYNC_MIGRATE) && folio_test_writeback(folio)) goto isolate_fail_put; - if ((mode & ISOLATE_ASYNC_MIGRATE) && folio_test_dirty(folio)) { - bool migrate_dirty; + is_dirty = folio_test_dirty(folio); + + if (((mode & ISOLATE_ASYNC_MIGRATE) && is_dirty) + || (mapping && is_unevictable)) { + bool migrate_dirty = true; + bool is_unmovable; /* - * Only pages without mappings or that have a - * ->migrate_folio callback are possible to migrate - * without blocking. However, we can be racing with - * truncation so it's necessary to lock the page - * to stabilise the mapping as truncation holds - * the page lock until after the page is removed - * from the page cache. + * Only folios without mappings or that have + * a ->migrate_folio callback are possible to migrate + * without blocking. + * + * Folios from unmovable mappings are not migratable. + * + * However, we can be racing with truncation, which can + * free the mapping that we need to check. Truncation + * holds the folio lock until after the folio is removed + * from the page so holding it ourselves is sufficient. + * + * To avoid this folio locking to inspect every folio + * with mapping for being unmovable, we assume every + * such folio is also unevictable, which is a cheaper + * test. If our assumption goes wrong, it's not a bug, + * just potentially wasted cycles. */ if (!folio_trylock(folio)) goto isolate_fail_put; mapping = folio_mapping(folio); - migrate_dirty = !mapping || - mapping->a_ops->migrate_folio; + if ((mode & ISOLATE_ASYNC_MIGRATE) && is_dirty) { + migrate_dirty = !mapping || + mapping->a_ops->migrate_folio; + } + is_unmovable = mapping && mapping_unmovable(mapping); folio_unlock(folio); - if (!migrate_dirty) + if (!migrate_dirty || is_unmovable) goto isolate_fail_put; } diff --git a/virt/kvm/guest_mem.c b/virt/kvm/guest_mem.c index c81d2bb9ae93..85903c32163f 100644 --- a/virt/kvm/guest_mem.c +++ b/virt/kvm/guest_mem.c @@ -390,7 +390,7 @@ static int __kvm_gmem_create(struct kvm *kvm, loff_t size, u64 flags, inode->i_size = size; mapping_set_gfp_mask(inode->i_mapping, GFP_HIGHUSER); mapping_set_large_folios(inode->i_mapping); - mapping_set_unevictable(inode->i_mapping); + /* this also sets the mapping as unevictable */ mapping_set_unmovable(inode->i_mapping); fd = get_unused_fd_flags(0);