gfs2: Truncate address space when flipping GFS2_DIF_JDATA flag

Andreas Gruenbacher · mehmetb0 · commit ee38e6b15b61 · 2025-02-14T15:17:40.000+03:00
BugLink: https://bugs.launchpad.net/bugs/2098441 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 <huk23@m.fudan.edu.cn>, Jiaji Qin <jjtan24@m.fudan.edu.cn> Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Noah Wager <noah.wager@canonical.com> Signed-off-by: Koichiro Den <koichiro.den@canonical.com>
diff --git a/fs/gfs2/file.c b/fs/gfs2/file.c
@@ -257,6 +257,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
`@@ -257,6 +257,7 @@ static int do_gfs2_set_flags(struct inode *inode, u32 reqflags, u32 mask,`
`257`	`257`	`error = filemap_fdatawait(inode->i_mapping);`
`258`	`258`	`if (error)`
`259`	`259`	`goto out;`
	`260`	`+ truncate_inode_pages(inode->i_mapping, 0);`
`260`	`261`	`if (new_flags & GFS2_DIF_JDATA)`
`261`	`262`	`gfs2_ordered_del_inode(ip);`
`262`	`263`	`}`