xfs: remove duplicate INO1_WRITTEN check

Commit a23eca8844 ("xfs: fix exchange-range reflink flag clearing
issue with INO1_WRITTEN") duplicated commit b2d5a81dae ("xfs: fix
exchange-range reflink flag clearing issue with INO1_WRITTEN"), so
xmi_can_exchange_reflink_flags() ended up with two identical
XFS_EXCHMAPS_INO1_WRITTEN checks.  The second one is dead code,
since the first one already returned false.  Remove it.

Signed-off-by: Jiangshan Yi <yijiangshan@kylinos.cn>
Reviewed-by: Darrick J. Wong <djwong@kernel.org>
Signed-off-by: Carlos Maiolino <cem@kernel.org>
This commit is contained in:
Jiangshan Yi 2026-09-14 16:21:11 +08:00 committed by Carlos Maiolino
parent 14e379600d
commit ce2b91bebc

View File

@ -959,16 +959,6 @@ xmi_can_exchange_reflink_flags(
{
struct xfs_mount *mp = req->ip1->i_mount;
/*
* The INO1_WRITTEN optimization can skip exchanging hole and
* unwritten mappings, which means we cannot guarantee that all
* shared extents actually moved to the other file. Clearing the
* reflink flag of an inode that still holds shared extents breaks
* the CoW write path, so refuse to exchange the flags in that case.
*/
if (req->flags & XFS_EXCHMAPS_INO1_WRITTEN)
return false;
/*
* The INO1_WRITTEN optimization can skip exchanging hole and
* unwritten mappings, which means we cannot guarantee that all