Bug 2848 - Association ID not correctly set upon association
Association ID not correctly set upon association
Status: RESOLVED FIXED
Product: ns-3
Classification: Unclassified
Component: wifi
ns-3.27
All All
: P3 minor
Assigned To: sebastien.deronne
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2018-01-11 04:02 UTC by Rediet
Modified: 2018-01-13 06:19 UTC (History)
1 user (show)

See Also:


Attachments
Patch without tests (wrt changeset 13245) (11.64 KB, patch)
2018-01-11 04:02 UTC, Rediet
Details | Diff
patch (8.43 KB, patch)
2018-01-11 17:46 UTC, sebastien.deronne
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Rediet 2018-01-11 04:02:17 UTC
Created attachment 2995 [details]
Patch without tests (wrt changeset 13245)

Currently the association ID (AID) returned by the AP is always set to 0. This patch aims at setting the correct AID (in an incremental manner) and maintaining a register of all MAC@-AID pairs within the AP.
Unit-tests will follow.
Comment 1 sebastien.deronne 2018-01-11 04:24:57 UTC
Fixed in PCF repository, I'll push the solution to ns-3-dev.
Comment 2 Rediet 2018-01-11 08:00:07 UTC
OK (it's a good thing that I didn't spend too much time trying to set up the tests then :)).
Would it be possible to make it independent from PCF-related procedures? When moving to HE MU, we will need to have something equivalent to m_cfPollingList in order to get the STA-ID from the MAC@ and inversely.
Comment 3 sebastien.deronne 2018-01-11 08:08:57 UTC
(In reply to Rediet from comment #2)
> OK (it's a good thing that I didn't spend too much time trying to set up the
> tests then :)).
> Would it be possible to make it independent from PCF-related procedures?
> When moving to HE MU, we will need to have something equivalent to
> m_cfPollingList in order to get the STA-ID from the MAC@ and inversely.

Yes I will look at it.
Comment 4 sebastien.deronne 2018-01-11 16:06:42 UTC
I had not seen your patch, I will have a look.
Comment 5 sebastien.deronne 2018-01-11 17:46:33 UTC
Created attachment 2997 [details]
patch

I merged and reworked our both patches. I think it is ready to be pushed.
Comment 6 Rediet 2018-01-12 03:07:22 UTC
OK for me. Thanks for the quick replies.
Comment 7 sebastien.deronne 2018-01-13 06:19:09 UTC
Pushed in changeset 13251:51268ff4614e