lkml.org 
[lkml]   [2020]   [Sep]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.8 116/255] ext4: correctly restore system zone info when remount fails
    Date
    From: Jan Kara <jack@suse.cz>

    [ Upstream commit 0f5bde1db174f6c471f0bd27198575719dabe3e5 ]

    When remounting filesystem fails late during remount handling and
    block_validity mount option is also changed during the remount, we fail
    to restore system zone information to a state matching the mount option.
    This is mostly harmless, just the block validity checking will not match
    the situation described by the mount option. Make sure these two are always
    consistent.

    Reported-by: Lukas Czerner <lczerner@redhat.com>
    Reviewed-by: Lukas Czerner <lczerner@redhat.com>
    Signed-off-by: Jan Kara <jack@suse.cz>
    Link: https://lore.kernel.org/r/20200728130437.7804-7-jack@suse.cz
    Signed-off-by: Theodore Ts'o <tytso@mit.edu>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    fs/ext4/block_validity.c | 8 --------
    fs/ext4/super.c | 29 +++++++++++++++++++++--------
    2 files changed, 21 insertions(+), 16 deletions(-)

    diff --git a/fs/ext4/block_validity.c b/fs/ext4/block_validity.c
    index e830a9d4e10d3..11aa37693e436 100644
    --- a/fs/ext4/block_validity.c
    +++ b/fs/ext4/block_validity.c
    @@ -254,14 +254,6 @@ int ext4_setup_system_zone(struct super_block *sb)
    int flex_size = ext4_flex_bg_size(sbi);
    int ret;

    - if (!test_opt(sb, BLOCK_VALIDITY)) {
    - if (sbi->system_blks)
    - ext4_release_system_zone(sb);
    - return 0;
    - }
    - if (sbi->system_blks)
    - return 0;
    -
    system_blks = kzalloc(sizeof(*system_blks), GFP_KERNEL);
    if (!system_blks)
    return -ENOMEM;
    diff --git a/fs/ext4/super.c b/fs/ext4/super.c
    index 54d1c09329e55..4c8253188d8df 100644
    --- a/fs/ext4/super.c
    +++ b/fs/ext4/super.c
    @@ -4698,11 +4698,13 @@ no_journal:

    ext4_set_resv_clusters(sb);

    - err = ext4_setup_system_zone(sb);
    - if (err) {
    - ext4_msg(sb, KERN_ERR, "failed to initialize system "
    - "zone (%d)", err);
    - goto failed_mount4a;
    + if (test_opt(sb, BLOCK_VALIDITY)) {
    + err = ext4_setup_system_zone(sb);
    + if (err) {
    + ext4_msg(sb, KERN_ERR, "failed to initialize system "
    + "zone (%d)", err);
    + goto failed_mount4a;
    + }
    }

    ext4_ext_init(sb);
    @@ -5716,9 +5718,16 @@ static int ext4_remount(struct super_block *sb, int *flags, char *data)
    ext4_register_li_request(sb, first_not_zeroed);
    }

    - err = ext4_setup_system_zone(sb);
    - if (err)
    - goto restore_opts;
    + /*
    + * Handle creation of system zone data early because it can fail.
    + * Releasing of existing data is done when we are sure remount will
    + * succeed.
    + */
    + if (test_opt(sb, BLOCK_VALIDITY) && !sbi->system_blks) {
    + err = ext4_setup_system_zone(sb);
    + if (err)
    + goto restore_opts;
    + }

    if (sbi->s_journal == NULL && !(old_sb_flags & SB_RDONLY)) {
    err = ext4_commit_super(sb, 1);
    @@ -5740,6 +5749,8 @@ static int ext4_remount(struct super_block *sb, int *flags, char *data)
    }
    }
    #endif
    + if (!test_opt(sb, BLOCK_VALIDITY) && sbi->system_blks)
    + ext4_release_system_zone(sb);

    /*
    * Some options can be enabled by ext4 and/or by VFS mount flag
    @@ -5761,6 +5772,8 @@ restore_opts:
    sbi->s_commit_interval = old_opts.s_commit_interval;
    sbi->s_min_batch_time = old_opts.s_min_batch_time;
    sbi->s_max_batch_time = old_opts.s_max_batch_time;
    + if (!test_opt(sb, BLOCK_VALIDITY) && sbi->system_blks)
    + ext4_release_system_zone(sb);
    #ifdef CONFIG_QUOTA
    sbi->s_jquota_fmt = old_opts.s_jquota_fmt;
    for (i = 0; i < EXT4_MAXQUOTAS; i++) {
    --
    2.25.1


    \
     
     \ /
      Last update: 2020-09-01 17:44    [W:4.198 / U:0.524 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site