Re: [PATCH 09/15] IB/srpt: Fix srpt_close_session()

2016-01-06 Thread Sagi Grimberg
On 06/01/2016 16:34, Bart Van Assche wrote: On 01/06/2016 03:21 PM, Sagi Grimberg wrote: Avoid that srpt_close_session() waits if it doesn't have to wait. Can you explain when it doesn't have to wait? is it possible that srpt_release_channel_work() was already triggered? isn't that a problem

Re: [PATCH 09/15] IB/srpt: Fix srpt_close_session()

2016-01-06 Thread Bart Van Assche
On 01/06/2016 03:21 PM, Sagi Grimberg wrote: Avoid that srpt_close_session() waits if it doesn't have to wait. Can you explain when it doesn't have to wait? is it possible that srpt_release_channel_work() was already triggered? isn't that a problem? Hello Sagi, The target core can decide to

Re: [PATCH 09/15] IB/srpt: Fix srpt_close_session()

2016-01-06 Thread Sagi Grimberg
Avoid that srpt_close_session() waits if it doesn't have to wait. Can you explain when it doesn't have to wait? is it possible that srpt_release_channel_work() was already triggered? isn't that a problem? -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a

Re: [PATCH 09/15] IB/srpt: Fix srpt_close_session()

2016-01-05 Thread Christoph Hellwig
On Tue, Jan 05, 2016 at 03:24:49PM +0100, Bart Van Assche wrote: > Avoid that srpt_close_session() waits if it doesn't have to wait. > Additionally, increase the time during which srpt_close_session() > waits until closing a session has finished. This makes it easier > to detect session shutdown bu

[PATCH 09/15] IB/srpt: Fix srpt_close_session()

2016-01-05 Thread Bart Van Assche
Avoid that srpt_close_session() waits if it doesn't have to wait. Additionally, increase the time during which srpt_close_session() waits until closing a session has finished. This makes it easier to detect session shutdown bugs. Signed-off-by: Bart Van Assche Cc: Christoph Hellwig --- drivers/