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: ath10k: increase rx buffer size to 2048 #1992

Merged
merged 1 commit into from
Apr 25, 2020

Conversation

ecsv
Copy link
Contributor

@ecsv ecsv commented Apr 25, 2020

Before, only frames with a maximum size of 1528 bytes could be transmitted between two 802.11s nodes.

For batman-adv for instance, which adds its own header to each frame, we typically need an MTU of at least 1532 bytes to be able to transmit without fragmentation.

This patch now increases the maxmimum frame size from 1528 to 1656 bytes.

Tested with two ath10k devices in 802.11s mode, as well as with batman-adv on top of 802.11s with forwarding disabled.

Fix originally found and developed by @greearb and is already part of ath10k-ct

Link: greearb/ath10k-ct#89
Link: greearb/ath10k-ct@9e5ab25
Forwarded: openwrt/openwrt#2959

Before, only frames with a maximum size of 1528 bytes could be
transmitted between two 802.11s nodes.

For batman-adv for instance, which adds its own header to each frame,
we typically need an MTU of at least 1532 bytes to be able to transmit
without fragmentation.

This patch now increases the maxmimum frame size from 1528 to 1656
bytes.

Tested with two ath10k devices in 802.11s mode, as well as with
batman-adv on top of 802.11s with forwarding disabled.

Fix originally found and developed by Ben Greear.
@mweinelt mweinelt added the 0. type: enhancement The changeset is an enhancement label Apr 25, 2020
@mweinelt
Copy link
Contributor

Tested and LGTM. Allows ping with payloads > 1466 Bytes.

@mweinelt mweinelt merged commit 8bc602b into freifunk-gluon:master Apr 25, 2020
@mweinelt mweinelt added this to the 2020.2 milestone Apr 27, 2020
@mweinelt mweinelt added 4. has: upstream-patches This changeset backports upstream patches 5. needs: stable backport The changeset need to be backported to supported stable releases labels Apr 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. type: enhancement The changeset is an enhancement 4. has: upstream-patches This changeset backports upstream patches 5. needs: stable backport The changeset need to be backported to supported stable releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants