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

Advance swss #5099

Closed
wants to merge 18 commits into from
Closed

Advance swss #5099

wants to merge 18 commits into from

Conversation

sydlogan
Copy link
Contributor

@sydlogan sydlogan commented Aug 4, 2020

Signed-off-by: Syd Logan syd.logan@broadcom.com

… support VS gearbox phy feature

* scripts and configuration needed to support a second syncd docker (physyncd)
* physyncd supports gearbox device and phy SAI APIs and runs multiple instances of syncd, one per phy in the device
* support for VS target (sonic-sairedis vslib has been extended to support a virtual BCM81724 gearbox PHY).

HLD is located at https://github.com/Azure/SONiC/blob/b817a12fd89520d3fd26bbc5897487928e7f6de7/doc/gearbox/gearbox_mgr_design.md

**- Why I did it**

This work is part of the gearbox phy joint effort between Microsoft and Broadcom, and is based
on multi-switch support in sonic-sairedis.

**- How I did it**

Overall feature was implemented across several projects. The collective pull requests (some in late stages of review at this point):

sonic-net/sonic-utilities#931 - CLI (merged)
sonic-net/sonic-swss-common#347 - Minor changes (merged)
sonic-net/sonic-swss#1321 - gearsyncd, config parsers, changes to orchargent to create gearbox phy on supported systems
sonic-net/sonic-sairedis#624 - physyncd, virtual BCM81724 gearbox phy added to vslib

**- How to verify it**

In a vslib build:

root@sonic:/home/admin# show gearbox interfaces status
  PHY Id    Interface        MAC Lanes    MAC Lane Speed        PHY Lanes    PHY Lane Speed    Line Lanes    Line Lane Speed    Oper    Admin
--------  -----------  ---------------  ----------------  ---------------  ----------------  ------------  -----------------  ------  -------
       1   Ethernet48  121,122,123,124               25G  200,201,202,203               25G       204,205                50G    down     down
       1   Ethernet49  125,126,127,128               25G  206,207,208,209               25G       210,211                50G    down     down
       1   Ethernet50      69,70,71,72               25G  212,213,214,215               25G           216               100G    down     down

In addition, docker ps | grep phy should show a physyncd docker running.

  Signed-off-by: syd.logan@broadcom.com
* ea30f2f swss: gearsyncd should return 0 on exit (sonic-net#1376)
* 7a4c15d Remove 00-copp.config.json from swss debian package. (sonic-net#1366)
* 69761b2 fix undefined var in rates lua scripts (sonic-net#1365)
* 26e1723 [fdborch] Fixed Orchagent crash in FDB flush on port disable. (sonic-net#1369)
* a1530e3 [tlm_teamd]: Try to add LAG again, when teamd is not ready first time (sonic-net#1347)
* a9479e6 [vs] Incorporate python3 best practices into DVSLib (sonic-net#1357)
* d5a18a0 [dvs] Mark unstable tests as xfail (sonic-net#1356)
@sydlogan sydlogan closed this Aug 4, 2020
@sydlogan sydlogan deleted the advance-swss branch August 4, 2020 20:14
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