diff options
author | Christoph Hellwig <hch@infradead.org> | 2011-08-23 04:28:13 -0400 |
---|---|---|
committer | Alex Elder <aelder@sgi.com> | 2011-10-11 22:15:01 -0400 |
commit | 4a06fd262dbeb70a2c315f7259e063efa493fe3d (patch) | |
tree | cca4aee6c419a8b3dc267176bb43d93048593919 /fs/xfs/xfs_super.c | |
parent | 2b3ffd7eb7b4392e3657c5046b055ca9f1f7cf5e (diff) |
xfs: remove i_iocount
We now have an i_dio_count filed and surrounding infrastructure to wait
for direct I/O completion instead of i_icount, and we have never needed
to iocount waits for buffered I/O given that we only set the page uptodate
after finishing all required work. Thus remove i_iocount, and replace
the actually needed waits with calls to inode_dio_wait.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Alex Elder <aelder@sgi.com>
Diffstat (limited to 'fs/xfs/xfs_super.c')
-rw-r--r-- | fs/xfs/xfs_super.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/fs/xfs/xfs_super.c b/fs/xfs/xfs_super.c index 2366c54cc4fa..54d5e102ffe1 100644 --- a/fs/xfs/xfs_super.c +++ b/fs/xfs/xfs_super.c | |||
@@ -796,8 +796,6 @@ xfs_fs_destroy_inode( | |||
796 | if (is_bad_inode(inode)) | 796 | if (is_bad_inode(inode)) |
797 | goto out_reclaim; | 797 | goto out_reclaim; |
798 | 798 | ||
799 | xfs_ioend_wait(ip); | ||
800 | |||
801 | ASSERT(XFS_FORCED_SHUTDOWN(ip->i_mount) || ip->i_delayed_blks == 0); | 799 | ASSERT(XFS_FORCED_SHUTDOWN(ip->i_mount) || ip->i_delayed_blks == 0); |
802 | 800 | ||
803 | /* | 801 | /* |
@@ -837,7 +835,6 @@ xfs_fs_inode_init_once( | |||
837 | inode_init_once(VFS_I(ip)); | 835 | inode_init_once(VFS_I(ip)); |
838 | 836 | ||
839 | /* xfs inode */ | 837 | /* xfs inode */ |
840 | atomic_set(&ip->i_iocount, 0); | ||
841 | atomic_set(&ip->i_pincount, 0); | 838 | atomic_set(&ip->i_pincount, 0); |
842 | spin_lock_init(&ip->i_flags_lock); | 839 | spin_lock_init(&ip->i_flags_lock); |
843 | init_waitqueue_head(&ip->i_ipin_wait); | 840 | init_waitqueue_head(&ip->i_ipin_wait); |
@@ -914,9 +911,8 @@ xfs_fs_write_inode( | |||
914 | * of forcing it all the way to stable storage using a | 911 | * of forcing it all the way to stable storage using a |
915 | * synchronous transaction we let the log force inside the | 912 | * synchronous transaction we let the log force inside the |
916 | * ->sync_fs call do that for thus, which reduces the number | 913 | * ->sync_fs call do that for thus, which reduces the number |
917 | * of synchronous log foces dramatically. | 914 | * of synchronous log forces dramatically. |
918 | */ | 915 | */ |
919 | xfs_ioend_wait(ip); | ||
920 | error = xfs_log_inode(ip); | 916 | error = xfs_log_inode(ip); |
921 | if (error) | 917 | if (error) |
922 | goto out; | 918 | goto out; |
@@ -1681,7 +1677,6 @@ init_xfs_fs(void) | |||
1681 | printk(KERN_INFO XFS_VERSION_STRING " with " | 1677 | printk(KERN_INFO XFS_VERSION_STRING " with " |
1682 | XFS_BUILD_OPTIONS " enabled\n"); | 1678 | XFS_BUILD_OPTIONS " enabled\n"); |
1683 | 1679 | ||
1684 | xfs_ioend_init(); | ||
1685 | xfs_dir_startup(); | 1680 | xfs_dir_startup(); |
1686 | 1681 | ||
1687 | error = xfs_init_zones(); | 1682 | error = xfs_init_zones(); |