On Montag, 6. November 2017 09:28:42 CET Sebastian Gottschall wrote: > Am 06.11.2017 um 09:23 schrieb Sven Eckelmann: > > On Sonntag, 5. November 2017 10:22:22 CET Sebastian Gottschall wrote: > >> the assumption made in this patch is obviously wrong (at least for more > >> recent firmwares and 9984) > >> my log is flooded with messages like > >> [208802.803537] ath10k_pci 0001:03:00.0: Invalid VHT mcs 15 peer stats > >> [208805.108515] ath10k_pci 0001:03:00.0: Invalid VHT mcs 15 peer stats > >> [208821.747621] ath10k_pci 0001:03:00.0: Invalid VHT mcs 15 peer stats > >> [208822.516599] ath10k_pci 0001:03:00.0: Invalid VHT mcs 15 peer stats > >> [208841.257780] ath10k_pci 0001:03:00.0: Invalid VHT mcs 15 peer stats [...] > > This patch only splits WMI_RATE_PREAMBLE_HT & WMI_RATE_PREAMBLE_VHT. And for > > WMI_RATE_PREAMBLE_HT (*not VHT*), it uses a slightly different approach. But > > the WMI_RATE_PREAMBLE_VHT part (which you see in your logs) is basically > > untouched. > > then a question follows up. is this check really neccessary? Until we find out what the heck VHT MCS 15 should mean in this context - maybe. But to the message - no, the message is most likely not necessary for each received "invalid" peer tx stat. Kind regards. Sven
Attachment:
signature.asc
Description: This is a digitally signed message part.
