On Fri, Nov 06 2015, rgoldwyn@xxxxxxx wrote:
> From: Goldwyn Rodrigues <rgoldwyn@xxxxxxxx>
>
> If a spare device was marked faulty, it would not be reflected
> in receiving nodes because it would mark it as activated and continue.
> Continue the operation, so it may be set as faulty.
Yes, that makes sense.
Applied, thanks.
NeilBrown
>
> Signed-off-by: Goldwyn Rodrigues <rgoldwyn@xxxxxxxx>
> ---
> drivers/md/md.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/md/md.c b/drivers/md/md.c
> index 4222285..b98cf04 100644
> --- a/drivers/md/md.c
> +++ b/drivers/md/md.c
> @@ -9013,7 +9013,6 @@ static void check_sb_changes(struct mddev *mddev, struct md_rdev *rdev)
> ret = remove_and_add_spares(mddev, rdev2);
> pr_info("Activated spare: %s\n",
> bdevname(rdev2->bdev,b));
> - continue;
> }
> /* device faulty
> * We just want to do the minimum to mark the disk
> --
> 1.8.5.6
Attachment:
signature.asc
Description: PGP signature
