[PATCH 2/2] xfs_repair: Properly set lost+found inode as used
Carlos Maiolino
cmaiolino at redhat.com
Tue Nov 8 12:46:30 CST 2011
This patch makes mk_orphanage() to properly set the inode link count of
the recently allocated inode in the AVL tree, avoiding the lost+found
directory to be bypass the link count check in phase7 and possibly leaving
lost+found directory with a wrong link count.
---
repair/phase6.c | 17 ++++++++++++++++-
1 files changed, 16 insertions(+), 1 deletions(-)
diff --git a/repair/phase6.c b/repair/phase6.c
index adad61d..0e0e294 100644
--- a/repair/phase6.c
+++ b/repair/phase6.c
@@ -823,6 +823,8 @@ mk_orphanage(xfs_mount_t *mp)
xfs_inode_t *ip;
xfs_inode_t *pip;
xfs_fsblock_t first;
+ ino_tree_node_t *irec;
+ int ino_offset = 0;
int i;
int committed;
int error;
@@ -875,6 +877,18 @@ mk_orphanage(xfs_mount_t *mp)
ORPHANAGE, error);
}
ip->i_d.di_nlink++; /* account for . */
+ ino = ip->i_ino;
+
+ irec = find_inode_rec(mp,
+ XFS_INO_TO_AGNO(mp, ino),
+ XFS_INO_TO_AGINO(mp,ino));
+ ino_offset = get_inode_offset(mp, ino, irec);
+
+ /* Set the inode allocated to lost+found as used in the AVL
+ * tree, so it is not bypassed in phase 7
+ */
+ set_inode_used(irec,ino_offset);
+ add_inode_ref(irec,ino_offset);
/*
* now that we know the transaction will stay around,
@@ -902,6 +916,8 @@ mk_orphanage(xfs_mount_t *mp)
XFS_INO_TO_AGINO(mp, mp->m_sb.sb_rootino)), 0);
+ add_inode_reached(irec,ino_offset);
+
libxfs_trans_log_inode(tp, pip, XFS_ILOG_CORE);
libxfs_dir_init(tp, ip, pip);
libxfs_trans_log_inode(tp, ip, XFS_ILOG_CORE);
@@ -912,7 +928,6 @@ mk_orphanage(xfs_mount_t *mp)
ORPHANAGE, error);
}
- ino = ip->i_ino;
libxfs_trans_commit(tp, XFS_TRANS_RELEASE_LOG_RES|XFS_TRANS_SYNC);
--
1.7.6.4
More information about the xfs
mailing list