diff options
| author | Brian Foster <bfoster@redhat.com> | 2016-06-01 17:38:15 +1000 |
|---|---|---|
| committer | Dave Chinner <david@fromorbit.com> | 2016-06-01 17:38:15 +1000 |
| commit | 0c871f9a101a290e61af02df49087d118543caeb (patch) | |
| tree | d5b551afdc4dda0731a330436e84560fec5ee949 /fs/xfs/xfs_bmap_util.c | |
| parent | a3916e528b917851a4d2379e2fd2579ad5f2b5a7 (diff) | |
| download | linux-0-day-0c871f9a101a290e61af02df49087d118543caeb.tar.gz linux-0-day-0c871f9a101a290e61af02df49087d118543caeb.tar.xz | |
xfs: remove spurious shutdown type check from xfs_bmap_finish()
The static checker reports that after commit 8d99fe92fed0 ("xfs: fix
efi/efd error handling to avoid fs shutdown hangs"), the code has been
reworked such that error == -EFSCORRUPTED is not possible in this
codepath.
Remove the spurious error check and just use SHUTDOWN_META_IO_ERROR
unconditionally.
Signed-off-by: Brian Foster <bfoster@redhat.com>
Reviewed-by: Carlos Maiolino <cmaiolino@redhat.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Dave Chinner <david@fromorbit.com>
Diffstat (limited to 'fs/xfs/xfs_bmap_util.c')
| -rw-r--r-- | fs/xfs/xfs_bmap_util.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/fs/xfs/xfs_bmap_util.c b/fs/xfs/xfs_bmap_util.c index 586bb64e674ba..3ce3c61af32a4 100644 --- a/fs/xfs/xfs_bmap_util.c +++ b/fs/xfs/xfs_bmap_util.c @@ -125,9 +125,7 @@ xfs_bmap_finish( if (committed) { xfs_efi_release(efi); xfs_force_shutdown((*tp)->t_mountp, - (error == -EFSCORRUPTED) ? - SHUTDOWN_CORRUPT_INCORE : - SHUTDOWN_META_IO_ERROR); + SHUTDOWN_META_IO_ERROR); } return error; } |