From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ryan O'Hara <rohara@redhat.com>
Date: Mon, 24 Jan 2011 14:57:12 -0600
Subject: [Cluster-devel] [PATCH] fence_scsi: always do sg_turs before
 registration
In-Reply-To: <1295902307.4673.6230.camel@ayanami.boston.devel.redhat.com>
References: <1295895478-20961-1-git-send-email-rohara@redhat.com>
	<1295902307.4673.6230.camel@ayanami.boston.devel.redhat.com>
Message-ID: <20110124205711.GD17856@redhat.com>
List-Id: <cluster-devel.redhat.com>
To: cluster-devel.redhat.com
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit

On Mon, Jan 24, 2011 at 03:51:47PM -0500, Lon Hohberger wrote:
> On Mon, 2011-01-24 at 12:57 -0600, Ryan O'Hara wrote:
> > This patch fixes a with the original patch posted last week. Before
> > attempting to register with a device, we should call sg_turs on the
> > device. If that device is a dm-multipath device, we should call sg_turs
> > on all the paths. This can be done by simply moving the call to do_reset
> > to do_register and do_register_ignore.
> > 
> > Also, the patch adds a call to do_reset before doing a "status" action.
> > If a node has been fenced and then proceeds to check its status, it will
> > be in "unit attention" state.
> > 
> > Resolves: rhbz#640343
> 
> Was reusing the same patch synopsis intentional here?  It doesn't look
> like a replacement of the patch; it looks like an additional patch. 
> 
> -- Lon

Not intentional, no. Its the same bug, and the 2nd patch is just a
tweak to the first patch so that is properly resets all paths for
dm-mp devices.

Sorry for any confusion.

Ryan