Skip to content

Commit

Permalink
Fix AgentVoqSwitchFullScaleDsfNodesTest.systemPortScaleTest in dual s…
Browse files Browse the repository at this point in the history
…tage

Summary: As titled. The calculation is fixed in new release to only account 3 queues for single VOQ creation.

Reviewed By: daiwei1983

Differential Revision: D68470872

fbshipit-source-id: d6e92819b86ede067f55925684bc79aa56a09275
  • Loading branch information
Ron He authored and facebook-github-bot committed Jan 22, 2025
1 parent d05df03 commit 5e7b26f
Showing 1 changed file with 1 addition and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -110,12 +110,8 @@ TEST_F(AgentVoqSwitchWithMultipleDsfNodesTest, remoteSystemPort) {
<< " after sysPortsFree: " << afterSysPortsFree
<< " voqsFree: " << afterVoqsFree;
EXPECT_EVENTUALLY_EQ(beforeSysPortsFree - 1, afterSysPortsFree);
// 8 VOQs allocated per sys port for single stage
// 4 VOQs allocated per sys port for dual stage - 3 VOQs for the system
// port itself (since it's <16k port), and 1 VOQ for the 16k+ port. Please
// see 3Q2Q mode VOQ allocation for more details.
EXPECT_EVENTUALLY_EQ(
isDualStage3Q2QMode() ? beforeVoqsFree - 4 : beforeVoqsFree - 8,
isDualStage3Q2QMode() ? beforeVoqsFree - 3 : beforeVoqsFree - 8,
afterVoqsFree);
});
};
Expand Down

0 comments on commit 5e7b26f

Please sign in to comment.