Re: [PATCH] SUNRPC: Allow one callback request to be received from two sk_buff

2014-01-22 Thread shaobingqing
2014/1/21 Trond Myklebust : > > On Jan 21, 2014, at 3:08, shaobingqing wrote: > >> 2014/1/21 Trond Myklebust : >>> On Mon, 2014-01-20 at 14:59 +0800, shaobingqing wrote: In current code, there only one struct rpc_rqst is prealloced. If one callback request is received from two sk_buff,

Re: [PATCH] SUNRPC: Allow one callback request to be received from two sk_buff

2014-01-22 Thread shaobingqing
2014/1/23 J. Bruce Fields : > On Tue, Jan 21, 2014 at 08:35:36AM -0700, Trond Myklebust wrote: >> >> On Jan 21, 2014, at 3:08, shaobingqing wrote: >> >> > 2014/1/21 Trond Myklebust : >> >> On Mon, 2014-01-20 at 14:59 +0800, shaobingqing wrote: >> >>> In current code, there only one struct

Re: [PATCH] SUNRPC: Allow one callback request to be received from two sk_buff

2014-01-22 Thread J. Bruce Fields
On Tue, Jan 21, 2014 at 08:35:36AM -0700, Trond Myklebust wrote: > > On Jan 21, 2014, at 3:08, shaobingqing wrote: > > > 2014/1/21 Trond Myklebust : > >> On Mon, 2014-01-20 at 14:59 +0800, shaobingqing wrote: > >>> In current code, there only one struct rpc_rqst is prealloced. If one > >>>

Re: [PATCH] SUNRPC: Allow one callback request to be received from two sk_buff

2014-01-22 Thread J. Bruce Fields
On Tue, Jan 21, 2014 at 08:35:36AM -0700, Trond Myklebust wrote: On Jan 21, 2014, at 3:08, shaobingqing shaobingq...@bwstor.com.cn wrote: 2014/1/21 Trond Myklebust trond.mykleb...@primarydata.com: On Mon, 2014-01-20 at 14:59 +0800, shaobingqing wrote: In current code, there only one

Re: [PATCH] SUNRPC: Allow one callback request to be received from two sk_buff

2014-01-22 Thread shaobingqing
2014/1/23 J. Bruce Fields bfie...@fieldses.org: On Tue, Jan 21, 2014 at 08:35:36AM -0700, Trond Myklebust wrote: On Jan 21, 2014, at 3:08, shaobingqing shaobingq...@bwstor.com.cn wrote: 2014/1/21 Trond Myklebust trond.mykleb...@primarydata.com: On Mon, 2014-01-20 at 14:59 +0800,

Re: [PATCH] SUNRPC: Allow one callback request to be received from two sk_buff

2014-01-22 Thread shaobingqing
2014/1/21 Trond Myklebust trond.mykleb...@primarydata.com: On Jan 21, 2014, at 3:08, shaobingqing shaobingq...@bwstor.com.cn wrote: 2014/1/21 Trond Myklebust trond.mykleb...@primarydata.com: On Mon, 2014-01-20 at 14:59 +0800, shaobingqing wrote: In current code, there only one struct

Re: [PATCH] SUNRPC: Allow one callback request to be received from two sk_buff

2014-01-21 Thread Trond Myklebust
On Jan 21, 2014, at 3:08, shaobingqing wrote: > 2014/1/21 Trond Myklebust : >> On Mon, 2014-01-20 at 14:59 +0800, shaobingqing wrote: >>> In current code, there only one struct rpc_rqst is prealloced. If one >>> callback request is received from two sk_buff, the xprt_alloc_bc_request >>> would

Re: [PATCH] SUNRPC: Allow one callback request to be received from two sk_buff

2014-01-21 Thread shaobingqing
2014/1/21 Trond Myklebust : > On Mon, 2014-01-20 at 14:59 +0800, shaobingqing wrote: >> In current code, there only one struct rpc_rqst is prealloced. If one >> callback request is received from two sk_buff, the xprt_alloc_bc_request >> would be execute two times with the same transport->xid. The

Re: [PATCH] SUNRPC: Allow one callback request to be received from two sk_buff

2014-01-21 Thread shaobingqing
2014/1/21 Trond Myklebust trond.mykleb...@primarydata.com: On Mon, 2014-01-20 at 14:59 +0800, shaobingqing wrote: In current code, there only one struct rpc_rqst is prealloced. If one callback request is received from two sk_buff, the xprt_alloc_bc_request would be execute two times with the

Re: [PATCH] SUNRPC: Allow one callback request to be received from two sk_buff

2014-01-21 Thread Trond Myklebust
On Jan 21, 2014, at 3:08, shaobingqing shaobingq...@bwstor.com.cn wrote: 2014/1/21 Trond Myklebust trond.mykleb...@primarydata.com: On Mon, 2014-01-20 at 14:59 +0800, shaobingqing wrote: In current code, there only one struct rpc_rqst is prealloced. If one callback request is received from

Re: [PATCH] SUNRPC: Allow one callback request to be received from two sk_buff

2014-01-20 Thread Trond Myklebust
On Mon, 2014-01-20 at 14:59 +0800, shaobingqing wrote: > In current code, there only one struct rpc_rqst is prealloced. If one > callback request is received from two sk_buff, the xprt_alloc_bc_request > would be execute two times with the same transport->xid. The first time >

Re: [PATCH] SUNRPC: Allow one callback request to be received from two sk_buff

2014-01-20 Thread Sergei Shtylyov
Hello. On 20-01-2014 10:59, shaobingqing wrote: In current code, there only one struct rpc_rqst is prealloced. If one callback request is received from two sk_buff, the xprt_alloc_bc_request would be execute two times with the same transport->xid. The first time xprt_alloc_bc_request will

Re: [PATCH] SUNRPC: Allow one callback request to be received from two sk_buff

2014-01-20 Thread Sergei Shtylyov
Hello. On 20-01-2014 10:59, shaobingqing wrote: In current code, there only one struct rpc_rqst is prealloced. If one callback request is received from two sk_buff, the xprt_alloc_bc_request would be execute two times with the same transport-xid. The first time xprt_alloc_bc_request will alloc

Re: [PATCH] SUNRPC: Allow one callback request to be received from two sk_buff

2014-01-20 Thread Trond Myklebust
On Mon, 2014-01-20 at 14:59 +0800, shaobingqing wrote: In current code, there only one struct rpc_rqst is prealloced. If one callback request is received from two sk_buff, the xprt_alloc_bc_request would be execute two times with the same transport-xid. The first time xprt_alloc_bc_request

[PATCH] SUNRPC: Allow one callback request to be received from two sk_buff

2014-01-19 Thread shaobingqing
In current code, there only one struct rpc_rqst is prealloced. If one callback request is received from two sk_buff, the xprt_alloc_bc_request would be execute two times with the same transport->xid. The first time xprt_alloc_bc_request will alloc one struct rpc_rqst and the TCP_RCV_COPY_DATA bit

[PATCH] SUNRPC: Allow one callback request to be received from two sk_buff

2014-01-19 Thread shaobingqing
In current code, there only one struct rpc_rqst is prealloced. If one callback request is received from two sk_buff, the xprt_alloc_bc_request would be execute two times with the same transport-xid. The first time xprt_alloc_bc_request will alloc one struct rpc_rqst and the TCP_RCV_COPY_DATA bit