From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Tucker Subject: Re: [RFC, PATCH 3/15] knfsd: initialize RDMA transport in sunrpc Date: Fri, 18 May 2007 14:25:50 -0500 Message-ID: <1179516350.23385.165.camel@trinity.ogc.int> References: <1179510296.23385.116.camel@trinity.ogc.int> <464DF9E0.3050104@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Cc: Neil Brown , Tom Talpey , Linux NFS Mailing List , Peter Leckie , Greg Banks To: chuck.lever@oracle.com Return-path: Received: from sc8-sf-mx2-b.sourceforge.net ([10.3.1.92] helo=mail.sourceforge.net) by sc8-sf-list2-new.sourceforge.net with esmtp (Exim 4.43) id 1Hp85J-0003H7-7n for nfs@lists.sourceforge.net; Fri, 18 May 2007 12:25:53 -0700 Received: from rrcs-71-42-183-126.sw.biz.rr.com ([71.42.183.126] helo=smtp.opengridcomputing.com) by mail.sourceforge.net with esmtp (Exim 4.44) id 1Hp85L-0004Wh-Sn for nfs@lists.sourceforge.net; Fri, 18 May 2007 12:25:56 -0700 In-Reply-To: <464DF9E0.3050104@oracle.com> List-Id: "Discussion of NFS under Linux development, interoperability, and testing." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: nfs-bounces@lists.sourceforge.net Errors-To: nfs-bounces@lists.sourceforge.net On Fri, 2007-05-18 at 15:09 -0400, Chuck Lever wrote: > As part of the client side transport switch, I constructed a transport > "register" and "unregister" interface so that when a transport module is > loaded, it's init routine is invoked automatically, and likewise when it > is unloaded its cleanup routine is automatically invoked. > > I'd like to see the same here. I don't think we want to create a > situation where you have to make source code changes in order to add new > transport capabilities. Especially the distributors would be allergic > to such a constraint. > > Please consider this change while going about the process of integrating > the server-side transport switch with your RDMA transport. Agreed. > > Tom Tucker wrote: > > Add calls to the svc_rdma initialization and cleanup routines to the > > sunrpc module init logic. > > > > Signed-off-by: Tom Tucker > > --- > > > > net/sunrpc/sunrpc_syms.c | 10 +++++++++- > > 1 files changed, 9 insertions(+), 1 deletions(-) > > > > diff --git a/net/sunrpc/sunrpc_syms.c b/net/sunrpc/sunrpc_syms.c > > index 43ecf62..16a50c9 100644 > > --- a/net/sunrpc/sunrpc_syms.c > > +++ b/net/sunrpc/sunrpc_syms.c > > @@ -20,7 +20,9 @@ #include > > #include > > #include > > #include > > - > > +#ifdef CONFIG_NFSD_RDMA > > +#include > > +#endif > > > > /* RPC scheduler */ > > EXPORT_SYMBOL(rpc_execute); > > @@ -158,6 +160,9 @@ #endif > > cache_register(&ip_map_cache); > > cache_register(&unix_gid_cache); > > init_socket_xprt(); > > +#ifdef CONFIG_NFSD_RDMA > > + err = svc_rdma_init(); > > +#endif > > out: > > return err; > > } > > @@ -175,6 +180,9 @@ cleanup_sunrpc(void) > > #ifdef RPC_DEBUG > > rpc_unregister_sysctl(); > > #endif > > +#ifdef CONFIG_NFSD_RDMA > > + svc_rdma_cleanup(); > > +#endif > > #ifdef CONFIG_PROC_FS > > rpc_proc_exit(); > > #endif > > > > > > ------------------------------------------------------------------------- > > This SF.net email is sponsored by DB2 Express > > Download DB2 Express C - the FREE version of DB2 express and take > > control of your XML. No limits. Just data. Click to get it now. > > http://sourceforge.net/powerbar/db2/ > > _______________________________________________ > > NFS maillist - NFS@lists.sourceforge.net > > https://lists.sourceforge.net/lists/listinfo/nfs > ------------------------------------------------------------------------- This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now. http://sourceforge.net/powerbar/db2/ _______________________________________________ NFS maillist - NFS@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/nfs