[stgt] [PATCH V1] iser: Don't release IB resources if were not	allocated
    Roi Dayan 
    roid at mellanox.com
       
    Sun Aug 25 13:20:39 CEST 2013
    
    
  
The lld exit function is being called even if the lld was not
initialized.
Added a check if IB resources were allocated before trying to
release.
Signed-off-by: Roi Dayan <roid at mellanox.com>
---
Hi Tomo,
This is instead of the previus one.
I added more information to the commit message.
This patch should be applied over Andy's patch:
[PATCH] iser: Don't wait until iser_ib_init to init list_heads
Thanks,
Roi
 usr/iscsi/iser.c |   12 +++++++-----
 1 files changed, 7 insertions(+), 5 deletions(-)
diff --git a/usr/iscsi/iser.c b/usr/iscsi/iser.c
index be2afeb..3e96cb7 100644
--- a/usr/iscsi/iser.c
+++ b/usr/iscsi/iser.c
@@ -3408,13 +3408,15 @@ static void iser_ib_release(void)
 		free(dev);
 	}
 
-	tgt_event_del(cma_listen_id->channel->fd);
+	if (cma_listen_id) {
+		tgt_event_del(cma_listen_id->channel->fd);
 
-	err = rdma_destroy_id(cma_listen_id);
-	if (err)
-		eprintf("rdma_destroy_id failed: (errno=%d %m)\n", errno);
+		err = rdma_destroy_id(cma_listen_id);
+		if (err)
+			eprintf("rdma_destroy_id failed: (errno=%d %m)\n", errno);
 
-	rdma_destroy_event_channel(rdma_evt_channel);
+		rdma_destroy_event_channel(rdma_evt_channel);
+	}
 }
 
 static int iser_send_nop = 1;
-- 
1.7.8.2
--
To unsubscribe from this list: send the line "unsubscribe stgt" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
    
    
More information about the stgt
mailing list