search

sonic-net / sonic-platform-daemons

Platform module daemons for SONiC
Other
23 stars 152 forks source link

Support 800G ifname in xcvrd #416

Closed AnoopKamath closed 10 months ago

AnoopKamath commented 10 months ago

Description

Add 800G ifname in xcvrd get_interface_speed() which gets the port speed from the host interface name.

Motivation and Context

How Has This Been Tested?

Verified with enabling 800G module app code

202305 commit PR: https://github.com/sonic-net/sonic-platform-daemons/pull/420

Additional Information (Optional)

AnoopKamath commented 10 months ago

@prgeor, @mihirpat1 , Please review this

mihirpat1 commented 10 months ago

@StormLiangMS @yxieca - Can you please help to cherry-pick this to 202305 and 202311? MSFT ADO - 26246820

StormLiangMS commented 9 months ago

hi @mihirpat1 why we need this for 202305? there is no 800G product in 202305.

mihirpat1 commented 9 months ago

hi @mihirpat1 why we need this for 202305? there is no 800G product in 202305.

@AnoopKamath - Can you please help in responding to @StormLiangMS.

AnoopKamath commented 9 months ago

Hi @StormLiangMS, we are already supporting 800G in 2x400G mode and want to enable 800G app code now in 202305. @prgeor can add more details. This HLD https://github.com/sonic-net/SONiC/blob/master/doc/sfp-cmis/CMIS-custom-SI-settings.md was also added which is required to support certain 800G modules.

@prgeor, @mihirpat1, Can you please review and help merge double commit these changes to 202305? https://github.com/sonic-net/sonic-platform-daemons/pull/420

AnoopKamath commented 9 months ago

Hi @StormLiangMS, can you please cherry pick this to 202311?

mssonicbld commented 8 months ago

Cherry-pick PR to 202311: https://github.com/sonic-net/sonic-platform-daemons/pull/429