Bug 2271 – Decision to enable RTS/CTS handshake for A-MPDUs should be based on the comparison between RtsCtsThreshold and the total A-MPDU size

Bug 2271 - Decision to enable RTS/CTS handshake for A-MPDUs should be based on the comparison between RtsCtsThreshold and the total A-MPDU size


Summary:	Decision to enable RTS/CTS handshake for A-MPDUs should be based on the compa...

Status:	RESOLVED FIXED

Product:	ns-3
Classification:	Unclassified
Component:	wifi
Version:	pre-release
Hardware:	All All

Importance:	P5 minor
Assigned To:	sebastien.deronne

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2016-01-19 16:39 UTC by sebastien.deronne
Modified:	2016-02-05 14:22 UTC (History)
CC List:	1 user (show)

See Also:

Attachments
proposed changes (11.83 KB, patch) 2016-01-19 16:39 UTC, sebastien.deronne	Details \| Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description sebastien.deronne 2016-01-19 16:39:05 UTC

Created attachment 2243 [details]
proposed changes

According to the standard,the RTS threshold should apply to the PSDU, so the check should be moved to MacLow so that it compares the threshold against the total A-MPDU size. Furthermore, the description provided in the attribute should be adapted.

Note that the description for the fragmentation threshold attribute should specify that it does not apply to A-MPDU (but this is already OK with the current implementation).

Comment 1 sebastien.deronne 2016-01-31 14:44:11 UTC

I plan to deliver those changes in the short term since the fix is quite obvious and was already agreed upfront by Ghada.

Comment 2 sebastien.deronne 2016-02-05 14:22:27 UTC

pushed in changeset 11853:dfae11557a03