From 3f8af81457759e9a71d5061f84c23d89bff96f89 Mon Sep 17 00:00:00 2001 Date: Sat, 6 Mar 2010 02:29:49 +0000 Subject: [RHEL5.7 PATCH 5/8] dm mpath: skip activate_path for failed paths BZ 673058 Upstream commit f7b934c8127deebf4eb56fbe4a4ae0da16b6dbcd Author: Moger, Babu Date: Sat Mar 6 02:29:49 2010 +0000 dm mpath: skip activate_path for failed paths This patch adds two minor fixes while processing device mapper path activation. 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. In some case this might cause prolonged retries unnecessarily. Change the misleading message if the path being activated fails with SCSI_DH_NOSYS. Signed-off-by: Babu Moger Signed-off-by: Alasdair G Kergon --- drivers/md/dm-mpath.c | 7 +++++-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/md/dm-mpath.c b/drivers/md/dm-mpath.c index c49bc0e..4de9a33 100644 --- a/drivers/md/dm-mpath.c +++ b/drivers/md/dm-mpath.c @@ -449,6 +449,9 @@ static void process_queued_ios(void *data) init_required = 1; } else { list_for_each_entry(tmp, &pgpath->pg->pgpaths, list) { + /* Skip failed paths */ + if (!tmp->path.is_active) + continue; if (queue_delayed_work(kmpath_handlerd, &tmp->activate_path, m->pg_init_delay ? m->pg_init_delay_secs * HZ : 0)) @@ -1199,8 +1202,8 @@ static void pg_init_done(void *data, int errors) errors = 0; break; } - DMERR("Cannot failover device because scsi_dh_%s was not " - "loaded.", m->hw_handler_name); + DMERR("Could not failover the device: Handler scsi_dh_%s " + "Error %d.", m->hw_handler_name, errors); /* * Fail path for now, so we do not ping pong */ -- 1.7.3.4