Re: [PATCHv2 0/3] nilfs2 add feature flags

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi, 

At Thu, 22 Jul 2010 03:22:17 +0900,
Ryusuke Konishi wrote:
> 
> This is a revised patchset that introduces feature flags and makes
> nilfs refuse to mount future incompatibile filesystems.
> 
> The first patch is just for improving readability of super block
> format.  The second patch extends super block and adds three flags to
> record feature sets (i.e. "compat features", "read-only compat
> features", and "incompat features").  The semantics of these flags is
> mimic of ext3.  At present, no feature flags are defined.
> 
> The third patch makes nilfs reject filesystems with unknown features.
> This serves as a safety for future disk format change which is not
> forward compatible.
> 
> * Change from the v1 patchset is as follows:
> 
>  - insert the same test of feature flags also before proceed with
>    recovery for read-only mount.

This is, I think, one of the mandatory functionality to take
"experimental flag" away.  Fixing disk format is important, 
but any unexpected incompatible disk format change may happen in future.

At least, nilfs can protect its filesystem from destructing by itself.

Thank you for the improvement!


> Thanks,
> Ryusuke Konishi
> --
>  fs/nilfs2/nilfs.h         |    2 ++
>  fs/nilfs2/super.c         |   39 +++++++++++++++++++++++++++++++++++++++
>  fs/nilfs2/the_nilfs.c     |   20 ++++++++++++++++++++
>  include/linux/nilfs2_fs.h |   41 +++++++++++++++++++++++++++--------------
>  4 files changed, 88 insertions(+), 14 deletions(-)
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-nilfs" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> 
> 



-- 
Jiro SEKIBA <jir@xxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-nilfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Filesystem Development]     [Linux BTRFS]     [Linux CIFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]

  Powered by Linux