Re: [RFC,PATCH 2/14] knfsd: delete per transport

[Greg Banks <gnb@sgi.com>]

On Fri, May 18, 2007 at 04:44:22PM +1000, Neil Brown wrote:
> On Friday May 18, gnb@sgi.com wrote:
> > 
> > I considered renaming the existing svc_sock to svc_xprt, which
> > would be another horribly unpronouncable identifier(*) but would at
> > least be appropriately abstract and consistent with the client.
> > 
> > Then the naming would all be obvious and correct: svc_xprt_foo()
> > is a generic transport function, svc_sock_foo() is a more specific
> > socket-based transport function used as common code between TCP
> > and UDP transports.  Yay.
> > 
> > But it would be a much larger patch so I shied away.
> 
> Sensible.
> 
> > 
> > I'm happy to do the naming any way you want, just let me know.
> 
> I accept that "ipsock" is not accurate (as it is the 'sock' bit that
> is significant, the 'ip' is fairly irrelevant) but it is at least
> better than tcpip.  Another option is
>    svc_detach_sock / svc_free_sock
> 
> Which avoid mentioned "svc_sock", but still says what you want.
> 
> So go with that, unless you find it objectionable, in which case got
> with svc_ipsock_*.

I really like having the module at the start and the verb at the end,
so I'll go for svc_ipsock_foo().

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 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