From patchwork Thu Feb 16 15:07:00 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: David Howells X-Patchwork-Id: 13143336 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 A5D77C636CC for ; Thu, 16 Feb 2023 15:07:17 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 3CFED6B0074; Thu, 16 Feb 2023 10:07:17 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 380176B0075; Thu, 16 Feb 2023 10:07:17 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 2215B6B007B; Thu, 16 Feb 2023 10:07:17 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id 135566B0074 for ; Thu, 16 Feb 2023 10:07:17 -0500 (EST) Received: from smtpin06.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay07.hostedemail.com (Postfix) with ESMTP id 77E80160219 for ; Thu, 16 Feb 2023 15:07:16 +0000 (UTC) X-FDA: 80473483272.06.2EDEA6D Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by imf01.hostedemail.com (Postfix) with ESMTP id 929DA4000C for ; Thu, 16 Feb 2023 15:07:14 +0000 (UTC) Authentication-Results: imf01.hostedemail.com; dkim=pass header.d=redhat.com header.s=mimecast20190719 header.b=SjnulXlH; dmarc=pass (policy=none) header.from=redhat.com; spf=pass (imf01.hostedemail.com: domain of dhowells@redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=dhowells@redhat.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1676560034; 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=TMTMNPHQNijU8uUEBNxhNrrsQnlnvW6SaQn8r5zH6Oo=; b=XynXjwCRUdOXIV2BBzozXc/Cm0FHYNIQZmQTyDk1iGc+TObBvs8AAazTuyhpKihUrh4ASe CJuR5URlqy8RrBFddjXUtiwxztH4mipq73mQm/LjbW90fiXNJ+kjWLZli7jKyp3ea0TO7t j3ZOGlobUJOl+Wt+VZw2Hcbz5N5z7Ck= ARC-Authentication-Results: i=1; imf01.hostedemail.com; dkim=pass header.d=redhat.com header.s=mimecast20190719 header.b=SjnulXlH; dmarc=pass (policy=none) header.from=redhat.com; spf=pass (imf01.hostedemail.com: domain of dhowells@redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=dhowells@redhat.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1676560034; a=rsa-sha256; cv=none; b=U/OjjjjVuwjjXtxE+x6KE6qdlTJ24HmuJ8+OJJfcDyHxPsWhpGDQ3FFE61vVnWPJ6tQhgn 1EOxJzhGB3xNPQR4rmQO2tZYtLe7y8pQ1PHrbEDOi5MUTOsIW7UUGqBFjXVtuWoXlWWZAy nk/Zja7SmctWBShRT6cBSl2lEu5+jKU= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1676560033; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=TMTMNPHQNijU8uUEBNxhNrrsQnlnvW6SaQn8r5zH6Oo=; b=SjnulXlHa8AB0+nKbSLW/1D51Mi6EJEWCCpnuZvH6WWFrS1Dzy+/1Aagk3D0pj+UbCWAuJ doPPvgsxuedzN/FatSOsJXlcUp1Y+Fu7/zeGHXDhCcvg6IICnKRH2lmYRHGlZC4T7/xzkv alrlbalkoc8XfICwTa0UTPnXci8LBDk= Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-177-VDIE36s1OfyspGOi3TOcRw-1; Thu, 16 Feb 2023 10:07:12 -0500 X-MC-Unique: VDIE36s1OfyspGOi3TOcRw-1 Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.rdu2.redhat.com [10.11.54.6]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 98DF485D06A; Thu, 16 Feb 2023 15:07:10 +0000 (UTC) Received: from warthog.procyon.org.uk (unknown [10.33.36.24]) by smtp.corp.redhat.com (Postfix) with ESMTP id BAD9E2166B30; Thu, 16 Feb 2023 15:07:07 +0000 (UTC) From: David Howells To: Matthew Wilcox Cc: David Howells , Linus Torvalds , Jeff Layton , Christoph Hellwig , linux-afs@lists.infradead.org, linux-nfs@vger.kernel.org, linux-cifs@vger.kernel.org, ceph-devel@vger.kernel.org, v9fs-developer@lists.sourceforge.net, linux-erofs@lists.ozlabs.org, linux-ext4@vger.kernel.org, linux-cachefs@redhat.com, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, Rohith Surabattula , Steve French , Shyam Prasad N , Dave Wysochanski , Dominique Martinet , Ilya Dryomov , "Theodore Ts'o" , Andreas Dilger , linux-mm@kvack.org Subject: [PATCH v6 1/2] mm: Merge folio_has_private()/filemap_release_folio() call pairs Date: Thu, 16 Feb 2023 15:07:00 +0000 Message-Id: <20230216150701.3654894-2-dhowells@redhat.com> In-Reply-To: <20230216150701.3654894-1-dhowells@redhat.com> References: <20230216150701.3654894-1-dhowells@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.1 on 10.11.54.6 X-Rspam-User: X-Rspamd-Server: rspam02 X-Rspamd-Queue-Id: 929DA4000C X-Stat-Signature: y57jxuji8i4iq3xjteo66mqsgk9t5asz X-HE-Tag: 1676560034-199982 X-HE-Meta: U2FsdGVkX190Rkcz02XkpomUmwZ6g0eo9QpIf4NgQHAAfLLyvDE/cARytBtiE4wzQbwpGT/N7W0AfuxOnubW4IvRtmf8NjZFebPThTkjWyQaSHOg108/YU5wNmw8XGj+N1WVQPpWfZtIdiTT1VsF3fAlk62x2Gpa6/NZni7hVai4QcNOLOKMhGRoieFXfXyn2TUoHREmOv/K37I7MD3WlNnGEL+gOV0E+sLTj3kOPfqyQdwq+a4J4dEDi/i+MkuYGRd1DyTCl6JmE9GIokZlHaoadw9Ac/NTwRUmEDtiYpSe0gdU9sA+dRJzEqMIoRTipoH1PQa0x9cwhWW7EJLOHdlEsgRNh2HYmLq9olv95Ahpd2wPo29TGBtOb1GU/pSgNCnZ3WARs2+nV4tUafkuKeNVNLtqpveS8vhs/vv5stasi9g5yaj4wzQfmvWx9QRgCEQ8LhfYN7/1N+fg0PWdp3rPCsOnfazxDOa/LajZ7teUPG1/9WU9w5xsI3f0BO5E8RyrWU9LvNCUn/aeELDxReuPlxcbc86i4TxlYGpkZaOPgnYjjvXLv9TJ1CCa+F0FQrpLpjgzGXCHy4JCeoNK0xjcHwEBsqlntrn8bqCsRztY4sjPAZ+gPD0TyyTZ8DlxRyNd5Pj4fOORVPDi7FtcqN4DHZc4aeVhpM4H6YCjF4ezcSyGdurtTIYKqDWYqb20exPgrqu4ICEvoN2IzHUPIBFTc4gQn0nIJaIJbFMKGYCQ7dE3k+LmJPB6X6Ugnm7I5hDFe18fON2+M+a2Rhk/waD+WWxexwFxO42Eg1lSu7TcLuEU0SzDE9QMwdNuAhtVSp6vx4bx8lQqH+5v1jvbNDXQgU3LWO3ezekV22nyWJF5UQrrT0dbM1phQ/uXRIWJBb1d33cGsN2MmPeNesNgOihyIuE+ywPxlFPRtg+d+nYjbJTzmn0iUPpzZIq0zQ1jWqIKWv4FH8g5GP5SJUW 4YlwSxlv EbifLuk1TUoHPL2jtyd3qDFrQDqrXaPJsobmE+/n7y1NaA+6U4P1QuudV2i/wW62QWSQb6NsxNvLtWyp8AsxZOApRuTG6TUWRMV/nZKKE6gedHoHpgT9Y1+T4qfYc2Om+PHkxhkRCQbOq4Ebv8H922/nkmKIIONnR3hSqRaCq9JYq98W8zCgzqtX3V2A6+NpelnIjmtwCHQ/3cpw6QRSe4YAb1oNtJ8fBOwLRrp4j/krT3t+ymH0rbJ8IENqnetG9OEuUfe6iDW4RH6zXUGGGN000tjwGNmmBQ0pIVVXEkj9JBZ8eUYnBvZ3gO/5+4NXQKoaz0Kk92j6lT3KmmM3St4YkbGRnZMUpSJud9hO3H8mayFcQmzes4OqOowisAR8VqCtAoSGfJoZ6i96yF7BWhdq3edbALRnraHUgAqth7lD7rW5So4qPcypCiBi7neeu3gDmwu0TPgTsnUV2yukC+O7wcliqz1rWVEW3Onc6UFvtRQDFaAcefyMM4SFjAOTTOwWUEWyyEJpAE1MhrByRmX9mU/tp7efQ1pH3QFN7a9RHXaQ+z+Ldl6LRDP/C/xtxDVBfJC+4Nh51Y2dLpAmYWrAwtG5N/jj3FAMUlJKP/TlmEGcf2F5KyCwtQJMlUlmModU8amvdZfBIFgAL8G6XtWBEmKI+65tn4Ttyt+GawKhbtDoGblT9FNs72oxrqpH6v63aMXOQD/1b9kbkKK0N3SES4kZbT8hd2MbyeTT3UapjIYvFVBSdJzqkHJbYsn6GbQXIHxdFRB5P0t0= 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: Make filemap_release_folio() check folio_has_private(). Then, in most cases, where a call to folio_has_private() is immediately followed by a call to filemap_release_folio(), we can get rid of the test in the pair. There are a couple of sites in mm/vscan.c that this can't so easily be done. In shrink_folio_list(), there are actually three cases (something different is done for incompletely invalidated buffers), but filemap_release_folio() elides two of them. In shrink_active_list(), we don't have have the folio lock yet, so the check allows us to avoid locking the page unnecessarily. A wrapper function to check if a folio needs release is provided for those places that still need to do it in the mm/ directory. This will acquire additional parts to the condition in a future patch. After this, the only remaining caller of folio_has_private() outside of mm/ is a check in fuse. Reported-by: Rohith Surabattula Suggested-by: Matthew Wilcox Signed-off-by: David Howells cc: Matthew Wilcox cc: Linus Torvalds cc: Steve French cc: Shyam Prasad N cc: Rohith Surabattula cc: Dave Wysochanski cc: Dominique Martinet cc: Ilya Dryomov cc: "Theodore Ts'o" cc: Andreas Dilger cc: linux-cachefs@redhat.com cc: linux-cifs@vger.kernel.org cc: linux-afs@lists.infradead.org cc: v9fs-developer@lists.sourceforge.net cc: ceph-devel@vger.kernel.org cc: linux-nfs@vger.kernel.org cc: linux-ext4@vger.kernel.org cc: linux-fsdevel@vger.kernel.org cc: linux-mm@kvack.org --- Notes: ver #5) - Rebased on linus/master. try_to_release_page() has now been entirely replaced by filemap_release_folio(), barring one comment. - Cleaned up some pairs in ext4. ver #4) - Split from fscache fix. - Moved folio_needs_release() to mm/internal.h and removed open-coded version from filemap_release_folio(). ver #3) - Fixed mapping_clear_release_always() to use clear_bit() not set_bit(). - Moved a '&&' to the correct line. ver #2) - Rewrote entirely according to Willy's suggestion[1]. fs/ext4/move_extent.c | 12 ++++-------- fs/splice.c | 3 +-- mm/filemap.c | 2 ++ mm/huge_memory.c | 3 +-- mm/internal.h | 8 ++++++++ mm/khugepaged.c | 3 +-- mm/memory-failure.c | 8 +++----- mm/migrate.c | 3 +-- mm/truncate.c | 6 ++---- mm/vmscan.c | 8 ++++---- 10 files changed, 27 insertions(+), 29 deletions(-) diff --git a/fs/ext4/move_extent.c b/fs/ext4/move_extent.c index 8dbb87edf24c..dedc9d445f24 100644 --- a/fs/ext4/move_extent.c +++ b/fs/ext4/move_extent.c @@ -339,10 +339,8 @@ move_extent_per_page(struct file *o_filp, struct inode *donor_inode, ext4_double_up_write_data_sem(orig_inode, donor_inode); goto data_copy; } - if ((folio_has_private(folio[0]) && - !filemap_release_folio(folio[0], 0)) || - (folio_has_private(folio[1]) && - !filemap_release_folio(folio[1], 0))) { + if (!filemap_release_folio(folio[0], 0) || + !filemap_release_folio(folio[1], 0)) { *err = -EBUSY; goto drop_data_sem; } @@ -361,10 +359,8 @@ move_extent_per_page(struct file *o_filp, struct inode *donor_inode, /* At this point all buffers in range are uptodate, old mapping layout * is no longer required, try to drop it now. */ - if ((folio_has_private(folio[0]) && - !filemap_release_folio(folio[0], 0)) || - (folio_has_private(folio[1]) && - !filemap_release_folio(folio[1], 0))) { + if (!filemap_release_folio(folio[0], 0) || + !filemap_release_folio(folio[1], 0)) { *err = -EBUSY; goto unlock_folios; } diff --git a/fs/splice.c b/fs/splice.c index 5969b7a1d353..e69eddaf9d7c 100644 --- a/fs/splice.c +++ b/fs/splice.c @@ -65,8 +65,7 @@ static bool page_cache_pipe_buf_try_steal(struct pipe_inode_info *pipe, */ folio_wait_writeback(folio); - if (folio_has_private(folio) && - !filemap_release_folio(folio, GFP_KERNEL)) + if (!filemap_release_folio(folio, GFP_KERNEL)) goto out_unlock; /* diff --git a/mm/filemap.c b/mm/filemap.c index c4d4ace9cc70..344146c170b0 100644 --- a/mm/filemap.c +++ b/mm/filemap.c @@ -3960,6 +3960,8 @@ bool filemap_release_folio(struct folio *folio, gfp_t gfp) struct address_space * const mapping = folio->mapping; BUG_ON(!folio_test_locked(folio)); + if (!folio_needs_release(folio)) + return true; if (folio_test_writeback(folio)) return false; diff --git a/mm/huge_memory.c b/mm/huge_memory.c index abe6cfd92ffa..8490c42dedb3 100644 --- a/mm/huge_memory.c +++ b/mm/huge_memory.c @@ -2702,8 +2702,7 @@ int split_huge_page_to_list(struct page *page, struct list_head *list) gfp = current_gfp_context(mapping_gfp_mask(mapping) & GFP_RECLAIM_MASK); - if (folio_test_private(folio) && - !filemap_release_folio(folio, gfp)) { + if (!filemap_release_folio(folio, gfp)) { ret = -EBUSY; goto out; } diff --git a/mm/internal.h b/mm/internal.h index bcf75a8b032d..c4c8e58e1d12 100644 --- a/mm/internal.h +++ b/mm/internal.h @@ -163,6 +163,14 @@ static inline void set_page_refcounted(struct page *page) set_page_count(page, 1); } +/* + * Return true if a folio needs ->release_folio() calling upon it. + */ +static inline bool folio_needs_release(struct folio *folio) +{ + return folio_has_private(folio); +} + extern unsigned long highest_memmap_pfn; /* diff --git a/mm/khugepaged.c b/mm/khugepaged.c index 90acfea40c13..e257e0a13ad1 100644 --- a/mm/khugepaged.c +++ b/mm/khugepaged.c @@ -1944,8 +1944,7 @@ static int collapse_file(struct mm_struct *mm, unsigned long addr, goto out_unlock; } - if (folio_has_private(folio) && - !filemap_release_folio(folio, GFP_KERNEL)) { + if (!filemap_release_folio(folio, GFP_KERNEL)) { result = SCAN_PAGE_HAS_PRIVATE; folio_putback_lru(folio); goto out_unlock; diff --git a/mm/memory-failure.c b/mm/memory-failure.c index c77a9e37e27e..a4f809c11ae9 100644 --- a/mm/memory-failure.c +++ b/mm/memory-failure.c @@ -843,14 +843,12 @@ static int truncate_error_page(struct page *p, unsigned long pfn, struct folio *folio = page_folio(p); int err = mapping->a_ops->error_remove_page(mapping, p); - if (err != 0) { + if (err != 0) pr_info("%#lx: Failed to punch page: %d\n", pfn, err); - } else if (folio_has_private(folio) && - !filemap_release_folio(folio, GFP_NOIO)) { + else if (!filemap_release_folio(folio, GFP_NOIO)) pr_info("%#lx: failed to release buffers\n", pfn); - } else { + else ret = MF_RECOVERED; - } } else { /* * If the file system doesn't support it just invalidate diff --git a/mm/migrate.c b/mm/migrate.c index a4d3fc65085f..db867bb80128 100644 --- a/mm/migrate.c +++ b/mm/migrate.c @@ -915,8 +915,7 @@ static int fallback_migrate_folio(struct address_space *mapping, * Buffers may be managed in a filesystem specific way. * We must have no buffers or drop them. */ - if (folio_test_private(src) && - !filemap_release_folio(src, GFP_KERNEL)) + if (!filemap_release_folio(src, GFP_KERNEL)) return mode == MIGRATE_SYNC ? -EAGAIN : -EBUSY; return migrate_folio(mapping, dst, src, mode); diff --git a/mm/truncate.c b/mm/truncate.c index 7b4ea4c4a46b..8378aabb5294 100644 --- a/mm/truncate.c +++ b/mm/truncate.c @@ -19,7 +19,6 @@ #include #include #include -#include /* grr. try_to_release_page */ #include #include #include "internal.h" @@ -276,7 +275,7 @@ static long mapping_evict_folio(struct address_space *mapping, if (folio_ref_count(folio) > folio_nr_pages(folio) + folio_has_private(folio) + 1) return 0; - if (folio_has_private(folio) && !filemap_release_folio(folio, 0)) + if (!filemap_release_folio(folio, 0)) return 0; return remove_mapping(mapping, folio); @@ -573,8 +572,7 @@ static int invalidate_complete_folio2(struct address_space *mapping, if (folio->mapping != mapping) return 0; - if (folio_has_private(folio) && - !filemap_release_folio(folio, GFP_KERNEL)) + if (!filemap_release_folio(folio, GFP_KERNEL)) return 0; spin_lock(&mapping->host->i_lock); diff --git a/mm/vmscan.c b/mm/vmscan.c index bf3eedf0209c..0fd6623adccb 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -1996,7 +1996,7 @@ static unsigned int shrink_folio_list(struct list_head *folio_list, * (refcount == 1) it can be freed. Otherwise, leave * the folio on the LRU so it is swappable. */ - if (folio_has_private(folio)) { + if (folio_needs_release(folio)) { if (!filemap_release_folio(folio, sc->gfp_mask)) goto activate_locked; if (!mapping && folio_ref_count(folio) == 1) { @@ -2641,9 +2641,9 @@ static void shrink_active_list(unsigned long nr_to_scan, } if (unlikely(buffer_heads_over_limit)) { - if (folio_test_private(folio) && folio_trylock(folio)) { - if (folio_test_private(folio)) - filemap_release_folio(folio, 0); + if (folio_needs_release(folio) && + folio_trylock(folio)) { + filemap_release_folio(folio, 0); folio_unlock(folio); } }