Re: vfat BKL/lock_super regression in v2.6.26-rc3-g8f59342

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Linus Torvalds
Date: Tuesday, August 19, 2008 - 5:56 pm

On Tue, 19 Aug 2008, Linus Torvalds wrote:

.. and by "finished the write" I mean "closed the file", not "end of 
write() system call". Ie it's one of those things where it really doesn't 
mostly make much sense to try to give any kinds of flush guarantees until 
the user has basically shown that he's all done with writing.

If you have something like removable media, and you actually remove it 
while you have a "cp -R" in progress, it damn well won't matter whether we 
were synchronous or not. But if you remove it after the "cp" has actually 
finished, it's a lot more understandable if somebody expects it to be on 
disk.

So one thing we could perhaps consider is to make FAT in particular 
consider "sync" mounts to be about open/close consistency, not about 
per-write-system-call consistency. So the "close()" wouldn't return until 
the file is on disk, but we wouldn't force a synchronous rewrite the inode 
or the file allocation table thousands of times just because the file was 
big.

FAT really is kind of different. I suspect we could just change what 
"sync" means for it. But it would probably be good to have a VFS-level 
notion of open-close consistency. It is, after all, what NFS is already 
supposed to give you, so there is precedence for that being a useful IO 
serialization model.

Al, what do you think?

		Linus
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
vfat BKL/lock_super regression in v2.6.26-rc3-g8f59342, Bart Trojanowski, (Tue Aug 19, 3:03 pm)
[PATCH] make lock_super recursive to simulate BKL, Bart Trojanowski, (Tue Aug 19, 3:06 pm)
Re: vfat BKL/lock_super regression in v2.6.26-rc3-g8f59342, Linus Torvalds, (Tue Aug 19, 3:17 pm)
Re: [PATCH] make lock_super recursive to simulate BKL, Linus Torvalds, (Tue Aug 19, 3:21 pm)
Re: vfat BKL/lock_super regression in v2.6.26-rc3-g8f59342, Bart Trojanowski, (Tue Aug 19, 5:03 pm)
Re: vfat BKL/lock_super regression in v2.6.26-rc3-g8f59342, Linus Torvalds, (Tue Aug 19, 5:11 pm)
Re: vfat BKL/lock_super regression in v2.6.26-rc3-g8f59342, Bart Trojanowski, (Tue Aug 19, 5:18 pm)
Re: vfat BKL/lock_super regression in v2.6.26-rc3-g8f59342, Bart Trojanowski, (Tue Aug 19, 5:24 pm)
Re: vfat BKL/lock_super regression in v2.6.26-rc3-g8f59342, Linus Torvalds, (Tue Aug 19, 5:43 pm)
Re: vfat BKL/lock_super regression in v2.6.26-rc3-g8f59342, Linus Torvalds, (Tue Aug 19, 5:56 pm)
Re: [PATCH] make lock_super recursive to simulate BKL, Bart Trojanowski, (Tue Aug 19, 6:14 pm)
Re: vfat BKL/lock_super regression in v2.6.26-rc3-g8f59342, Bart Trojanowski, (Tue Aug 19, 7:27 pm)
[PATCH] document additional vfat mount options, Bart Trojanowski, (Fri Aug 22, 5:54 pm)
Re: [PATCH] document additional vfat mount options, Grant Coady, (Fri Aug 22, 7:33 pm)
Re: [PATCH] document additional vfat mount options, OGAWA Hirofumi, (Fri Aug 22, 8:10 pm)
Re: [PATCH] document additional vfat mount options, Bart Trojanowski, (Fri Aug 22, 8:12 pm)
[PATCH] document additional vfat mount options, Bart Trojanowski, (Fri Aug 22, 8:14 pm)
Re: [PATCH] document additional vfat mount options, OGAWA Hirofumi, (Fri Aug 22, 8:27 pm)
Re: [PATCH] document additional vfat mount options, Bart Trojanowski, (Sat Aug 23, 6:11 am)
Re: [PATCH] document additional vfat mount options, OGAWA Hirofumi, (Sat Aug 23, 7:47 am)