Skip to content
This repository has been archived by the owner on Apr 14, 2024. It is now read-only.

Commit

Permalink
Fix jumbo error being reported multiple times
Browse files Browse the repository at this point in the history
Also found and fixed another related bug where the loop to check pkt
size for trunc/jumbo ran for frameCount instead of
frameSizeVariableCount() - this change will reduce preflight check time
when frameSizeVariableCount is less than frameCount

Fixes #290
  • Loading branch information
pstavirs committed Dec 29, 2019
1 parent 44a9ead commit 64fe90e
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion common/streambase.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -608,7 +608,7 @@ bool StreamBase::preflightCheck(QStringList &result) const
bool pass = true;
bool chkTrunc = true;
bool chkJumbo = true;
int count = isFrameSizeVariable() ? frameCount() : 1;
int count = isFrameSizeVariable() ? frameSizeVariableCount() : 1;

for (int i = 0; i < count; i++)
{
Expand All @@ -627,6 +627,7 @@ bool StreamBase::preflightCheck(QStringList &result) const
{
result << QObject::tr("Jumbo frames may be truncated or dropped "
"if not supported by the hardware");
chkJumbo = false;
pass = false;
}

Expand Down

0 comments on commit 64fe90e

Please sign in to comment.