fs: fold invalidate_list into invalidate_inodes
authorChristoph Hellwig <hch@lst.de>
Sun, 24 Oct 2010 17:40:33 +0000 (19:40 +0200)
committerAl Viro <viro@zeniv.linux.org.uk>
Tue, 26 Oct 2010 01:26:15 +0000 (21:26 -0400)
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/inode.c

index 1bf2be41257a144550003481e5b7aa5f1187f012..f6b3b52f1278e5b757b780fb271922a908875acc 100644 (file)
@@ -477,15 +477,24 @@ static void dispose_list(struct list_head *head)
        }
 }
 
-/*
- * Invalidate all inodes for a device.
+/**
+ * invalidate_inodes   - attempt to free all inodes on a superblock
+ * @sb:                superblock to operate on
+ *
+ * Attempts to free all inodes for a given superblock.  If there were any
+ * busy inodes return a non-zero value, else zero.
  */
-static int invalidate_list(struct list_head *head, struct list_head *dispose)
+int invalidate_inodes(struct super_block *sb)
 {
-       struct inode *inode, *next;
        int busy = 0;
+       struct inode *inode, *next;
+       LIST_HEAD(dispose);
 
-       list_for_each_entry_safe(inode, next, head, i_sb_list) {
+       down_write(&iprune_sem);
+
+       spin_lock(&inode_lock);
+       fsnotify_unmount_inodes(&sb->s_inodes);
+       list_for_each_entry_safe(inode, next, &sb->s_inodes, i_sb_list) {
                if (inode->i_state & I_NEW)
                        continue;
                if (atomic_read(&inode->i_count)) {
@@ -499,34 +508,14 @@ static int invalidate_list(struct list_head *head, struct list_head *dispose)
                 * Move the inode off the IO lists and LRU once I_FREEING is
                 * set so that it won't get moved back on there if it is dirty.
                 */
-               list_move(&inode->i_lru, dispose);
+               list_move(&inode->i_lru, &dispose);
                list_del_init(&inode->i_wb_list);
                if (!(inode->i_state & (I_DIRTY | I_SYNC)))
                        percpu_counter_dec(&nr_inodes_unused);
        }
-       return busy;
-}
-
-/**
- *     invalidate_inodes       - discard the inodes on a device
- *     @sb: superblock
- *
- *     Discard all of the inodes for a given superblock. If the discard
- *     fails because there are busy inodes then a non zero value is returned.
- *     If the discard is successful all the inodes have been discarded.
- */
-int invalidate_inodes(struct super_block *sb)
-{
-       int busy;
-       LIST_HEAD(throw_away);
-
-       down_write(&iprune_sem);
-       spin_lock(&inode_lock);
-       fsnotify_unmount_inodes(&sb->s_inodes);
-       busy = invalidate_list(&sb->s_inodes, &throw_away);
        spin_unlock(&inode_lock);
 
-       dispose_list(&throw_away);
+       dispose_list(&dispose);
        up_write(&iprune_sem);
 
        return busy;