diff options
| author | Theodore Ts'o <tytso@mit.edu> | 2009-01-06 20:53:16 +0100 |
|---|---|---|
| committer | Theodore Ts'o <tytso@mit.edu> | 2009-01-06 20:53:16 +0100 |
| commit | 83982b6f47201c4c7767210d24d7d8c99567a0b3 (patch) | |
| tree | f1c19b1c6b5a34dd0c5de7519b78414b7a9267ee /fs/ext4/ialloc.c | |
| parent | block: Add Kconfig help which notes that ext4 needs CONFIG_LBD (diff) | |
| download | linux-83982b6f47201c4c7767210d24d7d8c99567a0b3.tar.xz linux-83982b6f47201c4c7767210d24d7d8c99567a0b3.zip | |
ext4: Remove "extents" mount option
This mount option is largely superfluous, and in fact the way it was
implemented was buggy; if a filesystem which did not have the extents
feature flag was mounted -o extents, the filesystem would attempt to
create and use extents-based file even though the extents feature flag
was not eabled. The simplest thing to do is to nuke the mount option
entirely. It's not all that useful to force the non-creation of new
extent-based files if the filesystem can support it.
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/ext4/ialloc.c')
| -rw-r--r-- | fs/ext4/ialloc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/ext4/ialloc.c b/fs/ext4/ialloc.c index 369c34c64292..4fb86a0061d0 100644 --- a/fs/ext4/ialloc.c +++ b/fs/ext4/ialloc.c @@ -917,7 +917,7 @@ got: if (err) goto fail_free_drop; - if (test_opt(sb, EXTENTS)) { + if (EXT4_HAS_INCOMPAT_FEATURE(sb, EXT4_FEATURE_INCOMPAT_EXTENTS)) { /* set extent flag only for directory, file and normal symlink*/ if (S_ISDIR(mode) || S_ISREG(mode) || S_ISLNK(mode)) { EXT4_I(inode)->i_flags |= EXT4_EXTENTS_FL; |