diff options
author | Chuck Lever <chuck.lever@oracle.com> | 2016-06-29 17:53:43 (GMT) |
---|---|---|
committer | Anna Schumaker <Anna.Schumaker@Netapp.com> | 2016-07-11 19:50:43 (GMT) |
commit | 7a89f9c626e337ba6528d8a2829b228c933877fb (patch) | |
tree | fb37fd67ab882087a162d6ae0d1966909f7ee4e6 /net/mpls | |
parent | 3d4cf35bd4fab56c3aa0ec4323fccb24970aaf79 (diff) | |
download | linux-7a89f9c626e337ba6528d8a2829b228c933877fb.tar.xz |
xprtrdma: Honor ->send_request API contract
Commit c93c62231cf5 ("xprtrdma: Disconnect on registration failure")
added a disconnect for some RPC marshaling failures. This is needed
only in a handful of cases, but it was triggering for simple stuff
like temporary resource shortages. Try to straighten this out.
Fix up the lower layers so they don't return -ENOMEM or other error
codes that the RPC client's FSM doesn't explicitly recognize.
Also fix up the places in the send_request path that do want a
disconnect. For example, when ib_post_send or ib_post_recv fail,
this is a sign that there is a send or receive queue resource
miscalculation. That should be rare, and is a sign of a software
bug. But xprtrdma can recover: disconnect to reset the transport and
start over.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Tested-by: Steve Wise <swise@opengridcomputing.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
Diffstat (limited to 'net/mpls')
0 files changed, 0 insertions, 0 deletions