Commit bd62ad7a authored by Jan Kara's avatar Jan Kara Committed by Linus Torvalds

ocfs2: move dquot_initialize() in ocfs2_delete_inode() somewhat later

Move dquot_initalize() call in ocfs2_delete_inode() after the moment we
verify inode is actually a sane one to delete.  We certainly don't want
to initialize quota for system inodes etc.  This also avoids calling
into quota code from downconvert thread.

Add more details into the comment why bailing out from
ocfs2_delete_inode() when we are in downconvert thread is OK.
Signed-off-by: default avatarJan Kara <jack@suse.cz>
Reviewed-by: default avatarMark Fasheh <mfasheh@suse.de>
Reviewed-by: default avatarSrinivas Eeda <srinivas.eeda@oracle.com>
Cc: Joel Becker <jlbec@evilplan.org>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 7bf619c1
...@@ -831,11 +831,13 @@ static int ocfs2_inode_is_valid_to_delete(struct inode *inode) ...@@ -831,11 +831,13 @@ static int ocfs2_inode_is_valid_to_delete(struct inode *inode)
goto bail; goto bail;
} }
/* If we're coming from downconvert_thread we can't go into our own /*
* voting [hello, deadlock city!], so unforuntately we just * If we're coming from downconvert_thread we can't go into our own
* have to skip deleting this guy. That's OK though because * voting [hello, deadlock city!] so we cannot delete the inode. But
* the node who's doing the actual deleting should handle it * since we dropped last inode ref when downconverting dentry lock,
* anyway. */ * we cannot have the file open and thus the node doing unlink will
* take care of deleting the inode.
*/
if (current == osb->dc_task) if (current == osb->dc_task)
goto bail; goto bail;
...@@ -981,8 +983,6 @@ static void ocfs2_delete_inode(struct inode *inode) ...@@ -981,8 +983,6 @@ static void ocfs2_delete_inode(struct inode *inode)
if (is_bad_inode(inode) || !OCFS2_I(inode)->ip_blkno) if (is_bad_inode(inode) || !OCFS2_I(inode)->ip_blkno)
goto bail; goto bail;
dquot_initialize(inode);
if (!ocfs2_inode_is_valid_to_delete(inode)) { if (!ocfs2_inode_is_valid_to_delete(inode)) {
/* It's probably not necessary to truncate_inode_pages /* It's probably not necessary to truncate_inode_pages
* here but we do it for safety anyway (it will most * here but we do it for safety anyway (it will most
...@@ -991,6 +991,8 @@ static void ocfs2_delete_inode(struct inode *inode) ...@@ -991,6 +991,8 @@ static void ocfs2_delete_inode(struct inode *inode)
goto bail; goto bail;
} }
dquot_initialize(inode);
/* We want to block signals in delete_inode as the lock and /* We want to block signals in delete_inode as the lock and
* messaging paths may return us -ERESTARTSYS. Which would * messaging paths may return us -ERESTARTSYS. Which would
* cause us to exit early, resulting in inodes being orphaned * cause us to exit early, resulting in inodes being orphaned
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment