[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <8f71afdb-f762-ac59-c71f-8241ad2e9144@huaweicloud.com>
Date: Mon, 26 Jun 2023 10:31:34 +0800
From: Yu Kuai <yukuai1@...weicloud.com>
To: linan666@...weicloud.com, song@...nel.org
Cc: linux-raid@...r.kernel.org, linux-kernel@...r.kernel.org,
linan122@...wei.com, yi.zhang@...wei.com, houtao1@...wei.com,
yangerkun@...wei.com, "yukuai (C)" <yukuai3@...wei.com>
Subject: Re: [PATCH 2/3] md: remove redundant check in fix_read_error()
在 2023/06/24 1:32, linan666@...weicloud.com 写道:
> From: Li Nan <linan122@...wei.com>
>
> In fix_read_error(), 'success' will be checked immediately after assigning
> it, if it is set to 1 then the loop will break. Checking it again in
> condition of loop is redundant. Clean it up.
LGTM
Reviewed-by: Yu Kuai <yukuai3@...wei.com>
>
> Signed-off-by: Li Nan <linan122@...wei.com>
> ---
> drivers/md/raid1.c | 2 +-
> drivers/md/raid10.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
> :
> diff --git a/drivers/md/raid1.c b/drivers/md/raid1.c
> index 3570da63969b..0391c2d0c109 100644
> --- a/drivers/md/raid1.c
> +++ b/drivers/md/raid1.c
> @@ -2318,7 +2318,7 @@ static void fix_read_error(struct r1conf *conf, int read_disk,
> d++;
> if (d == conf->raid_disks * 2)
> d = 0;
> - } while (!success && d != read_disk);
> + } while (d != read_disk);
>
> if (!success) {
> /* Cannot read from anywhere - mark it bad */
> diff --git a/drivers/md/raid10.c b/drivers/md/raid10.c
> index 94ae294c8a3c..a36e53fce21f 100644
> --- a/drivers/md/raid10.c
> +++ b/drivers/md/raid10.c
> @@ -2790,7 +2790,7 @@ static void fix_read_error(struct r10conf *conf, struct mddev *mddev, struct r10
> sl++;
> if (sl == conf->copies)
> sl = 0;
> - } while (!success && sl != slot);
> + } while (sl != slot);
> rcu_read_unlock();
>
> if (!success) {
>
Powered by blists - more mailing lists