Re: [PATCH v3 09/11] SUNRPC: Introduce xprt_commit_rqst()

2015-12-16 Thread Anna Schumaker
Hi Chuck, Sorry for the last minute comment. On 12/14/2015 04:19 PM, Chuck Lever wrote: > I'm about to add code in the RPC/RDMA reply handler between the > xprt_lookup_rqst() and xprt_complete_rqst() call site that needs > to execute outside of spinlock critical sections. > > Add a hook to

Re: [PATCH v3 09/11] SUNRPC: Introduce xprt_commit_rqst()

2015-12-16 Thread Chuck Lever
Hi Anna- > On Dec 16, 2015, at 8:48 AM, Anna Schumaker wrote: > > Hi Chuck, > > Sorry for the last minute comment. > > On 12/14/2015 04:19 PM, Chuck Lever wrote: >> I'm about to add code in the RPC/RDMA reply handler between the >> xprt_lookup_rqst() and

[PATCH v3 09/11] SUNRPC: Introduce xprt_commit_rqst()

2015-12-14 Thread Chuck Lever
I'm about to add code in the RPC/RDMA reply handler between the xprt_lookup_rqst() and xprt_complete_rqst() call site that needs to execute outside of spinlock critical sections. Add a hook to remove an rpc_rqst from the pending list once the transport knows its going to invoke