ext4: rename two functions with 'check'
The naming styles are different for some functions with 'check' in their names. Some of them are like: ext4_check_quota_consistency ext4_check_test_dummy_encryption ext4_check_opt_consistency ext4_check_descriptors ext4_check_feature_compatibility While the others looks like below: ext4_geometry_check ext4_journal_data_mode_check This is not a big deal and boils down to personal preference. But I'd like to make them consistent. Signed-off-by: Jason Yan <yanaijie@huawei.com> Link: https://lore.kernel.org/r/20230323140517.1070239-6-yanaijie@huawei.com Signed-off-by: Theodore Ts'o <tytso@mit.edu>
This commit is contained in:
parent
dcbf87589d
commit
68e624398f
|
@ -4713,7 +4713,7 @@ static int ext4_check_feature_compatibility(struct super_block *sb,
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int ext4_geometry_check(struct super_block *sb,
|
static int ext4_check_geometry(struct super_block *sb,
|
||||||
struct ext4_super_block *es)
|
struct ext4_super_block *es)
|
||||||
{
|
{
|
||||||
struct ext4_sb_info *sbi = EXT4_SB(sb);
|
struct ext4_sb_info *sbi = EXT4_SB(sb);
|
||||||
|
@ -4922,7 +4922,7 @@ out:
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int ext4_journal_data_mode_check(struct super_block *sb)
|
static int ext4_check_journal_data_mode(struct super_block *sb)
|
||||||
{
|
{
|
||||||
if (test_opt(sb, DATA_FLAGS) == EXT4_MOUNT_JOURNAL_DATA) {
|
if (test_opt(sb, DATA_FLAGS) == EXT4_MOUNT_JOURNAL_DATA) {
|
||||||
printk_once(KERN_WARNING "EXT4-fs: Warning: mounting with "
|
printk_once(KERN_WARNING "EXT4-fs: Warning: mounting with "
|
||||||
|
@ -5162,7 +5162,7 @@ static int __ext4_fill_super(struct fs_context *fc, struct super_block *sb)
|
||||||
if (ext4_encoding_init(sb, es))
|
if (ext4_encoding_init(sb, es))
|
||||||
goto failed_mount;
|
goto failed_mount;
|
||||||
|
|
||||||
if (ext4_journal_data_mode_check(sb))
|
if (ext4_check_journal_data_mode(sb))
|
||||||
goto failed_mount;
|
goto failed_mount;
|
||||||
|
|
||||||
sb->s_flags = (sb->s_flags & ~SB_POSIXACL) |
|
sb->s_flags = (sb->s_flags & ~SB_POSIXACL) |
|
||||||
|
@ -5264,7 +5264,7 @@ static int __ext4_fill_super(struct fs_context *fc, struct super_block *sb)
|
||||||
goto failed_mount;
|
goto failed_mount;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (ext4_geometry_check(sb, es))
|
if (ext4_check_geometry(sb, es))
|
||||||
goto failed_mount;
|
goto failed_mount;
|
||||||
|
|
||||||
timer_setup(&sbi->s_err_report, print_daily_error_info, 0);
|
timer_setup(&sbi->s_err_report, print_daily_error_info, 0);
|
||||||
|
|
Loading…
Reference in New Issue