xfs: remove a duplicate assert in xfs_setattr_size

Christoph Hellwig · cmaiolino · commit 0f7d2a9e0208 · 2026-04-07T13:03:52.000+02:00
There already is an assert that checks for uid and gid changes besides a
lot of others at the beginning of the function.

Signed-off-by: Christoph Hellwig &lt;hch@lst.de&gt;
Reviewed-by: Carlos Maiolino &lt;cmaiolino@redhat.com&gt;
Signed-off-by: Carlos Maiolino &lt;cem@kernel.org&gt;
diff --git a/fs/xfs/xfs_iops.c b/fs/xfs/xfs_iops.c
@@ -1109,7 +1109,6 @@ xfs_setattr_size(
 		xfs_inode_clear_eofblocks_tag(ip);
 	}
 
-	ASSERT(!(iattr->ia_valid & (ATTR_UID | ATTR_GID)));
 	setattr_copy(idmap, inode, iattr);
 	xfs_trans_log_inode(tp, ip, XFS_ILOG_CORE);
 

Original file line number	Diff line number	Diff line change
`@@ -1109,7 +1109,6 @@ xfs_setattr_size(`
`1109`	`1109`	`xfs_inode_clear_eofblocks_tag(ip);`
`1110`	`1110`	`}`
`1111`	`1111`
`1112`		`- ASSERT(!(iattr->ia_valid & (ATTR_UID \| ATTR_GID)));`
`1113`	`1112`	`setattr_copy(idmap, inode, iattr);`
`1114`	`1113`	`xfs_trans_log_inode(tp, ip, XFS_ILOG_CORE);`
`1115`	`1114`