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 binary package version mismatch #2328

Closed
pawmas opened this issue Dec 1, 2018 · 3 comments
Closed

Broadcom SAI binary package version mismatch #2328

pawmas opened this issue Dec 1, 2018 · 3 comments

Comments

@pawmas
Copy link

pawmas commented Dec 1, 2018

Hi SONiC team,

I'm just a little confused. I didn't find any information that SONiC team is contributing its own SAI implementation for Broadcom ASIC chips and as far as I know SONiC is using Broadcom SAI binary package. Am I correct?

So in the sai.mk at master branch I can see that to build SONiC image there is used Broadcom SAI binary package in version 3.1.3.5. On the second side we have available binary package of SAI implementation from official Broadcom's Github repo but we can find there only older version - 3.1.3.4.

Why this difference between 3.1.3.4 and 3.1.3.5? Does it mean that SONiC/Microsoft team contribute their own SAI implementation for Broadcom chips? Or Broadcom company will release 3.1.3.5 binary package soon (just SONiC team has access to it earlier)?

Thanks in advance for explain my concerns.

@yxieca
Copy link
Contributor

yxieca commented Dec 2, 2018

@pawmas

You are correct, SONiC team doesn't come up with another implementation of browdcom SAI.

Please contact Broadcom for individual release of SAI.

Thanks and regards,
Ying

@stcheng
Copy link
Contributor

stcheng commented Dec 3, 2018

Yes. We mainly update the subversion due to some early bug fixes.

@pawmas
Copy link
Author

pawmas commented Dec 3, 2018

@stcheng ,
But fixed bugs was still development under Broadcom company as @yxieca mentioned?

Thanks

@pawmas pawmas closed this as completed Dec 4, 2018
vivekrnv added a commit to vivekrnv/sonic-buildimage that referenced this issue Jun 27, 2022
639d10e (HEAD, origin/202012) [PFC_WD] [202012] Avoid applying ZeroBuffer Profiles to ingress PG when a PFC storm is detected (sonic-net#2310)
475ae19 [202012] [cherry-pick] Apply `DSCP_TO_TC_MAP` from `PORT_QOS_MAP|global` to switch level (sonic-net#2328)
aa6f855 [ci] Change artifact reference pipeline to common lib pipeline. (sonic-net#2294)
752f8c5 [ci] Use correct branch when downloading artifact. (sonic-net#2292)
b3fcc5d [ci] Improve azp trigger settings to automaticlly support new release branch. (sonic-net#2289)

Signed-off-by: Vivek Reddy <vkarri@nvidia.com>
qiluo-msft pushed a commit that referenced this issue Jul 7, 2022
Update sonic-swss submodule

```
639d10e [PFC_WD] [202012] Avoid applying ZeroBuffer Profiles to ingress PG when a PFC storm is detected (#2310)
475ae19 [202012] [cherry-pick] Apply `DSCP_TO_TC_MAP` from `PORT_QOS_MAP|global` to switch level (#2328)
aa6f855 [ci] Change artifact reference pipeline to common lib pipeline. (#2294)
752f8c5 [ci] Use correct branch when downloading artifact. (#2292)
b3fcc5d [ci] Improve azp trigger settings to automaticlly support new release branch. (#2289)
```
theasianpianist added a commit to theasianpianist/sonic-buildimage that referenced this issue Sep 2, 2022
Include following commits:
839625d Replace cmp in acl_loader with operator.eq (sonic-net#2328)
af245ad [route_check]: Ignore standalone tunnel routes (sonic-net#2332)
c9aa65c Revert "[DHCPv6] [202012] Update the dhcpv6_relay config/show cli (sonic-net#2271)" (sonic-net#2336)
6b9cdc9 Made changes to only allow a valid IPv4 or Ipv6 string for dhcp cli (sonic-net#2296)

Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
theasianpianist added a commit to theasianpianist/sonic-buildimage that referenced this issue Sep 2, 2022
Include following commits:

414e239 update unit tests for swap allocator
a91a492 consider swap checking memory in installer
f0ce586 [route_check]: Ignore standalone tunnel routes (sonic-net#2325)
3af8ba4 Replace cmp in acl_loader with operator.eq (sonic-net#2328)
899ba12 Subinterface vrf bind issue fix (sonic-net#2211)
e45b47a [VRF]Adding CLI checks to ensure Vrf is valid in interface bind and static route commands (sonic-net#2333)
f82835e [doc]: Add MACsec CLI doc (sonic-net#2334)
666bdc0 [sonic-package-manager] Drop 'expires_in' (sonic-net#2002)
52ac8ac Handle non-front-panel ports in is_rj45_port (sonic-net#2327)
42ed6d5 [service_mgmt]: Fix fetch MULTI_INST_DEPENDENT bug in service_mgmt.sh.j2 (sonic-net#2319)
d1a2d72 correct an error by changing "show bgp summary" to "show bfd summary" (sonic-net#2324)
7d409a0 Update VRF unbind command (sonic-net#2331)
e14f679 Fix issue: port_type is referenced before initialized (sonic-net#2323)
7704f63 Fix issue: exception in is_rj45_port in multi ASIC env (sonic-net#2313)
6fc4f15 Delete .DS_Store (sonic-net#2244)
ece4049 Fix bug with checking VRF's routes in route_check.py  (sonic-net#2301)
20c6d18 [decode-syseeprom] Fix setting use_db based on support_eeprom_db (sonic-net#2270)
9282e6c Fix vrf UT failed issue (sonic-net#2309)
37eb2b3 add lacp_rate to portchannel (sonic-net#2036)

Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
liat-grozovik pushed a commit that referenced this issue Sep 4, 2022
Update sonic-utilities submodule pointer to include the following:

Replace cmp in acl_loader with operator.eq (#2328)
Subinterface vrf bind issue fix (#2211)
[VRF]Adding CLI checks to ensure Vrf is valid in interface bind and static route commands (#2333)
[doc]: Add MACsec CLI doc (#2334)
[sonic-package-manager] Drop 'expires_in' (#2002)
Handle non-front-panel ports in is_rj45_port (#2327)
[service_mgmt]: Fix fetch MULTI_INST_DEPENDENT bug in service_mgmt.sh.j2 (#2319)
correct an error by changing "show bgp summary" to "show bfd summary" (#2324)
Update VRF unbind command (#2331)
Fix issue: port_type is referenced before initialized (#2323)
Fix issue: exception in is_rj45_port in multi ASIC env (#2313)
Delete .DS_Store (#2244)
Fix bug with checking VRF's routes in route_check.py (#2301)
[decode-syseeprom] Fix setting use_db based on support_eeprom_db (#2270)
Fix vrf UT failed issue (#2309)
add lacp_rate to portchannel (#2036)
qiluo-msft pushed a commit that referenced this issue Sep 6, 2022
Include following commits:
```
839625d Replace cmp in acl_loader with operator.eq (#2328)
af245ad [route_check]: Ignore standalone tunnel routes (#2332)
c9aa65c Revert "[DHCPv6] [202012] Update the dhcpv6_relay config/show cli (#2271)" (#2336)
6b9cdc9 Made changes to only allow a valid IPv4 or Ipv6 string for dhcp cli (#2296)
```
yxieca added a commit to yxieca/sonic-buildimage that referenced this issue Sep 9, 2022
…x-kernel] advance submodule head

linkmgrd:
* ab5b2c1 2022-09-02 | Fix mux config (sonic-net#128) (HEAD -> 202205, github/202205) [Longxiang Lyu]

utilities:
* 7de9305 2022-09-07 | [generate dump]Added error message when saisdkdump fails (sonic-net#2356) (HEAD -> 202205, github/202205) [Sudharsan Dhamal Gopalarathnam]
* c5b0a6d 2022-09-07 | [counterpoll]Fixing counterpoll show for tunnel and acl stats (sonic-net#2355) [Sudharsan Dhamal Gopalarathnam]
* 1452b44 2022-09-05 | [GCU] Fix missing backend in dry run (sonic-net#2347) [jingwenxie]
* bc7b845 2022-09-04 | Add Password Hardening CLI support (sonic-net#2338) [davidpil2002]
* 55e8948 2022-09-06 | [fast-reboot]Avoid stopping masked services during fast-reboot (sonic-net#2335) [Sudharsan Dhamal Gopalarathnam]
* f7d69d4 2022-08-30 | Replace cmp in acl_loader with operator.eq (sonic-net#2328) [Zhaohui Sun]
* 4054ebb 2022-09-05 | Add verification for override (sonic-net#2305) [jingwenxie]
* 729d811 2022-05-30 | Fix sonic-installer and 'show version' command crash when database docker not running issue. (sonic-net#2183) [Hua Liu]

platform-daemons:
* 36ba7c0 2022-09-07 | [ycable] cleanup logic for creating grpc future ready (sonic-net#289) (HEAD -> 202205) [vdahiya12]
* 2a9db73 2022-09-01 | [ycabled] fix insert events from xcvrd;cleanup some mux toggle logic (sonic-net#287) [vdahiya12]

platform-common:
* d7c990d 2022-09-03 | [CMIS] 'get_transceiver_info' should return 'None' when CMIS cable EEPROM is not ready  (sonic-net#305) (HEAD -> 202205) [Kebo Liu]

linux-kernel:
* 25ea052 2022-08-31 | [patch]: Add accpt_untracked_na kernel param (sonic-net#292) (HEAD -> 202205) [Lawrence Lee]

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
yxieca added a commit that referenced this issue Sep 10, 2022
…x-kernel] advance submodule head (#12025)

linkmgrd:
* ab5b2c1 2022-09-02 | Fix mux config (#128) (HEAD -> 202205, github/202205) [Longxiang Lyu]

utilities:
* 7de9305 2022-09-07 | [generate dump]Added error message when saisdkdump fails (#2356) (HEAD -> 202205, github/202205) [Sudharsan Dhamal Gopalarathnam]
* c5b0a6d 2022-09-07 | [counterpoll]Fixing counterpoll show for tunnel and acl stats (#2355) [Sudharsan Dhamal Gopalarathnam]
* 1452b44 2022-09-05 | [GCU] Fix missing backend in dry run (#2347) [jingwenxie]
* bc7b845 2022-09-04 | Add Password Hardening CLI support (#2338) [davidpil2002]
* 55e8948 2022-09-06 | [fast-reboot]Avoid stopping masked services during fast-reboot (#2335) [Sudharsan Dhamal Gopalarathnam]
* f7d69d4 2022-08-30 | Replace cmp in acl_loader with operator.eq (#2328) [Zhaohui Sun]
* 4054ebb 2022-09-05 | Add verification for override (#2305) [jingwenxie]
* 729d811 2022-05-30 | Fix sonic-installer and 'show version' command crash when database docker not running issue. (#2183) [Hua Liu]

platform-daemons:
* 36ba7c0 2022-09-07 | [ycable] cleanup logic for creating grpc future ready (#289) (HEAD -> 202205) [vdahiya12]
* 2a9db73 2022-09-01 | [ycabled] fix insert events from xcvrd;cleanup some mux toggle logic (#287) [vdahiya12]

platform-common:
* d7c990d 2022-09-03 | [CMIS] 'get_transceiver_info' should return 'None' when CMIS cable EEPROM is not ready  (#305) (HEAD -> 202205) [Kebo Liu]

linux-kernel:
* 25ea052 2022-08-31 | [patch]: Add accpt_untracked_na kernel param (#292) (HEAD -> 202205) [Lawrence Lee]

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

Signed-off-by: Ying Xie <ying.xie@microsoft.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

No branches or pull requests

3 participants