diff mbox series

ath10k: Disable 4addr source port learning in 10.4 FW by default

Message ID 20181109061121.4184-1-murugana@codeaurora.org (mailing list archive)
State New, archived
Headers show
Series ath10k: Disable 4addr source port learning in 10.4 FW by default | expand

Commit Message

Sathishkumar Muruganandam Nov. 9, 2018, 6:11 a.m. UTC 
Currently in 10.4 FW, all the received 4addr frames are processed for 
source port learning which is enabled by default. This learning can't 
be disabled by default in FW since it breaks backward compatibility.

Since ath10k uses mac80211 based 4addr mode, source port learning done 
in 10.4 FW is redundant and also causes issues when 3addr frames are
transmitted/received for a 4addr station.

One such visible functional impact is when GTK rekey frame from
hostapd based AP to 4addr STA is dropped in AP's 10.4 FW. This is since
GTK rekey EAPOL frame is 3addr frame on AP interface and STA enabled
with 4addr is already allowed for receiving 3addr EAPOL frames.

Source port learning implementation in 10.4 FW drops this 3addr GTK rekey
frame in AP destinated for 4addr STA causing disassociation and
re-association for every GTK rekey session. GTK rekey issue is not seen
when learning is disabled in FW.

To prevent such issues without breaking backward compatibility, FW
advertises new service bit making the source port learning configurable and
this learning is being currently disabled during ath10k vdev creation.

* Tested HW: QCA9984
* Tested FW: 10.4-3.6.0.1-00003

Signed-off-by: Sathishkumar Muruganandam <murugana@codeaurora.org>
---
 drivers/net/wireless/ath/ath10k/mac.c | 11 +++++++++++
 drivers/net/wireless/ath/ath10k/wmi.c |  2 ++
 drivers/net/wireless/ath/ath10k/wmi.h | 21 ++++++++++++++++++++-
 3 files changed, 33 insertions(+), 1 deletion(-)

Comments

Sebastian Gottschall Nov. 9, 2018, 6:43 a.m. UTC | #1 
thank you very much. waiting for this fix since a long time. i also 
reported this issue years ago with no response

Sebastian

Am 09.11.2018 um 07:11 schrieb Sathishkumar Muruganandam:
> One such visible functional impact is when GTK rekey frame from
Sebastian Gottschall Nov. 10, 2018, 2:07 p.m. UTC | #2 
after tested 2 WDS devices (one WDS AP, one WDS STA) with 9984 (netgear 
r7800) i found out that the issue persists. gtk rekeying fails, so 
disconnect and reconnect after 60 minutes.
but the issue is not present with other classic atheros 802.11n WDS STA 
devices. so there is still a issue with 9984 WDS STA

Sebastian

Am 09.11.2018 um 07:11 schrieb Sathishkumar Muruganandam:
> Currently in 10.4 FW, all the received 4addr frames are processed for
> source port learning which is enabled by default. This learning can't
> be disabled by default in FW since it breaks backward compatibility.
>
> Since ath10k uses mac80211 based 4addr mode, source port learning done
> in 10.4 FW is redundant and also causes issues when 3addr frames are
> transmitted/received for a 4addr station.
>
> One such visible functional impact is when GTK rekey frame from
> hostapd based AP to 4addr STA is dropped in AP's 10.4 FW. This is since
> GTK rekey EAPOL frame is 3addr frame on AP interface and STA enabled
> with 4addr is already allowed for receiving 3addr EAPOL frames.
>
> Source port learning implementation in 10.4 FW drops this 3addr GTK rekey
> frame in AP destinated for 4addr STA causing disassociation and
> re-association for every GTK rekey session. GTK rekey issue is not seen
> when learning is disabled in FW.
>
> To prevent such issues without breaking backward compatibility, FW
> advertises new service bit making the source port learning configurable and
> this learning is being currently disabled during ath10k vdev creation.
>
> * Tested HW: QCA9984
> * Tested FW: 10.4-3.6.0.1-00003
>
> Signed-off-by: Sathishkumar Muruganandam <murugana@codeaurora.org>
> ---
>   drivers/net/wireless/ath/ath10k/mac.c | 11 +++++++++++
>   drivers/net/wireless/ath/ath10k/wmi.c |  2 ++
>   drivers/net/wireless/ath/ath10k/wmi.h | 21 ++++++++++++++++++++-
>   3 files changed, 33 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c
> index c5130fa264eb..aac4d842d504 100644
> --- a/drivers/net/wireless/ath/ath10k/mac.c
> +++ b/drivers/net/wireless/ath/ath10k/mac.c
> @@ -5154,6 +5154,17 @@ static int ath10k_add_interface(struct ieee80211_hw *hw,
>   		goto err;
>   	}
>   
> +	if (test_bit(WMI_SERVICE_VDEV_DISABLE_4_ADDR_SRC_LRN_SUPPORT,
> +		     ar->wmi.svc_map)) {
> +		vdev_param = ar->wmi.vdev_param->disable_4addr_src_lrn;
> +		ret = ath10k_wmi_vdev_set_param(ar, arvif->vdev_id, vdev_param,
> +						WMI_VDEV_DISABLE_4_ADDR_SRC_LRN);
> +		if (ret && ret != -EOPNOTSUPP) {
> +			ath10k_warn(ar, "failed to disable 4addr src lrn vdev %i: %d\n",
> +				    arvif->vdev_id, ret);
> +		}
> +	}
> +
>   	ar->free_vdev_map &= ~(1LL << arvif->vdev_id);
>   	spin_lock_bh(&ar->data_lock);
>   	list_add(&arvif->list, &ar->arvifs);
> diff --git a/drivers/net/wireless/ath/ath10k/wmi.c b/drivers/net/wireless/ath/ath10k/wmi.c
> index 659513bf4ddc..8757ae297084 100644
> --- a/drivers/net/wireless/ath/ath10k/wmi.c
> +++ b/drivers/net/wireless/ath/ath10k/wmi.c
> @@ -825,6 +825,7 @@ static struct wmi_vdev_param_map wmi_vdev_param_map = {
>   	.meru_vc = WMI_VDEV_PARAM_UNSUPPORTED,
>   	.rx_decap_type = WMI_VDEV_PARAM_UNSUPPORTED,
>   	.bw_nss_ratemask = WMI_VDEV_PARAM_UNSUPPORTED,
> +	.disable_4addr_src_lrn = WMI_VDEV_PARAM_UNSUPPORTED,
>   };
>   
>   /* 10.X WMI VDEV param map */
> @@ -900,6 +901,7 @@ static struct wmi_vdev_param_map wmi_10x_vdev_param_map = {
>   	.meru_vc = WMI_VDEV_PARAM_UNSUPPORTED,
>   	.rx_decap_type = WMI_VDEV_PARAM_UNSUPPORTED,
>   	.bw_nss_ratemask = WMI_VDEV_PARAM_UNSUPPORTED,
> +	.disable_4addr_src_lrn = WMI_VDEV_PARAM_UNSUPPORTED,
>   };
>   
>   static struct wmi_vdev_param_map wmi_10_2_4_vdev_param_map = {
> diff --git a/drivers/net/wireless/ath/ath10k/wmi.h b/drivers/net/wireless/ath/ath10k/wmi.h
> index 58e33ab9e0e9..3e17be938420 100644
> --- a/drivers/net/wireless/ath/ath10k/wmi.h
> +++ b/drivers/net/wireless/ath/ath10k/wmi.h
> @@ -205,6 +205,7 @@ enum wmi_service {
>   	WMI_SERVICE_SPOOF_MAC_SUPPORT,
>   	WMI_SERVICE_TX_DATA_ACK_RSSI,
>   	WMI_SERVICE_VDEV_DIFFERENT_BEACON_INTERVAL_SUPPORT,
> +	WMI_SERVICE_VDEV_DISABLE_4_ADDR_SRC_LRN_SUPPORT,
>   
>   	/* keep last */
>   	WMI_SERVICE_MAX,
> @@ -359,6 +360,9 @@ enum wmi_10_4_service {
>   	WMI_10_4_SERVICE_PEER_TID_CONFIGS_SUPPORT,
>   	WMI_10_4_SERVICE_VDEV_BCN_RATE_CONTROL,
>   	WMI_10_4_SERVICE_VDEV_DIFFERENT_BEACON_INTERVAL_SUPPORT,
> +	WMI_10_4_SERVICE_HTT_ASSERT_TRIGGER_SUPPORT,
> +	WMI_10_4_SERVICE_VDEV_FILTER_NEIGHBOR_RX_PACKETS,
> +	WMI_10_4_SERVICE_VDEV_DISABLE_4_ADDR_SRC_LRN_SUPPORT,
>   };
>   
>   static inline char *wmi_service_name(int service_id)
> @@ -786,6 +790,8 @@ static inline void wmi_10_4_svc_map(const __le32 *in, unsigned long *out,
>   	       WMI_SERVICE_TX_DATA_ACK_RSSI, len);
>   	SVCMAP(WMI_10_4_SERVICE_VDEV_DIFFERENT_BEACON_INTERVAL_SUPPORT,
>   	       WMI_SERVICE_VDEV_DIFFERENT_BEACON_INTERVAL_SUPPORT, len);
> +	SVCMAP(WMI_10_4_SERVICE_VDEV_DISABLE_4_ADDR_SRC_LRN_SUPPORT,
> +	       WMI_SERVICE_VDEV_DISABLE_4_ADDR_SRC_LRN_SUPPORT, len);
>   }
>   
>   #undef SVCMAP
> @@ -5065,6 +5071,7 @@ struct wmi_vdev_param_map {
>   	u32 bw_nss_ratemask;
>   	u32 inc_tsf;
>   	u32 dec_tsf;
> +	u32 disable_4addr_src_lrn;
>   };
>   
>   #define WMI_VDEV_PARAM_UNSUPPORTED 0
> @@ -5404,7 +5411,19 @@ enum wmi_10_4_vdev_param {
>   	WMI_10_4_VDEV_PARAM_ATF_SSID_SCHED_POLICY,
>   	WMI_10_4_VDEV_PARAM_DISABLE_DYN_BW_RTS,
>   	WMI_10_4_VDEV_PARAM_TSF_DECREMENT,
> -};
> +	WMI_10_4_VDEV_PARAM_SELFGEN_FIXED_RATE,
> +	WMI_10_4_VDEV_PARAM_AMPDU_SUBFRAME_SIZE_PER_AC,
> +	WMI_10_4_VDEV_PARAM_NSS_VHT160,
> +	WMI_10_4_VDEV_PARAM_NSS_VHT80_80,
> +	WMI_10_4_VDEV_PARAM_AMSDU_SUBFRAME_SIZE_PER_AC,
> +	WMI_10_4_VDEV_PARAM_DISABLE_CABQ,
> +	WMI_10_4_VDEV_PARAM_SIFS_TRIGGER_RATE,
> +	WMI_10_4_VDEV_PARAM_TX_POWER,
> +	WMI_10_4_VDEV_PARAM_ENABLE_DISABLE_RTT_RESPONDER_ROLE,
> +	WMI_10_4_VDEV_PARAM_DISABLE_4_ADDR_SRC_LRN,
> +};
> +
> +#define WMI_VDEV_DISABLE_4_ADDR_SRC_LRN 1
>   
>   #define WMI_VDEV_PARAM_TXBF_SU_TX_BFEE BIT(0)
>   #define WMI_VDEV_PARAM_TXBF_MU_TX_BFEE BIT(1)
Sathishkumar Muruganandam Nov. 11, 2018, 7:13 a.m. UTC | #3 
Hi Sebastian,

> after tested 2 WDS devices (one WDS AP, one WDS STA) with 9984 (netgear
> r7800) i found out that the issue persists. gtk rekeying fails, so disconnect and
> reconnect after 60 minutes.

Are you using the latest FW 10.4-3.6.0.1-00003 for 9984 ( mainly on the AP side ) ?

(https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git/commit/?id=a00e61b9203186561dc90de1ff7ed4d5f5e920d5)

Thanks,
Sathishkumar
Sebastian Gottschall Nov. 12, 2018, 2:44 a.m. UTC | #4 
hint for a possible fix


disable_4addr_src_lrn is not set in 10_4 vdev parameter map. so this 
feature cannot work


--- wmi.c       (Revision 4214)
+++ wmi.c       (Arbeitskopie)
@@ -972,6 +972,7 @@
         .meru_vc = WMI_VDEV_PARAM_UNSUPPORTED,
         .rx_decap_type = WMI_VDEV_PARAM_UNSUPPORTED,
         .bw_nss_ratemask = WMI_VDEV_PARAM_UNSUPPORTED,
+       .disable_4addr_src_lrn = WMI_VDEV_PARAM_UNSUPPORTED,
  };

  static struct wmi_vdev_param_map wmi_10_4_vdev_param_map = {
@@ -1049,6 +1050,7 @@
         .bw_nss_ratemask = WMI_10_4_VDEV_PARAM_BW_NSS_RATEMASK,
         .inc_tsf = WMI_10_4_VDEV_PARAM_TSF_INCREMENT,
         .dec_tsf = WMI_10_4_VDEV_PARAM_TSF_DECREMENT,
+       .disable_4addr_src_lrn = WMI_10_4_VDEV_PARAM_DISABLE_4_ADDR_SRC_LRN,
  };

Am 11.11.2018 um 08:13 schrieb murugana@codeaurora.org:
> Hi Sebastian,
>
>> after tested 2 WDS devices (one WDS AP, one WDS STA) with 9984 (netgear
>> r7800) i found out that the issue persists. gtk rekeying fails, so disconnect and
>> reconnect after 60 minutes.
> Are you using the latest FW 10.4-3.6.0.1-00003 for 9984 ( mainly on the AP side ) ?
>
> (https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git/commit/?id=a00e61b9203186561dc90de1ff7ed4d5f5e920d5)
>
> Thanks,
> Sathishkumar
>
>
>
Sebastian Gottschall Nov. 12, 2018, 4:10 a.m. UTC | #5 
i tested my proposal now and it fixes the issue. the submitted patch was 
simply broken at that small point

Am 12.11.2018 um 03:44 schrieb Sebastian Gottschall:
> hint for a possible fix
>
>
> disable_4addr_src_lrn is not set in 10_4 vdev parameter map. so this 
> feature cannot work
>
>
> --- wmi.c       (Revision 4214)
> +++ wmi.c       (Arbeitskopie)
> @@ -972,6 +972,7 @@
>         .meru_vc = WMI_VDEV_PARAM_UNSUPPORTED,
>         .rx_decap_type = WMI_VDEV_PARAM_UNSUPPORTED,
>         .bw_nss_ratemask = WMI_VDEV_PARAM_UNSUPPORTED,
> +       .disable_4addr_src_lrn = WMI_VDEV_PARAM_UNSUPPORTED,
>  };
>
>  static struct wmi_vdev_param_map wmi_10_4_vdev_param_map = {
> @@ -1049,6 +1050,7 @@
>         .bw_nss_ratemask = WMI_10_4_VDEV_PARAM_BW_NSS_RATEMASK,
>         .inc_tsf = WMI_10_4_VDEV_PARAM_TSF_INCREMENT,
>         .dec_tsf = WMI_10_4_VDEV_PARAM_TSF_DECREMENT,
> +       .disable_4addr_src_lrn = 
> WMI_10_4_VDEV_PARAM_DISABLE_4_ADDR_SRC_LRN,
>  };
>
> Am 11.11.2018 um 08:13 schrieb murugana@codeaurora.org:
>> Hi Sebastian,
>>
>>> after tested 2 WDS devices (one WDS AP, one WDS STA) with 9984 (netgear
>>> r7800) i found out that the issue persists. gtk rekeying fails, so 
>>> disconnect and
>>> reconnect after 60 minutes.
>> Are you using the latest FW 10.4-3.6.0.1-00003 for 9984 ( mainly on 
>> the AP side ) ?
>>
>> (https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git/commit/?id=a00e61b9203186561dc90de1ff7ed4d5f5e920d5) 
>>
>>
>> Thanks,
>> Sathishkumar
>>
>>
>>
Sathishkumar Muruganandam Nov. 12, 2018, 5:34 a.m. UTC | #6 
On 2018-11-12 09:40, Sebastian Gottschall wrote:
> i tested my proposal now and it fixes the issue. the submitted patch
> was simply broken at that small point
> 

You're right. Thanks for your finding !

Due to some reason, my patch formatting missed the vdev param
assignments to 10.4 and 10.2.4 (unsupported).

Sorry for the confusion, let me resend the patch.
Sebastian Gottschall Nov. 12, 2018, 7:33 a.m. UTC | #7 
no problem. glad to help you.

Sebastian

Am 12.11.2018 um 06:34 schrieb Sathishkumar Muruganandam:
> On 2018-11-12 09:40, Sebastian Gottschall wrote:
>> i tested my proposal now and it fixes the issue. the submitted patch
>> was simply broken at that small point
>>
>
> You're right. Thanks for your finding !
>
> Due to some reason, my patch formatting missed the vdev param
> assignments to 10.4 and 10.2.4 (unsupported).
>
> Sorry for the confusion, let me resend the patch.
>
Sebastian Gottschall Nov. 13, 2018, 1:34 p.m. UTC | #8 
is it possible to supply a firmware for 99X0 based devices which suffer 
from the same issue? i know the firmware codebase is the same for these 
chipsets and there are alot of embedded
devices around which this older one. unfortunatly this device suffers 
from the same issue

Sebastian

Am 12.11.2018 um 08:33 schrieb Sebastian Gottschall:
> no problem. glad to help you.
>
> Sebastian
>
> Am 12.11.2018 um 06:34 schrieb Sathishkumar Muruganandam:
>> On 2018-11-12 09:40, Sebastian Gottschall wrote:
>>> i tested my proposal now and it fixes the issue. the submitted patch
>>> was simply broken at that small point
>>>
>>
>> You're right. Thanks for your finding !
>>
>> Due to some reason, my patch formatting missed the vdev param
>> assignments to 10.4 and 10.2.4 (unsupported).
>>
>> Sorry for the confusion, let me resend the patch.
>>
>
diff mbox series

Patch

diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c
index c5130fa264eb..aac4d842d504 100644
--- a/drivers/net/wireless/ath/ath10k/mac.c
+++ b/drivers/net/wireless/ath/ath10k/mac.c
@@ -5154,6 +5154,17 @@  static int ath10k_add_interface(struct ieee80211_hw *hw,
 		goto err;
 	}
 
+	if (test_bit(WMI_SERVICE_VDEV_DISABLE_4_ADDR_SRC_LRN_SUPPORT,
+		     ar->wmi.svc_map)) {
+		vdev_param = ar->wmi.vdev_param->disable_4addr_src_lrn;
+		ret = ath10k_wmi_vdev_set_param(ar, arvif->vdev_id, vdev_param,
+						WMI_VDEV_DISABLE_4_ADDR_SRC_LRN);
+		if (ret && ret != -EOPNOTSUPP) {
+			ath10k_warn(ar, "failed to disable 4addr src lrn vdev %i: %d\n",
+				    arvif->vdev_id, ret);
+		}
+	}
+
 	ar->free_vdev_map &= ~(1LL << arvif->vdev_id);
 	spin_lock_bh(&ar->data_lock);
 	list_add(&arvif->list, &ar->arvifs);
diff --git a/drivers/net/wireless/ath/ath10k/wmi.c b/drivers/net/wireless/ath/ath10k/wmi.c
index 659513bf4ddc..8757ae297084 100644
--- a/drivers/net/wireless/ath/ath10k/wmi.c
+++ b/drivers/net/wireless/ath/ath10k/wmi.c
@@ -825,6 +825,7 @@  static struct wmi_vdev_param_map wmi_vdev_param_map = {
 	.meru_vc = WMI_VDEV_PARAM_UNSUPPORTED,
 	.rx_decap_type = WMI_VDEV_PARAM_UNSUPPORTED,
 	.bw_nss_ratemask = WMI_VDEV_PARAM_UNSUPPORTED,
+	.disable_4addr_src_lrn = WMI_VDEV_PARAM_UNSUPPORTED,
 };
 
 /* 10.X WMI VDEV param map */
@@ -900,6 +901,7 @@  static struct wmi_vdev_param_map wmi_10x_vdev_param_map = {
 	.meru_vc = WMI_VDEV_PARAM_UNSUPPORTED,
 	.rx_decap_type = WMI_VDEV_PARAM_UNSUPPORTED,
 	.bw_nss_ratemask = WMI_VDEV_PARAM_UNSUPPORTED,
+	.disable_4addr_src_lrn = WMI_VDEV_PARAM_UNSUPPORTED,
 };
 
 static struct wmi_vdev_param_map wmi_10_2_4_vdev_param_map = {
diff --git a/drivers/net/wireless/ath/ath10k/wmi.h b/drivers/net/wireless/ath/ath10k/wmi.h
index 58e33ab9e0e9..3e17be938420 100644
--- a/drivers/net/wireless/ath/ath10k/wmi.h
+++ b/drivers/net/wireless/ath/ath10k/wmi.h
@@ -205,6 +205,7 @@  enum wmi_service {
 	WMI_SERVICE_SPOOF_MAC_SUPPORT,
 	WMI_SERVICE_TX_DATA_ACK_RSSI,
 	WMI_SERVICE_VDEV_DIFFERENT_BEACON_INTERVAL_SUPPORT,
+	WMI_SERVICE_VDEV_DISABLE_4_ADDR_SRC_LRN_SUPPORT,
 
 	/* keep last */
 	WMI_SERVICE_MAX,
@@ -359,6 +360,9 @@  enum wmi_10_4_service {
 	WMI_10_4_SERVICE_PEER_TID_CONFIGS_SUPPORT,
 	WMI_10_4_SERVICE_VDEV_BCN_RATE_CONTROL,
 	WMI_10_4_SERVICE_VDEV_DIFFERENT_BEACON_INTERVAL_SUPPORT,
+	WMI_10_4_SERVICE_HTT_ASSERT_TRIGGER_SUPPORT,
+	WMI_10_4_SERVICE_VDEV_FILTER_NEIGHBOR_RX_PACKETS,
+	WMI_10_4_SERVICE_VDEV_DISABLE_4_ADDR_SRC_LRN_SUPPORT,
 };
 
 static inline char *wmi_service_name(int service_id)
@@ -786,6 +790,8 @@  static inline void wmi_10_4_svc_map(const __le32 *in, unsigned long *out,
 	       WMI_SERVICE_TX_DATA_ACK_RSSI, len);
 	SVCMAP(WMI_10_4_SERVICE_VDEV_DIFFERENT_BEACON_INTERVAL_SUPPORT,
 	       WMI_SERVICE_VDEV_DIFFERENT_BEACON_INTERVAL_SUPPORT, len);
+	SVCMAP(WMI_10_4_SERVICE_VDEV_DISABLE_4_ADDR_SRC_LRN_SUPPORT,
+	       WMI_SERVICE_VDEV_DISABLE_4_ADDR_SRC_LRN_SUPPORT, len);
 }
 
 #undef SVCMAP
@@ -5065,6 +5071,7 @@  struct wmi_vdev_param_map {
 	u32 bw_nss_ratemask;
 	u32 inc_tsf;
 	u32 dec_tsf;
+	u32 disable_4addr_src_lrn;
 };
 
 #define WMI_VDEV_PARAM_UNSUPPORTED 0
@@ -5404,7 +5411,19 @@  enum wmi_10_4_vdev_param {
 	WMI_10_4_VDEV_PARAM_ATF_SSID_SCHED_POLICY,
 	WMI_10_4_VDEV_PARAM_DISABLE_DYN_BW_RTS,
 	WMI_10_4_VDEV_PARAM_TSF_DECREMENT,
-};
+	WMI_10_4_VDEV_PARAM_SELFGEN_FIXED_RATE,
+	WMI_10_4_VDEV_PARAM_AMPDU_SUBFRAME_SIZE_PER_AC,
+	WMI_10_4_VDEV_PARAM_NSS_VHT160,
+	WMI_10_4_VDEV_PARAM_NSS_VHT80_80,
+	WMI_10_4_VDEV_PARAM_AMSDU_SUBFRAME_SIZE_PER_AC,
+	WMI_10_4_VDEV_PARAM_DISABLE_CABQ,
+	WMI_10_4_VDEV_PARAM_SIFS_TRIGGER_RATE,
+	WMI_10_4_VDEV_PARAM_TX_POWER,
+	WMI_10_4_VDEV_PARAM_ENABLE_DISABLE_RTT_RESPONDER_ROLE,
+	WMI_10_4_VDEV_PARAM_DISABLE_4_ADDR_SRC_LRN,
+};
+
+#define WMI_VDEV_DISABLE_4_ADDR_SRC_LRN 1
 
 #define WMI_VDEV_PARAM_TXBF_SU_TX_BFEE BIT(0)
 #define WMI_VDEV_PARAM_TXBF_MU_TX_BFEE BIT(1)