diff options
| author | Theodore Ts'o <tytso@mit.edu> | 2012-02-17 13:54:47 -0500 |
|---|---|---|
| committer | Theodore Ts'o <tytso@mit.edu> | 2012-02-17 15:28:21 -0500 |
| commit | 991211f676f49c6cf30af368dade2f81287f1fa8 (patch) | |
| tree | 37cab633fe2195d1558a0674f03810b2474fbb7f /lib/e2p | |
| parent | 3ae682bb0a5e025963b25ff5ac5d53e2115483f2 (diff) | |
| download | e2fsprogs-991211f676f49c6cf30af368dade2f81287f1fa8.tar.gz | |
libext2fs, libe2p: Reserve RO_COMPAT_REPLICA feature
The replica is a feature which stores multiple copies of the key
metadata blocks so a single block failure in failure-prone media
(read: certain types of flash storage) doesn't take out the entire
file system.
Discussion on the upstream list proved not to be very positive on this
feature; the arguments were that it added complexity that wasn't
warrented, since common practice in industry is to insist on reliable
media, and if media is unreliable, you're kind of toast anyway (unless
the file system is being used as the back-end store of a cluster file
system where checksuming and data replication is happening above the
local disk file system level). So, this feature is being developed
out of tree.
We reserve the code points so that other people won't accidentally
step on them. Since it's not upstream, it's a soft reservation, but
it's not like we have any shortage of RO_COMPAT features. We are a
bit more tight on reserved inodes, but EXT2_BOOT_LOADER_INO and
EXT2_UNDEL_DIR_INO are not currently used anywhere, and
EXT2_EXCLUDE_INO is a reservation for another out-of-tree feature.
There are no features currently being discussed which require a
reserved inode, but if a need were to arise, we can claw back code
point reservations that were never used or not in tree, as those will
always be considered lower priority than in-tree features.
Cc: Aditya Kali <adityakali@google.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'lib/e2p')
| -rw-r--r-- | lib/e2p/feature.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/lib/e2p/feature.c b/lib/e2p/feature.c index 63486f37..19e6f0cc 100644 --- a/lib/e2p/feature.c +++ b/lib/e2p/feature.c @@ -64,6 +64,8 @@ static struct feature feature_list[] = { "bigalloc"}, { E2P_FEATURE_RO_INCOMPAT, EXT4_FEATURE_RO_COMPAT_METADATA_CSUM, "metadata_csum"}, + { E2P_FEATURE_RO_INCOMPAT, EXT4_FEATURE_RO_COMPAT_REPLICA, + "replica" }, { E2P_FEATURE_INCOMPAT, EXT2_FEATURE_INCOMPAT_COMPRESSION, "compression" }, |