summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEric Sandeen <sandeen@redhat.com>2016-10-20 06:44:53 +0200
committerDave Chinner <david@fromorbit.com>2016-10-20 06:44:53 +0200
commitfe23759eaf2f6540de20c1623f066aad967ff9c9 (patch)
treeaed055ad6ea66c50dde4bedc43d2338966044526
parentxfs: don't take the IOLOCK exclusive for direct I/O page invalidation (diff)
downloadlinux-fe23759eaf2f6540de20c1623f066aad967ff9c9.tar.xz
linux-fe23759eaf2f6540de20c1623f066aad967ff9c9.zip
xfs: remove pointless error goto in xfs_bmap_remap_alloc
The commit: f65306ea xfs: map an inode's offset to an exact physical block added a pointless error0: target; remove it. Addresses-Coverity-Id: 1373865 Signed-off-by: Eric Sandeen <sandeen@redhat.com> Reviewed-by: Bill O'Donnell <billodo@redhat.com> Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com> Signed-off-by: Dave Chinner <david@fromorbit.com>
-rw-r--r--fs/xfs/libxfs/xfs_bmap.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/fs/xfs/libxfs/xfs_bmap.c b/fs/xfs/libxfs/xfs_bmap.c
index 0283b7eaf973..80bdb11ca6bf 100644
--- a/fs/xfs/libxfs/xfs_bmap.c
+++ b/fs/xfs/libxfs/xfs_bmap.c
@@ -3974,9 +3974,6 @@ xfs_bmap_remap_alloc(
* allocating, so skip that check by pretending to be freeing.
*/
error = xfs_alloc_fix_freelist(&args, XFS_ALLOC_FLAG_FREEING);
- if (error)
- goto error0;
-error0:
xfs_perag_put(args.pag);
if (error)
trace_xfs_bmap_remap_alloc_error(ap->ip, error, _RET_IP_);