[lvc-project] [PATCH] gfs2: eliminate unreachable code in gfs2_freeze_super()

[Fedor Pchelkin]

Приветствую, Алексей!  Спасибо за патч, по оформлению вопросов нет.

On Sat, 01. Nov 12:55, Alexey Velichayshiy wrote:
> The general error check after gfs2_do_thaw() made the subsequent
> specific error handlers for -EBUSY and -EIO unreachable. Remove
> the redundant general check to ensure proper error handling
> for specific cases while preserving the retry mechanism.
> 
> Found by Linux Verification Center (linuxtesting.org) with SVACE.
> 
> Signed-off-by: Alexey Velichayshiy <a.velichayshiy at ispras.ru>
> ---
>  fs/gfs2/super.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/fs/gfs2/super.c b/fs/gfs2/super.c
> index 644b2d1e7276..03b407a925dc 100644
> --- a/fs/gfs2/super.c
> +++ b/fs/gfs2/super.c
> @@ -750,8 +750,6 @@ static int gfs2_freeze_super(struct super_block *sb, enum freeze_holder who,
>  		}
>  
>  		error = gfs2_do_thaw(sdp, who, freeze_owner);
> -		if (error)
> -			goto out;

Тут конечно трудно понять, чего хотел автор кода, но эту проверку он внёс
тем же патчем, что и добавил вызов gfs2_do_thaw() - коммит b77b4a4815a9
("gfs2: Rework freeze / thaw logic").

Выглядит будто обработка кодов ошибок ниже (EBUSY и проч.) относится к
gfs2_lock_fs_check_clean(), т.е. нужно перенести её перед вызовом
gfs2_do_thaw(), а `if (error)` после gfs2_do_thaw() оставить.

Просьба выслать вторую версию патча, пометив как [PATCH v2] также сюда в
lvc-patches.

>  
>  		if (error == -EBUSY)
>  			fs_err(sdp, "waiting for recovery before freeze\n");
> -- 
> 2.43.0