From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757182Ab2DTPuh (ORCPT ); Fri, 20 Apr 2012 11:50:37 -0400 Received: from e8.ny.us.ibm.com ([32.97.182.138]:45889 "EHLO e8.ny.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753058Ab2DTPuf (ORCPT ); Fri, 20 Apr 2012 11:50:35 -0400 Subject: Re: [RESEND][PATCH] [SCSI] scsi_dh: allow 3rd party multipath drivers to use scsi_dh_detach From: Chandra Seetharaman Reply-To: sekharan@us.ibm.com To: Mike Snitzer , James Bottomley Cc: linux-scsi@vger.kernel.org, Hannes Reinecke , linux-kernel@vger.kernel.org In-Reply-To: <1334935036.13001.27.camel@dabdike> References: <20111215214440.GA17677@redhat.com> <4EF039D5.5010201@suse.de> <20120405144721.GA18437@redhat.com> <20120420144538.GB8155@redhat.com> <1334935036.13001.27.camel@dabdike> Content-Type: text/plain; charset="UTF-8" Organization: IBM Date: Fri, 20 Apr 2012 10:49:51 -0500 Message-ID: <1334936991.3420.21.camel@chandra-lucid.austin.ibm.com> Mime-Version: 1.0 X-Mailer: Evolution 2.28.3 Content-Transfer-Encoding: 7bit X-Content-Scanned: Fidelis XPS MAILER x-cbid: 12042015-9360-0000-0000-000005986C6A Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Mike, First off, sorry for not responding on this earlier. When I saw the original patch, I was thinking it is a generic policy issue, and James would be a better person to make decision on this, and forgot about it. Now only realized that James was waiting on me. I am not very religious either, and agree with James on changing the Changelog. With that change, Acked-by: Chandra Seetharaman Regards, Chandra On Fri, 2012-04-20 at 19:17 +0400, James Bottomley wrote: > On Fri, 2012-04-20 at 10:45 -0400, Mike Snitzer wrote: > > Allow 3rd party multipath drivers to programatically detach a scsi_dh > > using the scsi_dh_detach() interface. This is as improvement over > > requiring them to write 'detach' to /sys/block/sdX/queue/dh_state > > > > End result is both Linux and 3rd party multipath drivers can coexist > > without compromising Linux's default handling of multipath LUNs. > > > > Linux has suffered from races associated with attaching a scsi_dh to a > > device too late (after an HBA driver has started the SCSI device scan). > > Attaching a scsi_dh too late results in default sense handling that does > > not silently fail IO to passive paths, which creates excessive delays > > and IO errors during normal boot on a system with hundreds of LUNs. > > > > To fix this the appropriate scsi_dh must be attached before the HBA > > driver(s) are even loaded. But some scsi_dh are known to conflict with > > 3rd party multipath drivers (e.g. both scsi_dh_alua and scsi_dh_emc > > conflict with EMC PowerPath). This patch allows 3rd party drivers to > > resolve the conflict by detaching an attached scsi_dh. > > This changelog is a bit misleading, isn't it? > > The basic problem is that binary only third party drivers can't use the > scsi_dh_detach callback because it's GPL only. The only module I know > which suffers this problem is powerpath, is that right? > > So this patch actually relaxes our GPL only policy to allow powerpath to > detach correctly. I really don't like the characterisation in the > current proposed changelog of this being a "third party" problem because > quite a few third party modules actually provide source and are thus GPL > compliant. > > The whole point of GPL only symbols is to make life difficult for binary > modules, so I could just say this might be indicative of the success of > that policy. On the other hand, I'm not going to be religious about > this. If you get the ack of the dh handler maintainer (Chandra > Seetharaman) I'll apply this policy relaxation with a proper changelog > that says what we're doing (allowing powerpath to bind to the symbol). > > James > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-scsi" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >