Commit ba69f9ab authored by Jan Kara's avatar Jan Kara Committed by Theodore Ts'o
Browse files

ext4: Don't use delayed allocation by default when used instead of ext3 



When ext4 driver is used to mount a filesystem instead of the ext3 file
system driver (through CONFIG_EXT4_USE_FOR_EXT23), do not enable delayed
allocation by default since some ext3 users and application writers have
developed unfortunate expectations about the safety of writing files on
systems subject to sudden and violent death without using fsync().

Signed-off-by: default avatarJan Kara <jack@suse.cz>
Signed-off-by: default avatar"Theodore Ts'o" <tytso@mit.edu>
parent 37f328eb

fs/ext4/super.c

+16 −9
Original line number Diff line number Diff line
@@ -68,7 +68,21 @@ static int ext4_statfs(struct dentry *dentry, struct kstatfs *buf); 
static int ext4_unfreeze(struct super_block *sb);

static void ext4_write_super(struct super_block *sb);

static int ext4_freeze(struct super_block *sb);

static int ext4_get_sb(struct file_system_type *fs_type, int flags,

		       const char *dev_name, void *data, struct vfsmount *mnt);



#if !defined(CONFIG_EXT3_FS) && !defined(CONFIG_EXT3_FS_MODULE) && defined(CONFIG_EXT4_USE_FOR_EXT23)

static struct file_system_type ext3_fs_type = {

	.owner		= THIS_MODULE,

	.name		= "ext3",

	.get_sb		= ext4_get_sb,

	.kill_sb	= kill_block_super,

	.fs_flags	= FS_REQUIRES_DEV,

};

#define IS_EXT3_SB(sb) ((sb)->s_bdev->bd_holder == &ext3_fs_type)

#else

#define IS_EXT3_SB(sb) (0)

#endif



ext4_fsblk_t ext4_block_bitmap(struct super_block *sb,

			       struct ext4_group_desc *bg)
@@ -2539,6 +2553,7 @@ static int ext4_fill_super(struct super_block *sb, void *data, int silent) 
	 * enable delayed allocation by default

	 * Use -o nodelalloc to turn it off

	 */

	if (!IS_EXT3_SB(sb))

		set_opt(sbi->s_mount_opt, DELALLOC);



	if (!parse_options((char *) data, sb, &journal_devnum,
@@ -4096,14 +4111,6 @@ static inline void unregister_as_ext2(void) { } 
#endif



#if !defined(CONFIG_EXT3_FS) && !defined(CONFIG_EXT3_FS_MODULE) && defined(CONFIG_EXT4_USE_FOR_EXT23)

static struct file_system_type ext3_fs_type = {

	.owner		= THIS_MODULE,

	.name		= "ext3",

	.get_sb		= ext4_get_sb,

	.kill_sb	= kill_block_super,

	.fs_flags	= FS_REQUIRES_DEV,

};



static inline void register_as_ext3(void)

{

	int err = register_filesystem(&ext3_fs_type);

CRA Git | Maintained and supported by SUSTech CRA and CCSE