From: Jan Kara Date: Mon, 29 May 2017 17:24:55 +0000 (-0400) Subject: ext4: fix fdatasync(2) after extent manipulation operations X-Git-Tag: v3.2.93~55 X-Git-Url: http://git.openpandora.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=99da954f8787a01791c59517caa7aa3cc28eb0ae;p=pandora-kernel.git ext4: fix fdatasync(2) after extent manipulation operations commit 67a7d5f561f469ad2fa5154d2888258ab8e6df7c upstream. Currently, extent manipulation operations such as hole punch, range zeroing, or extent shifting do not record the fact that file data has changed and thus fdatasync(2) has a work to do. As a result if we crash e.g. after a punch hole and fdatasync, user can still possibly see the punched out data after journal replay. Test generic/392 fails due to these problems. Fix the problem by properly marking that file data has changed in these operations. Fixes: a4bb6b64e39abc0e41ca077725f2a72c868e7622 Signed-off-by: Jan Kara Signed-off-by: Theodore Ts'o [bwh: Backported to 3.2: Only the punch-hole operation is supported, and it's in extents.c.] Signed-off-by: Ben Hutchings --- diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c index bb40a70ed412..aa54e82d74c3 100644 --- a/fs/ext4/extents.c +++ b/fs/ext4/extents.c @@ -5023,6 +5023,8 @@ out: ext4_orphan_del(handle, inode); inode->i_mtime = inode->i_ctime = ext4_current_time(inode); ext4_mark_inode_dirty(handle, inode); + if (err >= 0) + ext4_update_inode_fsync_trans(handle, inode, 1); ext4_journal_stop(handle); return err; }