Re: md/raid1:Remove unnecessary judgement.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, 31 Mar 2012 10:05:23 +0800 "majianpeng" <majianpeng@xxxxxxxxx> wrote:

> >From 033a0330eb210750ec09202ed95696b54d9afeba Mon Sep 17 00:00:00 2001
> From: majianpeng <majianpeng@xxxxxxxxx>
> Date: Sat, 31 Mar 2012 09:57:42 +0800
> Subject: [PATCH] md/raid1:Remove unnecessary judgement. 
> If success == 1 then  break the do{}while,so did not judge !success.
> 
> 
> Signed-off-by: majianpeng <majianpeng@xxxxxxxxx>
> ---
>  drivers/md/raid1.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/md/raid1.c b/drivers/md/raid1.c
> index 4a40a20..649941b 100644
> --- a/drivers/md/raid1.c
> +++ b/drivers/md/raid1.c
> @@ -1624,7 +1624,7 @@ static int fix_sync_read_error(struct r1bio *r1_bio)
>  			d++;
>  			if (d == conf->raid_disks * 2)
>  				d = 0;
> -		} while (!success && d != r1_bio->read_disk);
> +		} while (d != r1_bio->read_disk);
>  
>  		if (!success) {
>  			char b[BDEVNAME_SIZE];

This is another micro-optimisation that doesn't really buy us anything.

I actually like the fact that the 'while' condition makes it clear that lot
loop doesn't continue after we reach 'success'.  So I think I'll leave it the
way it is.

Thanks,
NeilBrown

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux RAID Wiki]     [ATA RAID]     [Linux SCSI Target Infrastructure]     [Linux Block]     [Linux IDE]     [Linux SCSI]     [Linux Hams]     [Device Mapper]     [Device Mapper Cryptographics]     [Kernel]     [Linux Admin]     [Linux Net]     [GFS]     [RPM]     [git]     [Yosemite Forum]


  Powered by Linux