Commit fee5150c authored Oct 10, 2018 by Andreas Gruenbacher

gfs2: Fix iomap buffered write support for journaled files (2)



It turns out that the fix in commit 6636c3cc56 is bad; the assertion
that the iomap code no longer creates buffer heads is incorrect for
filesystems that set the IOMAP_F_BUFFER_HEAD flag.

Instead, what's happening is that gfs2_iomap_begin_write treats all
files that have the jdata flag set as journaled files, which is
incorrect as long as those files are inline ("stuffed").  We're handling
stuffed files directly via the page cache, which is why we ended up with
pages without buffer heads in gfs2_page_add_databufs.

Fix this by handling stuffed journaled files correctly in
gfs2_iomap_begin_write.

This reverts commit 6636c3cc5690c11631e6366cf9a28fb99c8b25bb.

Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>

parent dc480feb

fs/gfs2/bmap.c

+1 −5

Original line number	Diff line number	Diff line
		@@ -975,10 +975,6 @@ static void gfs2_iomap_journaled_page_done(struct inode *inode, loff_t pos,
		{
		struct gfs2_inode *ip = GFS2_I(inode);

		if (!page_has_buffers(page)) {
		create_empty_buffers(page, inode->i_sb->s_blocksize,
		(1 << BH_Dirty)\|(1 << BH_Uptodate));
		}
		gfs2_page_add_databufs(ip, page, offset_in_page(pos), copied);
		}

		@@ -1061,7 +1057,7 @@ static int gfs2_iomap_begin_write(struct inode *inode, loff_t pos,
		}
		}
		release_metapath(&mp);
		if (gfs2_is_jdata(ip))
		if (!gfs2_is_stuffed(ip) && gfs2_is_jdata(ip))
		iomap->page_done = gfs2_iomap_journaled_page_done;
		return 0;

Admin message