Page MenuHomeFreeBSD
Differential D47776

rtwn: enable receiving AMSDU in AMPDU
ClosedPublic
Actions

Authored by adrian on Nov 27 2024, 4:40 AM.
Tags
Referenced Files
Unknown Object (File)
Fri, Feb 27, 2:58 PM
Unknown Object (File)
Fri, Feb 27, 10:37 AM
Unknown Object (File)
Sun, Feb 8, 1:17 PM
Unknown Object (File)
Sun, Feb 8, 8:21 AM
Unknown Object (File)
Sat, Feb 7, 8:08 PM
Unknown Object (File)
Fri, Jan 30, 9:34 PM
Unknown Object (File)
Jan 28 2026, 5:57 PM
Unknown Object (File)
Jan 23 2026, 6:15 AM
View All 81 Files
Subscribers
bz
imp

Details

Reviewers
None
Group Reviewers
wireless
Commits
rG05c3851b20e0: rtwn: enable receiving AMSDU in AMPDU
Summary

All of the supported NICs should support this.

Locally tested:

  • RTL8192CU, STA mode
  • RTL8192EU, STA mode
  • RTL8821AU, STA mode
  • RTL8812AU, STA mode

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 60806
Build 57690: arc lint + arc unit

Event Timeline

adrian created this revision.Nov 27 2024, 4:40 AM
Herald added a subscriber: imp. · View Herald TranscriptNov 27 2024, 4:40 AM
adrian requested review of this revision.Nov 27 2024, 4:40 AM
Harbormaster completed remote builds in B60805: Diff 147023.Nov 27 2024, 4:40 AM
adrian updated this revision to Diff 147024.Nov 27 2024, 4:40 AM

url

Harbormaster completed remote builds in B60806: Diff 147024.Nov 27 2024, 4:40 AM
adrian added a parent revision: D47775: rtwn: enable FCS in the recive config to work around truncated frames.Nov 27 2024, 4:41 AM
adrian added a reviewer: wireless.
adrian added a project: wireless.
bz added a subscriber: bz.Nov 28 2024, 1:15 AM

Likely fine but should make sure that all the different devices/firmware really can do that.

adrian added a comment.Dec 1 2024, 7:20 PM
In D47776#1090169, @bz wrote:

Likely fine but should make sure that all the different devices/firmware really can do that.

I've checked; RTL8188 and RTL8192 NICs support it fine. There's some other instabilities, but not linked to this!

adrian added a child revision: D47860: rtwn: remove the conditional compilation around the sc_ht40 option..Dec 2 2024, 5:32 AM
This revision was not accepted when it landed; it landed in state Needs Review.Dec 5 2024, 7:31 AM
Closed by commit rG05c3851b20e0: rtwn: enable receiving AMSDU in AMPDU (authored by adrian). · Explain Why
This revision was automatically updated to reflect the committed changes.
adrian added a commit: rG05c3851b20e0: rtwn: enable receiving AMSDU in AMPDU.

Revision Contents
Changeset List

PathSize
sys/
dev/
rtwn/
1 line

Diff 147024

View Options

sys/dev/rtwn/if_rtwn.c

Loading...