Re: [RFC, PATCH 13/35] svc: Change services to use new svc_create_xprt service

[Greg Banks <gnb@sgi.com>]

On Tue, Oct 02, 2007 at 11:44:50AM -0400, Chuck Lever wrote:
> On Oct 1, 2007, at 3:27 PM, Tom Tucker wrote:
> >
> >-static int find_socket(struct svc_serv *serv, int proto)
> >+static int find_xprt(struct svc_serv *serv, char *proto)
> > {
> > 	struct svc_sock *svsk;
> > 	int found = 0;
> > 	list_for_each_entry(svsk, &serv->sv_permsocks, sk_list)
> >-		if (svsk->sk_sk->sk_protocol == proto) {
> >+		if (strcmp(svsk->sk_xprt.xpt_class->xcl_name, proto) == 0) {
> > 			found = 1;
> > 			break;
> > 		}
> 
> This is scary.  :-)
> 
> First, I think we would be better off making the server transport API  
> stronger by not allowing ULPs to dig around in svc_xprt or the  
> svc_xprt_class structures directly.  Perhaps you could provide a  
> method for obtaining the transport's NETID.

Or a function to do the match:

struct svc_xprt_ops {
    ...
    int (*xpo_netid_supported)(const char *netid);
    ...
};

Or, your could push the find_xprt() function down into svc_xprt.c
where such futzing belongs.

> Second, is there any guarantee that the string name of the underlying  
> protocol is the same as the name of the transport class?  Is there  
> any relationship between the transport name and the NETIDs it supports?

This is already confused: our "tcp" transport supports both TCP/IPv4
and TCP/IPv6, which are two separate netids "tcp4" and "tcp6" in
TLI jargon.

Greg.
-- 
Greg Banks, R&D Software Engineer, SGI Australian Software Group.
Apparently, I'm Bedevere.  Which MPHG character are you?
I don't speak for SGI.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
NFS maillist  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs