From patchwork Tue Jan 5 22:43:44 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: babu moger X-Patchwork-Id: 71191 X-Patchwork-Delegate: agk@redhat.com Received: from mx01.util.phx2.redhat.com (mx1-phx2.redhat.com [209.132.183.26]) by demeter.kernel.org (8.14.3/8.14.2) with ESMTP id o05MmVW3005560 for ; Tue, 5 Jan 2010 22:48:32 GMT Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by mx01.util.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id o05MiE76024252; Tue, 5 Jan 2010 17:44:15 -0500 Received: from int-mx01.intmail.prod.int.phx2.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id o05MiD06004672 for ; Tue, 5 Jan 2010 17:44:13 -0500 Received: from mx1.redhat.com (ext-mx06.extmail.prod.ext.phx2.redhat.com [10.5.110.10]) by int-mx01.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id o05Mi8FD012362 for ; Tue, 5 Jan 2010 17:44:08 -0500 Received: from na3sys009aog113.obsmtp.com (na3sys009aog113.obsmtp.com [74.125.149.209]) by mx1.redhat.com (8.13.8/8.13.8) with ESMTP id o05Mhpl4016340 for ; Tue, 5 Jan 2010 17:43:52 -0500 Received: from source ([147.145.40.20]) by na3sys009aob113.postini.com ([74.125.148.12]) with SMTP ID DSNKS0PApwtwtDt652IyR0SBUrHFSk7VUoT7@postini.com; Tue, 05 Jan 2010 14:43:53 PST Received: from milmhbs0.lsil.com (mhbs.lsil.com [147.145.1.30]) by mail0.lsil.com (8.12.11/8.12.11) with ESMTP id o05MhnZJ001039; Tue, 5 Jan 2010 14:43:49 -0800 (PST) Received: from coshub01.lsi.com (coshub01.co.lsil.com [172.21.36.64]) by milmhbs0.lsil.com (8.12.11/8.12.11) with ESMTP id o05MhnGo014335; Tue, 5 Jan 2010 14:43:49 -0800 Received: from cosmail01.lsi.com ([172.21.36.24]) by coshub01.lsi.com ([172.21.36.64]) with mapi; Tue, 5 Jan 2010 15:43:48 -0700 From: "Moger, Babu" To: "dm-devel@redhat.com" , "linux-scsi@vger.kernel.org" Date: Tue, 5 Jan 2010 15:43:44 -0700 Thread-Topic: [PATCH] dm-mpath: Two minor fixes while processing activate_path Thread-Index: AcqOWImTyUbr9wVyQMKwUczFDstnuw== Message-ID: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: acceptlanguage: en-US MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.67 on 10.5.11.11 X-Scanned-By: MIMEDefang 2.67 on 10.5.110.10 X-Scanned-By: MIMEDefang 2.39 X-RedHat-Spam-Score: -3.895 (AWL,RCVD_IN_DNSWL_MED) X-MIME-Autoconverted: from quoted-printable to 8bit by lists01.pubmisc.prod.ext.phx2.redhat.com id o05MiD06004672 X-loop: dm-devel@redhat.com Subject: [dm-devel] [PATCH] dm-mpath: Two minor fixes while processing activate_path X-BeenThere: dm-devel@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk Reply-To: device-mapper development List-Id: device-mapper development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dm-devel-bounces@redhat.com Errors-To: dm-devel-bounces@redhat.com --- linux-2.6.33-rc2/drivers/md/dm-mpath.c.orig 2010-01-05 12:47:26.000000000 -0400 +++ linux-2.6.33-rc2/drivers/md/dm-mpath.c 2010-01-05 17:07:35.000000000 -0400 @@ -461,6 +461,9 @@ 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++; } @@ -1143,8 +1146,7 @@ static void pg_init_done(void *data, int errors = 0; break; } - DMERR("Cannot failover device because scsi_dh_%s was not " - "loaded.", m->hw_handler_name); + DMERR("Could not failover device. Error %d.", errors); /* * Fail path for now, so we do not ping pong */