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

[WIP] enable xcvrd with xcvr emulation in the VS environment #21242

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

ishidawataru
Copy link
Collaborator

Why I did it

Implementing sonic-net/SONiC#1849

sonic-net/sonic-sairedis#1458 needs to be merged and included in this PR before merging.

SAI_PORT_ATTR_HOST_TX_READY_STATUS support is required to make xcvrd work.

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

admin@sonic:~$ show platform summary
Platform: x86_64-kvm_x86_64-r0
HwSKU: Force10-S6000
ASIC: vs
ASIC Count: 1
Serial Number: N/A
Model Number: N/A
Hardware Revision: N/A
admin@sonic:~$ show interface transceiver eeprom Ethernet0
Ethernet0: SFP EEPROM detected
        Active Firmware: N/A
        Active application selected code assigned to host lane 1: N/A
        Active application selected code assigned to host lane 2: N/A
        Active application selected code assigned to host lane 3: N/A
        Active application selected code assigned to host lane 4: N/A
        Active application selected code assigned to host lane 5: N/A
        Active application selected code assigned to host lane 6: N/A
        Active application selected code assigned to host lane 7: N/A
        Active application selected code assigned to host lane 8: N/A
        Application Advertisement: 400GAUI-4-S C2M (Annex 120G) - Host Assign (0x1) - 400GBASE-DR4 (Cl 124) - Media Assign (0x1)
                                   200GBASE-CR2 (Clause 162) - Host Assign (0x5) - 200GBASE-DR4 (Cl 121) - Media Assign (0x5)
        CMIS Rev: 5.2
        Connector: MPO 1x16
        Encoding: N/A
        Extended Identifier: Power Class 8 (10.0W Max)
        Extended RateSelect Compliance: N/A
        Host Lane Count: N/A
        Identifier: QSFP-DD Double Density 8X Pluggable Transceiver
        Inactive Firmware: N/A
        Length Cable Assembly(m): 100.0
        Media Interface Technology: 850 nm VCSEL
        Media Lane Count: N/A
        Module Hardware Rev: 0.0
        Nominal Bit Rate(100Mbs): 0
        Specification compliance: sm_media_interface
        Supported Max Laser Frequency: N/A
        Supported Max TX Power: N/A
        Supported Min Laser Frequency: N/A
        Supported Min TX Power: N/A
        Vendor Date Code(YYYY-MM-DD Lot): 2024-12-14 
        Vendor Name: xcvr-emu
        Vendor OUI: 01-02-03
        Vendor PN: 
        Vendor Rev: 01
        Vendor SN: 0123456789
admin@sonic:~$ xcvr-emush
> transceiver 0
transceiver(0)> read VendorName
Read: bank=0, page=0, offset=129, length=16, res.data=b'xcvr-emu\x00\x00\x00\x00\x00\x00\x00\x00'
00h:129-144    | VendorName | xcvr-emu
transceiver(0)>

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

the metadata file doesn't exist on pizza box vs platforms.

Signed-off-by: Wataru Ishida <[email protected]>
the package is installed when pmon container starts.
see dockers/docker-platform-monitor/docker_init.j2 for details.

Signed-off-by: Wataru Ishida <[email protected]>
```
make target/python-wheels/bookworm/xcvr_emu-1.0.1-py3-none-any.whl
```

Signed-off-by: Wataru Ishida <[email protected]>
Signed-off-by: Wataru Ishida <[email protected]>
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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