Browse Source

qcacld-3.0: Modify TDLS log for link setup/Teardown

qcacld-2.0 to qcacld-3.0 propagation

As a part of logging improvement, modify redundant
logs, change certain log levels in TDLS link setup/
teardown path to make logs more useful.

Change-Id: Ibb0ee1edb5bbb1aed57bccbc15354031b0362198
CRs-Fixed: 861452
Agrawal Ashish 9 years ago
parent
commit
a5c2bdc4a7
2 changed files with 13 additions and 2 deletions
  1. 2 1
      core/hdd/src/wlan_hdd_tdls.c
  2. 11 1
      core/sme/src/csr/csr_tdls_process.c

+ 2 - 1
core/hdd/src/wlan_hdd_tdls.c

@@ -2962,6 +2962,7 @@ void wlan_hdd_tdls_indicate_teardown(hdd_adapter_t *pAdapter,
 	wlan_hdd_tdls_set_peer_link_status(curr_peer,
 	wlan_hdd_tdls_set_peer_link_status(curr_peer,
 					   eTDLS_LINK_TEARING,
 					   eTDLS_LINK_TEARING,
 					   eTDLS_LINK_UNSPECIFIED);
 					   eTDLS_LINK_UNSPECIFIED);
+	hdd_info("Teardown reason %d", reason);
 	cfg80211_tdls_oper_request(pAdapter->dev,
 	cfg80211_tdls_oper_request(pAdapter->dev,
 				   curr_peer->peerMac,
 				   curr_peer->peerMac,
 				   NL80211_TDLS_TEARDOWN, reason, GFP_KERNEL);
 				   NL80211_TDLS_TEARDOWN, reason, GFP_KERNEL);
@@ -4614,7 +4615,7 @@ static int __wlan_hdd_cfg80211_tdls_oper(struct wiphy *wiphy,
 		return -ENOTSUPP;
 		return -ENOTSUPP;
 	default:
 	default:
 		QDF_TRACE(QDF_MODULE_ID_HDD, QDF_TRACE_LEVEL_ERROR,
 		QDF_TRACE(QDF_MODULE_ID_HDD, QDF_TRACE_LEVEL_ERROR,
-			  "%s: unsupported event", __func__);
+			  "%s: unsupported event %d", __func__, oper);
 		return -ENOTSUPP;
 		return -ENOTSUPP;
 	}
 	}
 	EXIT();
 	EXIT();

+ 11 - 1
core/sme/src/csr/csr_tdls_process.c

@@ -129,6 +129,8 @@ QDF_STATUS csr_tdls_send_mgmt_req(tHalHandle hHal, uint8_t sessionId,
 	tdlsSendMgmtCmd->u.tdlsCmd.size = sizeof(tTdlsSendMgmtCmdInfo);
 	tdlsSendMgmtCmd->u.tdlsCmd.size = sizeof(tTdlsSendMgmtCmdInfo);
 	sme_push_command(pMac, tdlsSendMgmtCmd, false);
 	sme_push_command(pMac, tdlsSendMgmtCmd, false);
 	status = QDF_STATUS_SUCCESS;
 	status = QDF_STATUS_SUCCESS;
+	sms_log(pMac, LOG1,
+		FL("Successfully posted eSmeCommandTdlsSendMgmt to SME"));
 	return status;
 	return status;
 }
 }
 
 
@@ -201,6 +203,8 @@ QDF_STATUS csr_tdls_change_peer_sta(tHalHandle hHal, uint8_t sessionId,
 				sizeof(tTdlsAddStaCmdInfo);
 				sizeof(tTdlsAddStaCmdInfo);
 			sme_push_command(pMac, tdlsAddStaCmd, false);
 			sme_push_command(pMac, tdlsAddStaCmd, false);
 			status = QDF_STATUS_SUCCESS;
 			status = QDF_STATUS_SUCCESS;
+			sms_log(pMac, LOG1,
+			FL("Successfully posted eSmeCommandTdlsAddPeer to SME to modify peer "));
 		}
 		}
 	}
 	}
 
 
@@ -271,6 +275,8 @@ QDF_STATUS csr_tdls_send_link_establish_params(tHalHandle hHal,
 				sizeof(tTdlsLinkEstablishCmdInfo);
 				sizeof(tTdlsLinkEstablishCmdInfo);
 			sme_push_command(pMac, tdlsLinkEstablishCmd, false);
 			sme_push_command(pMac, tdlsLinkEstablishCmd, false);
 			status = QDF_STATUS_SUCCESS;
 			status = QDF_STATUS_SUCCESS;
+			sms_log(pMac, LOG1,
+			FL("Successfully posted eSmeCommandTdlsLinkEstablish to SME"));
 		}
 		}
 	}
 	}
 
 
@@ -308,6 +314,8 @@ QDF_STATUS csr_tdls_add_peer_sta(tHalHandle hHal, uint8_t sessionId,
 				sizeof(tTdlsAddStaCmdInfo);
 				sizeof(tTdlsAddStaCmdInfo);
 			sme_push_command(pMac, tdlsAddStaCmd, false);
 			sme_push_command(pMac, tdlsAddStaCmd, false);
 			status = QDF_STATUS_SUCCESS;
 			status = QDF_STATUS_SUCCESS;
+			sms_log(pMac, LOG1,
+			FL("Successfully posted eSmeCommandTdlsAddPeer to SME"));
 		}
 		}
 	}
 	}
 
 
@@ -344,6 +352,8 @@ QDF_STATUS csr_tdls_del_peer_sta(tHalHandle hHal, uint8_t sessionId,
 				sizeof(tTdlsDelStaCmdInfo);
 				sizeof(tTdlsDelStaCmdInfo);
 			sme_push_command(pMac, tdlsDelStaCmd, false);
 			sme_push_command(pMac, tdlsDelStaCmd, false);
 			status = QDF_STATUS_SUCCESS;
 			status = QDF_STATUS_SUCCESS;
+			sms_log(pMac, LOG1,
+			FL("Successfully posted eSmeCommandTdlsDelPeer to SME"));
 		}
 		}
 	}
 	}
 
 
@@ -423,7 +433,7 @@ QDF_STATUS csr_tdls_process_send_mgmt(tpAniSirGlobal pMac, tSmeCmd *cmd)
 
 
 	}
 	}
 	/* Send the request to PE. */
 	/* Send the request to PE. */
-	sms_log(pMac, LOG1, "sending TDLS Mgmt Frame req to PE ");
+	sms_log(pMac, LOG1, FL("sending TDLS Mgmt Frame req to PE "));
 	status = tdls_send_message(pMac, eWNI_SME_TDLS_SEND_MGMT_REQ,
 	status = tdls_send_message(pMac, eWNI_SME_TDLS_SEND_MGMT_REQ,
 				   (void *)tdlsSendMgmtReq,
 				   (void *)tdlsSendMgmtReq,
 				   sizeof(tSirTdlsSendMgmtReq) +
 				   sizeof(tSirTdlsSendMgmtReq) +