----- Original Message -----
> From: "Brian Foster" <bfoster@xxxxxxxxxx>
> > @@ -1912,17 +2013,17 @@ _("32 bit Project IDs always enabled on CRC enabled
> > filesytems\n"));
> > * tried to use crc=0,finobt=1, then issue a warning before
> > * turning them off.
> > */
> > - if (finobt && finobtflag) {
> > + if (sb_feat.finobt && sb_feat.finobtflag) {
>
> Since the code above drops finobtflag, I don't think we'll ever hit
> this. Indeed, I can now create a crc=0,finobt=1 fs, which shouldn't
> happen.
>
> Brian
>
Finobtflag is dropped by a later patch in the set entirely. After all patches, the line is:
if (sb_feat.finobt && mopts.subopt_params[M_FINOBT].seen)
Which indeed works as it should:
mkfs.xfs -f -m crc=0,finobt=1 /dev/vdb2
warning: finobt not supported without CRC support, disabled.
Cheers,
Jan
--
Jan Tulak
jtulak@xxxxxxxxxx
_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs
