On Wed, Jan 15, 2020 at 10:19:25AM +0100, Jan Kara wrote: > On Tue 14-01-20 08:38:18, Christoph Hellwig wrote: > > On Tue, Jan 14, 2020 at 10:05:07AM +0100, Jan Kara wrote: > > > > > > Well, XFS always performs buffered writeback using unwritten extents so at > > > least the immediate problem of stale data exposure ext4 has does not happen > > > there AFAICT. > > > > Currently XFS never uses unwritten extents when converting delalloc > > extents. > > I see, it is a long time since I last looked at that part of XFS code. So > then I think XFS might be prone to the same kind of race and data exposure > as I outlined in [1]... I think not using unwritten extents for filling holes inside i_size will always lead to the potential for stale data exposure in one form or another. Because of that Darrick has started looking into always using unwritten extents for buffered writes inside i_size.
