qcacld-3.0: Use cached data for get ml link state

Optimize Ml link state updates to reduce frequent communication from
firmware, preventing unnecessary runtime resume and minimizing power
penalties.

This change is to introduce cached ML link state data, ensuring
efficient communication until the expiration of ML link state cache.

CRs-Fixed: 3650673
Change-Id: I3fb742ed18fa7e364f25fa9ed10092d3a372391b

core/hdd/inc/hdd_config.h

@@ -1,6 +1,6 @@
 /*
  * Copyright (c) 2012-2021 The Linux Foundation. All rights reserved.
- * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved.
+ * Copyright (c) 2021-2024 Qualcomm Innovation Center, Inc. All rights reserved.
  *
  * Permission to use, copy, modify, and/or distribute this software for
  * any purpose with or without fee is hereby granted, provided that the
@@ -1194,6 +1194,40 @@ struct dhcp_server {
 #define CFG_WLAN_CLUB_GET_STA_IN_LL_STA_REQ
 #endif /* FEATURE_CLUB_LL_STATS_AND_GET_STATION */
 
+#ifdef WLAN_FEATURE_11BE_MLO
+/*
+ * <ini>
+ * link_state_cache_expiry_time - Expiry time for cached ml link state
+ *
+ * @Min: 0
+ * @Max: 5000
+ * Default: 400
+ *
+ * This ini is used as duration in milliseconds for which cached ml link state
+ * are valid. Driver sends the cached information as response, if it gets the
+ * ml link state request with in this duration. Otherwise driver sends new
+ * request to the firmware to get the updated ml link state.
+ *
+ * Supported Feature: MLO STA
+ *
+ * Usage: External
+ *
+ * </ini>
+ */
+#define CFG_LINK_STATE_CACHE_EXPIRY  CFG_INI_UINT( \
+			"link_state_cache_expiry_time", \
+			0, \
+			5000, \
+			400, \
+			CFG_VALUE_OR_DEFAULT, \
+			"link state cache expiry")
+
+#define CFG_LINK_STATE_CACHE_EXPIRY_ALL \
+	CFG(CFG_LINK_STATE_CACHE_EXPIRY)
+#else
+#define CFG_LINK_STATE_CACHE_EXPIRY_ALL
+#endif /* WLAN_FEATURE_11BE_MLO */
+
 /**
  * enum host_log_level - Debug verbose level imposed by user
  * @HOST_LOG_LEVEL_NONE: no trace will be logged.
@@ -1348,5 +1382,6 @@ enum host_log_level {
 	SAR_SAFETY_FEATURE_ALL \
 	CFG_GET_WIFI_FEATURES_ALL \
 	CFG_CPU_CXPC_THRESHOLD_ALL \
-	CFG(CFG_EXCLUDE_SELFTX_FROM_CCA_BUSY_TIME)
+	CFG(CFG_EXCLUDE_SELFTX_FROM_CCA_BUSY_TIME) \
+	CFG_LINK_STATE_CACHE_EXPIRY_ALL
 #endif

core/hdd/inc/wlan_hdd_cfg.h

@@ -1,6 +1,6 @@
 /*
  * Copyright (c) 2012-2021 The Linux Foundation. All rights reserved.
- * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved.
+ * Copyright (c) 2021-2024 Qualcomm Innovation Center, Inc. All rights reserved.
  *
  * Permission to use, copy, modify, and/or distribute this software for
  * any purpose with or without fee is hereby granted, provided that the
@@ -207,6 +207,10 @@ struct hdd_config {
 	uint16_t cpu_cxpc_threshold;
 #endif
 	bool exclude_selftx_from_cca_busy;
+#ifdef WLAN_FEATURE_11BE_MLO
+	/* ml link state cache expiry time*/
+	qdf_time_t link_state_cache_expiry_time;
+#endif
 };
 
 /**

core/hdd/inc/wlan_hdd_main.h

@@ -1,6 +1,6 @@
 /*
  * Copyright (c) 2012-2021 The Linux Foundation. All rights reserved.
- * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved.
+ * Copyright (c) 2021-2024 Qualcomm Innovation Center, Inc. All rights reserved.
  *
  * Permission to use, copy, modify, and/or distribute this software for
  * any purpose with or without fee is hereby granted, provided that the
@@ -1295,6 +1295,7 @@ struct wlan_hdd_tx_power {
  * @link_info: Data structure to hold link specific information
  * @tx_power: Structure to hold connection tx Power info
  * @tx_latency_cfg: configuration for per-link transmit latency statistics
+ * @link_state_cached_timestamp: link state cached timestamp
  */
 struct hdd_adapter {
 	uint32_t magic;
@@ -1489,6 +1490,9 @@ struct hdd_adapter {
 #ifdef WLAN_FEATURE_TX_LATENCY_STATS
 	struct cdp_tx_latency_config tx_latency_cfg;
 #endif
+#ifdef WLAN_FEATURE_11BE_MLO
+	qdf_time_t link_state_cached_timestamp;
+#endif
 };
 
 #define WLAN_HDD_GET_STATION_CTX_PTR(link_info) (&(link_info)->session.station)

core/hdd/inc/wlan_hdd_mlo.h

@@ -1,6 +1,6 @@
 /*
  * Copyright (c) 2021, The Linux Foundation. All rights reserved.
- * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved.
+ * Copyright (c) 2021-2024 Qualcomm Innovation Center, Inc. All rights reserved.
  *
  * Permission to use, copy, modify, and/or distribute this software for any
  * purpose with or without fee is hereby granted, provided that the above
@@ -280,6 +280,7 @@ void hdd_mlo_t2lm_unregister_callback(struct wlan_objmgr_vdev *vdev);
 
 /**
  * wlan_handle_mlo_link_state_operation() - mlo link state operation
+ * @adapter: HDD adapter
  * @wiphy: wiphy pointer
  * @vdev: vdev handler
  * @hdd_ctx: hdd context
@@ -290,7 +291,8 @@ void hdd_mlo_t2lm_unregister_callback(struct wlan_objmgr_vdev *vdev);
  *
  * Return: 0 on success and error number otherwise.
  */
-int wlan_handle_mlo_link_state_operation(struct wiphy *wiphy,
+int wlan_handle_mlo_link_state_operation(struct hdd_adapter *adapter,
+					 struct wiphy *wiphy,
 					 struct wlan_objmgr_vdev *vdev,
 					 struct hdd_context *hdd_ctx,
 					 const void *data, int data_len);
@@ -323,6 +325,13 @@ QDF_STATUS wlan_hdd_send_t2lm_event(struct wlan_objmgr_vdev *vdev,
 int wlan_hdd_cfg80211_process_ml_link_state(struct wiphy *wiphy,
 					    struct wireless_dev *wdev,
 					    const void *data, int data_len);
+/**
+ * hdd_update_link_state_cached_timestamp() - update link state cached timestamp
+ * @adapter: HDD adapter
+ *
+ * Return: none
+ */
+void hdd_update_link_state_cached_timestamp(struct hdd_adapter *adapter);
 
 /**
  * hdd_derive_link_address_from_mld() - Function to derive link address from
@@ -404,7 +413,8 @@ void hdd_mlo_t2lm_unregister_callback(struct wlan_objmgr_vdev *vdev)
 }
 
 static inline int
-wlan_handle_mlo_link_state_operation(struct wiphy *wiphy,
+wlan_handle_mlo_link_state_operation(struct hdd_adapter *adapter,
+				     struct wiphy *wiphy,
 				     struct wlan_objmgr_vdev *vdev,
 				     struct hdd_context *hdd_ctx,
 				     const void *data, int data_len)
@@ -435,6 +445,12 @@ QDF_STATUS wlan_hdd_send_t2lm_event(struct wlan_objmgr_vdev *vdev,
 {
 	return QDF_STATUS_E_NOSUPPORT;
 }
+
+static inline
+void hdd_update_link_state_cached_timestamp(struct hdd_adapter *adapter)
+{
+}
+
 #define FEATURE_ML_LINK_STATE_COMMANDS
 #endif
 #endif

core/hdd/src/wlan_hdd_main.c

@@ -1,6 +1,6 @@
 /*
  * Copyright (c) 2012-2021 The Linux Foundation. All rights reserved.
- * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved.
+ * Copyright (c) 2021-2024 Qualcomm Innovation Center, Inc. All rights reserved.
  *
  * Permission to use, copy, modify, and/or distribute this software for
  * any purpose with or without fee is hereby granted, provided that the
@@ -1728,6 +1728,35 @@ hdd_init_get_sta_in_ll_stats_config(struct hdd_adapter *adapter)
 }
 #endif /* FEATURE_CLUB_LL_STATS_AND_GET_STATION */
 
+#ifdef WLAN_FEATURE_11BE_MLO
+
+static void
+hdd_init_link_state_cfg(struct hdd_config *config,
+			struct wlan_objmgr_psoc *psoc)
+{
+	config->link_state_cache_expiry_time =
+		cfg_get(psoc, CFG_LINK_STATE_CACHE_EXPIRY);
+}
+
+static void
+hdd_init_link_state_config(struct hdd_adapter *adapter)
+{
+	adapter->link_state_cached_timestamp = 0;
+}
+
+#else
+static void
+hdd_init_link_state_cfg(struct hdd_config *config,
+			struct wlan_objmgr_psoc *psoc)
+{
+}
+
+static void
+hdd_init_link_state_config(struct hdd_adapter *adapter)
+{
+}
+#endif /* WLAN_FEATURE_11BE_MLO */
+
 #ifdef WLAN_FEATURE_IGMP_OFFLOAD
 static void
 hdd_intersect_igmp_offload_setting(struct wlan_objmgr_psoc *psoc,
@@ -6946,6 +6975,7 @@ hdd_alloc_station_adapter(struct hdd_context *hdd_ctx, tSirMacAddr mac_addr,
 
 	qdf_atomic_init(&adapter->is_ll_stats_req_pending);
 	hdd_init_get_sta_in_ll_stats_config(adapter);
+	hdd_init_link_state_config(adapter);
 
 	return adapter;
 
@@ -14537,6 +14567,7 @@ static void hdd_cfg_params_init(struct hdd_context *hdd_ctx)
 
 	config->exclude_selftx_from_cca_busy =
 			cfg_get(psoc, CFG_EXCLUDE_SELFTX_FROM_CCA_BUSY_TIME);
+	hdd_init_link_state_cfg(config, psoc);
 }
 
 #ifdef CONNECTION_ROAMING_CFG

core/hdd/src/wlan_hdd_mlo.c

@@ -1,6 +1,6 @@
 /*
  * Copyright (c) 2021, The Linux Foundation. All rights reserved.
- * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved.
+ * Copyright (c) 2021-2024 Qualcomm Innovation Center, Inc. All rights reserved.
  *
  * Permission to use, copy, modify, and/or distribute this software for any
  * purpose with or without fee is hereby granted, provided that the above
@@ -637,8 +637,8 @@ __wlan_hdd_cfg80211_process_ml_link_state(struct wiphy *wiphy,
 	if (!wlan_vdev_mlme_is_mlo_vdev(vdev))
 		goto release_ref;
 
-	ret = wlan_handle_mlo_link_state_operation(wiphy, vdev, hdd_ctx,
-						   data, data_len);
+	ret = wlan_handle_mlo_link_state_operation(adapter, wiphy, vdev,
+						   hdd_ctx, data, data_len);
 
 release_ref:
 	hdd_objmgr_put_vdev_by_user(vdev, WLAN_OSIF_ID);
@@ -785,7 +785,112 @@ static char *link_state_status_id_to_str(uint32_t status)
 	return "Undefined link state status ID";
 }
 
-static QDF_STATUS wlan_hdd_link_state_request(struct wiphy *wiphy,
+static bool
+wlan_hdd_link_state_request_needed(struct hdd_adapter *adapter)
+{
+	qdf_time_t link_state_cached_duration = 0;
+
+	link_state_cached_duration =
+				qdf_system_ticks_to_msecs(qdf_system_ticks()) -
+				adapter->link_state_cached_timestamp;
+	if (link_state_cached_duration <=
+		adapter->hdd_ctx->config->link_state_cache_expiry_time)
+		return false;
+
+	return true;
+}
+
+#ifdef WLAN_FEATURE_11BE_MLO
+
+void hdd_update_link_state_cached_timestamp(struct hdd_adapter *adapter)
+{
+	adapter->link_state_cached_timestamp =
+		qdf_system_ticks_to_msecs(qdf_system_ticks());
+}
+#endif /* WLAN_FEATURE_11BE_MLO */
+
+static QDF_STATUS
+wlan_hdd_cached_link_state_request(struct hdd_adapter *adapter,
+				   struct wiphy *wiphy,
+				   struct wlan_objmgr_psoc *psoc,
+				   struct wlan_objmgr_vdev *vdev)
+{
+	QDF_STATUS status = QDF_STATUS_E_INVAL;
+	struct ml_link_state_info_event link_state_event = {0};
+	struct wlan_hdd_link_info *link_info;
+	struct hdd_station_ctx *sta_ctx;
+	int skb_len;
+	struct sk_buff *reply_skb = NULL;
+	int errno;
+	struct qdf_mac_addr *mld_addr;
+	uint8_t link_iter = 0;
+
+	hdd_adapter_for_each_link_info(adapter, link_info) {
+
+		if (link_iter >= WLAN_MAX_ML_BSS_LINKS) {
+			hdd_err("Invalid number of link info");
+			return -EINVAL;
+		}
+
+		sta_ctx = WLAN_HDD_GET_STATION_CTX_PTR(link_info);
+		link_state_event.link_info[link_iter].link_id =
+				sta_ctx->conn_info.ieee_link_id;
+		link_state_event.link_info[link_iter].link_status =
+				link_info->is_mlo_vdev_active;
+		link_state_event.link_info[link_iter].vdev_id =
+				link_info->vdev_id;
+		link_state_event.link_info[link_iter].chan_freq =
+				sta_ctx->ch_info.freq;
+
+		if (sta_ctx->conn_info.ieee_link_id == WLAN_INVALID_LINK_ID)
+			continue;
+
+		link_iter++;
+
+		hdd_debug_rl("vdev id %d sta_ctx->conn_info.ieee_link_id %d is_mlo_vdev_active %d ",
+			     link_info->vdev_id, sta_ctx->conn_info.ieee_link_id,
+			     link_info->is_mlo_vdev_active);
+	}
+
+	link_state_event.num_mlo_vdev_link_info = link_iter;
+	link_state_event.vdev_id = wlan_vdev_get_id(vdev);
+	link_state_event.status = 0;
+	mld_addr = (struct qdf_mac_addr *)wlan_vdev_mlme_get_mldaddr(vdev);
+	link_state_event.mldaddr = *mld_addr;
+
+	hdd_debug_rl("cached link_state_resp: vdev id %d status %d num %d MAC addr " QDF_MAC_ADDR_FMT,
+		     link_state_event.vdev_id, link_state_event.status,
+		     link_state_event.num_mlo_vdev_link_info,
+		     QDF_MAC_ADDR_REF(link_state_event.mldaddr.bytes));
+
+	skb_len = hdd_get_ml_link_state_response_len(&link_state_event);
+
+	reply_skb = wlan_cfg80211_vendor_cmd_alloc_reply_skb(wiphy, skb_len);
+	if (!reply_skb) {
+		hdd_err("Get stats - alloc reply_skb failed");
+		status = QDF_STATUS_E_NOMEM;
+		return status;
+	}
+
+	status = hdd_ml_generate_link_state_resp_nlmsg(
+			reply_skb, psoc, &link_state_event,
+			link_state_event.num_mlo_vdev_link_info);
+	if (QDF_IS_STATUS_ERROR(status)) {
+		hdd_err("Failed to pack nl response");
+		goto free_skb;
+	}
+
+	errno = wlan_cfg80211_vendor_cmd_reply(reply_skb);
+
+	return qdf_status_from_os_return(errno);
+
+free_skb:
+	wlan_cfg80211_vendor_free_skb(reply_skb);
+	return QDF_STATUS_SUCCESS;
+}
+
+static QDF_STATUS wlan_hdd_link_state_request(struct hdd_adapter *adapter,
+					      struct wiphy *wiphy,
 					      struct wlan_objmgr_psoc *psoc,
 					      struct wlan_objmgr_vdev *vdev)
 {
@@ -807,6 +912,16 @@ static QDF_STATUS wlan_hdd_link_state_request(struct wiphy *wiphy,
 	if (!wiphy || !vdev)
 		return status;
 
+	if (adapter->device_mode != QDF_STA_MODE)
+		return QDF_STATUS_SUCCESS;
+
+	if (!wlan_hdd_link_state_request_needed(adapter)) {
+		hdd_debug_rl("sending cached link state request");
+		status = wlan_hdd_cached_link_state_request(adapter, wiphy,
+							    psoc, vdev);
+		return status;
+	}
+
 	request = osif_request_alloc(&params);
 	if (!request)
 		return QDF_STATUS_E_NOMEM;
@@ -857,6 +972,8 @@ static QDF_STATUS wlan_hdd_link_state_request(struct wiphy *wiphy,
 			  link_state_event->link_info[num_info].link_status);
 	}
 
+	hdd_update_link_state_cached_timestamp(adapter);
+
 	skb_len = hdd_get_ml_link_state_response_len(link_state_event);
 
 	reply_skb = wlan_cfg80211_vendor_cmd_alloc_reply_skb(
@@ -879,6 +996,7 @@ static QDF_STATUS wlan_hdd_link_state_request(struct wiphy *wiphy,
 	osif_request_put(request);
 
 	errno = wlan_cfg80211_vendor_cmd_reply(reply_skb);
+
 	return qdf_status_from_os_return(errno);
 
 free_skb:
@@ -891,7 +1009,8 @@ free_event:
 
 #define MLD_MAX_SUPPORTED_LINKS 2
 
-int wlan_handle_mlo_link_state_operation(struct wiphy *wiphy,
+int wlan_handle_mlo_link_state_operation(struct hdd_adapter *adapter,
+					 struct wiphy *wiphy,
 					 struct wlan_objmgr_vdev *vdev,
 					 struct hdd_context *hdd_ctx,
 					 const void *data, int data_len)
@@ -925,8 +1044,8 @@ int wlan_handle_mlo_link_state_operation(struct wiphy *wiphy,
 	ml_link_op = nla_get_u8(link_oper_attr);
 	switch (ml_link_op) {
 	case QCA_WLAN_VENDOR_LINK_STATE_OP_GET:
-		status = wlan_hdd_link_state_request(wiphy, hdd_ctx->psoc,
-						     vdev);
+		status = wlan_hdd_link_state_request(adapter, wiphy,
+						     hdd_ctx->psoc, vdev);
 		return qdf_status_to_os_return(status);
 	case QCA_WLAN_VENDOR_LINK_STATE_OP_SET:
 		if (policy_mgr_is_set_link_in_progress(hdd_ctx->psoc)) {

core/hdd/src/wlan_hdd_stats.c

@@ -3451,8 +3451,10 @@ static int wlan_hdd_send_ll_stats_req(struct wlan_hdd_link_info *link_info,
 		sme_radio_tx_mem_free();
 		ret = -ETIMEDOUT;
 	} else {
-		if (QDF_IS_STATUS_SUCCESS(vdev_req_status))
+		if (QDF_IS_STATUS_SUCCESS(vdev_req_status)) {
 			hdd_update_station_stats_cached_timestamp(adapter);
+			hdd_update_link_state_cached_timestamp(adapter);
+		}
 
 		adapter->ll_stats_failure_count = 0;
 	}
@@ -9291,6 +9293,7 @@ int wlan_hdd_get_station_stats(struct wlan_hdd_link_info *link_info)
 
 	/* update get stats cached time stamp */
 	hdd_update_station_stats_cached_timestamp(link_info->adapter);
+	hdd_update_link_state_cached_timestamp(link_info->adapter);
 	copy_station_stats_to_adapter(link_info, stats);
 out:
 	wlan_cfg80211_mc_cp_stats_free_stats_event(stats);