[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-rdma
Subject:    Re: [PATCH v3 1/3] IB/srp: Track connection state properly
From:       Or Gerlitz <or.gerlitz () gmail ! com>
Date:       2012-12-20 20:06:55
Message-ID: CAJZOPZLE8hRR9vKq_iXU46tyEQhz6fsQzLfh+J9N+0BL9xLyrA () mail ! gmail ! com
[Download RAW message or body]

On Thu, Dec 20, 2012 at 6:27 PM, David Dillow <dave@thedillows.org> wrote:
> On Thu, 2012-12-20 at 17:27 +0200, Or Gerlitz wrote:

>> Dave, did you gave the patches a try?

> This one looks to be an obvious fix, but I have not tested the others; I
> have been relying on Bart, Alex, and yourself for the validation.

Taking into account that we've been going back and forth w.r.t that
validation -- that is initially things weren't working on our setups,
later with picking some patches from Bart's tree, that host removal
went OK, and now with the patches posted here, it doesn't go fine on
my setup  -- all in all, sounds like a 2nd opinion which is strengthen
by a test is needed here. You only need to boot with Roland's for-next
+ Bart's patches, connect to a target, take down the IB link and issue
dd. No multipath scheme is needed for the very basic validation, can
you?
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread]