| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Thu, 3 Mar 2022 08:56:04 -0800
From: Jeff Johnson <quic_jjohnson@...cinc.com>
To: Youghandhar Chintala <youghand@...eaurora.org>,
<ath10k@...ts.infradead.org>
CC: <linux-wireless@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<pillair@...eaurora.org>, <dianders@...omium.org>,
<kuabhs@...omium.org>, <briannorris@...omium.org>,
<mpubbise@...eaurora.org>
Subject: Re: [PATCH v4 2/2] ath10k:trigger sta disconnect on hardware restart
On 3/3/2022 3:55 AM, Youghandhar Chintala wrote:
nit: in subject add space after ath10k:
> Currently after the hardware restart triggered from the driver,
> the station interface connection remains intact, since a disconnect
> trigger is not sent to userspace. This can lead to a problem in
> targets where the wifi mac sequence is added by the firmware.
>
> After the target restart, during subsytem recovery, the target
nit: s/subsytem/subsystem/
> restarts its wifi mac sequence number. Hence AP to which our device
> is connected will receive frames with a wifi mac sequence number jump
> to the past, thereby resulting in the AP dropping all these frames,
> until the frame arrives with a wifi mac sequence number which AP was
> expecting.
>
> To avoid such frame drops, its better to trigger a station disconnect
> upon target hardware restart which can be done with API
> ieee80211_reconfig_disconnect exposed to mac80211.
>
> The other targets are not affected by this change, since the hardware
> params flag is not set.
>
> Tested-on: WCN3990 hw1.0 SNOC WLAN.HL.3.1-01040-QCAHLSWMTPLZ-1
> QCA6174 hw3.2 PCI WLAN.RM.4.4.1-00110-QCARMSWP-1
> QCA6174 hw3.2 SDIO WLAN.RMH.4.4.1-00048
<https://wireless.wiki.kernel.org/en/users/drivers/ath10k/submittingpatches#tested-on_tag>
there should be a Tested-on: tag for each line/target
>
> Signed-off-by: Youghandhar Chintala <youghand@...eaurora.org>
> ---
> drivers/net/wireless/ath/ath10k/core.c | 25 +++++++++++++++++++++++++
> drivers/net/wireless/ath/ath10k/hw.h | 2 ++
> 2 files changed, 27 insertions(+)
>
> diff --git a/drivers/net/wireless/ath/ath10k/core.c b/drivers/net/wireless/ath/ath10k/core.c
> index 9e1f483e1362..2092bfd02cd1 100644
> --- a/drivers/net/wireless/ath/ath10k/core.c
> +++ b/drivers/net/wireless/ath/ath10k/core.c
> @@ -94,6 +94,7 @@ static const struct ath10k_hw_params ath10k_hw_params_list[] = {
> .credit_size_workaround = false,
> .tx_stats_over_pktlog = true,
> .dynamic_sar_support = false,
> + .hw_restart_disconnect = false,
> },
> {
> .id = QCA988X_HW_2_0_VERSION,
> @@ -131,6 +132,7 @@ static const struct ath10k_hw_params ath10k_hw_params_list[] = {
> .credit_size_workaround = false,
> .tx_stats_over_pktlog = true,
> .dynamic_sar_support = false,
> + .hw_restart_disconnect = false,
> },
> {
> .id = QCA9887_HW_1_0_VERSION,
> @@ -169,6 +171,7 @@ static const struct ath10k_hw_params ath10k_hw_params_list[] = {
> .credit_size_workaround = false,
> .tx_stats_over_pktlog = false,
> .dynamic_sar_support = false,
> + .hw_restart_disconnect = false,
> },
> {
> .id = QCA6174_HW_3_2_VERSION,
> @@ -202,6 +205,7 @@ static const struct ath10k_hw_params ath10k_hw_params_list[] = {
> .bmi_large_size_download = true,
> .supports_peer_stats_info = true,
> .dynamic_sar_support = true,
> + .hw_restart_disconnect = false,
> },
> {
> .id = QCA6174_HW_2_1_VERSION,
> @@ -239,6 +243,7 @@ static const struct ath10k_hw_params ath10k_hw_params_list[] = {
> .credit_size_workaround = false,
> .tx_stats_over_pktlog = false,
> .dynamic_sar_support = false,
> + .hw_restart_disconnect = false,
> },
> {
> .id = QCA6174_HW_2_1_VERSION,
> @@ -276,6 +281,7 @@ static const struct ath10k_hw_params ath10k_hw_params_list[] = {
> .credit_size_workaround = false,
> .tx_stats_over_pktlog = false,
> .dynamic_sar_support = false,
> + .hw_restart_disconnect = false,
> },
> {
> .id = QCA6174_HW_3_0_VERSION,
> @@ -313,6 +319,7 @@ static const struct ath10k_hw_params ath10k_hw_params_list[] = {
> .credit_size_workaround = false,
> .tx_stats_over_pktlog = false,
> .dynamic_sar_support = false,
> + .hw_restart_disconnect = false,
> },
> {
> .id = QCA6174_HW_3_2_VERSION,
> @@ -354,6 +361,7 @@ static const struct ath10k_hw_params ath10k_hw_params_list[] = {
> .tx_stats_over_pktlog = false,
> .supports_peer_stats_info = true,
> .dynamic_sar_support = true,
> + .hw_restart_disconnect = false,
> },
> {
> .id = QCA99X0_HW_2_0_DEV_VERSION,
> @@ -397,6 +405,7 @@ static const struct ath10k_hw_params ath10k_hw_params_list[] = {
> .credit_size_workaround = false,
> .tx_stats_over_pktlog = false,
> .dynamic_sar_support = false,
> + .hw_restart_disconnect = false,
> },
> {
> .id = QCA9984_HW_1_0_DEV_VERSION,
> @@ -447,6 +456,7 @@ static const struct ath10k_hw_params ath10k_hw_params_list[] = {
> .credit_size_workaround = false,
> .tx_stats_over_pktlog = false,
> .dynamic_sar_support = false,
> + .hw_restart_disconnect = false,
> },
> {
> .id = QCA9888_HW_2_0_DEV_VERSION,
> @@ -494,6 +504,7 @@ static const struct ath10k_hw_params ath10k_hw_params_list[] = {
> .credit_size_workaround = false,
> .tx_stats_over_pktlog = false,
> .dynamic_sar_support = false,
> + .hw_restart_disconnect = false,
> },
> {
> .id = QCA9377_HW_1_0_DEV_VERSION,
> @@ -531,6 +542,7 @@ static const struct ath10k_hw_params ath10k_hw_params_list[] = {
> .credit_size_workaround = false,
> .tx_stats_over_pktlog = false,
> .dynamic_sar_support = false,
> + .hw_restart_disconnect = false,
> },
> {
> .id = QCA9377_HW_1_1_DEV_VERSION,
> @@ -570,6 +582,7 @@ static const struct ath10k_hw_params ath10k_hw_params_list[] = {
> .credit_size_workaround = false,
> .tx_stats_over_pktlog = false,
> .dynamic_sar_support = false,
> + .hw_restart_disconnect = false,
> },
> {
> .id = QCA9377_HW_1_1_DEV_VERSION,
> @@ -600,6 +613,7 @@ static const struct ath10k_hw_params ath10k_hw_params_list[] = {
> .uart_pin_workaround = true,
> .credit_size_workaround = true,
> .dynamic_sar_support = false,
> + .hw_restart_disconnect = false,
> },
> {
> .id = QCA4019_HW_1_0_DEV_VERSION,
> @@ -644,6 +658,7 @@ static const struct ath10k_hw_params ath10k_hw_params_list[] = {
> .credit_size_workaround = false,
> .tx_stats_over_pktlog = false,
> .dynamic_sar_support = false,
> + .hw_restart_disconnect = false,
> },
> {
> .id = WCN3990_HW_1_0_DEV_VERSION,
> @@ -674,6 +689,7 @@ static const struct ath10k_hw_params ath10k_hw_params_list[] = {
> .credit_size_workaround = false,
> .tx_stats_over_pktlog = false,
> .dynamic_sar_support = true,
> + .hw_restart_disconnect = true,
> },
> };
>
> @@ -2442,6 +2458,7 @@ EXPORT_SYMBOL(ath10k_core_napi_sync_disable);
> static void ath10k_core_restart(struct work_struct *work)
> {
> struct ath10k *ar = container_of(work, struct ath10k, restart_work);
> + struct ath10k_vif *arvif;
> int ret;
>
> set_bit(ATH10K_FLAG_CRASH_FLUSH, &ar->dev_flags);
> @@ -2480,6 +2497,14 @@ static void ath10k_core_restart(struct work_struct *work)
> ar->state = ATH10K_STATE_RESTARTING;
> ath10k_halt(ar);
> ath10k_scan_finish(ar);
> + if (ar->hw_params.hw_restart_disconnect) {
> + list_for_each_entry(arvif, &ar->arvifs, list) {
> + if (arvif->is_up &&
> + arvif->vdev_type == WMI_VDEV_TYPE_STA)
> + ieee80211_hw_restart_disconnect(arvif->vif);
> + }
> + }
> +
> ieee80211_restart_hw(ar->hw);
> break;
> case ATH10K_STATE_OFF:
> diff --git a/drivers/net/wireless/ath/ath10k/hw.h b/drivers/net/wireless/ath/ath10k/hw.h
> index 5215a6816d71..93acf0dd580a 100644
> --- a/drivers/net/wireless/ath/ath10k/hw.h
> +++ b/drivers/net/wireless/ath/ath10k/hw.h
> @@ -633,6 +633,8 @@ struct ath10k_hw_params {
> bool supports_peer_stats_info;
>
> bool dynamic_sar_support;
> +
> + bool hw_restart_disconnect;
> };
>
> struct htt_resp;
Powered by blists - more mailing lists