From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Date: Tue, 4 Apr 2023 08:32:14 -0700 Subject: [Cluster-devel] [PATCH v2 08/23] iomap: hoist iomap_readpage_ctx from the iomap_readahead/_folio In-Reply-To: <20230404145319.2057051-9-aalbersh@redhat.com> References: <20230404145319.2057051-1-aalbersh@redhat.com> <20230404145319.2057051-9-aalbersh@redhat.com> Message-ID: List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit On Tue, Apr 04, 2023 at 04:53:04PM +0200, Andrey Albershteyn wrote: > Make filesystems create readpage context, similar as > iomap_writepage_ctx in write path. This will allow filesystem to > pass _ops to iomap for ioend configuration (->prepare_ioend) which > in turn would be used to set BIO end callout (bio->bi_end_io). > > This will be utilized in further patches by fs-verity to verify > pages on BIO completion by XFS. Hmm. Can't we just have a version of the readpage helper that just gets the ops passed instead of creating all this boilerplate code? For writepage XFS embedds the context in it's own structure, so it's kindof needed, but I don't think we'll need that here.