Skip to content

Commit

Permalink
Revert "netkvm: do not complete packets in sending path"
Browse files Browse the repository at this point in the history
https://bugzilla.redhat.com/show_bug.cgi?id=2067167
https://bugzilla.redhat.com/show_bug.cgi?id=2063594

This reverts commit 5fcd9ea.
Mentioned commit was intended to solve the problem
https://bugzilla.redhat.com/show_bug.cgi?id=1972487

But during later tests it was found that the commit has triggered
other problems in perf6 and mpe tests
We revert the commit and continue the investigation

Signed-off-by: Yuri Benditovich <yuri.benditovich@daynix.com>
  • Loading branch information
ybendito authored and YanVugenfirer committed Apr 6, 2022
1 parent 402fc56 commit 315c637
Showing 1 changed file with 2 additions and 13 deletions.
15 changes: 2 additions & 13 deletions NetKVM/Common/ParaNdis_TX.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -424,22 +424,11 @@ void CParaNdisTX::NBLMappingDone(CNBL *NBLHolder)
if (NBLHolder->MappingSucceeded() && m_VirtQueue.Alive())
{
m_SendQueue.Enqueue(NBLHolder);
// do not do anything if there is dpc processing for this TX
if (m_DpcWaiting) {
return;
}

#if defined(ENABLE_COMPLETE_FROM_SEND)
DoPendingTasks(NBLHolder);
#else
CRawCNBList nbToFree;
CRawCNBLList completedNBLs;
if (m_DpcWaiting == 0)
{
TDPCSpinLocker LockedContext(m_Lock);
SendMapped(false, completedNBLs);
DoPendingTasks(NBLHolder);
}
PostProcessPendingTask(nbToFree, completedNBLs);
#endif
}
else
{
Expand Down

0 comments on commit 315c637

Please sign in to comment.