diff options
author | Jie Liu <jeff.liu@oracle.com> | 2014-07-23 21:33:28 -0400 |
---|---|---|
committer | Dave Chinner <david@fromorbit.com> | 2014-07-23 21:33:28 -0400 |
commit | 8fe657760d5c8372cd4cf022c4cdb9307a3af995 (patch) | |
tree | ef5106de56953650218ad211f94f109ed93b9d1b /fs/xfs | |
parent | 2451337dd043901b5270b7586942abe564443e3d (diff) |
xfs: remove redundant stat assignment in xfs_bulkstat_one_int
From: Jie Liu <jeff.liu@oracle.com>
Remove the redundant BULKSTAT_RV_NOTHING assignment in case of call
xfs_iget() failed at xfs_bulkstat_one_int().
Signed-off-by: Jie Liu <jeff.liu@oracle.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Dave Chinner <david@fromorbit.com>
Diffstat (limited to 'fs/xfs')
-rw-r--r-- | fs/xfs/xfs_itable.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/fs/xfs/xfs_itable.c b/fs/xfs/xfs_itable.c index 7e54992bcae9..ca15214f1772 100644 --- a/fs/xfs/xfs_itable.c +++ b/fs/xfs/xfs_itable.c | |||
@@ -76,10 +76,8 @@ xfs_bulkstat_one_int( | |||
76 | error = xfs_iget(mp, NULL, ino, | 76 | error = xfs_iget(mp, NULL, ino, |
77 | (XFS_IGET_DONTCACHE | XFS_IGET_UNTRUSTED), | 77 | (XFS_IGET_DONTCACHE | XFS_IGET_UNTRUSTED), |
78 | XFS_ILOCK_SHARED, &ip); | 78 | XFS_ILOCK_SHARED, &ip); |
79 | if (error) { | 79 | if (error) |
80 | *stat = BULKSTAT_RV_NOTHING; | ||
81 | goto out_free; | 80 | goto out_free; |
82 | } | ||
83 | 81 | ||
84 | ASSERT(ip != NULL); | 82 | ASSERT(ip != NULL); |
85 | ASSERT(ip->i_imap.im_blkno != 0); | 83 | ASSERT(ip->i_imap.im_blkno != 0); |
@@ -136,7 +134,6 @@ xfs_bulkstat_one_int( | |||
136 | IRELE(ip); | 134 | IRELE(ip); |
137 | 135 | ||
138 | error = formatter(buffer, ubsize, ubused, buf); | 136 | error = formatter(buffer, ubsize, ubused, buf); |
139 | |||
140 | if (!error) | 137 | if (!error) |
141 | *stat = BULKSTAT_RV_DIDONE; | 138 | *stat = BULKSTAT_RV_DIDONE; |
142 | 139 | ||