Re: [PATCH 13/15] IB/srpt: Detect session shutdown reliably

2016-01-06 Thread Bart Van Assche
On 01/06/2016 06:21 AM, Christoph Hellwig wrote: On Tue, Jan 05, 2016 at 03:26:49PM +0100, Bart Van Assche wrote: The Last WQE Reached event is only generated after one or more work requests have been queued on the QP associated with a session. Since session shutdown can start before any work

Re: [PATCH 13/15] IB/srpt: Detect session shutdown reliably

2016-01-06 Thread Bart Van Assche
On 01/06/2016 03:39 PM, Sagi Grimberg wrote: On 05/01/2016 16:26, Bart Van Assche wrote: The Last WQE Reached event is only generated after one or more work requests have been queued on the QP associated with a session. Since session shutdown can start before any work requests have been queued,

Re: [PATCH 13/15] IB/srpt: Detect session shutdown reliably

2016-01-06 Thread Sagi Grimberg
On 05/01/2016 16:26, Bart Van Assche wrote: The Last WQE Reached event is only generated after one or more work requests have been queued on the QP associated with a session. Since session shutdown can start before any work requests have been queued, use a zero-length RDMA write to wait until

Re: [PATCH 13/15] IB/srpt: Detect session shutdown reliably

2016-01-06 Thread Christoph Hellwig
On Wed, Jan 06, 2016 at 03:46:34PM +0100, Bart Van Assche wrote: > I will make the patch description more detailed. Sorry if some of this code > is hard to follow but that's because of the high level of concurrency in > the SRP target driver. Some time ago I documented how session management in

[PATCH 13/15] IB/srpt: Detect session shutdown reliably

2016-01-05 Thread Bart Van Assche
The Last WQE Reached event is only generated after one or more work requests have been queued on the QP associated with a session. Since session shutdown can start before any work requests have been queued, use a zero-length RDMA write to wait until a QP has been drained. Signed-off-by: Bart Van

Re: [PATCH 13/15] IB/srpt: Detect session shutdown reliably

2016-01-05 Thread Christoph Hellwig
On Tue, Jan 05, 2016 at 03:26:49PM +0100, Bart Van Assche wrote: > The Last WQE Reached event is only generated after one or more work > requests have been queued on the QP associated with a session. Since > session shutdown can start before any work requests have been queued, > use a zero-length