svcrdma: RDMA transport driver close path cleanup.
Subsystems (2)
A message that is not in the archive (sent off-list or not yet received): [email protected]
Quoted 7 lines · Tom Tucker
This is an updated version of the patchset that includes changes per your recommendations, plus:
- cleanup: changed the return type on
svc_rdma_send_errorto void
- bug: added copy of transport address to the rqstp structure in rdma read completion logic
- bug: fixedsvc_rdma_acceptrace that was found by inspection,
OK, thanks. A minor whine: whatever procedure you're using to mail these seems to alphabetize them like:
11/20 1/20 12/20 13/20 14/20 15/20 16/20 17/20 18/20 19/20 20/20 2/20 3/20 ...
So the threading doesn't help apply them in the right order.
Anyway, not a big problem. And I'd also be just as happy to take them from git if that'd be simpler. (But from a quick glance I didn't see a branch with the identical patches?)
--b.
Quoted 129 lines
The original description follows:
This patchset fixes a number of defects in the close path of the SVCRDMA transport provider. The defects were found by injecting errors on the transport at random intervals while running concurrent iozone tests on IB and iWARP mounts. With this set of patches applied I was able to run the above tests for several hours without causing a crash or leaking resources. The transport recovers correctly on a client reconnect.
This patchset is based on 2.6 top of tree, however, I think these fixes are good candidates for a 2.6.25 dot release as well. I have a set that is already merged for that release if you would like them. They are available in my git tree on linux-nfs.org in the '2.6.25' branch. Please let me know if you would like me to post them here.
[PATCH 1/20] svcrdma: Simplify receive buffer posting
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 17 +---------------- net/sunrpc/xprtrdma/svc_rdma_sendto.c | 10 ++++++++++ net/sunrpc/xprtrdma/svc_rdma_transport.c | 19 ------------------- 3 files changed, 11 insertions(+), 35 deletions(-)[PATCH 2/20] svcrdma: Fix race with
dto_taskletinsvc_rdma_sendnet/sunrpc/xprtrdma/svc_rdma_transport.c | 40 ++++++++++++++++------------- 1 files changed, 22 insertions(+), 18 deletions(-)[PATCH 3/20] svcrdma: Fix return value in
svc_rdma_sendnet/sunrpc/xprtrdma/svc_rdma_transport.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)[PATCH 4/20] svcrdma: Add put of connection ESTABLISHED reference in
rdma_cma_handlernet/sunrpc/xprtrdma/svc_rdma_transport.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-)[PATCH 5/20] svcrdma: Free context on
ib_post_recverrornet/sunrpc/xprtrdma/svc_rdma_transport.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-)[PATCH 6/20] svcrdma: Free context on
post_recverror insend_replynet/sunrpc/xprtrdma/svc_rdma_sendto.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-)[PATCH 7/20] svcrdma: Fix error handling during listening endpoint creation
net/sunrpc/xprtrdma/svc_rdma_transport.c | 24 +++++++++++++----------- 1 files changed, 13 insertions(+), 11 deletions(-)[PATCH 8/20] svcrdma: Return error from
rdma_read_xdrso caller knows to free contextnet/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 23 ++++++++++++++++------- 1 files changed, 16 insertions(+), 7 deletions(-)[PATCH 9/20] svcrdma: Remove unused
READ_DONEcontext flags bitinclude/linux/sunrpc/svc_rdma.h | 1 - net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 1 - net/sunrpc/xprtrdma/svc_rdma_transport.c | 1 - 3 files changed, 0 insertions(+), 3 deletions(-)[PATCH 10/20] svcrdma: Simplify
RDMA_READdeferral buffer managementinclude/linux/sunrpc/svc_rdma.h | 1 + net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 58 ++++++------------------------ net/sunrpc/xprtrdma/svc_rdma_transport.c | 5 ++- 3 files changed, 16 insertions(+), 48 deletions(-)[PATCH 11/20] svcrdma: Use standard Linux lists for context cache
include/linux/sunrpc/svc_rdma.h | 5 ++- net/sunrpc/xprtrdma/svc_rdma_transport.c | 47 ++++++++++++++++------------- 2 files changed, 29 insertions(+), 23 deletions(-)[PATCH 12/20] svcrdma: Shrink scope of spinlock on RQ CQ
net/sunrpc/xprtrdma/svc_rdma_transport.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)[PATCH 13/20] svcrdma: Move destroy to kernel thread
include/linux/sunrpc/svc_rdma.h | 1 + net/sunrpc/xprtrdma/svc_rdma_transport.c | 17 ++++++++++++++--- 2 files changed, 15 insertions(+), 3 deletions(-)[PATCH 14/20] svcrdma: Add reference for each SQ/RQ WR
net/sunrpc/xprtrdma/svc_rdma_transport.c | 18 ++++++++++++++++-- 1 files changed, 16 insertions(+), 2 deletions(-)[PATCH 15/20] svcrdma: Move the QP and
cm_iddestruction tosvc_rdma_freenet/sunrpc/xprtrdma/svc_rdma_transport.c | 40 +++++++++++++++--------------- 1 files changed, 20 insertions(+), 20 deletions(-)[PATCH 16/20] svcrdma: Cleanup queued, but unprocessed I/O in
svc_rdma_freenet/sunrpc/xprtrdma/svc_rdma_transport.c | 38 +++++++++++++++++++++++++++-- 1 files changed, 35 insertions(+), 3 deletions(-)[PATCH 17/20] svcrdma: Use ib verbs version of
dma_unmapnet/sunrpc/xprtrdma/svc_rdma_transport.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-)[PATCH 18/20] svcrdma: Set rqstp transport address in
rdma_read_completefunctionnet/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-)[PATCH 19/20] svcrdma: Copy transport address and arm CQ before calling
rdma_acceptnet/sunrpc/xprtrdma/svc_rdma_transport.c | 23 ++++++++++++++--------- 1 files changed, 14 insertions(+), 9 deletions(-)[PATCH 20/20] svcrdma: Change
svc_rdma_send_errorreturn type to voidinclude/linux/sunrpc/svc_rdma.h | 4 ++-- net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 2 +- net/sunrpc/xprtrdma/svc_rdma_transport.c | 9 ++++----- 3 files changed, 7 insertions(+), 8 deletions(-)Signed-off-by Tom Tucker <[email protected]>.
The for-bfields branch should contain those same patches.
Tom
Quoted 177 lines · J. Bruce Fields
Quoted 8 lines · Tom Tucker
This is an updated version of the patchset that includes changes per your recommendations, plus:
- cleanup: changed the return type on
svc_rdma_send_errorto void
- bug: added copy of transport address to the rqstp structure in rdma read
completion logic
- bug: fixedsvc_rdma_acceptrace that was found by inspection,OK, thanks. A minor whine: whatever procedure you're using to mail these seems to alphabetize them like:
11/20 1/20 12/20 13/20 14/20 15/20 16/20 17/20 18/20 19/20 20/20 2/20 3/20 ...So the threading doesn't help apply them in the right order.
Anyway, not a big problem. And I'd also be just as happy to take them from git if that'd be simpler. (But from a quick glance I didn't see a branch with the identical patches?)
--b.
Quoted 129 lines
The original description follows:
This patchset fixes a number of defects in the close path of the SVCRDMA transport provider. The defects were found by injecting errors on the transport at random intervals while running concurrent iozone tests on IB and iWARP mounts. With this set of patches applied I was able to run the above tests for several hours without causing a crash or leaking resources. The transport recovers correctly on a client reconnect.
This patchset is based on 2.6 top of tree, however, I think these fixes are good candidates for a 2.6.25 dot release as well. I have a set that is already merged for that release if you would like them. They are available in my git tree on linux-nfs.org in the '2.6.25' branch. Please let me know if you would like me to post them here.
[PATCH 1/20] svcrdma: Simplify receive buffer posting
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 17 +---------------- net/sunrpc/xprtrdma/svc_rdma_sendto.c | 10 ++++++++++ net/sunrpc/xprtrdma/svc_rdma_transport.c | 19 ------------------- 3 files changed, 11 insertions(+), 35 deletions(-)[PATCH 2/20] svcrdma: Fix race with
dto_taskletinsvc_rdma_send
net/sunrpc/xprtrdma/svc_rdma_transport.c| 40 +++++++++++++++
+-------------1 files changed, 22 insertions(+), 18 deletions(-)[PATCH 3/20] svcrdma: Fix return value in
svc_rdma_send
net/sunrpc/xprtrdma/svc_rdma_transport.c| 2 +-1 files changed, 1 insertions(+), 1 deletions(-)[PATCH 4/20] svcrdma: Add put of connection ESTABLISHED reference in
rdma_cma_handler
net/sunrpc/xprtrdma/svc_rdma_transport.c| 1 +1 files changed, 1 insertions(+), 0 deletions(-)[PATCH 5/20] svcrdma: Free context on
ib_post_recverror
net/sunrpc/xprtrdma/svc_rdma_transport.c| 2 ++1 files changed, 2 insertions(+), 0 deletions(-)[PATCH 6/20] svcrdma: Free context on
post_recverror insend_reply
net/sunrpc/xprtrdma/svc_rdma_sendto.c| 3 ++-1 files changed, 2 insertions(+), 1 deletions(-)[PATCH 7/20] svcrdma: Fix error handling during listening endpoint creation
net/sunrpc/xprtrdma/svc_rdma_transport.c| 24 ++++++++++++
+-----------1 files changed, 13 insertions(+), 11 deletions(-)[PATCH 8/20] svcrdma: Return error from
rdma_read_xdrso caller knows to free context
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c| 23 +++++++++++++++
+-------1 files changed, 16 insertions(+), 7 deletions(-)[PATCH 9/20] svcrdma: Remove unused
READ_DONEcontext flags bitinclude/linux/sunrpc/svc_rdma.h | 1 - net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 1 - net/sunrpc/xprtrdma/svc_rdma_transport.c | 1 - 3 files changed, 0 insertions(+), 3 deletions(-)[PATCH 10/20] svcrdma: Simplify
RDMA_READdeferral buffer management
include/linux/sunrpc/svc_rdma.h| 1 +net/sunrpc/xprtrdma/svc_rdma_recvfrom.c| 58 ++++++------------------------ net/sunrpc/xprtrdma/svc_rdma_transport.c | 5 ++- 3 files changed, 16 insertions(+), 48 deletions(-)[PATCH 11/20] svcrdma: Use standard Linux lists for context cache
include/linux/sunrpc/svc_rdma.h| 5 ++-net/sunrpc/xprtrdma/svc_rdma_transport.c| 47 +++++++++++++++
+-------------2 files changed, 29 insertions(+), 23 deletions(-)[PATCH 12/20] svcrdma: Shrink scope of spinlock on RQ CQ
net/sunrpc/xprtrdma/svc_rdma_transport.c| 4 ++--1 files changed, 2 insertions(+), 2 deletions(-)[PATCH 13/20] svcrdma: Move destroy to kernel thread
include/linux/sunrpc/svc_rdma.h| 1 +net/sunrpc/xprtrdma/svc_rdma_transport.c| 17 ++++++++++++++---2 files changed, 15 insertions(+), 3 deletions(-)[PATCH 14/20] svcrdma: Add reference for each SQ/RQ WR
net/sunrpc/xprtrdma/svc_rdma_transport.c| 18 ++++++++++++++++--1 files changed, 16 insertions(+), 2 deletions(-)[PATCH 15/20] svcrdma: Move the QP and
cm_iddestruction tosvc_rdma_free
net/sunrpc/xprtrdma/svc_rdma_transport.c| 40 ++++++++++++++
+---------------1 files changed, 20 insertions(+), 20 deletions(-)[PATCH 16/20] svcrdma: Cleanup queued, but unprocessed I/O in
svc_rdma_free
net/sunrpc/xprtrdma/svc_rdma_transport.c| 38 ++++++++++++++++++++
+++++++--1 files changed, 35 insertions(+), 3 deletions(-)[PATCH 17/20] svcrdma: Use ib verbs version of
dma_unmap
net/sunrpc/xprtrdma/svc_rdma_transport.c| 8 ++++----1 files changed, 4 insertions(+), 4 deletions(-)[PATCH 18/20] svcrdma: Set rqstp transport address in
rdma_read_completefunction
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c| 1 +1 files changed, 1 insertions(+), 0 deletions(-)[PATCH 19/20] svcrdma: Copy transport address and arm CQ before calling
rdma_accept
net/sunrpc/xprtrdma/svc_rdma_transport.c| 23 +++++++++++++
+---------1 files changed, 14 insertions(+), 9 deletions(-)[PATCH 20/20] svcrdma: Change
svc_rdma_send_errorreturn type to voidQuoted 8 lines
include/linux/sunrpc/svc_rdma.h | 4 ++-- net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 2 +- net/sunrpc/xprtrdma/svc_rdma_transport.c | 9 ++++----- 3 files changed, 7 insertions(+), 8 deletions(-)Signed-off-by Tom Tucker <[email protected]>.
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Sorry for the delay:
Quoted 1 line · Tom Tucker
The for-bfields branch should contain those same patches.
The series in the for-bfield branch includes two patches at the start that I don't think were in the series you mailed ("svcrdma: Remove extra check for XPT_DEAD bit in svc_xprt_enqueue" and "svcrdma: Remove unused header files from svc_xprt.c").
Small patches, so probably doesn't matter either way, but we may as well go with whichever result was actually tested, if only one was?
--b.
Quoted 198 lines
Tom
Quoted 192 lines · J. Bruce Fields
Quoted 8 lines · Tom Tucker
This is an updated version of the patchset that includes changes per your recommendations, plus:
- cleanup: changed the return type on
svc_rdma_send_errorto void
- bug: added copy of transport address to the rqstp structure in rdma read
completion logic
- bug: fixedsvc_rdma_acceptrace that was found by inspection,OK, thanks. A minor whine: whatever procedure you're using to mail these seems to alphabetize them like:
11/20 1/20 12/20 13/20 14/20 15/20 16/20 17/20 18/20 19/20 20/20 2/20 3/20 ...So the threading doesn't help apply them in the right order.
Anyway, not a big problem. And I'd also be just as happy to take them from git if that'd be simpler. (But from a quick glance I didn't see a branch with the identical patches?)
--b.
Quoted 143 lines
The original description follows:
This patchset fixes a number of defects in the close path of the SVCRDMA
transport provider. The defects were found by injecting errors on the transport at random intervals while running concurrent iozone tests on
IB and iWARP mounts. With this set of patches applied I was able to run
the above tests for several hours without causing a crash or leaking resources. The transport recovers correctly on a client reconnect.This patchset is based on 2.6 top of tree, however, I think these fixes are
good candidates for a 2.6.25 dot release as well. I have a set that is
already merged for that release if you would like them. They are available in
my git tree on linux-nfs.org in the '2.6.25' branch. Please let me know if you
would like me to post them here.[PATCH 1/20] svcrdma: Simplify receive buffer posting
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 17 +---------------- net/sunrpc/xprtrdma/svc_rdma_sendto.c | 10 ++++++++++ net/sunrpc/xprtrdma/svc_rdma_transport.c | 19 ------------------- 3 files changed, 11 insertions(+), 35 deletions(-)[PATCH 2/20] svcrdma: Fix race with
dto_taskletinsvc_rdma_send
net/sunrpc/xprtrdma/svc_rdma_transport.c| 40 +++++++++++++++
+-------------1 files changed, 22 insertions(+), 18 deletions(-)[PATCH 3/20] svcrdma: Fix return value in
svc_rdma_send
net/sunrpc/xprtrdma/svc_rdma_transport.c| 2 +-1 files changed, 1 insertions(+), 1 deletions(-)[PATCH 4/20] svcrdma: Add put of connection ESTABLISHED reference in
rdma_cma_handler
net/sunrpc/xprtrdma/svc_rdma_transport.c| 1 +1 files changed, 1 insertions(+), 0 deletions(-)[PATCH 5/20] svcrdma: Free context on
ib_post_recverror
net/sunrpc/xprtrdma/svc_rdma_transport.c| 2 ++1 files changed, 2 insertions(+), 0 deletions(-)[PATCH 6/20] svcrdma: Free context on
post_recverror insend_reply
net/sunrpc/xprtrdma/svc_rdma_sendto.c| 3 ++-1 files changed, 2 insertions(+), 1 deletions(-)[PATCH 7/20] svcrdma: Fix error handling during listening endpoint creation
net/sunrpc/xprtrdma/svc_rdma_transport.c| 24 ++++++++++++
+-----------1 files changed, 13 insertions(+), 11 deletions(-)[PATCH 8/20] svcrdma: Return error from
rdma_read_xdrso caller knows to free context
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c| 23 +++++++++++++++
+-------1 files changed, 16 insertions(+), 7 deletions(-)[PATCH 9/20] svcrdma: Remove unused
READ_DONEcontext flags bitinclude/linux/sunrpc/svc_rdma.h | 1 - net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 1 - net/sunrpc/xprtrdma/svc_rdma_transport.c | 1 - 3 files changed, 0 insertions(+), 3 deletions(-)[PATCH 10/20] svcrdma: Simplify
RDMA_READdeferral buffer management
include/linux/sunrpc/svc_rdma.h| 1 +net/sunrpc/xprtrdma/svc_rdma_recvfrom.c| 58 ++++++------------------------ net/sunrpc/xprtrdma/svc_rdma_transport.c | 5 ++- 3 files changed, 16 insertions(+), 48 deletions(-)[PATCH 11/20] svcrdma: Use standard Linux lists for context cache
include/linux/sunrpc/svc_rdma.h| 5 ++-net/sunrpc/xprtrdma/svc_rdma_transport.c| 47 +++++++++++++++
+-------------2 files changed, 29 insertions(+), 23 deletions(-)[PATCH 12/20] svcrdma: Shrink scope of spinlock on RQ CQ
net/sunrpc/xprtrdma/svc_rdma_transport.c| 4 ++--1 files changed, 2 insertions(+), 2 deletions(-)[PATCH 13/20] svcrdma: Move destroy to kernel thread
include/linux/sunrpc/svc_rdma.h| 1 +net/sunrpc/xprtrdma/svc_rdma_transport.c| 17 ++++++++++++++---2 files changed, 15 insertions(+), 3 deletions(-)[PATCH 14/20] svcrdma: Add reference for each SQ/RQ WR
net/sunrpc/xprtrdma/svc_rdma_transport.c| 18 ++++++++++++++++--1 files changed, 16 insertions(+), 2 deletions(-)[PATCH 15/20] svcrdma: Move the QP and
cm_iddestruction tosvc_rdma_free
net/sunrpc/xprtrdma/svc_rdma_transport.c| 40 ++++++++++++++
+---------------1 files changed, 20 insertions(+), 20 deletions(-)[PATCH 16/20] svcrdma: Cleanup queued, but unprocessed I/O in
svc_rdma_free
net/sunrpc/xprtrdma/svc_rdma_transport.c| 38 ++++++++++++++++++++
+++++++--1 files changed, 35 insertions(+), 3 deletions(-)[PATCH 17/20] svcrdma: Use ib verbs version of
dma_unmap
net/sunrpc/xprtrdma/svc_rdma_transport.c| 8 ++++----1 files changed, 4 insertions(+), 4 deletions(-)[PATCH 18/20] svcrdma: Set rqstp transport address in
rdma_read_completefunction
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c| 1 +1 files changed, 1 insertions(+), 0 deletions(-)[PATCH 19/20] svcrdma: Copy transport address and arm CQ before calling
rdma_accept
net/sunrpc/xprtrdma/svc_rdma_transport.c| 23 +++++++++++++
+---------1 files changed, 14 insertions(+), 9 deletions(-)[PATCH 20/20] svcrdma: Change
svc_rdma_send_errorreturn type to voidQuoted 8 lines
include/linux/sunrpc/svc_rdma.h | 4 ++-- net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 2 +- net/sunrpc/xprtrdma/svc_rdma_transport.c | 9 ++++----- 3 files changed, 7 insertions(+), 8 deletions(-)Signed-off-by Tom Tucker <[email protected]>.
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Quoted 9 lines · J. Bruce Fields
Sorry for the delay:
Quoted 1 line · Tom Tucker
The for-bfields branch should contain those same patches.
The series in the for-bfield branch includes two patches at the start that I don't think were in the series you mailed ("svcrdma: Remove extra check for
XPT_DEADbit insvc_xprt_enqueue" and "svcrdma: Remove unused header files fromsvc_xprt.c").
Ah. So I think I understand the confusion.
I separated changes to the generic transport switch from changes to the RDMA provider and mailed them in separate patchsets.
I do believe that I have posted both the generic transport changes and the RDMA specific changes to the list. I've also posted a new set based on some Connectathon testing.
All that aside... the tree in for-bfields has had iozone testing and Connectathon testing on x86-64.
Tom
Quoted 209 lines
Small patches, so probably doesn't matter either way, but we may as well go with whichever result was actually tested, if only one was?
--b.
Quoted 198 lines
Tom
Quoted 192 lines · J. Bruce Fields
Quoted 8 lines · Tom Tucker
This is an updated version of the patchset that includes changes per your recommendations, plus:
- cleanup: changed the return type on
svc_rdma_send_errorto void
- bug: added copy of transport address to the rqstp structure in rdma read
completion logic
- bug: fixedsvc_rdma_acceptrace that was found by inspection,OK, thanks. A minor whine: whatever procedure you're using to mail these seems to alphabetize them like:
11/20 1/20 12/20 13/20 14/20 15/20 16/20 17/20 18/20 19/20 20/20 2/20 3/20 ...So the threading doesn't help apply them in the right order.
Anyway, not a big problem. And I'd also be just as happy to take them from git if that'd be simpler. (But from a quick glance I didn't see a branch with the identical patches?)
--b.
Quoted 143 lines
The original description follows:
This patchset fixes a number of defects in the close path of the SVCRDMA
transport provider. The defects were found by injecting errors on the transport at random intervals while running concurrent iozone tests on
IB and iWARP mounts. With this set of patches applied I was able to run
the above tests for several hours without causing a crash or leaking resources. The transport recovers correctly on a client reconnect.This patchset is based on 2.6 top of tree, however, I think these fixes are
good candidates for a 2.6.25 dot release as well. I have a set that is
already merged for that release if you would like them. They are available in
my git tree on linux-nfs.org in the '2.6.25' branch. Please let me know if you
would like me to post them here.[PATCH 1/20] svcrdma: Simplify receive buffer posting
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 17 +---------------- net/sunrpc/xprtrdma/svc_rdma_sendto.c | 10 ++++++++++ net/sunrpc/xprtrdma/svc_rdma_transport.c | 19 ------------------- 3 files changed, 11 insertions(+), 35 deletions(-)[PATCH 2/20] svcrdma: Fix race with
dto_taskletinsvc_rdma_send
net/sunrpc/xprtrdma/svc_rdma_transport.c| 40 +++++++++++++++
+-------------1 files changed, 22 insertions(+), 18 deletions(-)[PATCH 3/20] svcrdma: Fix return value in
svc_rdma_send
net/sunrpc/xprtrdma/svc_rdma_transport.c| 2 +-1 files changed, 1 insertions(+), 1 deletions(-)[PATCH 4/20] svcrdma: Add put of connection ESTABLISHED reference in
rdma_cma_handler
net/sunrpc/xprtrdma/svc_rdma_transport.c| 1 +1 files changed, 1 insertions(+), 0 deletions(-)[PATCH 5/20] svcrdma: Free context on
ib_post_recverror
net/sunrpc/xprtrdma/svc_rdma_transport.c| 2 ++1 files changed, 2 insertions(+), 0 deletions(-)[PATCH 6/20] svcrdma: Free context on
post_recverror insend_reply
net/sunrpc/xprtrdma/svc_rdma_sendto.c| 3 ++-1 files changed, 2 insertions(+), 1 deletions(-)[PATCH 7/20] svcrdma: Fix error handling during listening endpoint creation
net/sunrpc/xprtrdma/svc_rdma_transport.c| 24 ++++++++++++
+-----------1 files changed, 13 insertions(+), 11 deletions(-)[PATCH 8/20] svcrdma: Return error from
rdma_read_xdrso caller knows to free context
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c| 23 +++++++++++++++
+-------1 files changed, 16 insertions(+), 7 deletions(-)[PATCH 9/20] svcrdma: Remove unused
READ_DONEcontext flags bitinclude/linux/sunrpc/svc_rdma.h | 1 - net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 1 - net/sunrpc/xprtrdma/svc_rdma_transport.c | 1 - 3 files changed, 0 insertions(+), 3 deletions(-)[PATCH 10/20] svcrdma: Simplify
RDMA_READdeferral buffer management
include/linux/sunrpc/svc_rdma.h| 1 +net/sunrpc/xprtrdma/svc_rdma_recvfrom.c| 58 ++++++------------------------ net/sunrpc/xprtrdma/svc_rdma_transport.c | 5 ++- 3 files changed, 16 insertions(+), 48 deletions(-)[PATCH 11/20] svcrdma: Use standard Linux lists for context cache
include/linux/sunrpc/svc_rdma.h| 5 ++-net/sunrpc/xprtrdma/svc_rdma_transport.c| 47 +++++++++++++++
+-------------2 files changed, 29 insertions(+), 23 deletions(-)[PATCH 12/20] svcrdma: Shrink scope of spinlock on RQ CQ
net/sunrpc/xprtrdma/svc_rdma_transport.c| 4 ++--1 files changed, 2 insertions(+), 2 deletions(-)[PATCH 13/20] svcrdma: Move destroy to kernel thread
include/linux/sunrpc/svc_rdma.h| 1 +net/sunrpc/xprtrdma/svc_rdma_transport.c| 17 ++++++++++++++---2 files changed, 15 insertions(+), 3 deletions(-)[PATCH 14/20] svcrdma: Add reference for each SQ/RQ WR
net/sunrpc/xprtrdma/svc_rdma_transport.c| 18 ++++++++++++++++--1 files changed, 16 insertions(+), 2 deletions(-)[PATCH 15/20] svcrdma: Move the QP and
cm_iddestruction tosvc_rdma_free
net/sunrpc/xprtrdma/svc_rdma_transport.c| 40 ++++++++++++++
+---------------1 files changed, 20 insertions(+), 20 deletions(-)[PATCH 16/20] svcrdma: Cleanup queued, but unprocessed I/O in
svc_rdma_free
net/sunrpc/xprtrdma/svc_rdma_transport.c| 38 ++++++++++++++++++++
+++++++--1 files changed, 35 insertions(+), 3 deletions(-)[PATCH 17/20] svcrdma: Use ib verbs version of
dma_unmap
net/sunrpc/xprtrdma/svc_rdma_transport.c| 8 ++++----1 files changed, 4 insertions(+), 4 deletions(-)[PATCH 18/20] svcrdma: Set rqstp transport address in
rdma_read_completefunction
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c| 1 +1 files changed, 1 insertions(+), 0 deletions(-)[PATCH 19/20] svcrdma: Copy transport address and arm CQ before calling
rdma_accept
net/sunrpc/xprtrdma/svc_rdma_transport.c| 23 +++++++++++++
+---------1 files changed, 14 insertions(+), 9 deletions(-)[PATCH 20/20] svcrdma: Change
svc_rdma_send_errorreturn type to voidQuoted 8 lines
include/linux/sunrpc/svc_rdma.h | 4 ++-- net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 2 +- net/sunrpc/xprtrdma/svc_rdma_transport.c | 9 ++++----- 3 files changed, 7 insertions(+), 8 deletions(-)Signed-off-by Tom Tucker <[email protected]>.
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
A message that is not in the archive (sent off-list or not yet received): 1211162414.31725.15.camel-SMNkleLxa3ZimH42XvhXlA@public.gmane.org
Quoted 11 lines · Tom Tucker
Quoted 9 lines · J. Bruce Fields
Sorry for the delay:
Quoted 1 line · Tom Tucker
The for-bfields branch should contain those same patches.
The series in the for-bfield branch includes two patches at the start that I don't think were in the series you mailed ("svcrdma: Remove extra check for
XPT_DEADbit insvc_xprt_enqueue" and "svcrdma: Remove unused header files fromsvc_xprt.c").
Quoted 4 lines
Ah. So I think I understand the confusion.
I separated changes to the generic transport switch from changes to the RDMA provider and mailed them in separate patchsets.
The subject of the first two patches that update svc_xprt.c were misnamed in my git tree. They had 'svcrdma' as a prefix instead of 'svc'. I have fixed this and updated the tree.
Sorry for the confusion.
Tom
Quoted 224 lines
I do believe that I have posted both the generic transport changes and the RDMA specific changes to the list. I've also posted a new set based on some Connectathon testing.
All that aside... the tree in for-bfields has had iozone testing and Connectathon testing on x86-64.
Tom
Quoted 205 lines
Small patches, so probably doesn't matter either way, but we may as well go with whichever result was actually tested, if only one was?
--b.
Quoted 198 lines
Tom
Quoted 192 lines · J. Bruce Fields
Quoted 8 lines · Tom Tucker
This is an updated version of the patchset that includes changes per your recommendations, plus:
- cleanup: changed the return type on
svc_rdma_send_errorto void
- bug: added copy of transport address to the rqstp structure in rdma read
completion logic
- bug: fixedsvc_rdma_acceptrace that was found by inspection,OK, thanks. A minor whine: whatever procedure you're using to mail these seems to alphabetize them like:
11/20 1/20 12/20 13/20 14/20 15/20 16/20 17/20 18/20 19/20 20/20 2/20 3/20 ...So the threading doesn't help apply them in the right order.
Anyway, not a big problem. And I'd also be just as happy to take them from git if that'd be simpler. (But from a quick glance I didn't see a branch with the identical patches?)
--b.
Quoted 143 lines
The original description follows:
This patchset fixes a number of defects in the close path of the SVCRDMA
transport provider. The defects were found by injecting errors on the transport at random intervals while running concurrent iozone tests on
IB and iWARP mounts. With this set of patches applied I was able to run
the above tests for several hours without causing a crash or leaking resources. The transport recovers correctly on a client reconnect.This patchset is based on 2.6 top of tree, however, I think these fixes are
good candidates for a 2.6.25 dot release as well. I have a set that is
already merged for that release if you would like them. They are available in
my git tree on linux-nfs.org in the '2.6.25' branch. Please let me know if you
would like me to post them here.[PATCH 1/20] svcrdma: Simplify receive buffer posting
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 17 +---------------- net/sunrpc/xprtrdma/svc_rdma_sendto.c | 10 ++++++++++ net/sunrpc/xprtrdma/svc_rdma_transport.c | 19 ------------------- 3 files changed, 11 insertions(+), 35 deletions(-)[PATCH 2/20] svcrdma: Fix race with
dto_taskletinsvc_rdma_send
net/sunrpc/xprtrdma/svc_rdma_transport.c| 40 +++++++++++++++
+-------------1 files changed, 22 insertions(+), 18 deletions(-)[PATCH 3/20] svcrdma: Fix return value in
svc_rdma_send
net/sunrpc/xprtrdma/svc_rdma_transport.c| 2 +-1 files changed, 1 insertions(+), 1 deletions(-)[PATCH 4/20] svcrdma: Add put of connection ESTABLISHED reference in
rdma_cma_handler
net/sunrpc/xprtrdma/svc_rdma_transport.c| 1 +1 files changed, 1 insertions(+), 0 deletions(-)[PATCH 5/20] svcrdma: Free context on
ib_post_recverror
net/sunrpc/xprtrdma/svc_rdma_transport.c| 2 ++1 files changed, 2 insertions(+), 0 deletions(-)[PATCH 6/20] svcrdma: Free context on
post_recverror insend_reply
net/sunrpc/xprtrdma/svc_rdma_sendto.c| 3 ++-1 files changed, 2 insertions(+), 1 deletions(-)[PATCH 7/20] svcrdma: Fix error handling during listening endpoint creation
net/sunrpc/xprtrdma/svc_rdma_transport.c| 24 ++++++++++++
+-----------1 files changed, 13 insertions(+), 11 deletions(-)[PATCH 8/20] svcrdma: Return error from
rdma_read_xdrso caller knows to free context
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c| 23 +++++++++++++++
+-------1 files changed, 16 insertions(+), 7 deletions(-)[PATCH 9/20] svcrdma: Remove unused
READ_DONEcontext flags bitinclude/linux/sunrpc/svc_rdma.h | 1 - net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 1 - net/sunrpc/xprtrdma/svc_rdma_transport.c | 1 - 3 files changed, 0 insertions(+), 3 deletions(-)[PATCH 10/20] svcrdma: Simplify
RDMA_READdeferral buffer management
include/linux/sunrpc/svc_rdma.h| 1 +net/sunrpc/xprtrdma/svc_rdma_recvfrom.c| 58 ++++++------------------------ net/sunrpc/xprtrdma/svc_rdma_transport.c | 5 ++- 3 files changed, 16 insertions(+), 48 deletions(-)[PATCH 11/20] svcrdma: Use standard Linux lists for context cache
include/linux/sunrpc/svc_rdma.h| 5 ++-net/sunrpc/xprtrdma/svc_rdma_transport.c| 47 +++++++++++++++
+-------------2 files changed, 29 insertions(+), 23 deletions(-)[PATCH 12/20] svcrdma: Shrink scope of spinlock on RQ CQ
net/sunrpc/xprtrdma/svc_rdma_transport.c| 4 ++--1 files changed, 2 insertions(+), 2 deletions(-)[PATCH 13/20] svcrdma: Move destroy to kernel thread
include/linux/sunrpc/svc_rdma.h| 1 +net/sunrpc/xprtrdma/svc_rdma_transport.c| 17 ++++++++++++++---2 files changed, 15 insertions(+), 3 deletions(-)[PATCH 14/20] svcrdma: Add reference for each SQ/RQ WR
net/sunrpc/xprtrdma/svc_rdma_transport.c| 18 ++++++++++++++++--1 files changed, 16 insertions(+), 2 deletions(-)[PATCH 15/20] svcrdma: Move the QP and
cm_iddestruction tosvc_rdma_free
net/sunrpc/xprtrdma/svc_rdma_transport.c| 40 ++++++++++++++
+---------------1 files changed, 20 insertions(+), 20 deletions(-)[PATCH 16/20] svcrdma: Cleanup queued, but unprocessed I/O in
svc_rdma_free
net/sunrpc/xprtrdma/svc_rdma_transport.c| 38 ++++++++++++++++++++
+++++++--1 files changed, 35 insertions(+), 3 deletions(-)[PATCH 17/20] svcrdma: Use ib verbs version of
dma_unmap
net/sunrpc/xprtrdma/svc_rdma_transport.c| 8 ++++----1 files changed, 4 insertions(+), 4 deletions(-)[PATCH 18/20] svcrdma: Set rqstp transport address in
rdma_read_completefunction
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c| 1 +1 files changed, 1 insertions(+), 0 deletions(-)[PATCH 19/20] svcrdma: Copy transport address and arm CQ before calling
rdma_accept
net/sunrpc/xprtrdma/svc_rdma_transport.c| 23 +++++++++++++
+---------1 files changed, 14 insertions(+), 9 deletions(-)[PATCH 20/20] svcrdma: Change
svc_rdma_send_errorreturn type to voidQuoted 8 lines
include/linux/sunrpc/svc_rdma.h | 4 ++-- net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 2 +- net/sunrpc/xprtrdma/svc_rdma_transport.c | 9 ++++----- 3 files changed, 7 insertions(+), 8 deletions(-)Signed-off-by Tom Tucker <[email protected]>.
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.htmlQuoted 4 lines
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Quoted 11 lines · J. Bruce Fields
Quoted 7 lines · Tom Tucker
This is an updated version of the patchset that includes changes per your recommendations, plus:
- cleanup: changed the return type on
svc_rdma_send_errorto void
- bug: added copy of transport address to the rqstp structure in rdma read completion logic
- bug: fixedsvc_rdma_acceptrace that was found by inspection,OK, thanks. A minor whine: whatever procedure you're using to mail these seems to alphabetize them like:
Ok, sorry about this. I was relying on the git send-email threading to get the order.
I've modified my tool to format the subject like "%02/%02". This will get you 01/20, 02/20, etc... So they should sort alphabetically for subsequent submissions.
Tom
Quoted 157 lines
11/20 1/20 12/20 13/20 14/20 15/20 16/20 17/20 18/20 19/20 20/20 2/20 3/20 ...So the threading doesn't help apply them in the right order.
Anyway, not a big problem. And I'd also be just as happy to take them from git if that'd be simpler. (But from a quick glance I didn't see a branch with the identical patches?)
--b.
Quoted 121 lines
The original description follows:
This patchset fixes a number of defects in the close path of the SVCRDMA transport provider. The defects were found by injecting errors on the transport at random intervals while running concurrent iozone tests on IB and iWARP mounts. With this set of patches applied I was able to run the above tests for several hours without causing a crash or leaking resources. The transport recovers correctly on a client reconnect.
This patchset is based on 2.6 top of tree, however, I think these fixes are good candidates for a 2.6.25 dot release as well. I have a set that is already merged for that release if you would like them. They are available in my git tree on linux-nfs.org in the '2.6.25' branch. Please let me know if you would like me to post them here.
[PATCH 1/20] svcrdma: Simplify receive buffer posting
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 17 +---------------- net/sunrpc/xprtrdma/svc_rdma_sendto.c | 10 ++++++++++ net/sunrpc/xprtrdma/svc_rdma_transport.c | 19 ------------------- 3 files changed, 11 insertions(+), 35 deletions(-)[PATCH 2/20] svcrdma: Fix race with
dto_taskletinsvc_rdma_sendnet/sunrpc/xprtrdma/svc_rdma_transport.c | 40 ++++++++++++++++------------- 1 files changed, 22 insertions(+), 18 deletions(-)[PATCH 3/20] svcrdma: Fix return value in
svc_rdma_sendnet/sunrpc/xprtrdma/svc_rdma_transport.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)[PATCH 4/20] svcrdma: Add put of connection ESTABLISHED reference in
rdma_cma_handlernet/sunrpc/xprtrdma/svc_rdma_transport.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-)[PATCH 5/20] svcrdma: Free context on
ib_post_recverrornet/sunrpc/xprtrdma/svc_rdma_transport.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-)[PATCH 6/20] svcrdma: Free context on
post_recverror insend_replynet/sunrpc/xprtrdma/svc_rdma_sendto.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-)[PATCH 7/20] svcrdma: Fix error handling during listening endpoint creation
net/sunrpc/xprtrdma/svc_rdma_transport.c | 24 +++++++++++++----------- 1 files changed, 13 insertions(+), 11 deletions(-)[PATCH 8/20] svcrdma: Return error from
rdma_read_xdrso caller knows to free contextnet/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 23 ++++++++++++++++------- 1 files changed, 16 insertions(+), 7 deletions(-)[PATCH 9/20] svcrdma: Remove unused
READ_DONEcontext flags bitinclude/linux/sunrpc/svc_rdma.h | 1 - net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 1 - net/sunrpc/xprtrdma/svc_rdma_transport.c | 1 - 3 files changed, 0 insertions(+), 3 deletions(-)[PATCH 10/20] svcrdma: Simplify
RDMA_READdeferral buffer managementinclude/linux/sunrpc/svc_rdma.h | 1 + net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 58 ++++++------------------------ net/sunrpc/xprtrdma/svc_rdma_transport.c | 5 ++- 3 files changed, 16 insertions(+), 48 deletions(-)[PATCH 11/20] svcrdma: Use standard Linux lists for context cache
include/linux/sunrpc/svc_rdma.h | 5 ++- net/sunrpc/xprtrdma/svc_rdma_transport.c | 47 ++++++++++++++++------------- 2 files changed, 29 insertions(+), 23 deletions(-)[PATCH 12/20] svcrdma: Shrink scope of spinlock on RQ CQ
net/sunrpc/xprtrdma/svc_rdma_transport.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)[PATCH 13/20] svcrdma: Move destroy to kernel thread
include/linux/sunrpc/svc_rdma.h | 1 + net/sunrpc/xprtrdma/svc_rdma_transport.c | 17 ++++++++++++++--- 2 files changed, 15 insertions(+), 3 deletions(-)[PATCH 14/20] svcrdma: Add reference for each SQ/RQ WR
net/sunrpc/xprtrdma/svc_rdma_transport.c | 18 ++++++++++++++++-- 1 files changed, 16 insertions(+), 2 deletions(-)[PATCH 15/20] svcrdma: Move the QP and
cm_iddestruction tosvc_rdma_freenet/sunrpc/xprtrdma/svc_rdma_transport.c | 40 +++++++++++++++--------------- 1 files changed, 20 insertions(+), 20 deletions(-)[PATCH 16/20] svcrdma: Cleanup queued, but unprocessed I/O in
svc_rdma_freenet/sunrpc/xprtrdma/svc_rdma_transport.c | 38 +++++++++++++++++++++++++++-- 1 files changed, 35 insertions(+), 3 deletions(-)[PATCH 17/20] svcrdma: Use ib verbs version of
dma_unmapnet/sunrpc/xprtrdma/svc_rdma_transport.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-)[PATCH 18/20] svcrdma: Set rqstp transport address in
rdma_read_completefunctionnet/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-)[PATCH 19/20] svcrdma: Copy transport address and arm CQ before calling
rdma_acceptnet/sunrpc/xprtrdma/svc_rdma_transport.c | 23 ++++++++++++++--------- 1 files changed, 14 insertions(+), 9 deletions(-)[PATCH 20/20] svcrdma: Change
svc_rdma_send_errorreturn type to voidQuoted 8 lines
include/linux/sunrpc/svc_rdma.h | 4 ++-- net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 2 +- net/sunrpc/xprtrdma/svc_rdma_transport.c | 9 ++++----- 3 files changed, 7 insertions(+), 8 deletions(-)Signed-off-by Tom Tucker <[email protected]>.
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
A message that is not in the archive (sent off-list or not yet received): 1210686283.16787.75.camel-SMNkleLxa3ZimH42XvhXlA@public.gmane.org
Quoted 9 lines · Tom Tucker
Quoted 8 lines · J. Bruce Fields
Quoted 7 lines · Tom Tucker
This is an updated version of the patchset that includes changes per your recommendations, plus:
- cleanup: changed the return type on
svc_rdma_send_errorto void
- bug: added copy of transport address to the rqstp structure in rdma read completion logic
- bug: fixedsvc_rdma_acceptrace that was found by inspection,
Quoted 5 lines
Quoted 2 lines
OK, thanks. A minor whine: whatever procedure you're using to mail these seems to alphabetize them like:
Ok, sorry about this. I was relying on the git send-email threading to get the order.
Unfortunately it doesn't do that if all patches are in reply to #0. Threading helps more when they are all chained, but it's really ugly and less useful especially with a large number of patches.
This script I use: git-send-patch, available here:
http://git.linux-nfs.org/?p=bhalevy/git-tools.git;a=blob_plain;f=git-send-patch;hb=master
Tries to keep the order by watching your mailq (in auto mode) and not sending a patch before its predecessor was sent out. This usually works, though I think mail queues in between you and the list may still send stuff out of order...
Benny
Quoted 167 lines
I've modified my tool to format the subject like "%02/%02". This will get you 01/20, 02/20, etc... So they should sort alphabetically for subsequent submissions.
Tom
Quoted 148 lines
11/20 1/20 12/20 13/20 14/20 15/20 16/20 17/20 18/20 19/20 20/20 2/20 3/20 ...So the threading doesn't help apply them in the right order.
Anyway, not a big problem. And I'd also be just as happy to take them from git if that'd be simpler. (But from a quick glance I didn't see a branch with the identical patches?)
--b.
Quoted 125 lines
The original description follows:
This patchset fixes a number of defects in the close path of the SVCRDMA transport provider. The defects were found by injecting errors on the transport at random intervals while running concurrent iozone tests on IB and iWARP mounts. With this set of patches applied I was able to run the above tests for several hours without causing a crash or leaking resources. The transport recovers correctly on a client reconnect.
This patchset is based on 2.6 top of tree, however, I think these fixes are good candidates for a 2.6.25 dot release as well. I have a set that is already merged for that release if you would like them. They are available in my git tree on linux-nfs.org in the '2.6.25' branch. Please let me know if you would like me to post them here.
[PATCH 1/20] svcrdma: Simplify receive buffer posting
net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 17 +---------------- net/sunrpc/xprtrdma/svc_rdma_sendto.c | 10 ++++++++++ net/sunrpc/xprtrdma/svc_rdma_transport.c | 19 ------------------- 3 files changed, 11 insertions(+), 35 deletions(-)[PATCH 2/20] svcrdma: Fix race with
dto_taskletinsvc_rdma_sendnet/sunrpc/xprtrdma/svc_rdma_transport.c | 40 ++++++++++++++++------------- 1 files changed, 22 insertions(+), 18 deletions(-)[PATCH 3/20] svcrdma: Fix return value in
svc_rdma_sendnet/sunrpc/xprtrdma/svc_rdma_transport.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)[PATCH 4/20] svcrdma: Add put of connection ESTABLISHED reference in
rdma_cma_handlernet/sunrpc/xprtrdma/svc_rdma_transport.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-)[PATCH 5/20] svcrdma: Free context on
ib_post_recverrornet/sunrpc/xprtrdma/svc_rdma_transport.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-)[PATCH 6/20] svcrdma: Free context on
post_recverror insend_replynet/sunrpc/xprtrdma/svc_rdma_sendto.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-)[PATCH 7/20] svcrdma: Fix error handling during listening endpoint creation
net/sunrpc/xprtrdma/svc_rdma_transport.c | 24 +++++++++++++----------- 1 files changed, 13 insertions(+), 11 deletions(-)[PATCH 8/20] svcrdma: Return error from
rdma_read_xdrso caller knows to free contextnet/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 23 ++++++++++++++++------- 1 files changed, 16 insertions(+), 7 deletions(-)[PATCH 9/20] svcrdma: Remove unused
READ_DONEcontext flags bitinclude/linux/sunrpc/svc_rdma.h | 1 - net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 1 - net/sunrpc/xprtrdma/svc_rdma_transport.c | 1 - 3 files changed, 0 insertions(+), 3 deletions(-)[PATCH 10/20] svcrdma: Simplify
RDMA_READdeferral buffer managementinclude/linux/sunrpc/svc_rdma.h | 1 + net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 58 ++++++------------------------ net/sunrpc/xprtrdma/svc_rdma_transport.c | 5 ++- 3 files changed, 16 insertions(+), 48 deletions(-)[PATCH 11/20] svcrdma: Use standard Linux lists for context cache
include/linux/sunrpc/svc_rdma.h | 5 ++- net/sunrpc/xprtrdma/svc_rdma_transport.c | 47 ++++++++++++++++------------- 2 files changed, 29 insertions(+), 23 deletions(-)[PATCH 12/20] svcrdma: Shrink scope of spinlock on RQ CQ
net/sunrpc/xprtrdma/svc_rdma_transport.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)[PATCH 13/20] svcrdma: Move destroy to kernel thread
include/linux/sunrpc/svc_rdma.h | 1 + net/sunrpc/xprtrdma/svc_rdma_transport.c | 17 ++++++++++++++--- 2 files changed, 15 insertions(+), 3 deletions(-)[PATCH 14/20] svcrdma: Add reference for each SQ/RQ WR
net/sunrpc/xprtrdma/svc_rdma_transport.c | 18 ++++++++++++++++-- 1 files changed, 16 insertions(+), 2 deletions(-)[PATCH 15/20] svcrdma: Move the QP and
cm_iddestruction tosvc_rdma_freenet/sunrpc/xprtrdma/svc_rdma_transport.c | 40 +++++++++++++++--------------- 1 files changed, 20 insertions(+), 20 deletions(-)[PATCH 16/20] svcrdma: Cleanup queued, but unprocessed I/O in
svc_rdma_freenet/sunrpc/xprtrdma/svc_rdma_transport.c | 38 +++++++++++++++++++++++++++-- 1 files changed, 35 insertions(+), 3 deletions(-)[PATCH 17/20] svcrdma: Use ib verbs version of
dma_unmapnet/sunrpc/xprtrdma/svc_rdma_transport.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-)[PATCH 18/20] svcrdma: Set rqstp transport address in
rdma_read_completefunctionnet/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-)[PATCH 19/20] svcrdma: Copy transport address and arm CQ before calling
rdma_acceptnet/sunrpc/xprtrdma/svc_rdma_transport.c | 23 ++++++++++++++--------- 1 files changed, 14 insertions(+), 9 deletions(-)[PATCH 20/20] svcrdma: Change
svc_rdma_send_errorreturn type to voidinclude/linux/sunrpc/svc_rdma.h | 4 ++-- net/sunrpc/xprtrdma/svc_rdma_recvfrom.c | 2 +- net/sunrpc/xprtrdma/svc_rdma_transport.c | 9 ++++----- 3 files changed, 7 insertions(+), 8 deletions(-)Quoted 7 lines
Quoted 3 lines
Signed-off-by Tom Tucker <[email protected]>.
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html