xfs: reject swapon for inodes on a zoned file system earlier

Christoph Hellwig · cmaiolino · commit 8e5a2441e186 · 2025-08-19T14:38:39.000+02:00
No point in going down into the iomap mapping loop when we know it
will be rejected.

Signed-off-by: Christoph Hellwig &lt;hch@lst.de&gt;
Reviewed-by: Damien Le Moal &lt;dlemoal@kernel.org&gt;
Reviewed-by: Hans Holmberg &lt;hans.holmberg@wdc.com&gt;
Signed-off-by: Carlos Maiolino &lt;cem@kernel.org&gt;
diff --git a/fs/xfs/xfs_aops.c b/fs/xfs/xfs_aops.c
@@ -760,6 +760,9 @@ xfs_vm_swap_activate(
 {
 	struct xfs_inode		*ip = XFS_I(file_inode(swap_file));
 
+	if (xfs_is_zoned_inode(ip))
+		return -EINVAL;
+
 	/*
 	 * Swap file activation can race against concurrent shared extent
 	 * removal in files that have been cloned.  If this happens,

Original file line number	Diff line number	Diff line change
`@@ -760,6 +760,9 @@ xfs_vm_swap_activate(`
`760`	`760`	`{`
`761`	`761`	`struct xfs_inode *ip = XFS_I(file_inode(swap_file));`
`762`	`762`
	`763`	`+ if (xfs_is_zoned_inode(ip))`
	`764`	`+ return -EINVAL;`
	`765`	`+`
`763`	`766`	`/*`
`764`	`767`	`* Swap file activation can race against concurrent shared extent`
`765`	`768`	`* removal in files that have been cloned. If this happens,`