diff options
| author | Mahesh A Saptasagar <c_msapta@qti.qualcomm.com> | 2015-06-17 12:35:06 +0530 |
|---|---|---|
| committer | Gerrit - the friendly Code Review server <code-review@localhost> | 2015-06-29 07:24:25 -0700 |
| commit | 55ea7e0b217bc762d6d36a88bc341fbd637a7fe2 (patch) | |
| tree | 7a09d715aecfb08e112a2232965a07d2d2a1871a | |
| parent | 78bf7c871d1fce4ee82804e55f2afa11d6f3074a (diff) | |
qcacld-2.0: Address excessive logging issues during load/unload
prima to qcacld-2.0 propagation
Resolve excessive logging issues by moving the log levels from
ERROR to INFO in hdd_validate_context function and also don't
print the failure status of hdd_validate_context function in
__hdd_ioctl and __hdd_hostapd_ioctl functions.
Change-Id: Ieedccaa6c9724c336e68cc373e588990049fdeb0
CRs-Fixed: 802404
| -rw-r--r-- | CORE/HDD/src/wlan_hdd_hostapd.c | 1 | ||||
| -rwxr-xr-x | CORE/HDD/src/wlan_hdd_main.c | 9 |
2 files changed, 3 insertions, 7 deletions
diff --git a/CORE/HDD/src/wlan_hdd_hostapd.c b/CORE/HDD/src/wlan_hdd_hostapd.c index b3abe719dc62..a9e0a772f18c 100644 --- a/CORE/HDD/src/wlan_hdd_hostapd.c +++ b/CORE/HDD/src/wlan_hdd_hostapd.c @@ -568,7 +568,6 @@ static int __hdd_hostapd_ioctl(struct net_device *dev, pHddCtx = WLAN_HDD_GET_CTX(pAdapter); ret = wlan_hdd_validate_context(pHddCtx); if (ret) { - hddLog(VOS_TRACE_LEVEL_ERROR, "%s: invalid context", __func__); ret = -EBUSY; goto exit; } diff --git a/CORE/HDD/src/wlan_hdd_main.c b/CORE/HDD/src/wlan_hdd_main.c index b05d81809e67..290bd7276f24 100755 --- a/CORE/HDD/src/wlan_hdd_main.c +++ b/CORE/HDD/src/wlan_hdd_main.c @@ -855,21 +855,20 @@ static void hdd_vos_trace_enable(VOS_MODULE_ID moduleId, v_U32_t bitmask) */ int wlan_hdd_validate_context(hdd_context_t *pHddCtx) { - ENTER(); if (NULL == pHddCtx || NULL == pHddCtx->cfg_ini) { - hddLog(LOGE, FL("HDD context is Null")); + hddLog(LOG1, FL("HDD context is Null")); return -ENODEV; } if (pHddCtx->isLogpInProgress) { - hddLog(LOGE, FL("LOGP in Progress. Ignore!!!")); + hddLog(LOG1, FL("LOGP in Progress. Ignore!!!")); return -EAGAIN; } if ((pHddCtx->isLoadInProgress) || (pHddCtx->isUnloadInProgress)) { - hddLog(LOGE, FL("Unloading/Loading in Progress. Ignore!!!")); + hddLog(LOG1, FL("Unloading/Loading in Progress. Ignore!!!")); return -EAGAIN; } return 0; @@ -6165,8 +6164,6 @@ static int __hdd_ioctl(struct net_device *dev, struct ifreq *ifr, int cmd) pHddCtx = WLAN_HDD_GET_CTX(pAdapter); ret = wlan_hdd_validate_context(pHddCtx); if (ret) { - VOS_TRACE(VOS_MODULE_ID_HDD, VOS_TRACE_LEVEL_ERROR, - "%s: invalid context", __func__); ret = -EBUSY; goto exit; } |