diff options
| author | Jan Kara <jack@suse.cz> | 2009-01-15 16:51:29 -0500 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-01-15 19:39:42 -0500 |
| commit | 6b7021ef7e1a703c7092daeceda063951b22b4f6 (patch) | |
| tree | ee2a92121f102bc8e948c17a3fa4a357f6687f2a | |
| parent | 634a84f8d5b74da497688d3346f6809c28239eda (diff) | |
ext2: also update the inode on disk when dir is IS_DIRSYNC
We used to just write changed page for IS_DIRSYNC inodes. But we also
have to update the directory inode itself just for the case that we've
allocated a new block and changed i_size.
[akpm@linux-foundation.org: still sync the data page]
Signed-off-by: Jan Kara <jack@suse.cz>
Tested-by: Pavel Machek <pavel@suse.cz>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
| -rw-r--r-- | fs/ext2/dir.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/fs/ext2/dir.c b/fs/ext2/dir.c index 9a0fc400f91..2999d72153b 100644 --- a/fs/ext2/dir.c +++ b/fs/ext2/dir.c | |||
| @@ -95,10 +95,13 @@ static int ext2_commit_chunk(struct page *page, loff_t pos, unsigned len) | |||
| 95 | mark_inode_dirty(dir); | 95 | mark_inode_dirty(dir); |
| 96 | } | 96 | } |
| 97 | 97 | ||
| 98 | if (IS_DIRSYNC(dir)) | 98 | if (IS_DIRSYNC(dir)) { |
| 99 | err = write_one_page(page, 1); | 99 | err = write_one_page(page, 1); |
| 100 | else | 100 | if (!err) |
| 101 | err = ext2_sync_inode(dir); | ||
| 102 | } else { | ||
| 101 | unlock_page(page); | 103 | unlock_page(page); |
| 104 | } | ||
| 102 | 105 | ||
| 103 | return err; | 106 | return err; |
| 104 | } | 107 | } |
