Gitweb:
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=6f16bf655c5795586dd2ac96a7c70e0b9a378746
Commit: 6f16bf655c5795586dd2ac96a7c70e0b9a378746
Parent: 68e2b740c4b5394680cfefccddbdb486c5866a4c
Author: Mark Fasheh <[EMAIL PROTECTED]>
AuthorDate: Tue Mar 20 17:17:54 2007 -0700
Committer: Mark Fasheh <[EMAIL PROTECTED]>
CommitDate: Thu Apr 26 14:40:55 2007 -0700
ocfs2: small cleanup of ocfs2_request_delete()
There are two checks in there (one for inode newness, one for other mounted
nodes) which are unnecessary, so remove them. The DLM will allow the trylock
in either case without any messaging overhead.
Removing these makes ocfs2_request_delete() a one liner function, so just
move the trylock out one level into ocfs2_query_inode_wipe().
Signed-off-by: Mark Fasheh <[EMAIL PROTECTED]>
---
fs/ocfs2/inode.c | 46 +++++++++++++---------------------------------
1 files changed, 13 insertions(+), 33 deletions(-)
diff --git a/fs/ocfs2/inode.c b/fs/ocfs2/inode.c
index e1e4882..08d57a3 100644
--- a/fs/ocfs2/inode.c
+++ b/fs/ocfs2/inode.c
@@ -767,35 +767,6 @@ bail:
return ret;
}
-static int ocfs2_request_delete(struct inode *inode)
-{
- int status = 0;
- struct ocfs2_super *osb = OCFS2_SB(inode->i_sb);
-
- if (ocfs2_inode_is_new(inode))
- return 0;
-
- if (ocfs2_node_map_is_only(osb, &osb->mounted_map,
- osb->node_num))
- return 0;
- /*
- * This is how ocfs2 determines whether an inode is still live
- * within the cluster. Every node takes a shared read lock on
- * the inode open lock in ocfs2_read_locked_inode(). When we
- * get to ->delete_inode(), each node tries to convert it's
- * lock to an exclusive. Trylocks are serialized by the inode
- * meta data lock. If the upconvert suceeds, we know the inode
- * is no longer live and can be deleted.
- *
- * Though we call this with the meta data lock held, the
- * trylock keeps us from ABBA deadlock.
- */
- status = ocfs2_try_open_lock(inode, 1);
- if (status < 0 && status != -EAGAIN)
- mlog_errno(status);
- return status;
-}
-
/* Query the cluster to determine whether we should wipe an inode from
* disk or not.
*
@@ -848,10 +819,19 @@ static int ocfs2_query_inode_wipe(struct inode *inode,
goto bail;
}
- status = ocfs2_request_delete(inode);
- /* -EAGAIN means that other nodes are still using the
- * inode. We're done here though, so avoid doing anything on
- * disk and let them worry about deleting it. */
+ /*
+ * This is how ocfs2 determines whether an inode is still live
+ * within the cluster. Every node takes a shared read lock on
+ * the inode open lock in ocfs2_read_locked_inode(). When we
+ * get to ->delete_inode(), each node tries to convert it's
+ * lock to an exclusive. Trylocks are serialized by the inode
+ * meta data lock. If the upconvert suceeds, we know the inode
+ * is no longer live and can be deleted.
+ *
+ * Though we call this with the meta data lock held, the
+ * trylock keeps us from ABBA deadlock.
+ */
+ status = ocfs2_try_open_lock(inode, 1);
if (status == -EAGAIN) {
status = 0;
mlog(0, "Skipping delete of %llu because it is in use on"
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
