commit 7dc0e6c3c1b50d0c643fca06758871ddaa2bdeba
author Hanumantha Reddy Pothula <c_hpothu@qti.qualcomm.com> Fri Mar 06 15:11:16 2015 +0530
committer Satyanarayana Dash <c_sadash@qti.qualcomm.com> Sat Mar 07 01:33:53 2015 +0530
tree f44798b9950f90503459b3c655ff3d63731960ff
parent 1de3e69806079cebe8350ed55901a61b5eae56c4

wlan: Cleanup North-Bound load/unload logs

If IOCTLs or CFG callbacks are issued continiously during driver
load/unload or during SSR, observing stability issues because of
excessive logging.
Hence move log level to INFO from ERROR in wlan_hdd_validate_context
function and don't print return status in its caller functions.

Change-Id: I8fd9b25e0c774e1e12b01128eb86783c9dec2ab7
CRs-Fixed: 800631

CORE/HDD/src/wlan_hdd_early_suspend.c

diff --git a/CORE/HDD/src/wlan_hdd_early_suspend.c b/CORE/HDD/src/wlan_hdd_early_suspend.c
index a96df9b..bbde004 100644
--- a/CORE/HDD/src/wlan_hdd_early_suspend.c
+++ b/CORE/HDD/src/wlan_hdd_early_suspend.c
@@ -459,7 +459,7 @@
     hdd_context_t *pHddCtx;
     int status;
 
-    hddLog(LOG1, FL("Reconfiguring NS Offload"));
+    ENTER();
     if (NULL == pAdapter)
     {
         hddLog(LOGE, FL("Adapter is invalid"));
@@ -470,7 +470,6 @@
     status = wlan_hdd_validate_context(pHddCtx);
     if (0 != status)
     {
-        hddLog(LOGE, FL("HDD context is invalid, status = %d"), status);
         return;
     }
 
@@ -503,8 +502,7 @@
      */
     wlan_hdd_set_mc_addr_list(pAdapter, TRUE);
 #endif
-
-
+    EXIT();
 }
 
 void hdd_ipv6_notifier_work_queue(struct work_struct *work)
@@ -523,11 +521,11 @@
     VOS_STATUS vos_status;
     int status;
 
+    ENTER();
     pHddCtx = container_of(nb, hdd_context_t, ipv6_notifier);
     status = wlan_hdd_validate_context(pHddCtx);
     if (0 != status)
     {
-        hddLog(LOGE, FL("HDD context is invalid"));
         return NOTIFY_DONE;
     }
 
@@ -553,7 +551,7 @@
         vos_status = hdd_get_next_adapter(pHddCtx, pAdapterNode, &pNext);
         pAdapterNode = pNext;
     }
-
+    EXIT();
     return NOTIFY_DONE;
 }
 
@@ -716,11 +714,8 @@
     pHddCtx = WLAN_HDD_GET_CTX(pAdapter);
 
     ret = wlan_hdd_validate_context(pHddCtx);
-
     if (0 != ret)
     {
-        VOS_TRACE( VOS_MODULE_ID_HDD, VOS_TRACE_LEVEL_ERROR,
-                   FL("HDD context is not valid"));
         return;
     }
 
@@ -937,6 +932,7 @@
     {
        vos_mem_free(selfIPv6AddrValid);
     }
+    EXIT();
     return;
 }
 #endif
@@ -948,7 +944,7 @@
     hdd_context_t *pHddCtx;
     int status;
 
-    hddLog(LOG1, FL("Reconfiguring ARP Offload"));
+    ENTER();
     if (NULL == pAdapter)
     {
         hddLog(LOGE, FL("Adapter is invalid"));
@@ -958,7 +954,6 @@
     status = wlan_hdd_validate_context(pHddCtx);
     if (0 != status)
     {
-        hddLog(LOGE, FL("HDD context is invalid, status = %d"), status);
         return;
     }
 
@@ -980,6 +975,7 @@
         // of IPv4 notifier again.
         hdd_conf_arp_offload(pAdapter, 2);
     }
+    EXIT();
 }
 
 void hdd_ipv4_notifier_work_queue(struct work_struct *work)
@@ -1002,11 +998,11 @@
     VOS_STATUS vos_status;
     int status;
 
+    ENTER();
     pHddCtx = container_of(nb, hdd_context_t, ipv4_notifier);
     status = wlan_hdd_validate_context(pHddCtx);
     if (0 != status)
     {
-        hddLog(LOGE, FL("HDD context is invalid"));
         return NOTIFY_DONE;
     }
 
@@ -1049,7 +1045,7 @@
         vos_status = hdd_get_next_adapter(pHddCtx, pAdapterNode, &pNext);
         pAdapterNode = pNext;
     }
-
+    EXIT();
     return NOTIFY_DONE;
 }