[lvc-project] [PATCH v2] gfs2: fix unreachable error handling after freeze/thaw rework

[Fedor Pchelkin]

On Tue, 11. Nov 23:40, Alexey Velichayshiy wrote:
> @@ -747,11 +747,7 @@ static int gfs2_freeze_super(struct super_block *sb, enum freeze_holder who,
>  			set_bit(SDF_FREEZE_INITIATOR, &sdp->sd_flags);
>  			set_bit(SDF_FROZEN, &sdp->sd_flags);
>  			break;
> -		}
> -
> -		error = gfs2_do_thaw(sdp, who, freeze_owner);
> -
> -		if (error == -EBUSY)

Какая-то непонятка в патче, которая всё ещё не позволяет его применить.
Почему после gfs2_do_thaw() выше отсутствует if (error) goto out; ?
Просьба готовить патч поверх _чистого_ актуального состояния репозитория
Торвальдса
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/

> +		} else if (error == -EBUSY)
>  			fs_err(sdp, "waiting for recovery before freeze\n");
>  		else if (error == -EIO) {
>  			fs_err(sdp, "Fatal IO error: cannot freeze gfs2 due "
> @@ -760,6 +756,11 @@ static int gfs2_freeze_super(struct super_block *sb, enum freeze_holder who,
>  		} else {
>  			fs_err(sdp, "error freezing FS: %d\n", error);
>  		}
> +
> +		error = gfs2_do_thaw(sdp, who, freeze_owner);
> +		if (error)
> +			goto out;
> +
>  		fs_err(sdp, "retrying...\n");
>  		msleep(1000);
>  	}
> -- 
> 2.43.0