In my most recent Napalm patch, I'd moved all the code that accesses
the function_desc in nfs_worker_thread.c into nfs_worker_thread.c,
hopefully making it easier to understand.

Looking at it today, I remember that I've had a long time question.
Both uses of xdr_free() use xdr_decode_func().

Frank added a free_function() in 2014:

fdcd2f9e (Frank S. Filz                2014-03-07 18:27:37 -0500   78)  
.service_function = nfs_null,
fdcd2f9e (Frank S. Filz                2014-03-07 18:27:37 -0500   79)  
.free_function = nfs_null_free,

There's xdr_decode_func() and xdr_encode_func().

3d060451 (Jim Lieb                     2013-03-14 14:36:30 -0700   80)  
.xdr_decode_func = (xdrproc_t) xdr_void,
3d060451 (Jim Lieb                     2013-03-14 14:36:30 -0700   81)  
.xdr_encode_func = (xdrproc_t) xdr_void,

Should there be a xdr_free_func(), too?  Or instead?

Yes, there's a method to my madness.  I'm hoping to make the decode
and encode functions zero copy....


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel

Reply via email to