[XFS] Cleanup references to i_sem.
authorNathan Scott <nathans@sgi.com>
Fri, 17 Mar 2006 06:26:14 +0000 (17:26 +1100)
committerNathan Scott <nathans@sgi.com>
Fri, 17 Mar 2006 06:26:14 +0000 (17:26 +1100)
SGI-PV: 904196
SGI-Modid: xfs-linux-melb:xfs-kern:25480a

Signed-off-by: Nathan Scott <nathans@sgi.com>
fs/xfs/linux-2.6/xfs_lrw.c

index 2ede4bb7ecd0770da1a28329fff21ab5b7b0975e..ba07b83f988d4b4a08308fa0cc494d6574cea161 100644 (file)
@@ -249,9 +249,8 @@ xfs_read(
        if (n < size)
                size = n;
 
-       if (XFS_FORCED_SHUTDOWN(mp)) {
+       if (XFS_FORCED_SHUTDOWN(mp))
                return -EIO;
-       }
 
        if (unlikely(ioflags & IO_ISDIRECT))
                mutex_lock(&inode->i_mutex);
@@ -267,7 +266,7 @@ xfs_read(
                                        dmflags, &locktype);
                if (ret) {
                        xfs_iunlock(ip, XFS_IOLOCK_SHARED);
-                       goto unlock_isem;
+                       goto unlock_mutex;
                }
        }
 
@@ -281,7 +280,7 @@ xfs_read(
 
        xfs_iunlock(ip, XFS_IOLOCK_SHARED);
 
-unlock_isem:
+unlock_mutex:
        if (unlikely(ioflags & IO_ISDIRECT))
                mutex_unlock(&inode->i_mutex);
        return ret;
@@ -573,7 +572,7 @@ xfs_write(
        vrwlock_t               locktype;
        size_t                  ocount = 0, count;
        loff_t                  pos;
-       int                     need_isem = 1, need_flush = 0;
+       int                     need_i_mutex = 1, need_flush = 0;
 
        XFS_STATS_INC(xs_write_calls);
 
@@ -622,14 +621,14 @@ xfs_write(
                        return XFS_ERROR(-EINVAL);
 
                if (!VN_CACHED(vp) && pos < i_size_read(inode))
-                       need_isem = 0;
+                       need_i_mutex = 0;
 
                if (VN_CACHED(vp))
                        need_flush = 1;
        }
 
 relock:
-       if (need_isem) {
+       if (need_i_mutex) {
                iolock = XFS_IOLOCK_EXCL;
                locktype = VRWLOCK_WRITE;
 
@@ -651,7 +650,7 @@ start:
                                        S_ISBLK(inode->i_mode));
        if (error) {
                xfs_iunlock(xip, XFS_ILOCK_EXCL|iolock);
-               goto out_unlock_isem;
+               goto out_unlock_mutex;
        }
 
        new_size = pos + count;
@@ -663,7 +662,7 @@ start:
                loff_t          savedsize = pos;
                int             dmflags = FILP_DELAY_FLAG(file);
 
-               if (need_isem)
+               if (need_i_mutex)
                        dmflags |= DM_FLAGS_IMUX;
 
                xfs_iunlock(xip, XFS_ILOCK_EXCL);
@@ -672,7 +671,7 @@ start:
                                      dmflags, &locktype);
                if (error) {
                        xfs_iunlock(xip, iolock);
-                       goto out_unlock_isem;
+                       goto out_unlock_mutex;
                }
                xfs_ilock(xip, XFS_ILOCK_EXCL);
                eventsent = 1;
@@ -710,7 +709,7 @@ start:
                                        isize, pos + count);
                if (error) {
                        xfs_iunlock(xip, XFS_ILOCK_EXCL|iolock);
-                       goto out_unlock_isem;
+                       goto out_unlock_mutex;
                }
        }
        xfs_iunlock(xip, XFS_ILOCK_EXCL);
@@ -731,7 +730,7 @@ start:
                        error = -remove_suid(file->f_dentry);
                if (unlikely(error)) {
                        xfs_iunlock(xip, iolock);
-                       goto out_unlock_isem;
+                       goto out_unlock_mutex;
                }
        }
 
@@ -747,14 +746,14 @@ retry:
                                        -1, FI_REMAPF_LOCKED);
                }
 
-               if (need_isem) {
+               if (need_i_mutex) {
                        /* demote the lock now the cached pages are gone */
                        XFS_ILOCK_DEMOTE(mp, io, XFS_IOLOCK_EXCL);
                        mutex_unlock(&inode->i_mutex);
 
                        iolock = XFS_IOLOCK_SHARED;
                        locktype = VRWLOCK_WRITE_DIRECT;
-                       need_isem = 0;
+                       need_i_mutex = 0;
                }
 
                xfs_rw_enter_trace(XFS_DIOWR_ENTER, io, (void *)iovp, segs,
@@ -772,7 +771,7 @@ retry:
                        pos += ret;
                        count -= ret;
 
-                       need_isem = 1;
+                       need_i_mutex = 1;
                        ioflags &= ~IO_ISDIRECT;
                        xfs_iunlock(xip, iolock);
                        goto relock;
@@ -794,14 +793,14 @@ retry:
            !(ioflags & IO_INVIS)) {
 
                xfs_rwunlock(bdp, locktype);
-               if (need_isem)
+               if (need_i_mutex)
                        mutex_unlock(&inode->i_mutex);
                error = XFS_SEND_NAMESP(xip->i_mount, DM_EVENT_NOSPACE, vp,
                                DM_RIGHT_NULL, vp, DM_RIGHT_NULL, NULL, NULL,
                                0, 0, 0); /* Delay flag intentionally  unused */
                if (error)
                        goto out_nounlocks;
-               if (need_isem)
+               if (need_i_mutex)
                        mutex_lock(&inode->i_mutex);
                xfs_rwlock(bdp, locktype);
                pos = xip->i_d.di_size;
@@ -905,9 +904,9 @@ retry:
                        if (error)
                                goto out_unlock_internal;
                }
-       
+
                xfs_rwunlock(bdp, locktype);
-               if (need_isem)
+               if (need_i_mutex)
                        mutex_unlock(&inode->i_mutex);
 
                error = sync_page_range(inode, mapping, pos, ret);
@@ -918,8 +917,8 @@ retry:
 
  out_unlock_internal:
        xfs_rwunlock(bdp, locktype);
- out_unlock_isem:
-       if (need_isem)
+ out_unlock_mutex:
+       if (need_i_mutex)
                mutex_unlock(&inode->i_mutex);
  out_nounlocks:
        return -error;