8000 Pr show int switchport config and status performance improvement. by vikumarks · Pull Request #3887 · sonic-net/sonic-utilities · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Pr show int switchport config and status performance improvement. #3887

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Pr show int switchport config and status performance improvement. #3887

wants to merge 4 commits into from
+86 −91

Conversation

vikumarks
Copy link

What I did

Optimize the code, getting 15 times performance improvement

How I did it

How to verify it

time show int switchport config 

admin@str-7060x6-64pe-stress-01:~$ time show int switchport config
Interface    Mode    Untagged    Tagged
-----------  ------  ----------  --------
Ethernet0    routed  2
Ethernet1    routed  2
Ethernet16   routed
Ethernet17   routed
...
....

Ethernet112  routed  2
....
.....
Ethernet499  routed
Ethernet500  routed              2
Ethernet501  routed
Ethernet502  routed
Ethernet503  routed
Ethernet512  routed
Ethernet513  routed

real    0m12.485s
user    0m5.013s
sys     0m2.697s
admin@str-7060x6-64pe-stress-01:~$

After Code change

admin@sonic-s6100-dut1:~$ time show int switchport config
Interface    Mode    Untagged    Tagged
-----------  ------  ----------  --------
Ethernet0    routed  2
Ethernet1    routed  2
Ethernet1
8000
6   routed
Ethernet17   routed
...
....

Ethernet112  routed  2
....
.....
Ethernet499  routed
Ethernet500  routed              2
Ethernet501  routed
Ethernet502  routed
Ethernet503  routed
Ethernet512  routed
Ethernet513  routed

real    0m0.824s
user    0m0.512s
sys     0m0.195s
admin@sonic-s6100-dut1:~$

vikumarks and others added 2 commits May 19, 2025 17:53
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines Azure Pipelines
Copy link
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines Azure Pipelines
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
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vikumarks 3271 @mssonicbld
0