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

[Broadcom SAI] upgrade to version 3.1.3.5-6 #1916

Closed
wants to merge 1 commit into from

Conversation

yxieca
Copy link
Contributor

@yxieca yxieca commented Aug 11, 2018

Signed-off-by: Ying Xie ying.xie@microsoft.com

- What I did
Upgrade Broadcom SAI to 3.1.3.5-6

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
@yxieca yxieca requested a review from lguohan August 11, 2018 04:27
@lguohan
Copy link
Collaborator

lguohan commented Aug 11, 2018

we have header changes, need to update the SAI header in sairedis

@zhenggen-xu
Copy link
Collaborator

It would be good to list the feature added and/or bugs fixed when libsai was upgraded.
In this particular upgrade, what was the change?

@wendani
Copy link
Contributor

wendani commented Aug 11, 2018

This upgrade adds the handler to support sai attribute SAI_PORT_ATTR_PKT_TX_ENABLE

Copy link
Collaborator

@lguohan lguohan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we have header changes, need to update the SAI header in sairedis

@yxieca
Copy link
Contributor Author

yxieca commented Aug 13, 2018

Closing this PR. It is likely that we need to upgrade to the next version to pick up the SAI header changes.

@yxieca yxieca closed this Aug 13, 2018
lguohan pushed a commit that referenced this pull request Sep 21, 2021
Commits on Sep 18, 2021
[pfcwd] Fix the polling interval time granularity (#1912) 98a3ae8

Commits on Sep 20, 2021
[202012][teammgrd]: Improve LAGs cleanup on shutdown (#1916) 5a4678e 

Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
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.

4 participants