Jelajahi Sumber

qcacld-3.0: Prioritize connection req frames in pe queue

If multiple auth req comes, it added to the header of message queue
but in between mc thread could be processing some beacon frame. while
processing beacon, SAP gets assoc req and adds it to head of message
queue and will be processed first instead of auth req. After sending
of accoc rsp, mc thread will start to process queued auth req, which
is at the top of message queue which will result in deauthentication.

Reverts commit 360c5d590ec8 ("qcacld-3.0: Prioritize connection req
frames in pe queue")

Change-Id: Iaf511be1160e05cfcd9d60f18be4339587308089
CRs-Fixed: 2192836
Abhinav Kumar 7 tahun lalu
induk
melakukan
c93e5ad6c5
1 mengubah file dengan 3 tambahan dan 14 penghapusan
  1. 3 14
      core/mac/src/sys/legacy/src/system/src/sys_entry_func.c

+ 3 - 14
core/mac/src/sys/legacy/src/system/src/sys_entry_func.c

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2012-2017 The Linux Foundation. All rights reserved.
+ * Copyright (c) 2012-2018 The Linux Foundation. All rights reserved.
  *
  * Previously licensed under the ISC license by Qualcomm Atheros, Inc.
  *
@@ -164,19 +164,8 @@ sys_bbt_process_message_core(tpAniSirGlobal mac_ctx, struct scheduler_msg *msg,
 				mac_ctx->sys.gSysFrameCount[type][subtype]);
 		}
 
-		/*
-		 * Post the message to PE Queue. Prioritize the
-		 * Auth and assoc frames.
-		 */
-		if ((subtype == SIR_MAC_MGMT_AUTH) ||
-		   (subtype == SIR_MAC_MGMT_ASSOC_RSP) ||
-		   (subtype == SIR_MAC_MGMT_REASSOC_RSP) ||
-		   (subtype == SIR_MAC_MGMT_ASSOC_REQ) ||
-		   (subtype == SIR_MAC_MGMT_REASSOC_REQ))
-			ret = (tSirRetStatus)
-				   lim_post_msg_high_priority(mac_ctx, msg);
-		else
-			ret = (tSirRetStatus) lim_post_msg_api(mac_ctx, msg);
+		/* Post the message to PE Queue */
+		ret = (tSirRetStatus) lim_post_msg_api(mac_ctx, msg);
 		if (ret != eSIR_SUCCESS) {
 			pe_err("posting to LIM2 failed, ret %d\n", ret);
 			goto fail;