Bug 2836

Summary:	Missing VHT information in radiotap header when A-MPDU is used
Product:	ns-3	Reporter:	sebastien.deronne
Component:	wifi	Assignee:	sebastien.deronne
Status:	RESOLVED FIXED
Severity:	minor	CC:	ns-bugs, tomh
Priority:	P3
Version:	ns-3.27
Hardware:	All
OS:	All

Description sebastien.deronne 2017-12-05 16:44:42 UTC

When 802.11ac is used and A-MPDU is enabled, I noticed the radiotap header does not contain the VHT information fields.

Comment 1 sebastien.deronne 2017-12-05 16:45:12 UTC

Solution to fix:

             header.SetAmpduStatus (aMpdu.mpduRefNumber, ampduStatusFlags, hdr.GetCrc ());
           }
 
-        if (preamble == WIFI_PREAMBLE_VHT)
+        if (txVector.GetMode ().GetModulationClass () == WIFI_MOD_CLASS_VHT)
           {
             uint16_t vhtKnown = RadiotapHeader::VHT_KNOWN_NONE;
             uint8_t vhtFlags = RadiotapHeader::VHT_FLAGS_NONE;

Comment 2 Tom Henderson 2017-12-05 16:50:38 UTC

(In reply to sebastien.deronne from comment #1)
> Solution to fix:
> 
>              header.SetAmpduStatus (aMpdu.mpduRefNumber, ampduStatusFlags,
> hdr.GetCrc ());
>            }
>  
> -        if (preamble == WIFI_PREAMBLE_VHT)
> +        if (txVector.GetMode ().GetModulationClass () == WIFI_MOD_CLASS_VHT)
>            {
>              uint16_t vhtKnown = RadiotapHeader::VHT_KNOWN_NONE;
>              uint8_t vhtFlags = RadiotapHeader::VHT_FLAGS_NONE;

Looks sufficient to push immediately unless you are waiting for some community input.

Comment 3 sebastien.deronne 2017-12-05 16:52:17 UTC

(In reply to Tom Henderson from comment #2)
> (In reply to sebastien.deronne from comment #1)
> > Solution to fix:
> > 
> >              header.SetAmpduStatus (aMpdu.mpduRefNumber, ampduStatusFlags,
> > hdr.GetCrc ());
> >            }
> >  
> > -        if (preamble == WIFI_PREAMBLE_VHT)
> > +        if (txVector.GetMode ().GetModulationClass () == WIFI_MOD_CLASS_VHT)
> >            {
> >              uint16_t vhtKnown = RadiotapHeader::VHT_KNOWN_NONE;
> >              uint8_t vhtFlags = RadiotapHeader::VHT_FLAGS_NONE;
> 
> Looks sufficient to push immediately unless you are waiting for some
> community input.

I will push it.

Comment 4 sebastien.deronne 2017-12-05 17:07:36 UTC

Pushed in changeset 13198:44744f91e4cf