diff options
| author | Dennis Jeon <dennis.jeon@broadcom.corp-partner.google.com> | 2023-03-02 15:42:43 +0900 |
|---|---|---|
| committer | Isaac Chiou <isaacchiou@google.com> | 2023-03-02 09:53:53 +0000 |
| commit | f099d4fa5c9c83f8146c7d1507754f4d72dd72da (patch) | |
| tree | d6fe8f11f9fef4f9b7a2383affadeed9e453b1f6 | |
| parent | 35a6f33c96cd2341b3a7a4cc2fe0d305e96a19f4 (diff) | |
| download | bcm4389-f099d4fa5c9c83f8146c7d1507754f4d72dd72da.tar.gz | |
bcmdhd: Disable LOGSET_BEYOND memdump for customer platformandroid-t-qpr3-beta-3_r0.5android-t-qpr3-beta-3_r0.4android-t-qpr3-beta-3_r0.3android-t-qpr3-beta-3.1_r0.5android-t-qpr3-beta-3.1_r0.4android-t-qpr3-beta-3.1_r0.3android-13.0.0_r0.92android-13.0.0_r0.85android-13.0.0_r0.84android-13.0.0_r0.83android-13.0.0_r0.82android-13.0.0_r0.127android-13.0.0_r0.126android-13.0.0_r0.125android-13.0.0_r0.124android-13.0.0_r0.123android-13.0.0_r0.121android-13.0.0_r0.117android-13.0.0_r0.116android-13.0.0_r0.115android-13.0.0_r0.114android-13.0.0_r0.113android-13.0.0_r0.112android-13.0.0_r0.107android-13.0.0_r0.106android-13.0.0_r0.105android-13.0.0_r0.104android-13.0.0_r0.103android-13.0.0_r0.100android-gs-tangorpro-5.10-android13-qpr3android-gs-raviole-5.10-t-qpr3-beta-3android-gs-raviole-5.10-android13-qpr3android-gs-pantah-5.10-t-qpr3-beta-3android-gs-pantah-5.10-android13-qpr3android-gs-lynx-5.10-android13-qpr3android-gs-felix-5.10-android13-qpr3-candroid-gs-felix-5.10-android13-qpr3android-gs-bluejay-5.10-t-qpr3-beta-3android-gs-bluejay-5.10-android13-qpr3
As this logset beyond range memdump is only useful for debug purpose of the fw logs and no specific use of functionality.
Disable this memdump collection for customer platform and keep it only for our reference platforms.
Bug: 270914537
Bug: 270587504
Test: Verified with pa/2407362 which mimics the issue
Change-Id: Ic79b7800bf56daff32aa777c46c265b1f0413de9
Signed-off-by: Dennis Jeon <dennis.jeon@broadcom.corp-partner.google.com>
| -rw-r--r-- | Kbuild | 3 | ||||
| -rw-r--r-- | dhd_debug.c | 4 |
2 files changed, 7 insertions, 0 deletions
@@ -947,6 +947,9 @@ else ifneq ($(CONFIG_ARCH_HISI),) # Allow wl event forwarding as network packet DHDCFLAGS += -DWL_EVENT_ENAB + # Enable memdump for logset beyond range only internal builds + DHDCFLAGS += -DDHD_LOGSET_BEYOND_MEMDUMP + ifneq ($(CONFIG_BCMDHD_PCIE),) # LB RXP Flow control to avoid OOM DHDCFLAGS += -DLB_RXP_STOP_THR=200 -DLB_RXP_STRT_THR=199 diff --git a/dhd_debug.c b/dhd_debug.c index a7f4933..72e9689 100644 --- a/dhd_debug.c +++ b/dhd_debug.c @@ -1129,6 +1129,7 @@ dhd_dbg_msgtrace_log_parser(dhd_pub_t *dhdp, void *event_data, if (logset >= event_log_max_sets) { DHD_ERROR(("%s logset: %d max: %d out of range queried: %d\n", __FUNCTION__, logset, event_log_max_sets, event_log_max_sets_queried)); +#ifdef DHD_LOGSET_BEYOND_MEMDUMP #ifdef DHD_FW_COREDUMP if (event_log_max_sets_queried && !dhd_memdump_is_scheduled(dhdp)) { DHD_ERROR(("%s: collect socram for DUMP_TYPE_LOGSET_BEYOND_RANGE\n", @@ -1137,6 +1138,9 @@ dhd_dbg_msgtrace_log_parser(dhd_pub_t *dhdp, void *event_data, dhd_bus_mem_dump(dhdp); } #endif /* DHD_FW_COREDUMP */ +#else + goto exit; +#endif /* DHD_LOGSET_BEYOND_MEMDUMP */ } block = ltoh16(*((uint16 *)(data + 2))); |