diff options
author | Chuck Lever <chuck.lever@oracle.com> | 2007-09-11 18:00:47 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2007-10-09 17:16:56 -0400 |
commit | 906462af4c707ba0238f3579fdb2b594c4ea29c3 (patch) | |
tree | f3c15855aaef2353700f1bc91c56108c50ca4150 /net/sunrpc/rpcb_clnt.c | |
parent | 2429cbf6a1566b8e92436d615387e4250feab46b (diff) |
SUNRPC: Split another new rpcbind retry error code from EACCES
Add more new error code processing to the kernel's rpcbind client
and to call_bind_status() to distinguish two cases:
Case 1: the remote has replied that the program/version tuple is not
registered (returns EACCES)
Case 2: retry with a lesser rpcbind version (rpcb now returns EPFNOSUPPORT)
This change allows more specific error processing for each of these two
cases. We now fail case 2 instead of retrying... it's a server
configuration error not to support even rpcbind version 2. And don't
expose this new error code to user land -- convert it to EIO before
failing the RPC.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'net/sunrpc/rpcb_clnt.c')
-rw-r--r-- | net/sunrpc/rpcb_clnt.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/sunrpc/rpcb_clnt.c b/net/sunrpc/rpcb_clnt.c index 9b06a217ab86..d7b9f02489e6 100644 --- a/net/sunrpc/rpcb_clnt.c +++ b/net/sunrpc/rpcb_clnt.c | |||
@@ -377,7 +377,7 @@ void rpcb_getport_async(struct rpc_task *task) | |||
377 | } | 377 | } |
378 | if (info[xprt->bind_index].rpc_proc == NULL) { | 378 | if (info[xprt->bind_index].rpc_proc == NULL) { |
379 | xprt->bind_index = 0; | 379 | xprt->bind_index = 0; |
380 | status = -EACCES; /* tell caller to try again later */ | 380 | status = -EPFNOSUPPORT; |
381 | dprintk("RPC: %5u %s: no more getport versions available\n", | 381 | dprintk("RPC: %5u %s: no more getport versions available\n", |
382 | task->tk_pid, __FUNCTION__); | 382 | task->tk_pid, __FUNCTION__); |
383 | goto bailout_nofree; | 383 | goto bailout_nofree; |