Re: [PATCH, RFC] xfs: add heuristic to flush on rename

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Apr 25, 2014 at 02:42:21PM -0500, Eric Sandeen wrote:
> Add a heuristic to flush data to a file which looks like it's
> going through a tmpfile/rename dance, but not fsynced.
> 
> I had a report of a system with many 0-length files after
> package updates; as it turns out, the user had basically
> done 'yum update' and punched the power button when it was
> done.

So yum didn't run sync() on completion of the update? That seems
rather dangerous to me - IMO system updates need to be guaranteed to
be stable by the update mechanisms, not to leave the system state to
chance if power fails or the system crashes immediately after an
update...


> Granted, the admin should not do this.  Granted, the package
> manager should ensure persistence of files it updated.

Yes, yes it should. Problem solved without needing to touch XFS.

> Ext4, however, added a heuristic like this for just this case;
> someone who writes file.tmp, then renames over file, but
> never issues an fsync.

You mean like rsync does all the time for every file it copies?

> Now, this does smack of O_PONIES, but I would hope that it's
> fairly benign.  If someone already synced the tmpfile, it's
> a no-op.

I'd suggest it will greatly impact rsync speed and have impact on
the resultant filesystem layout as it guarantees interleaving of
metadata and data on disk....

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx

_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs




[Index of Archives]     [Linux XFS Devel]     [Linux Filesystem Development]     [Filesystem Testing]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux