On Mon, 2014-05-19 at 14:23 -0700, Nicholas A. Bellinger wrote: > On Mon, 2014-05-19 at 17:44 +0300, Sagi Grimberg wrote: > > This patchset addresses several issues in connection > > (graceful/ungraceful) teardown. > > > > Patch #1 addresses a possible np_thread hang when isert_accept_np > > retook isert_np semaphore and never left it. > > > > Patch #2 addresses connection teardown hang when isert_wait_conn > > had waited for iser disconnected handler that never arrived. > > > > Patch #3 addresses other RDMA_CM events that were not properly > > handled. > > > > Patch #4 makes sure that ib_isert unload will properly cleanup > > before finishing. > > > > This patchset survived several nights of: > > - Unstable link (cable pull of both initiator and target ports). > > - stack unload during live IO and unstable link. > > > > Slava will continue iSER target qualification efforts (Thanks Slava!). > > > > Sagi Grimberg (4): > > Target/iser: Bail from accept_np if np_thread is trying to close > > Target/iser: Fix hangs in connection teardown > > Target/iser: Improve cm events handling > > Target/iser: Wait for proper cleanup before unloading > > > > drivers/infiniband/ulp/isert/ib_isert.c | 60 +++++++++++++++---------------- > > drivers/infiniband/ulp/isert/ib_isert.h | 2 +- > > 2 files changed, 30 insertions(+), 32 deletions(-) > > > > Minus the minor bit in patch #3 (dropping that unnecessary check now), > the series looks fine. > > Applied to for-next. > Btw, adding a CC for >= v3.10.y code as well, as these four look like stable material. --nab -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
