* Re: [PATCH] xfs: Default XFS_RT to Y if CONFIG_BLK_DEV_ZONED is enabled
2025-08-12 2:55 ` [PATCH] xfs: Default XFS_RT to Y if CONFIG_BLK_DEV_ZONED is enabled Damien Le Moal
@ 2025-08-12 7:42 ` Carlos Maiolino
2025-08-12 7:45 ` Christoph Hellwig
2025-08-19 13:11 ` Carlos Maiolino
2 siblings, 0 replies; 4+ messages in thread
From: Carlos Maiolino @ 2025-08-12 7:42 UTC (permalink / raw)
To: Damien Le Moal; +Cc: linux-xfs, Christoph Hellwig
On Tue, Aug 12, 2025 at 11:55:19AM +0900, Damien Le Moal wrote:
> XFS support for zoned block devices requires the realtime subvolume
> support (XFS_RT) to be enabled. Change the default configuration value
> of XFS_RT from N to CONFIG_BLK_DEV_ZONED to align with this requirement.
> This change still allows the user to disable XFS_RT if this feature is
> not desired for the user use case.
>
> Suggested-by: Christoph Hellwig <hch@lst.de>
> Signed-off-by: Damien Le Moal <dlemoal@kernel.org>
> ---
> fs/xfs/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/fs/xfs/Kconfig b/fs/xfs/Kconfig
> index 68724dec3d19..9367f60718dd 100644
> --- a/fs/xfs/Kconfig
> +++ b/fs/xfs/Kconfig
> @@ -105,6 +105,7 @@ config XFS_POSIX_ACL
> config XFS_RT
> bool "XFS Realtime subvolume support"
> depends on XFS_FS
> + default BLK_DEV_ZONED
> help
> If you say Y here you will be able to mount and use XFS filesystems
> which contain a realtime subvolume. The realtime subvolume is a
This looks as a good compromise to me as it still allows distributions
to disable it if they don't want to have XFS_RT enabled even if zoned
devices are supported.
Reviewed-by: Carlos Maiolino <cmaiolino@redhat.com>
> --
> 2.50.1
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] xfs: Default XFS_RT to Y if CONFIG_BLK_DEV_ZONED is enabled
2025-08-12 2:55 ` [PATCH] xfs: Default XFS_RT to Y if CONFIG_BLK_DEV_ZONED is enabled Damien Le Moal
2025-08-12 7:42 ` Carlos Maiolino
@ 2025-08-12 7:45 ` Christoph Hellwig
2025-08-19 13:11 ` Carlos Maiolino
2 siblings, 0 replies; 4+ messages in thread
From: Christoph Hellwig @ 2025-08-12 7:45 UTC (permalink / raw)
To: Damien Le Moal; +Cc: Carlos Maiolino, linux-xfs, Christoph Hellwig
Looks good:
Reviewed-by: Christoph Hellwig <hch@lst.de>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] xfs: Default XFS_RT to Y if CONFIG_BLK_DEV_ZONED is enabled
2025-08-12 2:55 ` [PATCH] xfs: Default XFS_RT to Y if CONFIG_BLK_DEV_ZONED is enabled Damien Le Moal
2025-08-12 7:42 ` Carlos Maiolino
2025-08-12 7:45 ` Christoph Hellwig
@ 2025-08-19 13:11 ` Carlos Maiolino
2 siblings, 0 replies; 4+ messages in thread
From: Carlos Maiolino @ 2025-08-19 13:11 UTC (permalink / raw)
To: linux-xfs, Damien Le Moal; +Cc: Christoph Hellwig
On Tue, 12 Aug 2025 11:55:19 +0900, Damien Le Moal wrote:
> XFS support for zoned block devices requires the realtime subvolume
> support (XFS_RT) to be enabled. Change the default configuration value
> of XFS_RT from N to CONFIG_BLK_DEV_ZONED to align with this requirement.
> This change still allows the user to disable XFS_RT if this feature is
> not desired for the user use case.
>
>
> [...]
Applied to for-next, thanks!
[1/1] xfs: Default XFS_RT to Y if CONFIG_BLK_DEV_ZONED is enabled
commit: 9ce43caa4b7be707638d49ad4fb358b6ff646e91
Best regards,
--
Carlos Maiolino <cem@kernel.org>
^ permalink raw reply [flat|nested] 4+ messages in thread