From 07bbe12d29397350c316b5da822ffa4394214624 Mon Sep 17 00:00:00 2001 From: Rajeev Kumar Date: Tue, 24 Oct 2017 13:06:05 -0700 Subject: [PATCH] qcacld-3.0: Remove faulty unload check from stop bss callback API Stop BSS callback runs in driver's internal scheduler thread context which should not check for driver load/unload state which is required for external thread of execution. Stop BSS callback is supposed to free the peer object which is created by HDD when driver gets unloaded. Change-Id: I3b8586498f3e9783be3ca4ba8cc06b7f1e9d70a2 CRs-Fixed: 2131209 --- core/hdd/src/wlan_hdd_softap_tx_rx.c | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/core/hdd/src/wlan_hdd_softap_tx_rx.c b/core/hdd/src/wlan_hdd_softap_tx_rx.c index bb85ae3ba4ee..242c935d8bee 100644 --- a/core/hdd/src/wlan_hdd_softap_tx_rx.c +++ b/core/hdd/src/wlan_hdd_softap_tx_rx.c @@ -1041,15 +1041,10 @@ QDF_STATUS hdd_softap_stop_bss(struct hdd_adapter *adapter) hdd_ctx = WLAN_HDD_GET_CTX(adapter); - /* bss deregister is not allowed during wlan driver loading or - * unloading + /* This is stop bss callback running in scheduler thread so do not + * driver unload in progress check otherwise it can lead to peer + * object leak */ - if (cds_is_load_or_unload_in_progress()) { - hdd_err("Loading_unloading in Progress, state: 0x%x. Ignore!!!", - cds_get_driver_state()); - return QDF_STATUS_E_PERM; - } - qdf_status = hdd_softap_deregister_bc_sta(adapter); if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {