Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756050Ab0LPDts (ORCPT ); Wed, 15 Dec 2010 22:49:48 -0500 Received: from fieldses.org ([174.143.236.118]:43233 "EHLO fieldses.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755922Ab0LPDtr (ORCPT ); Wed, 15 Dec 2010 22:49:47 -0500 Date: Wed, 15 Dec 2010 22:49:46 -0500 To: Takuma Umeya Cc: linux-nfs@vger.kernel.org, linux-kernel@vger.kernel.org, chuck.lever@oracle.com Subject: Re: [PATCH] nfs4: set source address when callback is generated Message-ID: <20101216034945.GH9646@fieldses.org> References: <20101215050901.GB13532@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20101215050901.GB13532@redhat.com> User-Agent: Mutt/1.5.20 (2009-06-14) From: "J. Bruce Fields" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4586 Lines: 98 On Wed, Dec 15, 2010 at 02:09:01PM +0900, Takuma Umeya wrote: > when callback is generated in NFSv4 server, it doesn't set the source > address. When an alias IP is utilized on NFSv4 server and suppose the > client is accessing via that alias IP (e.g. eth0:0), the client invokes > the callback to the IP address that is set on the original device (e.g. > eth0). This behavior results in timeout of xprt. > The patch sets the IP address that the client should invoke callback to. > > Signed-off-by: Takuma Umeya > > > diff --git a/fs/nfsd/nfs4callback.c b/fs/nfsd/nfs4callback.c > index 143da2e..bb5de7f 100644 > --- a/fs/nfsd/nfs4callback.c > +++ b/fs/nfsd/nfs4callback.c > @@ -483,6 +483,7 @@ int setup_callback_client(struct nfs4_client *clp, struct nfs4_cb_conn *conn) > .net = &init_net, > .address = (struct sockaddr *) &conn->cb_addr, > .addrsize = conn->cb_addrlen, > + .saddress = (struct sockaddr *) &conn->cb_saddr, > .timeout = &timeparms, > .program = &cb_program, > .version = 0, > diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c > index 116cab9..c7d3171 100644 > --- a/fs/nfsd/nfs4state.c > +++ b/fs/nfsd/nfs4state.c > @@ -1177,9 +1177,11 @@ find_unconfirmed_client_by_str(const char *dname, unsigned int hashval, > } > > static void > -gen_callback(struct nfs4_client *clp, struct nfsd4_setclientid *se, u32 scopeid) > +gen_callback(struct nfs4_client *clp, struct nfsd4_setclientid *se, > + u32 scopeid, struct svc_rqst *rqstp) > { > struct nfs4_cb_conn *conn = &clp->cl_cb_conn; > + struct sockaddr s_sa; > unsigned short expected_family; > > /* Currently, we only support tcp and tcp6 for the callback channel */ > @@ -1205,6 +1207,19 @@ gen_callback(struct nfs4_client *clp, struct nfsd4_setclientid *se, u32 scopeid) > > conn->cb_prog = se->se_callback_prog; > conn->cb_ident = se->se_callback_ident; > + switch(expected_family){ > + case AF_INET: > + ((struct sockaddr_in *)&s_sa)->sin_family = AF_INET; > + ((struct sockaddr_in *)&s_sa)->sin_addr = rqstp-> > + rq_daddr.addr; > + break; > + case AF_INET6: > + ((struct sockaddr_in6 *)&s_sa)->sin6_family = AF_INET6; > + ((struct sockaddr_in6 *)&s_sa)->sin6_addr = rqstp-> > + rq_daddr.addr6; > + break; > + } > + rpc_copy_addr((struct sockaddr *)&conn->cb_saddr, &s_sa); Chuck, is there any reason we use this special svc_addr_u union and not a struct sockaddr_storage? This seems more verbose than necessary.... --b. > return; > out_err: > conn->cb_addr.ss_family = AF_UNSPEC; > @@ -1876,7 +1891,7 @@ nfsd4_setclientid(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate, > * for consistent minorversion use throughout: > */ > new->cl_minorversion = 0; > - gen_callback(new, setclid, rpc_get_scope_id(sa)); > + gen_callback(new, setclid, rpc_get_scope_id(sa), rqstp); > add_to_unconfirmed(new, strhashval); > setclid->se_clientid.cl_boot = new->cl_clientid.cl_boot; > setclid->se_clientid.cl_id = new->cl_clientid.cl_id; > diff --git a/fs/nfsd/state.h b/fs/nfsd/state.h > index 39adc27..0cee03d 100644 > --- a/fs/nfsd/state.h > +++ b/fs/nfsd/state.h > @@ -95,6 +95,7 @@ struct nfs4_delegation { > struct nfs4_cb_conn { > /* SETCLIENTID info */ > struct sockaddr_storage cb_addr; > + struct sockaddr_storage cb_saddr; > size_t cb_addrlen; > u32 cb_prog; /* used only in 4.0 case; > per-session otherwise */ > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/