diff options
author | Dennis Jeon <dennis.jeon@broadcom.com> | 2021-07-27 11:27:38 +0900 |
---|---|---|
committer | Ahmed ElArabawy <arabawy@google.com> | 2021-07-27 14:14:06 -0700 |
commit | 958be197c2311ae16c51e1857df8fad7eae72fc7 (patch) | |
tree | ffe74c64e7aaf0b2c2a6bb55aff27a00197214e5 | |
parent | ea2920c92ace0bf0fb5792866a70b98c3abb5453 (diff) | |
download | wlan-958be197c2311ae16c51e1857df8fad7eae72fc7.tar.gz |
HAL: Add logs to SSR command for debugging
HAL starting log
07-21 16:38:04.038 383 383 I WifiHAL : Start Restart Handler
07-21 16:38:04.038 383 383 I WifiHAL : Register SSR handler:0x5766bd4c48
call recovery (cmd wifi trigger-recovery)
07-21 16:38:54.596 383 383 I WifiHAL : Trigger subsystem restart
07-21 16:38:54.596 383 383 I WifiHAL : found interface wlan0
07-21 16:38:54.604 383 383 I WifiHAL : Trigger ssr handler registered handler:0x5766bd4c48
BUG: 194087289
Test: No code change, just logs
Change-Id: I874a361e3a26a000ecca0584f9489d66dc2b7081
-rwxr-xr-x | bcmdhd/wifi_hal/wifi_logger.cpp | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/bcmdhd/wifi_hal/wifi_logger.cpp b/bcmdhd/wifi_hal/wifi_logger.cpp index 364e4fd..8fcf6a4 100755 --- a/bcmdhd/wifi_hal/wifi_logger.cpp +++ b/bcmdhd/wifi_hal/wifi_logger.cpp @@ -946,17 +946,17 @@ public: : WifiCommand("SetRestartHandler", handle, id), mHandler(handler), mBuff(NULL) { } int start() { - ALOGV("Start Restart Handler"); + ALOGI("Start Restart Handler"); registerVendorHandler(BRCM_OUI, BRCM_VENDOR_EVENT_HANGED); return WIFI_SUCCESS; } virtual int cancel() { - ALOGV("Clear Restart Handler"); + ALOGI("Clear Restart Handler"); /* unregister alert handler */ unregisterVendorHandler(BRCM_OUI, BRCM_VENDOR_EVENT_HANGED); wifi_unregister_cmd(wifiHandle(), id()); - ALOGD("Success to clear restarthandler"); + ALOGI("Success to clear restarthandler"); return WIFI_SUCCESS; } @@ -980,7 +980,7 @@ public: if (it.get_type() == LOGGER_ATTRIBUTE_HANG_REASON) { mBuff = (char *)it.get_data(); } else { - ALOGW("Ignoring invalid attribute type = %d, size = %d", + ALOGI("Ignoring invalid attribute type = %d, size = %d", it.get_type(), it.get_len()); } } @@ -988,7 +988,7 @@ public: if (*mHandler.on_subsystem_restart) { (*mHandler.on_subsystem_restart)(mBuff); } else { - ALOGW("No Restart handler registered"); + ALOGI("No Restart handler registered"); } } return NL_OK; @@ -1211,6 +1211,7 @@ wifi_error wifi_set_subsystem_restart_handler(wifi_handle handle, } /* Cache the handler to use it for trigger subsystem restart */ + ALOGI("Register SSR handler:%p", handler); info->restart_handler = handler; return result; } @@ -1243,6 +1244,7 @@ wifi_error wifi_trigger_subsystem_restart(wifi_handle handle) if (result != WIFI_SUCCESS) { cmd->releaseRef(); strncpy(error_str, "WIFI_ERROR_UNKNOWN", sizeof(error_str)); + ALOGE("Failed to create SSR"); goto exit; } @@ -1250,9 +1252,11 @@ wifi_error wifi_trigger_subsystem_restart(wifi_handle handle) exit: if (info->restart_handler.on_subsystem_restart) { + ALOGI("Trigger ssr handler registered handler:%p", + info->restart_handler.on_subsystem_restart); (info->restart_handler.on_subsystem_restart)(error_str); } else { - ALOGW("No trigger ssr handler registered"); + ALOGI("No trigger ssr handler registered"); } return result; |