Andreas, you wrote:
> Daniel, you write:
> > Andreas, you wrote:
> > > We should go with "EXT2_FEATURE_COMPAT_DIR_INDEX 0x0008"
> > > because the on-disk layout is 100% compatible with older kernels, so
> > > no reason to force read-only for those systems.  I'm guessing Ted had
> > > put RO_COMPAT_BTREE_DIR in there in anticipation, but it was never used.
> > 
> > By the way, did you mean:
> > 
> >     #define EXT2_FEATURE_COMPAT_DIR_INDEX 0x0002
> > 
> > since there is only one other COMPAT feature currently defined?
> 
> Quick note - you need to use 0x0008 or higher (Ted is the authority on
> this).  The kernel ext2_fs.h is out of date compared to the one in e2fsprogs.
> The EXT3_FEATURE_COMPAT_HAS_JOURNAL and EXT2_FEATURE_COMPAT_IMAGIC_INODES
> is missing from the kernel header.

Since this could mess up somebody's superblock, albiet in a minor way, I
uploaded the correction right away without changing the patch name.

--
Daniel

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to