From: Brian Foster Date: Wed, 1 Jun 2016 07:38:15 +0000 (+1000) Subject: xfs: remove spurious shutdown type check from xfs_bmap_finish() X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=0c871f9a101a290e61af02df49087d118543caeb;p=GitHub%2Fmoto-9609%2Fandroid_kernel_motorola_exynos9610.git 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 Reviewed-by: Carlos Maiolino Reviewed-by: Christoph Hellwig Signed-off-by: Dave Chinner --- 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; }