Bug 2779 – Beacon jitter not enabled by default

Bug 2779 - Beacon jitter not enabled by default


Summary:	Beacon jitter not enabled by default

Status:	RESOLVED FIXED

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

Importance:	P5 enhancement
Assigned To:	sebastien.deronne

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2017-08-31 09:38 UTC by Rediet
Modified:	2017-09-13 17:14 UTC (History)
CC List:	2 users (show)

See Also:

Attachments
Simple patch (662 bytes, patch) 2017-08-31 09:38 UTC, Rediet	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 Rediet 2017-08-31 09:38:53 UTC

Created attachment 2910 [details]
Simple patch

Beacon jitter is disabled by default. This is transparent when setting up one BSS, but when deploying multiple BSSs it might have some side effects (see discussion on https://groups.google.com/forum/#!topic/ns-3-users/7L1ayb5bzR8).
It should thus be better to enable it by default, so as to avoid confusing users.

Comment 1 sebastien.deronne 2017-09-13 13:23:30 UTC

OK, I will push to the mainstream.

Comment 2 sebastien.deronne 2017-09-13 15:19:36 UTC

This introduces regression problems because of the randomness that is now introduced. I suggest to remove this randomness in failing tests.

Comment 3 Tom Henderson 2017-09-13 15:35:56 UTC

(In reply to sebastien.deronne from comment #2)
> This introduces regression problems because of the randomness that is now
> introduced. I suggest to remove this randomness in failing tests.

Agreed.

Comment 4 sebastien.deronne 2017-09-13 17:14:01 UTC

changeset 13059:037e9046f940