Page MenuHomeFreeBSD
Differential D47776

rtwn: enable receiving AMSDU in AMPDU
ClosedPublic
Actions

Authored by adrian on Wed, Nov 27, 4:40 AM.
Tags
Referenced Files
Unknown Object (File)
Thu, Dec 5, 8:28 AM
Unknown Object (File)
Wed, Dec 4, 10:26 AM
Unknown Object (File)
Tue, Dec 3, 9:27 PM
Unknown Object (File)
Sun, Dec 1, 5:49 AM
Unknown Object (File)
Sun, Dec 1, 12:58 AM
Unknown Object (File)
Sun, Dec 1, 12:55 AM
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.Wed, Nov 27, 4:40 AM
Herald added a subscriber: imp. · View Herald TranscriptWed, Nov 27, 4:40 AM
adrian requested review of this revision.Wed, Nov 27, 4:40 AM
Harbormaster completed remote builds in B60805: Diff 147023.Wed, Nov 27, 4:40 AM
adrian updated this revision to Diff 147024.Wed, Nov 27, 4:40 AM

url

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

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

adrian added a comment.Sun, Dec 1, 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..Mon, Dec 2, 5:32 AM
This revision was not accepted when it landed; it landed in state Needs Review.Thu, Dec 5, 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...