[lustre-devel] [PATCH 19/22] ext4: don't check before replay
Andreas Dilger
adilger at whamcloud.com
Mon Jul 22 18:57:41 PDT 2019
Actually, I think this patch would be OK to push upstream.
Cheers, Andreas
> On Jul 21, 2019, at 23:29, NeilBrown <neilb at suse.com> wrote:
>
>> On Sun, Jul 21 2019, James Simmons wrote:
>>
>> When ldiskfs run in failover mode whith read-only disk.
>> Part of allocation updates are lost and ldiskfs may fail
>> while mounting this is due to inconsistent state of
>> group-descriptor. Group-descriptor check is added after
>> journal replay.
>
> I think this needs to be enabled by a mount option or super-block flag.
>
> NeilBrown
>
>
>>
>> Signed-off-by: James Simmons <jsimmons at infradead.org>
>> ---
>> fs/ext4/super.c | 11 ++++++-----
>> 1 file changed, 6 insertions(+), 5 deletions(-)
>>
>> diff --git a/fs/ext4/super.c b/fs/ext4/super.c
>> index a3179b2..b818acb 100644
>> --- a/fs/ext4/super.c
>> +++ b/fs/ext4/super.c
>> @@ -4255,11 +4255,6 @@ static int ext4_fill_super(struct super_block *sb, void *data, int silent)
>> }
>> }
>> sbi->s_gdb_count = db_count;
>> - if (!ext4_check_descriptors(sb, logical_sb_block, &first_not_zeroed)) {
>> - ext4_msg(sb, KERN_ERR, "group descriptors corrupted!");
>> - ret = -EFSCORRUPTED;
>> - goto failed_mount2;
>> - }
>>
>> timer_setup(&sbi->s_err_report, print_daily_error_info, 0);
>>
>> @@ -4401,6 +4396,12 @@ static int ext4_fill_super(struct super_block *sb, void *data, int silent)
>> sbi->s_journal->j_commit_callback = ext4_journal_commit_callback;
>>
>> no_journal:
>> + if (!ext4_check_descriptors(sb, logical_sb_block, &first_not_zeroed)) {
>> + ext4_msg(sb, KERN_ERR, "group descriptors corrupted!");
>> + ret = -EFSCORRUPTED;
>> + goto failed_mount_wq;
>> + }
>> +
>> if (!test_opt(sb, NO_MBCACHE)) {
>> sbi->s_ea_block_cache = ext4_xattr_create_cache();
>> if (!sbi->s_ea_block_cache) {
>> --
>> 1.8.3.1
More information about the lustre-devel
mailing list