On May 11, 2015, at 3:16 AM, Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote: > On Thu, May 07, 2015 at 07:20:28PM -0400, Chuck Lever wrote: >> Clean up: Merge bc_svc.c into backchannel_rqst.c, and adopt a more >> consistent naming scheme for bc_send(). > > Seems like a bit of an odd choise as none of the callchain bc_send > is in really touches backchannel_rqst.c. > > How about just killing bc_send() and open coding it in bc_svc_process? OK, that sounds reasonable. > And while we're at it - bc_svc_process seems totally generic RPC > code that might want to be moved to net/sunrpc and converted to > a workqueue. That is underway. -- Chuck Lever chuck[dot]lever[at]oracle[dot]com -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html