Skip to content

Comments

[DPU] Increase Bulker limit and pop batch size#24168

Merged
kperumalbfn merged 4 commits intosonic-net:masterfrom
vivekrnv:zmq_batch_optimization
Oct 2, 2025
Merged

[DPU] Increase Bulker limit and pop batch size#24168
kperumalbfn merged 4 commits intosonic-net:masterfrom
vivekrnv:zmq_batch_optimization

Conversation

@vivekrnv
Copy link
Contributor

Why I did it

Increase the pop batch size and Max Bulker limit to 65536 to speed up applying the high volume Dash configuration

Depends on
sonic-net/sonic-sairedis#1660
sonic-net/sonic-swss-common#1084
sonic-net/sonic-swss#3910

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

root@sonic:/home/admin# ps -aux | grep orch
root       11118  1.5  0.4 464804 267368 pts/0   Sl   02:50   0:00 /usr/bin/orchagent -d /var/log/swss -b 65536 -z zmq_sync -k 65536 -m B0:CF:0E:20:8E:DE -q tcp://eth0-midplane

2025 Sep 30 18:48:38.911835 sonic NOTICE swss#orchagent: :- main: Setting maximum bulk size in bulk mode as 65536

Apply Scale config and verify

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

  • 202205
  • 202211
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

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)

@vivekrnv vivekrnv requested a review from lguohan as a code owner September 30, 2025 19:03
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

# Set orchagent pop batch size to 128 for faster link notification handling
# during route-churn
ORCHAGENT_ARGS+="-b 128 "
elif [[ x"$LOCALHOST_SWITCHTYPE" == x"dpu" ]]; then
Copy link
Contributor

Choose a reason for hiding this comment

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

does this work for kvm dpus? Could you try it out?

@prabhataravind
Copy link
Contributor

Change lgtm except a minor comment on kvm testbeds. If that is confirmed to be working, we should be ok.

@prabhataravind
Copy link
Contributor

@vivekrnv could you add similar changes to platform/vs/docker-sonic-vs/orchagent.sh?

@prabhataravind
Copy link
Contributor

@prsunny for viz

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vivekrnv
Copy link
Contributor Author

vivekrnv commented Oct 1, 2025

@vivekrnv could you add similar changes to platform/vs/docker-sonic-vs/orchagent.sh?

Added

@prsunny
Copy link
Contributor

prsunny commented Oct 2, 2025

@kperumalbfn , please help with merge

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to msft-202506: Azure/sonic-buildimage-msft#1691

croos12 pushed a commit to croos12/sonic-buildimage that referenced this pull request Oct 6, 2025
FengPan-Frank pushed a commit to FengPan-Frank/sonic-buildimage that referenced this pull request Dec 4, 2025
[DPU] Increase Bulker limit and pop batch size

Signed-off-by: Feng Pan <fenpan@microsoft.com>
spandan-nexthop pushed a commit to spandan-nexthop/sonic-buildimage that referenced this pull request Feb 15, 2026
[DPU] Increase Bulker limit and pop batch size
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants