From mboxrd@z Thu Jan 1 00:00:00 1970 From: Darrick J. Wong Date: Thu, 8 Jul 2021 21:23:45 -0700 Subject: [Cluster-devel] [PATCH v3 1/3] iomap: Permit pages without an iop to enter writeback In-Reply-To: <20210707115524.2242151-2-agruenba@redhat.com> References: <20210707115524.2242151-1-agruenba@redhat.com> <20210707115524.2242151-2-agruenba@redhat.com> Message-ID: <20210709042345.GS11588@locust> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit On Wed, Jul 07, 2021 at 01:55:22PM +0200, Andreas Gruenbacher wrote: > Create an iop in the writeback path if one doesn't exist. This allows us > to avoid creating the iop in some cases. We'll initially do that for pages > with inline data, but it can be extended to pages which are entirely within > an extent. It also allows for an iop to be removed from pages in the > future (eg page split). > > Co-developed-by: Matthew Wilcox (Oracle) > Signed-off-by: Matthew Wilcox (Oracle) > Signed-off-by: Andreas Gruenbacher > Reviewed-by: Christoph Hellwig Seems simple enough... Reviewed-by: Darrick J. Wong --D > --- > fs/iomap/buffered-io.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/fs/iomap/buffered-io.c b/fs/iomap/buffered-io.c > index 9023717c5188..598fcfabc337 100644 > --- a/fs/iomap/buffered-io.c > +++ b/fs/iomap/buffered-io.c > @@ -1334,14 +1334,13 @@ iomap_writepage_map(struct iomap_writepage_ctx *wpc, > struct writeback_control *wbc, struct inode *inode, > struct page *page, u64 end_offset) > { > - struct iomap_page *iop = to_iomap_page(page); > + struct iomap_page *iop = iomap_page_create(inode, page); > struct iomap_ioend *ioend, *next; > unsigned len = i_blocksize(inode); > u64 file_offset; /* file offset of page */ > int error = 0, count = 0, i; > LIST_HEAD(submit_list); > > - WARN_ON_ONCE(i_blocks_per_page(inode, page) > 1 && !iop); > WARN_ON_ONCE(iop && atomic_read(&iop->write_bytes_pending) != 0); > > /* > -- > 2.26.3 >