Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mac80211: silence mesh rate mismatch warning #3416

Merged

Conversation

blocktrron
Copy link
Member

Printing a warning everytime a rate mismatch is detected is a bit to much. With many mesh neighbors, this quickly results in the kernel log being spammed with these messages.

Printing a warning everytime a rate mismatch is detected is a bit to
much. With many mesh neighbors, this quickly results in the kernel log
being spammed with these messages.

Signed-off-by: David Bauer <[email protected]>
@maurerle
Copy link
Member

maurerle commented Jan 8, 2025

I have a node with 2 mesh partners, which is directly spammed very much with these records - like 30 messages per second.

ignoring basic rate mismatch for peer (local=40 peer=1)

Is this normal without silencing it?

What is needed to fix the mismatch warning? If both covr-x1860 are using latest gluon main - they should not have this warning anymore right?
Do you need to restart wifi or reboot or something to fix this, after both have the main version and the third router without it is removed?

@maurerle
Copy link
Member

maurerle commented Jan 8, 2025

reboot did not help - there are only two COVR-X1860 seeing each other with gluon-main firmware.
Both are spamming the ignoring basic rate mismatch for peer (local=40 peer=1) message.

This looks like both would send with 40, but the peer receives a mesh rate of 1 in both directions..?

+ ieee80211_sta_get_rates(sdata, ie, sband->band,
+ &basic_rates);
+
+- if (sdata->vif.bss_conf.basic_rates != basic_rates)
+- return false;
++ if (sdata->vif.bss_conf.basic_rates != basic_rates) {
++ wiphy_warn(sdata->wdev.wiphy,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Putting the warning in the first place was a bad idea, rate-limiting has no point in that.

@blocktrron blocktrron merged commit 0b23f89 into freifunk-gluon:main Jan 8, 2025
40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants