[PATCH 03/17] mkfs: Sanitise the superblock feature macros

[Jan Tulak]

----- Original Message -----
> From: "Brian Foster" <bfoster at redhat.com>
> > @@ -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 at redhat.com