* [PATCH 1/2] ath10k: fix low TX rates when IBSS and HT @ 2014-12-16 8:52 ` Janusz Dziedzic 0 siblings, 0 replies; 24+ messages in thread From: Janusz Dziedzic @ 2014-12-16 8:52 UTC (permalink / raw) To: ath10k; +Cc: Janusz Dziedzic, linux-wireless This fix TX problem when IBSS used in HT mode. Before we used 6Mbps all the time for TX direction. Reported-by: Yeoh Chun-Yeow <yeohchunyeow@gmail.com> Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> --- drivers/net/wireless/ath/ath10k/mac.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c index 5475f0f..c9e7995 100644 --- a/drivers/net/wireless/ath/ath10k/mac.c +++ b/drivers/net/wireless/ath/ath10k/mac.c @@ -1411,9 +1411,16 @@ static void ath10k_peer_assoc_h_qos(struct ath10k *ar, if (vif->bss_conf.qos) arg->peer_flags |= WMI_PEER_QOS; break; + case WMI_VDEV_TYPE_IBSS: + if (sta->wme) + arg->peer_flags |= WMI_PEER_QOS; + break; default: break; } + + ath10k_dbg(ar, ATH10K_DBG_MAC, "mac peer %pM qos %d\n", + sta->addr, !!(arg->peer_flags & WMI_PEER_QOS)); } static void ath10k_peer_assoc_h_phymode(struct ath10k *ar, -- 1.9.1 _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 1/2] ath10k: fix low TX rates when IBSS and HT @ 2014-12-16 8:52 ` Janusz Dziedzic 0 siblings, 0 replies; 24+ messages in thread From: Janusz Dziedzic @ 2014-12-16 8:52 UTC (permalink / raw) To: ath10k; +Cc: linux-wireless, Janusz Dziedzic This fix TX problem when IBSS used in HT mode. Before we used 6Mbps all the time for TX direction. Reported-by: Yeoh Chun-Yeow <yeohchunyeow@gmail.com> Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> --- drivers/net/wireless/ath/ath10k/mac.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c index 5475f0f..c9e7995 100644 --- a/drivers/net/wireless/ath/ath10k/mac.c +++ b/drivers/net/wireless/ath/ath10k/mac.c @@ -1411,9 +1411,16 @@ static void ath10k_peer_assoc_h_qos(struct ath10k *ar, if (vif->bss_conf.qos) arg->peer_flags |= WMI_PEER_QOS; break; + case WMI_VDEV_TYPE_IBSS: + if (sta->wme) + arg->peer_flags |= WMI_PEER_QOS; + break; default: break; } + + ath10k_dbg(ar, ATH10K_DBG_MAC, "mac peer %pM qos %d\n", + sta->addr, !!(arg->peer_flags & WMI_PEER_QOS)); } static void ath10k_peer_assoc_h_phymode(struct ath10k *ar, -- 1.9.1 ^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 2/2] ath10k: send (re)assoc peer command when NSS changed 2014-12-16 8:52 ` Janusz Dziedzic @ 2014-12-16 8:53 ` Janusz Dziedzic -1 siblings, 0 replies; 24+ messages in thread From: Janusz Dziedzic @ 2014-12-16 8:53 UTC (permalink / raw) To: ath10k; +Cc: Janusz Dziedzic, linux-wireless Assoc peer command contain information about NSS. When we will get IEEE80211_RC_NSS_CHANGED we should also send (re) assoc peer command to be sure firmware will know about it and RC will work correctly. Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> --- drivers/net/wireless/ath/ath10k/mac.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c index c9e7995..05fd620 100644 --- a/drivers/net/wireless/ath/ath10k/mac.c +++ b/drivers/net/wireless/ath/ath10k/mac.c @@ -3592,8 +3592,9 @@ static void ath10k_sta_rc_update_wk(struct work_struct *wk) sta->addr, smps, err); } - if (changed & IEEE80211_RC_SUPP_RATES_CHANGED) { - ath10k_dbg(ar, ATH10K_DBG_MAC, "mac update sta %pM supp rates\n", + if (changed & IEEE80211_RC_SUPP_RATES_CHANGED || + changed & IEEE80211_RC_NSS_CHANGED) { + ath10k_dbg(ar, ATH10K_DBG_MAC, "mac update sta %pM supp rates/nss\n", sta->addr); err = ath10k_station_assoc(ar, arvif->vif, sta, true); -- 1.9.1 _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 2/2] ath10k: send (re)assoc peer command when NSS changed @ 2014-12-16 8:53 ` Janusz Dziedzic 0 siblings, 0 replies; 24+ messages in thread From: Janusz Dziedzic @ 2014-12-16 8:53 UTC (permalink / raw) To: ath10k; +Cc: linux-wireless, Janusz Dziedzic Assoc peer command contain information about NSS. When we will get IEEE80211_RC_NSS_CHANGED we should also send (re) assoc peer command to be sure firmware will know about it and RC will work correctly. Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> --- drivers/net/wireless/ath/ath10k/mac.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c index c9e7995..05fd620 100644 --- a/drivers/net/wireless/ath/ath10k/mac.c +++ b/drivers/net/wireless/ath/ath10k/mac.c @@ -3592,8 +3592,9 @@ static void ath10k_sta_rc_update_wk(struct work_struct *wk) sta->addr, smps, err); } - if (changed & IEEE80211_RC_SUPP_RATES_CHANGED) { - ath10k_dbg(ar, ATH10K_DBG_MAC, "mac update sta %pM supp rates\n", + if (changed & IEEE80211_RC_SUPP_RATES_CHANGED || + changed & IEEE80211_RC_NSS_CHANGED) { + ath10k_dbg(ar, ATH10K_DBG_MAC, "mac update sta %pM supp rates/nss\n", sta->addr); err = ath10k_station_assoc(ar, arvif->vif, sta, true); -- 1.9.1 ^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH 2/2] ath10k: send (re)assoc peer command when NSS changed 2014-12-16 8:53 ` Janusz Dziedzic @ 2014-12-17 10:32 ` Kalle Valo -1 siblings, 0 replies; 24+ messages in thread From: Kalle Valo @ 2014-12-17 10:32 UTC (permalink / raw) To: Janusz Dziedzic; +Cc: linux-wireless, ath10k Janusz Dziedzic <janusz.dziedzic@tieto.com> writes: > Assoc peer command contain information about NSS. > When we will get IEEE80211_RC_NSS_CHANGED we should > also send (re) assoc peer command to be sure firmware > will know about it and RC will work correctly. > > Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> Does this fix a user visible bug or is this something you just found from doing code review? -- Kalle Valo _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 2/2] ath10k: send (re)assoc peer command when NSS changed @ 2014-12-17 10:32 ` Kalle Valo 0 siblings, 0 replies; 24+ messages in thread From: Kalle Valo @ 2014-12-17 10:32 UTC (permalink / raw) To: Janusz Dziedzic; +Cc: ath10k, linux-wireless Janusz Dziedzic <janusz.dziedzic@tieto.com> writes: > Assoc peer command contain information about NSS. > When we will get IEEE80211_RC_NSS_CHANGED we should > also send (re) assoc peer command to be sure firmware > will know about it and RC will work correctly. > > Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> Does this fix a user visible bug or is this something you just found from doing code review? -- Kalle Valo ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 2/2] ath10k: send (re)assoc peer command when NSS changed 2014-12-17 10:32 ` Kalle Valo @ 2014-12-18 5:28 ` Janusz Dziedzic -1 siblings, 0 replies; 24+ messages in thread From: Janusz Dziedzic @ 2014-12-18 5:28 UTC (permalink / raw) To: Kalle Valo; +Cc: linux-wireless, ath10k@lists.infradead.org On 17 December 2014 at 11:32, Kalle Valo <kvalo@qca.qualcomm.com> wrote: > Janusz Dziedzic <janusz.dziedzic@tieto.com> writes: > >> Assoc peer command contain information about NSS. >> When we will get IEEE80211_RC_NSS_CHANGED we should >> also send (re) assoc peer command to be sure firmware >> will know about it and RC will work correctly. >> >> Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> > > Does this fix a user visible bug or is this something you just found > from doing code review? > In case we will get only IEEE80211_RC_NSS_CHANGED we will not update FW configuration correctly. This is not the problem when IBSS, while we update two flags in one step IEEE80211_RC_SUPP_RATES_CHANGED | IEEE80211_RC_NSS_CHANGED. We could have problems when mac80211 will send only NSS_CHANGED flag. This seems could happen when we will get WLAN_EID_OPMODE_NOTIF. I didn't test such case, but I believe some AP could send such notification. So, this fix potential TP issue. BR Janusz _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 2/2] ath10k: send (re)assoc peer command when NSS changed @ 2014-12-18 5:28 ` Janusz Dziedzic 0 siblings, 0 replies; 24+ messages in thread From: Janusz Dziedzic @ 2014-12-18 5:28 UTC (permalink / raw) To: Kalle Valo; +Cc: ath10k@lists.infradead.org, linux-wireless On 17 December 2014 at 11:32, Kalle Valo <kvalo@qca.qualcomm.com> wrote: > Janusz Dziedzic <janusz.dziedzic@tieto.com> writes: > >> Assoc peer command contain information about NSS. >> When we will get IEEE80211_RC_NSS_CHANGED we should >> also send (re) assoc peer command to be sure firmware >> will know about it and RC will work correctly. >> >> Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> > > Does this fix a user visible bug or is this something you just found > from doing code review? > In case we will get only IEEE80211_RC_NSS_CHANGED we will not update FW configuration correctly. This is not the problem when IBSS, while we update two flags in one step IEEE80211_RC_SUPP_RATES_CHANGED | IEEE80211_RC_NSS_CHANGED. We could have problems when mac80211 will send only NSS_CHANGED flag. This seems could happen when we will get WLAN_EID_OPMODE_NOTIF. I didn't test such case, but I believe some AP could send such notification. So, this fix potential TP issue. BR Janusz ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 2/2] ath10k: send (re)assoc peer command when NSS changed 2014-12-18 5:28 ` Janusz Dziedzic @ 2014-12-18 6:02 ` Kalle Valo -1 siblings, 0 replies; 24+ messages in thread From: Kalle Valo @ 2014-12-18 6:02 UTC (permalink / raw) To: Janusz Dziedzic; +Cc: linux-wireless, ath10k@lists.infradead.org Janusz Dziedzic <janusz.dziedzic@tieto.com> writes: > On 17 December 2014 at 11:32, Kalle Valo <kvalo@qca.qualcomm.com> wrote: >> Janusz Dziedzic <janusz.dziedzic@tieto.com> writes: >> >>> Assoc peer command contain information about NSS. >>> When we will get IEEE80211_RC_NSS_CHANGED we should >>> also send (re) assoc peer command to be sure firmware >>> will know about it and RC will work correctly. >>> >>> Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> >> >> Does this fix a user visible bug or is this something you just found >> from doing code review? > > In case we will get only IEEE80211_RC_NSS_CHANGED we will not update > FW configuration correctly. This is not the problem when IBSS, while > we update two flags in one step IEEE80211_RC_SUPP_RATES_CHANGED | > IEEE80211_RC_NSS_CHANGED. > > We could have problems when mac80211 will send only NSS_CHANGED flag. > This seems could happen when we will get WLAN_EID_OPMODE_NOTIF. I > didn't test such case, but I believe some AP could send such > notification. So, this fix potential TP issue. Ok, I'll add to the commit log that this was found during code review (and doesn't fix a reported bug). -- Kalle Valo _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 2/2] ath10k: send (re)assoc peer command when NSS changed @ 2014-12-18 6:02 ` Kalle Valo 0 siblings, 0 replies; 24+ messages in thread From: Kalle Valo @ 2014-12-18 6:02 UTC (permalink / raw) To: Janusz Dziedzic; +Cc: ath10k@lists.infradead.org, linux-wireless Janusz Dziedzic <janusz.dziedzic@tieto.com> writes: > On 17 December 2014 at 11:32, Kalle Valo <kvalo@qca.qualcomm.com> wrote: >> Janusz Dziedzic <janusz.dziedzic@tieto.com> writes: >> >>> Assoc peer command contain information about NSS. >>> When we will get IEEE80211_RC_NSS_CHANGED we should >>> also send (re) assoc peer command to be sure firmware >>> will know about it and RC will work correctly. >>> >>> Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> >> >> Does this fix a user visible bug or is this something you just found >> from doing code review? > > In case we will get only IEEE80211_RC_NSS_CHANGED we will not update > FW configuration correctly. This is not the problem when IBSS, while > we update two flags in one step IEEE80211_RC_SUPP_RATES_CHANGED | > IEEE80211_RC_NSS_CHANGED. > > We could have problems when mac80211 will send only NSS_CHANGED flag. > This seems could happen when we will get WLAN_EID_OPMODE_NOTIF. I > didn't test such case, but I believe some AP could send such > notification. So, this fix potential TP issue. Ok, I'll add to the commit log that this was found during code review (and doesn't fix a reported bug). -- Kalle Valo ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] ath10k: fix low TX rates when IBSS and HT 2014-12-16 8:52 ` Janusz Dziedzic @ 2014-12-16 11:10 ` Yeoh Chun-Yeow -1 siblings, 0 replies; 24+ messages in thread From: Yeoh Chun-Yeow @ 2014-12-16 11:10 UTC (permalink / raw) To: Janusz Dziedzic Cc: linux-wireless@vger.kernel.org, ath10k@lists.infradead.org Hi, Janusz I have applied the three patches and tested with firmware 999.999.0.636 but not working. Any advice what's wrong? --- ChunYeow Some of my dmesg: [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 3 value 34 [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 4 value 34 [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params [ 640.450000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 peer_addr 04:f0:21:0c:a5:1c [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 16 value 0 [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 3 value 100 [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 beacon_interval 100 [ 640.450000] ath10k_pci 0000:00:00.0: vdev 0 set beacon tx mode to staggered [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 7 value 0 [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 start center_freq 5180 phymode 11na-ht20 [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev start id 0x0 flags: 0x0, freq 5180, mode 4, ch_flags: 0x4000000, max_power: 34 [ 640.460000] ath10k_pci 0000:00:00.0: WMI_VDEV_START_RESP_EVENTID [ 640.460000] ath10k_pci 0000:00:00.0: wmi mgmt vdev up id 0x0 assoc id 0 bssid 52:8d:75:e5:00:52 [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 up [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 cts_prot 0 [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 0 [ 640.460000] ath10k_pci 0000:00:00.0: WMI_TBTTOFFSET_UPDATE_EVENTID [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 slot_time 2 [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 7 value 2 [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 preamble 1n [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 8 value 1 [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 peer create 04:f0:21:0c:a5:43 (new sta) sta 1 / 16 peer 2 / 24 [ 640.460000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 peer_addr 04:f0:21:0c:a5:43 [ 640.460000] IPv6: ADDRCONF(NETDEV_CHANGE): mesh0: link becomes ready [ 640.470000] ath10k_pci 0000:00:00.0: mac sta 04:f0:21:0c:a5:43 associated [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 phymode 11a [ 640.470000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr 04:f0:21:0c:a5:43 (new) [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer 0x04:f0:21:0c:a5:43 set param 5 value 1 [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 33 [ 640.470000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for 04:f0:21:0c:a5:43 changed 00000004 bw 0 nss 1 smps 2 [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for 04:f0:21:0c:a5:43 changed 00000008 bw 0 nss 3 smps 2 [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta 04:f0:21:0c:a5:43 nss 3 [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer 0x04:f0:21:0c:a5:43 set param 5 value 3 [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta 04:f0:21:0c:a5:43 supp rates/nss [ 640.540000] ath10k_pci 0000:00:00.0: mac ht peer 04:f0:21:0c:a5:43 mcs cnt 24 nss 3 [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 phymode 11na-ht20 [ 640.540000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr 04:f0:21:0c:a5:43 (reassociate) [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer 0x04:f0:21:0c:a5:43 set param 5 value 3 [ 640.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 460 [ 641.400000] ath10k_pci 0000:00:00.0: rx skb 86be4b40 len 148 peer 04:f0:21:0c:a5:43 mcast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 642.170000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 [ 642.400000] ath10k_pci 0000:00:00.0: rx skb 86b49b40 len 108 peer 04:f0:21:0c:a5:43 mcast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 651.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 28 [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86af8540 len 60 peer 04:f0:21:0c:a5:43 mcast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86967600 len 116 peer 04:f0:21:0c:a5:43 ucast sn 0 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1741.550000] ath10k_pci 0000:00:00.0: rx skb 86967000 len 116 peer 04:f0:21:0c:a5:43 ucast sn 1 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1742.550000] ath10k_pci 0000:00:00.0: rx skb 86b8acc0 len 116 peer 04:f0:21:0c:a5:43 ucast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1743.550000] ath10k_pci 0000:00:00.0: rx skb 86a7a600 len 116 peer 04:f0:21:0c:a5:43 ucast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1746.560000] ath10k_pci 0000:00:00.0: rx skb 86a79cc0 len 60 peer 04:f0:21:0c:a5:43 ucast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1752.410000] ath10k_pci 0000:00:00.0: rx skb 86a9d540 len 116 peer 04:f0:21:0c:a5:43 ucast sn 5 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1753.410000] ath10k_pci 0000:00:00.0: rx skb 86a7c6c0 len 116 peer 04:f0:21:0c:a5:43 ucast sn 6 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1754.410000] ath10k_pci 0000:00:00.0: rx skb 86a7f6c0 len 116 peer 04:f0:21:0c:a5:43 ucast sn 7 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1755.420000] ath10k_pci 0000:00:00.0: rx skb 86afc240 len 116 peer 04:f0:21:0c:a5:43 ucast sn 8 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1756.420000] ath10k_pci 0000:00:00.0: rx skb 86a7d480 len 116 peer 04:f0:21:0c:a5:43 ucast sn 9 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1757.420000] ath10k_pci 0000:00:00.0: rx skb 86af0180 len 116 peer 04:f0:21:0c:a5:43 ucast sn 10 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 On Tue, Dec 16, 2014 at 4:52 PM, Janusz Dziedzic <janusz.dziedzic@tieto.com> wrote: > This fix TX problem when IBSS used in HT mode. > Before we used 6Mbps all the time for TX direction. > > Reported-by: Yeoh Chun-Yeow <yeohchunyeow@gmail.com> > Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> > --- > drivers/net/wireless/ath/ath10k/mac.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c > index 5475f0f..c9e7995 100644 > --- a/drivers/net/wireless/ath/ath10k/mac.c > +++ b/drivers/net/wireless/ath/ath10k/mac.c > @@ -1411,9 +1411,16 @@ static void ath10k_peer_assoc_h_qos(struct ath10k *ar, > if (vif->bss_conf.qos) > arg->peer_flags |= WMI_PEER_QOS; > break; > + case WMI_VDEV_TYPE_IBSS: > + if (sta->wme) > + arg->peer_flags |= WMI_PEER_QOS; > + break; > default: > break; > } > + > + ath10k_dbg(ar, ATH10K_DBG_MAC, "mac peer %pM qos %d\n", > + sta->addr, !!(arg->peer_flags & WMI_PEER_QOS)); > } > > static void ath10k_peer_assoc_h_phymode(struct ath10k *ar, > -- > 1.9.1 > > > _______________________________________________ > ath10k mailing list > ath10k@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/ath10k _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] ath10k: fix low TX rates when IBSS and HT @ 2014-12-16 11:10 ` Yeoh Chun-Yeow 0 siblings, 0 replies; 24+ messages in thread From: Yeoh Chun-Yeow @ 2014-12-16 11:10 UTC (permalink / raw) To: Janusz Dziedzic Cc: ath10k@lists.infradead.org, linux-wireless@vger.kernel.org Hi, Janusz I have applied the three patches and tested with firmware 999.999.0.636 but not working. Any advice what's wrong? --- ChunYeow Some of my dmesg: [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 3 value 34 [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 4 value 34 [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params [ 640.450000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 peer_addr 04:f0:21:0c:a5:1c [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 16 value 0 [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 3 value 100 [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 beacon_interval 100 [ 640.450000] ath10k_pci 0000:00:00.0: vdev 0 set beacon tx mode to staggered [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 7 value 0 [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 start center_freq 5180 phymode 11na-ht20 [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev start id 0x0 flags: 0x0, freq 5180, mode 4, ch_flags: 0x4000000, max_power: 34 [ 640.460000] ath10k_pci 0000:00:00.0: WMI_VDEV_START_RESP_EVENTID [ 640.460000] ath10k_pci 0000:00:00.0: wmi mgmt vdev up id 0x0 assoc id 0 bssid 52:8d:75:e5:00:52 [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 up [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 cts_prot 0 [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 0 [ 640.460000] ath10k_pci 0000:00:00.0: WMI_TBTTOFFSET_UPDATE_EVENTID [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 slot_time 2 [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 7 value 2 [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 preamble 1n [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 8 value 1 [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 peer create 04:f0:21:0c:a5:43 (new sta) sta 1 / 16 peer 2 / 24 [ 640.460000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 peer_addr 04:f0:21:0c:a5:43 [ 640.460000] IPv6: ADDRCONF(NETDEV_CHANGE): mesh0: link becomes ready [ 640.470000] ath10k_pci 0000:00:00.0: mac sta 04:f0:21:0c:a5:43 associated [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 phymode 11a [ 640.470000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr 04:f0:21:0c:a5:43 (new) [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer 0x04:f0:21:0c:a5:43 set param 5 value 1 [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 33 [ 640.470000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for 04:f0:21:0c:a5:43 changed 00000004 bw 0 nss 1 smps 2 [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for 04:f0:21:0c:a5:43 changed 00000008 bw 0 nss 3 smps 2 [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta 04:f0:21:0c:a5:43 nss 3 [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer 0x04:f0:21:0c:a5:43 set param 5 value 3 [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta 04:f0:21:0c:a5:43 supp rates/nss [ 640.540000] ath10k_pci 0000:00:00.0: mac ht peer 04:f0:21:0c:a5:43 mcs cnt 24 nss 3 [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 phymode 11na-ht20 [ 640.540000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr 04:f0:21:0c:a5:43 (reassociate) [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer 0x04:f0:21:0c:a5:43 set param 5 value 3 [ 640.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 460 [ 641.400000] ath10k_pci 0000:00:00.0: rx skb 86be4b40 len 148 peer 04:f0:21:0c:a5:43 mcast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 642.170000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 [ 642.400000] ath10k_pci 0000:00:00.0: rx skb 86b49b40 len 108 peer 04:f0:21:0c:a5:43 mcast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 651.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 28 [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86af8540 len 60 peer 04:f0:21:0c:a5:43 mcast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86967600 len 116 peer 04:f0:21:0c:a5:43 ucast sn 0 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1741.550000] ath10k_pci 0000:00:00.0: rx skb 86967000 len 116 peer 04:f0:21:0c:a5:43 ucast sn 1 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1742.550000] ath10k_pci 0000:00:00.0: rx skb 86b8acc0 len 116 peer 04:f0:21:0c:a5:43 ucast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1743.550000] ath10k_pci 0000:00:00.0: rx skb 86a7a600 len 116 peer 04:f0:21:0c:a5:43 ucast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1746.560000] ath10k_pci 0000:00:00.0: rx skb 86a79cc0 len 60 peer 04:f0:21:0c:a5:43 ucast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1752.410000] ath10k_pci 0000:00:00.0: rx skb 86a9d540 len 116 peer 04:f0:21:0c:a5:43 ucast sn 5 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1753.410000] ath10k_pci 0000:00:00.0: rx skb 86a7c6c0 len 116 peer 04:f0:21:0c:a5:43 ucast sn 6 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1754.410000] ath10k_pci 0000:00:00.0: rx skb 86a7f6c0 len 116 peer 04:f0:21:0c:a5:43 ucast sn 7 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1755.420000] ath10k_pci 0000:00:00.0: rx skb 86afc240 len 116 peer 04:f0:21:0c:a5:43 ucast sn 8 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1756.420000] ath10k_pci 0000:00:00.0: rx skb 86a7d480 len 116 peer 04:f0:21:0c:a5:43 ucast sn 9 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 [ 1757.420000] ath10k_pci 0000:00:00.0: rx skb 86af0180 len 116 peer 04:f0:21:0c:a5:43 ucast sn 10 rate_idx 0 vht_nss 0 freq 5180 band 1 flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 On Tue, Dec 16, 2014 at 4:52 PM, Janusz Dziedzic <janusz.dziedzic@tieto.com> wrote: > This fix TX problem when IBSS used in HT mode. > Before we used 6Mbps all the time for TX direction. > > Reported-by: Yeoh Chun-Yeow <yeohchunyeow@gmail.com> > Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> > --- > drivers/net/wireless/ath/ath10k/mac.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c > index 5475f0f..c9e7995 100644 > --- a/drivers/net/wireless/ath/ath10k/mac.c > +++ b/drivers/net/wireless/ath/ath10k/mac.c > @@ -1411,9 +1411,16 @@ static void ath10k_peer_assoc_h_qos(struct ath10k *ar, > if (vif->bss_conf.qos) > arg->peer_flags |= WMI_PEER_QOS; > break; > + case WMI_VDEV_TYPE_IBSS: > + if (sta->wme) > + arg->peer_flags |= WMI_PEER_QOS; > + break; > default: > break; > } > + > + ath10k_dbg(ar, ATH10K_DBG_MAC, "mac peer %pM qos %d\n", > + sta->addr, !!(arg->peer_flags & WMI_PEER_QOS)); > } > > static void ath10k_peer_assoc_h_phymode(struct ath10k *ar, > -- > 1.9.1 > > > _______________________________________________ > ath10k mailing list > ath10k@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] ath10k: fix low TX rates when IBSS and HT 2014-12-16 11:10 ` Yeoh Chun-Yeow @ 2014-12-16 11:22 ` Janusz Dziedzic -1 siblings, 0 replies; 24+ messages in thread From: Janusz Dziedzic @ 2014-12-16 11:22 UTC (permalink / raw) To: Yeoh Chun-Yeow; +Cc: linux-wireless@vger.kernel.org, ath10k@lists.infradead.org On 16 December 2014 at 12:10, Yeoh Chun-Yeow <yeohchunyeow@gmail.com> wrote: > Hi, Janusz > > I have applied the three patches and tested with firmware > 999.999.0.636 but not working. > > Any advice what's wrong? > mac peer 04:f0:21:0c:a5:43 qos 0 - this is suspect. What is your test procedure (I see some mesh ...)? Maybe additional patch for mesh is required? I tested this using ath9k + ath10k (636) + iw wlanx set type ibss iw ibss join <> 5180 HT [HT40+] or wpa_supplicant in ibss mode (HT20 limited in current version) Next in airlogs I saw we are using nss>1 and higher mcs But for sure qos = 1 is required. BR Janusz > --- > ChunYeow > > Some of my dmesg: > [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 3 value 34 > [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 4 value 34 > [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params > [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params > [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params > [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params > [ 640.450000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 > peer_addr 04:f0:21:0c:a5:1c > [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 16 value 0 > [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 3 value 100 > [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 beacon_interval 100 > [ 640.450000] ath10k_pci 0000:00:00.0: vdev 0 set beacon tx mode to staggered > [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 7 value 0 > [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 start center_freq > 5180 phymode 11na-ht20 > [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev start id 0x0 flags: > 0x0, freq 5180, mode 4, ch_flags: 0x4000000, max_power: 34 > [ 640.460000] ath10k_pci 0000:00:00.0: WMI_VDEV_START_RESP_EVENTID > [ 640.460000] ath10k_pci 0000:00:00.0: wmi mgmt vdev up id 0x0 assoc > id 0 bssid 52:8d:75:e5:00:52 > [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 up > [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 cts_prot 0 > [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 0 > [ 640.460000] ath10k_pci 0000:00:00.0: WMI_TBTTOFFSET_UPDATE_EVENTID > [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 slot_time 2 > [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 7 value 2 > [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 preamble 1n > [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 8 value 1 > [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 peer create > 04:f0:21:0c:a5:43 (new sta) sta 1 / 16 peer 2 / 24 > [ 640.460000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 > peer_addr 04:f0:21:0c:a5:43 > [ 640.460000] IPv6: ADDRCONF(NETDEV_CHANGE): mesh0: link becomes ready > [ 640.470000] ath10k_pci 0000:00:00.0: mac sta 04:f0:21:0c:a5:43 associated > [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 > [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 phymode 11a > [ 640.470000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr > 04:f0:21:0c:a5:43 (new) > [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer > 0x04:f0:21:0c:a5:43 set param 5 value 1 > [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 33 > [ 640.470000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 > [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for > 04:f0:21:0c:a5:43 changed 00000004 bw 0 nss 1 smps 2 > [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for > 04:f0:21:0c:a5:43 changed 00000008 bw 0 nss 3 smps 2 > [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta 04:f0:21:0c:a5:43 nss 3 > [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer > 0x04:f0:21:0c:a5:43 set param 5 value 3 > [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta > 04:f0:21:0c:a5:43 supp rates/nss > [ 640.540000] ath10k_pci 0000:00:00.0: mac ht peer 04:f0:21:0c:a5:43 > mcs cnt 24 nss 3 > [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 > [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 > phymode 11na-ht20 > [ 640.540000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr > 04:f0:21:0c:a5:43 (reassociate) > [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer > 0x04:f0:21:0c:a5:43 set param 5 value 3 > [ 640.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 460 > [ 641.400000] ath10k_pci 0000:00:00.0: rx skb 86be4b40 len 148 peer > 04:f0:21:0c:a5:43 mcast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 642.170000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 > [ 642.400000] ath10k_pci 0000:00:00.0: rx skb 86b49b40 len 108 peer > 04:f0:21:0c:a5:43 mcast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 651.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 28 > [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86af8540 len 60 peer > 04:f0:21:0c:a5:43 mcast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86967600 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 0 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1741.550000] ath10k_pci 0000:00:00.0: rx skb 86967000 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 1 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1742.550000] ath10k_pci 0000:00:00.0: rx skb 86b8acc0 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1743.550000] ath10k_pci 0000:00:00.0: rx skb 86a7a600 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1746.560000] ath10k_pci 0000:00:00.0: rx skb 86a79cc0 len 60 peer > 04:f0:21:0c:a5:43 ucast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1752.410000] ath10k_pci 0000:00:00.0: rx skb 86a9d540 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 5 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1753.410000] ath10k_pci 0000:00:00.0: rx skb 86a7c6c0 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 6 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1754.410000] ath10k_pci 0000:00:00.0: rx skb 86a7f6c0 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 7 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1755.420000] ath10k_pci 0000:00:00.0: rx skb 86afc240 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 8 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1756.420000] ath10k_pci 0000:00:00.0: rx skb 86a7d480 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 9 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1757.420000] ath10k_pci 0000:00:00.0: rx skb 86af0180 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 10 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > > > On Tue, Dec 16, 2014 at 4:52 PM, Janusz Dziedzic > <janusz.dziedzic@tieto.com> wrote: >> This fix TX problem when IBSS used in HT mode. >> Before we used 6Mbps all the time for TX direction. >> >> Reported-by: Yeoh Chun-Yeow <yeohchunyeow@gmail.com> >> Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> >> --- >> drivers/net/wireless/ath/ath10k/mac.c | 7 +++++++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c >> index 5475f0f..c9e7995 100644 >> --- a/drivers/net/wireless/ath/ath10k/mac.c >> +++ b/drivers/net/wireless/ath/ath10k/mac.c >> @@ -1411,9 +1411,16 @@ static void ath10k_peer_assoc_h_qos(struct ath10k *ar, >> if (vif->bss_conf.qos) >> arg->peer_flags |= WMI_PEER_QOS; >> break; >> + case WMI_VDEV_TYPE_IBSS: >> + if (sta->wme) >> + arg->peer_flags |= WMI_PEER_QOS; >> + break; >> default: >> break; >> } >> + >> + ath10k_dbg(ar, ATH10K_DBG_MAC, "mac peer %pM qos %d\n", >> + sta->addr, !!(arg->peer_flags & WMI_PEER_QOS)); >> } >> >> static void ath10k_peer_assoc_h_phymode(struct ath10k *ar, >> -- >> 1.9.1 >> >> >> _______________________________________________ >> ath10k mailing list >> ath10k@lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/ath10k _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] ath10k: fix low TX rates when IBSS and HT @ 2014-12-16 11:22 ` Janusz Dziedzic 0 siblings, 0 replies; 24+ messages in thread From: Janusz Dziedzic @ 2014-12-16 11:22 UTC (permalink / raw) To: Yeoh Chun-Yeow; +Cc: ath10k@lists.infradead.org, linux-wireless@vger.kernel.org On 16 December 2014 at 12:10, Yeoh Chun-Yeow <yeohchunyeow@gmail.com> wrote: > Hi, Janusz > > I have applied the three patches and tested with firmware > 999.999.0.636 but not working. > > Any advice what's wrong? > mac peer 04:f0:21:0c:a5:43 qos 0 - this is suspect. What is your test procedure (I see some mesh ...)? Maybe additional patch for mesh is required? I tested this using ath9k + ath10k (636) + iw wlanx set type ibss iw ibss join <> 5180 HT [HT40+] or wpa_supplicant in ibss mode (HT20 limited in current version) Next in airlogs I saw we are using nss>1 and higher mcs But for sure qos = 1 is required. BR Janusz > --- > ChunYeow > > Some of my dmesg: > [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 3 value 34 > [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 4 value 34 > [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params > [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params > [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params > [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params > [ 640.450000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 > peer_addr 04:f0:21:0c:a5:1c > [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 16 value 0 > [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 3 value 100 > [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 beacon_interval 100 > [ 640.450000] ath10k_pci 0000:00:00.0: vdev 0 set beacon tx mode to staggered > [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 7 value 0 > [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 start center_freq > 5180 phymode 11na-ht20 > [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev start id 0x0 flags: > 0x0, freq 5180, mode 4, ch_flags: 0x4000000, max_power: 34 > [ 640.460000] ath10k_pci 0000:00:00.0: WMI_VDEV_START_RESP_EVENTID > [ 640.460000] ath10k_pci 0000:00:00.0: wmi mgmt vdev up id 0x0 assoc > id 0 bssid 52:8d:75:e5:00:52 > [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 up > [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 cts_prot 0 > [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 0 > [ 640.460000] ath10k_pci 0000:00:00.0: WMI_TBTTOFFSET_UPDATE_EVENTID > [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 slot_time 2 > [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 7 value 2 > [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 preamble 1n > [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 8 value 1 > [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 peer create > 04:f0:21:0c:a5:43 (new sta) sta 1 / 16 peer 2 / 24 > [ 640.460000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 > peer_addr 04:f0:21:0c:a5:43 > [ 640.460000] IPv6: ADDRCONF(NETDEV_CHANGE): mesh0: link becomes ready > [ 640.470000] ath10k_pci 0000:00:00.0: mac sta 04:f0:21:0c:a5:43 associated > [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 > [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 phymode 11a > [ 640.470000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr > 04:f0:21:0c:a5:43 (new) > [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer > 0x04:f0:21:0c:a5:43 set param 5 value 1 > [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 33 > [ 640.470000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 > [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for > 04:f0:21:0c:a5:43 changed 00000004 bw 0 nss 1 smps 2 > [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for > 04:f0:21:0c:a5:43 changed 00000008 bw 0 nss 3 smps 2 > [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta 04:f0:21:0c:a5:43 nss 3 > [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer > 0x04:f0:21:0c:a5:43 set param 5 value 3 > [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta > 04:f0:21:0c:a5:43 supp rates/nss > [ 640.540000] ath10k_pci 0000:00:00.0: mac ht peer 04:f0:21:0c:a5:43 > mcs cnt 24 nss 3 > [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 > [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 > phymode 11na-ht20 > [ 640.540000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr > 04:f0:21:0c:a5:43 (reassociate) > [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer > 0x04:f0:21:0c:a5:43 set param 5 value 3 > [ 640.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 460 > [ 641.400000] ath10k_pci 0000:00:00.0: rx skb 86be4b40 len 148 peer > 04:f0:21:0c:a5:43 mcast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 642.170000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 > [ 642.400000] ath10k_pci 0000:00:00.0: rx skb 86b49b40 len 108 peer > 04:f0:21:0c:a5:43 mcast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 651.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 28 > [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86af8540 len 60 peer > 04:f0:21:0c:a5:43 mcast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86967600 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 0 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1741.550000] ath10k_pci 0000:00:00.0: rx skb 86967000 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 1 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1742.550000] ath10k_pci 0000:00:00.0: rx skb 86b8acc0 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1743.550000] ath10k_pci 0000:00:00.0: rx skb 86a7a600 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1746.560000] ath10k_pci 0000:00:00.0: rx skb 86a79cc0 len 60 peer > 04:f0:21:0c:a5:43 ucast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1752.410000] ath10k_pci 0000:00:00.0: rx skb 86a9d540 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 5 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1753.410000] ath10k_pci 0000:00:00.0: rx skb 86a7c6c0 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 6 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1754.410000] ath10k_pci 0000:00:00.0: rx skb 86a7f6c0 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 7 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1755.420000] ath10k_pci 0000:00:00.0: rx skb 86afc240 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 8 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1756.420000] ath10k_pci 0000:00:00.0: rx skb 86a7d480 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 9 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > [ 1757.420000] ath10k_pci 0000:00:00.0: rx skb 86af0180 len 116 peer > 04:f0:21:0c:a5:43 ucast sn 10 rate_idx 0 vht_nss 0 freq 5180 band 1 > flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 > > > On Tue, Dec 16, 2014 at 4:52 PM, Janusz Dziedzic > <janusz.dziedzic@tieto.com> wrote: >> This fix TX problem when IBSS used in HT mode. >> Before we used 6Mbps all the time for TX direction. >> >> Reported-by: Yeoh Chun-Yeow <yeohchunyeow@gmail.com> >> Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> >> --- >> drivers/net/wireless/ath/ath10k/mac.c | 7 +++++++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c >> index 5475f0f..c9e7995 100644 >> --- a/drivers/net/wireless/ath/ath10k/mac.c >> +++ b/drivers/net/wireless/ath/ath10k/mac.c >> @@ -1411,9 +1411,16 @@ static void ath10k_peer_assoc_h_qos(struct ath10k *ar, >> if (vif->bss_conf.qos) >> arg->peer_flags |= WMI_PEER_QOS; >> break; >> + case WMI_VDEV_TYPE_IBSS: >> + if (sta->wme) >> + arg->peer_flags |= WMI_PEER_QOS; >> + break; >> default: >> break; >> } >> + >> + ath10k_dbg(ar, ATH10K_DBG_MAC, "mac peer %pM qos %d\n", >> + sta->addr, !!(arg->peer_flags & WMI_PEER_QOS)); >> } >> >> static void ath10k_peer_assoc_h_phymode(struct ath10k *ar, >> -- >> 1.9.1 >> >> >> _______________________________________________ >> ath10k mailing list >> ath10k@lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] ath10k: fix low TX rates when IBSS and HT 2014-12-16 11:22 ` Janusz Dziedzic @ 2014-12-16 11:45 ` Janusz Dziedzic -1 siblings, 0 replies; 24+ messages in thread From: Janusz Dziedzic @ 2014-12-16 11:45 UTC (permalink / raw) To: Yeoh Chun-Yeow; +Cc: linux-wireless@vger.kernel.org, ath10k@lists.infradead.org On 16 December 2014 at 12:22, Janusz Dziedzic <janusz.dziedzic@tieto.com> wrote: > On 16 December 2014 at 12:10, Yeoh Chun-Yeow <yeohchunyeow@gmail.com> wrote: >> Hi, Janusz >> >> I have applied the three patches and tested with firmware >> 999.999.0.636 but not working. >> >> Any advice what's wrong? >> > mac peer 04:f0:21:0c:a5:43 qos 0 - this is suspect. > What is your test procedure (I see some mesh ...)? > Maybe additional patch for mesh is required? > > I tested this using ath9k + ath10k (636) + > iw wlanx set type ibss > iw ibss join <> 5180 HT [HT40+] > or wpa_supplicant in ibss mode (HT20 limited in current version) > > Next in airlogs I saw we are using nss>1 and higher mcs > But for sure qos = 1 is required. > I am not sure why sta->wme is not set while we have HT rates (is it allowed to have HT without WMM?) But anyway as a quick workaroud you can try something like this: @@ -1212,7 +1212,7 @@ static void ath10k_peer_assoc_h_ht(struct ath10k *ar, if (!ht_cap->ht_supported) return; - arg->peer_flags |= WMI_PEER_HT; + arg->peer_flags |= WMI_PEER_HT | WMI_PEER_QOS; arg->peer_max_mpdu = (1 << (IEEE80211_HT_MAX_AMPDU_FACTOR + ht_cap->ampdu_factor)) - 1; @@ -3464,6 +3464,13 @@ static int ath10k_set_key(struct ieee80211_hw *hw, enum set_key_cmd cmd, key->cipher == WLAN_CIPHER_SUITE_WEP104; int ret = 0; > BR > Janusz > > >> --- >> ChunYeow >> >> Some of my dmesg: >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 3 value 34 >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 4 value 34 >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 >> peer_addr 04:f0:21:0c:a5:1c >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 16 value 0 >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 3 value 100 >> [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 beacon_interval 100 >> [ 640.450000] ath10k_pci 0000:00:00.0: vdev 0 set beacon tx mode to staggered >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 7 value 0 >> [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 start center_freq >> 5180 phymode 11na-ht20 >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev start id 0x0 flags: >> 0x0, freq 5180, mode 4, ch_flags: 0x4000000, max_power: 34 >> [ 640.460000] ath10k_pci 0000:00:00.0: WMI_VDEV_START_RESP_EVENTID >> [ 640.460000] ath10k_pci 0000:00:00.0: wmi mgmt vdev up id 0x0 assoc >> id 0 bssid 52:8d:75:e5:00:52 >> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 up >> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 cts_prot 0 >> [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 0 >> [ 640.460000] ath10k_pci 0000:00:00.0: WMI_TBTTOFFSET_UPDATE_EVENTID >> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 slot_time 2 >> [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 7 value 2 >> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 preamble 1n >> [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 8 value 1 >> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 peer create >> 04:f0:21:0c:a5:43 (new sta) sta 1 / 16 peer 2 / 24 >> [ 640.460000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 >> peer_addr 04:f0:21:0c:a5:43 >> [ 640.460000] IPv6: ADDRCONF(NETDEV_CHANGE): mesh0: link becomes ready >> [ 640.470000] ath10k_pci 0000:00:00.0: mac sta 04:f0:21:0c:a5:43 associated >> [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 >> [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 phymode 11a >> [ 640.470000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr >> 04:f0:21:0c:a5:43 (new) >> [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer >> 0x04:f0:21:0c:a5:43 set param 5 value 1 >> [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 33 >> [ 640.470000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 >> [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for >> 04:f0:21:0c:a5:43 changed 00000004 bw 0 nss 1 smps 2 >> [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for >> 04:f0:21:0c:a5:43 changed 00000008 bw 0 nss 3 smps 2 >> [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta 04:f0:21:0c:a5:43 nss 3 >> [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer >> 0x04:f0:21:0c:a5:43 set param 5 value 3 >> [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta >> 04:f0:21:0c:a5:43 supp rates/nss >> [ 640.540000] ath10k_pci 0000:00:00.0: mac ht peer 04:f0:21:0c:a5:43 >> mcs cnt 24 nss 3 >> [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 >> [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 >> phymode 11na-ht20 >> [ 640.540000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr >> 04:f0:21:0c:a5:43 (reassociate) >> [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer >> 0x04:f0:21:0c:a5:43 set param 5 value 3 >> [ 640.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 460 >> [ 641.400000] ath10k_pci 0000:00:00.0: rx skb 86be4b40 len 148 peer >> 04:f0:21:0c:a5:43 mcast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 642.170000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 >> [ 642.400000] ath10k_pci 0000:00:00.0: rx skb 86b49b40 len 108 peer >> 04:f0:21:0c:a5:43 mcast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 651.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 28 >> [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86af8540 len 60 peer >> 04:f0:21:0c:a5:43 mcast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86967600 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 0 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1741.550000] ath10k_pci 0000:00:00.0: rx skb 86967000 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 1 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1742.550000] ath10k_pci 0000:00:00.0: rx skb 86b8acc0 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1743.550000] ath10k_pci 0000:00:00.0: rx skb 86a7a600 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1746.560000] ath10k_pci 0000:00:00.0: rx skb 86a79cc0 len 60 peer >> 04:f0:21:0c:a5:43 ucast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1752.410000] ath10k_pci 0000:00:00.0: rx skb 86a9d540 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 5 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1753.410000] ath10k_pci 0000:00:00.0: rx skb 86a7c6c0 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 6 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1754.410000] ath10k_pci 0000:00:00.0: rx skb 86a7f6c0 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 7 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1755.420000] ath10k_pci 0000:00:00.0: rx skb 86afc240 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 8 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1756.420000] ath10k_pci 0000:00:00.0: rx skb 86a7d480 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 9 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1757.420000] ath10k_pci 0000:00:00.0: rx skb 86af0180 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 10 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> >> >> On Tue, Dec 16, 2014 at 4:52 PM, Janusz Dziedzic >> <janusz.dziedzic@tieto.com> wrote: >>> This fix TX problem when IBSS used in HT mode. >>> Before we used 6Mbps all the time for TX direction. >>> >>> Reported-by: Yeoh Chun-Yeow <yeohchunyeow@gmail.com> >>> Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> >>> --- >>> drivers/net/wireless/ath/ath10k/mac.c | 7 +++++++ >>> 1 file changed, 7 insertions(+) >>> >>> diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c >>> index 5475f0f..c9e7995 100644 >>> --- a/drivers/net/wireless/ath/ath10k/mac.c >>> +++ b/drivers/net/wireless/ath/ath10k/mac.c >>> @@ -1411,9 +1411,16 @@ static void ath10k_peer_assoc_h_qos(struct ath10k *ar, >>> if (vif->bss_conf.qos) >>> arg->peer_flags |= WMI_PEER_QOS; >>> break; >>> + case WMI_VDEV_TYPE_IBSS: >>> + if (sta->wme) >>> + arg->peer_flags |= WMI_PEER_QOS; >>> + break; >>> default: >>> break; >>> } >>> + >>> + ath10k_dbg(ar, ATH10K_DBG_MAC, "mac peer %pM qos %d\n", >>> + sta->addr, !!(arg->peer_flags & WMI_PEER_QOS)); >>> } >>> >>> static void ath10k_peer_assoc_h_phymode(struct ath10k *ar, >>> -- >>> 1.9.1 >>> >>> >>> _______________________________________________ >>> ath10k mailing list >>> ath10k@lists.infradead.org >>> http://lists.infradead.org/mailman/listinfo/ath10k _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] ath10k: fix low TX rates when IBSS and HT @ 2014-12-16 11:45 ` Janusz Dziedzic 0 siblings, 0 replies; 24+ messages in thread From: Janusz Dziedzic @ 2014-12-16 11:45 UTC (permalink / raw) To: Yeoh Chun-Yeow; +Cc: ath10k@lists.infradead.org, linux-wireless@vger.kernel.org On 16 December 2014 at 12:22, Janusz Dziedzic <janusz.dziedzic@tieto.com> wrote: > On 16 December 2014 at 12:10, Yeoh Chun-Yeow <yeohchunyeow@gmail.com> wrote: >> Hi, Janusz >> >> I have applied the three patches and tested with firmware >> 999.999.0.636 but not working. >> >> Any advice what's wrong? >> > mac peer 04:f0:21:0c:a5:43 qos 0 - this is suspect. > What is your test procedure (I see some mesh ...)? > Maybe additional patch for mesh is required? > > I tested this using ath9k + ath10k (636) + > iw wlanx set type ibss > iw ibss join <> 5180 HT [HT40+] > or wpa_supplicant in ibss mode (HT20 limited in current version) > > Next in airlogs I saw we are using nss>1 and higher mcs > But for sure qos = 1 is required. > I am not sure why sta->wme is not set while we have HT rates (is it allowed to have HT without WMM?) But anyway as a quick workaroud you can try something like this: @@ -1212,7 +1212,7 @@ static void ath10k_peer_assoc_h_ht(struct ath10k *ar, if (!ht_cap->ht_supported) return; - arg->peer_flags |= WMI_PEER_HT; + arg->peer_flags |= WMI_PEER_HT | WMI_PEER_QOS; arg->peer_max_mpdu = (1 << (IEEE80211_HT_MAX_AMPDU_FACTOR + ht_cap->ampdu_factor)) - 1; @@ -3464,6 +3464,13 @@ static int ath10k_set_key(struct ieee80211_hw *hw, enum set_key_cmd cmd, key->cipher == WLAN_CIPHER_SUITE_WEP104; int ret = 0; > BR > Janusz > > >> --- >> ChunYeow >> >> Some of my dmesg: >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 3 value 34 >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 4 value 34 >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 >> peer_addr 04:f0:21:0c:a5:1c >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 16 value 0 >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 3 value 100 >> [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 beacon_interval 100 >> [ 640.450000] ath10k_pci 0000:00:00.0: vdev 0 set beacon tx mode to staggered >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 7 value 0 >> [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 start center_freq >> 5180 phymode 11na-ht20 >> [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev start id 0x0 flags: >> 0x0, freq 5180, mode 4, ch_flags: 0x4000000, max_power: 34 >> [ 640.460000] ath10k_pci 0000:00:00.0: WMI_VDEV_START_RESP_EVENTID >> [ 640.460000] ath10k_pci 0000:00:00.0: wmi mgmt vdev up id 0x0 assoc >> id 0 bssid 52:8d:75:e5:00:52 >> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 up >> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 cts_prot 0 >> [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 0 >> [ 640.460000] ath10k_pci 0000:00:00.0: WMI_TBTTOFFSET_UPDATE_EVENTID >> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 slot_time 2 >> [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 7 value 2 >> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 preamble 1n >> [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 8 value 1 >> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 peer create >> 04:f0:21:0c:a5:43 (new sta) sta 1 / 16 peer 2 / 24 >> [ 640.460000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 >> peer_addr 04:f0:21:0c:a5:43 >> [ 640.460000] IPv6: ADDRCONF(NETDEV_CHANGE): mesh0: link becomes ready >> [ 640.470000] ath10k_pci 0000:00:00.0: mac sta 04:f0:21:0c:a5:43 associated >> [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 >> [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 phymode 11a >> [ 640.470000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr >> 04:f0:21:0c:a5:43 (new) >> [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer >> 0x04:f0:21:0c:a5:43 set param 5 value 1 >> [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 33 >> [ 640.470000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 >> [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for >> 04:f0:21:0c:a5:43 changed 00000004 bw 0 nss 1 smps 2 >> [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for >> 04:f0:21:0c:a5:43 changed 00000008 bw 0 nss 3 smps 2 >> [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta 04:f0:21:0c:a5:43 nss 3 >> [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer >> 0x04:f0:21:0c:a5:43 set param 5 value 3 >> [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta >> 04:f0:21:0c:a5:43 supp rates/nss >> [ 640.540000] ath10k_pci 0000:00:00.0: mac ht peer 04:f0:21:0c:a5:43 >> mcs cnt 24 nss 3 >> [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 >> [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 >> phymode 11na-ht20 >> [ 640.540000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr >> 04:f0:21:0c:a5:43 (reassociate) >> [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer >> 0x04:f0:21:0c:a5:43 set param 5 value 3 >> [ 640.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 460 >> [ 641.400000] ath10k_pci 0000:00:00.0: rx skb 86be4b40 len 148 peer >> 04:f0:21:0c:a5:43 mcast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 642.170000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 >> [ 642.400000] ath10k_pci 0000:00:00.0: rx skb 86b49b40 len 108 peer >> 04:f0:21:0c:a5:43 mcast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 651.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 28 >> [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86af8540 len 60 peer >> 04:f0:21:0c:a5:43 mcast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86967600 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 0 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1741.550000] ath10k_pci 0000:00:00.0: rx skb 86967000 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 1 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1742.550000] ath10k_pci 0000:00:00.0: rx skb 86b8acc0 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1743.550000] ath10k_pci 0000:00:00.0: rx skb 86a7a600 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1746.560000] ath10k_pci 0000:00:00.0: rx skb 86a79cc0 len 60 peer >> 04:f0:21:0c:a5:43 ucast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1752.410000] ath10k_pci 0000:00:00.0: rx skb 86a9d540 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 5 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1753.410000] ath10k_pci 0000:00:00.0: rx skb 86a7c6c0 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 6 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1754.410000] ath10k_pci 0000:00:00.0: rx skb 86a7f6c0 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 7 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1755.420000] ath10k_pci 0000:00:00.0: rx skb 86afc240 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 8 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1756.420000] ath10k_pci 0000:00:00.0: rx skb 86a7d480 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 9 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> [ 1757.420000] ath10k_pci 0000:00:00.0: rx skb 86af0180 len 116 peer >> 04:f0:21:0c:a5:43 ucast sn 10 rate_idx 0 vht_nss 0 freq 5180 band 1 >> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >> >> >> On Tue, Dec 16, 2014 at 4:52 PM, Janusz Dziedzic >> <janusz.dziedzic@tieto.com> wrote: >>> This fix TX problem when IBSS used in HT mode. >>> Before we used 6Mbps all the time for TX direction. >>> >>> Reported-by: Yeoh Chun-Yeow <yeohchunyeow@gmail.com> >>> Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> >>> --- >>> drivers/net/wireless/ath/ath10k/mac.c | 7 +++++++ >>> 1 file changed, 7 insertions(+) >>> >>> diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c >>> index 5475f0f..c9e7995 100644 >>> --- a/drivers/net/wireless/ath/ath10k/mac.c >>> +++ b/drivers/net/wireless/ath/ath10k/mac.c >>> @@ -1411,9 +1411,16 @@ static void ath10k_peer_assoc_h_qos(struct ath10k *ar, >>> if (vif->bss_conf.qos) >>> arg->peer_flags |= WMI_PEER_QOS; >>> break; >>> + case WMI_VDEV_TYPE_IBSS: >>> + if (sta->wme) >>> + arg->peer_flags |= WMI_PEER_QOS; >>> + break; >>> default: >>> break; >>> } >>> + >>> + ath10k_dbg(ar, ATH10K_DBG_MAC, "mac peer %pM qos %d\n", >>> + sta->addr, !!(arg->peer_flags & WMI_PEER_QOS)); >>> } >>> >>> static void ath10k_peer_assoc_h_phymode(struct ath10k *ar, >>> -- >>> 1.9.1 >>> >>> >>> _______________________________________________ >>> ath10k mailing list >>> ath10k@lists.infradead.org >>> http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] ath10k: fix low TX rates when IBSS and HT 2014-12-16 11:45 ` Janusz Dziedzic @ 2014-12-16 13:32 ` Yeoh Chun-Yeow -1 siblings, 0 replies; 24+ messages in thread From: Yeoh Chun-Yeow @ 2014-12-16 13:32 UTC (permalink / raw) To: Janusz Dziedzic Cc: linux-wireless@vger.kernel.org, ath10k@lists.infradead.org Ok, I will try this and investigate further. --- ChunYeow On Tue, Dec 16, 2014 at 7:45 PM, Janusz Dziedzic <janusz.dziedzic@tieto.com> wrote: > On 16 December 2014 at 12:22, Janusz Dziedzic <janusz.dziedzic@tieto.com> wrote: >> On 16 December 2014 at 12:10, Yeoh Chun-Yeow <yeohchunyeow@gmail.com> wrote: >>> Hi, Janusz >>> >>> I have applied the three patches and tested with firmware >>> 999.999.0.636 but not working. >>> >>> Any advice what's wrong? >>> >> mac peer 04:f0:21:0c:a5:43 qos 0 - this is suspect. >> What is your test procedure (I see some mesh ...)? >> Maybe additional patch for mesh is required? >> >> I tested this using ath9k + ath10k (636) + >> iw wlanx set type ibss >> iw ibss join <> 5180 HT [HT40+] >> or wpa_supplicant in ibss mode (HT20 limited in current version) >> >> Next in airlogs I saw we are using nss>1 and higher mcs >> But for sure qos = 1 is required. >> > I am not sure why sta->wme is not set while we have HT rates (is it > allowed to have HT without WMM?) > But anyway as a quick workaroud you can try something like this: > > @@ -1212,7 +1212,7 @@ static void ath10k_peer_assoc_h_ht(struct ath10k *ar, > if (!ht_cap->ht_supported) > return; > > - arg->peer_flags |= WMI_PEER_HT; > + arg->peer_flags |= WMI_PEER_HT | WMI_PEER_QOS; > arg->peer_max_mpdu = (1 << (IEEE80211_HT_MAX_AMPDU_FACTOR + > ht_cap->ampdu_factor)) - 1; > > @@ -3464,6 +3464,13 @@ static int ath10k_set_key(struct ieee80211_hw > *hw, enum set_key_cmd cmd, > key->cipher == WLAN_CIPHER_SUITE_WEP104; > int ret = 0; > >> BR >> Janusz >> >> >>> --- >>> ChunYeow >>> >>> Some of my dmesg: >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 3 value 34 >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 4 value 34 >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 >>> peer_addr 04:f0:21:0c:a5:1c >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 16 value 0 >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 3 value 100 >>> [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 beacon_interval 100 >>> [ 640.450000] ath10k_pci 0000:00:00.0: vdev 0 set beacon tx mode to staggered >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 7 value 0 >>> [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 start center_freq >>> 5180 phymode 11na-ht20 >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev start id 0x0 flags: >>> 0x0, freq 5180, mode 4, ch_flags: 0x4000000, max_power: 34 >>> [ 640.460000] ath10k_pci 0000:00:00.0: WMI_VDEV_START_RESP_EVENTID >>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi mgmt vdev up id 0x0 assoc >>> id 0 bssid 52:8d:75:e5:00:52 >>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 up >>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 cts_prot 0 >>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 0 >>> [ 640.460000] ath10k_pci 0000:00:00.0: WMI_TBTTOFFSET_UPDATE_EVENTID >>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 slot_time 2 >>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 7 value 2 >>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 preamble 1n >>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 8 value 1 >>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 peer create >>> 04:f0:21:0c:a5:43 (new sta) sta 1 / 16 peer 2 / 24 >>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 >>> peer_addr 04:f0:21:0c:a5:43 >>> [ 640.460000] IPv6: ADDRCONF(NETDEV_CHANGE): mesh0: link becomes ready >>> [ 640.470000] ath10k_pci 0000:00:00.0: mac sta 04:f0:21:0c:a5:43 associated >>> [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 >>> [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 phymode 11a >>> [ 640.470000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr >>> 04:f0:21:0c:a5:43 (new) >>> [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer >>> 0x04:f0:21:0c:a5:43 set param 5 value 1 >>> [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 33 >>> [ 640.470000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 >>> [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for >>> 04:f0:21:0c:a5:43 changed 00000004 bw 0 nss 1 smps 2 >>> [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for >>> 04:f0:21:0c:a5:43 changed 00000008 bw 0 nss 3 smps 2 >>> [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta 04:f0:21:0c:a5:43 nss 3 >>> [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer >>> 0x04:f0:21:0c:a5:43 set param 5 value 3 >>> [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta >>> 04:f0:21:0c:a5:43 supp rates/nss >>> [ 640.540000] ath10k_pci 0000:00:00.0: mac ht peer 04:f0:21:0c:a5:43 >>> mcs cnt 24 nss 3 >>> [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 >>> [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 >>> phymode 11na-ht20 >>> [ 640.540000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr >>> 04:f0:21:0c:a5:43 (reassociate) >>> [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer >>> 0x04:f0:21:0c:a5:43 set param 5 value 3 >>> [ 640.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 460 >>> [ 641.400000] ath10k_pci 0000:00:00.0: rx skb 86be4b40 len 148 peer >>> 04:f0:21:0c:a5:43 mcast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 642.170000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 >>> [ 642.400000] ath10k_pci 0000:00:00.0: rx skb 86b49b40 len 108 peer >>> 04:f0:21:0c:a5:43 mcast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 651.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 28 >>> [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86af8540 len 60 peer >>> 04:f0:21:0c:a5:43 mcast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86967600 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 0 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1741.550000] ath10k_pci 0000:00:00.0: rx skb 86967000 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 1 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1742.550000] ath10k_pci 0000:00:00.0: rx skb 86b8acc0 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1743.550000] ath10k_pci 0000:00:00.0: rx skb 86a7a600 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1746.560000] ath10k_pci 0000:00:00.0: rx skb 86a79cc0 len 60 peer >>> 04:f0:21:0c:a5:43 ucast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1752.410000] ath10k_pci 0000:00:00.0: rx skb 86a9d540 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 5 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1753.410000] ath10k_pci 0000:00:00.0: rx skb 86a7c6c0 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 6 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1754.410000] ath10k_pci 0000:00:00.0: rx skb 86a7f6c0 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 7 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1755.420000] ath10k_pci 0000:00:00.0: rx skb 86afc240 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 8 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1756.420000] ath10k_pci 0000:00:00.0: rx skb 86a7d480 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 9 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1757.420000] ath10k_pci 0000:00:00.0: rx skb 86af0180 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 10 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> >>> >>> On Tue, Dec 16, 2014 at 4:52 PM, Janusz Dziedzic >>> <janusz.dziedzic@tieto.com> wrote: >>>> This fix TX problem when IBSS used in HT mode. >>>> Before we used 6Mbps all the time for TX direction. >>>> >>>> Reported-by: Yeoh Chun-Yeow <yeohchunyeow@gmail.com> >>>> Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> >>>> --- >>>> drivers/net/wireless/ath/ath10k/mac.c | 7 +++++++ >>>> 1 file changed, 7 insertions(+) >>>> >>>> diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c >>>> index 5475f0f..c9e7995 100644 >>>> --- a/drivers/net/wireless/ath/ath10k/mac.c >>>> +++ b/drivers/net/wireless/ath/ath10k/mac.c >>>> @@ -1411,9 +1411,16 @@ static void ath10k_peer_assoc_h_qos(struct ath10k *ar, >>>> if (vif->bss_conf.qos) >>>> arg->peer_flags |= WMI_PEER_QOS; >>>> break; >>>> + case WMI_VDEV_TYPE_IBSS: >>>> + if (sta->wme) >>>> + arg->peer_flags |= WMI_PEER_QOS; >>>> + break; >>>> default: >>>> break; >>>> } >>>> + >>>> + ath10k_dbg(ar, ATH10K_DBG_MAC, "mac peer %pM qos %d\n", >>>> + sta->addr, !!(arg->peer_flags & WMI_PEER_QOS)); >>>> } >>>> >>>> static void ath10k_peer_assoc_h_phymode(struct ath10k *ar, >>>> -- >>>> 1.9.1 >>>> >>>> >>>> _______________________________________________ >>>> ath10k mailing list >>>> ath10k@lists.infradead.org >>>> http://lists.infradead.org/mailman/listinfo/ath10k _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] ath10k: fix low TX rates when IBSS and HT @ 2014-12-16 13:32 ` Yeoh Chun-Yeow 0 siblings, 0 replies; 24+ messages in thread From: Yeoh Chun-Yeow @ 2014-12-16 13:32 UTC (permalink / raw) To: Janusz Dziedzic Cc: ath10k@lists.infradead.org, linux-wireless@vger.kernel.org Ok, I will try this and investigate further. --- ChunYeow On Tue, Dec 16, 2014 at 7:45 PM, Janusz Dziedzic <janusz.dziedzic@tieto.com> wrote: > On 16 December 2014 at 12:22, Janusz Dziedzic <janusz.dziedzic@tieto.com> wrote: >> On 16 December 2014 at 12:10, Yeoh Chun-Yeow <yeohchunyeow@gmail.com> wrote: >>> Hi, Janusz >>> >>> I have applied the three patches and tested with firmware >>> 999.999.0.636 but not working. >>> >>> Any advice what's wrong? >>> >> mac peer 04:f0:21:0c:a5:43 qos 0 - this is suspect. >> What is your test procedure (I see some mesh ...)? >> Maybe additional patch for mesh is required? >> >> I tested this using ath9k + ath10k (636) + >> iw wlanx set type ibss >> iw ibss join <> 5180 HT [HT40+] >> or wpa_supplicant in ibss mode (HT20 limited in current version) >> >> Next in airlogs I saw we are using nss>1 and higher mcs >> But for sure qos = 1 is required. >> > I am not sure why sta->wme is not set while we have HT rates (is it > allowed to have HT without WMM?) > But anyway as a quick workaroud you can try something like this: > > @@ -1212,7 +1212,7 @@ static void ath10k_peer_assoc_h_ht(struct ath10k *ar, > if (!ht_cap->ht_supported) > return; > > - arg->peer_flags |= WMI_PEER_HT; > + arg->peer_flags |= WMI_PEER_HT | WMI_PEER_QOS; > arg->peer_max_mpdu = (1 << (IEEE80211_HT_MAX_AMPDU_FACTOR + > ht_cap->ampdu_factor)) - 1; > > @@ -3464,6 +3464,13 @@ static int ath10k_set_key(struct ieee80211_hw > *hw, enum set_key_cmd cmd, > key->cipher == WLAN_CIPHER_SUITE_WEP104; > int ret = 0; > >> BR >> Janusz >> >> >>> --- >>> ChunYeow >>> >>> Some of my dmesg: >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 3 value 34 >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 4 value 34 >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 >>> peer_addr 04:f0:21:0c:a5:1c >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 16 value 0 >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 3 value 100 >>> [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 beacon_interval 100 >>> [ 640.450000] ath10k_pci 0000:00:00.0: vdev 0 set beacon tx mode to staggered >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 7 value 0 >>> [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 start center_freq >>> 5180 phymode 11na-ht20 >>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev start id 0x0 flags: >>> 0x0, freq 5180, mode 4, ch_flags: 0x4000000, max_power: 34 >>> [ 640.460000] ath10k_pci 0000:00:00.0: WMI_VDEV_START_RESP_EVENTID >>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi mgmt vdev up id 0x0 assoc >>> id 0 bssid 52:8d:75:e5:00:52 >>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 up >>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 cts_prot 0 >>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 0 >>> [ 640.460000] ath10k_pci 0000:00:00.0: WMI_TBTTOFFSET_UPDATE_EVENTID >>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 slot_time 2 >>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 7 value 2 >>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 preamble 1n >>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 8 value 1 >>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 peer create >>> 04:f0:21:0c:a5:43 (new sta) sta 1 / 16 peer 2 / 24 >>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 >>> peer_addr 04:f0:21:0c:a5:43 >>> [ 640.460000] IPv6: ADDRCONF(NETDEV_CHANGE): mesh0: link becomes ready >>> [ 640.470000] ath10k_pci 0000:00:00.0: mac sta 04:f0:21:0c:a5:43 associated >>> [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 >>> [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 phymode 11a >>> [ 640.470000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr >>> 04:f0:21:0c:a5:43 (new) >>> [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer >>> 0x04:f0:21:0c:a5:43 set param 5 value 1 >>> [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 33 >>> [ 640.470000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 >>> [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for >>> 04:f0:21:0c:a5:43 changed 00000004 bw 0 nss 1 smps 2 >>> [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for >>> 04:f0:21:0c:a5:43 changed 00000008 bw 0 nss 3 smps 2 >>> [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta 04:f0:21:0c:a5:43 nss 3 >>> [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer >>> 0x04:f0:21:0c:a5:43 set param 5 value 3 >>> [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta >>> 04:f0:21:0c:a5:43 supp rates/nss >>> [ 640.540000] ath10k_pci 0000:00:00.0: mac ht peer 04:f0:21:0c:a5:43 >>> mcs cnt 24 nss 3 >>> [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 >>> [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 >>> phymode 11na-ht20 >>> [ 640.540000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr >>> 04:f0:21:0c:a5:43 (reassociate) >>> [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer >>> 0x04:f0:21:0c:a5:43 set param 5 value 3 >>> [ 640.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 460 >>> [ 641.400000] ath10k_pci 0000:00:00.0: rx skb 86be4b40 len 148 peer >>> 04:f0:21:0c:a5:43 mcast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 642.170000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 >>> [ 642.400000] ath10k_pci 0000:00:00.0: rx skb 86b49b40 len 108 peer >>> 04:f0:21:0c:a5:43 mcast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 651.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 28 >>> [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86af8540 len 60 peer >>> 04:f0:21:0c:a5:43 mcast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86967600 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 0 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1741.550000] ath10k_pci 0000:00:00.0: rx skb 86967000 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 1 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1742.550000] ath10k_pci 0000:00:00.0: rx skb 86b8acc0 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1743.550000] ath10k_pci 0000:00:00.0: rx skb 86a7a600 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1746.560000] ath10k_pci 0000:00:00.0: rx skb 86a79cc0 len 60 peer >>> 04:f0:21:0c:a5:43 ucast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1752.410000] ath10k_pci 0000:00:00.0: rx skb 86a9d540 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 5 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1753.410000] ath10k_pci 0000:00:00.0: rx skb 86a7c6c0 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 6 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1754.410000] ath10k_pci 0000:00:00.0: rx skb 86a7f6c0 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 7 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1755.420000] ath10k_pci 0000:00:00.0: rx skb 86afc240 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 8 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1756.420000] ath10k_pci 0000:00:00.0: rx skb 86a7d480 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 9 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> [ 1757.420000] ath10k_pci 0000:00:00.0: rx skb 86af0180 len 116 peer >>> 04:f0:21:0c:a5:43 ucast sn 10 rate_idx 0 vht_nss 0 freq 5180 band 1 >>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>> >>> >>> On Tue, Dec 16, 2014 at 4:52 PM, Janusz Dziedzic >>> <janusz.dziedzic@tieto.com> wrote: >>>> This fix TX problem when IBSS used in HT mode. >>>> Before we used 6Mbps all the time for TX direction. >>>> >>>> Reported-by: Yeoh Chun-Yeow <yeohchunyeow@gmail.com> >>>> Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> >>>> --- >>>> drivers/net/wireless/ath/ath10k/mac.c | 7 +++++++ >>>> 1 file changed, 7 insertions(+) >>>> >>>> diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c >>>> index 5475f0f..c9e7995 100644 >>>> --- a/drivers/net/wireless/ath/ath10k/mac.c >>>> +++ b/drivers/net/wireless/ath/ath10k/mac.c >>>> @@ -1411,9 +1411,16 @@ static void ath10k_peer_assoc_h_qos(struct ath10k *ar, >>>> if (vif->bss_conf.qos) >>>> arg->peer_flags |= WMI_PEER_QOS; >>>> break; >>>> + case WMI_VDEV_TYPE_IBSS: >>>> + if (sta->wme) >>>> + arg->peer_flags |= WMI_PEER_QOS; >>>> + break; >>>> default: >>>> break; >>>> } >>>> + >>>> + ath10k_dbg(ar, ATH10K_DBG_MAC, "mac peer %pM qos %d\n", >>>> + sta->addr, !!(arg->peer_flags & WMI_PEER_QOS)); >>>> } >>>> >>>> static void ath10k_peer_assoc_h_phymode(struct ath10k *ar, >>>> -- >>>> 1.9.1 >>>> >>>> >>>> _______________________________________________ >>>> ath10k mailing list >>>> ath10k@lists.infradead.org >>>> http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] ath10k: fix low TX rates when IBSS and HT 2014-12-16 13:32 ` Yeoh Chun-Yeow @ 2014-12-17 3:56 ` Yeoh Chun-Yeow -1 siblings, 0 replies; 24+ messages in thread From: Yeoh Chun-Yeow @ 2014-12-17 3:56 UTC (permalink / raw) To: Janusz Dziedzic Cc: linux-wireless@vger.kernel.org, ath10k@lists.infradead.org Hi, Janusz I have figured out the problem, need the following patch to make sure the QoS is set which is not in my kernel. http://www.spinics.net/lists/linux-wireless/msg125241.html "sta->sta.wme = true" is needed in ibss.c. So applying these patches and it works, you can add "Tested-by" if you want. Thanks ---- Chun-Yeow On Tue, Dec 16, 2014 at 9:32 PM, Yeoh Chun-Yeow <yeohchunyeow@gmail.com> wrote: > Ok, I will try this and investigate further. > > --- > ChunYeow > > On Tue, Dec 16, 2014 at 7:45 PM, Janusz Dziedzic > <janusz.dziedzic@tieto.com> wrote: >> On 16 December 2014 at 12:22, Janusz Dziedzic <janusz.dziedzic@tieto.com> wrote: >>> On 16 December 2014 at 12:10, Yeoh Chun-Yeow <yeohchunyeow@gmail.com> wrote: >>>> Hi, Janusz >>>> >>>> I have applied the three patches and tested with firmware >>>> 999.999.0.636 but not working. >>>> >>>> Any advice what's wrong? >>>> >>> mac peer 04:f0:21:0c:a5:43 qos 0 - this is suspect. >>> What is your test procedure (I see some mesh ...)? >>> Maybe additional patch for mesh is required? >>> >>> I tested this using ath9k + ath10k (636) + >>> iw wlanx set type ibss >>> iw ibss join <> 5180 HT [HT40+] >>> or wpa_supplicant in ibss mode (HT20 limited in current version) >>> >>> Next in airlogs I saw we are using nss>1 and higher mcs >>> But for sure qos = 1 is required. >>> >> I am not sure why sta->wme is not set while we have HT rates (is it >> allowed to have HT without WMM?) >> But anyway as a quick workaroud you can try something like this: >> >> @@ -1212,7 +1212,7 @@ static void ath10k_peer_assoc_h_ht(struct ath10k *ar, >> if (!ht_cap->ht_supported) >> return; >> >> - arg->peer_flags |= WMI_PEER_HT; >> + arg->peer_flags |= WMI_PEER_HT | WMI_PEER_QOS; >> arg->peer_max_mpdu = (1 << (IEEE80211_HT_MAX_AMPDU_FACTOR + >> ht_cap->ampdu_factor)) - 1; >> >> @@ -3464,6 +3464,13 @@ static int ath10k_set_key(struct ieee80211_hw >> *hw, enum set_key_cmd cmd, >> key->cipher == WLAN_CIPHER_SUITE_WEP104; >> int ret = 0; >> >>> BR >>> Janusz >>> >>> >>>> --- >>>> ChunYeow >>>> >>>> Some of my dmesg: >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 3 value 34 >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 4 value 34 >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 >>>> peer_addr 04:f0:21:0c:a5:1c >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 16 value 0 >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 3 value 100 >>>> [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 beacon_interval 100 >>>> [ 640.450000] ath10k_pci 0000:00:00.0: vdev 0 set beacon tx mode to staggered >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 7 value 0 >>>> [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 start center_freq >>>> 5180 phymode 11na-ht20 >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev start id 0x0 flags: >>>> 0x0, freq 5180, mode 4, ch_flags: 0x4000000, max_power: 34 >>>> [ 640.460000] ath10k_pci 0000:00:00.0: WMI_VDEV_START_RESP_EVENTID >>>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi mgmt vdev up id 0x0 assoc >>>> id 0 bssid 52:8d:75:e5:00:52 >>>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 up >>>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 cts_prot 0 >>>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 0 >>>> [ 640.460000] ath10k_pci 0000:00:00.0: WMI_TBTTOFFSET_UPDATE_EVENTID >>>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 slot_time 2 >>>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 7 value 2 >>>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 preamble 1n >>>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 8 value 1 >>>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 peer create >>>> 04:f0:21:0c:a5:43 (new sta) sta 1 / 16 peer 2 / 24 >>>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 >>>> peer_addr 04:f0:21:0c:a5:43 >>>> [ 640.460000] IPv6: ADDRCONF(NETDEV_CHANGE): mesh0: link becomes ready >>>> [ 640.470000] ath10k_pci 0000:00:00.0: mac sta 04:f0:21:0c:a5:43 associated >>>> [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 >>>> [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 phymode 11a >>>> [ 640.470000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr >>>> 04:f0:21:0c:a5:43 (new) >>>> [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer >>>> 0x04:f0:21:0c:a5:43 set param 5 value 1 >>>> [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 33 >>>> [ 640.470000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 >>>> [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for >>>> 04:f0:21:0c:a5:43 changed 00000004 bw 0 nss 1 smps 2 >>>> [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for >>>> 04:f0:21:0c:a5:43 changed 00000008 bw 0 nss 3 smps 2 >>>> [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta 04:f0:21:0c:a5:43 nss 3 >>>> [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer >>>> 0x04:f0:21:0c:a5:43 set param 5 value 3 >>>> [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta >>>> 04:f0:21:0c:a5:43 supp rates/nss >>>> [ 640.540000] ath10k_pci 0000:00:00.0: mac ht peer 04:f0:21:0c:a5:43 >>>> mcs cnt 24 nss 3 >>>> [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 >>>> [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 >>>> phymode 11na-ht20 >>>> [ 640.540000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr >>>> 04:f0:21:0c:a5:43 (reassociate) >>>> [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer >>>> 0x04:f0:21:0c:a5:43 set param 5 value 3 >>>> [ 640.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 460 >>>> [ 641.400000] ath10k_pci 0000:00:00.0: rx skb 86be4b40 len 148 peer >>>> 04:f0:21:0c:a5:43 mcast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 642.170000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 >>>> [ 642.400000] ath10k_pci 0000:00:00.0: rx skb 86b49b40 len 108 peer >>>> 04:f0:21:0c:a5:43 mcast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 651.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 28 >>>> [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86af8540 len 60 peer >>>> 04:f0:21:0c:a5:43 mcast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86967600 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 0 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1741.550000] ath10k_pci 0000:00:00.0: rx skb 86967000 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 1 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1742.550000] ath10k_pci 0000:00:00.0: rx skb 86b8acc0 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1743.550000] ath10k_pci 0000:00:00.0: rx skb 86a7a600 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1746.560000] ath10k_pci 0000:00:00.0: rx skb 86a79cc0 len 60 peer >>>> 04:f0:21:0c:a5:43 ucast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1752.410000] ath10k_pci 0000:00:00.0: rx skb 86a9d540 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 5 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1753.410000] ath10k_pci 0000:00:00.0: rx skb 86a7c6c0 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 6 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1754.410000] ath10k_pci 0000:00:00.0: rx skb 86a7f6c0 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 7 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1755.420000] ath10k_pci 0000:00:00.0: rx skb 86afc240 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 8 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1756.420000] ath10k_pci 0000:00:00.0: rx skb 86a7d480 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 9 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1757.420000] ath10k_pci 0000:00:00.0: rx skb 86af0180 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 10 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> >>>> >>>> On Tue, Dec 16, 2014 at 4:52 PM, Janusz Dziedzic >>>> <janusz.dziedzic@tieto.com> wrote: >>>>> This fix TX problem when IBSS used in HT mode. >>>>> Before we used 6Mbps all the time for TX direction. >>>>> >>>>> Reported-by: Yeoh Chun-Yeow <yeohchunyeow@gmail.com> >>>>> Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> >>>>> --- >>>>> drivers/net/wireless/ath/ath10k/mac.c | 7 +++++++ >>>>> 1 file changed, 7 insertions(+) >>>>> >>>>> diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c >>>>> index 5475f0f..c9e7995 100644 >>>>> --- a/drivers/net/wireless/ath/ath10k/mac.c >>>>> +++ b/drivers/net/wireless/ath/ath10k/mac.c >>>>> @@ -1411,9 +1411,16 @@ static void ath10k_peer_assoc_h_qos(struct ath10k *ar, >>>>> if (vif->bss_conf.qos) >>>>> arg->peer_flags |= WMI_PEER_QOS; >>>>> break; >>>>> + case WMI_VDEV_TYPE_IBSS: >>>>> + if (sta->wme) >>>>> + arg->peer_flags |= WMI_PEER_QOS; >>>>> + break; >>>>> default: >>>>> break; >>>>> } >>>>> + >>>>> + ath10k_dbg(ar, ATH10K_DBG_MAC, "mac peer %pM qos %d\n", >>>>> + sta->addr, !!(arg->peer_flags & WMI_PEER_QOS)); >>>>> } >>>>> >>>>> static void ath10k_peer_assoc_h_phymode(struct ath10k *ar, >>>>> -- >>>>> 1.9.1 >>>>> >>>>> >>>>> _______________________________________________ >>>>> ath10k mailing list >>>>> ath10k@lists.infradead.org >>>>> http://lists.infradead.org/mailman/listinfo/ath10k _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] ath10k: fix low TX rates when IBSS and HT @ 2014-12-17 3:56 ` Yeoh Chun-Yeow 0 siblings, 0 replies; 24+ messages in thread From: Yeoh Chun-Yeow @ 2014-12-17 3:56 UTC (permalink / raw) To: Janusz Dziedzic Cc: ath10k@lists.infradead.org, linux-wireless@vger.kernel.org Hi, Janusz I have figured out the problem, need the following patch to make sure the QoS is set which is not in my kernel. http://www.spinics.net/lists/linux-wireless/msg125241.html "sta->sta.wme = true" is needed in ibss.c. So applying these patches and it works, you can add "Tested-by" if you want. Thanks ---- Chun-Yeow On Tue, Dec 16, 2014 at 9:32 PM, Yeoh Chun-Yeow <yeohchunyeow@gmail.com> wrote: > Ok, I will try this and investigate further. > > --- > ChunYeow > > On Tue, Dec 16, 2014 at 7:45 PM, Janusz Dziedzic > <janusz.dziedzic@tieto.com> wrote: >> On 16 December 2014 at 12:22, Janusz Dziedzic <janusz.dziedzic@tieto.com> wrote: >>> On 16 December 2014 at 12:10, Yeoh Chun-Yeow <yeohchunyeow@gmail.com> wrote: >>>> Hi, Janusz >>>> >>>> I have applied the three patches and tested with firmware >>>> 999.999.0.636 but not working. >>>> >>>> Any advice what's wrong? >>>> >>> mac peer 04:f0:21:0c:a5:43 qos 0 - this is suspect. >>> What is your test procedure (I see some mesh ...)? >>> Maybe additional patch for mesh is required? >>> >>> I tested this using ath9k + ath10k (636) + >>> iw wlanx set type ibss >>> iw ibss join <> 5180 HT [HT40+] >>> or wpa_supplicant in ibss mode (HT20 limited in current version) >>> >>> Next in airlogs I saw we are using nss>1 and higher mcs >>> But for sure qos = 1 is required. >>> >> I am not sure why sta->wme is not set while we have HT rates (is it >> allowed to have HT without WMM?) >> But anyway as a quick workaroud you can try something like this: >> >> @@ -1212,7 +1212,7 @@ static void ath10k_peer_assoc_h_ht(struct ath10k *ar, >> if (!ht_cap->ht_supported) >> return; >> >> - arg->peer_flags |= WMI_PEER_HT; >> + arg->peer_flags |= WMI_PEER_HT | WMI_PEER_QOS; >> arg->peer_max_mpdu = (1 << (IEEE80211_HT_MAX_AMPDU_FACTOR + >> ht_cap->ampdu_factor)) - 1; >> >> @@ -3464,6 +3464,13 @@ static int ath10k_set_key(struct ieee80211_hw >> *hw, enum set_key_cmd cmd, >> key->cipher == WLAN_CIPHER_SUITE_WEP104; >> int ret = 0; >> >>> BR >>> Janusz >>> >>> >>>> --- >>>> ChunYeow >>>> >>>> Some of my dmesg: >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 3 value 34 >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 4 value 34 >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set wmm params >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 >>>> peer_addr 04:f0:21:0c:a5:1c >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 16 value 0 >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 3 value 100 >>>> [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 beacon_interval 100 >>>> [ 640.450000] ath10k_pci 0000:00:00.0: vdev 0 set beacon tx mode to staggered >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi pdev set param 7 value 0 >>>> [ 640.450000] ath10k_pci 0000:00:00.0: mac vdev 0 start center_freq >>>> 5180 phymode 11na-ht20 >>>> [ 640.450000] ath10k_pci 0000:00:00.0: wmi vdev start id 0x0 flags: >>>> 0x0, freq 5180, mode 4, ch_flags: 0x4000000, max_power: 34 >>>> [ 640.460000] ath10k_pci 0000:00:00.0: WMI_VDEV_START_RESP_EVENTID >>>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi mgmt vdev up id 0x0 assoc >>>> id 0 bssid 52:8d:75:e5:00:52 >>>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 up >>>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 cts_prot 0 >>>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 0 >>>> [ 640.460000] ath10k_pci 0000:00:00.0: WMI_TBTTOFFSET_UPDATE_EVENTID >>>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 slot_time 2 >>>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 7 value 2 >>>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 preamble 1n >>>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 8 value 1 >>>> [ 640.460000] ath10k_pci 0000:00:00.0: mac vdev 0 peer create >>>> 04:f0:21:0c:a5:43 (new sta) sta 1 / 16 peer 2 / 24 >>>> [ 640.460000] ath10k_pci 0000:00:00.0: wmi peer create vdev_id 0 >>>> peer_addr 04:f0:21:0c:a5:43 >>>> [ 640.460000] IPv6: ADDRCONF(NETDEV_CHANGE): mesh0: link becomes ready >>>> [ 640.470000] ath10k_pci 0000:00:00.0: mac sta 04:f0:21:0c:a5:43 associated >>>> [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 >>>> [ 640.470000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 phymode 11a >>>> [ 640.470000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr >>>> 04:f0:21:0c:a5:43 (new) >>>> [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer >>>> 0x04:f0:21:0c:a5:43 set param 5 value 1 >>>> [ 640.470000] ath10k_pci 0000:00:00.0: wmi vdev id 0x0 set param 44 value 33 >>>> [ 640.470000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 >>>> [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for >>>> 04:f0:21:0c:a5:43 changed 00000004 bw 0 nss 1 smps 2 >>>> [ 640.540000] ath10k_pci 0000:00:00.0: mac sta rc update for >>>> 04:f0:21:0c:a5:43 changed 00000008 bw 0 nss 3 smps 2 >>>> [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta 04:f0:21:0c:a5:43 nss 3 >>>> [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer >>>> 0x04:f0:21:0c:a5:43 set param 5 value 3 >>>> [ 640.540000] ath10k_pci 0000:00:00.0: mac update sta >>>> 04:f0:21:0c:a5:43 supp rates/nss >>>> [ 640.540000] ath10k_pci 0000:00:00.0: mac ht peer 04:f0:21:0c:a5:43 >>>> mcs cnt 24 nss 3 >>>> [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 qos 0 >>>> [ 640.540000] ath10k_pci 0000:00:00.0: mac peer 04:f0:21:0c:a5:43 >>>> phymode 11na-ht20 >>>> [ 640.540000] ath10k_pci 0000:00:00.0: wmi peer assoc vdev 0 addr >>>> 04:f0:21:0c:a5:43 (reassociate) >>>> [ 640.540000] ath10k_pci 0000:00:00.0: wmi vdev 0 peer >>>> 0x04:f0:21:0c:a5:43 set param 5 value 3 >>>> [ 640.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 460 >>>> [ 641.400000] ath10k_pci 0000:00:00.0: rx skb 86be4b40 len 148 peer >>>> 04:f0:21:0c:a5:43 mcast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 642.170000] ath10k_pci 0000:00:00.0: mac monitor recalc started? 0 should? 0 >>>> [ 642.400000] ath10k_pci 0000:00:00.0: rx skb 86b49b40 len 108 peer >>>> 04:f0:21:0c:a5:43 mcast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 651.560000] ath10k_pci 0000:00:00.0: wmi event debug mesg len 28 >>>> [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86af8540 len 60 peer >>>> 04:f0:21:0c:a5:43 mcast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1740.540000] ath10k_pci 0000:00:00.0: rx skb 86967600 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 0 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1741.550000] ath10k_pci 0000:00:00.0: rx skb 86967000 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 1 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1742.550000] ath10k_pci 0000:00:00.0: rx skb 86b8acc0 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 2 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1743.550000] ath10k_pci 0000:00:00.0: rx skb 86a7a600 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 3 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1746.560000] ath10k_pci 0000:00:00.0: rx skb 86a79cc0 len 60 peer >>>> 04:f0:21:0c:a5:43 ucast sn 4 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1752.410000] ath10k_pci 0000:00:00.0: rx skb 86a9d540 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 5 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1753.410000] ath10k_pci 0000:00:00.0: rx skb 86a7c6c0 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 6 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1754.410000] ath10k_pci 0000:00:00.0: rx skb 86a7f6c0 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 7 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1755.420000] ath10k_pci 0000:00:00.0: rx skb 86afc240 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 8 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1756.420000] ath10k_pci 0000:00:00.0: rx skb 86a7d480 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 9 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> [ 1757.420000] ath10k_pci 0000:00:00.0: rx skb 86af0180 len 116 peer >>>> 04:f0:21:0c:a5:43 ucast sn 10 rate_idx 0 vht_nss 0 freq 5180 band 1 >>>> flag 0x200000 fcs-err 0 mic-err 0 amsdu-more 0 >>>> >>>> >>>> On Tue, Dec 16, 2014 at 4:52 PM, Janusz Dziedzic >>>> <janusz.dziedzic@tieto.com> wrote: >>>>> This fix TX problem when IBSS used in HT mode. >>>>> Before we used 6Mbps all the time for TX direction. >>>>> >>>>> Reported-by: Yeoh Chun-Yeow <yeohchunyeow@gmail.com> >>>>> Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> >>>>> --- >>>>> drivers/net/wireless/ath/ath10k/mac.c | 7 +++++++ >>>>> 1 file changed, 7 insertions(+) >>>>> >>>>> diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c >>>>> index 5475f0f..c9e7995 100644 >>>>> --- a/drivers/net/wireless/ath/ath10k/mac.c >>>>> +++ b/drivers/net/wireless/ath/ath10k/mac.c >>>>> @@ -1411,9 +1411,16 @@ static void ath10k_peer_assoc_h_qos(struct ath10k *ar, >>>>> if (vif->bss_conf.qos) >>>>> arg->peer_flags |= WMI_PEER_QOS; >>>>> break; >>>>> + case WMI_VDEV_TYPE_IBSS: >>>>> + if (sta->wme) >>>>> + arg->peer_flags |= WMI_PEER_QOS; >>>>> + break; >>>>> default: >>>>> break; >>>>> } >>>>> + >>>>> + ath10k_dbg(ar, ATH10K_DBG_MAC, "mac peer %pM qos %d\n", >>>>> + sta->addr, !!(arg->peer_flags & WMI_PEER_QOS)); >>>>> } >>>>> >>>>> static void ath10k_peer_assoc_h_phymode(struct ath10k *ar, >>>>> -- >>>>> 1.9.1 >>>>> >>>>> >>>>> _______________________________________________ >>>>> ath10k mailing list >>>>> ath10k@lists.infradead.org >>>>> http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] ath10k: fix low TX rates when IBSS and HT 2014-12-16 11:22 ` Janusz Dziedzic @ 2014-12-16 13:32 ` Yeoh Chun-Yeow -1 siblings, 0 replies; 24+ messages in thread From: Yeoh Chun-Yeow @ 2014-12-16 13:32 UTC (permalink / raw) To: Janusz Dziedzic Cc: linux-wireless@vger.kernel.org, ath10k@lists.infradead.org On Tue, Dec 16, 2014 at 7:22 PM, Janusz Dziedzic <janusz.dziedzic@tieto.com> wrote: > On 16 December 2014 at 12:10, Yeoh Chun-Yeow <yeohchunyeow@gmail.com> wrote: >> Hi, Janusz >> >> I have applied the three patches and tested with firmware >> 999.999.0.636 but not working. >> >> Any advice what's wrong? >> > mac peer 04:f0:21:0c:a5:43 qos 0 - this is suspect. Not so sure. > What is your test procedure (I see some mesh ...)? I named the interface as mesh0 but it is ibss or adhoc mode. I use the iw util to setup the adhoc interface. --- ChunYeow _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] ath10k: fix low TX rates when IBSS and HT @ 2014-12-16 13:32 ` Yeoh Chun-Yeow 0 siblings, 0 replies; 24+ messages in thread From: Yeoh Chun-Yeow @ 2014-12-16 13:32 UTC (permalink / raw) To: Janusz Dziedzic Cc: ath10k@lists.infradead.org, linux-wireless@vger.kernel.org On Tue, Dec 16, 2014 at 7:22 PM, Janusz Dziedzic <janusz.dziedzic@tieto.com> wrote: > On 16 December 2014 at 12:10, Yeoh Chun-Yeow <yeohchunyeow@gmail.com> wrote: >> Hi, Janusz >> >> I have applied the three patches and tested with firmware >> 999.999.0.636 but not working. >> >> Any advice what's wrong? >> > mac peer 04:f0:21:0c:a5:43 qos 0 - this is suspect. Not so sure. > What is your test procedure (I see some mesh ...)? I named the interface as mesh0 but it is ibss or adhoc mode. I use the iw util to setup the adhoc interface. --- ChunYeow ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] ath10k: fix low TX rates when IBSS and HT 2014-12-16 8:52 ` Janusz Dziedzic @ 2014-12-23 15:23 ` Kalle Valo -1 siblings, 0 replies; 24+ messages in thread From: Kalle Valo @ 2014-12-23 15:23 UTC (permalink / raw) To: Janusz Dziedzic; +Cc: linux-wireless, ath10k Janusz Dziedzic <janusz.dziedzic@tieto.com> writes: > This fix TX problem when IBSS used in HT mode. > Before we used 6Mbps all the time for TX direction. > > Reported-by: Yeoh Chun-Yeow <yeohchunyeow@gmail.com> > Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> Thanks, both patches applied. -- Kalle Valo _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] ath10k: fix low TX rates when IBSS and HT @ 2014-12-23 15:23 ` Kalle Valo 0 siblings, 0 replies; 24+ messages in thread From: Kalle Valo @ 2014-12-23 15:23 UTC (permalink / raw) To: Janusz Dziedzic; +Cc: ath10k, linux-wireless Janusz Dziedzic <janusz.dziedzic@tieto.com> writes: > This fix TX problem when IBSS used in HT mode. > Before we used 6Mbps all the time for TX direction. > > Reported-by: Yeoh Chun-Yeow <yeohchunyeow@gmail.com> > Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com> Thanks, both patches applied. -- Kalle Valo ^ permalink raw reply [flat|nested] 24+ messages in thread
end of thread, other threads:[~2014-12-23 15:24 UTC | newest] Thread overview: 24+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-12-16 8:52 [PATCH 1/2] ath10k: fix low TX rates when IBSS and HT Janusz Dziedzic 2014-12-16 8:52 ` Janusz Dziedzic 2014-12-16 8:53 ` [PATCH 2/2] ath10k: send (re)assoc peer command when NSS changed Janusz Dziedzic 2014-12-16 8:53 ` Janusz Dziedzic 2014-12-17 10:32 ` Kalle Valo 2014-12-17 10:32 ` Kalle Valo 2014-12-18 5:28 ` Janusz Dziedzic 2014-12-18 5:28 ` Janusz Dziedzic 2014-12-18 6:02 ` Kalle Valo 2014-12-18 6:02 ` Kalle Valo 2014-12-16 11:10 ` [PATCH 1/2] ath10k: fix low TX rates when IBSS and HT Yeoh Chun-Yeow 2014-12-16 11:10 ` Yeoh Chun-Yeow 2014-12-16 11:22 ` Janusz Dziedzic 2014-12-16 11:22 ` Janusz Dziedzic 2014-12-16 11:45 ` Janusz Dziedzic 2014-12-16 11:45 ` Janusz Dziedzic 2014-12-16 13:32 ` Yeoh Chun-Yeow 2014-12-16 13:32 ` Yeoh Chun-Yeow 2014-12-17 3:56 ` Yeoh Chun-Yeow 2014-12-17 3:56 ` Yeoh Chun-Yeow 2014-12-16 13:32 ` Yeoh Chun-Yeow 2014-12-16 13:32 ` Yeoh Chun-Yeow 2014-12-23 15:23 ` Kalle Valo 2014-12-23 15:23 ` Kalle Valo
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.