From: leonidk <leonidk@ad392aa1-c5ef-ae45-8dd8-e69d62a5ef86>
Date: Mon, 10 Apr 2006 09:03:24 +0000 (+0000)
Subject: [MTHCA] fixed a typo, producing a crash, and changed debug flags
X-Git-Url: https://openfabrics.org/gitweb/?a=commitdiff_plain;h=fcc19a9c1924d50f23afc527fd8693af2e3115fd;p=~shefty%2Frdma-win.git

[MTHCA] fixed a typo, producing a crash, and changed debug flags

git-svn-id: svn://openib.tc.cornell.edu/gen1@297 ad392aa1-c5ef-ae45-8dd8-e69d62a5ef86
---

diff --git a/trunk/hw/mthca/kernel/hca_verbs.c b/trunk/hw/mthca/kernel/hca_verbs.c
index 49b87a2a..30e096c7 100644
--- a/trunk/hw/mthca/kernel/hca_verbs.c
+++ b/trunk/hw/mthca/kernel/hca_verbs.c
@@ -561,7 +561,7 @@ mlnx_create_av (
 	struct ib_ah_attr ah_attr;
 	struct ib_ucontext *p_context = NULL;
 
-	HCA_ENTER(HCA_DBG_QP);
+	HCA_ENTER(HCA_DBG_AV);
 
 	if( p_umv_buf && p_umv_buf->command ) {
 		// sanity checks 
@@ -581,10 +581,10 @@ mlnx_create_av (
 	mlnx_conv_ibal_av( ib_dev, p_addr_vector,  &ah_attr );
 
 	ib_av_p = ibv_create_ah(ib_pd_p, &ah_attr, p_context, p_umv_buf);
-	if (IS_ERR(ib_pd_p)) {
-		err = PTR_ERR(ib_pd_p);
-		HCA_PRINT(TRACE_LEVEL_ERROR   ,HCA_DBG_QP,
-			("ibv_alloc_pd failed (%d)\n", err));
+	if (IS_ERR(ib_av_p)) {
+		err = PTR_ERR(ib_av_p);
+		HCA_PRINT(TRACE_LEVEL_ERROR ,HCA_DBG_AV,
+			("ibv_create_ah failed (%d)\n", err));
 		status = errno_to_iberr(err);
 		goto err_alloc_av;
 	}
@@ -603,7 +603,7 @@ mlnx_create_av (
 	
 err_alloc_av:	
 err_inval_params:
-	HCA_PRINT_EXIT(TRACE_LEVEL_ERROR ,HCA_DBG_QP,
+	HCA_PRINT_EXIT(TRACE_LEVEL_ERROR ,HCA_DBG_AV,
 		("completes with ERROR status %s\n", IB_GET_ERR_STR(status)));
 	return status;
 }
@@ -620,11 +620,11 @@ mlnx_query_av (
 	struct ib_ah *ib_ah_p = (struct ib_ah *)h_av;
 	PREP_IBDEV_FOR_PRINT(ib_ah_p->device)
 
-	HCA_ENTER(HCA_DBG_SHIM);
+	HCA_ENTER(HCA_DBG_AV);
 
 	// sanity checks
 	if( p_umv_buf && p_umv_buf->command ) {
-			HCA_PRINT (TRACE_LEVEL_ERROR, HCA_DBG_SHIM,
+			HCA_PRINT (TRACE_LEVEL_ERROR, HCA_DBG_AV,
 				("User mode is not supported yet\n"));
 			status = IB_UNSUPPORTED;
 			goto err_user_unsupported;
@@ -635,7 +635,7 @@ mlnx_query_av (
 	//TODO: not implemented in low-level driver
 	err = ibv_query_ah(ib_ah_p, &ah_attr)
 	if (err) {
-		HCA_PRINT(TRACE_LEVEL_ERROR, HCA_DBG_SHIM,
+		HCA_PRINT(TRACE_LEVEL_ERROR, HCA_DBG_AV,
 			("ibv_query_ah failed (%d)\n", err));
 		status = errno_to_iberr(err);
 		goto err_query_ah;
@@ -646,7 +646,7 @@ mlnx_query_av (
 
 	err = mlnx_conv_mthca_av( ib_ah_p, p_addr_vector );
 	if (err) {
-		HCA_PRINT (TRACE_LEVEL_ERROR, HCA_DBG_SHIM,
+		HCA_PRINT (TRACE_LEVEL_ERROR, HCA_DBG_AV,
 			("mlnx_conv_mthca_av failed (%d)\n", err));
 		status = errno_to_iberr(err);
 		goto err_conv_mthca_av;
@@ -658,7 +658,7 @@ mlnx_query_av (
 	
 err_conv_mthca_av:
 err_user_unsupported:
-	HCA_PRINT_EXIT(TRACE_LEVEL_ERROR  , HCA_DBG_SHIM,
+	HCA_PRINT_EXIT(TRACE_LEVEL_ERROR  , HCA_DBG_AV,
 		("completes with ERROR status %s\n", IB_GET_ERR_STR(status)));
 	return status;
 }
@@ -674,11 +674,11 @@ mlnx_modify_av (
 	struct ib_ah *ib_ah_p = (struct ib_ah *)h_av;
 	struct ib_device *ib_dev = ib_ah_p->pd->device;
 
-	HCA_ENTER(HCA_DBG_SHIM);
+	HCA_ENTER(HCA_DBG_AV);
 
 	// sanity checks
 	if( p_umv_buf && p_umv_buf->command ) {
-			HCA_PRINT(TRACE_LEVEL_ERROR   ,HCA_DBG_SHIM,
+			HCA_PRINT(TRACE_LEVEL_ERROR   ,HCA_DBG_AV,
 				("User mode is not supported yet\n"));
 			status = IB_UNSUPPORTED;
 			goto err_user_unsupported;
@@ -693,7 +693,7 @@ mlnx_modify_av (
 	//TODO: not implemented in low-level driver
 	err = ibv_modify_ah(ib_ah_p, &ah_attr)
 	if (err) {
-		HCA_PRINT (TRACE_LEVEL_ERROR, HCA_DBG_SHIM,
+		HCA_PRINT (TRACE_LEVEL_ERROR, HCA_DBG_AV,
 			("ibv_query_ah failed (%d)\n", err));
 		status = errno_to_iberr(err);
 		goto err_query_ah;
@@ -704,7 +704,7 @@ mlnx_modify_av (
 #endif
 
 err_user_unsupported:
-	HCA_PRINT_EXIT(TRACE_LEVEL_ERROR  , HCA_DBG_SHIM,
+	HCA_PRINT_EXIT(TRACE_LEVEL_ERROR  , HCA_DBG_AV,
 		("completes with ERROR status %s\n", IB_GET_ERR_STR(status)));
 	return status;
 }
@@ -718,19 +718,19 @@ mlnx_destroy_av (
 	struct ib_ah *ib_ah_p = (struct ib_ah *)h_av;
 	PREP_IBDEV_FOR_PRINT(ib_ah_p->device)
 
-	HCA_ENTER(HCA_DBG_SHIM);
+	HCA_ENTER(HCA_DBG_AV);
 
 	// destroy AV
 	err = ibv_destroy_ah( ib_ah_p );
 	if (err) {
-		HCA_PRINT (TRACE_LEVEL_ERROR ,HCA_DBG_SHIM,
+		HCA_PRINT (TRACE_LEVEL_ERROR ,HCA_DBG_AV,
 			("ibv_destroy_ah failed (%d)\n", err));
 		status = errno_to_iberr(err);
 		goto err_destroy_ah;
 	}
 
 err_destroy_ah:
-	HCA_PRINT_EXIT(TRACE_LEVEL_ERROR  , HCA_DBG_SHIM,
+	HCA_PRINT_EXIT(TRACE_LEVEL_ERROR  , HCA_DBG_AV,
 		("completes with ERROR status %s\n", IB_GET_ERR_STR(status)));
 	return status;
 }