Re: [PATCH v1 13/16] xprtrdma: Acquire MRs in rpcrdma_register_external()

2015-03-16 Thread Chuck Lever
On Mar 16, 2015, at 5:44 AM, Sagi Grimberg wrote: > On 3/13/2015 11:28 PM, Chuck Lever wrote: >> Acquiring 64 MRs in rpcrdma_buffer_get() while holding the buffer >> pool lock is expensive, and unnecessary because most modern adapters >> can transfer 100s of KBs of payload using just a single MR

Re: [PATCH v1 13/16] xprtrdma: Acquire MRs in rpcrdma_register_external()

2015-03-16 Thread Sagi Grimberg
On 3/13/2015 11:28 PM, Chuck Lever wrote: Acquiring 64 MRs in rpcrdma_buffer_get() while holding the buffer pool lock is expensive, and unnecessary because most modern adapters can transfer 100s of KBs of payload using just a single MR. Instead, acquire MRs one-at-a-time as chunks are registered

[PATCH v1 13/16] xprtrdma: Acquire MRs in rpcrdma_register_external()

2015-03-13 Thread Chuck Lever
Acquiring 64 MRs in rpcrdma_buffer_get() while holding the buffer pool lock is expensive, and unnecessary because most modern adapters can transfer 100s of KBs of payload using just a single MR. Instead, acquire MRs one-at-a-time as chunks are registered, and return them to rb_mws immediately duri