[openib-general] Re: [PATCH] [kdapl CM] Add more debug on connection destruction

[James Lentini]

Committed in revision 2518.


On Tue, 31 May 2005, Hal Rosenstock wrote:

halr> [kdapl CM] Add more debug on connection destruction
halr> Also, make naming of retry defines consistent
halr> 
halr> Signed-off-by: Hal Rosenstock <halr at voltaire.com>
halr> 
halr> Index: dapl_openib_cm.c
halr> ===================================================================
halr> --- dapl_openib_cm.c	(revision 2507)
halr> +++ dapl_openib_cm.c	(working copy)
halr> @@ -42,7 +42,7 @@
halr>  #define DAPL_IB_RNR_RETRY_COUNT     6
halr>  #define DAPL_IB_CM_RESPONSE_TIMEOUT 20      /* 4 sec */
halr>  #define DAPL_IB_MAX_CM_RETRIES      4
halr> -#define DAPL_IB_MAX_AT_RETRY        3
halr> +#define DAPL_IB_MAX_AT_RETRIES      3
halr>  
halr>  /* Should these be queried ? */
halr>  #define DAPL_IB_TARGET_MAX      4 /* responder resources (max_qp_ous_rd_atom) */
halr> @@ -65,6 +65,9 @@
halr>  	spin_unlock_irqrestore(&conn->lock, flags);
halr>  
halr>  	if (!in_callback) {
halr> +		dapl_dbg_log(DAPL_DBG_TYPE_CM,
halr> +			     "  >>> dapl_destroy_cm_id: conn %p CM ID %p\n",
halr> +			     conn, conn->cm_id);
halr>  		ib_destroy_cm_id(conn->cm_id);
halr>  		if (conn->ep)
halr>  			conn->ep->cm_handle = NULL;
halr> @@ -297,7 +300,7 @@
halr>  	if (rec_num <= 0) {
halr>  		printk(KERN_ERR "dapl_path_comp_handler: path resolution "
halr>  		       "failed %d retry %d!!!\n", rec_num, conn->retries + 1);
halr> -		if (++conn->retries > DAPL_IB_MAX_AT_RETRY) {
halr> +		if (++conn->retries > DAPL_IB_MAX_AT_RETRIES) {
halr>  			printk(KERN_ERR "dapl_path_comp_handler: ep_ptr 0x%p\n",
halr>  			       conn->ep);
halr>  			event = DAT_CONNECTION_EVENT_UNREACHABLE;
halr> @@ -346,7 +349,7 @@
halr>  	if (rec_num <= 0) {
halr>  		printk(KERN_ERR "dapl_rt_comp_handler: rec num %d retry %d\n",
halr>  		       rec_num, conn->retries + 1);
halr> -		if (++conn->retries > DAPL_IB_MAX_AT_RETRY) {
halr> +		if (++conn->retries > DAPL_IB_MAX_AT_RETRIES) {
halr>  			event = DAT_CONNECTION_EVENT_UNREACHABLE;
halr>  			goto error;
halr>  		}
halr> @@ -580,6 +583,9 @@
halr>  	struct dapl_ia *ia_ptr;
halr>  	int      ib_status;
halr>  
halr> +	dapl_dbg_log(DAPL_DBG_TYPE_CM,
halr> +		     "  >>> dapl_ib_reinit_ep: EP %p\n", ep_ptr);
halr> +
halr>  	ia_ptr = ep_ptr->header.owner_ia;
halr>  
halr>  	/*
halr> @@ -671,6 +677,10 @@
halr>   */
halr>  u32 dapl_ib_remove_conn_listener(struct dapl_ia *ia_ptr, struct dapl_sp *sp_ptr)
halr>  {
halr> +        dapl_dbg_log(DAPL_DBG_TYPE_CM,
halr> +		     "  >>> dapl_ib_remove_conn_listener: SP %p conn %p\n",
halr> +		     sp_ptr, sp_ptr->cm_srvc_handle);
halr> +
halr>  	/*
halr>  	 * This will hang if called from CM thread context...
halr>  	 * Move back to using WQ...
halr> 
halr> 
halr>