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 | |
| 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>
| -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 | ||
