diff options
| author | Zheng Liu <wenqing.lz@taobao.com> | 2013-01-13 17:08:13 +0800 |
|---|---|---|
| committer | Theodore Ts'o <tytso@mit.edu> | 2013-01-14 16:54:09 -0500 |
| commit | 9cf69bb825c2995352502f48ea64d1be1d263995 (patch) | |
| tree | 6e03f6a2cde4e2bed3481b4e3bfbabc0531dfe7c /misc/mke2fs.c | |
| parent | 9e85208e84bbdea9cfe6e8887502a1c4d92efbe5 (diff) | |
| download | e2fsprogs-9cf69bb825c2995352502f48ea64d1be1d263995.tar.gz | |
mke2fs: require the bigalloc feature explicity if the cluster size is set
When cluster-size is specified without the bigalloc feature, mke2fs
will ignore this argument silently. But user might think bigalloc
feature has been enabled unless they use the dumpe2fs command. So now
we ask user to set bigalloc feature explicity when cluster-size is
enabled. This can make sure that users understand what they are doing
because bigalloc might impact the performance for some workloads.
Signed-off-by: Zheng Liu <wenqing.lz@taobao.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'misc/mke2fs.c')
| -rw-r--r-- | misc/mke2fs.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/misc/mke2fs.c b/misc/mke2fs.c index 32b4c34e..4748f4ab 100644 --- a/misc/mke2fs.c +++ b/misc/mke2fs.c @@ -1847,6 +1847,11 @@ profile_error: blocksize*16); fs_param.s_log_cluster_size = int_log2(cluster_size >> EXT2_MIN_CLUSTER_LOG_SIZE); + } else if (cluster_size) { + com_err(program_name, 0, + _("specifying a cluster size requires the " + "bigalloc feature")); + exit(1); } else fs_param.s_log_cluster_size = fs_param.s_log_block_size; |