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

Update loopback-mode type #1034

Merged
merged 65 commits into from
Feb 16, 2024
Merged

Update loopback-mode type #1034

merged 65 commits into from
Feb 16, 2024

Conversation

sallylsy
Copy link
Contributor

@sallylsy sallylsy commented Jan 17, 2024

Change Scope

  • In openconfig-transport-types.yang, updates the current enums of loopback-mode-type.
  • Two of the existing enums are updated to new values. If users were using FACILITY or TERMINAL, they should update to ASIC_MAC_LOCAL/PHY_MAC_LOCAL or ASIC_MAC_REMOTE/PHY_MAC_REMOTE

Ref:

Reference to SAI implementation

SAI header

Arista EOS - traffic loopback supports local and remote options. It also allows specifying MAC and PHY layer options.

JunOS - set loopback supports local loopback only. JunOS does not expose a MAC or PHY level option from their configuration

Cisco IOS XR - loopback supports local (aka terminal or internal) and remote (aka facility or line) loopbacks. Cisco does not expose a MAC or PHY level option from their configuration.

sallylsy and others added 30 commits September 15, 2023 01:15
@OpenConfigBot
Copy link

OpenConfigBot commented Jan 17, 2024

No major YANG version changes in commit 2eb5cd6

@sallylsy sallylsy marked this pull request as ready for review January 17, 2024 01:17
@sallylsy sallylsy requested a review from a team as a code owner January 17, 2024 01:17
@dplore
Copy link
Member

dplore commented Jan 22, 2024

We will review in OC operators meeting on Jan 23, 2024

@dplore
Copy link
Member

dplore commented Jan 23, 2024

I am not a deep expert on transceiver loopbacks, but I thought some transceiver modules implement a PHY level loopback in the transceiver module (as opposed to the ASIC). Is this true?

@jnelson-sj do you have experience here? Does it make sense to have some loopback enum for a transceiver PHY level loopback?

@jnelson-sj
Copy link
Contributor

jnelson-sj commented Jan 23, 2024 via email

@dplore
Copy link
Member

dplore commented Jan 30, 2024

Hi John, thank you for the insight! My bias is to handle the 400ZR case separately as I think the ASIC MAC/PHY layer loopbacks proposed here are still applicable and do not appear to conflict with the 400ZR case.

+Aditya Bhiday @.***> Hi Darren, Yes, there are transceivers which perform loopbacks at different levels of the processing chain. For example, ZR transceivers have the following loopbacks defined in OIF-400GZR: [image: image.png] You can (hopefully) see in the diagram that there are 6 loopbacks defined here. The can be grouped into whether they provide a FACILITY or a TERMINAL loopback: FACILITY Loopbacks: - Media Side Tx - Tx Modem - Host Side Tx TERMINAL Loopbacks: - Media Side Rx - Rx Modem - Host Side Rx I think the PHY level loopback you are talking about may be the Host Side Tx - what the switch port sends on its tx is looped back to its rx with minimal processing in the transceiver. I wonder if there should be another dimension to the loopbacks - a depth. The FACILITY/TERMINAL gives the direction of the loopback, but it would also be good to have a DEEP/SHALLOW qualifier. This would be used to specify how much of the transceiver function is to be exercised in the loopback. Is this along the lines of what you are thinking? John

@dplore
Copy link
Member

dplore commented Jan 30, 2024

This was reviewed in the Jan 30, 2024 OC Operators meeting. The feedback was we should address how a system which does not expose MAC / PHY layer loopbacks, but only an 'interface' loopback should be handled. The suggestion was to
retain the existing FACILITY and TERMINAL enum values rather than deprecating them.

@dplore dplore self-assigned this Jan 30, 2024
Copy link
Member

@dplore dplore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Last call for comments please. This will merge on Feb 13, 2024

@dplore
Copy link
Member

dplore commented Feb 16, 2024

/gcbrun

@dplore
Copy link
Member

dplore commented Feb 16, 2024

/gcbrun

@dplore dplore merged commit aff3e9e into openconfig:master Feb 16, 2024
14 checks passed
romeyod pushed a commit to romeyod/aftNextHop that referenced this pull request Sep 19, 2024
* Add interface loopback-mode types for ASIC PHY and MAC

---------

Co-authored-by: Darren Loher <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants