From patchwork Thu Mar 28 16:33:52 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: David Howells X-Patchwork-Id: 13608967 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 58461CD1283 for ; Thu, 28 Mar 2024 16:34:43 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 9F3406B0092; Thu, 28 Mar 2024 12:34:42 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 97C486B0096; Thu, 28 Mar 2024 12:34:42 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 7F5D56B0099; Thu, 28 Mar 2024 12:34:42 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id 595DF6B0092 for ; Thu, 28 Mar 2024 12:34:42 -0400 (EDT) Received: from smtpin04.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id 07CAE1C02E6 for ; Thu, 28 Mar 2024 16:34:42 +0000 (UTC) X-FDA: 81946996404.04.7513725 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by imf19.hostedemail.com (Postfix) with ESMTP id 57B0B1A0008 for ; Thu, 28 Mar 2024 16:34:40 +0000 (UTC) Authentication-Results: imf19.hostedemail.com; dkim=pass header.d=redhat.com header.s=mimecast20190719 header.b=iV7WMMps; dmarc=pass (policy=none) header.from=redhat.com; spf=pass (imf19.hostedemail.com: domain of dhowells@redhat.com designates 170.10.133.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=1711643680; 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=ffhOAbislEtDui9sz/UMzHfpFDxhB+Cw1cNj4X99Z0A=; b=sILZJZGhWQlz5COLyPxOPcmP1Ubsli2tZ8yVgO60/KLJwT7GxVMsK2tRBod7EnOqQtbLaY THaN7hfPWoXMKbeT9vr9fgRG6L7G23Kkzy4t2JQJmxZIfZ3izF/CPMawjsrh9GfCFjU3VU vRyUM/2VAVlYyMEg40am8pibFx2lK18= ARC-Authentication-Results: i=1; imf19.hostedemail.com; dkim=pass header.d=redhat.com header.s=mimecast20190719 header.b=iV7WMMps; dmarc=pass (policy=none) header.from=redhat.com; spf=pass (imf19.hostedemail.com: domain of dhowells@redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=dhowells@redhat.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1711643680; a=rsa-sha256; cv=none; b=I2donI8r0gTukKjNqYt6SvWTlPBL1TIpIlZUCZrjsPjSO/D5z++nTrqTixEIIYzrdlzELl 0Xeosfi4yBYxJiZV7ETSFSiS+dTew9AxYH5csT0k//K9Ckx3Sw3h+6IeZhz5a40kTCUdoK 34+irhPgwrxGSvnNMUSDS+dGCRSeFI8= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1711643679; 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; bh=ffhOAbislEtDui9sz/UMzHfpFDxhB+Cw1cNj4X99Z0A=; b=iV7WMMpshcKzMlSPsuEjVxcVvXBFDI7LKV1O19Z2GoHa9hW8CSxYrJ53fusdjsToCwctsP 25MZ/6fWtBEWswPg76sr3EvjaFqpwC+6SENhVVKa9KBBCqxQPoU/T8lmazoBegX3lZ53p4 ZZYG+o8RdafLmoxFQhfjvWaHgGjjRkg= Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-452-2dgsIfgmMnmMsc1zau3KSw-1; Thu, 28 Mar 2024 12:34:36 -0400 X-MC-Unique: 2dgsIfgmMnmMsc1zau3KSw-1 Received: from smtp.corp.redhat.com (int-mx09.intmail.prod.int.rdu2.redhat.com [10.11.54.9]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 1B630800262; Thu, 28 Mar 2024 16:34:35 +0000 (UTC) Received: from warthog.procyon.org.com (unknown [10.42.28.146]) by smtp.corp.redhat.com (Postfix) with ESMTP id D4330492BC6; Thu, 28 Mar 2024 16:34:31 +0000 (UTC) From: David Howells To: Christian Brauner , Jeff Layton , Gao Xiang , Dominique Martinet Cc: David Howells , Matthew Wilcox , Steve French , Marc Dionne , Paulo Alcantara , Shyam Prasad N , Tom Talpey , Eric Van Hensbergen , Ilya Dryomov , netfs@lists.linux.dev, linux-cachefs@redhat.com, linux-afs@lists.infradead.org, linux-cifs@vger.kernel.org, linux-nfs@vger.kernel.org, ceph-devel@vger.kernel.org, v9fs@lists.linux.dev, linux-erofs@lists.ozlabs.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 00/26] netfs, afs, 9p, cifs: Rework netfs to use ->writepages() to copy to cache Date: Thu, 28 Mar 2024 16:33:52 +0000 Message-ID: <20240328163424.2781320-1-dhowells@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.4.1 on 10.11.54.9 X-Rspamd-Server: rspam09 X-Rspamd-Queue-Id: 57B0B1A0008 X-Stat-Signature: 9cmuhwe5rhftc1zdzsqhhr8co9kcmpu9 X-Rspam-User: X-HE-Tag: 1711643680-919837 X-HE-Meta: U2FsdGVkX19zNl69kwAJ9q3gEoYOxZrZSrEH0JNoMbEpD7yw20pkjMiDgGx52b8ilfK3gMnb44xcjtT0tBFsGppusVxYtFc2iEGuUQuotSwUyvQkUuGl3uTCbPOaQxwcriDQZBv4PDZ+frEJRZqMgM6Inn7rkue5Rcd3cjzyuE6PZFJPkcdE02Q91pJgjgRFKCLKqST4z35HyFYn/OblNozx47a3s2CUP8+NcogpTOcQzw0z9WiwlQKEifVKO0qNTUwvbKVfNmFiH0zYjgThBVeTDaGpa/HtYAJv+K3yJ6KX2gPo0QJRVOFj/DLhuza7+w9/KaxIJo76ZkdIqrHPsbBo3/4VMokYlpKLoTPtg2C5udUnKgzOKZVrjUPeN8C1dIAkoXxI6tB3W9odyts4CyzDC1+eirUJa74WCdczujHKlL8sMmvWmuwjcKRhYRSFYya6OTH0TO3oisOe6qbzX/dqGQpKpt2XrgQGkyY0qfVtd6xWnsT1YlLMzq35RWRuGoWNPDMLevibbYnFX9itfE+xzfmRNqywCRdjHbQ/Fz5u9WtzjQFyfxgzs+UBqxn9nMQkNb6v3Dcv2KO32hRbe1tzHmlnKqz65A1h4H0UNKBX40rutRchMW+usI+988n45BaAfDhqFJO/ovl4mFmK2H016UWFrk5MFMIBciABqG7QfX43TgdNnbbemYOqpQxp4mMlgjKJQtVyWLmw6pnLeFNQ+VX8+yb+0Vswcbh86a8bGPV5hp6Do7Bx3zdIX6ORChzp3HKFpWoA6ok8Xe+UZmcC8l6gnmJUYLg2L07vFEbwE8rcHoEUGkqeBcpHbyJkgSGWnq5+ogTd1i956r2fdTgtF8Ap85sEMvXnmaejlpBuo7r/5I9dKtYMtRLEhv+oomhebd/hpqoUuCxcVzU2tviINQSZgfKbrspKn614iasqmu13ct7cHupp66zh2DrRRByL5DlEiIkF0S+sHHY xwL6Mc5i 59/eYTqvClYXbd/Jq0lZ+qzkHuSEXShvVNuTOjcgQ/bxURw0AKOfEe4ksSo6zeYqnsvT8CxUb79eP1tb494XQPWhaGU3NOQtLIYCErAjnXC0taeXHoh2HjWFDmNzWVsd0rQkhfV5KhslEQLDbBV//kfIMPw== 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: List-Subscribe: List-Unsubscribe: Hi Christian, Willy, The primary purpose of these patches is to rework the netfslib writeback implementation such that pages read from the cache are written to the cache through ->writepages(), thereby allowing the fscache page flag to be retired. The reworking also: (1) builds on top of the new writeback_iter() infrastructure; (2) makes it possible to use vectored write RPCs as discontiguous streams of pages can be accommodated; (3) makes it easier to do simultaneous content crypto and stream division. (4) provides support for retrying writes and re-dividing a stream; (5) replaces the ->launder_folio() op, so that ->writepages() is used instead; (6) uses mempools to allocate the netfs_io_request and netfs_io_subrequest structs to avoid allocation failure in the writeback path. Some code that uses the fscache page flag is retained for compatibility purposes with nfs and ceph. The code is switched to using the synonymous private_2 label instead and marked with deprecation comments. I have a separate set of patches that convert cifs to use this code. -~- In this new implementation, writeback_iter() is used to pump folios, progressively creating two parallel, but separate streams. Either or both streams can contain gaps, and the subrequests in each stream can be of variable size, don't need to align with each other and don't need to align with the folios. (Note that more streams can be added if we have multiple servers to duplicate data to). Indeed, subrequests can cross folio boundaries, may cover several folios or a folio may be spanned by multiple subrequests, e.g.: +---+---+-----+-----+---+----------+ Folios: | | | | | | | +---+---+-----+-----+---+----------+ +------+------+ +----+----+ Upload: | | |.....| | | +------+------+ +----+----+ +------+------+------+------+------+ Cache: | | | | | | +------+------+------+------+------+ Data that got read from the server that needs copying to the cache is stored in folios that are marked dirty and have folio->private set to a special value. The progressive subrequest construction permits the algorithm to be preparing both the next upload to the server and the next write to the cache whilst the previous ones are already in progress. Throttling can be applied to control the rate of production of subrequests - and, in any case, we probably want to write them to the server in ascending order, particularly if the file will be extended. Content crypto can also be prepared at the same time as the subrequests and run asynchronously, with the prepped requests being stalled until the crypto catches up with them. This might also be useful for transport crypto, but that happens at a lower layer, so probably would be harder to pull off. The algorithm is split into three parts: (1) The issuer. This walks through the data, packaging it up, encrypting it and creating subrequests. The part of this that generates subrequests only deals with file positions and spans and so is usable for DIO/unbuffered writes as well as buffered writes. (2) The collector. This asynchronously collects completed subrequests, unlocks folios, frees crypto buffers and performs any retries. This runs in a work queue so that the issuer can return to the caller for writeback (so that the VM can have its kswapd thread back) or async writes. Collection is slightly complex as the collector has to work out where discontiguities happen in the folio list so that it doesn't try and collect folios that weren't included in the write out. (3) The retryer. This pauses the issuer, waits for all outstanding subrequests to complete and then goes through the failed subrequests to reissue them. This may involve reprepping them (with cifs, the credits must be renegotiated and a subrequest may need splitting), and doing RMW for content crypto if there's a conflicting change on the server. David David Howells (26): cifs: Fix duplicate fscache cookie warnings 9p: Clean up some kdoc and unused var warnings. netfs: Update i_blocks when write committed to pagecache netfs: Replace PG_fscache by setting folio->private and marking dirty mm: Remove the PG_fscache alias for PG_private_2 netfs: Remove deprecated use of PG_private_2 as a second writeback flag netfs: Make netfs_io_request::subreq_counter an atomic_t netfs: Use subreq_counter to allocate subreq debug_index values mm: Provide a means of invalidation without using launder_folio cifs: Use alternative invalidation to using launder_folio 9p: Use alternative invalidation to using launder_folio afs: Use alternative invalidation to using launder_folio netfs: Remove ->launder_folio() support netfs: Use mempools for allocating requests and subrequests mm: Export writeback_iter() netfs: Switch to using unsigned long long rather than loff_t netfs: Fix writethrough-mode error handling netfs: Add some write-side stats and clean up some stat names netfs: New writeback implementation netfs, afs: Implement helpers for new write code netfs, 9p: Implement helpers for new write code netfs, cachefiles: Implement helpers for new write code netfs: Cut over to using new writeback code netfs: Remove the old writeback code netfs: Miscellaneous tidy ups netfs, afs: Use writeback retry to deal with alternate keys fs/9p/vfs_addr.c | 60 +-- fs/9p/vfs_inode_dotl.c | 4 - fs/afs/file.c | 8 +- fs/afs/internal.h | 6 +- fs/afs/validation.c | 4 +- fs/afs/write.c | 187 ++++---- fs/cachefiles/io.c | 75 +++- fs/ceph/addr.c | 24 +- fs/ceph/inode.c | 2 + fs/netfs/Makefile | 3 +- fs/netfs/buffered_read.c | 40 +- fs/netfs/buffered_write.c | 832 ++++------------------------------- fs/netfs/direct_write.c | 30 +- fs/netfs/fscache_io.c | 14 +- fs/netfs/internal.h | 55 ++- fs/netfs/io.c | 155 +------ fs/netfs/main.c | 55 ++- fs/netfs/misc.c | 10 +- fs/netfs/objects.c | 81 +++- fs/netfs/output.c | 478 -------------------- fs/netfs/stats.c | 17 +- fs/netfs/write_collect.c | 813 ++++++++++++++++++++++++++++++++++ fs/netfs/write_issue.c | 673 ++++++++++++++++++++++++++++ fs/nfs/file.c | 8 +- fs/nfs/fscache.h | 6 +- fs/nfs/write.c | 4 +- fs/smb/client/cifsfs.h | 1 - fs/smb/client/file.c | 136 +----- fs/smb/client/fscache.c | 16 +- fs/smb/client/inode.c | 27 +- include/linux/fscache.h | 22 +- include/linux/netfs.h | 196 +++++---- include/linux/pagemap.h | 1 + include/net/9p/client.h | 2 + include/trace/events/netfs.h | 249 ++++++++++- mm/filemap.c | 52 ++- mm/page-writeback.c | 1 + net/9p/Kconfig | 1 + net/9p/client.c | 49 +++ net/9p/trans_fd.c | 1 - 40 files changed, 2492 insertions(+), 1906 deletions(-) delete mode 100644 fs/netfs/output.c create mode 100644 fs/netfs/write_collect.c create mode 100644 fs/netfs/write_issue.c