Messages in this thread |  | | From | (Linus Torvalds) | Subject | Re: [PATCH] filemap_fdatasync & related changes | Date | 29 Dec 2000 13:13:55 -0800 |
| |
In article <Pine.LNX.4.21.0012291446560.13194-100000@freak.distro.conectiva>, Marcelo Tosatti <marcelo@conectiva.com.br> wrote: > >Ok, here it is. > >I hope I got all locking and all special cases right. > >Comments ?
Looks good.
There's a few things this misses, the worst of which were all my bugs in the original description. Things like "don't unlock the page after calling writepage, becasue writepage will do it on its own". Details like that.
The worst problem actually ends up being the fact that the global sync() doesn't work, after all, because if it's associated with syncing the inodes it will only sync the dirty page list for inodes that are dirty. Now, this probably doesn't show up in testing, because mostly if you have dirty pages the inode too _will_ be dirty, but that only makes the bug more insidious.
So I'll work on this a bit to polish up these problems, but on the whole this is all fine.
Linus - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org Please read the FAQ at http://www.tux.org/lkml/
|  |