Browse Source

qcacld-3.0: Revert change to use TP_LOW lvl to reset tcp_adv_win_scale

tcp_adv_win_scale value does not get reset to the default
value by cnss daemon for WLAN_SVC_TP_LOW throughput level.
This could result in undesired behavior if the tcp param
value is used for TCP connections after disconnection or
wifi turn off.

Fix is to use the previous WLAN_SVC_TP_NONE level to
reset the tcp_adv_win_scale parameter.

Change-Id: I3d312abc2a873a7b35d8a411c60bce7832742c4b
CRs-Fixed: 2995589
Yeshwanth Sriram Guntuka 3 years ago
parent
commit
ec5f52c2ad
1 changed files with 5 additions and 7 deletions
  1. 5 7
      core/hdd/src/wlan_hdd_tx_rx.c

+ 5 - 7
core/hdd/src/wlan_hdd_tx_rx.c

@@ -3411,18 +3411,16 @@ void hdd_reset_tcp_delack(struct hdd_context *hdd_ctx)
 
 void hdd_reset_tcp_adv_win_scale(struct hdd_context *hdd_ctx)
 {
-	enum wlan_tp_level next_level = WLAN_SVC_TP_LOW;
+	enum wlan_tp_level next_level = WLAN_SVC_TP_NONE;
 	struct wlan_rx_tp_data rx_tp_data = {0};
 
 	if (!hdd_ctx->config->enable_tcp_adv_win_scale)
 		return;
 
-	if (hdd_ctx->cur_rx_level != next_level) {
-		rx_tp_data.rx_tp_flags |= TCP_ADV_WIN_SCL;
-		rx_tp_data.level = next_level;
-		hdd_ctx->cur_rx_level = next_level;
-		wlan_hdd_update_tcp_rx_param(hdd_ctx, &rx_tp_data);
-	}
+	rx_tp_data.rx_tp_flags |= TCP_ADV_WIN_SCL;
+	rx_tp_data.level = next_level;
+	hdd_ctx->cur_rx_level = WLAN_SVC_TP_NONE;
+	wlan_hdd_update_tcp_rx_param(hdd_ctx, &rx_tp_data);
 }
 
 /**