samsung-sm8650/android_kernel_samsung_sm8650-modules
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

qcacld-3.0: Move sta state to not connected after try disconnect

Browse Source 
qcacld-2.0 to qcacld-3.0 propagation

If a connect is received with previous connect already in progress,
sta connect state is moved from connecting to disconnecting and
disconnect is initiated to stop the previous connect.

If previous is present in sme/roam pending queue the disconnect
will abort the previous connect by removing the cmd from pending
queue. In this case the connect state is not moved back to
not connected from disconecting.

Thus the current connect command fails and all new connect command
will fail as the connect state will always be in disconnecting state.

To avoid this, move the connect state to not connected, once try
disconnect is issued internally.

Change-Id: Ia563f85df84cb9dbb3d3a82bc97127d39e7b1421
CRs-Fixed: 961559
This commit is contained in:
Abhishek Singh
2016-01-19 14:12:00 +05:30
committed by Akash Patel
parent c49931aaf5
commit f6479c5709
1 changed files with 38 additions and 26 deletions
Download Patch File Download Diff File Expand all files Collapse all files

64
core/hdd/src/wlan_hdd_cfg80211.c
View File

@@ -8739,6 +8739,7 @@ static int wlan_hdd_try_disconnect(hdd_adapter_t *pAdapter)
unsigned long rc; unsigned long rc;
hdd_station_ctx_t *pHddStaCtx; hdd_station_ctx_t *pHddStaCtx;
eMib_dot11DesiredBssType connectedBssType; eMib_dot11DesiredBssType connectedBssType;
int status, result = 0;
pHddStaCtx = WLAN_HDD_GET_STATION_CTX_PTR(pAdapter); pHddStaCtx = WLAN_HDD_GET_STATION_CTX_PTR(pAdapter);
@@ -8752,36 +8753,48 @@ static int wlan_hdd_try_disconnect(hdd_adapter_t *pAdapter)
eConnectionState_Disconnecting); eConnectionState_Disconnecting);
/* Issue disconnect to CSR */ /* Issue disconnect to CSR */
INIT_COMPLETION(pAdapter->disconnect_comp_var); INIT_COMPLETION(pAdapter->disconnect_comp_var);
if (CDF_STATUS_SUCCESS ==
sme_roam_disconnect(WLAN_HDD_GET_HAL_CTX(pAdapter), status = sme_roam_disconnect(WLAN_HDD_GET_HAL_CTX(pAdapter),
pAdapter->sessionId, pAdapter->sessionId,
eCSR_DISCONNECT_REASON_UNSPECIFIED)) { eCSR_DISCONNECT_REASON_UNSPECIFIED);
rc = wait_for_completion_timeout(&pAdapter-> /*
disconnect_comp_var, * Wait here instead of returning directly, this will block the
msecs_to_jiffies * next connect command and allow processing of the scan for
(WLAN_WAIT_TIME_DISCONNECT)); * ssid and the previous connect command in CSR. Else we might
if (!rc) { * hit some race conditions leading to SME and HDD out of sync.
hddLog(LOGE, */
FL("Sme disconnect event timed out session Id %d staDebugState %d"), if (CDF_STATUS_CMD_NOT_QUEUED == status) {
pAdapter->sessionId, hdd_info("Already disconnected or connect was in sme/roam pending list and removed by disconnect");
pHddStaCtx->staDebugState); } else if (0 != status) {
return -EALREADY; hdd_err("csrRoamDisconnect failure, returned %d",
} (int)status);
pHddStaCtx->staDebugState = status;
result = -EINVAL;
goto disconnected;
}
rc = wait_for_completion_timeout(
&pAdapter->disconnect_comp_var,
msecs_to_jiffies(WLAN_WAIT_TIME_DISCONNECT));
if (!rc && (CDF_STATUS_CMD_NOT_QUEUED != status)) {
hdd_err("Sme disconnect event timed out session Id %d staDebugState %d",
pAdapter->sessionId, pHddStaCtx->staDebugState);
result = -ETIMEDOUT;
} }
} else if (eConnectionState_Disconnecting == } else if (eConnectionState_Disconnecting ==
pHddStaCtx->conn_info.connState) { pHddStaCtx->conn_info.connState) {
rc = wait_for_completion_timeout(&pAdapter->disconnect_comp_var, rc = wait_for_completion_timeout(&pAdapter->disconnect_comp_var,
msecs_to_jiffies msecs_to_jiffies(WLAN_WAIT_TIME_DISCONNECT));
(WLAN_WAIT_TIME_DISCONNECT));
if (!rc) { if (!rc) {
hddLog(LOGE, hdd_err("Disconnect event timed out session Id %d staDebugState %d",
FL("Disconnect event timed out session Id %d staDebugState %d"),
pAdapter->sessionId, pHddStaCtx->staDebugState); pAdapter->sessionId, pHddStaCtx->staDebugState);
return -EALREADY; result = -ETIMEDOUT;
} }
} }
disconnected:
return 0; hdd_info("Set HDD connState to eConnectionState_NotConnected");
hdd_conn_set_connection_state(pAdapter, eConnectionState_NotConnected);
return result;
} }
/** /**
@@ -8952,8 +8965,7 @@ int wlan_hdd_disconnect(hdd_adapter_t *pAdapter, u16 reason)
* race conditions leading to SME and HDD out of sync. * race conditions leading to SME and HDD out of sync.
*/ */
if (CDF_STATUS_CMD_NOT_QUEUED == status) { if (CDF_STATUS_CMD_NOT_QUEUED == status) {
hddLog(LOG1, FL("status = %d, already disconnected"), hdd_info("Already disconnected or connect was in sme/roam pending list and removed by disconnect");
(int)status);
} else if (0 != status) { } else if (0 != status) {
hddLog(LOGE, hddLog(LOGE,
FL("csr_roam_disconnect failure, returned %d"), FL("csr_roam_disconnect failure, returned %d"),
@@ -8966,7 +8978,7 @@ int wlan_hdd_disconnect(hdd_adapter_t *pAdapter, u16 reason)
msecs_to_jiffies msecs_to_jiffies
(WLAN_WAIT_TIME_DISCONNECT)); (WLAN_WAIT_TIME_DISCONNECT));
if (!rc) { if (!rc && (CDF_STATUS_CMD_NOT_QUEUED != status)) {
hddLog(LOGE, hddLog(LOGE,
FL("Failed to disconnect, timed out")); FL("Failed to disconnect, timed out"));
result = -ETIMEDOUT; result = -ETIMEDOUT;