linux-xfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] mkfs: warn about V4 deprecation when creating new V4 filesystems
@ 2021-08-31 22:44 Darrick J. Wong
  2021-08-31 23:29 ` Dave Chinner
  0 siblings, 1 reply; 4+ messages in thread
From: Darrick J. Wong @ 2021-08-31 22:44 UTC (permalink / raw)
  To: Eric Sandeen; +Cc: xfs

From: Darrick J. Wong <djwong@kernel.org>

The V4 filesystem format is deprecated in the upstream Linux kernel.  In
September 2025 it will be turned off by default in the kernel and five
years after that, support will be removed entirely.  Warn people
formatting new filesystems with the old format, particularly since V4 is
not the default.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>
---
 mkfs/xfs_mkfs.c |    9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/mkfs/xfs_mkfs.c b/mkfs/xfs_mkfs.c
index 53904677..b8c11ce9 100644
--- a/mkfs/xfs_mkfs.c
+++ b/mkfs/xfs_mkfs.c
@@ -2103,6 +2103,15 @@ _("Directory ftype field always enabled on CRC enabled filesystems\n"));
 		}
 
 	} else {	/* !crcs_enabled */
+		/*
+		 * The V4 filesystem format is deprecated in the upstream Linux
+		 * kernel.  In September 2025 it will be turned off by default
+		 * in the kernel and in September 2030 support will be removed
+		 * entirely.
+		 */
+		fprintf(stdout,
+_("V4 filesystems are deprecated and will not be supported by future versions.\n"));
+
 		/*
 		 * The kernel doesn't support crc=0,finobt=1 filesystems.
 		 * If crcs are not enabled and the user has not explicitly

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] mkfs: warn about V4 deprecation when creating new V4 filesystems
  2021-08-31 22:44 [PATCH] mkfs: warn about V4 deprecation when creating new V4 filesystems Darrick J. Wong
@ 2021-08-31 23:29 ` Dave Chinner
  2021-08-31 23:33   ` Darrick J. Wong
  0 siblings, 1 reply; 4+ messages in thread
From: Dave Chinner @ 2021-08-31 23:29 UTC (permalink / raw)
  To: Darrick J. Wong; +Cc: Eric Sandeen, xfs

On Tue, Aug 31, 2021 at 03:44:38PM -0700, Darrick J. Wong wrote:
> From: Darrick J. Wong <djwong@kernel.org>
> 
> The V4 filesystem format is deprecated in the upstream Linux kernel.  In
> September 2025 it will be turned off by default in the kernel and five
> years after that, support will be removed entirely.  Warn people
> formatting new filesystems with the old format, particularly since V4 is
> not the default.
> 
> Signed-off-by: Darrick J. Wong <djwong@kernel.org>
> ---
>  mkfs/xfs_mkfs.c |    9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/mkfs/xfs_mkfs.c b/mkfs/xfs_mkfs.c
> index 53904677..b8c11ce9 100644
> --- a/mkfs/xfs_mkfs.c
> +++ b/mkfs/xfs_mkfs.c
> @@ -2103,6 +2103,15 @@ _("Directory ftype field always enabled on CRC enabled filesystems\n"));
>  		}
>  
>  	} else {	/* !crcs_enabled */
> +		/*
> +		 * The V4 filesystem format is deprecated in the upstream Linux
> +		 * kernel.  In September 2025 it will be turned off by default
> +		 * in the kernel and in September 2030 support will be removed
> +		 * entirely.
> +		 */
> +		fprintf(stdout,
> +_("V4 filesystems are deprecated and will not be supported by future versions.\n"));
> +
>  		/*
>  		 * The kernel doesn't support crc=0,finobt=1 filesystems.
>  		 * If crcs are not enabled and the user has not explicitly
> 

Looks good to me.

Reviewed-by: Dave Chinner <dchinner@redhat.com>

Do we need to update the mkfs filter in fstests now?

Cheers,

Dave.
-- 
Dave Chinner
david@fromorbit.com

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] mkfs: warn about V4 deprecation when creating new V4 filesystems
  2021-08-31 23:29 ` Dave Chinner
@ 2021-08-31 23:33   ` Darrick J. Wong
  2021-08-31 23:53     ` Darrick J. Wong
  0 siblings, 1 reply; 4+ messages in thread
From: Darrick J. Wong @ 2021-08-31 23:33 UTC (permalink / raw)
  To: Dave Chinner; +Cc: Eric Sandeen, xfs

On Wed, Sep 01, 2021 at 09:29:27AM +1000, Dave Chinner wrote:
> On Tue, Aug 31, 2021 at 03:44:38PM -0700, Darrick J. Wong wrote:
> > From: Darrick J. Wong <djwong@kernel.org>
> > 
> > The V4 filesystem format is deprecated in the upstream Linux kernel.  In
> > September 2025 it will be turned off by default in the kernel and five
> > years after that, support will be removed entirely.  Warn people
> > formatting new filesystems with the old format, particularly since V4 is
> > not the default.
> > 
> > Signed-off-by: Darrick J. Wong <djwong@kernel.org>
> > ---
> >  mkfs/xfs_mkfs.c |    9 +++++++++
> >  1 file changed, 9 insertions(+)
> > 
> > diff --git a/mkfs/xfs_mkfs.c b/mkfs/xfs_mkfs.c
> > index 53904677..b8c11ce9 100644
> > --- a/mkfs/xfs_mkfs.c
> > +++ b/mkfs/xfs_mkfs.c
> > @@ -2103,6 +2103,15 @@ _("Directory ftype field always enabled on CRC enabled filesystems\n"));
> >  		}
> >  
> >  	} else {	/* !crcs_enabled */
> > +		/*
> > +		 * The V4 filesystem format is deprecated in the upstream Linux
> > +		 * kernel.  In September 2025 it will be turned off by default
> > +		 * in the kernel and in September 2030 support will be removed
> > +		 * entirely.
> > +		 */
> > +		fprintf(stdout,
> > +_("V4 filesystems are deprecated and will not be supported by future versions.\n"));
> > +
> >  		/*
> >  		 * The kernel doesn't support crc=0,finobt=1 filesystems.
> >  		 * If crcs are not enabled and the user has not explicitly
> > 
> 
> Looks good to me.
> 
> Reviewed-by: Dave Chinner <dchinner@redhat.com>
> 
> Do we need to update the mkfs filter in fstests now?

AFAICT, no, because (so far) the only fstests that care about mkfs
output only look for messages on stderr, and dump stdout to $seqres.full
or /dev/null.

--D

> Cheers,
> 
> Dave.
> -- 
> Dave Chinner
> david@fromorbit.com

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] mkfs: warn about V4 deprecation when creating new V4 filesystems
  2021-08-31 23:33   ` Darrick J. Wong
@ 2021-08-31 23:53     ` Darrick J. Wong
  0 siblings, 0 replies; 4+ messages in thread
From: Darrick J. Wong @ 2021-08-31 23:53 UTC (permalink / raw)
  To: Dave Chinner; +Cc: Eric Sandeen, xfs

On Tue, Aug 31, 2021 at 04:33:00PM -0700, Darrick J. Wong wrote:
> On Wed, Sep 01, 2021 at 09:29:27AM +1000, Dave Chinner wrote:
> > On Tue, Aug 31, 2021 at 03:44:38PM -0700, Darrick J. Wong wrote:
> > > From: Darrick J. Wong <djwong@kernel.org>
> > > 
> > > The V4 filesystem format is deprecated in the upstream Linux kernel.  In
> > > September 2025 it will be turned off by default in the kernel and five
> > > years after that, support will be removed entirely.  Warn people
> > > formatting new filesystems with the old format, particularly since V4 is
> > > not the default.
> > > 
> > > Signed-off-by: Darrick J. Wong <djwong@kernel.org>
> > > ---
> > >  mkfs/xfs_mkfs.c |    9 +++++++++
> > >  1 file changed, 9 insertions(+)
> > > 
> > > diff --git a/mkfs/xfs_mkfs.c b/mkfs/xfs_mkfs.c
> > > index 53904677..b8c11ce9 100644
> > > --- a/mkfs/xfs_mkfs.c
> > > +++ b/mkfs/xfs_mkfs.c
> > > @@ -2103,6 +2103,15 @@ _("Directory ftype field always enabled on CRC enabled filesystems\n"));
> > >  		}
> > >  
> > >  	} else {	/* !crcs_enabled */
> > > +		/*
> > > +		 * The V4 filesystem format is deprecated in the upstream Linux
> > > +		 * kernel.  In September 2025 it will be turned off by default
> > > +		 * in the kernel and in September 2030 support will be removed
> > > +		 * entirely.
> > > +		 */
> > > +		fprintf(stdout,
> > > +_("V4 filesystems are deprecated and will not be supported by future versions.\n"));
> > > +
> > >  		/*
> > >  		 * The kernel doesn't support crc=0,finobt=1 filesystems.
> > >  		 * If crcs are not enabled and the user has not explicitly
> > > 
> > 
> > Looks good to me.
> > 
> > Reviewed-by: Dave Chinner <dchinner@redhat.com>
> > 
> > Do we need to update the mkfs filter in fstests now?
> 
> AFAICT, no, because (so far) the only fstests that care about mkfs
> output only look for messages on stderr, and dump stdout to $seqres.full
> or /dev/null.

So I'll immediately put my foot in my mouth; there /was/ a patch in my
fstests tree to fix the one test that broke: xfs/449.

--D

> --D
> 
> > Cheers,
> > 
> > Dave.
> > -- 
> > Dave Chinner
> > david@fromorbit.com

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-08-31 23:53 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-31 22:44 [PATCH] mkfs: warn about V4 deprecation when creating new V4 filesystems Darrick J. Wong
2021-08-31 23:29 ` Dave Chinner
2021-08-31 23:33   ` Darrick J. Wong
2021-08-31 23:53     ` Darrick J. Wong

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).