Re: [RFC,PATCH 3/14] knfsd: prepare reply per transport

["J. Bruce Fields" <bfields@fieldses.org>]

On Fri, May 18, 2007 at 12:01:37AM -0400, bfields wrote:
> On Fri, May 18, 2007 at 01:16:30PM +1000, Greg Banks wrote:
> > On Thu, May 17, 2007 at 02:16:46AM -0700, Iyer, Rahul wrote:
> > > I've written some code to do NFSv4.1 callbacks and wound up implementing
> > > another "transport". Since in v4.1 it's actually possible for clients to
> > > send requests (fore-channel) and receive callbacks (back channel) over
> > > the same connection,
> > 
> > A very sensible thing to do, but hard to retrofit to the existing Linux
> > code without significant surgery.  As you've discovered, the server and
> > client code are two mostly-separate code bases (with mostly-separate
> > authors and maintainers) that happen to link into the same module.
> > Unifying these would be a major job.  I'd love to see it happen, for
> > example to unify the XDR buffering code, but it would be an uphill
> > battle technically and possibly politically also.  For some reason,
> > code (like lockd) that lives in both the server and client side tends
> > to be neglected by both camps.
> > 
> > > Given this, a unified transport switch would really rock. 
> > > [...] Maybe I'm oversimplifying, but this seems doable.
> > 
> > Yes...eventually.
> 
> And the easiest approach may be to go ahead and introduce the completely
> separate server-side transport switch and then later find ways increase
> code sharing between the two in incremental steps....

(But I don't think people who are interested in doing this should be
scared off by "politics".  I'm sure careful patches would be welcomed by
everyone.  Care is required, though--small differences between the
assumptions made in the two code bases can tricky.  Anyway, that's my
lame excuse for why I ended up with such messy rpcsec_gss privacy code.)

--b.

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