linux-xfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] xfs: Default XFS_RT to Y if CONFIG_BLK_DEV_ZONED is enabled
@ 2025-08-12  2:55 ` Damien Le Moal
  2025-08-12  7:42   ` Carlos Maiolino
                     ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Damien Le Moal @ 2025-08-12  2:55 UTC (permalink / raw)
  To: Carlos Maiolino, linux-xfs; +Cc: Christoph Hellwig

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
-- 
2.50.1


^ permalink raw reply related	[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-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

end of thread, other threads:[~2025-08-19 13:11 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <7hiMOJZATLYgHbQYIoT4OdW2dzyJjPDOkOD-RgRwvftGN2A2K30y_c-13YQKqUTXVF0P369P3fzB1jYPMGhIRQ==@protonmail.internalid>
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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).