aboutsummaryrefslogtreecommitdiffstats
path: root/fs/ext2/super.c
diff options
context:
space:
mode:
authorBernard Blackham <bernard@blackham.com.au>2005-04-16 18:25:45 -0400
committerLinus Torvalds <torvalds@ppc970.osdl.org>2005-04-16 18:25:45 -0400
commite072c6f2af57fb8ad9e0f29bfff3f79edf7bdd55 (patch)
tree9d72262a63754b39df4ebfed5bc74855f0408c3a /fs/ext2/super.c
parent614a7d6a76b7fb37bb399047eb3ccf86cafbf60d (diff)
[PATCH] ext2 corruption - regression between 2.6.9 and 2.6.10
Whilst trying to stress test a Promise SX8 card, we stumbled across some nasty filesystem corruption in ext2. Our tests involved creating an ext2 partition, mounting, running several concurrent fsx's over it, umounting, and fsck'ing, all scripted[1]. The fsck would always return with errors. This regression was traced back to a change between 2.6.9 and 2.6.10, which moves the functionality of ext2_put_inode into ext2_clear_inode. The attached patch reverses this change, and eliminated the source of corruption. Mingming Cao <cmm@us.ibm.com> said: I think his patch for ext2 is correct. The corruption on ext3 is not the same issue he saw on ext2. I believe that's the race between discard reservation and reservation in-use that we already fixed it in 2.6.12- rc1. For the problem related to ext2, at the time when we design reservation for ext3, we decide we only need to discard the reservation at the last file close, so we have ext3_discard_reservation on iput_final- >ext3_clear_inode. The ext2 handle discard preallocation differently at that time, it discard the preallocation at each iput(), not in input_final(), so we think it's unnecessary to thrash it so frequently, and the right thing to do, as we did for ext3 reservation, discard preallocation on last iput(). So we moved the ext2_discard_preallocation from ext2_put_inode(0 to ext2_clear_inode. Since ext2 preallocation is doing pre-allocation on disk, so it is possible that at the unmount time, someone is still hold the reference of the inode, so the preallocation for a file is not discard yet, so we still mark those blocks allocated on disk, while they are not actually in the inode's block map, so fsck will catch/fix that error later. This is not a issue for ext3, as ext3 reservation(pre-allocation) is done in memory. Signed-off-by: Andrew Morton <akpm@osdl.org> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'fs/ext2/super.c')
-rw-r--r--fs/ext2/super.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/fs/ext2/super.c b/fs/ext2/super.c
index 37ca77a157ba..661c3d98d946 100644
--- a/fs/ext2/super.c
+++ b/fs/ext2/super.c
@@ -198,11 +198,8 @@ static void ext2_clear_inode(struct inode *inode)
198 ei->i_default_acl = EXT2_ACL_NOT_CACHED; 198 ei->i_default_acl = EXT2_ACL_NOT_CACHED;
199 } 199 }
200#endif 200#endif
201 if (!is_bad_inode(inode))
202 ext2_discard_prealloc(inode);
203} 201}
204 202
205
206#ifdef CONFIG_QUOTA 203#ifdef CONFIG_QUOTA
207static ssize_t ext2_quota_read(struct super_block *sb, int type, char *data, size_t len, loff_t off); 204static ssize_t ext2_quota_read(struct super_block *sb, int type, char *data, size_t len, loff_t off);
208static ssize_t ext2_quota_write(struct super_block *sb, int type, const char *data, size_t len, loff_t off); 205static ssize_t ext2_quota_write(struct super_block *sb, int type, const char *data, size_t len, loff_t off);
@@ -213,6 +210,7 @@ static struct super_operations ext2_sops = {
213 .destroy_inode = ext2_destroy_inode, 210 .destroy_inode = ext2_destroy_inode,
214 .read_inode = ext2_read_inode, 211 .read_inode = ext2_read_inode,
215 .write_inode = ext2_write_inode, 212 .write_inode = ext2_write_inode,
213 .put_inode = ext2_put_inode,
216 .delete_inode = ext2_delete_inode, 214 .delete_inode = ext2_delete_inode,
217 .put_super = ext2_put_super, 215 .put_super = ext2_put_super,
218 .write_super = ext2_write_super, 216 .write_super = ext2_write_super,