[dm-devel] [PATCH] dm mpath: Skip calling activate_path if the path is already failed

[Moger, Babu]

Alasdair, Can you please pick up this patch in your next update..

Hannes, Any comments from you? I know you have already reviewed this in another thread.

Thanks
Babu Moger 

> -----Original Message-----
> From: dm-devel-bounces at redhat.com [mailto:dm-devel-bounces at redhat.com] On
> Behalf Of Moger, Babu
> Sent: Wednesday, July 29, 2009 10:47 AM
> To: dm-devel at redhat.com
> Subject: [dm-devel] [PATCH] dm mpath: Skip calling activate_path if the
> path is already failed
> 
> This patch adds a check to skip failed paths while calling activate_path.
> If the path is already failed then activate_path will fail for sure. We
> don't have to call in that case.
> 
> Signed-off-by: Babu Moger <babu.moger at lsi.com>
> ---
> 
> 
> --- linux-2.6.31-rc3/drivers/md/dm-mpath.c.orig	2009-07-27
> 10:30:39.000000000 -0500
> +++ linux-2.6.31-rc3/drivers/md/dm-mpath.c	2009-07-27
> 10:35:50.000000000 -0500
> @@ -454,6 +454,10 @@ static void process_queued_ios(struct wo
>  		m->pg_init_count++;
>  		m->pg_init_required = 0;
>  		list_for_each_entry(tmp, &pgpath->pg->pgpaths, list) {
> +			/* Skip failed paths */
> +			if (!tmp->is_active)
> +				continue;
> +
>  			if (queue_work(kmpath_handlerd, &tmp->activate_path))
>  				m->pg_init_in_progress++;
>  		}
> 
> 
> 
> --
> dm-devel mailing list
> dm-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/dm-devel