diff options
author | Yan Zheng <zheng.yan@oracle.com> | 2009-01-05 15:43:42 -0500 |
---|---|---|
committer | Chris Mason <chris.mason@oracle.com> | 2009-01-05 15:43:42 -0500 |
commit | ec051c0f929afe5c42c24bb07abf577c616c208c (patch) | |
tree | a9b75cd422cf7dde92c9dbb8828feb062e2c35ec | |
parent | 2d69a0f88459fae35df3ddef4934a2dad67e2765 (diff) |
Btrfs: avoid orphan inode caused by log replay
drop_one_dir_item does not properly update inode's link count. It can be
reproduced by executing following commands:
#touch test
#sync
#rm -f test
#dd if=/dev/zero bs=4k count=1 of=test conv=fsync
#echo b > /proc/sysrq-trigger
This fixes it by adding an BTRFS_ORPHAN_ITEM_KEY for the inode
Signed-off-by: Yan Zheng <zheng.yan@oracle.com>
-rw-r--r-- | fs/btrfs/tree-log.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/fs/btrfs/tree-log.c b/fs/btrfs/tree-log.c index 33eee256ee81..b1c2921f5bef 100644 --- a/fs/btrfs/tree-log.c +++ b/fs/btrfs/tree-log.c | |||
@@ -50,6 +50,9 @@ | |||
50 | static int __btrfs_log_inode(struct btrfs_trans_handle *trans, | 50 | static int __btrfs_log_inode(struct btrfs_trans_handle *trans, |
51 | struct btrfs_root *root, struct inode *inode, | 51 | struct btrfs_root *root, struct inode *inode, |
52 | int inode_only); | 52 | int inode_only); |
53 | static int link_to_fixup_dir(struct btrfs_trans_handle *trans, | ||
54 | struct btrfs_root *root, | ||
55 | struct btrfs_path *path, u64 objectid); | ||
53 | 56 | ||
54 | /* | 57 | /* |
55 | * tree logging is a special write ahead log used to make sure that | 58 | * tree logging is a special write ahead log used to make sure that |
@@ -638,8 +641,10 @@ static noinline int drop_one_dir_item(struct btrfs_trans_handle *trans, | |||
638 | inode = read_one_inode(root, location.objectid); | 641 | inode = read_one_inode(root, location.objectid); |
639 | BUG_ON(!inode); | 642 | BUG_ON(!inode); |
640 | 643 | ||
641 | btrfs_inc_nlink(inode); | 644 | ret = link_to_fixup_dir(trans, root, path, location.objectid); |
645 | BUG_ON(ret); | ||
642 | ret = btrfs_unlink_inode(trans, root, dir, inode, name, name_len); | 646 | ret = btrfs_unlink_inode(trans, root, dir, inode, name, name_len); |
647 | BUG_ON(ret); | ||
643 | kfree(name); | 648 | kfree(name); |
644 | 649 | ||
645 | iput(inode); | 650 | iput(inode); |