Re: [PATCH v2 11/12] IB/srp: Eliminate free_reqs list

2014-10-21 Thread Christoph Hellwig
On Mon, Oct 20, 2014 at 01:47:53PM +0200, Bart Van Assche wrote: > The only reason why patches 10/12 and 11/12 are separate patches > is to reduce the size of individual patches and hence to make it > easier to review these patches. If everyone agrees I'm fine with > folding patch 11/12 into patch

Re: [PATCH v2 11/12] IB/srp: Eliminate free_reqs list

2014-10-20 Thread Bart Van Assche
On 10/17/14 12:59, Christoph Hellwig wrote: On Tue, Oct 07, 2014 at 03:06:54PM +0200, Bart Van Assche wrote: The free_reqs list is no longer needed now that we are using tags assigned by the block layer. Hence remove it. Is there any good reason not to fold this into the previous patch? The

Re: [PATCH v2 11/12] IB/srp: Eliminate free_reqs list

2014-10-19 Thread Sagi Grimberg
On 10/17/2014 1:59 PM, Christoph Hellwig wrote: On Tue, Oct 07, 2014 at 03:06:54PM +0200, Bart Van Assche wrote: The free_reqs list is no longer needed now that we are using tags assigned by the block layer. Hence remove it. Is there any good reason not to fold this into the previous patch?

Re: [PATCH v2 11/12] IB/srp: Eliminate free_reqs list

2014-10-17 Thread Christoph Hellwig
On Tue, Oct 07, 2014 at 03:06:54PM +0200, Bart Van Assche wrote: > The free_reqs list is no longer needed now that we are using > tags assigned by the block layer. Hence remove it. Is there any good reason not to fold this into the previous patch? -- To unsubscribe from this list: send the line "

[PATCH v2 11/12] IB/srp: Eliminate free_reqs list

2014-10-07 Thread Bart Van Assche
The free_reqs list is no longer needed now that we are using tags assigned by the block layer. Hence remove it. Signed-off-by: Bart Van Assche Cc: Sagi Grimberg Cc: Sebastian Parschauer --- drivers/infiniband/ulp/srp/ib_srp.c | 24 +--- drivers/infiniband/ulp/srp/ib_srp.h |