On Mon, Oct 21, 2019 at 05:32:43PM +0800, kaixuxia wrote:
> The xfs_bumplink() call has set the inode log fieldmask XFS_ILOG_CORE,
> so the next xfs_trans_log_inode() call is not necessary.
>
> Signed-off-by: kaixuxia <kaixuxia@xxxxxxxxxxx>
Looks ok,
Reviewed-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx>
--D
> ---
> fs/xfs/xfs_inode.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/fs/xfs/xfs_inode.c b/fs/xfs/xfs_inode.c
> index 18f4b26..b0c81f1d 100644
> --- a/fs/xfs/xfs_inode.c
> +++ b/fs/xfs/xfs_inode.c
> @@ -3327,7 +3327,6 @@ struct xfs_iunlink {
> goto out_trans_cancel;
>
> xfs_bumplink(tp, wip);
> - xfs_trans_log_inode(tp, wip, XFS_ILOG_CORE);
> VFS_I(wip)->i_state &= ~I_LINKABLE;
> }
>
> --
> 1.8.3.1
>
> --
> kaixuxia
