diff options
| author | Brian Foster <bfoster@redhat.com> | 2016-06-01 09:38:15 +0200 |
|---|---|---|
| committer | Dave Chinner <david@fromorbit.com> | 2016-06-01 09:38:15 +0200 |
| commit | 0c871f9a101a290e61af02df49087d118543caeb (patch) | |
| tree | d5b551afdc4dda0731a330436e84560fec5ee949 | |
| parent | xfs: fix broken multi-fsb buffer logging (diff) | |
| download | linux-0c871f9a101a290e61af02df49087d118543caeb.tar.xz linux-0c871f9a101a290e61af02df49087d118543caeb.zip | |
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>
| -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 586bb64e674b..3ce3c61af32a 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; } |