dquot: cleanup dquot drop routine
authorChristoph Hellwig <hch@infradead.org>
Wed, 3 Mar 2010 14:05:05 +0000 (09:05 -0500)
committerJan Kara <jack@suse.cz>
Thu, 4 Mar 2010 23:20:30 +0000 (00:20 +0100)
Get rid of the drop dquot operation - it is now always called from
the filesystem and if a filesystem really needs it's own (which none
currently does) it can just call into it's own routine directly.

Rename the now static low-level dquot_drop helper to __dquot_drop
and vfs_dq_drop to dquot_drop to have a consistent namespace.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jan Kara <jack@suse.cz>
22 files changed:
Documentation/filesystems/Locking
fs/ext2/ialloc.c
fs/ext2/super.c
fs/ext3/ialloc.c
fs/ext3/super.c
fs/ext4/ialloc.c
fs/ext4/super.c
fs/jfs/inode.c
fs/jfs/jfs_inode.c
fs/jfs/super.c
fs/ocfs2/inode.c
fs/ocfs2/quota_global.c
fs/quota/dquot.c
fs/reiserfs/inode.c
fs/reiserfs/namei.c
fs/reiserfs/super.c
fs/udf/ialloc.c
fs/udf/inode.c
fs/ufs/ialloc.c
fs/ufs/super.c
include/linux/quota.h
include/linux/quotaops.h

index 4574e0272bddf0a6124846ffc6c3db8b7d406800..fa10e4bf8e5e56188e430b7e8a55705c6fb1b44c 100644 (file)
@@ -461,7 +461,6 @@ in sys_read() and friends.
 --------------------------- dquot_operations -------------------------------
 prototypes:
        int (*initialize) (struct inode *, int);
-       int (*drop) (struct inode *);
        int (*write_dquot) (struct dquot *);
        int (*acquire_dquot) (struct dquot *);
        int (*release_dquot) (struct dquot *);
@@ -475,7 +474,6 @@ What filesystem should expect from the generic quota functions:
 
                FS recursion    Held locks when called
 initialize:    yes             maybe dqonoff_sem
-drop:          yes             -
 write_dquot:   yes             dqonoff_sem or dqptr_sem
 acquire_dquot: yes             dqonoff_sem or dqptr_sem
 release_dquot: yes             dqonoff_sem or dqptr_sem
index d12f9809559cdb12e0348a5841aa282ae53ff955..88b71972c626c001426e88d3f2e0b571890d7188 100644 (file)
@@ -122,7 +122,7 @@ void ext2_free_inode (struct inode * inode)
                /* Quota is already initialized in iput() */
                ext2_xattr_delete_inode(inode);
                dquot_free_inode(inode);
-               vfs_dq_drop(inode);
+               dquot_drop(inode);
        }
 
        es = EXT2_SB(sb)->s_es;
@@ -608,7 +608,7 @@ fail_free_drop:
        dquot_free_inode(inode);
 
 fail_drop:
-       vfs_dq_drop(inode);
+       dquot_drop(inode);
        inode->i_flags |= S_NOQUOTA;
        inode->i_nlink = 0;
        unlock_new_inode(inode);
index 98815d2a56647c2f83344a541fe349f90c3a7321..42e4a303b6758628aa87c23d18acbcb6e13110c5 100644 (file)
@@ -195,7 +195,7 @@ static void ext2_clear_inode(struct inode *inode)
 {
        struct ext2_block_alloc_info *rsv = EXT2_I(inode)->i_block_alloc_info;
 
-       vfs_dq_drop(inode);
+       dquot_drop(inode);
        ext2_discard_reservation(inode);
        EXT2_I(inode)->i_block_alloc_info = NULL;
        if (unlikely(rsv))
index 8bf00e997c38f929a69d53e60458427927854e10..7d7238f9f6f3cba8db8dbddb24e6f219e56be361 100644 (file)
@@ -126,7 +126,7 @@ void ext3_free_inode (handle_t *handle, struct inode * inode)
        vfs_dq_init(inode);
        ext3_xattr_delete_inode(handle, inode);
        dquot_free_inode(inode);
-       vfs_dq_drop(inode);
+       dquot_drop(inode);
 
        is_directory = S_ISDIR(inode->i_mode);
 
@@ -622,7 +622,7 @@ fail_free_drop:
        dquot_free_inode(inode);
 
 fail_drop:
-       vfs_dq_drop(inode);
+       dquot_drop(inode);
        inode->i_flags |= S_NOQUOTA;
        inode->i_nlink = 0;
        unlock_new_inode(inode);
index 2277b1a98e622edb44d394b1805d1416d366eb47..0163d0dae124f5c2c09d05ce72cd9f05479c9351 100644 (file)
@@ -529,7 +529,7 @@ static void ext3_clear_inode(struct inode *inode)
 {
        struct ext3_block_alloc_info *rsv = EXT3_I(inode)->i_block_alloc_info;
 
-       vfs_dq_drop(inode);
+       dquot_drop(inode);
        ext3_discard_reservation(inode);
        EXT3_I(inode)->i_block_alloc_info = NULL;
        if (unlikely(rsv))
@@ -753,7 +753,6 @@ static ssize_t ext3_quota_write(struct super_block *sb, int type,
 
 static const struct dquot_operations ext3_quota_operations = {
        .initialize     = dquot_initialize,
-       .drop           = dquot_drop,
        .write_dquot    = ext3_write_dquot,
        .acquire_dquot  = ext3_acquire_dquot,
        .release_dquot  = ext3_release_dquot,
index b0d744cf8b95a62dc6f7c04c35b5b9524e674f5e..ca8986e4b528270b27f0e53dbb2fc8efeea2e980 100644 (file)
@@ -220,7 +220,7 @@ void ext4_free_inode(handle_t *handle, struct inode *inode)
        vfs_dq_init(inode);
        ext4_xattr_delete_inode(handle, inode);
        dquot_free_inode(inode);
-       vfs_dq_drop(inode);
+       dquot_drop(inode);
 
        is_directory = S_ISDIR(inode->i_mode);
 
@@ -1077,7 +1077,7 @@ fail_free_drop:
        dquot_free_inode(inode);
 
 fail_drop:
-       vfs_dq_drop(inode);
+       dquot_drop(inode);
        inode->i_flags |= S_NOQUOTA;
        inode->i_nlink = 0;
        unlock_new_inode(inode);
index 56554c8850ecd160639408875448eb1b8a828e34..035516c80df28d112ddeddf7c3080be46f47873c 100644 (file)
@@ -761,7 +761,7 @@ static void destroy_inodecache(void)
 
 static void ext4_clear_inode(struct inode *inode)
 {
-       vfs_dq_drop(inode);
+       dquot_drop(inode);
        ext4_discard_preallocations(inode);
        if (EXT4_JOURNAL(inode))
                jbd2_journal_release_jbd_inode(EXT4_SB(inode->i_sb)->s_journal,
@@ -1014,7 +1014,6 @@ static ssize_t ext4_quota_write(struct super_block *sb, int type,
 
 static const struct dquot_operations ext4_quota_operations = {
        .initialize     = dquot_initialize,
-       .drop           = dquot_drop,
 #ifdef CONFIG_QUOTA
        .get_reserved_space = ext4_get_reserved_space,
 #endif
index 2562d18988f752028afd311625248db15ceb77a0..22fa412c52898306e1b72ba634b2daba617c177f 100644 (file)
@@ -160,7 +160,7 @@ void jfs_delete_inode(struct inode *inode)
                 */
                vfs_dq_init(inode);
                dquot_free_inode(inode);
-               vfs_dq_drop(inode);
+               dquot_drop(inode);
        }
 
        clear_inode(inode);
index 7762f33e062bce360e15ef549f26ba1b165f2720..72b30895422cbba40b5fc79e7e921c8000b724a6 100644 (file)
@@ -162,7 +162,7 @@ struct inode *ialloc(struct inode *parent, umode_t mode)
        return inode;
 
 fail_drop:
-       vfs_dq_drop(inode);
+       dquot_drop(inode);
        inode->i_flags |= S_NOQUOTA;
 fail_unlock:
        inode->i_nlink = 0;
index 4086fa59341937f443966f9c46e240888a284287..266699deb1c68825d29eee529d0109dcf9ce50a0 100644 (file)
@@ -133,7 +133,7 @@ static void jfs_destroy_inode(struct inode *inode)
 
 static void jfs_clear_inode(struct inode *inode)
 {
-       vfs_dq_drop(inode);
+       dquot_drop(inode);
 }
 
 static int jfs_statfs(struct dentry *dentry, struct kstatfs *buf)
index 13eb5d467c40d0bde078536f9866354ca2191586..00eb6a095e686f126294a6c7713287038f4e0a64 100644 (file)
@@ -1087,7 +1087,7 @@ void ocfs2_clear_inode(struct inode *inode)
        mlog_bug_on_msg(OCFS2_SB(inode->i_sb) == NULL,
                        "Inode=%lu\n", inode->i_ino);
 
-       vfs_dq_drop(inode);
+       dquot_drop(inode);
 
        /* To preven remote deletes we hold open lock before, now it
         * is time to unlock PR and EX open locks. */
index b654bd103b6f648e8aa8cecd02054c506948d0af..4dca38f487cf265e1a901ac5dbbdd68dc9a9989c 100644 (file)
@@ -852,7 +852,6 @@ static void ocfs2_destroy_dquot(struct dquot *dquot)
 
 const struct dquot_operations ocfs2_quota_operations = {
        .initialize     = dquot_initialize,
-       .drop           = dquot_drop,
        .write_dquot    = ocfs2_write_dquot,
        .acquire_dquot  = ocfs2_acquire_dquot,
        .release_dquot  = ocfs2_release_dquot,
index 78ce4c48ad77335647b3b9be8e0e04b60a4f387f..cd83c5b871bab94dea6a41ccd93fda8e3ad1c24d 100644 (file)
@@ -1358,7 +1358,7 @@ EXPORT_SYMBOL(dquot_initialize);
 /*
  *     Release all quotas referenced by inode
  */
-int dquot_drop(struct inode *inode)
+static void __dquot_drop(struct inode *inode)
 {
        int cnt;
        struct dquot *put[MAXQUOTAS];
@@ -1370,32 +1370,31 @@ int dquot_drop(struct inode *inode)
        }
        up_write(&sb_dqopt(inode->i_sb)->dqptr_sem);
        dqput_all(put);
-       return 0;
 }
-EXPORT_SYMBOL(dquot_drop);
 
-/* Wrapper to remove references to quota structures from inode */
-void vfs_dq_drop(struct inode *inode)
-{
-       /* Here we can get arbitrary inode from clear_inode() so we have
-        * to be careful. OTOH we don't need locking as quota operations
-        * are allowed to change only at mount time */
-       if (!IS_NOQUOTA(inode) && inode->i_sb && inode->i_sb->dq_op
-           && inode->i_sb->dq_op->drop) {
-               int cnt;
-               /* Test before calling to rule out calls from proc and such
-                 * where we are not allowed to block. Note that this is
-                * actually reliable test even without the lock - the caller
-                * must assure that nobody can come after the DQUOT_DROP and
-                * add quota pointers back anyway */
-               for (cnt = 0; cnt < MAXQUOTAS; cnt++)
-                       if (inode->i_dquot[cnt])
-                               break;
-               if (cnt < MAXQUOTAS)
-                       inode->i_sb->dq_op->drop(inode);
-       }
-}
-EXPORT_SYMBOL(vfs_dq_drop);
+void dquot_drop(struct inode *inode)
+{
+       int cnt;
+
+       if (IS_NOQUOTA(inode))
+               return;
+
+       /*
+        * Test before calling to rule out calls from proc and such
+        * where we are not allowed to block. Note that this is
+        * actually reliable test even without the lock - the caller
+        * must assure that nobody can come after the DQUOT_DROP and
+        * add quota pointers back anyway.
+        */
+       for (cnt = 0; cnt < MAXQUOTAS; cnt++) {
+               if (inode->i_dquot[cnt])
+                       break;
+       }
+
+       if (cnt < MAXQUOTAS)
+               __dquot_drop(inode);
+}
+EXPORT_SYMBOL(dquot_drop);
 
 /*
  * inode_reserved_space is managed internally by quota, and protected by
@@ -1812,7 +1811,6 @@ EXPORT_SYMBOL(dquot_commit_info);
  */
 const struct dquot_operations dquot_operations = {
        .initialize     = dquot_initialize,
-       .drop           = dquot_drop,
        .write_dquot    = dquot_commit,
        .acquire_dquot  = dquot_acquire,
        .release_dquot  = dquot_release,
@@ -2029,7 +2027,7 @@ static int vfs_load_quota_inode(struct inode *inode, int type, int format_id,
                 * When S_NOQUOTA is set, remove dquot references as no more
                 * references can be added
                 */
-               sb->dq_op->drop(inode);
+               __dquot_drop(inode);
        }
 
        error = -EIO;
index 99a5e5a8ab5a4930f251f88c4f506a90f21edfd7..f07c3b69247d88a3032520e214ebd6b44a0322c8 100644 (file)
@@ -1964,7 +1964,7 @@ int reiserfs_new_inode(struct reiserfs_transaction_handle *th,
       out_end_trans:
        journal_end(th, th->t_super, th->t_blocks_allocated);
        /* Drop can be outside and it needs more credits so it's better to have it outside */
-       vfs_dq_drop(inode);
+       dquot_drop(inode);
        inode->i_flags |= S_NOQUOTA;
        make_bad_inode(inode);
 
index 9d4dcf0b07cbdcd65353915de08e9a27e9f99848..9dea84e8a79a73442188120fbcafb9fa669ef9b1 100644 (file)
@@ -546,7 +546,7 @@ static int reiserfs_add_entry(struct reiserfs_transaction_handle *th,
 */
 static int drop_new_inode(struct inode *inode)
 {
-       vfs_dq_drop(inode);
+       dquot_drop(inode);
        make_bad_inode(inode);
        inode->i_flags |= S_NOQUOTA;
        iput(inode);
index 6b24e70e329b42b33916e469615553a530f0fd3f..34f7cd0cb02dbad112474e8586b5a63d9fffda46 100644 (file)
@@ -580,7 +580,7 @@ out:
 
 static void reiserfs_clear_inode(struct inode *inode)
 {
-       vfs_dq_drop(inode);
+       dquot_drop(inode);
 }
 
 #ifdef CONFIG_QUOTA
@@ -623,7 +623,6 @@ static int reiserfs_quota_on(struct super_block *, int, int, char *, int);
 
 static const struct dquot_operations reiserfs_quota_operations = {
        .initialize = dquot_initialize,
-       .drop = dquot_drop,
        .write_dquot = reiserfs_write_dquot,
        .acquire_dquot = reiserfs_acquire_dquot,
        .release_dquot = reiserfs_release_dquot,
index e1856b89c9c88486b0fd2e331241534243b2a1d5..15c6e992e587c4844144fc6cb7a089213b06d8d8 100644 (file)
@@ -37,7 +37,7 @@ void udf_free_inode(struct inode *inode)
         * as writing the quota to disk may need the lock as well.
         */
        dquot_free_inode(inode);
-       vfs_dq_drop(inode);
+       dquot_drop(inode);
 
        clear_inode(inode);
 
@@ -156,7 +156,7 @@ struct inode *udf_new_inode(struct inode *dir, int mode, int *err)
        vfs_dq_init(inode);
        ret = dquot_alloc_inode(inode);
        if (ret) {
-               vfs_dq_drop(inode);
+               dquot_drop(inode);
                inode->i_flags |= S_NOQUOTA;
                inode->i_nlink = 0;
                iput(inode);
index 859389a3832bdbd2602373758152c209662e6cbb..1199e8e21ee21ef83d7d6cf8d556f7bc8e0c9e92 100644 (file)
@@ -109,7 +109,7 @@ void udf_clear_inode(struct inode *inode)
                        (unsigned long long)iinfo->i_lenExtents);
        }
 
-       vfs_dq_drop(inode);
+       dquot_drop(inode);
        kfree(iinfo->i_ext.i_data);
        iinfo->i_ext.i_data = NULL;
 }
index 02f77882c5732c8f9afd733a35d4f59d1c3e1f87..67b4bdb056fb8ee6385d4b9b071df0907f2ae174 100644 (file)
@@ -96,7 +96,7 @@ void ufs_free_inode (struct inode * inode)
        is_directory = S_ISDIR(inode->i_mode);
 
        dquot_free_inode(inode);
-       vfs_dq_drop(inode);
+       dquot_drop(inode);
 
        clear_inode (inode);
 
@@ -358,7 +358,7 @@ cg_found:
        vfs_dq_init(inode);
        err = dquot_alloc_inode(inode);
        if (err) {
-               vfs_dq_drop(inode);
+               dquot_drop(inode);
                goto fail_without_unlock;
        }
 
index 95d61cb3a5b8186a20347820d80ad9e5c33043a7..66b63a7516159f766d7c4822e9770fce2bf94105 100644 (file)
@@ -1434,7 +1434,7 @@ static void destroy_inodecache(void)
 
 static void ufs_clear_inode(struct inode *inode)
 {
-       vfs_dq_drop(inode);
+       dquot_drop(inode);
 }
 
 #ifdef CONFIG_QUOTA
index 422e6aa78edc48a2f03c264cd7ef684d2e8b3191..aec2e9dac2d7b6a5d074b6910a9e69d791c65b9c 100644 (file)
@@ -296,7 +296,6 @@ struct quota_format_ops {
 /* Operations working with dquots */
 struct dquot_operations {
        int (*initialize) (struct inode *, int);
-       int (*drop) (struct inode *);
        int (*write_dquot) (struct dquot *);            /* Ordinary dquot write */
        struct dquot *(*alloc_dquot)(struct super_block *, int);        /* Allocate memory for new dquot */
        void (*destroy_dquot)(struct dquot *);          /* Free memory for dquot */
index fa27b7218c822635d13610378f936db40bfed547..a5ebd1abccd8d9b068682fd25fe0419e5cf05e19 100644 (file)
@@ -24,7 +24,7 @@ void inode_claim_rsv_space(struct inode *inode, qsize_t number);
 void inode_sub_rsv_space(struct inode *inode, qsize_t number);
 
 int dquot_initialize(struct inode *inode, int type);
-int dquot_drop(struct inode *inode);
+void dquot_drop(struct inode *inode);
 struct dquot *dqget(struct super_block *sb, unsigned int id, int type);
 void dqput(struct dquot *dquot);
 int dquot_scan_active(struct super_block *sb,
@@ -64,7 +64,6 @@ int vfs_set_dqinfo(struct super_block *sb, int type, struct if_dqinfo *ii);
 int vfs_get_dqblk(struct super_block *sb, int type, qid_t id, struct if_dqblk *di);
 int vfs_set_dqblk(struct super_block *sb, int type, qid_t id, struct if_dqblk *di);
 
-void vfs_dq_drop(struct inode *inode);
 int dquot_transfer(struct inode *inode, struct iattr *iattr);
 int vfs_dq_quota_on_remount(struct super_block *sb);
 
@@ -210,7 +209,7 @@ static inline void vfs_dq_init(struct inode *inode)
 {
 }
 
-static inline void vfs_dq_drop(struct inode *inode)
+static inline void dquot_drop(struct inode *inode)
 {
 }