Leverage the fanout's ASIC to send PFC storm #12
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Currently, the PFC storm is sent using packet socket on the host CPU, which causes some drawbacks - There is strict timing requirement, which can not be guaranteed - The more ports under PFC storm, the higher probability that the timing requirement is not satisfied We will leverage the fanout's ASIC to send PFC storm - Use SDK API to enable lossless traffic, set xoff to 0, and setup mappings for the PGs and the corresponding IEEE priorities Limitation Still keep sending pfc frame with cpu in the storm docker, but rename it to pfc_gen_cpu.py in the docker. If use the ASIC to send the pfc frame, ASIC could not send specified count of the pfc frame. need to still use the CPU to send the pfc frame.such as the test_pfc_counter test Signed-off-by: Stephen Sun <stephens@nvidia.com>
nhe-NV
force-pushed
the
sneding_pfc_with_asic
branch
from
9fbeb5a
to
899de5a
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Currently, the PFC storm is sent using packet socket on the host CPU, which causes some drawbacks
We will leverage the fanout's ASIC to send PFC storm
After the change, the pfc related test case which is using onxy as fanout will send the pfc frame with ASIC instead of using the CPU.
New docker storm docker need to be build with the Docker file provided in this PR.
Description of PR
Summary:
Fixes # (issue)
Type of change
Back port request
Approach
What is the motivation for this PR?
To make the pfc related test case much more stable.
How did you do it?
Change sending pfc frame with CPU to ASIC on the onxy fanout.
How did you verify/test it?
Run the pfc related test case on the setup which has onxy fanout, test case could pass stablly.
Any platform specific information?
Supported testbed topology if it's a new test case?
Documentation