Skip to content

ieee80211: use response rate for fragmented-frame ACK duration - #1200

Open
mgonzalezlopezudc wants to merge 1 commit into
inet-framework:masterfrom
mgonzalezlopezudc:fix/ieee80211-audit-fragment-ack-rate
Open

mgonzalezlopezudc wants to merge 1 commit into
inet-framework:masterfrom
mgonzalezlopezudc:fix/ieee80211-audit-fragment-ack-rate

Conversation

@mgonzalezlopezudc

Copy link
Copy Markdown
Contributor

Summary

A fragmented DCF frame reserved its next ACK at the data rate. Select the next fragment’s response ACK mode; the 54/24-Mbps case now reserves the additional four microseconds.

Scope

Originator protection mechanism; no configuration change.

Dependencies and merge order

No prerequisite within this audit series; this topic was built and tested independently on the pinned base.

Validation

Pinned test base: 7287f347aaca36e557c708d6930a6f5a450b2833.

The isolated topic head passed an assertions-enabled debug build (-O0 -Wall -g0; debug symbols omitted for storage) and the focused checks below. The larger integration source tree also passed debug and release compilation; no separate release build is claimed for this topic.

Executed focused test commands, from the repository root after source setenv -q:

inet_run_module_tests -m debug -f 'Ieee80211FragmentAckDuration_1\.test'

Results: 1 module cases passed.

Limits and review status

Independent of fractional Duration rounding.

Draft pending final independent review and reconciliation with current upstream. Live PR overlap has not been reverified against the pinned test base.

Reserve the ACK following the next DCF fragment using its response mode. A production exchange at 54 Mbps with 24 Mbps ACKs now reserves the missing four microseconds. The focused module test and all four fragmentation fingerprints pass.

Change: src.ieee80211.duration | behavior.change.fix | test | wifi-audit
@mgonzalezlopezudc
mgonzalezlopezudc marked this pull request as ready for review September 15, 2026 23:28
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.

1 participant