wlan: Modify TDLS log for link setup/Teardown
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
diff --git a/CORE/HDD/src/wlan_hdd_cfg80211.c b/CORE/HDD/src/wlan_hdd_cfg80211.c
index 40f4435..a0c899e 100644
--- a/CORE/HDD/src/wlan_hdd_cfg80211.c
+++ b/CORE/HDD/src/wlan_hdd_cfg80211.c
@@ -15162,7 +15162,7 @@
if ( NULL == pTdlsPeer ) {
hddLog(VOS_TRACE_LEVEL_INFO, "%s: " MAC_ADDRESS_STR
- " peer not exsting",
+ " peer not existing",
__func__, MAC_ADDR_ARRAY(peer));
return -EINVAL;
}
@@ -15249,6 +15249,9 @@
hddTdlsPeer_t *connPeer = NULL;
tANI_U8 suppChannelLen = 0;
+ VOS_TRACE(VOS_MODULE_ID_HDD, VOS_TRACE_LEVEL_INFO,
+ " %s : NL80211_TDLS_ENABLE_LINK for " MAC_ADDRESS_STR,
+ __func__, MAC_ADDR_ARRAY(peer));
pTdlsPeer = wlan_hdd_tdls_find_peer(pAdapter, peer, TRUE);
memset(&staDesc, 0, sizeof(staDesc));
if ( NULL == pTdlsPeer ) {
@@ -15298,7 +15301,8 @@
TDLS_OFF_CHANNEL_BW_OFFSET,
TDLS_CHANNEL_SWITCH_DISABLE);
if (ret != VOS_STATUS_SUCCESS) {
- hddLog(VOS_TRACE_LEVEL_ERROR, FL("Failed to send TDLS switch channel request"));
+ hddLog(VOS_TRACE_LEVEL_ERROR,
+ FL("Failed to send TDLS switch channel request"));
}
}
else
@@ -15343,8 +15347,8 @@
if (ret <= 0)
{
VOS_TRACE(VOS_MODULE_ID_HDD, VOS_TRACE_LEVEL_ERROR,
- "%s: Link Establish Request Faled Status %ld",
- __func__, ret);
+ FL("Link Establish Request Failed Status %ld"),
+ ret);
return -EINVAL;
}
}
@@ -15489,8 +15493,13 @@
tANI_U16 numCurrTdlsPeers = 0;
hddTdlsPeer_t *connPeer = NULL;
+ VOS_TRACE(VOS_MODULE_ID_HDD, VOS_TRACE_LEVEL_INFO,
+ " %s : NL80211_TDLS_DISABLE_LINK for " MAC_ADDRESS_STR,
+ __func__, MAC_ADDR_ARRAY(peer));
+
pTdlsPeer = wlan_hdd_tdls_find_peer(pAdapter, peer, TRUE);
+
if ( NULL == pTdlsPeer ) {
hddLog(VOS_TRACE_LEVEL_ERROR, "%s: " MAC_ADDRESS_STR
" (oper %d) not exsting. ignored",
@@ -15590,7 +15599,7 @@
if (0 != status)
{
VOS_TRACE( VOS_MODULE_ID_HDD, VOS_TRACE_LEVEL_ERROR,
- "%s: Error in TDLS Teardown", __func__);
+ FL("Error in TDLS Teardown"));
return status;
}
break;
@@ -15605,7 +15614,7 @@
if (0 != status)
{
VOS_TRACE( VOS_MODULE_ID_HDD, VOS_TRACE_LEVEL_ERROR,
- "%s: Error in TDLS Setup", __func__);
+ FL("Error in TDLS Setup"));
return status;
}
break;
@@ -15613,7 +15622,7 @@
case NL80211_TDLS_DISCOVERY_REQ:
/* We don't support in-driver setup/teardown/discovery */
VOS_TRACE(VOS_MODULE_ID_HDD, VOS_TRACE_LEVEL_WARN,
- "%s: We don't support in-driver setup/teardown/discovery "
+ "%s: Driver doesn't support in-driver setup/teardown/discovery "
,__func__);
return -ENOTSUPP;
default:
diff --git a/CORE/HDD/src/wlan_hdd_tdls.c b/CORE/HDD/src/wlan_hdd_tdls.c
index f3ce0f7..cfcb7d0 100644
--- a/CORE/HDD/src/wlan_hdd_tdls.c
+++ b/CORE/HDD/src/wlan_hdd_tdls.c
@@ -2975,6 +2975,8 @@
wlan_hdd_tdls_set_peer_link_status(curr_peer,
eTDLS_LINK_TEARING,
eTDLS_LINK_UNSPECIFIED);
+ VOS_TRACE(VOS_MODULE_ID_HDD, VOS_TRACE_LEVEL_ERROR,
+ FL("Setting NL80211_TDLS_TEARDOWN, reason %d"), reason);
cfg80211_tdls_oper_request(pAdapter->dev,
curr_peer->peerMac,
NL80211_TDLS_TEARDOWN,