Hi Darrick,
On Wed, Apr 21, 2021 at 08:55:14AM -0700, Darrick J. Wong wrote:
> On Wed, Apr 21, 2021 at 04:57:16PM +0800, Gao Xiang wrote:
> > Kernel commit b96cb835e37c ("xfs: deprecate the V4 format") started
> > the process of retiring the old format to close off attack surfaces
> > and to encourage users to migrate onto V5.
> >
> > This also prints warning to users when mkfs as well.
> >
> > Suggested-by: Darrick J. Wong <djwong@xxxxxxxxxx>
> > Signed-off-by: Gao Xiang <hsiangkao@xxxxxxxxxx>
>
> Looks fine to me; but does this cause any golden output failures in
> fstests?
ok, I will check mkfs_filter later.
(btw, just in case... would you mind reply the message below:
https://lore.kernel.org/r/20210420200029.GA3028214@xxxxxxxxxxxxxxxxxx
so I could refine the series and send out the next version later?
Also I still have no idea how to handle [PATCH v2 1/2], since
I'd like just add some comments here to explain why sb_ifree,
sb_icount doesn't matter and such logic can be (and will be)
totally removed after [PATCH v2 2/2] in short time. )
Thanks,
Gao Xiang
>
> --D
>
> > ---
> > mkfs/xfs_mkfs.c | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/mkfs/xfs_mkfs.c b/mkfs/xfs_mkfs.c
> > index 0eac5336..ef09f8b3 100644
> > --- a/mkfs/xfs_mkfs.c
> > +++ b/mkfs/xfs_mkfs.c
> > @@ -4022,6 +4022,10 @@ main(
> > validate_extsize_hint(mp, &cli);
> > validate_cowextsize_hint(mp, &cli);
> >
> > + if (!cli.sb_feat.crcs_enabled)
> > + fprintf(stderr,
> > +_("Deprecated V4 format (-mcrc=0) will not be supported after September 2030.\n"));
> > +
> > /* Print the intended geometry of the fs. */
> > if (!quiet || dry_run) {
> > struct xfs_fsop_geom geo;
> > --
> > 2.27.0
> >
>
