On Mon, Dec 07, 2009 at 12:34:28PM +0100, Roel Kluin wrote: > IS_ERR returns 1 or 0, PTR_ERR returns the error value. Whoops, thanks, looks like that was my mistake, from 79a3f20b641f9f93787ada49d1d7cfa98ee5a11e "rpc: use count of pipe openers to wait for first open". --b. > > Signed-off-by: Roel Kluin <roel.kluin@xxxxxxxxx> > --- > net/sunrpc/auth_gss/auth_gss.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/net/sunrpc/auth_gss/auth_gss.c b/net/sunrpc/auth_gss/auth_gss.c > index fc6a43c..129d75e 100644 > --- a/net/sunrpc/auth_gss/auth_gss.c > +++ b/net/sunrpc/auth_gss/auth_gss.c > @@ -485,7 +485,7 @@ gss_refresh_upcall(struct rpc_task *task) > dprintk("RPC: %5u gss_refresh_upcall for uid %u\n", task->tk_pid, > cred->cr_uid); > gss_msg = gss_setup_upcall(task->tk_client, gss_auth, cred); > - if (IS_ERR(gss_msg) == -EAGAIN) { > + if (PTR_ERR(gss_msg) == -EAGAIN) { > /* XXX: warning on the first, under the assumption we > * shouldn't normally hit this case on a refresh. */ > warn_gssd(); -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html