First pass at merging Bart's HA work
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest5 of 11 patches 30 repliesIn review
5 of 11 patches are in this archive. The cover letter announces 11.
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 3/11 | IB/srp: don't send anything on a bad QPdrivers/ | +8 | 2 replies | |
| 4/11 | IB/srp: keep processing commands during host removaldrivers/ | +6 | 2 replies | |
| 8/11 | srp_transport: Fix attribute registrationdrivers/ | +2 | 0 replies | |
| 9/11 | srp_transport: Simplify attribute initialization codedrivers/ | +4 | 1 reply | |
| 11/11 | IB/srp: Allow SRP disconnect through sysfsDocumentation/ drivers/ drivers/ +1 more | +46 | 0 replies |
Reviews trailers per patch
None of the 5 patches has a Reviewed-by or Acked-by.
| Patch | Bart Van Assche |
|---|---|
| Series | Needs-to-be-signed-off-by |
| 3/11 | Needs-to-be-signed-off-by |
| 4/11 | |
| 8/11 | |
| 9/11 | |
| 11/11 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH 00/11] First pass at merging Bart's HA work
Here is a first, UNTESTED, pass at preparing a merge of Bart's SRP HA work to upstream. It is not complete, as I have not yet added the transport layer error handling and related patches. It is also currently missing the patch to maintain a single connection for an I_T nexus.
I swapped Ishai's code to recreate QP/CQs for each connection, as that adds recovery from fatal hardware errors, and reduces the code needed to avoid stale completions. Similarly, Vu's patch slightly reduces reconnect times.
Target blocking/unblocking hasn't been added from Bart's patch that removes the SRP_TARGET_CONNECTING state; I think it is probably the right thing to do, but want to think it over a bit more.
Similarly, I'm looking for a clean way to start the reconnection effort as soon as we get an error through the CQ -- we know that any pending commands are lost to us at that point, so we should be able to kick them back upstream quickly. This should allow multipath to reissue them on one of the remaining good paths.
This series compiles, but is otherwise UNTESTED. I'll be working on that over the next few days, with an eye on getting as much of Bart's work into 3.8 as possible.
One may also pull this series from github:
git pull git://github.com/dillow/srp-initiator.git ha-merge-v1
Bart Van Assche (7):
IB/srp: enlarge block layer timeout
IB/srp: keep processing commands during host removal IB/srp: Document sysfs attributes
srp_transport: Fix attribute registration
srp_transport: Simplify attribute initialization code
srp_transport: Document sysfs attributes IB/srp: Allow SRP disconnect through sysfs
David Dillow (2):
IB/srp: simplify state tracking
IB/srp: don't send anything on a bad QP
Ishai Rabinovitz (1):
IB/srp: destroy and recreate QP and CQs on each connection
Vu Pham (1):
IB/srp: send disconnect request without waiting for CM timewait exit
Documentation/ABI/stable/sysfs-driver-ib_srp | 156 +++++++++++++++ Documentation/ABI/stable/sysfs-transport-srp | 19 ++ drivers/infiniband/ulp/srp/ib_srp.c | 274 +++++++++++++++----------- drivers/infiniband/ulp/srp/ib_srp.h | 13 +- drivers/scsi/scsi_transport_srp.c | 51 +++--- include/scsi/scsi_transport_srp.h | 8 + 6 files changed, 378 insertions(+), 143 deletions(-) create mode 100644 Documentation/ABI/stable/sysfs-driver-ib_srp create mode 100644 Documentation/ABI/stable/sysfs-transport-srp
Signature · 6 lines
1.7.7.6 -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html