qcacld-3.0: Add check to prevent TWT CMD during suspend

Flush TWT commands when cfg80211 suspend is in progress
and schedule them for execution when cfg80211 resumes.
This prevents WMI from crashing due to concurrent
WMI_WOW_ENABLE_CMDID and twt requester enable commands
during suspend.

Change-Id: I505b789805d8a83564bf25d3f6c5c28d6d390966
CRs-Fixed: 3567817

components/umac/twt/core/src/wlan_twt_cfg.c

@@ -438,3 +438,16 @@ wlan_twt_get_restricted_support(struct wlan_objmgr_psoc *psoc, bool *val)
 
 	return QDF_STATUS_SUCCESS;
 }
+
+bool
+wlan_twt_get_pmo_allowed(struct wlan_objmgr_psoc *psoc)
+{
+	struct twt_psoc_priv_obj *twt_psoc_obj;
+
+	twt_psoc_obj = wlan_twt_psoc_get_comp_private_obj(psoc);
+
+	if (twt_psoc_obj->twt_pmo_disabled)
+		return false;
+	else
+		return true;
+}

components/umac/twt/core/src/wlan_twt_cfg.h

@@ -233,6 +233,14 @@ wlan_twt_cfg_get_support_in_11n_mode(struct wlan_objmgr_psoc *psoc,
 QDF_STATUS
 wlan_twt_get_restricted_support(struct wlan_objmgr_psoc *psoc, bool *val);
 
+/**
+ * wlan_twt_get_pmo_allowed() - Get pmo allowed
+ * @psoc: psoc handler
+ *
+ * Return: True if twt pmo is allowed otherwise false
+ */
+bool
+wlan_twt_get_pmo_allowed(struct wlan_objmgr_psoc *psoc);
 #else
 
 static inline QDF_STATUS wlan_twt_cfg_init(struct wlan_objmgr_psoc *psoc)
@@ -348,6 +356,12 @@ wlan_twt_get_restricted_support(struct wlan_objmgr_psoc *psoc, bool *val)
 {
 	return QDF_STATUS_SUCCESS;
 }
+
+static inline bool
+wlan_twt_get_pmo_allowed(struct wlan_objmgr_psoc *psoc)
+{
+	return true;
+}
 #endif
 
 #endif /* End of _WLAN_TWT_CFG_H */

components/umac/twt/dispatcher/inc/wlan_twt_ucfg_ext_api.h

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2022 Qualcomm Innovation Center, Inc. All rights reserved.
+ * Copyright (c) 2022-2023 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
@@ -313,6 +313,13 @@ void ucfg_twt_get_work_params(struct wlan_objmgr_vdev *vdev,
  * Return: QDF_STATUS
  */
 QDF_STATUS ucfg_twt_cfg_set_responder(struct wlan_objmgr_psoc *psoc, bool val);
+/**
+ * ucfg_twt_get_pmo_allowed() - Get twt allowed
+ * @psoc: psoc handler
+ *
+ * Return: QDF_STATUS_SUCCESS
+ */
+bool ucfg_twt_get_pmo_allowed(struct wlan_objmgr_psoc *psoc);
 #else
 static inline
 QDF_STATUS ucfg_twt_psoc_open(struct wlan_objmgr_psoc *psoc)
@@ -431,5 +438,12 @@ bool ucfg_twt_cfg_is_twt_enabled(struct wlan_objmgr_psoc *psoc)
 {
 	return false;
 }
+
+static inline
+bool ucfg_twt_get_pmo_allowed(struct wlan_objmgr_psoc *psoc)
+{
+	return false;
+}
+
 #endif
 #endif

components/umac/twt/dispatcher/src/wlan_twt_ucfg_ext_api.c

@@ -241,3 +241,9 @@ void ucfg_twt_get_work_params(
 {
 	return wlan_twt_get_work_params(vdev, params, next_action);
 }
+
+bool ucfg_twt_get_pmo_allowed(struct wlan_objmgr_psoc *psoc)
+{
+	return wlan_twt_get_pmo_allowed(psoc);
+}
+

core/hdd/inc/wlan_hdd_twt.h

@@ -372,6 +372,28 @@ QDF_STATUS hdd_get_twt_requestor(struct wlan_objmgr_psoc *psoc, bool *val);
  * Return: QDF_STATUS
  */
 QDF_STATUS hdd_get_twt_responder(struct wlan_objmgr_psoc *psoc, bool *val);
+
+/**
+ * wlan_hdd_resume_pmo_twt() - resume twt worker
+ * @hdd_ctx: hdd context
+ *
+ * Return: None
+ */
+void wlan_hdd_resume_pmo_twt(struct hdd_context *hdd_ctx);
+/**
+ * wlan_hdd_suspend_pmo_twt() - suspend twt worker
+ * @hdd_ctx: hdd context
+ *
+ * Return: None
+ */
+void wlan_hdd_suspend_pmo_twt(struct hdd_context *hdd_ctx);
+/**
+ * wlan_hdd_is_twt_pmo_allowed() - check twt disabled
+ * @hdd_ctx: hdd context
+ *
+ * Return: true if twt pmo is allowed otherwise false
+ */
+bool wlan_hdd_is_twt_pmo_allowed(struct hdd_context *hdd_ctx);
 #else
 static inline void hdd_update_tgt_twt_cap(struct hdd_context *hdd_ctx,
 					  struct wma_tgt_cfg *cfg)
@@ -489,6 +511,19 @@ QDF_STATUS hdd_get_twt_responder(struct wlan_objmgr_psoc *psoc, bool *val)
 	return QDF_STATUS_E_NOSUPPORT;
 }
 
+static inline void wlan_hdd_resume_pmo_twt(struct hdd_context *hdd_ctx)
+{
+}
+
+static inline void wlan_hdd_suspend_pmo_twt(struct hdd_context *hdd_ctx)
+{
+}
+
+static inline bool wlan_hdd_is_twt_pmo_allowed(struct hdd_context *hdd_ctx)
+{
+	return true;
+}
+
 #define FEATURE_VENDOR_SUBCMD_WIFI_CONFIG_TWT
 
 #endif

core/hdd/src/wlan_hdd_power.c

@@ -2355,6 +2355,7 @@ static int __wlan_hdd_cfg80211_resume_wlan(struct wiphy *wiphy)
 	}
 
 	ucfg_pmo_notify_system_resume(hdd_ctx->psoc);
+	wlan_hdd_resume_pmo_twt(hdd_ctx);
 
 	qdf_mtrace(QDF_MODULE_ID_HDD, QDF_MODULE_ID_HDD,
 		   TRACE_CODE_HDD_CFG80211_RESUME_WLAN,
@@ -2626,6 +2627,8 @@ static int __wlan_hdd_cfg80211_suspend_wlan(struct wiphy *wiphy,
 		return -EAGAIN;
 	}
 
+	wlan_hdd_suspend_pmo_twt(hdd_ctx);
+
 	/*
 	 * Suspend IPA early before proceeding to suspend other entities like
 	 * firmware to avoid any race conditions.

core/hdd/src/wlan_hdd_twt.c

@@ -82,7 +82,8 @@ QDF_STATUS hdd_send_twt_responder_enable_cmd(struct hdd_context *hdd_ctx)
 
 void wlan_twt_concurrency_update(struct hdd_context *hdd_ctx)
 {
-	qdf_sched_work(0, &hdd_ctx->twt_en_dis_work);
+	if (wlan_hdd_is_twt_pmo_allowed(hdd_ctx))
+		qdf_sched_work(0, &hdd_ctx->twt_en_dis_work);
 }
 
 void hdd_twt_update_work_handler(void *data)
@@ -5057,3 +5058,23 @@ int wlan_hdd_cfg80211_wifi_twt_config(struct wiphy *wiphy,
 	return errno;
 }
 
+void wlan_hdd_resume_pmo_twt(struct hdd_context *hdd_ctx)
+{
+	wlan_twt_concurrency_update(hdd_ctx);
+}
+
+void wlan_hdd_suspend_pmo_twt(struct hdd_context *hdd_ctx)
+{
+	qdf_flush_work(&hdd_ctx->twt_en_dis_work);
+}
+
+bool wlan_hdd_is_twt_pmo_allowed(struct hdd_context *hdd_ctx)
+{
+	bool twt_pmo_allowed = false;
+
+	twt_pmo_allowed = ucfg_twt_get_pmo_allowed(hdd_ctx->psoc);
+	hdd_debug("twt_disabled_allowed %d ", twt_pmo_allowed);
+
+	return twt_pmo_allowed;
+}
+