Skip to content

restart USB to Ethernet management interface if not RUNNING #3999

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

eladnachman
Copy link

PROBLEM:
Sometimes after "config reload" command the management interface (eth0) connected over ETH-USB device is
UP/RUNNING on a remote peer side, but
stays UP/not-running on the local side.
Well-known negotiation problem of ETH-over-USB devices (ASIX).

SOLUTION:
If the management interface is a USB device and is not RUNNING (/sys/class/net/${if}/operstate is not "up") at the end of the "config reload" command
bring it down and up with delay to trigger renegotiation.

What I did

restart USB to Ethernet management interface if not at RUNNING state

How I did it

If the management interface is a USB device and is not at RUNNING state at the end of the config reload command, bring it down and up with delay

How to verify it

Manual test

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

PROBLEM:
Sometimes after "config reload" command the management
interface (eth0) connected over ETH-USB device is
  UP/RUNNING on a remote peer side, but
  stays UP/not-running on the local side.
Well-known negotiation problem of ETH-over-USB devices (ASIX).

SOLUTION:
If the management interface is a USB device and is not RUNNING
(/sys/class/net/${if}/operstate is not "up") at the end of the
"config reload" command
bring it down and up with delay to trigger renegotiation.

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

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@eladnachman
Copy link
Author

Also required merge to 202505

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.

3 participants