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

[cherry-pick] Tunnel Scoped TC MAP and Remarking (#1413) #1451

Merged
merged 1 commit into from
Mar 23, 2022

Conversation

bingwang-ms
Copy link
Contributor

This PR add following behavior to tunnel encap and decap

Tunnel Encap:

SAI_TUNNEL_ATTR_ENCAP_QOS_TC_AND_COLOR_TO_DSCP_MAP for remarking the packet based on normalized TC as per port TC MAP binding
SAI_TUNNEL_ATTR_ENCAP_QOS_TC_TO_QUEUE_MAP for queue selection based on normalized TC as per port TC MAP binding

Tunnel Decap:

SAI_TUNNEL_ATTR_DECAP_QOS_DSCP_TO_TC_MAP new 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_MAP for mapping the normalized TC to priority group mainly for PFC generation.
DSCP mode takes effect as configured for both cases.

Signed-off-by: Jai Kumar jai.kumar@broadcom.com

This PR add following behavior to tunnel encap and decap

Tunnel Encap:

 SAI_TUNNEL_ATTR_ENCAP_QOS_TC_AND_COLOR_TO_DSCP_MAP for remarking the packet based on normalized TC as per port TC MAP binding
 SAI_TUNNEL_ATTR_ENCAP_QOS_TC_TO_QUEUE_MAP for queue selection based on normalized TC as per port TC MAP binding

Tunnel Decap:

 SAI_TUNNEL_ATTR_DECAP_QOS_DSCP_TO_TC_MAP new 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_MAP for mapping the normalized TC to priority group mainly for PFC generation.
DSCP mode takes effect as configured for both cases.

Signed-off-by: Jai Kumar <jai.kumar@broadcom.com>
@lguohan
Copy link
Collaborator

lguohan commented Mar 23, 2022

i think 202012 is using 1.7

@lguohan
Copy link
Collaborator

lguohan commented Mar 23, 2022

can you generate pr for v1.7 as well?

@bingwang-ms
Copy link
Contributor Author

can you generate pr for v1.7 as well?

Sure, will do that.

@richardyu-ms richardyu-ms merged commit d0f3333 into opencomputeproject:v1.9 Mar 23, 2022
@kcudnik
Copy link
Collaborator

kcudnik commented Mar 29, 2022

if approved then would also need to be cherrypicked: #1462

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

Successfully merging this pull request may close these issues.

6 participants