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

[Ingrasys] Add platform support for S9280-64X with Barefoot ASIC #1776

Closed
wants to merge 1 commit into from

Conversation

cytsai0409
Copy link
Contributor

- What I did
Add Ingrasys S9280-64x (64*100G) platform support with Barefoot ASIC

- How I did it
Add platform drivers and configuration files

- How to verify it
Do sanity checks on the device

- Description for the changelog
[Ingrasys] Add platform support for S9280-64X with Barefoot ASIC

@cytsai0409
Copy link
Contributor Author

cytsai0409 commented Jun 8, 2018

resubmit this pull request since build errors on 201712 branch will be fixed by #1775

@cytsai0409
Copy link
Contributor Author

retest this please

@arkadiyshapiro
Copy link

lets make sure this PR goes to sonic master. All Barefoot ASIC support is there in master now.

@cytsai0409
Copy link
Contributor Author

Code already merged in master branch in the PR(#1880), so close this PR.

@cytsai0409 cytsai0409 closed this Aug 6, 2018
stephenxs added a commit to stephenxs/sonic-buildimage that referenced this pull request Jun 29, 2021
3226163 [BufferOrch] Don't call SAI API for BUFFER_POOL/PROFILE handling in case the op is DEL and the SAI OID is NULL (sonic-net#1786)
6c88e47 [Dynamic Buffer Calc][Mellanox] Bug fixes and enhancements for the lua plugins for buffer pool calculation and headroom checking (sonic-net#1781)
e86b900 [MPLS] sonic-swss changes for MPLS (sonic-net#1686)
4c8e2b5 [Dynamic Buffer Calc] Avoid creating lossy PG for admin down ports during initialization (sonic-net#1776)
3602124 [VS test stability] Skip flaky test for DPB (sonic-net#1807)
c37cc1c Support for in-band-mgmt via management VRF (sonic-net#1726)
1e3a532 Fix config prompt question issue (sonic-net#1799)

Signed-off-by: Stephen Sun <stephens@nvidia.com>
stephenxs added a commit to stephenxs/sonic-buildimage that referenced this pull request Jun 29, 2021
bb383be2 [Dynamic Buffer Calc][Mellanox] Bug fixes and enhancements for the lua plugins for buffer pool calculation and headroom checking (sonic-net#1781)
f949dfe9 [Dynamic Buffer Calc] Avoid creating lossy PG for admin down ports during initialization (sonic-net#1776)
def0a914 Fix config prompt question issue (sonic-net#1799)
21f97506 [ci]: Merge azure pipelines from master to 202012 branch (sonic-net#1764)
a83a2a42 [vstest]: add dvs_route fixture
849bdf9c [Mux] Add support for mux metrics to State DB (sonic-net#1757)
386de717 [qosorch] Dot1p map list initialization fix (sonic-net#1746)
f99abdca [sub intf] Port object reference count update (sonic-net#1712)
4a00042d [vstest/nhg]: use dvs_route fixture to make test_nhg more robust

Signed-off-by: Stephen Sun <stephens@nvidia.com>
theasianpianist pushed a commit to theasianpianist/sonic-buildimage that referenced this pull request Feb 5, 2022
…ring initialization (sonic-net#1776)

- What I did
Avoid creating lossy PGs for admin down ports during system initialization
Make ADMIN_DOWN the default state of a port in dynamic buffer manager.
Explicitly remove configured PGs from an admin down port during initialization
This is to make sure the items will be added to m_ready_list.
Use APPL_DB as the source of initial PGs and queues for port ready deducting for warm reboot
Signed-off-by: Stephen Sun stephens@nvidia.com

- Why I did it
Lossy PGs for admin down ports should not be applied to APPL_DB.
Originally, we had the logic to remove lossy/lossless PGs when shutting a port. The same logic should be applied when the system is starting. Allocating lossy PG for an admin down port doesn't affect the system negatively except for wasting buffer. So we would like to resolve it.

- How I verified it
Manually test
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.

2 participants