gfs2: Truncate address space when flipping GFS2_DIF_JDATA flag

Andreas Gruenbacher · gregkh · commit 2a40a140e11f · 2025-02-01T18:30:09.000+01:00
commit 7c9d9223802fbed4dee1ae301661bf346964c9d2 upstream.

Truncate an inode's address space when flipping the GFS2_DIF_JDATA flag:
depending on that flag, the pages in the address space will either use
buffer heads or iomap_folio_state structs, and we cannot mix the two.

Reported-by: Kun Hu &lt;huk23@m.fudan.edu.cn&gt;, Jiaji Qin &lt;jjtan24@m.fudan.edu.cn&gt;
Signed-off-by: Andreas Gruenbacher &lt;agruenba@redhat.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
diff --git a/fs/gfs2/file.c b/fs/gfs2/file.c
@@ -250,6 +250,7 @@ static int do_gfs2_set_flags(struct inode *inode, u32 reqflags, u32 mask)
 		error = filemap_fdatawait(inode->i_mapping);
 		if (error)
 			goto out;
+		truncate_inode_pages(inode->i_mapping, 0);
 		if (new_flags & GFS2_DIF_JDATA)
 			gfs2_ordered_del_inode(ip);
 	}

Original file line number	Diff line number	Diff line change
`@@ -250,6 +250,7 @@ static int do_gfs2_set_flags(struct inode *inode, u32 reqflags, u32 mask)`
`250`	`250`	`error = filemap_fdatawait(inode->i_mapping);`
`251`	`251`	`if (error)`
`252`	`252`	`goto out;`
	`253`	`+ truncate_inode_pages(inode->i_mapping, 0);`
`253`	`254`	`if (new_flags & GFS2_DIF_JDATA)`
`254`	`255`	`gfs2_ordered_del_inode(ip);`
`255`	`256`	`}`