diff options
Diffstat (limited to 'fs/quota/dquot.c')
-rw-r--r-- | fs/quota/dquot.c | 28 |
1 files changed, 16 insertions, 12 deletions
diff --git a/fs/quota/dquot.c b/fs/quota/dquot.c index a1470fda366c..fcc8ae75d874 100644 --- a/fs/quota/dquot.c +++ b/fs/quota/dquot.c | |||
@@ -76,7 +76,7 @@ | |||
76 | #include <linux/buffer_head.h> | 76 | #include <linux/buffer_head.h> |
77 | #include <linux/capability.h> | 77 | #include <linux/capability.h> |
78 | #include <linux/quotaops.h> | 78 | #include <linux/quotaops.h> |
79 | #include <linux/writeback.h> /* for inode_lock, oddly enough.. */ | 79 | #include "../internal.h" /* ugh */ |
80 | 80 | ||
81 | #include <asm/uaccess.h> | 81 | #include <asm/uaccess.h> |
82 | 82 | ||
@@ -900,7 +900,7 @@ static void add_dquot_ref(struct super_block *sb, int type) | |||
900 | int reserved = 0; | 900 | int reserved = 0; |
901 | #endif | 901 | #endif |
902 | 902 | ||
903 | spin_lock(&inode_lock); | 903 | spin_lock(&inode_sb_list_lock); |
904 | list_for_each_entry(inode, &sb->s_inodes, i_sb_list) { | 904 | list_for_each_entry(inode, &sb->s_inodes, i_sb_list) { |
905 | spin_lock(&inode->i_lock); | 905 | spin_lock(&inode->i_lock); |
906 | if ((inode->i_state & (I_FREEING|I_WILL_FREE|I_NEW)) || | 906 | if ((inode->i_state & (I_FREEING|I_WILL_FREE|I_NEW)) || |
@@ -915,19 +915,23 @@ static void add_dquot_ref(struct super_block *sb, int type) | |||
915 | #endif | 915 | #endif |
916 | __iget(inode); | 916 | __iget(inode); |
917 | spin_unlock(&inode->i_lock); | 917 | spin_unlock(&inode->i_lock); |
918 | spin_unlock(&inode_lock); | 918 | spin_unlock(&inode_sb_list_lock); |
919 | 919 | ||
920 | iput(old_inode); | 920 | iput(old_inode); |
921 | __dquot_initialize(inode, type); | 921 | __dquot_initialize(inode, type); |
922 | /* We hold a reference to 'inode' so it couldn't have been | 922 | |
923 | * removed from s_inodes list while we dropped the inode_lock. | 923 | /* |
924 | * We cannot iput the inode now as we can be holding the last | 924 | * We hold a reference to 'inode' so it couldn't have been |
925 | * reference and we cannot iput it under inode_lock. So we | 925 | * removed from s_inodes list while we dropped the |
926 | * keep the reference and iput it later. */ | 926 | * inode_sb_list_lock We cannot iput the inode now as we can be |
927 | * holding the last reference and we cannot iput it under | ||
928 | * inode_sb_list_lock. So we keep the reference and iput it | ||
929 | * later. | ||
930 | */ | ||
927 | old_inode = inode; | 931 | old_inode = inode; |
928 | spin_lock(&inode_lock); | 932 | spin_lock(&inode_sb_list_lock); |
929 | } | 933 | } |
930 | spin_unlock(&inode_lock); | 934 | spin_unlock(&inode_sb_list_lock); |
931 | iput(old_inode); | 935 | iput(old_inode); |
932 | 936 | ||
933 | #ifdef CONFIG_QUOTA_DEBUG | 937 | #ifdef CONFIG_QUOTA_DEBUG |
@@ -1008,7 +1012,7 @@ static void remove_dquot_ref(struct super_block *sb, int type, | |||
1008 | struct inode *inode; | 1012 | struct inode *inode; |
1009 | int reserved = 0; | 1013 | int reserved = 0; |
1010 | 1014 | ||
1011 | spin_lock(&inode_lock); | 1015 | spin_lock(&inode_sb_list_lock); |
1012 | list_for_each_entry(inode, &sb->s_inodes, i_sb_list) { | 1016 | list_for_each_entry(inode, &sb->s_inodes, i_sb_list) { |
1013 | /* | 1017 | /* |
1014 | * We have to scan also I_NEW inodes because they can already | 1018 | * We have to scan also I_NEW inodes because they can already |
@@ -1022,7 +1026,7 @@ static void remove_dquot_ref(struct super_block *sb, int type, | |||
1022 | remove_inode_dquot_ref(inode, type, tofree_head); | 1026 | remove_inode_dquot_ref(inode, type, tofree_head); |
1023 | } | 1027 | } |
1024 | } | 1028 | } |
1025 | spin_unlock(&inode_lock); | 1029 | spin_unlock(&inode_sb_list_lock); |
1026 | #ifdef CONFIG_QUOTA_DEBUG | 1030 | #ifdef CONFIG_QUOTA_DEBUG |
1027 | if (reserved) { | 1031 | if (reserved) { |
1028 | printk(KERN_WARNING "VFS (%s): Writes happened after quota" | 1032 | printk(KERN_WARNING "VFS (%s): Writes happened after quota" |