F5/unison
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

wifi: Iterate over the in-flight MPDUs in reverse order

Browse Source 
A newly added MPDU is likely to have a Sequence Number higher than the
previous ones, hence it likely has to be inserted at the end of the list.
This commit is contained in:
Stefano Avallone
2021-10-01 13:46:22 +02:00
parent cf8ef91a29
commit aba6227449
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/wifi/model/block-ack-manager.cc
View File

@@ -236,8 +236,8 @@ BlockAckManager::StorePacket (Ptr<WifiMacQueueItem> mpdu)
// store the packet and keep the list sorted in increasing order of sequence number
// with respect to the starting sequence number
PacketQueueI it = agreementIt->second.second.begin ();
while (it != agreementIt->second.second.end ())
auto it = agreementIt->second.second.rbegin ();
while (it != agreementIt->second.second.rend ())
{
if (mpdu->GetHeader ().GetSequenceControl () == (*it)->GetHeader ().GetSequenceControl ())
{
@@ -247,15 +247,15 @@ BlockAckManager::StorePacket (Ptr<WifiMacQueueItem> mpdu)
uint16_t dist = agreementIt->second.first.GetDistance ((*it)->GetHeader ().GetSequenceNumber ());
if (mpduDist < dist ||
(mpduDist == dist && mpdu->GetHeader ().GetFragmentNumber () < (*it)->GetHeader ().GetFragmentNumber ()))
if (mpduDist > dist ||
(mpduDist == dist && mpdu->GetHeader ().GetFragmentNumber () > (*it)->GetHeader ().GetFragmentNumber ()))
{
break;
}
it++;
}
agreementIt->second.second.insert (it, mpdu);
agreementIt->second.second.insert (std::prev (it.base ()), mpdu);
agreementIt->second.first.NotifyTransmittedMpdu (mpdu);
mpdu->SetInFlight ();
}