Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: [Snappi][IxNetwork] Require Prio<->Queues mapping for (test and background) flows with Prio4 and above #12967

Open
amitpawar12 opened this issue May 23, 2024 · 0 comments

Comments

@amitpawar12
Copy link

Issue Description

Some of the TGEN used for the testing along with SNAPPI have default queue size of 4.

With 8 queues, it is easy to map the PRIO flows with the hardware queues (usually one-one mapping).

However, with less than 8 queues, could result in different results. With TGEN, that we used, it was observed that flows with PRIO4 and above, did not have any hardware queues assigned to them. This resulted in improper behavior especially when PFC (for Prio4) was involved during the test.

Current code:
pfc.pfc_class_0 = 0
pfc.pfc_class_1 = 1
pfc.pfc_class_2 = 2
pfc.pfc_class_3 = 3
pfc.pfc_class_4 = 4
pfc.pfc_class_5 = 5
pfc.pfc_class_6 = 6
pfc.pfc_class_7 = 7

There should check in script to queue lengths and allow for dynamic queue vs prio assignments.

@kamalsahu0001

Results you see

We observed that PFCs for 4 were not getting counted on the TGEN and usually, it reacted to the PFCs received for PRIO3.

Results you expected to see

We have a workaround for now, to assign specific queues for Prio4 and above.

Is it platform specific

generic

Relevant log output

No response

Output of show version

No response

Attach files (if any)

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

1 participant