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