[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110519174420.GD12525@obsidianresearch.com>
Date: Thu, 19 May 2011 11:44:20 -0600
From: Jason Gunthorpe <jgunthorpe@...idianresearch.com>
To: Bart Van Assche <bvanassche@....org>
Cc: "Nicholas A. Bellinger" <nab@...ux-iscsi.org>,
linux-kernel <linux-kernel@...r.kernel.org>,
linux-scsi <linux-scsi@...r.kernel.org>,
linux-rmda <linux-rdma@...r.kernel.org>,
Roland Dreier <roland@...estorage.com>,
Vu Pham <vu@...lanox.com>, David Dillow <dillowda@...l.gov>,
James Bottomley <James.Bottomley@...senpartnership.com>
Subject: Re: [RFC] ib_srpt: initial .40-rc1 drivers/infiniband/ulp/srpt
merge
On Thu, May 19, 2011 at 07:29:21PM +0200, Bart Van Assche wrote:
> Regarding APM: the Linux kernel already has multipath support and
> duplicate functionality is in general not welcomed. So you will have
> to come up with a very good reason before APM support in ib_srp or
> ib_srpt would be considered for acceptance.
APM is a feature of IB's tranport layerthat provides disruption free
lossless failover to a backup network path completely transparently to
the application (eg SRP in this case).
This plays a similar role in the IB stack to the realtime spanning
tree protocol that ethernet uses, or the multipath IP routing that is
already in the net stack. I don't see it duplicating SCSI multipath at
all. On very large IB networks (eg > 1000 switches) the rate of link
failures is high enough that taking an application level recovery on
hundreds of nodes due to a link failure is troublesome.
If ib_srpt is implemented properly it may already support APM because
APM is supported by the IB CM that it relies on to setup connections,
and any SRP client could already request APM functionality during
connection setup.
Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists