Cleaned up some debug messages.

2 files changed, 15 insertions, 34 deletions

diff --git a/src/drivers/net/ipoib.c b/src/drivers/net/ipoib.c
index f3b821c4..81011405 100644
--- a/src/drivers/net/ipoib.c
+++ b/src/drivers/net/ipoib.c
@@ -322,16 +322,6 @@ static int ipoib_get_path_record ( struct ipoib_device *ipoib,
  	static uint32_t tid = 0;
 	int rc;
 
-	int get_path_record(struct ib_gid *dgid, uint16_t *dlid_p,
-			    uint8_t *sl_p, uint8_t *rate_p);
-	uint16_t tmp_dlid;
-	uint8_t tmp_sl;
-	uint8_t tmp_rate;
-	get_path_record ( gid, &tmp_dlid, &tmp_sl, &tmp_rate );
-	DBG ( "get_path_record() gives dlid = %04x, sl = %02x, rate = %02x\n",
-	      tmp_dlid, tmp_sl, tmp_rate );
-
-
 	/* Allocate I/O buffer */
 	iobuf = alloc_iob ( sizeof ( *path_record ) );
 	if ( ! iobuf )
@@ -513,10 +503,6 @@ static void ipoib_meta_complete_recv ( struct ib_device *ibdev __unused,
 		/* Update path cache */
 		iob_put ( iobuf, completion->len );
 		iob_pull ( iobuf, sizeof ( struct ib_global_route_header ) );
-
-		DBG ( "Metadata RX:\n" );
-		DBG_HD ( iobuf->data, iob_len ( iobuf ) );
-
 		path_record = iobuf->data;
 		path = &ipoib_path_cache[ipoib_path_cache_idx];
 		memcpy ( &path->gid, &path_record->dgid,
@@ -716,8 +702,11 @@ int ipoib_probe ( struct ib_device *ibdev ) {
  */
 void ipoib_remove ( struct ib_device *ibdev ) {
 	struct net_device *netdev = ib_get_ownerdata ( ibdev );
+	struct ipoib_device *ipoib = netdev->priv;
 
 	unregister_netdev ( netdev );
+	ipoib_destroy_qset ( ipoib, &ipoib->data );
+	ipoib_destroy_qset ( ipoib, &ipoib->meta );
 	netdev_nullify ( netdev );
 	netdev_put ( netdev );
 }
diff --git a/src/drivers/net/mlx_ipoib/mt25218.c b/src/drivers/net/mlx_ipoib/mt25218.c
index 0fd832e7..5f3c3c4d 100644
--- a/src/drivers/net/mlx_ipoib/mt25218.c
+++ b/src/drivers/net/mlx_ipoib/mt25218.c
@@ -138,10 +138,10 @@ static int arbel_cmd ( struct arbel *arbel, unsigned long command,
 	unsigned int i;
 	int rc;
 
-	DBGC ( arbel, "Arbel %p command %02x in %zx%s out %zx%s\n",
-	       arbel, opcode, in_len,
-	       ( ( command & ARBEL_HCR_IN_MBOX ) ? "(mbox)" : "" ), out_len,
-	       ( ( command & ARBEL_HCR_OUT_MBOX ) ? "(mbox)" : "" ) );
+	DBGC2 ( arbel, "Arbel %p command %02x in %zx%s out %zx%s\n",
+		arbel, opcode, in_len,
+		( ( command & ARBEL_HCR_IN_MBOX ) ? "(mbox)" : "" ), out_len,
+		( ( command & ARBEL_HCR_OUT_MBOX ) ? "(mbox)" : "" ) );
 
 	/* Check that HCR is free */
 	if ( ( rc = arbel_cmd_wait ( arbel, &hcr ) ) != 0 ) {
@@ -168,14 +168,10 @@ static int arbel_cmd ( struct arbel *arbel, unsigned long command,
 		     opcode, opcode,
 		     opcode_modifier, op_mod,
 		     go, 1 );
-
-	DBG_HD ( &hcr, sizeof ( hcr ) );
+	DBGC2_HD ( arbel, &hcr, sizeof ( hcr ) );
 	if ( in_len ) {
-		size_t dump_len = in_len;
-		if ( dump_len > 256 )
-			dump_len = 256;
-		//		DBG ( "Input:\n" );
-		//		DBG_HD ( in, dump_len );
+		DBGC2 ( arbel, "Input:\n" );
+		DBGC2_HD ( arbel, in, ( ( in_len < 256 ) ? in_len : 256 ) );
 	}
 
 	/* Issue command */
@@ -207,13 +203,9 @@ static int arbel_cmd ( struct arbel *arbel, unsigned long command,
 	hcr.u.dwords[3] = readl ( arbel->config + ARBEL_HCR_REG ( 3 ) );
 	hcr.u.dwords[4] = readl ( arbel->config + ARBEL_HCR_REG ( 4 ) );
 	memcpy ( out, out_buffer, out_len );
-
 	if ( out_len ) {
-		size_t dump_len = out_len;
-		if ( dump_len > 256 )
-			dump_len = 256;
-		//		DBG ( "Output:\n" );
-		//		DBG_HD ( out, dump_len );
+		DBGC2 ( arbel, "Output:\n" );
+		DBGC2_HD ( arbel, out, ( ( out_len < 256 ) ? out_len : 256 ) );
 	}
 
 	return 0;
@@ -737,9 +729,9 @@ static void arbel_ring_doorbell ( struct arbel *arbel,
 				  union arbelprm_doorbell_register *db_reg,
 				  unsigned int offset ) {
 
-	DBG ( "arbel_ring_doorbell %08lx:%08lx to %lx\n",
-	      db_reg->dword[0], db_reg->dword[1],
-	      virt_to_phys ( arbel->uar + offset ) );
+	DBGC2 ( arbel, "Arbel %p ringing doorbell %08lx:%08lx at %lx\n",
+		arbel, db_reg->dword[0], db_reg->dword[1],
+		virt_to_phys ( arbel->uar + offset ) );
 
 	barrier();
 	writel ( db_reg->dword[0], ( arbel->uar + offset + 0 ) );