[v1.7] Tunnel Scoped TC MAP and Remarking #1452
Merged
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
Signed-off-by: bingwang <bingwang@microsoft.com>
bingwang-ms
changed the title
kcudnik
approved these changes
Mar 23, 2022
|
@JaiOCP , please review, thx |
|
@kcudnik Could you help merge this PR? Thanks |
richardyu-ms
added a commit
to richardyu-ms/SAI
that referenced
this pull request
Apr 8, 2022
In order to enable the function for Tunnel Scoped TC MAP in opencomputeproject#1452, we need to enable the tunnel related interfaces, attribute and method tables. Add the method sai headers files. Test done: make -C ./meta/ Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
richardyu-ms
added a commit
to richardyu-ms/SAI
that referenced
this pull request
Apr 10, 2022
opencomputeproject#1452，only leave TC map Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
richardyu-ms
added a commit
to richardyu-ms/SAI
that referenced
this pull request
Apr 11, 2022
opencomputeproject#1452，only leave TC map Signed-off-by: richardyu-ms <richard.yu@microsoft.com> Signed-off-by: root <root@a3188c47ac11>
richardyu-ms
added a commit
to richardyu-ms/SAI
that referenced
this pull request
Apr 11, 2022
opencomputeproject#1452，only leave TC map Signed-off-by: richardyu-ms <richard.yu@microsoft.com> Signed-off-by: root <root@a3188c47ac11> Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
richardyu-ms
added a commit
to richardyu-ms/SAI
that referenced
this pull request
Apr 11, 2022
opencomputeproject#1452，only leave TC map Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
richardyu-ms
added a commit
to richardyu-ms/SAI
that referenced
this pull request
Apr 11, 2022
opencomputeproject#1452，only leave TC map Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
richardyu-ms
added a commit
that referenced
this pull request
Apr 26, 2022
* revert unnecessary changes Signed-off-by: richardyu-ms <richard.yu@microsoft.com> * take back two removed method for bulk operation and add keyworkds for the ipsec Signed-off-by: richardyu-ms <richard.yu@microsoft.com> * remove ipsec related interfaces and attributes Signed-off-by: richardyu-ms <richard.yu@microsoft.com> * Remove unnecessary change related to ECN and VxLan udp souce port in PR #1452，only leave TC map Signed-off-by: richardyu-ms <richard.yu@microsoft.com> * In order to keep the enum value compatiable with latter version, add vxlan source port and ECN Signed-off-by: richardyu-ms <richard.yu@microsoft.com> * Set enum values Signed-off-by: richardyu-ms <richard.yu@microsoft.com> * add test to make sure the value of enum is right Signed-off-by: richardyu-ms <richard.yu@microsoft.com> * refactor the testing code Signed-off-by: richardyu-ms <richard.yu@microsoft.com> * remove a test method and remove a unused enum which might cause enum inconsistence between versions Signed-off-by: richardyu-ms <richard.yu@microsoft.com> * remove the enum values which might cause value holes Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
# for free
to join this conversation on GitHub.
Already have an account?
# 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.
Signed-off-by: bingwang bingwang@microsoft.com
This PR is a cherry-pick of PR #1413, and resolve cherry-pick conflicts as well.
This PR add following behavior to tunnel encap and decap
Tunnel Encap:
SAI_TUNNEL_ATTR_ENCAP_QOS_TC_AND_COLOR_TO_DSCP_MAPfor remarking the packet based on normalized TC as per port TC MAP bindingSAI_TUNNEL_ATTR_ENCAP_QOS_TC_TO_QUEUE_MAPfor queue selection based on normalized TC as per port TC MAP bindingTunnel Decap:
SAI_TUNNEL_ATTR_DECAP_QOS_DSCP_TO_TC_MAPnew TC MAP can be bind'ed to the tunnel for decap. This MAP overrides port MAP if configured.SAI_TUNNEL_ATTR_DECAP_QOS_TC_TO_PRIORITY_GROUP_MAPfor mapping the normalized TC to priority group mainly for PFC generation.DSCP mode takes effect as configured for both cases.